Re: [Qemu-devel] [PATCH v9 16/20] memory: Single byte swap along the I/O path

[Richard Henderson]

On 8/23/19 11:36 AM, Tony Nguyen wrote:
> Now that MemOp has been pushed down into the memory API, and
> callers are encoding endianness, we can collapse byte swaps
> along the I/O path into the accelerator and target independent
> adjust_endianness.
> 
> Collapsing byte swaps along the I/O path enables additional endian
> inversion logic, e.g. SPARC64 Invert Endian TTE bit, with redundant
> byte swaps cancelling out.
> 
> Suggested-by: Richard Henderson <address@hidden>
> Signed-off-by: Tony Nguyen <address@hidden>
> ---
>  accel/tcg/cputlb.c     | 42 ++--------------------------
>  exec.c                 | 17 +++---------
>  hw/virtio/virtio-pci.c | 10 +++----
>  memory.c               | 33 ++++++++--------------
>  memory_ldst.inc.c      | 63 ------------------------------------------
>  5 files changed, 23 insertions(+), 142 deletions(-)

Reviewed-by: Richard Henderson <address@hidden>


r~