Mingming Cao Wrote: > ext4: Fix ext4_mb_init_cache return error > > From: Mingming Cao <cmm@xxxxxxxxxx> > > ext4_mb_init_cache() incorrectly always return EIO on success. This > causes the caller of ext4_mb_init_cache() fail when it checks > the return value. > > Signed-off-by: Mingming Cao <cmm@xxxxxxxxxx> > --- > fs/ext4/mballoc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Index: linux-2.6.26-rc4/fs/ext4/mballoc.c > =================================================================== > --- linux-2.6.26-rc4.orig/fs/ext4/mballoc.c 2008-06-01 12:55:19.000000000 -0700 > +++ linux-2.6.26-rc4/fs/ext4/mballoc.c 2008-06-01 12:55:23.000000000 -0700 > @@ -798,7 +798,7 @@ static int ext4_mb_init_cache(struct pag > for (i = 0; i < groups_per_page && bh[i]; i++) > wait_on_buffer(bh[i]); > > - err = -EIO; > + err = 0; > for (i = 0; i < groups_per_page && bh[i]; i++) > if (!buffer_uptodate(bh[i])) > goto out; > > I think the fix should be err = -EIO; for (i = 0; i < groups_per_page && bh[i]; i++) if (!buffer_uptodate(bh[i])) goto out; + err = 0; Is that right? Shen Feng -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html