Re: [XEN v2 08/11] xen/arm: guest_walk: LPAE specific bits should be enclosed within "ifndef CONFIG_ARM_PA_32"

[Julien Grall <julien@xxxxxxx>]

Hi Stefano,

On 19/01/2023 23:43, Stefano Stabellini wrote:
> On Tue, 17 Jan 2023, Ayan Kumar Halder wrote:
> > In the subsequent patch, we introduce "CONFIG_ARM_PA_32" to support
> > 32 bit physical addresses. Thus, the code specific to
> > "Large Page Address Extension" (ie LPAE) should be enclosed within
> > "ifndef CONFIG_ARM_PA_32".
> >
> > Refer xen/arch/arm/include/asm/short-desc.h, "short_desc_l1_supersec_t"
> > unsigned int extbase1:4;    /* Extended base address, PA[35:32] */
> > unsigned int extbase2:4;    /* Extended base address, PA[39:36] */
> >
> > Thus, extbase1 and extbase2 are not valid when 32 bit physical addresses
> > are supported.
> >
> > Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>
> This patch should be merged with patch 9: we shouldn't start to use a
> new kconfig symbol before it is defined.
I have asked the adaptation to be in a separate patch to avoid having a 
melting-pot patch.
So if you want the new Kconfig to be defined first, then we should 
reshuffle it.
BTW, the reshuffle will likely be necessary if you want to check for 
truncation in earlier patch.
Cheers,

--
Julien Grall