xfs
[Top] [All Lists]
<prev [Date] next>
[Advanced]
 <prev [Thread] next>

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

from [Christoph Hellwig] [Permanent Link][Original]
To: Felix Blyakher <felixb@xxxxxxx>
Subject: Re: [PATCH 1/2] xfs: a couple getbmap cleanups
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Thu, 9 Apr 2009 13:11:35 -0400
Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
In-reply-to: <9EF64CF3-FDFC-4777-94A5-7295E827ABA4@xxxxxxx>
References: <20090224133858.GB15820@xxxxxxxxxxxxx> <9EF64CF3-FDFC-4777-94A5-7295E827ABA4@xxxxxxx>
User-agent: Mutt/1.5.18 (2008-05-17) 
On Mon, Apr 06, 2009 at 07:57:48PM -0500, Felix Blyakher wrote:
> Shouldn't we set error to ENOMEM here?

Yes, good catch.

> Should the callers be taught to handle ENOMEM now?

The kernel callchain handles it, and in userspace the only caller
(xfs_io) will handled it by printing an out of memory message.
[More with this subject...]
<Prev in Thread] Current Thread [Next in Thread>
Previous by Date:  Re: [XFS updates] XFS development tree branch, master, updated. v2.6.28-rc3-20838-g5123bc3, Christoph Hellwig
Next by Date:  Re: [PATCH 2/2] xfs: fix getbmap vs mmap deadlock, Christoph Hellwig
Previous by Thread:  Re: [PATCH 1/2] xfs: a couple getbmap cleanups, Felix Blyakher
Next by Thread:  Re: [PATCH 1/2] xfs: a couple getbmap cleanups, Felix Blyakher
Indexes:  [Date] [Thread] [Top] [All Lists]