motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-07-31 14:23:53 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

hw/ppc/spapr: Convert DIRTY_HPTE() macro as hpte_set_dirty() method

Browse source 
Convert DIRTY_HPTE() macro as hpte_set_dirty() method.

sPAPR data structures including the hash page table are big-endian
regardless of current CPU endian mode, so use the big-endian LD/ST
API to access the hash PTEs.

Reviewed-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Harsh Prateek Bora <harshpb@linux.ibm.com>
Message-ID: <20241220213103.6314-6-philmd@linaro.org>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
This commit is contained in:
Philippe Mathieu-Daudé 2024-12-20 22:31:02 +01:00 • committed by Nicholas Piggin
parent 735f9c878a
commit c2ac9f4c29
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
hw/ppc/spapr.c
View file

@ -1425,7 +1425,11 @@ static void hpte_set_clean(SpaprMachineState *s, unsigned index)
ldq_be_p(hpte_get_ptr(s, index)) & ~HPTE64_V_HPTE_DIRTY); ldq_be_p(hpte_get_ptr(s, index)) & ~HPTE64_V_HPTE_DIRTY);
} }
#define DIRTY_HPTE(_hpte) ((*(uint64_t *)(_hpte)) |= tswap64(HPTE64_V_HPTE_DIRTY)) static void hpte_set_dirty(SpaprMachineState *s, unsigned index)
{
stq_be_p(hpte_get_ptr(s, index),
ldq_be_p(hpte_get_ptr(s, index)) | HPTE64_V_HPTE_DIRTY);
}
/* /*
* Get the fd to access the kernel htab, re-opening it if necessary * Get the fd to access the kernel htab, re-opening it if necessary
@ -1636,7 +1640,7 @@ int spapr_reallocate_hpt(SpaprMachineState *spapr, int shift, Error **errp)
spapr->htab_shift = shift; spapr->htab_shift = shift;
for (i = 0; i < size / HASH_PTE_SIZE_64; i++) { for (i = 0; i < size / HASH_PTE_SIZE_64; i++) {
DIRTY_HPTE(hpte_get_ptr(spapr, i)); hpte_set_dirty(spapr, i);
} }
} }
/* We're setting up a hash table, so that means we're not radix */ /* We're setting up a hash table, so that means we're not radix */