Whamcloud - gitweb
LU-4423 lustre: don't declare extern variables in C files.
[fs/lustre-release.git] / lustre / ldlm / ldlm_pool.c
index 4328274..3c518aa 100644 (file)
  *
  * You should have received a copy of the GNU General Public License
  * version 2 along with this program; If not, see
- * http://www.sun.com/software/products/lustre/docs/GPLv2.pdf
- *
- * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
- * CA 95054 USA or visit www.sun.com if you need additional information or
- * have any questions.
+ * http://www.gnu.org/licenses/gpl-2.0.html
  *
  * GPL HEADER END
  */
@@ -27,7 +23,7 @@
  * Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2010, 2015, Intel Corporation.
+ * Copyright (c) 2010, 2017, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
  * calculated as the number of locks in LRU * lock live time in seconds. If
  * CLV > SLV - lock is canceled.
  *
- * Client has LVF, that is, lock volume factor which regulates how much sensitive
- * client should be about last SLV from server. The higher LVF is the more locks
- * will be canceled on client. Default value for it is 1. Setting LVF to 2 means
- * that client will cancel locks 2 times faster.
+ * Client has LVF, that is, lock volume factor which regulates how much
+ * sensitive client should be about last SLV from server. The higher LVF is the
+ * more locks will be canceled on client. Default value for it is 1. Setting
+ * LVF to 2 means that client will cancel locks 2 times faster.
  *
  * Locks on a client will be canceled more intensively in these cases:
  * (1) if SLV is smaller, that is, load is higher on the server;
  * if flow is getting thinner, more and more particles become outside of it and
  * as particles are locks, they should be canceled.
  *
- * General idea of this belongs to Vitaly Fertman (vitaly@clusterfs.com). Andreas
- * Dilger (adilger@clusterfs.com) proposed few nice ideas like using LVF and many
- * cleanups. Flow definition to allow more easy understanding of the logic belongs
- * to Nikita Danilov (nikita@clusterfs.com) as well as many cleanups and fixes.
- * And design and implementation are done by Yury Umanets (umka@clusterfs.com).
+ * General idea of this belongs to Vitaly Fertman (vitaly@clusterfs.com).
+ * Andreas Dilger(adilger@clusterfs.com) proposed few nice ideas like using LVF
+ * and many cleanups. Flow definition to allow more easy understanding of the
+ * logic belongs to Nikita Danilov(nikita@clusterfs.com) as well as many
+ * cleanups and fixes. And design and implementation are done by Yury Umanets
+ * (umka@clusterfs.com).
  *
  * Glossary for terms used:
  *
  * pl_server_lock_volume - Current server lock volume (calculated);
  *
  * As it may be seen from list above, we have few possible tunables which may
- * affect behavior much. They all may be modified via proc. However, they also
+ * affect behavior much. They all may be modified via sysfs. However, they also
  * give a possibility for constructing few pre-defined behavior policies. If
  * none of predefines is suitable for a working pattern being used, new one may
- * be "constructed" via proc tunables.
+ * be "constructed" via sysfs tunables.
  */
 
 #define DEBUG_SUBSYSTEM S_LDLM
 
-#include <linux/kthread.h>
+#include <linux/workqueue.h>
+#include <libcfs/linux/linux-mem.h>
 #include <lustre_dlm.h>
 #include <cl_object.h>
 #include <obd_class.h>
 /*
  * 50 ldlm locks for 1MB of RAM.
  */
-#define LDLM_POOL_HOST_L ((NUM_CACHEPAGES >> (20 - PAGE_CACHE_SHIFT)) * 50)
+#define LDLM_POOL_HOST_L ((NUM_CACHEPAGES >> (20 - PAGE_SHIFT)) * 50)
 
 /*
  * Maximal possible grant step plan in %.
  */
 #define LDLM_POOL_SLV_SHIFT (10)
 
-extern struct proc_dir_entry *ldlm_ns_proc_dir;
-
 static inline __u64 dru(__u64 val, __u32 shift, int round_up)
 {
-        return (val + (round_up ? (1 << shift) - 1 : 0)) >> shift;
+       return (val + (round_up ? (1 << shift) - 1 : 0)) >> shift;
 }
 
 static inline __u64 ldlm_pool_slv_max(__u32 L)
 {
-        /*
-         * Allow to have all locks for 1 client for 10 hrs.
-         * Formula is the following: limit * 10h / 1 client.
-         */
-        __u64 lim = (__u64)L *  LDLM_POOL_MAX_AGE / 1;
-        return lim;
+       /*
+        * Allow to have all locks for 1 client for 10 hrs.
+        * Formula is the following: limit * 10h / 1 client.
+        */
+       __u64 lim = (__u64)L *  LDLM_POOL_MAX_AGE / 1;
+       return lim;
 }
 
 static inline __u64 ldlm_pool_slv_min(__u32 L)
 {
-        return 1;
+       return 1;
 }
 
 enum {
-        LDLM_POOL_FIRST_STAT = 0,
-        LDLM_POOL_GRANTED_STAT = LDLM_POOL_FIRST_STAT,
-        LDLM_POOL_GRANT_STAT,
-        LDLM_POOL_CANCEL_STAT,
-        LDLM_POOL_GRANT_RATE_STAT,
-        LDLM_POOL_CANCEL_RATE_STAT,
-        LDLM_POOL_GRANT_PLAN_STAT,
-        LDLM_POOL_SLV_STAT,
-        LDLM_POOL_SHRINK_REQTD_STAT,
-        LDLM_POOL_SHRINK_FREED_STAT,
-        LDLM_POOL_RECALC_STAT,
-        LDLM_POOL_TIMING_STAT,
-        LDLM_POOL_LAST_STAT
+       LDLM_POOL_FIRST_STAT = 0,
+       LDLM_POOL_GRANTED_STAT = LDLM_POOL_FIRST_STAT,
+       LDLM_POOL_GRANT_STAT,
+       LDLM_POOL_CANCEL_STAT,
+       LDLM_POOL_GRANT_RATE_STAT,
+       LDLM_POOL_CANCEL_RATE_STAT,
+       LDLM_POOL_GRANT_PLAN_STAT,
+       LDLM_POOL_SLV_STAT,
+       LDLM_POOL_SHRINK_REQTD_STAT,
+       LDLM_POOL_SHRINK_FREED_STAT,
+       LDLM_POOL_RECALC_STAT,
+       LDLM_POOL_TIMING_STAT,
+       LDLM_POOL_LAST_STAT
 };
 
 static inline struct ldlm_namespace *ldlm_pl2ns(struct ldlm_pool *pl)
 {
-        return container_of(pl, struct ldlm_namespace, ns_pool);
+       return container_of(pl, struct ldlm_namespace, ns_pool);
 }
 
 /**
@@ -191,29 +187,29 @@ static inline struct ldlm_namespace *ldlm_pl2ns(struct ldlm_pool *pl)
  */
 static inline int ldlm_pool_t2gsp(unsigned int t)
 {
-        /*
-         * This yields 1% grant step for anything below LDLM_POOL_GSP_STEP
-         * and up to 30% for anything higher than LDLM_POOL_GSP_STEP.
-         *
-         * How this will affect execution is the following:
-         *
-         * - for thread period 1s we will have grant_step 1% which good from
-         * pov of taking some load off from server and push it out to clients.
-         * This is like that because 1% for grant_step means that server will
-         * not allow clients to get lots of locks in short period of time and
-         * keep all old locks in their caches. Clients will always have to
-         * get some locks back if they want to take some new;
-         *
-         * - for thread period 10s (which is default) we will have 23% which
-         * means that clients will have enough of room to take some new locks
-         * without getting some back. All locks from this 23% which were not
-         * taken by clients in current period will contribute in SLV growing.
-         * SLV growing means more locks cached on clients until limit or grant
-         * plan is reached.
-         */
-        return LDLM_POOL_MAX_GSP -
-                ((LDLM_POOL_MAX_GSP - LDLM_POOL_MIN_GSP) >>
-                 (t >> LDLM_POOL_GSP_STEP_SHIFT));
+       /*
+        * This yields 1% grant step for anything below LDLM_POOL_GSP_STEP
+        * and up to 30% for anything higher than LDLM_POOL_GSP_STEP.
+        *
+        * How this will affect execution is the following:
+        *
+        * - for thread period 1s we will have grant_step 1% which good from
+        * pov of taking some load off from server and push it out to clients.
+        * This is like that because 1% for grant_step means that server will
+        * not allow clients to get lots of locks in short period of time and
+        * keep all old locks in their caches. Clients will always have to
+        * get some locks back if they want to take some new;
+        *
+        * - for thread period 10s (which is default) we will have 23% which
+        * means that clients will have enough of room to take some new locks
+        * without getting some back. All locks from this 23% which were not
+        * taken by clients in current period will contribute in SLV growing.
+        * SLV growing means more locks cached on clients until limit or grant
+        * plan is reached.
+        */
+       return LDLM_POOL_MAX_GSP -
+               ((LDLM_POOL_MAX_GSP - LDLM_POOL_MIN_GSP) >>
+                (t >> LDLM_POOL_GSP_STEP_SHIFT));
 }
 
 static inline int ldlm_pool_granted(struct ldlm_pool *pl)
@@ -248,13 +244,13 @@ static void ldlm_pool_recalc_grant_plan(struct ldlm_pool *pl)
  */
 static void ldlm_pool_recalc_slv(struct ldlm_pool *pl)
 {
-        int granted;
-        int grant_plan;
-        int round_up;
-        __u64 slv;
-        __u64 slv_factor;
-        __u64 grant_usage;
-        __u32 limit;
+       int granted;
+       int grant_plan;
+       int round_up;
+       __u64 slv;
+       __u64 slv_factor;
+       __u64 grant_usage;
+       __u32 limit;
 
        slv = pl->pl_server_lock_volume;
        grant_plan = pl->pl_grant_plan;
@@ -262,28 +258,27 @@ static void ldlm_pool_recalc_slv(struct ldlm_pool *pl)
        granted = ldlm_pool_granted(pl);
        round_up = granted < limit;
 
-        grant_usage = max_t(int, limit - (granted - grant_plan), 1);
+       grant_usage = max_t(int, limit - (granted - grant_plan), 1);
 
-        /*
-         * Find out SLV change factor which is the ratio of grant usage
-         * from limit. SLV changes as fast as the ratio of grant plan
-         * consumption. The more locks from grant plan are not consumed
-         * by clients in last interval (idle time), the faster grows
-         * SLV. And the opposite, the more grant plan is over-consumed
-         * (load time) the faster drops SLV.
-         */
-        slv_factor = (grant_usage << LDLM_POOL_SLV_SHIFT);
-        do_div(slv_factor, limit);
-        slv = slv * slv_factor;
-        slv = dru(slv, LDLM_POOL_SLV_SHIFT, round_up);
+       /*
+        * Find out SLV change factor which is the ratio of grant usage
+        * from limit. SLV changes as fast as the ratio of grant plan
+        * consumption. The more locks from grant plan are not consumed
+        * by clients in last interval (idle time), the faster grows
+        * SLV. And the opposite, the more grant plan is over-consumed
+        * (load time) the faster drops SLV.
+        */
+       slv_factor = (grant_usage << LDLM_POOL_SLV_SHIFT);
+       do_div(slv_factor, limit);
+       slv = slv * slv_factor;
+       slv = dru(slv, LDLM_POOL_SLV_SHIFT, round_up);
 
-        if (slv > ldlm_pool_slv_max(limit)) {
-                slv = ldlm_pool_slv_max(limit);
-        } else if (slv < ldlm_pool_slv_min(limit)) {
-                slv = ldlm_pool_slv_min(limit);
-        }
+       if (slv > ldlm_pool_slv_max(limit))
+               slv = ldlm_pool_slv_max(limit);
+       else if (slv < ldlm_pool_slv_min(limit))
+               slv = ldlm_pool_slv_min(limit);
 
-        pl->pl_server_lock_volume = slv;
+       pl->pl_server_lock_volume = slv;
 }
 
 /**
@@ -316,19 +311,19 @@ static void ldlm_pool_recalc_stats(struct ldlm_pool *pl)
  */
 static void ldlm_srv_pool_push_slv(struct ldlm_pool *pl)
 {
-        struct obd_device *obd;
-
-        /*
-         * Set new SLV in obd field for using it later without accessing the
-         * pool. This is required to avoid race between sending reply to client
-         * with new SLV and cleanup server stack in which we can't guarantee
-         * that namespace is still alive. We know only that obd is alive as
-         * long as valid export is alive.
-         */
-        obd = ldlm_pl2ns(pl)->ns_obd;
-        LASSERT(obd != NULL);
+       struct obd_device *obd;
+
+       /*
+        * Set new SLV in obd field for using it later without accessing the
+        * pool. This is required to avoid race between sending reply to client
+        * with new SLV and cleanup server stack in which we can't guarantee
+        * that namespace is still alive. We know only that obd is alive as
+        * long as valid export is alive.
+        */
+       obd = ldlm_pl2ns(pl)->ns_obd;
+       LASSERT(obd != NULL);
        write_lock(&obd->obd_pool_lock);
-        obd->obd_pool_slv = pl->pl_server_lock_volume;
+       obd->obd_pool_slv = pl->pl_server_lock_volume;
        write_unlock(&obd->obd_pool_lock);
 }
 
@@ -339,38 +334,39 @@ static void ldlm_srv_pool_push_slv(struct ldlm_pool *pl)
  */
 static int ldlm_srv_pool_recalc(struct ldlm_pool *pl)
 {
-        time_t recalc_interval_sec;
-        ENTRY;
+       time64_t recalc_interval_sec;
+
+       ENTRY;
 
-        recalc_interval_sec = cfs_time_current_sec() - pl->pl_recalc_time;
-        if (recalc_interval_sec < pl->pl_recalc_period)
-                RETURN(0);
+       recalc_interval_sec = ktime_get_real_seconds() - pl->pl_recalc_time;
+       if (recalc_interval_sec < pl->pl_recalc_period)
+               RETURN(0);
 
        spin_lock(&pl->pl_lock);
-       recalc_interval_sec = cfs_time_current_sec() - pl->pl_recalc_time;
+       recalc_interval_sec = ktime_get_real_seconds() - pl->pl_recalc_time;
        if (recalc_interval_sec < pl->pl_recalc_period) {
                spin_unlock(&pl->pl_lock);
                RETURN(0);
        }
-        /*
-         * Recalc SLV after last period. This should be done
-         * _before_ recalculating new grant plan.
-         */
-        ldlm_pool_recalc_slv(pl);
-
-        /*
-         * Make sure that pool informed obd of last SLV changes.
-         */
-        ldlm_srv_pool_push_slv(pl);
-
-        /*
-         * Update grant_plan for new period.
-         */
-        ldlm_pool_recalc_grant_plan(pl);
-
-        pl->pl_recalc_time = cfs_time_current_sec();
-        lprocfs_counter_add(pl->pl_stats, LDLM_POOL_TIMING_STAT,
-                            recalc_interval_sec);
+       /*
+        * Recalc SLV after last period. This should be done
+        * _before_ recalculating new grant plan.
+        */
+       ldlm_pool_recalc_slv(pl);
+
+       /*
+        * Make sure that pool informed obd of last SLV changes.
+        */
+       ldlm_srv_pool_push_slv(pl);
+
+       /*
+        * Update grant_plan for new period.
+        */
+       ldlm_pool_recalc_grant_plan(pl);
+
+       pl->pl_recalc_time = ktime_get_real_seconds();
+       lprocfs_counter_add(pl->pl_stats, LDLM_POOL_TIMING_STAT,
+                           recalc_interval_sec);
        spin_unlock(&pl->pl_lock);
        RETURN(0);
 }
@@ -403,29 +399,29 @@ static int ldlm_srv_pool_shrink(struct ldlm_pool *pl,
 
        spin_lock(&pl->pl_lock);
 
-        /*
-         * We want shrinker to possibly cause cancellation of @nr locks from
-         * clients or grant approximately @nr locks smaller next intervals.
-         *
-         * This is why we decreased SLV by @nr. This effect will only be as
-         * long as one re-calc interval (1s these days) and this should be
-         * enough to pass this decreased SLV to all clients. On next recalc
-         * interval pool will either increase SLV if locks load is not high
-         * or will keep on same level or even decrease again, thus, shrinker
-         * decreased SLV will affect next recalc intervals and this way will
-         * make locking load lower.
-         */
-        if (nr < pl->pl_server_lock_volume) {
-                pl->pl_server_lock_volume = pl->pl_server_lock_volume - nr;
-        } else {
-                limit = ldlm_pool_get_limit(pl);
-                pl->pl_server_lock_volume = ldlm_pool_slv_min(limit);
-        }
-
-        /*
-         * Make sure that pool informed obd of last SLV changes.
-         */
-        ldlm_srv_pool_push_slv(pl);
+       /*
+        * We want shrinker to possibly cause cancellation of @nr locks from
+        * clients or grant approximately @nr locks smaller next intervals.
+        *
+        * This is why we decreased SLV by @nr. This effect will only be as
+        * long as one re-calc interval (1s these days) and this should be
+        * enough to pass this decreased SLV to all clients. On next recalc
+        * interval pool will either increase SLV if locks load is not high
+        * or will keep on same level or even decrease again, thus, shrinker
+        * decreased SLV will affect next recalc intervals and this way will
+        * make locking load lower.
+        */
+       if (nr < pl->pl_server_lock_volume) {
+               pl->pl_server_lock_volume = pl->pl_server_lock_volume - nr;
+       } else {
+               limit = ldlm_pool_get_limit(pl);
+               pl->pl_server_lock_volume = ldlm_pool_slv_min(limit);
+       }
+
+       /*
+        * Make sure that pool informed obd of last SLV changes.
+        */
+       ldlm_srv_pool_push_slv(pl);
        spin_unlock(&pl->pl_lock);
 
        /*
@@ -440,17 +436,17 @@ static int ldlm_srv_pool_shrink(struct ldlm_pool *pl,
  */
 static int ldlm_srv_pool_setup(struct ldlm_pool *pl, int limit)
 {
-        struct obd_device *obd;
+       struct obd_device *obd;
 
-        obd = ldlm_pl2ns(pl)->ns_obd;
-        LASSERT(obd != NULL && obd != LP_POISON);
-        LASSERT(obd->obd_type != LP_POISON);
+       obd = ldlm_pl2ns(pl)->ns_obd;
+       LASSERT(obd != NULL && obd != LP_POISON);
+       LASSERT(obd->obd_type != LP_POISON);
        write_lock(&obd->obd_pool_lock);
-        obd->obd_pool_limit = limit;
+       obd->obd_pool_limit = limit;
        write_unlock(&obd->obd_pool_lock);
 
-        ldlm_pool_set_limit(pl, limit);
-        return 0;
+       ldlm_pool_set_limit(pl, limit);
+       return 0;
 }
 
 /**
@@ -458,17 +454,17 @@ static int ldlm_srv_pool_setup(struct ldlm_pool *pl, int limit)
  */
 static void ldlm_cli_pool_pop_slv(struct ldlm_pool *pl)
 {
-        struct obd_device *obd;
+       struct obd_device *obd;
 
-        /*
-         * Get new SLV and Limit from obd which is updated with coming
-         * RPCs.
-         */
-        obd = ldlm_pl2ns(pl)->ns_obd;
-        LASSERT(obd != NULL);
+       /*
+        * Get new SLV and Limit from obd which is updated with coming
+        * RPCs.
+        */
+       obd = ldlm_pl2ns(pl)->ns_obd;
+       LASSERT(obd != NULL);
        read_lock(&obd->obd_pool_lock);
-        pl->pl_server_lock_volume = obd->obd_pool_slv;
-        ldlm_pool_set_limit(pl, obd->obd_pool_limit);
+       pl->pl_server_lock_volume = obd->obd_pool_slv;
+       ldlm_pool_set_limit(pl, obd->obd_pool_limit);
        read_unlock(&obd->obd_pool_lock);
 }
 
@@ -477,42 +473,43 @@ static void ldlm_cli_pool_pop_slv(struct ldlm_pool *pl)
  */
 static int ldlm_cli_pool_recalc(struct ldlm_pool *pl)
 {
-        time_t recalc_interval_sec;
+       time64_t recalc_interval_sec;
        int ret;
-        ENTRY;
 
-        recalc_interval_sec = cfs_time_current_sec() - pl->pl_recalc_time;
-        if (recalc_interval_sec < pl->pl_recalc_period)
-                RETURN(0);
+       ENTRY;
+
+       recalc_interval_sec = ktime_get_real_seconds() - pl->pl_recalc_time;
+       if (recalc_interval_sec < pl->pl_recalc_period)
+               RETURN(0);
 
        spin_lock(&pl->pl_lock);
        /*
         * Check if we need to recalc lists now.
         */
-       recalc_interval_sec = cfs_time_current_sec() - pl->pl_recalc_time;
+       recalc_interval_sec = ktime_get_real_seconds() - pl->pl_recalc_time;
        if (recalc_interval_sec < pl->pl_recalc_period) {
                spin_unlock(&pl->pl_lock);
-                RETURN(0);
-        }
+               RETURN(0);
+       }
 
-        /*
-         * Make sure that pool knows last SLV and Limit from obd.
-         */
-        ldlm_cli_pool_pop_slv(pl);
+       /*
+        * Make sure that pool knows last SLV and Limit from obd.
+        */
+       ldlm_cli_pool_pop_slv(pl);
        spin_unlock(&pl->pl_lock);
 
-        /*
-         * Do not cancel locks in case lru resize is disabled for this ns.
-         */
-        if (!ns_connect_lru_resize(ldlm_pl2ns(pl)))
+       /*
+        * Do not cancel locks in case lru resize is disabled for this ns.
+        */
+       if (!ns_connect_lru_resize(ldlm_pl2ns(pl)))
                GOTO(out, ret = 0);
 
-        /*
-         * In the time of canceling locks on client we do not need to maintain
-         * sharp timing, we only want to cancel locks asap according to new SLV.
-         * It may be called when SLV has changed much, this is why we do not
-         * take into account pl->pl_recalc_time here.
-         */
+       /*
+        * In the time of canceling locks on client we do not need to maintain
+        * sharp timing, we only want to cancel locks asap according to new SLV.
+        * It may be called when SLV has changed much, this is why we do not
+        * take into account pl->pl_recalc_time here.
+        */
        ret = ldlm_cancel_lru(ldlm_pl2ns(pl), 0, LCF_ASYNC,
                              LDLM_LRU_FLAG_LRUR);
 
@@ -522,7 +519,7 @@ out:
         * Time of LRU resizing might be longer than period,
         * so update after LRU resizing rather than before it.
         */
-       pl->pl_recalc_time = cfs_time_current_sec();
+       pl->pl_recalc_time = ktime_get_real_seconds();
        lprocfs_counter_add(pl->pl_stats, LDLM_POOL_TIMING_STAT,
                            recalc_interval_sec);
        spin_unlock(&pl->pl_lock);
@@ -537,21 +534,21 @@ out:
 static int ldlm_cli_pool_shrink(struct ldlm_pool *pl,
                                int nr, gfp_t gfp_mask)
 {
-        struct ldlm_namespace *ns;
+       struct ldlm_namespace *ns;
        int unused;
 
-        ns = ldlm_pl2ns(pl);
+       ns = ldlm_pl2ns(pl);
 
-        /*
-         * Do not cancel locks in case lru resize is disabled for this ns.
-         */
-        if (!ns_connect_lru_resize(ns))
-                RETURN(0);
+       /*
+        * Do not cancel locks in case lru resize is disabled for this ns.
+        */
+       if (!ns_connect_lru_resize(ns))
+               RETURN(0);
 
-        /*
-         * Make sure that pool knows last SLV and Limit from obd.
-         */
-        ldlm_cli_pool_pop_slv(pl);
+       /*
+        * Make sure that pool knows last SLV and Limit from obd.
+        */
+       ldlm_cli_pool_pop_slv(pl);
 
        spin_lock(&ns->ns_lock);
        unused = ns->ns_nr_unused;
@@ -564,29 +561,29 @@ static int ldlm_cli_pool_shrink(struct ldlm_pool *pl,
 }
 
 static struct ldlm_pool_ops ldlm_srv_pool_ops = {
-        .po_recalc = ldlm_srv_pool_recalc,
-        .po_shrink = ldlm_srv_pool_shrink,
-        .po_setup  = ldlm_srv_pool_setup
+       .po_recalc = ldlm_srv_pool_recalc,
+       .po_shrink = ldlm_srv_pool_shrink,
+       .po_setup  = ldlm_srv_pool_setup
 };
 
 static struct ldlm_pool_ops ldlm_cli_pool_ops = {
-        .po_recalc = ldlm_cli_pool_recalc,
-        .po_shrink = ldlm_cli_pool_shrink
+       .po_recalc = ldlm_cli_pool_recalc,
+       .po_shrink = ldlm_cli_pool_shrink
 };
 
 /**
  * Pool recalc wrapper. Will call either client or server pool recalc callback
  * depending what pool \a pl is used.
  */
-int ldlm_pool_recalc(struct ldlm_pool *pl)
+time64_t ldlm_pool_recalc(struct ldlm_pool *pl)
 {
-       time_t recalc_interval_sec;
+       time64_t recalc_interval_sec;
        int count;
 
-       recalc_interval_sec = cfs_time_current_sec() - pl->pl_recalc_time;
+       recalc_interval_sec = ktime_get_real_seconds() - pl->pl_recalc_time;
        if (recalc_interval_sec > 0) {
                spin_lock(&pl->pl_lock);
-               recalc_interval_sec = cfs_time_current_sec() -
+               recalc_interval_sec = ktime_get_real_seconds() -
                        pl->pl_recalc_time;
 
                if (recalc_interval_sec > 0) {
@@ -610,14 +607,13 @@ int ldlm_pool_recalc(struct ldlm_pool *pl)
                                    count);
        }
 
-       recalc_interval_sec = pl->pl_recalc_time - cfs_time_current_sec() +
+       recalc_interval_sec = pl->pl_recalc_time - ktime_get_real_seconds() +
                              pl->pl_recalc_period;
        if (recalc_interval_sec <= 0) {
                /* DEBUG: should be re-removed after LU-4536 is fixed */
-               CDEBUG(D_DLMTRACE, "%s: Negative interval(%ld), "
-                      "too short period(%ld)\n",
+               CDEBUG(D_DLMTRACE, "%s: Negative interval(%lld), too short period(%lld)\n",
                       pl->pl_name, recalc_interval_sec,
-                      pl->pl_recalc_period);
+                      (s64)pl->pl_recalc_period);
 
                /* Prevent too frequent recalculation. */
                recalc_interval_sec = 1;
@@ -632,22 +628,23 @@ int ldlm_pool_recalc(struct ldlm_pool *pl)
  */
 int ldlm_pool_shrink(struct ldlm_pool *pl, int nr, gfp_t gfp_mask)
 {
-        int cancel = 0;
-
-        if (pl->pl_ops->po_shrink != NULL) {
-                cancel = pl->pl_ops->po_shrink(pl, nr, gfp_mask);
-                if (nr > 0) {
-                        lprocfs_counter_add(pl->pl_stats,
-                                            LDLM_POOL_SHRINK_REQTD_STAT,
-                                            nr);
-                        lprocfs_counter_add(pl->pl_stats,
-                                            LDLM_POOL_SHRINK_FREED_STAT,
-                                            cancel);
-                        CDEBUG(D_DLMTRACE, "%s: request to shrink %d locks, "
-                               "shrunk %d\n", pl->pl_name, nr, cancel);
-                }
-        }
-        return cancel;
+       int cancel = 0;
+
+       if (pl->pl_ops->po_shrink != NULL) {
+               cancel = pl->pl_ops->po_shrink(pl, nr, gfp_mask);
+               if (nr > 0) {
+                       lprocfs_counter_add(pl->pl_stats,
+                                           LDLM_POOL_SHRINK_REQTD_STAT,
+                                           nr);
+                       lprocfs_counter_add(pl->pl_stats,
+                                           LDLM_POOL_SHRINK_FREED_STAT,
+                                           cancel);
+                       CDEBUG(D_DLMTRACE,
+                              "%s: request to shrink %d locks, shrunk %d\n",
+                              pl->pl_name, nr, cancel);
+               }
+       }
+       return cancel;
 }
 
 /**
@@ -658,9 +655,9 @@ int ldlm_pool_shrink(struct ldlm_pool *pl, int nr, gfp_t gfp_mask)
  */
 int ldlm_pool_setup(struct ldlm_pool *pl, int limit)
 {
-        if (pl->pl_ops->po_setup != NULL)
-                return(pl->pl_ops->po_setup(pl, limit));
-        return 0;
+       if (pl->pl_ops->po_setup != NULL)
+               return pl->pl_ops->po_setup(pl, limit);
+       return 0;
 }
 
 static int lprocfs_pool_state_seq_show(struct seq_file *m, void *unused)
@@ -700,163 +697,195 @@ static int lprocfs_pool_state_seq_show(struct seq_file *m, void *unused)
                   granted, limit);
        return 0;
 }
-LPROC_SEQ_FOPS_RO(lprocfs_pool_state);
 
-static int lprocfs_grant_speed_seq_show(struct seq_file *m, void *unused)
+LDEBUGFS_SEQ_FOPS_RO(lprocfs_pool_state);
+
+static ssize_t grant_speed_show(struct kobject *kobj, struct attribute *attr,
+                               char *buf)
 {
-       struct ldlm_pool *pl = m->private;
-       int               grant_speed;
+       struct ldlm_pool *pl = container_of(kobj, struct ldlm_pool,
+                                           pl_kobj);
+       int grant_speed;
 
        spin_lock(&pl->pl_lock);
        /* serialize with ldlm_pool_recalc */
        grant_speed = atomic_read(&pl->pl_grant_rate) -
                        atomic_read(&pl->pl_cancel_rate);
        spin_unlock(&pl->pl_lock);
-       return lprocfs_uint_seq_show(m, &grant_speed);
+       return sprintf(buf, "%d\n", grant_speed);
 }
+LUSTRE_RO_ATTR(grant_speed);
 
-LDLM_POOL_PROC_READER_SEQ_SHOW(grant_plan, int);
-LPROC_SEQ_FOPS_RO(lprocfs_grant_plan);
+LDLM_POOL_SYSFS_READER_SHOW(grant_plan, int);
+LUSTRE_RO_ATTR(grant_plan);
 
-LDLM_POOL_PROC_READER_SEQ_SHOW(recalc_period, int);
-LDLM_POOL_PROC_WRITER(recalc_period, int);
-static ssize_t lprocfs_recalc_period_seq_write(struct file *file,
-                                              const char __user *buf,
-                                              size_t len, loff_t *off)
-{
-       struct seq_file *seq = file->private_data;
+LDLM_POOL_SYSFS_READER_SHOW(recalc_period, int);
+LDLM_POOL_SYSFS_WRITER_STORE(recalc_period, int);
+LUSTRE_RW_ATTR(recalc_period);
+
+LDLM_POOL_SYSFS_READER_NOLOCK_SHOW(server_lock_volume, u64);
+LUSTRE_RO_ATTR(server_lock_volume);
+
+LDLM_POOL_SYSFS_READER_NOLOCK_SHOW(limit, atomic);
+LDLM_POOL_SYSFS_WRITER_NOLOCK_STORE(limit, atomic);
+LUSTRE_RW_ATTR(limit);
+
+LDLM_POOL_SYSFS_READER_NOLOCK_SHOW(granted, atomic);
+LUSTRE_RO_ATTR(granted);
+
+LDLM_POOL_SYSFS_READER_NOLOCK_SHOW(cancel_rate, atomic);
+LUSTRE_RO_ATTR(cancel_rate);
 
-       return lprocfs_wr_recalc_period(file, buf, len, seq->private);
+LDLM_POOL_SYSFS_READER_NOLOCK_SHOW(grant_rate, atomic);
+LUSTRE_RO_ATTR(grant_rate);
+
+LDLM_POOL_SYSFS_READER_NOLOCK_SHOW(lock_volume_factor, atomic);
+LDLM_POOL_SYSFS_WRITER_NOLOCK_STORE(lock_volume_factor, atomic);
+LUSTRE_RW_ATTR(lock_volume_factor);
+
+/* These are for pools in /sys/fs/lustre/ldlm/namespaces/.../pool */
+static struct attribute *ldlm_pl_attrs[] = {
+       &lustre_attr_grant_speed.attr,
+       &lustre_attr_grant_plan.attr,
+       &lustre_attr_recalc_period.attr,
+       &lustre_attr_server_lock_volume.attr,
+       &lustre_attr_limit.attr,
+       &lustre_attr_granted.attr,
+       &lustre_attr_cancel_rate.attr,
+       &lustre_attr_grant_rate.attr,
+       &lustre_attr_lock_volume_factor.attr,
+       NULL,
+};
+
+static void ldlm_pl_release(struct kobject *kobj)
+{
+       struct ldlm_pool *pl = container_of(kobj, struct ldlm_pool,
+                                           pl_kobj);
+       complete(&pl->pl_kobj_unregister);
 }
-LPROC_SEQ_FOPS(lprocfs_recalc_period);
 
-LPROC_SEQ_FOPS_RO_TYPE(ldlm_pool, u64);
-LPROC_SEQ_FOPS_RO_TYPE(ldlm_pool, atomic);
-LPROC_SEQ_FOPS_RW_TYPE(ldlm_pool_rw, atomic);
+static struct kobj_type ldlm_pl_ktype = {
+       .default_attrs  = ldlm_pl_attrs,
+       .sysfs_ops      = &lustre_sysfs_ops,
+       .release        = ldlm_pl_release,
+};
 
-LPROC_SEQ_FOPS_RO(lprocfs_grant_speed);
+static int ldlm_pool_sysfs_init(struct ldlm_pool *pl)
+{
+       struct ldlm_namespace *ns = ldlm_pl2ns(pl);
+       int err;
+
+       init_completion(&pl->pl_kobj_unregister);
+       err = kobject_init_and_add(&pl->pl_kobj, &ldlm_pl_ktype, &ns->ns_kobj,
+                                  "pool");
 
-static int ldlm_pool_proc_init(struct ldlm_pool *pl)
+       return err;
+}
+
+static int ldlm_pool_debugfs_init(struct ldlm_pool *pl)
 {
        struct ldlm_namespace *ns = ldlm_pl2ns(pl);
-       struct proc_dir_entry *parent_ns_proc;
+       struct dentry *debugfs_ns_parent;
        struct lprocfs_vars pool_vars[2];
-       char *var_name = NULL;
        int rc = 0;
-       ENTRY;
 
-       OBD_ALLOC(var_name, MAX_STRING_SIZE + 1);
-       if (!var_name)
-               RETURN(-ENOMEM);
+       ENTRY;
 
-       parent_ns_proc = ns->ns_proc_dir_entry;
-       if (parent_ns_proc == NULL) {
-               CERROR("%s: proc entry is not initialized\n",
+       debugfs_ns_parent = ns->ns_debugfs_entry;
+       if (IS_ERR_OR_NULL(debugfs_ns_parent)) {
+               CERROR("%s: debugfs entry is not initialized\n",
                       ldlm_ns_name(ns));
-               GOTO(out_free_name, rc = -EINVAL);
+               GOTO(out, rc = -EINVAL);
        }
-       pl->pl_proc_dir = lprocfs_register("pool", parent_ns_proc,
-                                          NULL, NULL);
-       if (IS_ERR(pl->pl_proc_dir)) {
-               rc = PTR_ERR(pl->pl_proc_dir);
-               pl->pl_proc_dir = NULL;
-               CERROR("%s: cannot create 'pool' proc entry: rc = %d\n",
+       pl->pl_debugfs_entry = ldebugfs_register("pool", debugfs_ns_parent,
+                                                NULL, NULL);
+       if (IS_ERR(pl->pl_debugfs_entry)) {
+               rc = PTR_ERR(pl->pl_debugfs_entry);
+               pl->pl_debugfs_entry = NULL;
+               CERROR("%s: cannot create 'pool' debugfs entry: rc = %d\n",
                       ldlm_ns_name(ns), rc);
-               GOTO(out_free_name, rc);
+               GOTO(out, rc);
        }
 
-       var_name[MAX_STRING_SIZE] = '\0';
        memset(pool_vars, 0, sizeof(pool_vars));
-       pool_vars[0].name = var_name;
-
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "server_lock_volume",
-                    &pl->pl_server_lock_volume, &ldlm_pool_u64_fops);
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "limit", &pl->pl_limit,
-                    &ldlm_pool_rw_atomic_fops);
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "granted",
-                    &pl->pl_granted, &ldlm_pool_atomic_fops);
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "grant_speed", pl,
-                    &lprocfs_grant_speed_fops);
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "cancel_rate",
-                    &pl->pl_cancel_rate, &ldlm_pool_atomic_fops);
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "grant_rate",
-                    &pl->pl_grant_rate, &ldlm_pool_atomic_fops);
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "grant_plan", pl,
-                    &lprocfs_grant_plan_fops);
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "recalc_period",
-                    pl, &lprocfs_recalc_period_fops);
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "lock_volume_factor",
-                    &pl->pl_lock_volume_factor, &ldlm_pool_rw_atomic_fops);
-       ldlm_add_var(&pool_vars[0], pl->pl_proc_dir, "state", pl,
+
+       ldlm_add_var(&pool_vars[0], pl->pl_debugfs_entry, "state", pl,
                     &lprocfs_pool_state_fops);
 
-        pl->pl_stats = lprocfs_alloc_stats(LDLM_POOL_LAST_STAT -
-                                           LDLM_POOL_FIRST_STAT, 0);
-        if (!pl->pl_stats)
-                GOTO(out_free_name, rc = -ENOMEM);
-
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_GRANTED_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "granted", "locks");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_GRANT_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "grant", "locks");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_CANCEL_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "cancel", "locks");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_GRANT_RATE_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "grant_rate", "locks/s");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_CANCEL_RATE_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "cancel_rate", "locks/s");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_GRANT_PLAN_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "grant_plan", "locks/s");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_SLV_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "slv", "slv");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_SHRINK_REQTD_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "shrink_request", "locks");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_SHRINK_FREED_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "shrink_freed", "locks");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_RECALC_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "recalc_freed", "locks");
-        lprocfs_counter_init(pl->pl_stats, LDLM_POOL_TIMING_STAT,
-                             LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
-                             "recalc_timing", "sec");
-       rc = lprocfs_register_stats(pl->pl_proc_dir, "stats", pl->pl_stats);
-
-        EXIT;
-out_free_name:
-        OBD_FREE(var_name, MAX_STRING_SIZE + 1);
-        return rc;
-}
-
-static void ldlm_pool_proc_fini(struct ldlm_pool *pl)
-{
-        if (pl->pl_stats != NULL) {
-                lprocfs_free_stats(&pl->pl_stats);
-                pl->pl_stats = NULL;
-        }
-        if (pl->pl_proc_dir != NULL) {
-                lprocfs_remove(&pl->pl_proc_dir);
-                pl->pl_proc_dir = NULL;
-        }
+       pl->pl_stats = lprocfs_alloc_stats(LDLM_POOL_LAST_STAT -
+                                          LDLM_POOL_FIRST_STAT, 0);
+       if (!pl->pl_stats)
+               GOTO(out, rc = -ENOMEM);
+
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_GRANTED_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "granted", "locks");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_GRANT_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "grant", "locks");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_CANCEL_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "cancel", "locks");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_GRANT_RATE_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "grant_rate", "locks/s");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_CANCEL_RATE_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "cancel_rate", "locks/s");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_GRANT_PLAN_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "grant_plan", "locks/s");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_SLV_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "slv", "slv");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_SHRINK_REQTD_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "shrink_request", "locks");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_SHRINK_FREED_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "shrink_freed", "locks");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_RECALC_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "recalc_freed", "locks");
+       lprocfs_counter_init(pl->pl_stats, LDLM_POOL_TIMING_STAT,
+                            LPROCFS_CNTR_AVGMINMAX | LPROCFS_CNTR_STDDEV,
+                            "recalc_timing", "sec");
+       rc = ldebugfs_register_stats(pl->pl_debugfs_entry, "stats",
+                                    pl->pl_stats);
+
+       EXIT;
+out:
+       return rc;
+}
+
+static void ldlm_pool_sysfs_fini(struct ldlm_pool *pl)
+{
+       kobject_put(&pl->pl_kobj);
+       wait_for_completion(&pl->pl_kobj_unregister);
+}
+
+static void ldlm_pool_debugfs_fini(struct ldlm_pool *pl)
+{
+       if (pl->pl_stats != NULL) {
+               lprocfs_free_stats(&pl->pl_stats);
+               pl->pl_stats = NULL;
+       }
+       if (pl->pl_debugfs_entry != NULL) {
+               ldebugfs_remove(&pl->pl_debugfs_entry);
+               pl->pl_debugfs_entry = NULL;
+       }
 }
 
 int ldlm_pool_init(struct ldlm_pool *pl, struct ldlm_namespace *ns,
                   int idx, enum ldlm_side client)
 {
        int rc;
+
        ENTRY;
 
        spin_lock_init(&pl->pl_lock);
        atomic_set(&pl->pl_granted, 0);
-       pl->pl_recalc_time = cfs_time_current_sec();
+       pl->pl_recalc_time = ktime_get_real_seconds();
        atomic_set(&pl->pl_lock_volume_factor, 1);
 
        atomic_set(&pl->pl_grant_rate, 0);
@@ -866,39 +895,44 @@ int ldlm_pool_init(struct ldlm_pool *pl, struct ldlm_namespace *ns,
        snprintf(pl->pl_name, sizeof(pl->pl_name), "ldlm-pool-%s-%d",
                 ldlm_ns_name(ns), idx);
 
-        if (client == LDLM_NAMESPACE_SERVER) {
-                pl->pl_ops = &ldlm_srv_pool_ops;
-                ldlm_pool_set_limit(pl, LDLM_POOL_HOST_L);
-                pl->pl_recalc_period = LDLM_POOL_SRV_DEF_RECALC_PERIOD;
-                pl->pl_server_lock_volume = ldlm_pool_slv_max(LDLM_POOL_HOST_L);
-        } else {
-                ldlm_pool_set_limit(pl, 1);
-                pl->pl_server_lock_volume = 0;
-                pl->pl_ops = &ldlm_cli_pool_ops;
-                pl->pl_recalc_period = LDLM_POOL_CLI_DEF_RECALC_PERIOD;
-        }
-        pl->pl_client_lock_volume = 0;
-        rc = ldlm_pool_proc_init(pl);
-        if (rc)
-                RETURN(rc);
+       if (client == LDLM_NAMESPACE_SERVER) {
+               pl->pl_ops = &ldlm_srv_pool_ops;
+               ldlm_pool_set_limit(pl, LDLM_POOL_HOST_L);
+               pl->pl_recalc_period = LDLM_POOL_SRV_DEF_RECALC_PERIOD;
+               pl->pl_server_lock_volume = ldlm_pool_slv_max(LDLM_POOL_HOST_L);
+       } else {
+               ldlm_pool_set_limit(pl, 1);
+               pl->pl_server_lock_volume = 0;
+               pl->pl_ops = &ldlm_cli_pool_ops;
+               pl->pl_recalc_period = LDLM_POOL_CLI_DEF_RECALC_PERIOD;
+       }
+       pl->pl_client_lock_volume = 0;
+       rc = ldlm_pool_debugfs_init(pl);
+       if (rc)
+               RETURN(rc);
+
+       rc = ldlm_pool_sysfs_init(pl);
+       if (rc)
+               RETURN(rc);
 
-        CDEBUG(D_DLMTRACE, "Lock pool %s is initialized\n", pl->pl_name);
+       CDEBUG(D_DLMTRACE, "Lock pool %s is initialized\n", pl->pl_name);
 
-        RETURN(rc);
+       RETURN(rc);
 }
 
 void ldlm_pool_fini(struct ldlm_pool *pl)
 {
-        ENTRY;
-        ldlm_pool_proc_fini(pl);
+       ENTRY;
+       ldlm_pool_sysfs_fini(pl);
+       ldlm_pool_debugfs_fini(pl);
 
-        /*
-         * Pool should not be used after this point. We can't free it here as
-         * it lives in struct ldlm_namespace, but still interested in catching
-         * any abnormal using cases.
-         */
-        POISON(pl, 0x5a, sizeof(*pl));
-        EXIT;
+       /*
+        * Pool should not be used after this point. We can't free it here as
+        * it lives in struct ldlm_namespace, but still interested in catching
+        * any abnormal using cases.
+        */
+       POISON(pl, 0x5a, sizeof(*pl));
+       EXIT;
 }
 
 /**
@@ -967,6 +1001,7 @@ void ldlm_pool_del(struct ldlm_pool *pl, struct ldlm_lock *lock)
 __u64 ldlm_pool_get_slv(struct ldlm_pool *pl)
 {
        __u64 slv;
+
        spin_lock(&pl->pl_lock);
        slv = pl->pl_server_lock_volume;
        spin_unlock(&pl->pl_lock);
@@ -993,6 +1028,7 @@ void ldlm_pool_set_slv(struct ldlm_pool *pl, __u64 slv)
 __u64 ldlm_pool_get_clv(struct ldlm_pool *pl)
 {
        __u64 slv;
+
        spin_lock(&pl->pl_lock);
        slv = pl->pl_client_lock_volume;
        spin_unlock(&pl->pl_lock);
@@ -1035,15 +1071,13 @@ __u32 ldlm_pool_get_lvf(struct ldlm_pool *pl)
        return atomic_read(&pl->pl_lock_volume_factor);
 }
 
-static struct ptlrpc_thread *ldlm_pools_thread;
 static struct shrinker *ldlm_pools_srv_shrinker;
 static struct shrinker *ldlm_pools_cli_shrinker;
-static struct completion ldlm_pools_comp;
 
 /*
-* count locks from all namespaces (if possible). Returns number of
-* cached locks.
-*/
+ * count locks from all namespaces (if possible). Returns number of
+ * cached locks.
+ */
 static unsigned long ldlm_pools_count(enum ldlm_side client, gfp_t gfp_mask)
 {
        unsigned long total = 0;
@@ -1054,9 +1088,6 @@ static unsigned long ldlm_pools_count(enum ldlm_side client, gfp_t gfp_mask)
        if (client == LDLM_NAMESPACE_CLIENT && !(gfp_mask & __GFP_FS))
                return 0;
 
-       CDEBUG(D_DLMTRACE, "Request to count %s locks from all pools\n",
-              client == LDLM_NAMESPACE_CLIENT ? "client" : "server");
-
        /*
         * Find out how many resources we may release.
         */
@@ -1112,7 +1143,7 @@ static unsigned long ldlm_pools_scan(enum ldlm_side client, int nr,
 
                /*
                 * Do not call shrink under ldlm_namespace_lock(client)
-               */
+                */
                mutex_lock(ldlm_namespace_lock(client));
                if (list_empty(ldlm_namespace_list(client))) {
                        mutex_unlock(ldlm_namespace_lock(client));
@@ -1153,7 +1184,8 @@ static unsigned long ldlm_pools_srv_scan(struct shrinker *s,
                               sc->gfp_mask);
 }
 
-static unsigned long ldlm_pools_cli_count(struct shrinker *s, struct shrink_control *sc)
+static unsigned long ldlm_pools_cli_count(struct shrinker *s,
+                                         struct shrink_control *sc)
 {
        return ldlm_pools_count(LDLM_NAMESPACE_CLIENT, sc->gfp_mask);
 }
@@ -1179,9 +1211,6 @@ static int ldlm_pools_shrink(enum ldlm_side client, int nr, gfp_t gfp_mask)
            !(gfp_mask & __GFP_FS))
                return -1;
 
-       CDEBUG(D_DLMTRACE, "Request to shrink %d %s locks from all pools\n",
-              nr, client == LDLM_NAMESPACE_CLIENT ? "client" : "server");
-
        total = ldlm_pools_count(client, gfp_mask);
 
        if (nr == 0 || total == 0)
@@ -1192,122 +1221,49 @@ static int ldlm_pools_shrink(enum ldlm_side client, int nr, gfp_t gfp_mask)
 
 static int ldlm_pools_srv_shrink(SHRINKER_ARGS(sc, nr_to_scan, gfp_mask))
 {
-        return ldlm_pools_shrink(LDLM_NAMESPACE_SERVER,
-                                 shrink_param(sc, nr_to_scan),
-                                 shrink_param(sc, gfp_mask));
+       return ldlm_pools_shrink(LDLM_NAMESPACE_SERVER,
+                                shrink_param(sc, nr_to_scan),
+                                shrink_param(sc, gfp_mask));
 }
 
 static int ldlm_pools_cli_shrink(SHRINKER_ARGS(sc, nr_to_scan, gfp_mask))
 {
-        return ldlm_pools_shrink(LDLM_NAMESPACE_CLIENT,
-                                 shrink_param(sc, nr_to_scan),
-                                 shrink_param(sc, gfp_mask));
+       return ldlm_pools_shrink(LDLM_NAMESPACE_CLIENT,
+                                shrink_param(sc, nr_to_scan),
+                                shrink_param(sc, gfp_mask));
 }
 
 #endif /* HAVE_SHRINKER_COUNT */
 
-int ldlm_pools_recalc(enum ldlm_side client)
+static time64_t ldlm_pools_recalc_delay(enum ldlm_side side)
 {
-       unsigned long nr_l = 0, nr_p = 0, l;
        struct ldlm_namespace *ns;
        struct ldlm_namespace *ns_old = NULL;
-       int nr, equal = 0;
        /* seconds of sleep if no active namespaces */
-       int time = client ? LDLM_POOL_CLI_DEF_RECALC_PERIOD :
-                           LDLM_POOL_SRV_DEF_RECALC_PERIOD;
-
-       /*
-        * No need to setup pool limit for client pools.
-        */
-       if (client == LDLM_NAMESPACE_SERVER) {
-               /*
-                * Check all modest namespaces first.
-                */
-               mutex_lock(ldlm_namespace_lock(client));
-               list_for_each_entry(ns, ldlm_namespace_list(client),
-                                   ns_list_chain)
-               {
-                       if (ns->ns_appetite != LDLM_NAMESPACE_MODEST)
-                               continue;
-
-                        l = ldlm_pool_granted(&ns->ns_pool);
-                        if (l == 0)
-                                l = 1;
-
-                        /*
-                         * Set the modest pools limit equal to their avg granted
-                         * locks + ~6%.
-                         */
-                        l += dru(l, LDLM_POOLS_MODEST_MARGIN_SHIFT, 0);
-                        ldlm_pool_setup(&ns->ns_pool, l);
-                        nr_l += l;
-                        nr_p++;
-                }
-
-                /*
-                 * Make sure that modest namespaces did not eat more that 2/3
-                 * of limit.
-                 */
-                if (nr_l >= 2 * (LDLM_POOL_HOST_L / 3)) {
-                        CWARN("\"Modest\" pools eat out 2/3 of server locks "
-                              "limit (%lu of %lu). This means that you have too "
-                              "many clients for this amount of server RAM. "
-                              "Upgrade server!\n", nr_l, LDLM_POOL_HOST_L);
-                        equal = 1;
-                }
-
+       time64_t delay = side == LDLM_NAMESPACE_SERVER ?
+                                LDLM_POOL_SRV_DEF_RECALC_PERIOD :
+                                LDLM_POOL_CLI_DEF_RECALC_PERIOD;
+       int nr;
+
+       /* Recalc at least ldlm_namespace_nr(side) namespaces. */
+       for (nr = ldlm_namespace_nr_read(side); nr > 0; nr--) {
+               int skip;
                /*
-                * The rest is given to greedy namespaces.
+                * Lock the list, get first @ns in the list, getref, move it
+                * to the tail, unlock and call pool recalc. This way we avoid
+                * calling recalc under @ns lock, which is really good as we
+                * get rid of potential deadlock on side nodes when canceling
+                * locks synchronously.
                 */
-               list_for_each_entry(ns, ldlm_namespace_list(client),
-                                   ns_list_chain)
-               {
-                       if (!equal && ns->ns_appetite != LDLM_NAMESPACE_GREEDY)
-                               continue;
-
-                        if (equal) {
-                                /*
-                                 * In the case 2/3 locks are eaten out by
-                                 * modest pools, we re-setup equal limit
-                                 * for _all_ pools.
-                                 */
-                                l = LDLM_POOL_HOST_L /
-                                       ldlm_namespace_nr_read(client);
-                        } else {
-                                /*
-                                 * All the rest of greedy pools will have
-                                 * all locks in equal parts.
-                                 */
-                                l = (LDLM_POOL_HOST_L - nr_l) /
-                                       (ldlm_namespace_nr_read(client) -
-                                         nr_p);
-                        }
-                        ldlm_pool_setup(&ns->ns_pool, l);
-                }
-               mutex_unlock(ldlm_namespace_lock(client));
-        }
-
-        /*
-         * Recalc at least ldlm_namespace_nr(client) namespaces.
-         */
-       for (nr = ldlm_namespace_nr_read(client); nr > 0; nr--) {
-                int     skip;
-                /*
-                 * Lock the list, get first @ns in the list, getref, move it
-                 * to the tail, unlock and call pool recalc. This way we avoid
-                 * calling recalc under @ns lock what is really good as we get
-                 * rid of potential deadlock on client nodes when canceling
-                 * locks synchronously.
-                 */
-               mutex_lock(ldlm_namespace_lock(client));
-               if (list_empty(ldlm_namespace_list(client))) {
-                       mutex_unlock(ldlm_namespace_lock(client));
+               mutex_lock(ldlm_namespace_lock(side));
+               if (list_empty(ldlm_namespace_list(side))) {
+                       mutex_unlock(ldlm_namespace_lock(side));
                        break;
                }
-               ns = ldlm_namespace_first_locked(client);
+               ns = ldlm_namespace_first_locked(side);
 
                if (ns_old == ns) { /* Full pass complete */
-                       mutex_unlock(ldlm_namespace_lock(client));
+                       mutex_unlock(ldlm_namespace_lock(side));
                        break;
                }
 
@@ -1322,8 +1278,8 @@ int ldlm_pools_recalc(enum ldlm_side client)
                 *   there).
                 */
                if (ldlm_ns_empty(ns)) {
-                       ldlm_namespace_move_to_inactive_locked(ns, client);
-                       mutex_unlock(ldlm_namespace_lock(client));
+                       ldlm_namespace_move_to_inactive_locked(ns, side);
+                       mutex_unlock(ldlm_namespace_lock(side));
                        continue;
                }
 
@@ -1343,144 +1299,116 @@ int ldlm_pools_recalc(enum ldlm_side client)
                }
                spin_unlock(&ns->ns_lock);
 
-               ldlm_namespace_move_to_active_locked(ns, client);
-               mutex_unlock(ldlm_namespace_lock(client));
+               ldlm_namespace_move_to_active_locked(ns, side);
+               mutex_unlock(ldlm_namespace_lock(side));
 
                /*
                 * After setup is done - recalc the pool.
                 */
                if (!skip) {
-                       int ttime = ldlm_pool_recalc(&ns->ns_pool);
-
-                       if (ttime < time)
-                               time = ttime;
-
+                       delay = min(delay, ldlm_pool_recalc(&ns->ns_pool));
                        ldlm_namespace_put(ns);
                }
-        }
-
-       /* Wake up the blocking threads from time to time. */
-       ldlm_bl_thread_wakeup();
+       }
 
-       return time;
+       return delay;
 }
 
-static int ldlm_pools_thread_main(void *arg)
-{
-       struct ptlrpc_thread *thread = (struct ptlrpc_thread *)arg;
-       int s_time, c_time;
-       ENTRY;
-
-       thread_set_flags(thread, SVC_RUNNING);
-       wake_up(&thread->t_ctl_waitq);
+static void ldlm_pools_recalc_task(struct work_struct *ws);
+static DECLARE_DELAYED_WORK(ldlm_pools_recalc_work, ldlm_pools_recalc_task);
 
-       CDEBUG(D_DLMTRACE, "%s: pool thread starting, process %d\n",
-              "ldlm_poold", current_pid());
+static void ldlm_pools_recalc_task(struct work_struct *ws)
+{
+       /* seconds of sleep if no active namespaces */
+       time64_t delay;
+#ifdef HAVE_SERVER_SUPPORT
+       struct ldlm_namespace *ns;
+       unsigned long nr_l = 0, nr_p = 0, l;
+       int equal = 0;
 
-        while (1) {
-                struct l_wait_info lwi;
+       /* Check all modest namespaces first. */
+       mutex_lock(ldlm_namespace_lock(LDLM_NAMESPACE_SERVER));
+       list_for_each_entry(ns, ldlm_namespace_list(LDLM_NAMESPACE_SERVER),
+                           ns_list_chain) {
+               if (ns->ns_appetite != LDLM_NAMESPACE_MODEST)
+                       continue;
 
-               /*
-                * Recal all pools on this tick.
-                */
-               s_time = ldlm_pools_recalc(LDLM_NAMESPACE_SERVER);
-               c_time = ldlm_pools_recalc(LDLM_NAMESPACE_CLIENT);
+               l = ldlm_pool_granted(&ns->ns_pool);
+               if (l == 0)
+                       l = 1;
 
                /*
-                * Wait until the next check time, or until we're
-                * stopped.
+                * Set the modest pools limit equal to their avg granted
+                * locks + ~6%.
                 */
-               lwi = LWI_TIMEOUT(cfs_time_seconds(min(s_time, c_time)),
-                                 NULL, NULL);
-                l_wait_event(thread->t_ctl_waitq,
-                             thread_is_stopping(thread) ||
-                             thread_is_event(thread),
-                             &lwi);
-
-                if (thread_test_and_clear_flags(thread, SVC_STOPPING))
-                        break;
-                else
-                        thread_test_and_clear_flags(thread, SVC_EVENT);
-        }
-
-       thread_set_flags(thread, SVC_STOPPED);
-       wake_up(&thread->t_ctl_waitq);
-
-       CDEBUG(D_DLMTRACE, "%s: pool thread exiting, process %d\n",
-               "ldlm_poold", current_pid());
-
-       complete_and_exit(&ldlm_pools_comp, 0);
-}
-
-static int ldlm_pools_thread_start(void)
-{
-       struct l_wait_info lwi = { 0 };
-       struct task_struct *task;
-       ENTRY;
-
-       if (ldlm_pools_thread != NULL)
-               RETURN(-EALREADY);
-
-       OBD_ALLOC_PTR(ldlm_pools_thread);
-       if (ldlm_pools_thread == NULL)
-               RETURN(-ENOMEM);
-
-       init_completion(&ldlm_pools_comp);
-       init_waitqueue_head(&ldlm_pools_thread->t_ctl_waitq);
+               l += dru(l, LDLM_POOLS_MODEST_MARGIN_SHIFT, 0);
+               ldlm_pool_setup(&ns->ns_pool, l);
+               nr_l += l;
+               nr_p++;
+       }
 
-       task = kthread_run(ldlm_pools_thread_main, ldlm_pools_thread,
-                          "ldlm_poold");
-       if (IS_ERR(task)) {
-               CERROR("Can't start pool thread, error %ld\n", PTR_ERR(task));
-               OBD_FREE(ldlm_pools_thread, sizeof(*ldlm_pools_thread));
-               ldlm_pools_thread = NULL;
-               RETURN(PTR_ERR(task));
+       /*
+        * Make sure than modest namespaces did not eat more that 2/3
+        * of limit.
+        */
+       if (nr_l >= 2 * (LDLM_POOL_HOST_L / 3)) {
+               CWARN("'Modest' pools eat out 2/3 of server locks limit (%lu of %lu). This means that you have too many clients for this amount of server RAM. Upgrade server!\n",
+                     nr_l, LDLM_POOL_HOST_L);
+               equal = 1;
        }
-       l_wait_event(ldlm_pools_thread->t_ctl_waitq,
-                    thread_is_running(ldlm_pools_thread), &lwi);
-       RETURN(0);
-}
 
-static void ldlm_pools_thread_stop(void)
-{
-       ENTRY;
+       /* The rest is given to greedy namespaces. */
+       list_for_each_entry(ns, ldlm_namespace_list(LDLM_NAMESPACE_SERVER),
+                           ns_list_chain) {
+               if (!equal && ns->ns_appetite != LDLM_NAMESPACE_GREEDY)
+                       continue;
 
-       if (ldlm_pools_thread == NULL) {
-               EXIT;
-               return;
+               if (equal) {
+                       /*
+                        * In the case 2/3 locks are eaten out by
+                        * modest pools, we re-setup equal limit
+                        * for _all_ pools.
+                        */
+                       l = LDLM_POOL_HOST_L /
+                               ldlm_namespace_nr_read(LDLM_NAMESPACE_SERVER);
+               } else {
+                       /*
+                        * All the rest of greedy pools will have
+                        * all locks in equal parts.
+                        */
+                       l = (LDLM_POOL_HOST_L - nr_l) /
+                               (ldlm_namespace_nr_read(LDLM_NAMESPACE_SERVER) -
+                                nr_p);
+               }
+               ldlm_pool_setup(&ns->ns_pool, l);
        }
+       mutex_unlock(ldlm_namespace_lock(LDLM_NAMESPACE_SERVER));
 
-       thread_set_flags(ldlm_pools_thread, SVC_STOPPING);
-       wake_up(&ldlm_pools_thread->t_ctl_waitq);
+       delay = min(ldlm_pools_recalc_delay(LDLM_NAMESPACE_SERVER),
+                   ldlm_pools_recalc_delay(LDLM_NAMESPACE_CLIENT));
+#else  /* !HAVE_SERVER_SUPPORT */
+       delay = ldlm_pools_recalc_delay(LDLM_NAMESPACE_CLIENT);
+#endif /* HAVE_SERVER_SUPPORT */
 
-       /*
-        * Make sure that pools thread is finished before freeing @thread.
-        * This fixes possible race and oops due to accessing freed memory
-        * in pools thread.
-        */
-       wait_for_completion(&ldlm_pools_comp);
-       OBD_FREE_PTR(ldlm_pools_thread);
-       ldlm_pools_thread = NULL;
-       EXIT;
+       /* Wake up the blocking threads from time to time. */
+       ldlm_bl_thread_wakeup();
+
+       schedule_delayed_work(&ldlm_pools_recalc_work, cfs_time_seconds(delay));
 }
 
 int ldlm_pools_init(void)
 {
-       int rc;
        DEF_SHRINKER_VAR(shsvar, ldlm_pools_srv_shrink,
                         ldlm_pools_srv_count, ldlm_pools_srv_scan);
        DEF_SHRINKER_VAR(shcvar, ldlm_pools_cli_shrink,
                         ldlm_pools_cli_count, ldlm_pools_cli_scan);
-       ENTRY;
 
-       rc = ldlm_pools_thread_start();
-       if (rc == 0) {
-               ldlm_pools_srv_shrinker =
-                       set_shrinker(DEFAULT_SEEKS, &shsvar);
-               ldlm_pools_cli_shrinker =
-                       set_shrinker(DEFAULT_SEEKS, &shcvar);
-       }
-       RETURN(rc);
+       schedule_delayed_work(&ldlm_pools_recalc_work,
+                             LDLM_POOL_CLI_DEF_RECALC_PERIOD);
+       ldlm_pools_srv_shrinker = set_shrinker(DEFAULT_SEEKS, &shsvar);
+       ldlm_pools_cli_shrinker = set_shrinker(DEFAULT_SEEKS, &shcvar);
+
+       return 0;
 }
 
 void ldlm_pools_fini(void)
@@ -1493,18 +1421,18 @@ void ldlm_pools_fini(void)
                remove_shrinker(ldlm_pools_cli_shrinker);
                ldlm_pools_cli_shrinker = NULL;
        }
-       ldlm_pools_thread_stop();
+       cancel_delayed_work_sync(&ldlm_pools_recalc_work);
 }
 
 #else /* !HAVE_LRU_RESIZE_SUPPORT */
 int ldlm_pool_setup(struct ldlm_pool *pl, int limit)
 {
-        return 0;
+       return 0;
 }
 
-int ldlm_pool_recalc(struct ldlm_pool *pl)
+time64_t ldlm_pool_recalc(struct ldlm_pool *pl)
 {
-        return 0;
+       return 0;
 }
 
 int ldlm_pool_shrink(struct ldlm_pool *pl,
@@ -1521,57 +1449,57 @@ int ldlm_pool_init(struct ldlm_pool *pl, struct ldlm_namespace *ns,
 
 void ldlm_pool_fini(struct ldlm_pool *pl)
 {
-        return;
+       return;
 }
 
 void ldlm_pool_add(struct ldlm_pool *pl, struct ldlm_lock *lock)
 {
-        return;
+       return;
 }
 
 void ldlm_pool_del(struct ldlm_pool *pl, struct ldlm_lock *lock)
 {
-        return;
+       return;
 }
 
 __u64 ldlm_pool_get_slv(struct ldlm_pool *pl)
 {
-        return 1;
+       return 1;
 }
 
 void ldlm_pool_set_slv(struct ldlm_pool *pl, __u64 slv)
 {
-        return;
+       return;
 }
 
 __u64 ldlm_pool_get_clv(struct ldlm_pool *pl)
 {
-        return 1;
+       return 1;
 }
 
 void ldlm_pool_set_clv(struct ldlm_pool *pl, __u64 clv)
 {
-        return;
+       return;
 }
 
 __u32 ldlm_pool_get_limit(struct ldlm_pool *pl)
 {
-        return 0;
+       return 0;
 }
 
 void ldlm_pool_set_limit(struct ldlm_pool *pl, __u32 limit)
 {
-        return;
+       return;
 }
 
 __u32 ldlm_pool_get_lvf(struct ldlm_pool *pl)
 {
-        return 0;
+       return 0;
 }
 
 int ldlm_pools_init(void)
 {
-        return 0;
+       return 0;
 }
 
 void ldlm_pools_fini(void)
@@ -1579,8 +1507,4 @@ void ldlm_pools_fini(void)
        return;
 }
 
-int ldlm_pools_recalc(enum ldlm_side client)
-{
-       return 0;
-}
 #endif /* HAVE_LRU_RESIZE_SUPPORT */