Date: Fri, 9 Nov 2007 13:57:36 -0500
From: Chris Mason <chris.mason@oracle.com>
To: Chris Mason <chris.mason@oracle.com>
Cc: Zach Brown <zach.brown@oracle.com>,
       Trond Myklebust <trond.myklebust@fys.uio.no>,
       Peter Zijlstra <peterz@infradead.org>,
       Jens Axboe <jens.axboe@oracle.com>, linux-kernel@vger.kernel.org,
       akpm@linux-foundation.org, linux-aio@kvack.org
Subject: Re: dio_get_page() lockdep complaints
Message-ID: <20071109135736.04a4346a@think.oraclecorp.com>
In-Reply-To: <20071109135327.01bca1d9@think.oraclecorp.com>
References: <20070419073828.GB20928@kernel.dk>
	<1194627742.6289.175.camel@twins>
	<4734992C.7000408@oracle.com>
	<1194630300.7459.65.camel@heimdal.trondhjem.org>
	<47349D66.2040109@oracle.com>
	<20071109130156.4b05a652@think.oraclecorp.com>
	<4734A858.2010307@oracle.com>
	<20071109135327.01bca1d9@think.oraclecorp.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 810
Lines: 25

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/