Message ID | 20250205040341.2056361-10-richard.henderson@linaro.org |
---|---|
State | Superseded |
Headers | show |
Series | tcg: Cleanups after disallowing 64-on-32 | expand |
On 5/2/25 05:03, Richard Henderson wrote: > The declarations use vaddr for size. Which seems dubious, since TARGET_PAGE_SIZE is int IIUC. > > Signed-off-by: Richard Henderson <richard.henderson@linaro.org> > --- > accel/tcg/cputlb.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c > index 17e2251695..75d075d044 100644 > --- a/accel/tcg/cputlb.c > +++ b/accel/tcg/cputlb.c > @@ -1193,7 +1193,7 @@ void tlb_set_page_full(CPUState *cpu, int mmu_idx, > > void tlb_set_page_with_attrs(CPUState *cpu, vaddr addr, > hwaddr paddr, MemTxAttrs attrs, int prot, > - int mmu_idx, uint64_t size) > + int mmu_idx, vaddr size) > { > CPUTLBEntryFull full = { > .phys_addr = paddr, > @@ -1208,7 +1208,7 @@ void tlb_set_page_with_attrs(CPUState *cpu, vaddr addr, > > void tlb_set_page(CPUState *cpu, vaddr addr, > hwaddr paddr, int prot, > - int mmu_idx, uint64_t size) > + int mmu_idx, vaddr size) > { > tlb_set_page_with_attrs(cpu, addr, paddr, MEMTXATTRS_UNSPECIFIED, > prot, mmu_idx, size);
On 2/11/25 23:22, Philippe Mathieu-Daudé wrote: > On 5/2/25 05:03, Richard Henderson wrote: >> The declarations use vaddr for size. > > Which seems dubious, since TARGET_PAGE_SIZE is int IIUC. This parameter must handle guest huge pages. Most often this is 2MiB or 1GiB, which do fit in "int", but logically could be any size at all. So vaddr seems the correct type. r~
On 12/2/25 19:21, Richard Henderson wrote: > On 2/11/25 23:22, Philippe Mathieu-Daudé wrote: >> On 5/2/25 05:03, Richard Henderson wrote: >>> The declarations use vaddr for size. >> >> Which seems dubious, since TARGET_PAGE_SIZE is int IIUC. > > This parameter must handle guest huge pages. Most often this is 2MiB or > 1GiB, which do fit in "int", but logically could be any size at all. So > vaddr seems the correct type. OK, got it.
diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c index 17e2251695..75d075d044 100644 --- a/accel/tcg/cputlb.c +++ b/accel/tcg/cputlb.c @@ -1193,7 +1193,7 @@ void tlb_set_page_full(CPUState *cpu, int mmu_idx, void tlb_set_page_with_attrs(CPUState *cpu, vaddr addr, hwaddr paddr, MemTxAttrs attrs, int prot, - int mmu_idx, uint64_t size) + int mmu_idx, vaddr size) { CPUTLBEntryFull full = { .phys_addr = paddr, @@ -1208,7 +1208,7 @@ void tlb_set_page_with_attrs(CPUState *cpu, vaddr addr, void tlb_set_page(CPUState *cpu, vaddr addr, hwaddr paddr, int prot, - int mmu_idx, uint64_t size) + int mmu_idx, vaddr size) { tlb_set_page_with_attrs(cpu, addr, paddr, MEMTXATTRS_UNSPECIFIED, prot, mmu_idx, size);
The declarations use vaddr for size. Signed-off-by: Richard Henderson <richard.henderson@linaro.org> --- accel/tcg/cputlb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)