| Message ID | 1452877299-59267-4-git-send-email-jonathan.creekmore@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
> On Jan 15, 2016, at 11:01 AM, Jonathan Creekmore <jonathan.creekmore@gmail.com> wrote: > > Creates a section to contain scheduler entry pointers that are gathered > together into an array. This will allow, in a follow-on patch, scheduler > entries to be automatically gathered together into the array for > automatic parsing. > > CC: Ian Campbell <ian.campbell@citrix.com> > CC: Stefano Stabellini <stefano.stabellini@citrix.com> > CC: Keir Fraser <keir@xen.org> > CC: Jan Beulich <jbeulich@suse.com> > CC: Andrew Cooper <andrew.cooper3@citrix.com> > Signed-off-by: Jonathan Creekmore <jonathan.creekmore@gmail.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> > > --- > Changed since v4: > * Remove defensive check for schedulers since the credit scheduler > must always be present > > Changed since v3: > * Add defensive check for schedulers in the linker > > Changed since v1: > * rename the __start and __end symbols to better match > the rest of the file > --- > xen/arch/arm/xen.lds.S | 4 ++++ > xen/arch/x86/xen.lds.S | 4 ++++ > 2 files changed, 8 insertions(+) > > diff --git a/xen/arch/arm/xen.lds.S b/xen/arch/arm/xen.lds.S > index 0488f37..f501a2f 100644 > --- a/xen/arch/arm/xen.lds.S > +++ b/xen/arch/arm/xen.lds.S > @@ -57,6 +57,10 @@ SECTIONS > . = ALIGN(PAGE_SIZE); > *(.data.page_aligned) > *(.data) > + . = ALIGN(8); > + __start_schedulers_array = .; > + *(.data.schedulers) > + __end_schedulers_array = .; > *(.data.rel) > *(.data.rel.*) > CONSTRUCTORS > diff --git a/xen/arch/x86/xen.lds.S b/xen/arch/x86/xen.lds.S > index e18e08f..c1ce027 100644 > --- a/xen/arch/x86/xen.lds.S > +++ b/xen/arch/x86/xen.lds.S > @@ -80,6 +80,10 @@ SECTIONS > __stop___pre_ex_table = .; > > *(.data.read_mostly) > + . = ALIGN(8); > + __start_schedulers_array = .; > + *(.data.schedulers) > + __end_schedulers_array = .; > *(.data.rel.ro) > *(.data.rel.ro.*) > } :text > -- > 2.6.4 I am pretty sure, with Dario’s latest ACK on (2/5), that this patch is the only one in the series that has not been ACKed yet. Is there anything else that I need to do to get this series in, especially since the (1/5) CONFIG_EXPERT patch has already landed?
On Wed, 2016-01-20 at 08:34 -0600, Jonathan Creekmore wrote: > > > > diff --git a/xen/arch/arm/xen.lds.S b/xen/arch/arm/xen.lds.S > > index 0488f37..f501a2f 100644 > > --- a/xen/arch/arm/xen.lds.S > > +++ b/xen/arch/arm/xen.lds.S > > @@ -57,6 +57,10 @@ SECTIONS > > . = ALIGN(PAGE_SIZE); > > *(.data.page_aligned) > > *(.data) > > + . = ALIGN(8); > > + __start_schedulers_array = .; > > + *(.data.schedulers) > > + __end_schedulers_array = .; > > *(.data.rel) > > *(.data.rel.*) > > CONSTRUCTORS For this ARM change: Acked-by: Ian Campbell <ian.campbell@citrix.com> (FWIW I'd have done #3, #4 and #5 all in one patch, but no need to rework now). Ian.
diff --git a/xen/arch/arm/xen.lds.S b/xen/arch/arm/xen.lds.S index 0488f37..f501a2f 100644 --- a/xen/arch/arm/xen.lds.S +++ b/xen/arch/arm/xen.lds.S @@ -57,6 +57,10 @@ SECTIONS . = ALIGN(PAGE_SIZE); *(.data.page_aligned) *(.data) + . = ALIGN(8); + __start_schedulers_array = .; + *(.data.schedulers) + __end_schedulers_array = .; *(.data.rel) *(.data.rel.*) CONSTRUCTORS diff --git a/xen/arch/x86/xen.lds.S b/xen/arch/x86/xen.lds.S index e18e08f..c1ce027 100644 --- a/xen/arch/x86/xen.lds.S +++ b/xen/arch/x86/xen.lds.S @@ -80,6 +80,10 @@ SECTIONS __stop___pre_ex_table = .; *(.data.read_mostly) + . = ALIGN(8); + __start_schedulers_array = .; + *(.data.schedulers) + __end_schedulers_array = .; *(.data.rel.ro) *(.data.rel.ro.*) } :text