Re: [PATCH v5 5/5] sched: Use the auto-generated list of schedulers

[Jonathan Creekmore <jonathan.creekmore@gmail.com>]

Jan Beulich writes:

>>>> On 14.01.16 at 15:49, <jonathan.creekmore@gmail.com> wrote:
>> --- a/xen/common/schedule.c
>> +++ b/xen/common/schedule.c
>> @@ -64,20 +64,9 @@ static void poll_timer_fn(void *data);
>>  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
>> -};
>> +extern const struct scheduler *__start_schedulers_array[],
>> *__end_schedulers_array[];
>> +#define NUM_SCHEDULERS (__end_schedulers_array - __start_schedulers_array)
>> +static const struct scheduler **schedulers = __start_schedulers_array;
>
> #define or properly constified (to help the compiler recognize it
> won't ever change).

I will make it a define in the next version.