On 13.11.2024 19:51, Andrew Cooper wrote: > --- a/xen/include/xen/multiboot.h > +++ b/xen/include/xen/multiboot.h > @@ -17,7 +17,7 @@ > #ifndef __MULTIBOOT_H__ > #define __MULTIBOOT_H__ > > -#include "const.h" > +#include <xen/const.h> > > /* > * Multiboot header structure. > @@ -45,41 +45,43 @@ > > #ifndef __ASSEMBLY__ > > +#include <xen/types.h>
Isn't xen/stdint.h sufficient to include both here and ... > --- a/xen/include/xen/multiboot2.h > +++ b/xen/include/xen/multiboot2.h > @@ -114,6 +114,9 @@ > #define MULTIBOOT2_FRAMEBUFFER_TYPE_EGA_TEXT 2 > > #ifndef __ASSEMBLY__ > + > +#include <xen/types.h> > + > typedef struct { > uint32_t total_size; > uint32_t reserved; ... here? Jan