From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Tomas Subject: Re: [PATCH] ext4:fix unexpected error from ext4_reserve_global Date: Tue, 19 Jun 2007 10:05:24 +0400 Message-ID: <46777224.60201@clusterfs.com> References: <20070614152927.GC6517@localhost.sw.ru> <1181950884.3808.16.camel@dyn9047017103.beaverton.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Dmitriy Monakhov , linux-kernel@vger.kernel.org, linux-ext4@vger.kernel.org To: cmm@us.ibm.com Return-path: Received: from mail.rialcom.ru ([80.71.245.247]:65400 "EHLO mail.rialcom.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752472AbXFSGFs (ORCPT ); Tue, 19 Jun 2007 02:05:48 -0400 In-Reply-To: <1181950884.3808.16.camel@dyn9047017103.beaverton.ibm.com> Sender: linux-ext4-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org ACK, of course. thanks, Alex Mingming Cao wrote: > On Thu, 2007-06-14 at 19:29 +0400, Dmitriy Monakhov wrote: >> I just cant belive my eyes then i saw this at the first time... >> simple test: strace dd if=/dev/zero of=/mnt/file >> > Thanks for reporting it. > >> open("/dev/zero", O_RDONLY) = 0 >> close(1) = 0 >> open("/mnt/test/file", O_WRONLY|O_CREAT|O_TRUNC, 0666) = 1 >> read(0, "\0\0\0\0\0\0\0\0\0"..., 512) = 512 >> write(1, "\0\0\0\0\0\0\0\0"..., 512) = 512 >> read(0, "\0\0\0\0\0\0\0\0\0"..., 512) = 512 >> write(1, "\0\0\0\0\0\0\0\0"..., 512) = -1 ENOENT (No such fil >> e or directory) >> >> This strange error returned from ext4_reserve_global(). >> It's just typo because: >> a) In fact this is 100% ENOSPC situation >> b) simular function ext4_reserve_local() returns -ENOSPC >> > I agree. > > Patch is put in ext4 patch queue. Alex if you can Ack, that would be > great. > > > Thanks, > > Mingming >> Signed-off-by: Dmitriy Monakhov >> --- >> fs/ext4/balloc.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/fs/ext4/balloc.c b/fs/ext4/balloc.c >> index 4d7bfd2..43ae8f8 100644 >> --- a/fs/ext4/balloc.c >> +++ b/fs/ext4/balloc.c >> @@ -1920,7 +1920,7 @@ int ext4_reserve_global(struct super_block *sb, int blocks) >> { >> struct ext4_sb_info *sbi = EXT4_SB(sb); >> struct ext4_reservation_slot *rs; >> - int i, rc = -ENOENT; >> + int i, rc = -ENOSPC; >> __u64 free = 0; >> >> rs = sbi->s_reservation_slots; > > - > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html