Re: [PATCH v2] x86/Xen: tidy xen-head.S

[Juergen Gross <jgross@xxxxxxxx>]

On 16.06.23 08:36, Jan Beulich wrote:
> First of all move PV-only ELF notes inside the XEN_PV conditional; note
> that
> - HV_START_LOW is dropped altogether, as it was meaningful for 32-bit PV
>    only,
> - the 32-bit instance of VIRT_BASE is dropped, as it would be dead code
>    once inside the conditional,
> - while PADDR_OFFSET is not exactly unused for PVH, it defaults to zero
>    there, and the hypervisor (or tool stack) complains if it is present
>    but VIRT_BASE isn't.
> Then have the "supported features" note actually report reality: All
> three of the features there are supported and/or applicable only in
> certain cases.
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Reviewed-by: Juergen Gross <jgross@xxxxxxxx>


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature