Messages in this thread | | | Date | Fri, 9 Nov 2007 13:57:36 -0500 | From | Chris Mason <> | Subject | Re: dio_get_page() lockdep complaints |
| |
On Fri, 9 Nov 2007 13:53:27 -0500 Chris Mason <chris.mason@oracle.com> wrote:
> On Fri, 09 Nov 2007 10:35:04 -0800 > Zach Brown <zach.brown@oracle.com> 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@oracle.com> >
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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |