From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from asavdk4.altibox.net ([109.247.116.15]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1ieEye-0003D1-7M for barebox@lists.infradead.org; Mon, 09 Dec 2019 09:05:15 +0000 Date: Mon, 9 Dec 2019 10:05:04 +0100 From: Sam Ravnborg Message-ID: <20191209090504.GA32178@ravnborg.org> References: <20191209080029.3905-1-dev@lynxeye.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20191209080029.3905-1-dev@lynxeye.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH] lds: align RO_DATA_SECTION members To: Lucas Stach Cc: barebox@lists.infradead.org Hi Lucas. On Mon, Dec 09, 2019 at 09:00:29AM +0100, Lucas Stach wrote: > When using the RO_DATA_SECTION macro, all the individual components > are put into the rodata section. As they now end up in the same > section there is no alignment enforced between them anymore. This > leads to unaligned start pointers when any of the components have a > unaligned size. Enforce some alignment between them. In the kernel we have: #define ALIGN_FUNCTION() . = ALIGN(8) #define STRUCT_ALIGNMENT 32 #define STRUCT_ALIGN() . = ALIGN(STRUCT_ALIGNMENT) Could we use something similar here? Sam > > Fixes: c5d38e920101 (lds: Add and use RO_DATA_SECTION macro) > Signed-off-by: Lucas Stach > --- > include/asm-generic/barebox.lds.h | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/include/asm-generic/barebox.lds.h b/include/asm-generic/barebox.lds.h > index b6ca8eb2be27..113392d567fa 100644 > --- a/include/asm-generic/barebox.lds.h > +++ b/include/asm-generic/barebox.lds.h > @@ -17,6 +17,7 @@ > #endif > > #define BAREBOX_INITCALLS \ > + . = ALIGN(8); \ > __barebox_initcalls_start = .; \ > KEEP(*(.initcall.0)) \ > KEEP(*(.initcall.1)) \ > @@ -36,6 +37,7 @@ > __barebox_initcalls_end = .; > > #define BAREBOX_EXITCALLS \ > + . = ALIGN(8); \ > __barebox_exitcalls_start = .; \ > KEEP(*(.exitcall.0)) \ > KEEP(*(.exitcall.1)) \ > @@ -47,37 +49,44 @@ > __barebox_exitcalls_end = .; > > #define BAREBOX_CMDS \ > + . = ALIGN(8); \ > __barebox_cmd_start = .; \ > KEEP(*(SORT_BY_NAME(.barebox_cmd*))) \ > __barebox_cmd_end = .; > > #define BAREBOX_RATP_CMDS \ > + . = ALIGN(8); \ > __barebox_ratp_cmd_start = .; \ > KEEP(*(SORT_BY_NAME(.barebox_ratp_cmd*))) \ > __barebox_ratp_cmd_end = .; > > #define BAREBOX_SYMS \ > + . = ALIGN(8); \ > __usymtab_start = .; \ > KEEP(*(__usymtab)) \ > __usymtab_end = .; > > #define BAREBOX_MAGICVARS \ > + . = ALIGN(8); \ > __barebox_magicvar_start = .; \ > KEEP(*(SORT_BY_NAME(.barebox_magicvar*))) \ > __barebox_magicvar_end = .; > > #define BAREBOX_CLK_TABLE \ > + . = ALIGN(8); \ > __clk_of_table_start = .; \ > KEEP(*(.__clk_of_table)); \ > KEEP(*(.__clk_of_table_end)); \ > __clk_of_table_end = .; > > #define BAREBOX_DTB \ > + . = ALIGN(8); \ > __dtb_start = .; \ > KEEP(*(.dtb.rodata.*)); \ > __dtb_end = .; > > #define BAREBOX_IMD \ > + . = ALIGN(8); \ > KEEP(*(.barebox_imd_start)) \ > KEEP(*(.barebox_imd_1*)) \ > *(.barebox_imd_0*) \ > @@ -85,6 +94,7 @@ > > #ifdef CONFIG_PCI > #define BAREBOX_PCI_FIXUP \ > + . = ALIGN(8); \ > __start_pci_fixups_early = .; \ > KEEP(*(.pci_fixup_early)) \ > __end_pci_fixups_early = .; \ > @@ -99,6 +109,7 @@ > #endif > > #define BAREBOX_RSA_KEYS \ > + . = ALIGN(8); \ > __rsa_keys_start = .; \ > KEEP(*(.rsa_keys.rodata.*)); \ > __rsa_keys_end = .; \ > -- > 2.23.0 > > > _______________________________________________ > barebox mailing list > barebox@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/barebox _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox