On 08.07.2025 20:07, Alejandro Vallejo wrote:
> --- a/xen/include/xen/bootfdt.h
> +++ b/xen/include/xen/bootfdt.h
> @@ -107,6 +107,10 @@ struct boot_domain {
>      struct boot_module *initrd;
>  
>      const char *cmdline;
> +
> +#if __has_include(<asm/bootfdt.h>)
> +    struct arch_boot_domain arch;
> +#endif
>  };

I fear I still don't follow this. In patch 1 you made domU a common-but-not-
x86 field. Why can't domid, which is entirely generic, not similarly be a
common-but-only-x86-for-now field? What is put in arch-specific structures
should, in the common case at least, be truly (i.e. conceptually) be arch-
specific imo.

Jan

Reply via email to