Re: [PATCH 1/4] coroutine: Clean up superfluous inclusion of qemu/coroutine.h

2022-12-08 Thread Markus Armbruster
Stefan Hajnoczi writes: > Probably because block layer, aio.h, and coroutine_int.h header files > already include "qemu/coroutine.h"? Mostly, but not always. For instance, crypto/block-luks-priv.h compiles fine without it, and doesn't include it after this patch. > Reviewed-by: Stefan Hajnoczi

Re: [PATCH 1/4] coroutine: Clean up superfluous inclusion of qemu/coroutine.h

2022-12-08 Thread Stefan Hajnoczi
Probably because block layer, aio.h, and coroutine_int.h header files already include "qemu/coroutine.h"? Reviewed-by: Stefan Hajnoczi

[PATCH 1/4] coroutine: Clean up superfluous inclusion of qemu/coroutine.h

2022-12-08 Thread Markus Armbruster
Signed-off-by: Markus Armbruster --- crypto/block-luks-priv.h| 1 - include/block/raw-aio.h | 1 - include/scsi/pr-manager.h | 1 - nbd/nbd-internal.h | 1 - blockjob.c | 1 - crypto/block-luks.c | 1 - hw/9pfs/codir.c | 1 - hw/9pfs/cofile.