Multiple xlators: perform gettimeofday() not under lock

While it may slightly reduce accuracy, I think it's better to
acquire the time outside a lock and then memcpy the value while
under lock.

Compile-tested only!

updates: bz#1193929
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>

Change-Id: Ic8fb94a56c4fa2d3b13e59767e13646fb39342ba

2 files changed, 6 insertions, 6 deletions

diff --git a/xlators/features/locks/src/entrylk.c b/xlators/features/locks/src/entrylk.c
index ea78f92d200..60e7bed2a44 100644
--- a/xlators/features/locks/src/entrylk.c
+++ b/xlators/features/locks/src/entrylk.c
@@ -115,7 +115,6 @@ __stale_entrylk(xlator_t *this, pl_entry_lock_t *candidate_lock,
 {
     posix_locks_private_t *priv = NULL;
     struct timeval curr;
-    gettimeofday(&curr, NULL);
 
     priv = this->private;
 
@@ -123,6 +122,7 @@ __stale_entrylk(xlator_t *this, pl_entry_lock_t *candidate_lock,
      * chance?  Or just the locks we are attempting to acquire?
      */
     if (names_conflict(candidate_lock->basename, requested_lock->basename)) {
+        gettimeofday(&curr, NULL);
         *lock_age_sec = curr.tv_sec - candidate_lock->granted_time.tv_sec;
         if (*lock_age_sec > priv->revocation_secs)
             return _gf_true;
@@ -541,11 +541,11 @@ __lock_blocked_add(xlator_t *this, pl_inode_t *pinode, pl_dom_list_t *dom,
 {
     struct timeval now;
 
-    gettimeofday(&now, NULL);
-
     if (nonblock)
         goto out;
 
+    gettimeofday(&now, NULL);
+
     lock->blkd_time = now;
     list_add_tail(&lock->blocked_locks, &dom->blocked_entrylks);
 
diff --git a/xlators/features/locks/src/inodelk.c b/xlators/features/locks/src/inodelk.c
index eff58a79569..b8e005038d8 100644
--- a/xlators/features/locks/src/inodelk.c
+++ b/xlators/features/locks/src/inodelk.c
@@ -145,11 +145,11 @@ __stale_inodelk(xlator_t *this, pl_inode_lock_t *candidate_lock,
     struct timeval curr;
 
     priv = this->private;
-    gettimeofday(&curr, NULL);
     /* Question: Should we just prune them all given the
      * chance?  Or just the locks we are attempting to acquire?
      */
     if (inodelk_conflict(candidate_lock, requested_lock)) {
+        gettimeofday(&curr, NULL);
         *lock_age_sec = curr.tv_sec - candidate_lock->granted_time.tv_sec;
         if (*lock_age_sec > priv->revocation_secs)
             return _gf_true;
@@ -403,12 +403,12 @@ __lock_blocked_add(xlator_t *this, pl_dom_list_t *dom, pl_inode_lock_t *lock,
 {
     struct timeval now;
 
-    gettimeofday(&now, NULL);
-
     if (can_block == 0) {
         goto out;
     }
 
+    gettimeofday(&now, NULL);
+
     lock->blkd_time = now;
     list_add_tail(&lock->blocked_locks, &dom->blocked_inodelks);