Re: [PATCH v3 1/3] arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro

[Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>]

On 11/28/25 4:22 PM, Grygorii Strashko wrote:
> From: Grygorii Strashko <grygorii_strashko@xxxxxxxx>
>
> The domain_set_alloc_bitsize() is x86 PV32 specific and not used by other
> arch or common code, so remove it.
>
> Signed-off-by: Grygorii Strashko <grygorii_strashko@xxxxxxxx>
> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
> ---
> v3: no changes, added tag
>
>   xen/arch/arm/include/asm/mm.h   | 1 -
>   xen/arch/ppc/include/asm/mm.h   | 1 -
>   xen/arch/riscv/include/asm/mm.h | 1 -
>   3 files changed, 3 deletions(-)
>
> diff --git a/xen/arch/arm/include/asm/mm.h b/xen/arch/arm/include/asm/mm.h
> index f702f4a0d676..4c6cc64aa7ba 100644
> --- a/xen/arch/arm/include/asm/mm.h
> +++ b/xen/arch/arm/include/asm/mm.h
> @@ -313,7 +313,6 @@ struct page_info *get_page_from_gva(struct vcpu *v, vaddr_t 
> va,
>   /* Arch-specific portion of memory_op hypercall. */
>   long arch_memory_op(int op, XEN_GUEST_HANDLE_PARAM(void) arg);
>   
> -#define domain_set_alloc_bitsize(d) ((void)0)
>   #define domain_clamp_alloc_bitsize(d, b) (b)
>   
>   unsigned long domain_get_maximum_gpfn(struct domain *d);
> diff --git a/xen/arch/ppc/include/asm/mm.h b/xen/arch/ppc/include/asm/mm.h
> index a33eeec43bd6..e478291fdf55 100644
> --- a/xen/arch/ppc/include/asm/mm.h
> +++ b/xen/arch/ppc/include/asm/mm.h
> @@ -96,7 +96,6 @@ static inline struct page_info *virt_to_page(const void *v)
>   /* TODO: implement */
>   #define mfn_valid(mfn) ({ (void) (mfn); 0; })
>   
> -#define domain_set_alloc_bitsize(d) ((void)(d))
>   #define domain_clamp_alloc_bitsize(d, b) (b)
>   
>   #define PFN_ORDER(pfn_) ((pfn_)->v.free.order)
> diff --git a/xen/arch/riscv/include/asm/mm.h b/xen/arch/riscv/include/asm/mm.h
> index 9283616c0224..11e9f26bf0a5 100644
> --- a/xen/arch/riscv/include/asm/mm.h
> +++ b/xen/arch/riscv/include/asm/mm.h
> @@ -292,7 +292,6 @@ static inline bool arch_mfns_in_directmap(unsigned long 
> mfn, unsigned long nr)
>   /* TODO: implement */
>   #define mfn_valid(mfn) ({ (void)(mfn); 0; })
>   
> -#define domain_set_alloc_bitsize(d) ((void)(d))
>   #define domain_clamp_alloc_bitsize(d, b) ((void)(d), (b))
>   
>   #define PFN_ORDER(pg) ((pg)->v.free.order)
LGTM:
 Reviewed-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>

Thanks.

~ Oleksii