Re: [PATCH] - ia64_ctx.lock deadlock

From: Matthew Wilcox <willy_at_debian.org>
Date: 2003-12-30 09:31:33
On Sat, Dec 27, 2003 at 08:27:22AM -0600, Jack Steiner wrote:
> I hit a deadlock involving the ia64_ctx.lock. The lock
> may be taken in interrupt context to process an IPI from smp_flush_tlb_mm.
> 
> Here is a patch:
> -	spin_lock(&ia64_ctx.lock);
> +	spin_lock_irqsave(&ia64_ctx.lock, flags);

Would it be better to use spin_lock_irq() / spin_unlock_irq() here?
It seems to me this function is only called from task context.

-- 
"Next the statesmen will invent cheap lies, putting the blame upon 
the nation that is attacked, and every man will be glad of those
conscience-soothing falsities, and will diligently study them, and refuse
to examine any refutations of them; and thus he will by and by convince 
himself that the war is just, and will thank God for the better sleep 
he enjoys after this process of grotesque self-deception." -- Mark Twain
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Mon Dec 29 17:31:56 2003

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:21 EST