[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 12 Nov 2008 09:46:36 +1100 (EST)
From: "NeilBrown" <neilb@...e.de>
To: "Stefan Lippers-Hollmann" <s.L-H@....de>
Cc: gregkh@...e.de, linux-kernel@...r.kernel.org, maan@...temlinux.org,
stable@...nel.org
Subject: Re: patch
md-linear-fix-a-division-by-zero-bug-for-very-small-arrays.patch
added to 2.6.27-stable tree
On Wed, November 12, 2008 9:28 am, Stefan Lippers-Hollmann wrote:
> Hi
>
> On Dienstag, 11. November 2008, gregkh@...e.de wrote:
>>
>> --- a/drivers/md/linear.c
>> +++ b/drivers/md/linear.c
>> @@ -157,6 +157,8 @@ static linear_conf_t *linear_conf(mddev_
>>
>> min_spacing = conf->array_sectors / 2;
>> sector_div(min_spacing, PAGE_SIZE/sizeof(struct dev_info *));
>> + if (min_sectors == 0)
>> + min_sectors = 1;
>>
>> /* min_spacing is the minimum spacing that will fit the hash
>> * table in one PAGE. This may be much smaller than needed.
>
> drivers/md/linear.c: In function 'linear_conf':
> drivers/md/linear.c:160: error: 'min_sectors' undeclared (first use in
> this function)
> drivers/md/linear.c:160: error: (Each undeclared identifier is reported
> only once
> drivers/md/linear.c:160: error: for each function it appears in.)
> make[5]: *** [drivers/md/linear.o] Error 1
> make[4]: *** [drivers/md] Error 2
> make[4]: *** Waiting for unfinished jobs....
>
> This one obviously depends on:
>
> commit 23242fbb470ff4c8c4d41f178832cf1929273d7d
> Author: Andre Noll <maan@...temlinux.org>
> Date: Mon Oct 13 11:55:12 2008 +1100
>
> md: linear.c: Make two local variables sector-based.
>
> which in turn depends on previous changes.
Yes, but only syntactically, not semantically.
In 2.6.27, it needs to be
if (min_spacing == 0)
min_spacing = 1;
The other previous patches don't need to be backported.
Thanks for reporting this.
NeilBrown
>
> Regards
> Stefan Lippers-Hollmann
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists