]> git.ipfire.org Git - thirdparty/libvirt.git/commit
qemu_hotplug.c: user-friendlier setvcpus timeout error message
authorDaniel Henrique Barboza <danielhb413@gmail.com>
Fri, 18 Oct 2019 18:36:34 +0000 (15:36 -0300)
committerCole Robinson <crobinso@redhat.com>
Wed, 13 Nov 2019 20:03:40 +0000 (15:03 -0500)
commitf46eb511a2b0f6d875167f14bba09f25f799a14d
tree1e00565047b89c131defc8bb222c2b131d1ee9a1
parent2fe78a833ee67f70228f9c992904147faab5cc1b
qemu_hotplug.c: user-friendlier setvcpus timeout error message

The current 'setvcpus' timeout message requires a deeper
understanding of QEMU/Libvirt internals to proper react to it.
One who knows how setvcpus unplug work (it is an asynchronous
operation between QEMU and guest that Libvirt can't know for
sure if it failed, unless an explicit error happened during the
timeout period) will read the message and not assume a failed
operation. But the regular user, most often than not, will read
it and believe that the unplug operation failed.

This leads to situations where the user isn't exactly relieved
when accessing the guest and seeing that the unplug operation
worked. Instead, the user feel mislead by the timeout message
setvcpus threw.

Changing the timeout message to let the user know that the
unplug status is not known, and manual inspection in the guest
is required, is not a silver bullet. But it gives a more
realistic expectation of what happened, as best as we can tell
from Libvirt side anyways.

Reviewed-by: Cole Robinson <crobinso@redhat.com>
Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
src/qemu/qemu_hotplug.c