[PATCH 0/4] block: minor refactoring in preparation to the block layer API split

Emanuele Giuseppe Esposito posted 4 patches 2 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211124063640.3118897-1-eesposit@redhat.com
Maintainers: Hanna Reitz <hreitz@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Markus Armbruster <armbru@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
include/block/block_int.h      |  3 --
include/sysemu/blockdev.h      |  7 ++---
block.c                        |  4 ++-
block/monitor/block-hmp-cmds.c |  2 +-
blockdev.c                     | 54 ++++------------------------------
softmmu/vl.c                   | 25 +++++++++++++++-
6 files changed, 36 insertions(+), 59 deletions(-)

[PATCH 0/4] block: minor refactoring in preparation to the block layer API split

Posted by Emanuele Giuseppe Esposito 2 weeks ago
These patches are taken from my old patches and feedback of
my series "block layer: split block APIs in global state and I/O".

The reason for a separate series is that the original one is
already too long, and these patches are just refactoring the code,
mainly deleting or moving functions in blockdev.h and block_int.h.

Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>

Emanuele Giuseppe Esposito (4):
  block_int: make bdrv_backing_overridden static
  include/sysemu/blockdev.h: rename if_name in block_if_name
  include/sysemu/blockdev.h: move drive_add and inline drive_def
  include/sysemu/blockdev.h: remove drive_get_max_devs

 include/block/block_int.h      |  3 --
 include/sysemu/blockdev.h      |  7 ++---
 block.c                        |  4 ++-
 block/monitor/block-hmp-cmds.c |  2 +-
 blockdev.c                     | 54 ++++------------------------------
 softmmu/vl.c                   | 25 +++++++++++++++-
 6 files changed, 36 insertions(+), 59 deletions(-)

-- 
2.27.0


Re: [PATCH 0/4] block: minor refactoring in preparation to the block layer API split

Posted by Stefan Hajnoczi 1 week, 1 day ago
On Wed, Nov 24, 2021 at 01:36:36AM -0500, Emanuele Giuseppe Esposito wrote:
> These patches are taken from my old patches and feedback of
> my series "block layer: split block APIs in global state and I/O".
> 
> The reason for a separate series is that the original one is
> already too long, and these patches are just refactoring the code,
> mainly deleting or moving functions in blockdev.h and block_int.h.
> 
> Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
> 
> Emanuele Giuseppe Esposito (4):
>   block_int: make bdrv_backing_overridden static
>   include/sysemu/blockdev.h: rename if_name in block_if_name
>   include/sysemu/blockdev.h: move drive_add and inline drive_def
>   include/sysemu/blockdev.h: remove drive_get_max_devs
> 
>  include/block/block_int.h      |  3 --
>  include/sysemu/blockdev.h      |  7 ++---
>  block.c                        |  4 ++-
>  block/monitor/block-hmp-cmds.c |  2 +-
>  blockdev.c                     | 54 ++++------------------------------
>  softmmu/vl.c                   | 25 +++++++++++++++-
>  6 files changed, 36 insertions(+), 59 deletions(-)
> 
> -- 
> 2.27.0
> 

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>