[libvirt] [PATCH V2 1/2] locking: relax PID requirement

Daniel P. Berrange berrange at redhat.com
Wed Apr 22 09:18:02 UTC 2015


On Tue, Apr 21, 2015 at 04:24:29PM -0600, Jim Fehlig wrote:
> Some hypervisors like Xen do not have PIDs associated with domains.
> Relax the requirement for PID != 0 in the locking code so it can
> be used by hypervisors that do not represent domains as a process
> running on the host.
> 
> Signed-off-by: Jim Fehlig <jfehlig at suse.com>
> ---
> 
> V2:
> Change check for ownerPid to ownerId in lock daemon dispatcher
> 
> Change instance of VIR_WARN to VIR_DEBUG to prevent polluting the
> logs with warnings when creating Xen domains.
> 
>  src/locking/lock_daemon.c          |  2 +-
>  src/locking/lock_daemon_dispatch.c | 16 ++++++++--------
>  src/locking/lock_driver_lockd.c    |  7 ++-----
>  3 files changed, 11 insertions(+), 14 deletions(-)

ACK


Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|




More information about the libvir-list mailing list