> On 12/16/2025 6:20 AM  Anirudh Rayabharam <[email protected]> wrote:

[...]

> +#if IS_ENABLED(CONFIG_ARM64)
> +union hv_arm64_vp_execution_state {
> +     u16 as_uint16;
> +     struct {
> +             u16 cpl:2;

That looks oddly x86(-64)-specific (Current Priviledge Level).

Unless I'm mistaken, CPL doesn't belong here, and the bitfield isn't
used on ARM64. Provided the layout of the struct is correct, the
bitfield can have a better name of `reserved0` or something like that.

> +             u16 debug_active:1;
> +             u16 interruption_pending:1;
> +             u16 vtl:4;
> +             u16 virtualization_fault_active:1;
> +             u16 reserved:7;
> +     } __packed;

Reply via email to