]> git.ipfire.org Git - thirdparty/libvirt.git/commit
qemu: leave rerror policy at default when enospace is requested
authorLaine Stump <laine@laine.org>
Wed, 5 Oct 2011 15:19:28 +0000 (11:19 -0400)
committerLaine Stump <laine@laine.org>
Thu, 6 Oct 2011 18:49:13 +0000 (14:49 -0400)
commit91195b4321a1508139e36bcafc98201094af05de
tree38cc2c252551fa0a6306282d6a7113710b3c70bc
parent8644a379d7b22d0edaff854734c0b2c8fc5e52c6
qemu: leave rerror policy at default when enospace is requested

commit 12062ab set rerror=ignore when error_policy="enospace" was
selected (since the rerror option in qemu doesn't accept "enospc", as
the werror option does).

After that patch was already pushed, Paolo Bonzini noticed it and
commented that leaving rerror at the default ("report") would be a
better choice. This patch corrects the problem - if error_policy =
"enospace" is given, rerror is left off the qemu commandline,
effectively setting it to "report". For other values, rerror is still
set to match werror.

Additionally, the parsing of error_policy was changed to no longer
erroneously allow "default" as a choice - as with most other
attributes, if you want the default setting, just don't specify an
error_policy.

Finally, two ommissions in the first patch were corrected - a
long-dormant qemuxml2argv test for enospace was enabled, and fixed to
pass, and the argv2xml parser in qemu_command.c was updated to
recognize the different spelling on the qemu commandline.
src/conf/domain_conf.c
src/qemu/qemu_command.c
tests/qemuxml2argvdata/qemuxml2argv-disk-drive-error-policy-enospace.args
tests/qemuxml2argvtest.c