* 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@vger.kernel.org, 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@vger.kernel.org, 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@vger.kernel.org, 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@vger.kernel.org, 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:25 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