| Message ID | e786dd5bbd6c235c69f5b2e49d485397f155d393.1692816595.git.sanastasio@raptorengineering.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | ppc: Enable full Xen build | expand |
On 23.08.2023 22:07, Shawn Anastasio wrote: > A few files treewide depend on defininitions in headers that they > don't include. This works when arch headers end up including the > required headers by chance, but broke on ppc64 with only minimal/stub > arch headers. > > Signed-off-by: Shawn Anastasio <sanastasio@raptorengineering.com> Acked-by: Jan Beulich <jbeulich@suse.com>
diff --git a/xen/common/memory.c b/xen/common/memory.c index b1dcbaf551..fa165ebc14 100644 --- a/xen/common/memory.c +++ b/xen/common/memory.c @@ -28,6 +28,7 @@ #include <asm/current.h> #include <asm/hardirq.h> #include <asm/p2m.h> +#include <asm/page.h> #include <public/memory.h> #include <xsm/xsm.h> diff --git a/xen/common/symbols.c b/xen/common/symbols.c index 9377f41424..691e617925 100644 --- a/xen/common/symbols.c +++ b/xen/common/symbols.c @@ -19,6 +19,7 @@ #include <xen/virtual_region.h> #include <public/platform.h> #include <xen/guest_access.h> +#include <xen/errno.h> #ifdef SYMBOLS_ORIGIN extern const unsigned int symbols_offsets[]; diff --git a/xen/common/xmalloc_tlsf.c b/xen/common/xmalloc_tlsf.c index c603c39bb9..349b31cb4c 100644 --- a/xen/common/xmalloc_tlsf.c +++ b/xen/common/xmalloc_tlsf.c @@ -27,6 +27,7 @@ #include <xen/mm.h> #include <xen/pfn.h> #include <asm/time.h> +#include <asm/page.h> #define MAX_POOL_NAME_LEN 16 diff --git a/xen/include/xen/domain.h b/xen/include/xen/domain.h index d35af34841..81fb05a642 100644 --- a/xen/include/xen/domain.h +++ b/xen/include/xen/domain.h @@ -77,6 +77,7 @@ void arch_vcpu_destroy(struct vcpu *v); int map_vcpu_info(struct vcpu *v, unsigned long gfn, unsigned int offset); void unmap_vcpu_info(struct vcpu *v); +struct xen_domctl_createdomain; int arch_domain_create(struct domain *d, struct xen_domctl_createdomain *config, unsigned int flags); diff --git a/xen/include/xen/iommu.h b/xen/include/xen/iommu.h index 110693c59f..7368df9138 100644 --- a/xen/include/xen/iommu.h +++ b/xen/include/xen/iommu.h @@ -24,6 +24,7 @@ #include <xen/page-defs.h> #include <xen/pci.h> #include <xen/spinlock.h> +#include <xen/errno.h> #include <public/domctl.h> #include <public/hvm/ioreq.h> #include <asm/device.h> diff --git a/xen/include/xen/sched.h b/xen/include/xen/sched.h index b4f43cd410..d8c8dd85a6 100644 --- a/xen/include/xen/sched.h +++ b/xen/include/xen/sched.h @@ -21,6 +21,7 @@ #include <xen/smp.h> #include <xen/perfc.h> #include <asm/atomic.h> +#include <asm/current.h> #include <xen/vpci.h> #include <xen/wait.h> #include <public/xen.h>
A few files treewide depend on defininitions in headers that they don't include. This works when arch headers end up including the required headers by chance, but broke on ppc64 with only minimal/stub arch headers. Signed-off-by: Shawn Anastasio <sanastasio@raptorengineering.com> --- v2: - (xen/domain.h) Drop <public/domctl.h> include in favor of a forward declaration of struct xen_domctl_createdomain. xen/common/memory.c | 1 + xen/common/symbols.c | 1 + xen/common/xmalloc_tlsf.c | 1 + xen/include/xen/domain.h | 1 + xen/include/xen/iommu.h | 1 + xen/include/xen/sched.h | 1 + 6 files changed, 6 insertions(+) -- 2.30.2