Re: [PATCH 2/6] Make state driver device hotplug/update actually transient

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

 



On Mon, Nov 22, 2010 at 04:35:30PM -0500, Cole Robinson wrote:
> The current semantics of non-persistent hotplug/update are confusing: the
> changes will persist as long as the in memory domain definition isn't
> overwritten. This means hotplug changes stay around until the domain is
> redefined or libvirtd is restarted.
> 
> Call virDomainObjSetDefTransient at VM startup, so that we properly discard
> hotplug changes when the VM is shutdown.
> 
> Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
> ---
>  src/lxc/lxc_driver.c   |    3 +++
>  src/qemu/qemu_driver.c |    5 +++++
>  src/test/test_driver.c |   42 ++++++++++++++++++++++++------------------
>  src/uml/uml_driver.c   |    5 ++++-
>  4 files changed, 36 insertions(+), 19 deletions(-)

The downside is that we double the memory consumption for every
VM, regardless of whether we ever make any live config changes
to the VM. Calling SetDefTransient at time of change though
is probably too error prone.

ACK

Daniel

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]