Re: [libvirt] PATCH: 3/11: Remove unused virStateSigDispatcher() api

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

 



On Thu, Oct 30, 2008 at 01:36:45PM +0000, Daniel P. Berrange wrote:
> When we added the LXC driver, it originally needed to respond
> to signals. We removed that need a while ago, but left the
> infrastructure around. This patch removes the driver calls
> relating to signal progation, since they're unneccessary and

 uncontroversial +1

> we shouldn't follow this approach again in the future, because
> all drivers need to survive libvirtd restarts, which means they
> cannot rely in having signals from child processes.

So ultimately we could remove the signal part of the drivers
and maybe simplify the handling from qemudDispatchSignalEvent
Maybe we can extract the persistency mechanism and share it between
the wrapper processes then.

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
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]