include: Two cleanups around missing 'qemu/atomic.h'

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

Philippe Mathieu-Daudé posted 2 patches 1 year, 1 month ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20241217141326.98947-1-philmd@linaro.org

Maintainers: Stefan Hajnoczi <stefanha@redhat.com>, Kevin Wolf <kwolf@redhat.com>

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

Expand all Fold all

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

Posted by Philippe Mathieu-Daudé 1 year, 1 month ago

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(+)

-- 
2.45.2

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

Posted by Philippe Mathieu-Daudé 1 year, 1 month ago

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.

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

Posted by Ilya Leoshkevich 1 year, 1 month ago

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>

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

Posted by Richard Henderson 1 year, 1 month ago

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~