Whamcloud - gitweb
LU-12678 lnet: convert lpni_refcount to a kref
[fs/lustre-release.git] / lnet / lnet / router_proc.c
index f9d8d36..8f839d4 100644 (file)
@@ -1,9 +1,9 @@
 /*
  * Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved.
  *
- * Copyright (c) 2011, 2015, Intel Corporation.
+ * Copyright (c) 2011, 2017, Intel Corporation.
  *
- *   This file is part of Lustre, https://wiki.hpdd.intel.com/
+ *   This file is part of Lustre, https://wiki.whamcloud.com/
  *
  *   Portals is free software; you can redistribute it and/or
  *   modify it under the terms of version 2 of the GNU General Public
  */
 
 #define DEBUG_SUBSYSTEM S_LNET
+
+#include <linux/uaccess.h>
+
 #include <libcfs/libcfs.h>
 #include <lnet/lib-lnet.h>
 
 /* This is really lnet_proc.c. You might need to update sanity test 215
  * if any file format is changed. */
 
-static struct ctl_table_header *lnet_table_header = NULL;
-
 #define LNET_LOFFT_BITS                (sizeof(loff_t) * 8)
 /*
  * NB: max allowed LNET_CPT_BITS is 8 on 64-bit system and 2 on 32-bit system
  */
 #define LNET_PROC_CPT_BITS     (LNET_CPT_BITS + 1)
 /* change version, 16 bits or 8 bits */
-#define LNET_PROC_VER_BITS     MAX(((MIN(LNET_LOFFT_BITS, 64)) / 4), 8)
+#define LNET_PROC_VER_BITS             \
+       clamp_t(int, LNET_LOFFT_BITS / 4, 8, 16)
 
 #define LNET_PROC_HASH_BITS    LNET_PEER_HASH_BITS
 /*
@@ -81,9 +83,9 @@ static int __proc_lnet_stats(void *data, int write,
 {
        int              rc;
        struct lnet_counters *ctrs;
+       struct lnet_counters_common common;
        int              len;
-       char            *tmpstr;
-       const int        tmpsiz = 256; /* 7 %u and 4 __u64 */
+       char tmpstr[256]; /* 7 %u and 4 u64 */
 
        if (write) {
                lnet_counters_reset();
@@ -96,31 +98,28 @@ static int __proc_lnet_stats(void *data, int write,
        if (ctrs == NULL)
                return -ENOMEM;
 
-       LIBCFS_ALLOC(tmpstr, tmpsiz);
-       if (tmpstr == NULL) {
-               LIBCFS_FREE(ctrs, sizeof(*ctrs));
-               return -ENOMEM;
-       }
+       rc = lnet_counters_get(ctrs);
+       if (rc)
+               goto out_no_ctrs;
 
-       lnet_counters_get(ctrs);
+       common = ctrs->lct_common;
 
-       len = snprintf(tmpstr, tmpsiz,
-                      "%u %u %u %u %u %u %u %llu %llu "
-                      "%llu %llu",
-                      ctrs->msgs_alloc, ctrs->msgs_max,
-                      ctrs->errors,
-                      ctrs->send_count, ctrs->recv_count,
-                      ctrs->route_count, ctrs->drop_count,
-                      ctrs->send_length, ctrs->recv_length,
-                      ctrs->route_length, ctrs->drop_length);
+       len = scnprintf(tmpstr, sizeof(tmpstr),
+                       "%u %u %u %u %u %u %u %llu %llu "
+                       "%llu %llu",
+                       common.lcc_msgs_alloc, common.lcc_msgs_max,
+                       common.lcc_errors,
+                       common.lcc_send_count, common.lcc_recv_count,
+                       common.lcc_route_count, common.lcc_drop_count,
+                       common.lcc_send_length, common.lcc_recv_length,
+                       common.lcc_route_length, common.lcc_drop_length);
 
-       if (pos >= min_t(int, len, strlen(tmpstr)))
+       if (pos >= len)
                rc = 0;
        else
                rc = cfs_trace_copyout_string(buffer, nob,
                                              tmpstr + pos, "\n");
-
-       LIBCFS_FREE(tmpstr, tmpsiz);
+out_no_ctrs:
        LIBCFS_FREE(ctrs, sizeof(*ctrs));
        return rc;
 }
@@ -145,7 +144,7 @@ proc_lnet_routes(struct ctl_table *table, int write, void __user *buffer,
        int             ver;
        int             off;
 
-       CLASSERT(sizeof(loff_t) >= 4);
+       BUILD_BUG_ON(sizeof(loff_t) < 4);
 
        off = LNET_PROC_HOFF_GET(*ppos);
        ver = LNET_PROC_VER_GET(*ppos);
@@ -162,12 +161,12 @@ proc_lnet_routes(struct ctl_table *table, int write, void __user *buffer,
        s = tmpstr; /* points to current position in tmpstr[] */
 
        if (*ppos == 0) {
-               s += snprintf(s, tmpstr + tmpsiz - s, "Routing %s\n",
-                             the_lnet.ln_routing ? "enabled" : "disabled");
+               s += scnprintf(s, tmpstr + tmpsiz - s, "Routing %s\n",
+                              the_lnet.ln_routing ? "enabled" : "disabled");
                LASSERT(tmpstr + tmpsiz - s > 0);
 
-               s += snprintf(s, tmpstr + tmpsiz - s, "%-8s %4s %8s %7s %s\n",
-                             "net", "hops", "priority", "state", "router");
+               s += scnprintf(s, tmpstr + tmpsiz - s, "%-8s %4s %8s %7s %s\n",
+                              "net", "hops", "priority", "state", "router");
                LASSERT(tmpstr + tmpsiz - s > 0);
 
                lnet_net_lock(0);
@@ -221,18 +220,17 @@ proc_lnet_routes(struct ctl_table *table, int write, void __user *buffer,
                }
 
                if (route != NULL) {
-                       __u32        net        = rnet->lrn_net;
-                       __u32 hops              = route->lr_hops;
-                       unsigned int priority   = route->lr_priority;
-                       lnet_nid_t   nid        = route->lr_gateway->lpni_nid;
-                       int          alive      = lnet_is_route_alive(route);
-
-                       s += snprintf(s, tmpstr + tmpsiz - s,
-                                     "%-8s %4d %8u %7s %s\n",
-                                     libcfs_net2str(net), hops,
-                                     priority,
-                                     alive ? "up" : "down",
-                                     libcfs_nid2str(nid));
+                       __u32 net = rnet->lrn_net;
+                       __u32 hops = route->lr_hops;
+                       unsigned int priority = route->lr_priority;
+                       int alive = lnet_is_route_alive(route);
+
+                       s += scnprintf(s, tmpstr + tmpsiz - s,
+                                      "%-8s %4d %8u %7s %s\n",
+                                      libcfs_net2str(net), hops,
+                                      priority,
+                                      alive ? "up" : "down",
+                                      libcfs_nid2str(route->lr_nid));
                        LASSERT(tmpstr + tmpsiz - s > 0);
                }
 
@@ -287,11 +285,9 @@ proc_lnet_routers(struct ctl_table *table, int write, void __user *buffer,
        s = tmpstr; /* points to current position in tmpstr[] */
 
        if (*ppos == 0) {
-               s += snprintf(s, tmpstr + tmpsiz - s,
-                             "%-4s %7s %9s %6s %12s %9s %8s %7s %s\n",
-                             "ref", "rtr_ref", "alive_cnt", "state",
-                             "last_ping", "ping_sent", "deadline",
-                             "down_ni", "router");
+               s += scnprintf(s, tmpstr + tmpsiz - s,
+                              "%-4s %7s %5s %s\n",
+                              "ref", "rtr_ref", "alive", "router");
                LASSERT(tmpstr + tmpsiz - s > 0);
 
                lnet_net_lock(0);
@@ -300,7 +296,7 @@ proc_lnet_routers(struct ctl_table *table, int write, void __user *buffer,
                *ppos = LNET_PROC_POS_MAKE(0, ver, 0, off);
        } else {
                struct list_head *r;
-               struct lnet_peer_ni *peer = NULL;
+               struct lnet_peer *peer = NULL;
                int               skip = off - 1;
 
                lnet_net_lock(0);
@@ -315,9 +311,9 @@ proc_lnet_routers(struct ctl_table *table, int write, void __user *buffer,
                r = the_lnet.ln_routers.next;
 
                while (r != &the_lnet.ln_routers) {
-                       struct lnet_peer_ni *lp =
-                         list_entry(r, struct lnet_peer_ni,
-                                    lpni_rtr_list);
+                       struct lnet_peer *lp =
+                         list_entry(r, struct lnet_peer,
+                                    lp_rtr_list);
 
                        if (skip == 0) {
                                peer = lp;
@@ -329,48 +325,16 @@ proc_lnet_routers(struct ctl_table *table, int write, void __user *buffer,
                }
 
                if (peer != NULL) {
-                       lnet_nid_t nid = peer->lpni_nid;
-                       cfs_time_t now = cfs_time_current();
-                       cfs_time_t deadline = peer->lpni_ping_deadline;
-                       int nrefs     = atomic_read(&peer->lpni_refcount);
-                       int nrtrrefs  = peer->lpni_rtr_refcount;
-                       int alive_cnt = peer->lpni_alive_count;
-                       int alive     = peer->lpni_alive;
-                       int pingsent  = !peer->lpni_ping_notsent;
-                       int last_ping = cfs_duration_sec(cfs_time_sub(now,
-                                                    peer->lpni_ping_timestamp));
-                       int down_ni   = 0;
-                       struct lnet_route *rtr;
-
-                       if ((peer->lpni_ping_feats &
-                            LNET_PING_FEAT_NI_STATUS) != 0) {
-                               list_for_each_entry(rtr, &peer->lpni_routes,
-                                                   lr_gwlist) {
-                                       /* downis on any route should be the
-                                        * number of downis on the gateway */
-                                       if (rtr->lr_downis != 0) {
-                                               down_ni = rtr->lr_downis;
-                                               break;
-                                       }
-                               }
-                       }
-
-                       if (deadline == 0)
-                               s += snprintf(s, tmpstr + tmpsiz - s,
-                                             "%-4d %7d %9d %6s %12d %9d %8s %7d %s\n",
-                                             nrefs, nrtrrefs, alive_cnt,
-                                             alive ? "up" : "down", last_ping,
-                                             pingsent, "NA", down_ni,
-                                             libcfs_nid2str(nid));
-                       else
-                               s += snprintf(s, tmpstr + tmpsiz - s,
-                                             "%-4d %7d %9d %6s %12d %9d %8lu %7d %s\n",
-                                             nrefs, nrtrrefs, alive_cnt,
-                                             alive ? "up" : "down", last_ping,
-                                             pingsent,
-                                             cfs_duration_sec(cfs_time_sub(deadline, now)),
-                                             down_ni, libcfs_nid2str(nid));
-                       LASSERT(tmpstr + tmpsiz - s > 0);
+                       lnet_nid_t nid = peer->lp_primary_nid;
+                       int nrefs     = atomic_read(&peer->lp_refcount);
+                       int nrtrrefs  = peer->lp_rtr_refcount;
+                       int alive     = lnet_is_gateway_alive(peer);
+
+                       s += scnprintf(s, tmpstr + tmpsiz - s,
+                                      "%-4d %7d %5s %s\n",
+                                      nrefs, nrtrrefs,
+                                      alive ? "up" : "down",
+                                      libcfs_nid2str(nid));
                }
 
                lnet_net_unlock(0);
@@ -439,7 +403,7 @@ proc_lnet_peers(struct ctl_table *table, int write, void __user *buffer,
        if (*lenp == 0)
                return 0;
 
-       CLASSERT(LNET_PROC_HASH_BITS >= LNET_PEER_HASH_BITS);
+       BUILD_BUG_ON(LNET_PROC_HASH_BITS < LNET_PEER_HASH_BITS);
 
        if (cpt >= LNET_CPT_NUMBER) {
                *lenp = 0;
@@ -453,10 +417,10 @@ proc_lnet_peers(struct ctl_table *table, int write, void __user *buffer,
        s = tmpstr; /* points to current position in tmpstr[] */
 
        if (*ppos == 0) {
-               s += snprintf(s, tmpstr + tmpsiz - s,
-                             "%-24s %4s %5s %5s %5s %5s %5s %5s %5s %s\n",
-                             "nid", "refs", "state", "last", "max",
-                             "rtr", "min", "tx", "min", "queue");
+               s += scnprintf(s, tmpstr + tmpsiz - s,
+                              "%-24s %4s %5s %5s %5s %5s %5s %5s %5s %s\n",
+                              "nid", "refs", "state", "last", "max",
+                              "rtr", "min", "tx", "min", "queue");
                LASSERT(tmpstr + tmpsiz - s > 0);
 
                hoff++;
@@ -520,8 +484,8 @@ proc_lnet_peers(struct ctl_table *table, int write, void __user *buffer,
 
                if (peer != NULL) {
                        lnet_nid_t nid = peer->lpni_nid;
-                       int nrefs = atomic_read(&peer->lpni_refcount);
-                       int lastalive = -1;
+                       int nrefs = kref_read(&peer->lpni_kref);
+                       time64_t lastalive = -1;
                        char *aliveness = "NA";
                        int maxcr = (peer->lpni_net) ?
                          peer->lpni_net->net_tunables.lct_peer_tx_credits : 0;
@@ -533,30 +497,16 @@ proc_lnet_peers(struct ctl_table *table, int write, void __user *buffer,
 
                        if (lnet_isrouter(peer) ||
                            lnet_peer_aliveness_enabled(peer))
-                               aliveness = peer->lpni_alive ? "up" : "down";
-
-                       if (lnet_peer_aliveness_enabled(peer)) {
-                               cfs_time_t now = cfs_time_current();
-                               cfs_duration_t delta;
-
-                               delta = cfs_time_sub(now, peer->lpni_last_alive);
-                               lastalive = cfs_duration_sec(delta);
-
-                               /* No need to mess up peers contents with
-                                * arbitrarily long integers - it suffices to
-                                * know that lastalive is more than 10000s old
-                                */
-                               if (lastalive >= 10000)
-                                       lastalive = 9999;
-                       }
+                               aliveness = lnet_is_peer_ni_alive(peer) ?
+                                       "up" : "down";
 
                        lnet_net_unlock(cpt);
 
-                       s += snprintf(s, tmpstr + tmpsiz - s,
-                                     "%-24s %4d %5s %5d %5d %5d %5d %5d %5d %d\n",
-                                     libcfs_nid2str(nid), nrefs, aliveness,
-                                     lastalive, maxcr, rtrcr, minrtrcr, txcr,
-                                     mintxcr, txqnob);
+                       s += scnprintf(s, tmpstr + tmpsiz - s,
+                                      "%-24s %4d %5s %5lld %5d %5d %5d %5d %5d %d\n",
+                                      libcfs_nid2str(nid), nrefs, aliveness,
+                                      lastalive, maxcr, rtrcr, minrtrcr, txcr,
+                                      mintxcr, txqnob);
                        LASSERT(tmpstr + tmpsiz - s > 0);
 
                } else { /* peer is NULL */
@@ -612,9 +562,9 @@ static int __proc_lnet_buffers(void *data, int write,
 
        s = tmpstr; /* points to current position in tmpstr[] */
 
-       s += snprintf(s, tmpstr + tmpsiz - s,
-                     "%5s %5s %7s %7s\n",
-                     "pages", "count", "credits", "min");
+       s += scnprintf(s, tmpstr + tmpsiz - s,
+                      "%5s %5s %7s %7s\n",
+                      "pages", "count", "credits", "min");
        LASSERT(tmpstr + tmpsiz - s > 0);
 
        if (the_lnet.ln_rtrpools == NULL)
@@ -625,12 +575,12 @@ static int __proc_lnet_buffers(void *data, int write,
 
                lnet_net_lock(LNET_LOCK_EX);
                cfs_percpt_for_each(rbp, i, the_lnet.ln_rtrpools) {
-                       s += snprintf(s, tmpstr + tmpsiz - s,
-                                     "%5d %5d %7d %7d\n",
-                                     rbp[idx].rbp_npages,
-                                     rbp[idx].rbp_nbuffers,
-                                     rbp[idx].rbp_credits,
-                                     rbp[idx].rbp_mincredits);
+                       s += scnprintf(s, tmpstr + tmpsiz - s,
+                                      "%5d %5d %7d %7d\n",
+                                      rbp[idx].rbp_npages,
+                                      rbp[idx].rbp_nbuffers,
+                                      rbp[idx].rbp_credits,
+                                      rbp[idx].rbp_mincredits);
                        LASSERT(tmpstr + tmpsiz - s > 0);
                }
                lnet_net_unlock(LNET_LOCK_EX);
@@ -713,10 +663,10 @@ proc_lnet_nis(struct ctl_table *table, int write, void __user *buffer,
        s = tmpstr; /* points to current position in tmpstr[] */
 
        if (*ppos == 0) {
-               s += snprintf(s, tmpstr + tmpsiz - s,
-                             "%-24s %6s %5s %4s %4s %4s %5s %5s %5s\n",
-                             "nid", "status", "alive", "refs", "peer",
-                             "rtr", "max", "tx", "min");
+               s += scnprintf(s, tmpstr + tmpsiz - s,
+                              "%-24s %6s %5s %4s %4s %4s %5s %5s %5s\n",
+                              "nid", "status", "alive", "refs", "peer",
+                              "rtr", "max", "tx", "min");
                LASSERT (tmpstr + tmpsiz - s > 0);
        } else {
                struct lnet_ni *ni   = NULL;
@@ -727,19 +677,15 @@ proc_lnet_nis(struct ctl_table *table, int write, void __user *buffer,
                ni = lnet_get_ni_idx_locked(skip);
 
                if (ni != NULL) {
-                       struct lnet_tx_queue    *tq;
-                       char    *stat;
+                       struct lnet_tx_queue *tq;
+                       char *stat;
                        time64_t now = ktime_get_real_seconds();
-                       int     last_alive = -1;
-                       int     i;
-                       int     j;
+                       time64_t last_alive = -1;
+                       int i;
+                       int j;
 
                        if (the_lnet.ln_routing)
-                               last_alive = now - ni->ni_last_alive;
-
-                       /* @lo forever alive */
-                       if (ni->ni_net->net_lnd->lnd_type == LOLND)
-                               last_alive = 0;
+                               last_alive = now - ni->ni_net->net_last_alive;
 
                        lnet_ni_lock(ni);
                        LASSERT(ni->ni_status != NULL);
@@ -747,6 +693,12 @@ proc_lnet_nis(struct ctl_table *table, int write, void __user *buffer,
                                LNET_NI_STATUS_UP) ? "up" : "down";
                        lnet_ni_unlock(ni);
 
+                       /* @lo forever alive */
+                       if (ni->ni_net->net_lnd->lnd_type == LOLND) {
+                               last_alive = 0;
+                               stat = "up";
+                       }
+
                        /* we actually output credits information for
                         * TX queue of each partition */
                        cfs_percpt_for_each(tq, i, ni->ni_tx_queues) {
@@ -762,14 +714,14 @@ proc_lnet_nis(struct ctl_table *table, int write, void __user *buffer,
                                if (i != 0)
                                        lnet_net_lock(i);
 
-                               s += snprintf(s, tmpstr + tmpsiz - s,
-                                     "%-24s %6s %5d %4d %4d %4d %5d %5d %5d\n",
-                                     libcfs_nid2str(ni->ni_nid), stat,
-                                     last_alive, *ni->ni_refs[i],
-                                     ni->ni_net->net_tunables.lct_peer_tx_credits,
-                                     ni->ni_net->net_tunables.lct_peer_rtr_credits,
-                                     tq->tq_credits_max,
-                                     tq->tq_credits, tq->tq_credits_min);
+                               s += scnprintf(s, tmpstr + tmpsiz - s,
+                                      "%-24s %6s %5lld %4d %4d %4d %5d %5d %5d\n",
+                                      libcfs_nid2str(ni->ni_nid), stat,
+                                      last_alive, *ni->ni_refs[i],
+                                      ni->ni_net->net_tunables.lct_peer_tx_credits,
+                                      ni->ni_net->net_tunables.lct_peer_rtr_credits,
+                                      tq->tq_credits_max,
+                                      tq->tq_credits, tq->tq_credits_min);
                                if (i != 0)
                                        lnet_net_unlock(i);
                        }
@@ -844,11 +796,11 @@ static int __proc_lnet_portal_rotor(void *data, int write,
        int             rc;
        int             i;
 
-       LIBCFS_ALLOC(buf, buf_len);
-       if (buf == NULL)
-               return -ENOMEM;
-
        if (!write) {
+               LIBCFS_ALLOC(buf, buf_len);
+               if (buf == NULL)
+                       return -ENOMEM;
+
                lnet_res_lock(0);
 
                for (i = 0; portal_rotors[i].pr_value >= 0; i++) {
@@ -859,26 +811,28 @@ static int __proc_lnet_portal_rotor(void *data, int write,
                LASSERT(portal_rotors[i].pr_value == portal_rotor);
                lnet_res_unlock(0);
 
-               rc = snprintf(buf, buf_len,
-                             "{\n\tportals: all\n"
-                             "\trotor: %s\n\tdescription: %s\n}",
-                             portal_rotors[i].pr_name,
-                             portal_rotors[i].pr_desc);
+               rc = scnprintf(buf, buf_len,
+                              "{\n\tportals: all\n"
+                              "\trotor: %s\n\tdescription: %s\n}",
+                              portal_rotors[i].pr_name,
+                              portal_rotors[i].pr_desc);
 
                if (pos >= min_t(int, rc, buf_len)) {
                        rc = 0;
                } else {
                        rc = cfs_trace_copyout_string(buffer, nob,
-                                       buf + pos, "\n");
+                                                     buf + pos, "\n");
                }
-               goto out;
+               LIBCFS_FREE(buf, buf_len);
+
+               return rc;
        }
 
-       rc = cfs_trace_copyin_string(buf, buf_len, buffer, nob);
-       if (rc < 0)
-               goto out;
+       buf = memdup_user_nul(buffer, nob);
+       if (!buf)
+               return -ENOMEM;
 
-       tmp = cfs_trimwhite(buf);
+       tmp = strim(buf);
 
        rc = -EINVAL;
        lnet_res_lock(0);
@@ -891,8 +845,8 @@ static int __proc_lnet_portal_rotor(void *data, int write,
                }
        }
        lnet_res_unlock(0);
-out:
-       LIBCFS_FREE(buf, buf_len);
+       kfree(buf);
+
        return rc;
 }
 
@@ -911,78 +865,56 @@ static struct ctl_table lnet_table[] = {
         * to go via /proc for portability.
         */
        {
-               INIT_CTL_NAME
                .procname       = "stats",
                .mode           = 0644,
                .proc_handler   = &proc_lnet_stats,
        },
        {
-               INIT_CTL_NAME
                .procname       = "routes",
                .mode           = 0444,
                .proc_handler   = &proc_lnet_routes,
        },
        {
-               INIT_CTL_NAME
                .procname       = "routers",
                .mode           = 0444,
                .proc_handler   = &proc_lnet_routers,
        },
        {
-               INIT_CTL_NAME
                .procname       = "peers",
                .mode           = 0644,
                .proc_handler   = &proc_lnet_peers,
        },
        {
-               INIT_CTL_NAME
                .procname       = "buffers",
                .mode           = 0444,
                .proc_handler   = &proc_lnet_buffers,
        },
        {
-               INIT_CTL_NAME
                .procname       = "nis",
                .mode           = 0644,
                .proc_handler   = &proc_lnet_nis,
        },
        {
-               INIT_CTL_NAME
                .procname       = "portal_rotor",
                .mode           = 0644,
                .proc_handler   = &proc_lnet_portal_rotor,
        },
-       { 0 }
-};
-
-static struct ctl_table top_table[] = {
        {
-               INIT_CTL_NAME
-               .procname       = "lnet",
-               .mode           = 0555,
-               .data           = NULL,
-               .maxlen         = 0,
-               .child          = lnet_table,
+               .procname       = "lnet_lnd_timeout",
+               .data           = &lnet_lnd_timeout,
+               .maxlen         = sizeof(lnet_lnd_timeout),
+               .mode           = 0444,
+               .proc_handler   = &debugfs_doint,
        },
-       { 0 }
+       { .procname = NULL }
 };
 
-void
-lnet_proc_init(void)
+void lnet_router_debugfs_init(void)
 {
-#ifdef CONFIG_SYSCTL
-       if (lnet_table_header == NULL)
-               lnet_table_header = register_sysctl_table(top_table);
-#endif
+       lnet_insert_debugfs(lnet_table);
 }
 
-void
-lnet_proc_fini(void)
+void lnet_router_debugfs_fini(void)
 {
-#ifdef CONFIG_SYSCTL
-       if (lnet_table_header != NULL)
-               unregister_sysctl_table(lnet_table_header);
-
-       lnet_table_header = NULL;
-#endif
+       lnet_remove_debugfs(lnet_table);
 }