This is v3 of "block: Modify block-commit to use node-names". Changes:
* Add Eric's reviewed-by * Addressed Eric's review comments * Dropped HMP changes * Added helper function for setting the overlay, and set the overlay in bdrv_append() * Use bs->backing_file instead of bs->backing_hd->filename in block_stream Using node-names instead of filenames for block job operations over QMP is a superior method of identifying the block driver images to operate on, as it removes all pathname ambiguity. This series modifies block-commit and block-stream to use node-names, and creates a new QAPI command to allow stand-alone backing file changes on an image file. So that node-names can be used as desired for all block job operations, this series also auto-generates node-names for every BDS. User-specified node-names will override any autogenerated Jeff Cody (12): block: Auto-generate node_names for each BDS entry block: add helper function to determine if a BDS is in a chain block: Add overlay BDS pointer into the BlockDriverState struct block: add helper function to find the active layer of any BDS block: simplify bdrv_find_base() block: make 'top' argument to block-commit optional block: Accept node-name arguments for block-commit block: extend block-commit to accept a string for the backing file block: add ability for block-stream to use node-name block: add backing-file option to block-stream block: Add QMP documentation for block-stream block: add QAPI command to allow live backing file change block.c | 71 +++++++++-- block/commit.c | 9 +- block/stream.c | 11 +- blockdev.c | 291 ++++++++++++++++++++++++++++++++++++++++++---- hmp.c | 3 +- include/block/block.h | 6 +- include/block/block_int.h | 4 +- qapi-schema.json | 152 ++++++++++++++++++++++-- qmp-commands.hx | 187 +++++++++++++++++++++++++++-- tests/qemu-iotests/040 | 28 +++-- 10 files changed, 685 insertions(+), 77 deletions(-) -- 1.8.3.1