[v4] hw/ppc: Remove tswap() calls

[PATCH v4 0/6] hw/ppc: Remove tswap() calls

Posted by Philippe Mathieu-Daudé 1 year, 1 month ago

Since v3:
- Addressed Nick & Harsh  review comments

Remove the tswap() calls on ePAPR, and convert
them to big-endian LD/ST API on sPAPR.

Build-tested only.

Philippe Mathieu-Daudé (6):
  hw/ppc/spapr: Convert HPTE() macro as hpte_get_ptr() method
  hw/ppc/spapr: Convert HPTE_VALID() macro as hpte_is_valid() method
  hw/ppc/spapr: Convert HPTE_DIRTY() macro as hpte_is_dirty() method
  hw/ppc/spapr: Convert CLEAN_HPTE() macro as hpte_set_clean() method
  hw/ppc/spapr: Convert DIRTY_HPTE() macro as hpte_set_dirty() method
  hw/ppc/epapr: Do not swap ePAPR magic value

 hw/ppc/sam460ex.c     |  2 +-
 hw/ppc/spapr.c        | 63 +++++++++++++++++++++++++++++--------------
 hw/ppc/virtex_ml507.c |  2 +-
 3 files changed, 45 insertions(+), 22 deletions(-)

-- 
2.47.1

Re: [PATCH v4 0/6] hw/ppc: Remove tswap() calls

Posted by Philippe Mathieu-Daudé 1 year, 1 month ago

Hi Nick,

Ping? (series fully reviewed)

On 20/12/24 22:30, Philippe Mathieu-Daudé wrote:
> Since v3:
> - Addressed Nick & Harsh  review comments
> 
> Remove the tswap() calls on ePAPR, and convert
> them to big-endian LD/ST API on sPAPR.

Re: [PATCH v4 0/6] hw/ppc: Remove tswap() calls

Posted by Philippe Mathieu-Daudé 11 months, 1 week ago

ping

On 9/1/25 13:54, Philippe Mathieu-Daudé wrote:
> Hi Nick,
> 
> Ping? (series fully reviewed)
> 
> On 20/12/24 22:30, Philippe Mathieu-Daudé wrote:
>> Since v3:
>> - Addressed Nick & Harsh  review comments
>>
>> Remove the tswap() calls on ePAPR, and convert
>> them to big-endian LD/ST API on sPAPR.

Re: [PATCH v4 0/6] hw/ppc: Remove tswap() calls

Posted by Philippe Mathieu-Daudé 9 months, 2 weeks ago

On 6/3/25 16:23, Philippe Mathieu-Daudé wrote:
> ping

Merged as c894bdf78b3..c2ac9f4c297, thanks!

>>> Remove the tswap() calls on ePAPR, and convert
>>> them to big-endian LD/ST API on sPAPR.

Re: [PATCH v4 0/6] hw/ppc: Remove tswap() calls

Posted by Harsh Prateek Bora 1 year, 1 month ago

On Sat, 21 Dec, 2024, 3:02 am Philippe Mathieu-Daudé, <philmd@linaro.org>
wrote:

> Since v3:
> - Addressed Nick & Harsh  review comments
>
> Remove the tswap() calls on ePAPR, and convert
> them to big-endian LD/ST API on sPAPR.
>
> Build-tested only.
>
> Philippe Mathieu-Daudé (6):
>   hw/ppc/spapr: Convert HPTE() macro as hpte_get_ptr() method
>   hw/ppc/spapr: Convert HPTE_VALID() macro as hpte_is_valid() method
>   hw/ppc/spapr: Convert HPTE_DIRTY() macro as hpte_is_dirty() method
>   hw/ppc/spapr: Convert CLEAN_HPTE() macro as hpte_set_clean() method
>   hw/ppc/spapr: Convert DIRTY_HPTE() macro as hpte_set_dirty() method
>   hw/ppc/epapr: Do not swap ePAPR magic value
>

For spapr:
Reviewed-by: Harsh Prateek Bora <harshpb@linux.ibm.com>


>  hw/ppc/sam460ex.c     |  2 +-
>  hw/ppc/spapr.c        | 63 +++++++++++++++++++++++++++++--------------
>  hw/ppc/virtex_ml507.c |  2 +-
>  3 files changed, 45 insertions(+), 22 deletions(-)
>
> --
> 2.47.1
>
>
>