[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2/4] xen/link: Link .data.schedulers and CONSTRUCTERS in more appropriate locations



Hi Andrew,

On 6/19/19 9:11 PM, Andrew Cooper wrote:
Neither of these should live in .data

  * .data.schedulers is only ever read, so is moved into .rodata
  * CONSTRUCTORS is only ever read, and only at boot, so is moved to beside
    .init.rodata

Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
CC: Jan Beulich <JBeulich@xxxxxxxx>
CC: Wei Liu <wl@xxxxxxx>
CC: Roger Pau Monné <roger.pau@xxxxxxxxxx>
CC: Stefano Stabellini <sstabellini@xxxxxxxxxx>
CC: Julien Grall <julien.grall@xxxxxxx>
---
  xen/arch/arm/xen.lds.S | 11 ++++++-----
  xen/arch/x86/xen.lds.S | 11 ++++++-----
  2 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/xen/arch/arm/xen.lds.S b/xen/arch/arm/xen.lds.S
index 31d74a8..2b44e5d 100644
--- a/xen/arch/arm/xen.lds.S
+++ b/xen/arch/arm/xen.lds.S
@@ -66,6 +66,11 @@ SECTIONS
         *(.data.param)
         __param_end = .;
+ . = ALIGN(POINTER_ALIGN);

The alignment is going to be different on arm32 now. Please explain in the commit message why this is fine.

+       __start_schedulers_array = .;
+       *(.data.schedulers)
+       __end_schedulers_array = .;
+

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.