[PATCH] arm/link: Remove exception table sections

Andrew Cooper posted 1 patch 2 months, 1 week ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20250818081824.3538065-1-andrew.cooper3@citrix.com
xen/arch/arm/xen.lds.S | 10 ----------
1 file changed, 10 deletions(-)
[PATCH] arm/link: Remove exception table sections
Posted by Andrew Cooper 2 months, 1 week ago
This was almost certainly copy&paste from x86.  ARM does not us these nor
selects HAS_EX_TABLE, so drop the sections and their boundary markers.

No functional change.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Stefano Stabellini <sstabellini@kernel.org>
CC: Julien Grall <julien@xen.org>
CC: Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>
CC: Bertrand Marquis <bertrand.marquis@arm.com>
CC: Michal Orzel <michal.orzel@amd.com>

Found because I'm getting rid of .ex_table.pre on x86 too.
---
 xen/arch/arm/xen.lds.S | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/xen/arch/arm/xen.lds.S b/xen/arch/arm/xen.lds.S
index 9f30c3a13ed1..db17ff1efa98 100644
--- a/xen/arch/arm/xen.lds.S
+++ b/xen/arch/arm/xen.lds.S
@@ -86,16 +86,6 @@ SECTIONS
   } : text
 
   .data.read_mostly : {
-       /* Exception table */
-       __start___ex_table = .;
-       *(.ex_table)
-       __stop___ex_table = .;
-
-       /* Pre-exception table */
-       __start___pre_ex_table = .;
-       *(.ex_table.pre)
-       __stop___pre_ex_table = .;
-
        *(.data.read_mostly)
   } :text
 
-- 
2.39.5
Re: [PATCH] arm/link: Remove exception table sections
Posted by Julien Grall 2 months, 1 week ago
Hi Andrew,

On 18/08/2025 09:18, Andrew Cooper wrote:
> This was almost certainly copy&paste from x86.  ARM does not us these nor
> selects HAS_EX_TABLE, so drop the sections and their boundary markers.
> 
> No functional change.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>

Acked-by: Julien Grall <jgrall@amazon.com>

Cheers,

> ---
> CC: Stefano Stabellini <sstabellini@kernel.org>
> CC: Julien Grall <julien@xen.org>
> CC: Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>
> CC: Bertrand Marquis <bertrand.marquis@arm.com>
> CC: Michal Orzel <michal.orzel@amd.com>
> 
> Found because I'm getting rid of .ex_table.pre on x86 too.
> ---
>   xen/arch/arm/xen.lds.S | 10 ----------
>   1 file changed, 10 deletions(-)
> 
> diff --git a/xen/arch/arm/xen.lds.S b/xen/arch/arm/xen.lds.S
> index 9f30c3a13ed1..db17ff1efa98 100644
> --- a/xen/arch/arm/xen.lds.S
> +++ b/xen/arch/arm/xen.lds.S
> @@ -86,16 +86,6 @@ SECTIONS
>     } : text
>   
>     .data.read_mostly : {
> -       /* Exception table */
> -       __start___ex_table = .;
> -       *(.ex_table)
> -       __stop___ex_table = .;
> -
> -       /* Pre-exception table */
> -       __start___pre_ex_table = .;
> -       *(.ex_table.pre)
> -       __stop___pre_ex_table = .;
> -
>          *(.data.read_mostly)
>     } :text
>   

-- 
Julien Grall