On Thu, Jun 05, 2014 at 05:20:39PM +0800, Chunyan Liu wrote: > This patch series is to replace QEMUOptionParameter with QemuOpts, so that > only > one Qemu Option structure is kept in QEMU code. > > --- > Changes to v27: > * rebase to git master (code has been updated in this part since v27 sent > so rebase to make the patch series work.)
This series causes two qemu-iotests failures. For info on qemu-iotests: http://qemu-project.org/Documentation/QemuIoTests Luckily they look easy to fix, they are just differences in error messages. Please preserve error messages where possible, and otherwise just update the test .out file. When resolving these test failures, please make sure to keep bisectability by squashing the fix where the failure was introduced instead of adding new patches to the end of the series. 049 6s ... - output mismatch (see 049.out.bad) --- 049.out 2014-06-05 15:04:21.952358444 +0200 +++ 049.out.bad 2014-06-05 15:11:29.143000551 +0200 @@ -120,7 +120,7 @@ qemu-img create -f qcow2 -o size=foobar TEST_DIR/t.qcow2 qemu-img: Parameter 'size' expects a size -qemu-img: TEST_DIR/t.qcow2: Invalid options for file format 'qcow2' +qemu-img: TEST_DIR/t.qcow2: Invalid options for file format 'qcow2'. == Check correct interpretation of suffixes for cluster size == 061 7s ... - output mismatch (see 061.out.bad) --- 061.out 2014-06-05 15:04:21.952358444 +0200 +++ 061.out.bad 2014-06-05 15:12:08.024050567 +0200 @@ -281,7 +281,7 @@ qemu-img: Error while amending options: Invalid argument Unknown compatibility level 0.42. qemu-img: Error while amending options: Invalid argument -qemu-img: Invalid parameter 'foo' +Unknown option 'foo' qemu-img: Invalid options for file format 'qcow2' Changing the cluster size is not supported. qemu-img: Error while amending options: Operation not supported