[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20081114023812.GD20637@shell>
Date: Thu, 13 Nov 2008 21:38:12 -0500
From: Valerie Aurora Henson <vaurora@...hat.com>
To: Andreas Dilger <adilger@....com>
Cc: "Theodore Ts'o" <tytso@....edu>, linux-ext4@...r.kernel.org
Subject: Re: [RFC PATCH 10/17] signed int -> blk64_t to fix bugs at 2^31 - 2^32 blocks
On Thu, Nov 13, 2008 at 12:57:42PM -0700, Andreas Dilger wrote:
> On Nov 11, 2008 19:43 -0800, Valerie Aurora Henson wrote:
> > diff --git a/lib/ext2fs/alloc_tables.c b/lib/ext2fs/alloc_tables.c
> > index 7235f7d..71ad445 100644
> > --- a/lib/ext2fs/alloc_tables.c
> > +++ b/lib/ext2fs/alloc_tables.c
> > @@ -147,7 +147,7 @@ errcode_t ext2fs_allocate_group_table(ext2_filsys fs, dgrp_t group,
> > - int prev_block = 0;
> > + blk64_t prev_block = 0;
>
> > @@ -178,7 +178,7 @@ errcode_t ext2fs_allocate_group_table(ext2_filsys fs, dgrp_t group,
> > if (flexbg_size) {
> > - int prev_block = 0;
> > + blk64_t prev_block = 0;
>
> These appear to be defects in the base code and should be landed ASAP
> (as int -> blk_t) independently of this patch series.
Agreed. Ted, is this a good format for you or do you want me to
regenerate against something?
-VAL
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists