Re: [Xen-devel] [PATCH 1/2] xen/link: Link proc_info_list in .data instead of .init.data

[Oleksandr Tyshchenko <olekstysh@xxxxxxxxx>]

On Tue, Dec 4, 2018 at 7:13 PM Julien Grall <julien.grall@xxxxxxx> wrote:

Hi,

Hi, Julien

 

Title: xen/arm: link: ...

ok

 

On 12/4/18 11:42 AM, Oleksandr Tyshchenko wrote:
> From: Oleksandr Tyshchenko <oleksandr_tyshchenko@xxxxxxxx>
>
> To be able to use it for the hot pluged CPUs as well.

s/hot pluged/hot-plugged/

ok

 

>
> Signed-off-by: Oleksandr Tyshchenko <oleksandr_tyshchenko@xxxxxxxx>
> ---
>   xen/arch/arm/xen.lds.S | 10 ++++++----
>   1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/xen/arch/arm/xen.lds.S b/xen/arch/arm/xen.lds.S
> index 245a0e0..003301a 100644
> --- a/xen/arch/arm/xen.lds.S
> +++ b/xen/arch/arm/xen.lds.S
> @@ -88,6 +88,12 @@ SECTIONS
>          . = ALIGN(PAGE_SIZE);
>          *(.data.page_aligned)
>          *(.data)
> +
> +       . = ALIGN(POINTER_ALIGN);
> +       __proc_info_start = .;
> +       *(.init.proc.info)

This is a bit odd to see .init.* section in .data. So I think we want to
rename it.

indeed, will rename

 

Cheers,

--
Julien Grall

--

Regards,

Oleksandr Tyshchenko

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