Re: [PATCH v3 4/5] sched: Register the schedulers into the list

[Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>]

On Thu, Jan 07, 2016 at 11:29:20AM -0600, Jonathan Creekmore wrote:
> Adds a simple macro to place a pointer to a scheduler into an array
> section at compile time. Also, goes ahead and generates the array
> entries with each of the schedulers.
> 
> CC: George Dunlap <george.dunlap@eu.citrix.com>
> CC: Dario Faggioli <dario.faggioli@citrix.com>
> CC: Josh Whitehead <josh.whitehead@dornerworks.com>
> CC: Robert VanVossen <robert.vanvossen@dornerworks.com>
> Signed-off-by: Jonathan Creekmore <jonathan.creekmore@gmail.com>
> Acked-by: Dario Faggioli <dario.faggioli@citrix.com>
> Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
> Reviewed-by: Doug Goldstein <cardoe@cardoe.com>

Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> ---
>  xen/common/sched_arinc653.c | 2 ++
>  xen/common/sched_credit.c   | 2 ++
>  xen/common/sched_credit2.c  | 2 ++
>  xen/common/sched_rt.c       | 2 ++
>  xen/include/xen/sched-if.h  | 2 ++
>  5 files changed, 10 insertions(+)
> 
> diff --git a/xen/common/sched_arinc653.c b/xen/common/sched_arinc653.c
> index dbe02ed..3b59514 100644
> --- a/xen/common/sched_arinc653.c
> +++ b/xen/common/sched_arinc653.c
> @@ -767,6 +767,8 @@ const struct scheduler sched_arinc653_def = {
>      .tick_resume    = NULL,
>  };
>  
> +REGISTER_SCHEDULER(sched_arinc653_def);
> +
>  /*
>   * Local variables:
>   * mode: C
> diff --git a/xen/common/sched_credit.c b/xen/common/sched_credit.c
> index 0dce790..e586248 100644
> --- a/xen/common/sched_credit.c
> +++ b/xen/common/sched_credit.c
> @@ -2027,3 +2027,5 @@ const struct scheduler sched_credit_def = {
>      .tick_suspend   = csched_tick_suspend,
>      .tick_resume    = csched_tick_resume,
>  };
> +
> +REGISTER_SCHEDULER(sched_credit_def);
> diff --git a/xen/common/sched_credit2.c b/xen/common/sched_credit2.c
> index 3c49ffa..38b02d0 100644
> --- a/xen/common/sched_credit2.c
> +++ b/xen/common/sched_credit2.c
> @@ -2228,3 +2228,5 @@ const struct scheduler sched_credit2_def = {
>      .alloc_domdata  = csched2_alloc_domdata,
>      .free_domdata   = csched2_free_domdata,
>  };
> +
> +REGISTER_SCHEDULER(sched_credit2_def);
> diff --git a/xen/common/sched_rt.c b/xen/common/sched_rt.c
> index 3f1d047..7640cd0 100644
> --- a/xen/common/sched_rt.c
> +++ b/xen/common/sched_rt.c
> @@ -1199,3 +1199,5 @@ const struct scheduler sched_rtds_def = {
>      .wake           = rt_vcpu_wake,
>      .context_saved  = rt_context_saved,
>  };
> +
> +REGISTER_SCHEDULER(sched_rtds_def);
> diff --git a/xen/include/xen/sched-if.h b/xen/include/xen/sched-if.h
> index 493d43f..9c6e0f5 100644
> --- a/xen/include/xen/sched-if.h
> +++ b/xen/include/xen/sched-if.h
> @@ -170,6 +170,8 @@ extern const struct scheduler sched_credit2_def;
>  extern const struct scheduler sched_arinc653_def;
>  extern const struct scheduler sched_rtds_def;
>  
> +#define REGISTER_SCHEDULER(x) static const struct scheduler *x##_entry \
> +  __used_section(".data.schedulers") = &x;
>  
>  struct cpupool
>  {
> -- 
> 2.6.4
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel