mbox series

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

Message ID 20241217141326.98947-1-philmd@linaro.org
Headers show
Series include: Two cleanups around missing 'qemu/atomic.h' | expand

Message

Philippe Mathieu-Daudé Dec. 17, 2024, 2:13 p.m. UTC 
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' header
  qemu/coroutine: Include missing 'qemu/atomic.h' header

 include/exec/translation-block.h | 1 +
 include/qemu/coroutine.h         | 1 +
 2 files changed, 2 insertions(+)

Comments

Richard Henderson Dec. 17, 2024, 2:28 p.m. UTC | #1 
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 <richard.henderson@linaro.org>

r~
Ilya Leoshkevich Dec. 17, 2024, 2:37 p.m. UTC | #2 
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.
> 
> Philippe Mathieu-Daudé (2):
>   exec/translation-block: Include missing 'qemu/atomic.h' header
>   qemu/coroutine: Include missing 'qemu/atomic.h' header
> 
>  include/exec/translation-block.h | 1 +
>  include/qemu/coroutine.h         | 1 +
>  2 files changed, 2 insertions(+)

Acked-by: Ilya Leoshkevich <iii@linux.ibm.com>
Philippe Mathieu-Daudé Dec. 19, 2024, 5:10 p.m. UTC | #3 
On 17/12/24 15: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

Patches queued.