diff mbox

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

Message ID 1452187761-38328-5-git-send-email-jonathan.creekmore@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jonathan Creekmore Jan. 7, 2016, 5:29 p.m. UTC
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>
---
 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(+)

Comments

Konrad Rzeszutek Wilk Jan. 8, 2016, 3:48 p.m. UTC | #1
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
diff mbox

Patch

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
 {