[Top] [All Lists]

Re: [PATCH 1/2] xfs: a couple getbmap cleanups

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH 1/2] xfs: a couple getbmap cleanups
From: Felix Blyakher <felixb@xxxxxxx>
Date: Wed, 29 Apr 2009 10:02:11 -0500
Cc: xfs@xxxxxxxxxxx
In-reply-to: <20090429145048.GA15558@xxxxxxxxxxxxx>
References: <20090224133858.GB15820@xxxxxxxxxxxxx> <9EF64CF3-FDFC-4777-94A5-7295E827ABA4@xxxxxxx> <20090409171135.GA25303@xxxxxxxxxxxxx> <6F8D4AC9-D76A-43FC-AA5C-78C86A12E244@xxxxxxx> <20090429145048.GA15558@xxxxxxxxxxxxx>

On Apr 29, 2009, at 9:50 AM, Christoph Hellwig wrote:

On Wed, Apr 29, 2009 at 09:40:40AM -0500, Felix Blyakher wrote:

On Apr 9, 2009, at 12:11 PM, Christoph Hellwig wrote:

On Mon, Apr 06, 2009 at 07:57:48PM -0500, Felix Blyakher wrote:
Shouldn't we set error to ENOMEM here?

Yes, good catch.

Christoph, do you want to repost this patch with that change?

Yeah, I added it long ago.

I must have missed your updated patch.

Below is the updated version.  I've been
running xfsqa on it quite a bit.

Applied to the tree.

<Prev in Thread] Current Thread [Next in Thread>