Messages in this thread | | | Date | Wed, 10 Dec 2008 09:44:46 +0100 (CET) | From | Geert Uytterhoeven <> | Subject | Re: [PATCH] fork_init: fix division by zero |
| |
On Tue, 9 Dec 2008, Yuri Tikhonov wrote: > The following patch fixes divide-by-zero error for the > cases of really big PAGE_SIZEs (e.g. 256KB on ppc44x). > Support for such big page sizes on 44x is not present in the > current kernel yet, but coming soon. > > Also this patch fixes the comment for the max_threads > settings, as this didn't match the things actually done > in the code. > > Signed-off-by: Yuri Tikhonov <yur@emcraft.com> > Signed-off-by: Ilya Yanok <yanok@emcraft.com> > --- > kernel/fork.c | 8 ++++++-- > 1 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/kernel/fork.c b/kernel/fork.c > index 2a372a0..b0ac2fb 100644 > --- a/kernel/fork.c > +++ b/kernel/fork.c > @@ -181,10 +181,14 @@ void __init fork_init(unsigned long mempages) > > /* > * The default maximum number of threads is set to a safe > - * value: the thread structures can take up at most half > - * of memory. > + * value: the thread structures can take up at most > + * (1/8) part of memory. > */ > +#if (8 * THREAD_SIZE) > PAGE_SIZE > max_threads = mempages / (8 * THREAD_SIZE / PAGE_SIZE); > +#else > + max_threads = mempages * PAGE_SIZE / (8 * THREAD_SIZE); ^^^^^^^^^^^^^^^^^^^^ > +#endif
Can't this overflow, e.g. on 32-bit machines with HIGHMEM?
With kind regards,
Geert Uytterhoeven Software Architect
Sony Techsoft Centre Europe The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium
Phone: +32 (0)2 700 8453 Fax: +32 (0)2 700 8622 E-mail: Geert.Uytterhoeven@sonycom.com Internet: http://www.sony-europe.com/
A division of Sony Europe (Belgium) N.V. VAT BE 0413.825.160 · RPR Brussels Fortis · BIC GEBABEBB · IBAN BE41293037680010 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |