Re: [PATCH 42/51] hw/ppc: spapr: Use qemu_vfree() to free spapr->htab

[Daniel Henrique Barboza]

On 8/24/22 06:40, Bin Meng wrote:

From: Xuzhou Cheng <xuzhou.ch...@windriver.com>

spapr->htab is allocated by qemu_memalign(), hence we should use
qemu_vfree() to free it.

Fixes: c5f54f3e31bf ("pseries: Move hash page table allocation to reset time")
Fixes: b4db54132ffe ("target/ppc: Implement H_REGISTER_PROCESS_TABLE H_CALL"")
Signed-off-by: Xuzhou Cheng <xuzhou.ch...@windriver.com>
Signed-off-by: Bin Meng <bin.m...@windriver.com>
---

Reviewed-by: Daniel Henrique Barboza <danielhb...@gmail.com>

  hw/ppc/spapr.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
index bc9ba6e6dc..4034f4d130 100644
--- a/hw/ppc/spapr.c
+++ b/hw/ppc/spapr.c
@@ -1522,7 +1522,7 @@ int spapr_hpt_shift_for_ramsize(uint64_t ramsize)

void spapr_free_hpt(SpaprMachineState *spapr)

  {
-    g_free(spapr->htab);
+    qemu_vfree(spapr->htab);
      spapr->htab = NULL;
      spapr->htab_shift = 0;
      close_htab_fd(spapr);