Re: lockdep warning: events vs. bd_mutex

From: Tejun Heo
Date: Tue Sep 14 2010 - 03:50:05 EST


Hello,

On 09/14/2010 09:27 AM, KOSAKI Motohiro wrote:
> lru_add_drain_all() wait system_wq.
>
> Probably we have four choice.
> 1) simple revert buggy patch (I don't know how much serious original issue)
> 2) ata_scsi_dev_rescan() doesn't use system_wq
> 3) lru_add_drain_all() doesn't use system_wq
> 4) make IPI version lru_add_drain_all()
>
> I'd like to hear Tejun's opinion.

Yeah, it was reported before. The warning is bogus at this point.
The right thing to do would be improving lockdep annotation in
workqueue code but for now the easiest way out would be simply using
one of other system workqueues. I'll look into it.

Thanks.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/