Re: lib: Introduce priority array area manager

From: Geert Uytterhoeven
Date: Thu Feb 23 2017 - 04:29:17 EST


Hi Jiri,

On Thu, Feb 23, 2017 at 9:32 AM, Jiri Pirko <jiri@xxxxxxxxxxx> wrote:
> Thu, Feb 23, 2017 at 08:56:26AM CET, geert@xxxxxxxxxxxxxx wrote:
>>On Wed, Feb 22, 2017 at 8:02 PM, Linux Kernel Mailing List
>><linux-kernel@xxxxxxxxxxxxxxx> wrote:
>>> Web: https://git.kernel.org/torvalds/c/44091d29f2075972aede47ef17e1e70db3d51190
>>> Commit: 44091d29f2075972aede47ef17e1e70db3d51190
>>> Parent: b862815c3ee7b49ec20a9ab25da55a5f0bcbb95e
>>> Refname: refs/heads/master
>>> Author: Jiri Pirko <jiri@xxxxxxxxxxxx>
>>> AuthorDate: Fri Feb 3 10:29:06 2017 +0100
>>> Committer: David S. Miller <davem@xxxxxxxxxxxxx>
>>> CommitDate: Fri Feb 3 16:35:42 2017 -0500
>>>
>>> lib: Introduce priority array area manager
>>>
>>> This introduces a infrastructure for management of linear priority
>>> areas. Priority order in an array matters, however order of items inside
>>> a priority group does not matter.
>>>
>>> As an initial implementation, L-sort algorithm is used. It is quite
>>> trivial. More advanced algorithm called P-sort will be introduced as a
>>> follow-up. The infrastructure is prepared for other algos.
>>>
>>> Alongside this, a testing module is introduced as well.
>>>
>>> Signed-off-by: Jiri Pirko <jiri@xxxxxxxxxxxx>
>>> Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>
>>
>>> --- a/lib/Kconfig
>>> +++ b/lib/Kconfig
>>> @@ -550,4 +550,7 @@ config STACKDEPOT
>>> config SBITMAP
>>> bool
>>>
>>> +config PARMAN
>>> + tristate "parman"
>>
>>| parman (PARMAN) [N/m/y] (NEW) ?
>>|
>>| There is no help available for this option.
>>
>>Can you please add a description for this option?
>>Or drop the "parman" string if this is always selected by its kernel users, and
>>never intended to be enabled by the end user.
>
> I did it in the same way other similar lib dependencies do that. Does
> not make sense to have separate description for this, cause this is
> always only a dependency of a kernel user.

OK, so the user should not be asked about it...

> You suggeste to 'drop the "parman" string'. What do you mean by that
> exactly?

... and

- tristate "parman"
+ tristate

should do the trick.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds