Re: dio_get_page() lockdep complaints

From: Chris Mason
Date: Fri Nov 09 2007 - 13:58:58 EST


On Fri, 9 Nov 2007 13:53:27 -0500
Chris Mason <chris.mason@xxxxxxxxxx> wrote:

> On Fri, 09 Nov 2007 10:35:04 -0800
> Zach Brown <zach.brown@xxxxxxxxxx> wrote:
>
> > > Without getting into a huge patch, the best fix would just be
> > > switching to try lock. If the tail doesn't get packed, the world
> > > doesn't end.
> >
> > So, something like this?
>
> Reviewed-by: Chris Mason <chris.mason@xxxxxxxxxx>
>

Ugh, I thought the preallocation was getting freed elsewhere, but it
looks like I was wrong. We can't just skip the i_mutex after all,
sorry.

-chris
-
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/