Re: [PATCH v3 29/33] target/arm/ptw: replace target_ulong with uint64_t

[Richard Henderson]

On 5/1/25 12:35, Philippe Mathieu-Daudé wrote:
> On 1/5/25 08:23, Pierrick Bouvier wrote:
> > Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> > ---
> >   target/arm/ptw.c | 4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/target/arm/ptw.c b/target/arm/ptw.c
> > index d0a53d0987f..424d1b54275 100644
> > --- a/target/arm/ptw.c
> > +++ b/target/arm/ptw.c
> > @@ -1660,7 +1660,7 @@ static bool get_phys_addr_lpae(CPUARMState *env, 
> > S1Translate *ptw,
> >       uint64_t ttbr;
> >       hwaddr descaddr, indexmask, indexmask_grainsize;
> >       uint32_t tableattrs;
> > -    target_ulong page_size;
> > +    uint64_t page_size;
>
> Alternatively size_t.

No, this is not related to the size of any host-side object.

Bear in mind this particular page_size is used for the sizes of the intermediate page 
table levels.  So, at the very top level of the page table walk this will be quite large.



r~