Yeah it's the same but it should use errp. Sorry to both of you!
Paolo
On 9/2/23 06:19, Thomas Huth wrote:
> On 08/02/2023 18.19, Paolo Bonzini wrote:
>> The following changes since commit
>> ae2b5d8381a73b27f35f19c988d45c78bb4d5768:
>>
>> Merge tag 'pull-include-2023-02-06-v2' of
>> https://repo.or.cz/qemu/armbru into staging (2023-02-08 10:40:06 +0000)
>>
>> are available in the Git repository at:
>>
>> https://gitlab.com/bonzini/qemu.git tags/for-upstream
>>
>> for you to fetch changes up to e0de04cf9bd7cf03db16f33276679caf1724b75c:
>>
>> target/i386: fix ADOX followed by ADCX (2023-02-08 18:16:55 +0100)
>>
>> ----------------------------------------------------------------
>> * block/iscsi: fix double-free on BUSY or similar statuses
>> * catch [accel] entry without accelerator
>> * target/i386: various fixes for BMI and ADX instructions
>> * make the contents of meson-buildoptions.sh stable
>>
>> ----------------------------------------------------------------
>> Paolo Bonzini (8):
>> build: make meson-buildoptions.sh stable
>> remove unnecessary extern "C" blocks
>> block/iscsi: fix double-free on BUSY or similar statuses
>> vl: catch [accel] entry without accelerator
>
> You missed Philippe's review comment for that patch:
>
> https://lore.kernel.org/qemu-devel/8ec3abf5-f4aa-db40-cb7e-2f5733d93de3@linaro.org/
I interpreted "no response" as "the result is the same".