[PATCH v5 0/2] xen: Linker scripts synchronization

Michal Orzel posted 2 patches 2 years ago
Test gitlab-ci failed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20220411070300.77380-1-michal.orzel@arm.com
xen/arch/arm/xen.lds.S    |  45 +++----------
xen/arch/x86/xen.lds.S    |  97 +++------------------------
xen/include/xen/xen.lds.h | 137 ++++++++++++++++++++++++++++++++++++++
3 files changed, 157 insertions(+), 122 deletions(-)
create mode 100644 xen/include/xen/xen.lds.h
[PATCH v5 0/2] xen: Linker scripts synchronization
Posted by Michal Orzel 2 years ago
This patch series aims to do the first step towards linker scripts
synchronization. Linker scripts for arm and x86 share a lot of common
sections and in order to make the process of changing/improving/syncing
them, these sections shall be defined in just one place.

The first patch creates an empty header file xen.lds.h to store the
constructs mutual to both x86 and arm linker scripts. It also includes
this header in the scripts.

The second patch populates xen.lds.h with the first portion of common
macros and replaces the original contructs with these helpers.

Michal Orzel (2):
  xen: Introduce a header to store common linker scripts content
  xen: Populate xen.lds.h and make use of its macros

 xen/arch/arm/xen.lds.S    |  45 +++----------
 xen/arch/x86/xen.lds.S    |  97 +++------------------------
 xen/include/xen/xen.lds.h | 137 ++++++++++++++++++++++++++++++++++++++
 3 files changed, 157 insertions(+), 122 deletions(-)
 create mode 100644 xen/include/xen/xen.lds.h

-- 
2.25.1
Re: [PATCH v5 0/2] xen: Linker scripts synchronization
Posted by Julien Grall 2 years ago
Hi Michal,

On 11/04/2022 08:02, Michal Orzel wrote:
> This patch series aims to do the first step towards linker scripts
> synchronization. Linker scripts for arm and x86 share a lot of common
> sections and in order to make the process of changing/improving/syncing
> them, these sections shall be defined in just one place.
> 
> The first patch creates an empty header file xen.lds.h to store the
> constructs mutual to both x86 and arm linker scripts. It also includes
> this header in the scripts.
> 
> The second patch populates xen.lds.h with the first portion of common
> macros and replaces the original contructs with these helpers.
> 
> Michal Orzel (2):
>    xen: Introduce a header to store common linker scripts content
>    xen: Populate xen.lds.h and make use of its macros

I have committed the two patches.

Cheers,

-- 
Julien Grall