On Wed, 2010-12-08 at 15:03 +0530, Sahitya Tummala wrote:
> In the context of request processing thread, data mover lock is
> acquired after the host lock.  In another context, in the completion
> handler of data mover the locks are acquired in the reverse order,
> resulting in possible circular lock dependency warning. Hence,
> schedule a tasklet to process the dma completion so as to avoid
> nested locks.
> 

Can you just alter the location where the locks are taken ?

This lock,

spin_lock_irqsave(&host->lock, flags);

seems not fine grained at all .. That lock is just being held over very
large sections of code which it doesn't seems like it needs to be. It
doesn't appear to be protecting just the data structures.

Daniel


-- 

Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to