Re: [PATCH 6/8] lib: bitmap: support "N" as an alias for size of bitmap
From: Andy Shevchenko
Date: Tue Jan 26 2021 - 22:35:50 EST
On Tue, Jan 26, 2021 at 11:37:30PM +0200, Andy Shevchenko wrote:
> On Tue, Jan 26, 2021 at 12:11:39PM -0500, Paul Gortmaker wrote:
...
> > + if (str[0] == 'N') {
> > + *num = nbits - 1;
> > + return str + 1;
> > + }
>
> But locating it here makes possible to enter a priori invalid input, like N for
> start of the region.
>
> I think this should be separate helper which is called in places where it makes
> sense.
Okay, N is 31 on 32 core system... It is a bit counter intuitive, because it's
rather _L_ast than _N_umber of CPUs.
Changing letter?
--
With Best Regards,
Andy Shevchenko