[PATCH 0/3] plugins: fix endianness for qemu_plugin_mem_get_value()

Pierrick Bouvier posted 3 patches 1 week, 1 day ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260325024252.3369186-1-pierrick.bouvier@linaro.org
Maintainers: Richard Henderson <richard.henderson@linaro.org>, "Alex Bennée" <alex.bennee@linaro.org>, Pierrick Bouvier <pierrick.bouvier@linaro.org>, Alexandre Iooss <erdnaxe@crans.org>, Mahmoud Mandour <ma.mandourr@gmail.com>
There is a newer version of this series
include/plugins/qemu-plugin.h |  5 ++--
tcg/tcg-op-ldst.c             | 52 ++++++++++++++++++-----------------
tests/tcg/plugins/mem.c       | 12 ++++----
3 files changed, 36 insertions(+), 33 deletions(-)
[PATCH 0/3] plugins: fix endianness for qemu_plugin_mem_get_value()
Posted by Pierrick Bouvier 1 week, 1 day ago
This series fixes qemu_plugin_mem_get_value() endianness on non x64 hosts.
As well, it updates documentation to make clear that value returned is in
host-endian order. Finally, it fixes tests/tcg/plugins/mem.c.

Pierrick Bouvier (2):
  include/plugins/qemu-plugin.h: add note about endianness of value
    returned by qemu_plugin_mem_get_value
  tests/tcg/plugins/mem.c: fix endian swap in update_region_info

Richard Henderson (1):
  tcg: Pass host-endian values to plugin_gen_mem_callbacks_*

 include/plugins/qemu-plugin.h |  5 ++--
 tcg/tcg-op-ldst.c             | 52 ++++++++++++++++++-----------------
 tests/tcg/plugins/mem.c       | 12 ++++----
 3 files changed, 36 insertions(+), 33 deletions(-)

-- 
2.47.3
Re: [PATCH 0/3] plugins: fix endianness for qemu_plugin_mem_get_value()
Posted by Michael Tokarev 5 days, 19 hours ago
On 25.03.2026 05:42, Pierrick Bouvier wrote:
> This series fixes qemu_plugin_mem_get_value() endianness on non x64 hosts.
> As well, it updates documentation to make clear that value returned is in
> host-endian order. Finally, it fixes tests/tcg/plugins/mem.c.
> 
> Pierrick Bouvier (2):
>    include/plugins/qemu-plugin.h: add note about endianness of value
>      returned by qemu_plugin_mem_get_value
>    tests/tcg/plugins/mem.c: fix endian swap in update_region_info
> 
> Richard Henderson (1):
>    tcg: Pass host-endian values to plugin_gen_mem_callbacks_*

Hi!

Should this series be picked up for qemu-stable (10.2.x)?
(I'm not picking it up currently).

Thanks,

/mjt
Re: [PATCH 0/3] plugins: fix endianness for qemu_plugin_mem_get_value()
Posted by Michael Tokarev 5 days, 19 hours ago
On 28.03.2026 09:22, Michael Tokarev wrote:

> Should this series be picked up for qemu-stable (10.2.x)?
> (I'm not picking it up currently).

Ah, n/m - it's already marked for-stable.

Thank you for remembering about stable!
/mjt
Re: [PATCH 0/3] plugins: fix endianness for qemu_plugin_mem_get_value()
Posted by Pierrick Bouvier 5 days, 13 hours ago
On 3/27/26 11:24 PM, Michael Tokarev wrote:
> On 28.03.2026 09:22, Michael Tokarev wrote:
> 
>> Should this series be picked up for qemu-stable (10.2.x)?
>> (I'm not picking it up currently).
>

Yes :)

> Ah, n/m - it's already marked for-stable.
> 
> Thank you for remembering about stable!
> /mjt
Re: [PATCH 0/3] plugins: fix endianness for qemu_plugin_mem_get_value()
Posted by Pierrick Bouvier 1 week ago
On 3/24/26 7:42 PM, Pierrick Bouvier wrote:
> This series fixes qemu_plugin_mem_get_value() endianness on non x64 hosts.
> As well, it updates documentation to make clear that value returned is in
> host-endian order. Finally, it fixes tests/tcg/plugins/mem.c.
> 
> Pierrick Bouvier (2):
>    include/plugins/qemu-plugin.h: add note about endianness of value
>      returned by qemu_plugin_mem_get_value
>    tests/tcg/plugins/mem.c: fix endian swap in update_region_info
> 
> Richard Henderson (1):
>    tcg: Pass host-endian values to plugin_gen_mem_callbacks_*
> 
>   include/plugins/qemu-plugin.h |  5 ++--
>   tcg/tcg-op-ldst.c             | 52 ++++++++++++++++++-----------------
>   tests/tcg/plugins/mem.c       | 12 ++++----
>   3 files changed, 36 insertions(+), 33 deletions(-)
> 

This was merged into master (0b4c378fbc4e40e26003765c0c65f7f95e5eac28).

Regards,
Pierrick
Re: [PATCH 0/3] plugins: fix endianness for qemu_plugin_mem_get_value()
Posted by Pierrick Bouvier 1 week, 1 day ago
On 3/24/26 7:42 PM, Pierrick Bouvier wrote:
> This series fixes qemu_plugin_mem_get_value() endianness on non x64 hosts.
> As well, it updates documentation to make clear that value returned is in
> host-endian order. Finally, it fixes tests/tcg/plugins/mem.c.
> 
> Pierrick Bouvier (2):
>    include/plugins/qemu-plugin.h: add note about endianness of value
>      returned by qemu_plugin_mem_get_value
>    tests/tcg/plugins/mem.c: fix endian swap in update_region_info
> 
> Richard Henderson (1):
>    tcg: Pass host-endian values to plugin_gen_mem_callbacks_*
> 
>   include/plugins/qemu-plugin.h |  5 ++--
>   tcg/tcg-op-ldst.c             | 52 ++++++++++++++++++-----------------
>   tests/tcg/plugins/mem.c       | 12 ++++----
>   3 files changed, 36 insertions(+), 33 deletions(-)
> 

This series is a good candidate for stable also.

Thanks,
Pierrick