Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1586703 > unrolled thread

Re: lib: Introduce priority array area manager

Started byGeert Uytterhoeven <geert@linux-m68k.org>
First post2017-02-23 09:00 +0100
Last post2017-02-23 10:30 +0100
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: lib: Introduce priority array area manager Geert Uytterhoeven <geert@linux-m68k.org> - 2017-02-23 09:00 +0100
    Re: lib: Introduce priority array area manager Jiri Pirko <jiri@resnulli.us> - 2017-02-23 09:40 +0100
      Re: lib: Introduce priority array area manager Jiri Pirko <jiri@resnulli.us> - 2017-02-23 10:30 +0100
      Re: lib: Introduce priority array area manager Geert Uytterhoeven <geert@linux-m68k.org> - 2017-02-23 10:30 +0100

#1586703 — Re: lib: Introduce priority array area manager

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2017-02-23 09:00 +0100
SubjectRe: lib: Introduce priority array area manager
Message-ID<tdWtH-6D3-5@gated-at.bofh.it>
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

[toc] | [next] | [standalone]


#1586736

FromJiri Pirko <jiri@resnulli.us>
Date2017-02-23 09:40 +0100
Message-ID<tdX6q-7c9-17@gated-at.bofh.it>
In reply to#1586703
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

[toc] | [prev] | [next] | [standalone]


#1586773

FromJiri Pirko <jiri@resnulli.us>
Date2017-02-23 10:30 +0100
Message-ID<tdXSO-7Pd-7@gated-at.bofh.it>
In reply to#1586736
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!

[toc] | [prev] | [next] | [standalone]


#1586775

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2017-02-23 10:30 +0100
Message-ID<tdXSO-7Pd-9@gated-at.bofh.it>
In reply to#1586736
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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web