On Thu, Apr 11, 2013 at 12:04:08PM -0700, Tejun Heo wrote: > Hey, > > On Thu, Apr 11, 2013 at 09:58:54PM +0300, Michael S. Tsirkin wrote: > > > Hmm... how about adding a work_on_cpu_nested() which takes @subclass > > > argument? Wouldn't that be much cleaner? > > > > > > Thanks. > > > > Is that 3.9 material though? > > Why wouldn't it be? It's actually safer as it doesn't change any > logic. It's just updating lockdep annotation, which is what's needed > here anyway. > > Thanks. Hmm no, there's a real deadlock here: you are trying to flush from work1 from within work2 running on same workqueue. work2 can't event start running. The problem is not annotation. > -- > tejun -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
