[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] UpdateDevice: Allow startupPolicy update



On 05.11.2014 16:29, Peter Krempa wrote:
Add qemu: prefix to subject and mention that it's only with the _CONFIG
flag.

Fixed.


On 11/05/14 14:11, Michal Privoznik wrote:
Users might want to update startupPolicy via the
virDomainUpdateDeviceFlags API too. This patch
implements the feature on config layer.

Signed-off-by: Michal Privoznik <mprivozn redhat com>
---
  src/qemu/qemu_driver.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 6acaea8..6fc15c0 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -7437,6 +7437,7 @@ qemuDomainUpdateDeviceConfig(virQEMUCapsPtr qemuCaps,
          if (disk->src->format)
              orig->src->format = disk->src->format;
          disk->src->path = NULL;
+        orig->startupPolicy = disk->startupPolicy;

Looks on par with what we are doing now. I wanted to comment that if you
don't specify a startup policy you might want to keep the existing one,
but the code does not use this approach, thus ...

Well, in order to do that we would need to allow the following:

   startupPolicy='default'

to allow users to remove already set startupPolicy. But that's not supported currently, so I went the other way.


          break;

      case VIR_DOMAIN_DEVICE_NET:


ACK with the subject tweaked




Pushed now. Thanks.

Michal


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]