[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 0/2] change IVSHMEM endianess to LITTLE_ENDIAN

From: Daniel Henrique Barboza
Subject: Re: [PATCH 0/2] change IVSHMEM endianess to LITTLE_ENDIAN
Date: Wed, 24 Nov 2021 06:21:33 -0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0

On 11/23/21 20:22, Cédric Le Goater wrote:
On 11/23/21 22:19, Daniel Henrique Barboza wrote:

This small series fixes an issue reported in Gitlab [1] that
affects PowerPC big-endian and little-endian and probably all
other big-endians in the wild that might use 'ivshmem'.

It's not clear to me who is the maintainer/responsible for this device
(MAINTAINERS doesn't seem to have any 'ivhshmem' entries nor someone
that looks upon all hw/misc/* files) so I didn't add any CC in that
regard. 'qemu-ppc' is being copied for awareness since they are the
folks that are most likely being impacted by the bug.

[1] https://gitlab.com/qemu-project/qemu/-/issues/168

Do we want these fixes for 6.2 ?

No, I don't think it's necessary. Changing endianess is something that I'd 
do in the start of the 7.0 cycle. This bug has been around for years at this
point. It can wait a couple of months.

I'll re-send these with Thomas' Ack and corrections and with a "for-7.0" subject
for extra clarity.





Daniel Henrique Barboza (2):
   ivshmem.c: change endianness to LITTLE_ENDIAN
   ivshmem-test.c: enable test_ivshmem_server for ppc64 arch

  hw/misc/ivshmem.c          | 2 +-
  tests/qtest/ivshmem-test.c | 5 +----
  2 files changed, 2 insertions(+), 5 deletions(-)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]