From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45167) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1f9vmb-0007tV-20 for qemu-devel@nongnu.org; Sat, 21 Apr 2018 12:54:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1f9vma-0003AF-3e for qemu-devel@nongnu.org; Sat, 21 Apr 2018 12:54:41 -0400 From: Max Reitz Date: Sat, 21 Apr 2018 18:54:16 +0200 Message-Id: <20180421165423.30759-1-mreitz@redhat.com> Subject: [Qemu-devel] [PATCH 0/7] qemu-img: Improve option help for amend List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: qemu-devel@nongnu.org, Max Reitz , Kevin Wolf Currently, "qemu-img amend -f $format -o help" prints many things which it claims to be supported, but most of the time it's wrong. Usually that starts with the format already: No format but qcow2 supports option amendment, so we should never claim that a format supports any options when it actually does not support amendment in the first place. It goes on with the options themselves. The qcow2 driver does not support amendment of all creation options, so we should not claim it does. Actually knowing which formats are supported exactly would be a bit difficult (this would probably involve adding a field to QemuOptDesc, and I don't really want to do that), but what we can do instead is to at least advise the user that the options we print are all of the creation options and that we might not support amending all of them. There is a Bugzilla for this here: https://bugzilla.redhat.com/show_bug.cgi?id=1537956 On the way to address these issues, there is more to be done, though. bdrv_amend_options() does not have an Error parameter yet, and there is no real excuse for that. Also, "qemu-img create -o help" has its own little issue with formats that do not support creation: $ qemu-img create -f bochs -o help Supported options: qemu-img: util/qemu-option.c:219: qemu_opts_print_help: Assertion `list' failed. [1] 24831 abort (core dumped) qemu-img create -f bochs -o help Let's fix that, too. Max Reitz (7): qemu-img: Amendment support implies create_opts block: Add Error parameter to bdrv_amend_options qemu-option: Pull out "Supported options" print qemu-img: Add print_amend_option_help() qemu-img: Recognize no creation support in -o help iotests: Test help option for unsupporting formats iotests: Rework 113 include/block/block.h | 3 +- include/block/block_int.h | 3 +- block.c | 8 ++++-- block/qcow2.c | 72 ++++++++++++++++++++++++++++++---------------- qemu-img.c | 52 +++++++++++++++++++++++++++++---- util/qemu-option.c | 1 - tests/qemu-iotests/060.out | 4 +-- tests/qemu-iotests/061.out | 7 ----- tests/qemu-iotests/080.out | 4 +-- tests/qemu-iotests/082 | 9 ++++++ tests/qemu-iotests/082.out | 53 +++++++++++++++++++++++----------- tests/qemu-iotests/112.out | 3 -- tests/qemu-iotests/113 | 19 ++++++------ tests/qemu-iotests/113.out | 7 +++-- 14 files changed, 166 insertions(+), 79 deletions(-) -- 2.14.3