Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-11 Thread Jan Beulich
>>> On 08.01.16 at 22:22, wrote: > --- a/xen/common/Kconfig > +++ b/xen/common/Kconfig > @@ -51,4 +51,71 @@ config KEXEC > > If unsure, say Y. > > +# Enable schedulers > +menu "Schedulers" > + visible if EXPERT = "y" Does "visible if EXPERT" not

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-11 Thread Jan Beulich
>>> On 09.01.16 at 19:08, wrote: > On 09/01/16 17:50, Jonathan Creekmore wrote: >> However, if you would prefer me to remove the "If unsure" language >> completely, I can do that. The text came in before the whole >> CONFIG_EXPERT flag did. > > I would suggest dropping

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-11 Thread Doug Goldstein
On 1/11/16 10:49 AM, Jan Beulich wrote: On 11.01.16 at 17:31, wrote: >> There have been a good deal number of different downstreams that have >> been encouraging of changes like this but it seems like you are >> fundamentally opposed. Which is plainly discouraging to

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-11 Thread Doug Goldstein
On 1/11/16 9:43 AM, Jan Beulich wrote: On 11.01.16 at 16:10, wrote: >> Jan Beulich writes: >> On 08.01.16 at 22:22, wrote: +config SCHED_CREDIT + bool "Credit scheduler support" + default y >>> >>> I continue

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-11 Thread Jan Beulich
>>> On 11.01.16 at 17:31, wrote: > There have been a good deal number of different downstreams that have > been encouraging of changes like this but it seems like you are > fundamentally opposed. Which is plainly discouraging to people to > attempt to engage upstream. I could

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-11 Thread Jonathan Creekmore
Jan Beulich writes: On 08.01.16 at 22:22, wrote: >> --- a/xen/common/Kconfig >> +++ b/xen/common/Kconfig >> @@ -51,4 +51,71 @@ config KEXEC >> >>If unsure, say Y. >> >> +# Enable schedulers >> +menu "Schedulers" >> +visible if EXPERT = "y" > > Does

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-11 Thread Jan Beulich
>>> On 11.01.16 at 16:10, wrote: > Jan Beulich writes: > On 08.01.16 at 22:22, wrote: >>> +config SCHED_CREDIT >>> + bool "Credit scheduler support" >>> + default y >> >> I continue to think that not making the primary scheduler

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-09 Thread Andrew Cooper
On 08/01/16 21:22, Jonathan Creekmore wrote: > +# Enable schedulers > +menu "Schedulers" > + visible if EXPERT = "y" > + > +config SCHED_CREDIT > + bool "Credit scheduler support" > + default y > + ---help--- > + The traditional credit scheduler is a general purpose

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-09 Thread Jonathan Creekmore
Andrew Cooper writes: > On 08/01/16 21:22, Jonathan Creekmore wrote: >> +# Enable schedulers >> +menu "Schedulers" >> +visible if EXPERT = "y" >> + >> +config SCHED_CREDIT >> +bool "Credit scheduler support" >> +default y >> +---help--- >> + The traditional credit scheduler

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-09 Thread Andrew Cooper
On 09/01/16 17:50, Jonathan Creekmore wrote: > Andrew Cooper writes: > >> On 08/01/16 21:22, Jonathan Creekmore wrote: >>> +# Enable schedulers >>> +menu "Schedulers" >>> + visible if EXPERT = "y" >>> + >>> +config SCHED_CREDIT >>> + bool "Credit scheduler support" >>> + default y >>> +

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-09 Thread Jonathan Creekmore
Andrew Cooper writes: > On 09/01/16 17:50, Jonathan Creekmore wrote: >> Andrew Cooper writes: >> >>> On 08/01/16 21:22, Jonathan Creekmore wrote: +# Enable schedulers +menu "Schedulers" + visible if EXPERT = "y" + +config SCHED_CREDIT + bool "Credit scheduler

[Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

2016-01-08 Thread Jonathan Creekmore
Allow the schedulers to be independently enabled or disabled at compile-time. To match existing behavior, all four schedulers are compiled in by default, although the Credit2, RTDS, and ARINC653 are marked EXPERIMENTAL to match their not currently supported status. CC: George Dunlap