Re: [PATCH v4 6/7] tests/plugin/mem: add option to print memory accesses

[Richard Henderson]

On 7/2/24 11:44, Pierrick Bouvier wrote:
> +    case QEMU_PLUGIN_MEM_VALUE_U128:
> +        g_string_append_printf(out, "0x%.0"PRIx64"%"PRIx64,
> +                               value.data.u128.high, value.data.u128.low);

PRIx64 does not pad.
You need %016"PRIx64 for the low value.

Otherwise,
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


r~