* Re: lib: Introduce priority array area manager
[not found] <20170222190204.878696619DE@gitolite.kernel.org>
@ 2017-02-23 7:56 ` Geert Uytterhoeven
2017-02-23 8:32 ` Jiri Pirko
0 siblings, 1 reply; 4+ messages in thread
From: Geert Uytterhoeven @ 2017-02-23 7:56 UTC (permalink / raw)
To: Jiri Pirko; +Cc: netdev, Linux Kernel Mailing List
Hi Jiri,
On Wed, Feb 22, 2017 at 8:02 PM, Linux Kernel Mailing List
<linux-kernel@vger.kernel.org> wrote:
> Web: https://git.kernel.org/torvalds/c/44091d29f2075972aede47ef17e1e70db3d51190
> Commit: 44091d29f2075972aede47ef17e1e70db3d51190
> Parent: b862815c3ee7b49ec20a9ab25da55a5f0bcbb95e
> Refname: refs/heads/master
> Author: Jiri Pirko <jiri@mellanox.com>
> AuthorDate: Fri Feb 3 10:29:06 2017 +0100
> Committer: David S. Miller <davem@davemloft.net>
> 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@mellanox.com>
> Signed-off-by: David S. Miller <davem@davemloft.net>
> --- 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.
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: lib: Introduce priority array area manager
2017-02-23 7:56 ` lib: Introduce priority array area manager Geert Uytterhoeven
@ 2017-02-23 8:32 ` Jiri Pirko
2017-02-23 9:22 ` Geert Uytterhoeven
0 siblings, 1 reply; 4+ messages in thread
From: Jiri Pirko @ 2017-02-23 8:32 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: Jiri Pirko, netdev, Linux Kernel Mailing List
Thu, Feb 23, 2017 at 08:56:26AM CET, geert@linux-m68k.org wrote:
>Hi Jiri,
>
>On Wed, Feb 22, 2017 at 8:02 PM, Linux Kernel Mailing List
><linux-kernel@vger.kernel.org> wrote:
>> Web: https://git.kernel.org/torvalds/c/44091d29f2075972aede47ef17e1e70db3d51190
>> Commit: 44091d29f2075972aede47ef17e1e70db3d51190
>> Parent: b862815c3ee7b49ec20a9ab25da55a5f0bcbb95e
>> Refname: refs/heads/master
>> Author: Jiri Pirko <jiri@mellanox.com>
>> AuthorDate: Fri Feb 3 10:29:06 2017 +0100
>> Committer: David S. Miller <davem@davemloft.net>
>> 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@mellanox.com>
>> Signed-off-by: David S. Miller <davem@davemloft.net>
>
>> --- 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.
You suggeste to 'drop the "parman" string'. What do you mean by that
exactly?
Thanks.
Jiri
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: lib: Introduce priority array area manager
2017-02-23 8:32 ` Jiri Pirko
@ 2017-02-23 9:22 ` Geert Uytterhoeven
2017-02-23 9:25 ` Jiri Pirko
0 siblings, 1 reply; 4+ messages in thread
From: Geert Uytterhoeven @ 2017-02-23 9:22 UTC (permalink / raw)
To: Jiri Pirko; +Cc: Jiri Pirko, netdev, Linux Kernel Mailing List
Hi Jiri,
On Thu, Feb 23, 2017 at 9:32 AM, Jiri Pirko <jiri@resnulli.us> wrote:
> Thu, Feb 23, 2017 at 08:56:26AM CET, geert@linux-m68k.org wrote:
>>On Wed, Feb 22, 2017 at 8:02 PM, Linux Kernel Mailing List
>><linux-kernel@vger.kernel.org> wrote:
>>> Web: https://git.kernel.org/torvalds/c/44091d29f2075972aede47ef17e1e70db3d51190
>>> Commit: 44091d29f2075972aede47ef17e1e70db3d51190
>>> Parent: b862815c3ee7b49ec20a9ab25da55a5f0bcbb95e
>>> Refname: refs/heads/master
>>> Author: Jiri Pirko <jiri@mellanox.com>
>>> AuthorDate: Fri Feb 3 10:29:06 2017 +0100
>>> Committer: David S. Miller <davem@davemloft.net>
>>> 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@mellanox.com>
>>> Signed-off-by: David S. Miller <davem@davemloft.net>
>>
>>> --- 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@linux-m68k.org
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: lib: Introduce priority array area manager
2017-02-23 9:22 ` Geert Uytterhoeven
@ 2017-02-23 9:25 ` Jiri Pirko
0 siblings, 0 replies; 4+ messages in thread
From: Jiri Pirko @ 2017-02-23 9:25 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: Jiri Pirko, netdev, Linux Kernel Mailing List
Thu, Feb 23, 2017 at 10:22:22AM CET, geert@linux-m68k.org wrote:
>Hi Jiri,
>
>On Thu, Feb 23, 2017 at 9:32 AM, Jiri Pirko <jiri@resnulli.us> wrote:
>> Thu, Feb 23, 2017 at 08:56:26AM CET, geert@linux-m68k.org wrote:
>>>On Wed, Feb 22, 2017 at 8:02 PM, Linux Kernel Mailing List
>>><linux-kernel@vger.kernel.org> wrote:
>>>> Web: https://git.kernel.org/torvalds/c/44091d29f2075972aede47ef17e1e70db3d51190
>>>> Commit: 44091d29f2075972aede47ef17e1e70db3d51190
>>>> Parent: b862815c3ee7b49ec20a9ab25da55a5f0bcbb95e
>>>> Refname: refs/heads/master
>>>> Author: Jiri Pirko <jiri@mellanox.com>
>>>> AuthorDate: Fri Feb 3 10:29:06 2017 +0100
>>>> Committer: David S. Miller <davem@davemloft.net>
>>>> 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@mellanox.com>
>>>> Signed-off-by: David S. Miller <davem@davemloft.net>
>>>
>>>> --- 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.
Okay. I will push this through the net tree. Thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-02-23 9:29 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20170222190204.878696619DE@gitolite.kernel.org>
2017-02-23 7:56 ` lib: Introduce priority array area manager Geert Uytterhoeven
2017-02-23 8:32 ` Jiri Pirko
2017-02-23 9:22 ` Geert Uytterhoeven
2017-02-23 9:25 ` Jiri Pirko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).