On 25/04/2025 12:37 pm, Oleksii Kurochko wrote: > > > On 4/25/25 1:24 PM, Andrew Cooper wrote: >> From: Shawn Anastasio <[email protected]> >> >> Signed-off-by: Shawn Anastasio <[email protected]> >> --- >> CC: Jan Beulich <[email protected]> >> CC: Roger Pau Monné <[email protected]> >> CC: Stefano Stabellini <[email protected]> >> CC: Julien Grall <[email protected]> >> CC: Volodymyr Babchuk <[email protected]> >> CC: Bertrand Marquis <[email protected]> >> CC: Michal Orzel <[email protected]> >> CC: Oleksii Kurochko <[email protected]> >> CC: Shawn Anastasio <[email protected]> >> >> Possibly not for taking on it's own (might be better with the PPC >> implemenation of map_pages_to_xen()). >> >> Nevertheless, this patch alone now does trip: >> >> diff --git a/xen/common/vmap.c b/xen/common/vmap.c >> index d6991421f3f7..9552806dba39 100644 >> --- a/xen/common/vmap.c >> +++ b/xen/common/vmap.c >> @@ -228,6 +228,8 @@ void *__vmap(const mfn_t *mfn, unsigned int >> granularity, >> void *va = vm_alloc(nr * granularity, align, type); >> unsigned long cur = (unsigned long)va; >> >> + BUILD_BUG_ON(sizeof(pte_attr_t) != 4); >> + > Could you please clarify what kind of changes should done in the case if > sizeof(pte_attr_t) == 8 (unsigned long)?
Nothing. This is simply to demonstrate that __has_include() is working, and is making PPC different to all the other architectures. ~Andrew.
