[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1162402145.9524.501.camel@lnx-dfults.americas.sgi.com>
Date: Wed, 01 Nov 2006 11:29:04 -0600
From: Derek Fults <dfults@....com>
To: Randy Dunlap <randy.dunlap@...cle.com>
Cc: linux-kernel@...r.kernel.org, Andi Kleen <ak@...e.de>
Subject: Re: [PATCH] Add get_range, allows a hyhpenated range to get_options
On Wed, 2006-11-01 at 08:57 -0800, Randy Dunlap wrote:
> On Wed, 01 Nov 2006 10:22:36 -0600 Derek Fults wrote:
>
> > This allows a hyphenated range of positive numbers in the string passed
> > to command line helper function, get_options.
> > Currently the command line option "isolcpus=" takes as its argument a
> > list of cpus.
> > Format: <cpu number>,...,<cpu number>
> > This can get extremely long when isolating the majority of cpus on a
> > large system. Valid values of <cpu_number> include all cpus, 0 to
> > "number of CPUs in system - 1".
> >
> > Signed-off-by: Derek Fults <dfults@....com>
> >
> > Index: linux/lib/cmdline.c
> > ===================================================================
> > --- linux.orig/lib/cmdline.c 2006-09-19 22:42:06.000000000 -0500
> > +++ linux/lib/cmdline.c 2006-11-01 10:16:09.988659834 -0600
> > @@ -16,6 +16,21 @@
> > #include <linux/kernel.h>
> > #include <linux/string.h>
> >
> > +/* If a hyphen was found in get_option, this will handle the
> > + * range of numbers given.
>
> Still have trailing whitespace in the patch (2 lines above).
>
> I think that this comment should explain that the M-N range
> is handled by expanding it to an array of [M, M+1, ..., N].
>
How's this for a description?
/* If a hyphen was found in get_option, this will handle the
* range of numbers, M-N. This will expand the range and insert
* the values[M, M+1, ..., N] into the ints array in get_options.
*/
Thanks,
Derek
-
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