diff options
Diffstat (limited to 'xlators/features/locks/src/common.c')
| -rw-r--r-- | xlators/features/locks/src/common.c | 43 | 
1 files changed, 21 insertions, 22 deletions
diff --git a/xlators/features/locks/src/common.c b/xlators/features/locks/src/common.c index f6b9c3315..0bc7baa30 100644 --- a/xlators/features/locks/src/common.c +++ b/xlators/features/locks/src/common.c @@ -143,9 +143,9 @@ __pl_inode_is_empty (pl_inode_t *pl_inode)  void  pl_print_locker (char *str, int size, xlator_t *this, call_frame_t *frame)  { -        snprintf (str, size, "Pid=%llu, lk-owner=%llu, Transport=%p, Frame=%llu", +        snprintf (str, size, "Pid=%llu, lk-owner=%s, Transport=%p, Frame=%llu",                    (unsigned long long) frame->root->pid, -                  (unsigned long long) frame->root->lk_owner, +                  lkowner_utoa (&frame->root->lk_owner),                    (void *)frame->root->trans,                    (unsigned long long) frame->root->unique);  } @@ -187,7 +187,7 @@ pl_print_lockee (char *str, int size, fd_t *fd, loc_t *loc)  void  pl_print_lock (char *str, int size, int cmd, -               struct gf_flock *flock, uint64_t owner) +               struct gf_flock *flock, gf_lkowner_t *owner)  {          char *cmd_str = NULL;          char *type_str = NULL; @@ -235,11 +235,11 @@ pl_print_lock (char *str, int size, int cmd,          }          snprintf (str, size, "lock=FCNTL, cmd=%s, type=%s, " -                  "start=%llu, len=%llu, pid=%llu, lk-owner=%llu", +                  "start=%llu, len=%llu, pid=%llu, lk-owner=%s",                    cmd_str, type_str, (unsigned long long) flock->l_start,                    (unsigned long long) flock->l_len,                    (unsigned long long) flock->l_pid, -                  (unsigned long long) owner); +                  lkowner_utoa (owner));  } @@ -262,7 +262,7 @@ pl_trace_in (xlator_t *this, call_frame_t *frame, fd_t *fd, loc_t *loc,          if (domain)                  pl_print_inodelk (pl_lock, 256, cmd, flock, domain);          else -                pl_print_lock (pl_lock, 256, cmd, flock, frame->root->lk_owner); +                pl_print_lock (pl_lock, 256, cmd, flock, &frame->root->lk_owner);          gf_log (this->name, GF_LOG_INFO,                  "[REQUEST] Locker = {%s} Lockee = {%s} Lock = {%s}", @@ -312,7 +312,7 @@ pl_trace_out (xlator_t *this, call_frame_t *frame, fd_t *fd, loc_t *loc,          if (domain)                  pl_print_inodelk (pl_lock, 256, cmd, flock, domain);          else -                pl_print_lock (pl_lock, 256, cmd, flock, frame->root->lk_owner); +                pl_print_lock (pl_lock, 256, cmd, flock, &frame->root->lk_owner);          pl_print_verdict (verdict, 32, op_ret, op_errno); @@ -342,7 +342,7 @@ pl_trace_block (xlator_t *this, call_frame_t *frame, fd_t *fd, loc_t *loc,          if (domain)                  pl_print_inodelk (pl_lock, 256, cmd, flock, domain);          else -                pl_print_lock (pl_lock, 256, cmd, flock, frame->root->lk_owner); +                pl_print_lock (pl_lock, 256, cmd, flock, &frame->root->lk_owner);          gf_log (this->name, GF_LOG_INFO,                  "[BLOCKED] Locker = {%s} Lockee = {%s} Lock = {%s}", @@ -468,7 +468,7 @@ out:  /* Create a new posix_lock_t */  posix_lock_t *  new_posix_lock (struct gf_flock *flock, void *transport, pid_t client_pid, -                uint64_t owner, fd_t *fd) +                gf_lkowner_t *owner, fd_t *fd)  {          posix_lock_t *lock = NULL; @@ -494,7 +494,7 @@ new_posix_lock (struct gf_flock *flock, void *transport, pid_t client_pid,          lock->fd_num     = fd_to_fdnum (fd);          lock->fd         = fd;          lock->client_pid = client_pid; -        lock->owner      = owner; +        lock->owner      = *owner;          INIT_LIST_HEAD (&lock->list); @@ -569,8 +569,8 @@ int  same_owner (posix_lock_t *l1, posix_lock_t *l2)  { -                return ((l1->owner == l2->owner) && -                        (l1->transport  == l2->transport)); +        return (is_same_lkowner (&l1->owner, &l2->owner) && +                (l1->transport  == l2->transport));  } @@ -889,10 +889,9 @@ __grant_blocked_locks (xlator_t *this, pl_inode_t *pl_inode, struct list_head *g                          posix_lock_to_flock (l, &conf->user_flock);                          gf_log (this->name, GF_LOG_TRACE, -                                "%s (pid=%d) lk-owner:%"PRIu64" %"PRId64" - %"PRId64" => Granted", +                                "%s (pid=%d) lk-owner:%s %"PRId64" - %"PRId64" => Granted",                                  l->fl_type == F_UNLCK ? "Unlock" : "Lock", -                                l->client_pid, -                                l->owner, +                                l->client_pid, lkowner_utoa (&l->owner),                                  l->user_flock.l_start,                                  l->user_flock.l_len); @@ -958,7 +957,7 @@ pl_send_prelock_unlock (xlator_t *this, pl_inode_t *pl_inode,          unlock_lock = new_posix_lock (&flock, old_lock->transport, -                                      old_lock->client_pid, old_lock->owner, +                                      old_lock->client_pid, &old_lock->owner,                                        old_lock->fd);          GF_VALIDATE_OR_GOTO (this->name, unlock_lock, out);          ret = 0; @@ -1011,19 +1010,19 @@ pl_setlk (xlator_t *this, pl_inode_t *pl_inode, posix_lock_t *lock,                  if (__is_lock_grantable (pl_inode, lock)) {                          gf_log (this->name, GF_LOG_TRACE, -                                "%s (pid=%d) lk-owner:%"PRIu64" %"PRId64" - %"PRId64" => OK", +                                "%s (pid=%d) lk-owner:%s %"PRId64" - %"PRId64" => OK",                                  lock->fl_type == F_UNLCK ? "Unlock" : "Lock",                                  lock->client_pid, -                                lock->owner, +                                lkowner_utoa (&lock->owner),                                  lock->user_flock.l_start,                                  lock->user_flock.l_len);                          __insert_and_merge (pl_inode, lock);                  } else if (can_block) {                          gf_log (this->name, GF_LOG_TRACE, -                                "%s (pid=%d) lk-owner:%"PRIu64" %"PRId64" - %"PRId64" => Blocked", +                                "%s (pid=%d) lk-owner:%s %"PRId64" - %"PRId64" => Blocked",                                  lock->fl_type == F_UNLCK ? "Unlock" : "Lock",                                  lock->client_pid, -                                lock->owner, +                                lkowner_utoa (&lock->owner),                                  lock->user_flock.l_start,                                  lock->user_flock.l_len);                          lock->blocked = 1; @@ -1031,10 +1030,10 @@ pl_setlk (xlator_t *this, pl_inode_t *pl_inode, posix_lock_t *lock,                          ret = -1;                  } else {                          gf_log (this->name, GF_LOG_TRACE, -                                "%s (pid=%d) lk-owner:%"PRIu64" %"PRId64" - %"PRId64" => NOK", +                                "%s (pid=%d) lk-owner:%s %"PRId64" - %"PRId64" => NOK",                                  lock->fl_type == F_UNLCK ? "Unlock" : "Lock",                                  lock->client_pid, -                                lock->owner, +                                lkowner_utoa (&lock->owner),                                  lock->user_flock.l_start,                                  lock->user_flock.l_len);                          errno = EAGAIN;  | 
