[PATCH v2 0/3] xen/x86: move d->arch.physaddr_bitsize field handling to pv32

Grygorii Strashko posted 3 patches 2 weeks, 2 days ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20251127221244.3747918-1-grygorii._5Fstrashko@epam.com
There is a newer version of this series
xen/arch/arm/include/asm/mm.h     |  3 ---
xen/arch/ppc/include/asm/mm.h     |  3 ---
xen/arch/riscv/include/asm/mm.h   |  3 ---
xen/arch/x86/include/asm/domain.h |  8 +++++---
xen/arch/x86/include/asm/mm.h     |  7 +++++--
xen/arch/x86/pv/dom0_build.c      |  6 ++++--
xen/arch/x86/pv/domain.c          | 23 +++++++++++++++++++++++
xen/arch/x86/x86_64/mm.c          | 20 --------------------
xen/include/xen/mm.h              |  4 ++++
9 files changed, 41 insertions(+), 36 deletions(-)
[PATCH v2 0/3] xen/x86: move d->arch.physaddr_bitsize field handling to pv32
Posted by Grygorii Strashko 2 weeks, 2 days ago
From: Grygorii Strashko <grygorii_strashko@epam.com>

1) arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
2) add generic domain_clamp_alloc_bitsize() macro and clean up !x86 arches
3) move d->arch.physaddr_bitsize field handling to pv32 code

changes in v2:
- split on 3 patches
- move physaddr_bitsize in struct pv_domain
- make minimal style adjustments as requested

Grygorii Strashko (3):
  arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
  xen/mm: add generic domain_clamp_alloc_bitsize() macro
  xen/x86: move d->arch.physaddr_bitsize field handling to pv32

 xen/arch/arm/include/asm/mm.h     |  3 ---
 xen/arch/ppc/include/asm/mm.h     |  3 ---
 xen/arch/riscv/include/asm/mm.h   |  3 ---
 xen/arch/x86/include/asm/domain.h |  8 +++++---
 xen/arch/x86/include/asm/mm.h     |  7 +++++--
 xen/arch/x86/pv/dom0_build.c      |  6 ++++--
 xen/arch/x86/pv/domain.c          | 23 +++++++++++++++++++++++
 xen/arch/x86/x86_64/mm.c          | 20 --------------------
 xen/include/xen/mm.h              |  4 ++++
 9 files changed, 41 insertions(+), 36 deletions(-)

-- 
2.34.1
Re: [PATCH v2 0/3] xen/x86: move d->arch.physaddr_bitsize field handling to pv32
Posted by Jan Beulich 2 weeks, 1 day ago
On 27.11.2025 23:12, Grygorii Strashko wrote:
> From: Grygorii Strashko <grygorii_strashko@epam.com>
> 
> 1) arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
> 2) add generic domain_clamp_alloc_bitsize() macro and clean up !x86 arches
> 3) move d->arch.physaddr_bitsize field handling to pv32 code
> 
> changes in v2:
> - split on 3 patches
> - move physaddr_bitsize in struct pv_domain
> - make minimal style adjustments as requested
> 
> Grygorii Strashko (3):
>   arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
>   xen/mm: add generic domain_clamp_alloc_bitsize() macro

As I'm happy with these two being separate:
Acked-by: Jan Beulich <jbeulich@suse.com>

But of course Andrew's objection needs dealing with before they can go in. And
as he said, at least Arm maintainers need Cc-ing for them to actually be aware
that their ack is going to be needed.

Jan

>   xen/x86: move d->arch.physaddr_bitsize field handling to pv32
> 
>  xen/arch/arm/include/asm/mm.h     |  3 ---
>  xen/arch/ppc/include/asm/mm.h     |  3 ---
>  xen/arch/riscv/include/asm/mm.h   |  3 ---
>  xen/arch/x86/include/asm/domain.h |  8 +++++---
>  xen/arch/x86/include/asm/mm.h     |  7 +++++--
>  xen/arch/x86/pv/dom0_build.c      |  6 ++++--
>  xen/arch/x86/pv/domain.c          | 23 +++++++++++++++++++++++
>  xen/arch/x86/x86_64/mm.c          | 20 --------------------
>  xen/include/xen/mm.h              |  4 ++++
>  9 files changed, 41 insertions(+), 36 deletions(-)
>
Re: [PATCH v2 0/3] xen/x86: move d->arch.physaddr_bitsize field handling to pv32
Posted by Andrew Cooper 2 weeks, 2 days ago
On 27/11/2025 10:12 pm, Grygorii Strashko wrote:
> From: Grygorii Strashko <grygorii_strashko@epam.com>
>
> 1) arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
> 2) add generic domain_clamp_alloc_bitsize() macro and clean up !x86 arches
> 3) move d->arch.physaddr_bitsize field handling to pv32 code
>
> changes in v2:
> - split on 3 patches
> - move physaddr_bitsize in struct pv_domain
> - make minimal style adjustments as requested
>
> Grygorii Strashko (3):
>   arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
>   xen/mm: add generic domain_clamp_alloc_bitsize() macro

These two should be merged.  You also need to CC the ARM/PPC/RISC-V
maintainers.

~Andrew

Re: [PATCH v2 0/3] xen/x86: move d->arch.physaddr_bitsize field handling to pv32
Posted by Grygorii Strashko 2 weeks, 1 day ago

On 28.11.25 00:24, Andrew Cooper wrote:
> On 27/11/2025 10:12 pm, Grygorii Strashko wrote:
>> From: Grygorii Strashko <grygorii_strashko@epam.com>
>>
>> 1) arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
>> 2) add generic domain_clamp_alloc_bitsize() macro and clean up !x86 arches
>> 3) move d->arch.physaddr_bitsize field handling to pv32 code
>>
>> changes in v2:
>> - split on 3 patches
>> - move physaddr_bitsize in struct pv_domain
>> - make minimal style adjustments as requested
>>
>> Grygorii Strashko (3):
>>    arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
>>    xen/mm: add generic domain_clamp_alloc_bitsize() macro
> 
> These two should be merged.  You also need to CC the ARM/PPC/RISC-V
> maintainers.

Omg. missed to drop supress-cc=all :( will resend with patch 3 updated.

-- 
Best regards,
-grygorii


Re: [PATCH v2 0/3] xen/x86: move d->arch.physaddr_bitsize field handling to pv32
Posted by Jan Beulich 2 weeks, 1 day ago
On 27.11.2025 23:24, Andrew Cooper wrote:
> On 27/11/2025 10:12 pm, Grygorii Strashko wrote:
>> From: Grygorii Strashko <grygorii_strashko@epam.com>
>>
>> 1) arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
>> 2) add generic domain_clamp_alloc_bitsize() macro and clean up !x86 arches
>> 3) move d->arch.physaddr_bitsize field handling to pv32 code
>>
>> changes in v2:
>> - split on 3 patches
>> - move physaddr_bitsize in struct pv_domain
>> - make minimal style adjustments as requested
>>
>> Grygorii Strashko (3):
>>   arm/ppc/riscv: drop unused domain_set_alloc_bitsize() macro
>>   xen/mm: add generic domain_clamp_alloc_bitsize() macro
> 
> These two should be merged.

Why? They're dealing with two separate macros / functions ("set" vs "clamp").

Jan