Re: [PATCH v2 0/6] migration/block: disk activation rewrite

2024-12-17 Thread Fabiano Rosas
Peter Xu writes: > CI: https://gitlab.com/peterx/qemu/-/pipelines/1577280033 > (note: it's a pipeline of two patchsets, to save CI credits and time) > > v1: https://lore.kernel.org/r/20241204005138.702289-1-pet...@redhat.com > > This is v2 of the series, removing RFC tag, because my goal is to h

Re: [PATCH 0/2] include: Two cleanups around missing 'qemu/atomic.h'

2024-12-17 Thread Ilya Leoshkevich
On Tue, 2024-12-17 at 15:13 +0100, Philippe Mathieu-Daudé wrote: > We have 2 headers using qatomic_read() without including > its declaration from "qemu/atomic.h". Include the missing > header. For my own convenience I plan to merge these 2 patches > via my tree. > > Regards, > > Phil. > > Phili

[PATCH 1/2] exec/translation-block: Include missing 'qemu/atomic.h' header

2024-12-17 Thread Philippe Mathieu-Daudé
When moving tb_cflags() in commit 88d4b5138a8 ("tcg: Make tb_cflags() usable from target-agnostic code") we forgot to include "qemu/atomic.h", which declares qatomic_read(). Explicitly include it now to avoid issue when refactoring unrelated headers. Signed-off-by: Philippe Mathieu-Daudé --- inc

Re: [PATCH 0/2] include: Two cleanups around missing 'qemu/atomic.h'

2024-12-17 Thread Richard Henderson
On 12/17/24 08:13, Philippe Mathieu-Daudé wrote: Philippe Mathieu-Daudé (2): exec/translation-block: Include missing 'qemu/atomic.h' header qemu/coroutine: Include missing 'qemu/atomic.h' header Reviewed-by: Richard Henderson r~

[PATCH 2/2] qemu/coroutine: Include missing 'qemu/atomic.h' header

2024-12-17 Thread Philippe Mathieu-Daudé
Commit 944f3d5dd21 ("coroutine: Add qemu_co_mutex_assert_locked") added an inline method which uses qatomic_read(), itself declared in "qemu/atomic.h". Explicitly include it now to avoid issue when refactoring unrelated headers. Signed-off-by: Philippe Mathieu-Daudé --- include/qemu/coroutine.h

[PATCH 0/2] include: Two cleanups around missing 'qemu/atomic.h'

2024-12-17 Thread Philippe Mathieu-Daudé
We have 2 headers using qatomic_read() without including its declaration from "qemu/atomic.h". Include the missing header. For my own convenience I plan to merge these 2 patches via my tree. Regards, Phil. Philippe Mathieu-Daudé (2): exec/translation-block: Include missing 'qemu/atomic.h' head

[PATCH] meson.build: Disallow libnfs v6 to fix the broken macOS build

2024-12-17 Thread Thomas Huth
The macOS build in our CI is currently broken since homebrew updated libnfs to version 6 - and that version apparently comes with a big API breakage. Disallow that version for now to get the broken CI job working again. Once somebody had enough time to adapt our code in block/nfs.c, we can revert t