diff options
| author | Poornima G <pgurusid@redhat.com> | 2016-05-10 01:50:08 -0400 |
|---|---|---|
| committer | Raghavendra G <rgowdapp@redhat.com> | 2016-05-15 22:18:42 -0700 |
| commit | 58c577994b2bf67b6f8d9749d9c7d678edfc9ceb (patch) | |
| tree | e914a8c0c8986f856b1bd3f592586c5677e6585c /xlators | |
| parent | 5daa68bbdd698bc3532e71593d024d161d796a64 (diff) | |
leases: Fix the notify being lost
The notify() implemented doesn't really add any value
and the EXIT_IF_LEASES_OFF in notify() can cause the
notifications to be lost if leases if off.
Hence remove the notify definition from leases,
also fix a log level
Change-Id: I8bf30d26f08034a16a450ed811dcf8ac57d70a94
BUG: 1319992
Signed-off-by: Poornima G <pgurusid@redhat.com>
Reviewed-on: http://review.gluster.org/14275
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Raghavendra Talur <rtalur@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Diffstat (limited to 'xlators')
| -rw-r--r-- | xlators/features/leases/src/leases.c | 18 |
1 files changed, 3 insertions, 15 deletions
diff --git a/xlators/features/leases/src/leases.c b/xlators/features/leases/src/leases.c index 8783a5ce9c2..2c1db34bd9c 100644 --- a/xlators/features/leases/src/leases.c +++ b/xlators/features/leases/src/leases.c @@ -991,11 +991,11 @@ init (xlator_t *this) priv->timer_wheel = glusterfs_global_timer_wheel (this); if (!priv->timer_wheel) { - gf_msg (this->name, GF_LOG_ERROR, 0, LEASE_MSG_NO_TIMER_WHEEL, - "Initing the global timer wheel"); + gf_msg_debug (this->name, 0, "Initing the global timer wheel"); ret = glusterfs_global_timer_wheel_init (this->ctx); if (ret) { - gf_msg (this->name, GF_LOG_INFO, 0, LEASE_MSG_NO_TIMER_WHEEL, + gf_msg (this->name, GF_LOG_ERROR, 0, + LEASE_MSG_NO_TIMER_WHEEL, "Initing the global timer wheel failed"); goto out; } @@ -1059,18 +1059,6 @@ out: return ret; } -int -notify (xlator_t *this, int32_t event, void *data, ...) -{ - int ret = 0; - - EXIT_IF_LEASES_OFF (this, out); - - ret = default_notify (this, event, data); -out: - return ret; -} - struct xlator_fops fops = { /* Metadata modifying fops */ .fsetattr = leases_fsetattr, |