Re: [Xen-devel] [PATCH 1/4] build: Hook the schedulers into Kconfig

[Dario Faggioli <dario.faggioli@xxxxxxxxxx>]

On Thu, 2015-12-17 at 14:59 -0600, Jonathan Creekmore wrote:
> Allow the schedulers to be independently enabled or disabled at
> compile-time instead of just allowing the scheduler to be selected on
> the command line. 
>
Reading this quickly, that "instead" gave me a bit of an hard time. I'm
not a native English speaker, and I'm sure it's me that am wrong, but
for some reason the sentence made me think that the patch would somehow
disallow specifying a scheduler during boot, in Xen's command line.

Fact is, I don't think the phrase "instead of just allowing the
scheduler to be selected on the command line." adds much information,
and I'd just remove it.

> To match existing behavior, all four schedulers are
> compiled in by default, although the RTDS and ARINC653 are marked
> EXPERIMENTAL to match their not currently supported status.
> 
This may change shortly, but as of now, Credit2 is still experimental
too. I think I'd still recommend enabling it in the help file (i.e.,
I'm ok with "If unsure, say Y"), but we certainly should mark it with
"(EPERIMENTAL)".

I don't know much on how kconfig works, so all the changes to the
makefile, etc, I'm not able to review them properly.

On the other hand, the code here below...

> --- a/xen/common/schedule.c
> +++ b/xen/common/schedule.c
> @@ -38,8 +38,8 @@
> Â#include <public/sched.h>
> Â#include <xsm/xsm.h>
> Â
> -/* opt_sched: scheduler - default to credit */
> -static char __initdata opt_sched[10] = "credit";
> +/* opt_sched: scheduler - default to configured value */
> +static char __initdata opt_sched[10] = CONFIG_SCHED_DEFAULT;
> Âstring_param("sched", opt_sched);
> Â
> Â/* if sched_smt_power_savings is set,
> @@ -65,10 +65,18 @@ DEFINE_PER_CPU(struct schedule_data,
> schedule_data);
> ÂDEFINE_PER_CPU(struct scheduler *, scheduler);
> Â
> Âstatic const struct scheduler *schedulers[] = {
> +#ifdef CONFIG_SCHED_CREDIT
> ÂÂÂÂÂ&sched_credit_def,
> +#endif
> +#ifdef CONFIG_SCHED_CREDIT2
> ÂÂÂÂÂ&sched_credit2_def,
> +#endif
> +#ifdef CONFIG_SCHED_ARINC653
> ÂÂÂÂÂ&sched_arinc653_def,
> +#endif
> +#ifdef CONFIG_SCHED_RTDS
> ÂÂÂÂÂ&sched_rtds_def,
> +#endif
> Â};
> Â
... can have, with the changelog changed as shown, my:

ÂAcked-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>

Regards,
Daio
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel