[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 13/25] include/sysemu/blockdev.h: move drive_add and inlin
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH v3 13/25] include/sysemu/blockdev.h: move drive_add and inline drive_def |
Date: |
Thu, 21 Oct 2021 16:08:34 +0100 |
On Tue, Oct 12, 2021 at 04:48:54AM -0400, Emanuele Giuseppe Esposito wrote:
> drive_add is only used in softmmu/vl.c, so it can be a static
> function there, and drive_def is only a particular use case of
> qemu_opts_parse_noisily, so it can be inlined.
>
> Also remove drive_mark_claimed_by_board, as it is only defined
> but not implemented (nor used) anywhere.
>
> This also helps simplifying next patch.
>
> Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
> ---
> block/monitor/block-hmp-cmds.c | 2 +-
> blockdev.c | 27 +--------------------------
> include/sysemu/blockdev.h | 6 ++----
> softmmu/vl.c | 25 ++++++++++++++++++++++++-
> 4 files changed, 28 insertions(+), 32 deletions(-)
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
signature.asc
Description: PGP signature
- [PATCH v3 07/25] assertions for block_int global state API, (continued)
- [PATCH v3 07/25] assertions for block_int global state API, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 08/25] block: introduce assert_bdrv_graph_writable, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 10/25] assertions for blockjob_int.h, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 09/25] include/block/blockjob_int.h: split header into I/O and GS API, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 11/25] include/block/blockjob.h: global state API, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 13/25] include/sysemu/blockdev.h: move drive_add and inline drive_def, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 12/25] assertions for blockob.h global state API, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 21/25] block_int-common.h: split function pointers in BdrvChildClass, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 14/25] include/systemu/blockdev.h: global state API, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 19/25] block_int-common.h: split function pointers in BlockDriver, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 17/25] block/copy-before-write.h: global state API + assertions, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 15/25] assertions for blockdev.h global state API, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 16/25] include/block/snapshot: global state API + assertions, Emanuele Giuseppe Esposito, 2021/10/12
- [PATCH v3 20/25] block_int-common.h: assertion in the callers of BlockDriver function pointers, Emanuele Giuseppe Esposito, 2021/10/12