Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-14 Thread Julien Grall
Hi Michal, On 14/07/2025 13:37, Orzel, Michal wrote: On 14/07/2025 12:09, Julien Grall wrote: Hi Michal, On 14/07/2025 10:50, Orzel, Michal wrote: On 14/07/2025 11:45, Julien Grall wrote: Hi Michal, On 14/07/2025 08:37, Orzel, Michal wrote: On 12/07/2025 12:29, Julien Grall wrote: H

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-14 Thread Orzel, Michal
On 14/07/2025 12:09, Julien Grall wrote: > Hi Michal, > > On 14/07/2025 10:50, Orzel, Michal wrote: >> >> >> On 14/07/2025 11:45, Julien Grall wrote: >>> Hi Michal, >>> >>> On 14/07/2025 08:37, Orzel, Michal wrote: On 12/07/2025 12:29, Julien Grall wrote: > Hi Michal, > >

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-14 Thread Julien Grall
Hi Michal, On 14/07/2025 10:50, Orzel, Michal wrote: On 14/07/2025 11:45, Julien Grall wrote: Hi Michal, On 14/07/2025 08:37, Orzel, Michal wrote: On 12/07/2025 12:29, Julien Grall wrote: Hi Michal, On 04/07/2025 08:54, Michal Orzel wrote: When CONFIG_PDX_COMPRESSION=n, pdx_init_mask()

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-14 Thread Orzel, Michal
On 14/07/2025 11:45, Julien Grall wrote: > Hi Michal, > > On 14/07/2025 08:37, Orzel, Michal wrote: >> >> >> On 12/07/2025 12:29, Julien Grall wrote: >>> Hi Michal, >>> >>> On 04/07/2025 08:54, Michal Orzel wrote: When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and p

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-14 Thread Julien Grall
Hi Michal, On 14/07/2025 08:37, Orzel, Michal wrote: On 12/07/2025 12:29, Julien Grall wrote: Hi Michal, On 04/07/2025 08:54, Michal Orzel wrote: When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and pfn_pdx_hole_setup() are just stubs doing nothing. It does not make sense t

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-14 Thread Orzel, Michal
On 12/07/2025 12:29, Julien Grall wrote: > Hi Michal, > > On 04/07/2025 08:54, Michal Orzel wrote: >> When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and >> pfn_pdx_hole_setup() are just stubs doing nothing. It does not make >> sense to keep the two loops iterating over all th

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-12 Thread Julien Grall
Hi Michal, On 04/07/2025 08:54, Michal Orzel wrote: When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and pfn_pdx_hole_setup() are just stubs doing nothing. It does not make sense to keep the two loops iterating over all the memory banks. I saw this was already committed. But I

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-07 Thread Stefano Stabellini
On Fri, 4 Jul 2025, Michal Orzel wrote: > When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and > pfn_pdx_hole_setup() are just stubs doing nothing. It does not make > sense to keep the two loops iterating over all the memory banks. > > Signed-off-by: Michal Orzel Reviewed-by: St

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-07 Thread Orzel, Michal
On 07/07/2025 13:56, Hari Limaye wrote: > Hi Michal, > >> On Fri, Jul 04, 2025 at 09:54:28AM +, Michal Orzel wrote: >> When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and >> pfn_pdx_hole_setup() are just stubs doing nothing. It does not make >> sense to keep the two loops

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-07 Thread Hari Limaye
Apologies for noise, my previous message was missing the intended tags: On Fri, Jul 04, 2025 at 09:54:28AM +, Michal Orzel wrote: > When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and > pfn_pdx_hole_setup() are just stubs doing nothing. It does not make > sense to keep the tw

Re: [PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-07 Thread Hari Limaye
Hi Michal, > On Fri, Jul 04, 2025 at 09:54:28AM +, Michal Orzel wrote: > When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and > pfn_pdx_hole_setup() are just stubs doing nothing. It does not make > sense to keep the two loops iterating over all the memory banks. > > Signed-of

[PATCH 2/2] xen/arm: Skip loops in init_pdx() when no PDX compression is used

2025-07-04 Thread Michal Orzel
When CONFIG_PDX_COMPRESSION=n, pdx_init_mask(), pdx_region_mask() and pfn_pdx_hole_setup() are just stubs doing nothing. It does not make sense to keep the two loops iterating over all the memory banks. Signed-off-by: Michal Orzel --- xen/arch/arm/setup.c | 4 +++- 1 file changed, 3 insertions(+