Skip to contentSkip to conversation

Adding a callback for incoming sm_notify.

PATCH New Venkateswararao Jujjuri 1 participant 1 message May 7, 2014

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Adding a callback for incoming sm_notify.

From: rong zeng <[email protected]>

1. "sm-notify" callout name
2. monitored client name as in the SM_NOTIFY request
3. IP of the sender of the SM_NOITFY request.
4. state value in the SM_NOTIFY request

This new interface can be used by different HA-NFS product in its specific configuration and environment to recover from the client crash and stale lock scenarios.

---
 utils/statd/callback.c |   13 ++++++++-----
 1 files changed, 8 insertions(+), 5 deletions(-)
utils/statd/callback.c +8 -0
index d1cc139..58ba70b 100644
@@ -15,6 +15,7 @@
 #include "rpcmisc.h"
 #include "statd.h"
 #include "notlist.h"
+#include "ha-callout.h"
 
 /* Callback notify list. */
 /* notify_list *cbnl = NULL; ... never used */
@@ -87,6 +88,13 @@ sm_notify_1_svc(struct stat_chge *argp, struct svc_req *rqstp)
          xlog(D_CALL, "Received SM_NOTIFY from %s, state: %d",
                                     argp->mon_name, argp->state);
 
+         if (!statd_present_address(sap, ip_addr, sizeof(ip_addr))) {
+                  xlog_warn("Unrecognized sender address");
+                  return ((void *) &result);
+         }
+
+         ha_callout("sm-notify", argp->mon_name, ip_addr, argp->state);
+
          /* quick check - don't bother if we're not monitoring anyone */
          if (rtnl == NULL) {
                   xlog_warn("SM_NOTIFY from %s while not monitoring 

any hosts",
@@ -94,11 +102,6 @@ sm_notify_1_svc(struct stat_chge *argp, struct svc_req *rqstp)

                   return ((void *) &result);
          }

- if (!statd_present_address(sap, ip_addr, sizeof(ip_addr))) {
- xlog_warn("Unrecognized sender address");
- return ((void *) &result);
- }
-
/* okir change: statd doesn't remove the remote host from its
* internal monitor list when receiving an SM_NOTIFY call from
* it. Lockd will want to continue monitoring the remote host

Signature · 1 line
1.7.1