[PATCH 1/3] locking: relax PID requirement

[Jim Fehlig <jfehlig@xxxxxxxx>]

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@xxxxxxxx>
---
 src/locking/lock_daemon.c          |  2 +-
 src/locking/lock_daemon_dispatch.c | 49 +++++++++++---------------------------
 src/locking/lock_driver_lockd.c    |  7 ++----
 3 files changed, 17 insertions(+), 41 deletions(-)

diff --git a/src/locking/lock_daemon.c b/src/locking/lock_daemon.c
index bb165c0..042ff94 100644
--- a/src/locking/lock_daemon.c
+++ b/src/locking/lock_daemon.c
@@ -678,7 +678,7 @@ virLockDaemonClientFree(void *opaque)
                     signum = SIGKILL;
                 else
                     signum = 0;
-                if (virProcessKill(priv->clientPid, signum) < 0) {
+                if (priv->clientPid != 0 && virProcessKill(priv->clientPid, signum) < 0) {
                     if (errno == ESRCH)
                         break;
 
diff --git a/src/locking/lock_daemon_dispatch.c b/src/locking/lock_daemon_dispatch.c
index a7cee9d..f2704ec 100644
--- a/src/locking/lock_daemon_dispatch.c
+++ b/src/locking/lock_daemon_dispatch.c
@@ -62,11 +62,8 @@ virLockSpaceProtocolDispatchAcquireResource(virNetServerPtr server ATTRIBUTE_UNU
         goto cleanup;
     }
 
-    if (!priv->ownerPid) {
-        virReportError(VIR_ERR_OPERATION_INVALID, "%s",
-                       _("lock owner details have not been registered"));
-        goto cleanup;
-    }
+    if (!priv->ownerPid)
+        VIR_WARN("lock owner PID has not been registered");
 
     if (!(lockspace = virLockDaemonFindLockSpace(lockDaemon, args->path))) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
@@ -120,11 +117,8 @@ virLockSpaceProtocolDispatchCreateResource(virNetServerPtr server ATTRIBUTE_UNUS
         goto cleanup;
     }
 
-    if (!priv->ownerPid) {
-        virReportError(VIR_ERR_OPERATION_INVALID, "%s",
-                       _("lock owner details have not been registered"));
-        goto cleanup;
-    }
+    if (!priv->ownerPid)
+        VIR_WARN("lock owner PID has not been registered");
 
     if (!(lockspace = virLockDaemonFindLockSpace(lockDaemon, args->path))) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
@@ -169,11 +163,8 @@ virLockSpaceProtocolDispatchDeleteResource(virNetServerPtr server ATTRIBUTE_UNUS
         goto cleanup;
     }
 
-    if (!priv->ownerPid) {
-        virReportError(VIR_ERR_OPERATION_INVALID, "%s",
-                       _("lock owner details have not been registered"));
-        goto cleanup;
-    }
+    if (!priv->ownerPid)
+        VIR_WARN("lock owner PID has not been registered");
 
     if (!(lockspace = virLockDaemonFindLockSpace(lockDaemon, args->path))) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
@@ -218,11 +209,8 @@ virLockSpaceProtocolDispatchNew(virNetServerPtr server ATTRIBUTE_UNUSED,
         goto cleanup;
     }
 
-    if (!priv->ownerPid) {
-        virReportError(VIR_ERR_OPERATION_INVALID, "%s",
-                       _("lock owner details have not been registered"));
-        goto cleanup;
-    }
+    if (!priv->ownerPid)
+        VIR_WARN("lock owner PID has not been registered");
 
     if (!args->path || STREQ(args->path, "")) {
         virReportError(VIR_ERR_OPERATION_INVALID, "%s",
@@ -273,11 +261,8 @@ virLockSpaceProtocolDispatchRegister(virNetServerPtr server ATTRIBUTE_UNUSED,
         goto cleanup;
     }
 
-    if (priv->ownerPid) {
-        virReportError(VIR_ERR_OPERATION_INVALID, "%s",
-                       _("lock owner details have already been registered"));
-        goto cleanup;
-    }
+    if (priv->ownerPid)
+        VIR_WARN("lock owner PID has not been registered");
 
     if (VIR_STRDUP(priv->ownerName, args->owner.name) < 0)
         goto cleanup;
@@ -320,11 +305,8 @@ virLockSpaceProtocolDispatchReleaseResource(virNetServerPtr server ATTRIBUTE_UNU
         goto cleanup;
     }
 
-    if (!priv->ownerPid) {
-        virReportError(VIR_ERR_OPERATION_INVALID, "%s",
-                       _("lock owner details have not been registered"));
-        goto cleanup;
-    }
+    if (!priv->ownerPid)
+        VIR_WARN("lock owner PID has not been registered");
 
     if (!(lockspace = virLockDaemonFindLockSpace(lockDaemon, args->path))) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
@@ -370,11 +352,8 @@ virLockSpaceProtocolDispatchRestrict(virNetServerPtr server ATTRIBUTE_UNUSED,
         goto cleanup;
     }
 
-    if (!priv->ownerPid) {
-        virReportError(VIR_ERR_OPERATION_INVALID, "%s",
-                       _("lock owner details have not been registered"));
-        goto cleanup;
-    }
+    if (!priv->ownerPid)
+        VIR_WARN("lock owner PID has not been registered");
 
     priv->restricted = true;
     rv = 0;
diff --git a/src/locking/lock_driver_lockd.c b/src/locking/lock_driver_lockd.c
index 3a48a6a..109a79b 100644
--- a/src/locking/lock_driver_lockd.c
+++ b/src/locking/lock_driver_lockd.c
@@ -466,11 +466,8 @@ static int virLockManagerLockDaemonNew(virLockManagerPtr lock,
                            _("Missing ID parameter for domain object"));
             return -1;
         }
-        if (priv->pid == 0) {
-            virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
-                           _("Missing PID parameter for domain object"));
-            return -1;
-        }
+        if (priv->pid == 0)
+            VIR_WARN("Missing PID parameter for domain object");
         if (!priv->name) {
             virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                            _("Missing name parameter for domain object"));
-- 
1.8.4.5

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