On 06/11/2012 08:29 AM, Michal Privoznik wrote:
Currently, we are passing only one boolean (migrated) so there is
no real profit in this. But it creates starting position for
next patch.
---
src/qemu/qemu_driver.c | 16 ++++++++--------
src/qemu/qemu_migration.c | 20 +++++++++++++-------
src/qemu/qemu_process.c | 22 ++++++++++++----------
src/qemu/qemu_process.h | 8 ++++++--
4 files changed, 39 insertions(+), 27 deletions(-)
ACK - fairly mechanical and straightforward.
@@ -3985,7 +3985,8 @@ void qemuProcessStop(struct qemud_driver
*driver,
/* Reset Security Labels */
virSecurityManagerRestoreAllLabel(driver->securityManager,
- vm->def, migrated);
+ vm->def,
+ flags & VIR_QEMU_PROCESS_STOP_MIGRATED);
It's a bit odd that virSecurityManagerRestoreAllLabel uses an int
instead of a bool, but that's independent of your patch.
--
Eric Blake eblake(a)redhat.com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org