[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH 0/3] Prefer bdrv_lookup_bs() to find BDS nodes
From: |
Jeff Cody |
Subject: |
[Qemu-block] [PATCH 0/3] Prefer bdrv_lookup_bs() to find BDS nodes |
Date: |
Wed, 14 Oct 2015 09:15:59 -0400 |
This series is on top of Alberto's "Add 'blockdev-snapshot' command" series
BlockBackend devices and BlockDriverState node-names occupy the same
namespace. In addition, there are two methods that can be used in different
circumstances to look up a BlockDriverState: blk_by_name() and bdrv_find_node().
There is also a common interface, bdrv_lookup_bs(), that can look up either by
blk_by_name() or bdrv_find_node().
This makes bdrv_find_node() redundant for an external interface. And in the
cases where we just want the underlying BDS for a BlockBackend device, it is
simpler to use bdrv_lookup_bs() instead of blk_by_name().
This series makes bdrv_find_node() static and internal to block.c as a helper
function, and attempts to simplify some code when we are looking just for the
BDS of a BlockBackend device.
Jeff Cody (3):
block: Use bdrv_lookup_bs() instead of bdrv_find_node()
block: make bdrv_find_node() static
block: use bdrv_lookup_bs() over blk_by_name() for BDS only results
block.c | 30 +++++++++---------
block/block-backend.c | 2 +-
block/mirror.c | 2 +-
block/write-threshold.c | 2 +-
blockdev.c | 84 ++++++++++++++++++-------------------------------
include/block/block.h | 1 -
migration/block.c | 6 ++--
7 files changed, 50 insertions(+), 77 deletions(-)
--
1.9.3