Whamcloud - gitweb
LU-12635 build: Support for gcc -Wimplicit-fallthrough
[fs/lustre-release.git] / lustre / lod / lproc_lod.c
index b7621f1..cb2c780 100644 (file)
@@ -99,8 +99,19 @@ lod_dom_stripesize_seq_write(struct file *file, const char __user *buffer,
        /* 1GB is the limit */
        if (val > (1ULL << 30))
                return -ERANGE;
-       else if (val > 0)
-               lod_fix_desc_stripe_size(&val);
+       else if (val > 0) {
+               if (val < LOV_MIN_STRIPE_SIZE) {
+                       LCONSOLE_INFO("Increasing provided stripe size to "
+                                     "a minimum value %u\n",
+                                     LOV_MIN_STRIPE_SIZE);
+                       val = LOV_MIN_STRIPE_SIZE;
+               } else if (val & (LOV_MIN_STRIPE_SIZE - 1)) {
+                       val &= ~(LOV_MIN_STRIPE_SIZE - 1);
+                       LCONSOLE_WARN("Changing provided stripe size to %llu "
+                                     "(a multiple of minimum %u)\n",
+                                     val, LOV_MIN_STRIPE_SIZE);
+               }
+       }
 
        lod->lod_dom_max_stripesize = val;
 
@@ -176,15 +187,14 @@ LPROC_SEQ_FOPS(lod_stripesize);
  * \retval 0           on success
  * \retval negative    error code if failed
  */
-static int lod_stripeoffset_seq_show(struct seq_file *m, void *v)
+static ssize_t stripeoffset_show(struct kobject *kobj, struct attribute *attr,
+                                char *buf)
 {
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
 
-       LASSERT(dev != NULL);
-       lod  = lu2lod_dev(dev->obd_lu_dev);
-       seq_printf(m, "%lld\n", lod->lod_desc.ld_default_stripe_offset);
-       return 0;
+       return sprintf(buf, "%lld\n", lod->lod_desc.ld_default_stripe_offset);
 }
 
 /**
@@ -201,21 +211,19 @@ static int lod_stripeoffset_seq_show(struct seq_file *m, void *v)
  * \retval @count      on success
  * \retval negative    error code if failed
  */
-static ssize_t
-lod_stripeoffset_seq_write(struct file *file, const char __user *buffer,
-                          size_t count, loff_t *off)
+static ssize_t stripeoffset_store(struct kobject *kobj, struct attribute *attr,
+                                 const char *buffer, size_t count)
 {
-       struct seq_file *m = file->private_data;
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
        long val;
        int rc;
 
-       LASSERT(dev != NULL);
-       lod  = lu2lod_dev(dev->obd_lu_dev);
-       rc = kstrtol_from_user(buffer, count, 0, &val);
+       rc = kstrtol(buffer, 0, &val);
        if (rc)
                return rc;
+
        if (val < -1 || val > LOV_MAX_STRIPE_COUNT)
                return -ERANGE;
 
@@ -223,7 +231,7 @@ lod_stripeoffset_seq_write(struct file *file, const char __user *buffer,
 
        return count;
 }
-LPROC_SEQ_FOPS(lod_stripeoffset);
+LUSTRE_RW_ATTR(stripeoffset);
 
 /**
  * Show default striping pattern (LOV_PATTERN_*).
@@ -234,15 +242,14 @@ LPROC_SEQ_FOPS(lod_stripeoffset);
  * \retval 0           on success
  * \retval negative    error code if failed
  */
-static int lod_stripetype_seq_show(struct seq_file *m, void *v)
+static ssize_t stripetype_show(struct kobject *kobj, struct attribute *attr,
+                              char *buf)
 {
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
 
-       LASSERT(dev != NULL);
-       lod  = lu2lod_dev(dev->obd_lu_dev);
-       seq_printf(m, "%u\n", lod->lod_desc.ld_pattern);
-       return 0;
+       return sprintf(buf, "%u\n", lod->lod_desc.ld_pattern);
 }
 
 /**
@@ -257,19 +264,16 @@ static int lod_stripetype_seq_show(struct seq_file *m, void *v)
  * \retval @count      on success
  * \retval negative    error code if failed
  */
-static ssize_t
-lod_stripetype_seq_write(struct file *file, const char __user *buffer,
-                        size_t count, loff_t *off)
+static ssize_t stripetype_store(struct kobject *kobj, struct attribute *attr,
+                               const char *buffer, size_t count)
 {
-       struct seq_file *m = file->private_data;
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
        u32 pattern;
        int rc;
 
-       LASSERT(dev != NULL);
-       lod  = lu2lod_dev(dev->obd_lu_dev);
-       rc = kstrtouint_from_user(buffer, count, 0, &pattern);
+       rc = kstrtouint(buffer, 0, &pattern);
        if (rc)
                return rc;
 
@@ -278,7 +282,7 @@ lod_stripetype_seq_write(struct file *file, const char __user *buffer,
 
        return count;
 }
-LPROC_SEQ_FOPS(lod_stripetype);
+LUSTRE_RW_ATTR(stripetype);
 
 /**
  * Show default number of stripes.
@@ -289,16 +293,15 @@ LPROC_SEQ_FOPS(lod_stripetype);
  * \retval 0           on success,
  * \retval negative    error code if failed
  */
-static int lod_stripecount_seq_show(struct seq_file *m, void *v)
+static ssize_t stripecount_show(struct kobject *kobj, struct attribute *attr,
+                               char *buf)
 {
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
 
-       LASSERT(dev != NULL);
-       lod  = lu2lod_dev(dev->obd_lu_dev);
-       seq_printf(m, "%d\n",
-                  (__s16)(lod->lod_desc.ld_default_stripe_count + 1) - 1);
-       return 0;
+       return sprintf(buf, "%d\n",
+                      (s16)(lod->lod_desc.ld_default_stripe_count + 1) - 1);
 }
 
 /**
@@ -313,28 +316,28 @@ static int lod_stripecount_seq_show(struct seq_file *m, void *v)
  * \retval @count      on success
  * \retval negative    error code otherwise
  */
-static ssize_t
-lod_stripecount_seq_write(struct file *file, const char __user *buffer,
-                         size_t count, loff_t *off)
+static ssize_t stripecount_store(struct kobject *kobj, struct attribute *attr,
+                                 const char *buffer, size_t count)
 {
-       struct seq_file *m = file->private_data;
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
-       u32 stripe_count;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
+       int stripe_count;
        int rc;
 
-       LASSERT(dev != NULL);
-       lod = lu2lod_dev(dev->obd_lu_dev);
-       rc = kstrtouint_from_user(buffer, count, 0, &stripe_count);
+       rc = kstrtoint(buffer, 0, &stripe_count);
        if (rc)
                return rc;
 
+       if (stripe_count < -1)
+               return -ERANGE;
+
        lod_fix_desc_stripe_count(&stripe_count);
        lod->lod_desc.ld_default_stripe_count = stripe_count;
 
        return count;
 }
-LPROC_SEQ_FOPS(lod_stripecount);
+LUSTRE_RW_ATTR(stripecount);
 
 /**
  * Show number of targets.
@@ -345,17 +348,16 @@ LPROC_SEQ_FOPS(lod_stripecount);
  * \retval 0           on success
  * \retval negative    error code if failed
  */
-static int lod_numobd_seq_show(struct seq_file *m, void *v)
+static ssize_t numobd_show(struct kobject *kobj, struct attribute *attr,
+                          char *buf)
 {
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
 
-       LASSERT(dev != NULL);
-       lod  = lu2lod_dev(dev->obd_lu_dev);
-       seq_printf(m, "%u\n", lod->lod_desc.ld_tgt_count);
-       return 0;
+       return sprintf(buf, "%u\n", lod->lod_desc.ld_tgt_count);
 }
-LPROC_SEQ_FOPS_RO(lod_numobd);
+LUSTRE_RO_ATTR(numobd);
 
 /**
  * Show number of active targets.
@@ -366,17 +368,16 @@ LPROC_SEQ_FOPS_RO(lod_numobd);
  * \retval 0           on success
  * \retval negative    error code if failed
  */
-static int lod_activeobd_seq_show(struct seq_file *m, void *v)
+static ssize_t activeobd_show(struct kobject *kobj, struct attribute *attr,
+                             char *buf)
 {
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
 
-       LASSERT(dev != NULL);
-       lod  = lu2lod_dev(dev->obd_lu_dev);
-       seq_printf(m, "%u\n", lod->lod_desc.ld_active_tgt_count);
-       return 0;
+       return sprintf(buf, "%u\n", lod->lod_desc.ld_active_tgt_count);
 }
-LPROC_SEQ_FOPS_RO(lod_activeobd);
+LUSTRE_RO_ATTR(activeobd);
 
 /**
  * Show UUID of LOD device.
@@ -387,17 +388,16 @@ LPROC_SEQ_FOPS_RO(lod_activeobd);
  * \retval 0           on success
  * \retval negative    error code if failed
  */
-static int lod_desc_uuid_seq_show(struct seq_file *m, void *v)
+static ssize_t desc_uuid_show(struct kobject *kobj, struct attribute *attr,
+                             char *buf)
 {
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
 
-       LASSERT(dev != NULL);
-       lod  = lu2lod_dev(dev->obd_lu_dev);
-       seq_printf(m, "%s\n", lod->lod_desc.ld_uuid.uuid);
-       return 0;
+       return sprintf(buf, "%s\n", lod->lod_desc.ld_uuid.uuid);
 }
-LPROC_SEQ_FOPS_RO(lod_desc_uuid);
+LUSTRE_RO_ATTR(desc_uuid);
 
 /**
  * Show QoS priority parameter.
@@ -413,15 +413,15 @@ LPROC_SEQ_FOPS_RO(lod_desc_uuid);
  * \retval 0           on success
  * \retval negative    error code if failed
  */
-static int lod_qos_priofree_seq_show(struct seq_file *m, void *v)
+static ssize_t qos_prio_free_show(struct kobject *kobj, struct attribute *attr,
+                                 char *buf)
 {
-       struct obd_device *dev = m->private;
-       struct lod_device *lod = lu2lod_dev(dev->obd_lu_dev);
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
 
-       LASSERT(lod != NULL);
-       seq_printf(m, "%d%%\n",
-                  (lod->lod_qos.lq_prio_free * 100 + 255) >> 8);
-       return 0;
+       return sprintf(buf, "%d%%\n",
+                      (lod->lod_qos.lq_prio_free * 100 + 255) >> 8);
 }
 
 /**
@@ -440,20 +440,16 @@ static int lod_qos_priofree_seq_show(struct seq_file *m, void *v)
  * \retval @count      on success
  * \retval negative    error code if failed
  */
-static ssize_t
-lod_qos_priofree_seq_write(struct file *file, const char __user *buffer,
-                          size_t count, loff_t *off)
+static ssize_t qos_prio_free_store(struct kobject *kobj, struct attribute *attr,
+                                  const char *buffer, size_t count)
 {
-       struct seq_file *m = file->private_data;
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
        unsigned int val;
        int rc;
 
-       LASSERT(dev != NULL);
-       lod = lu2lod_dev(dev->obd_lu_dev);
-
-       rc = kstrtouint_from_user(buffer, count, 0, &val);
+       rc = kstrtouint(buffer, 0, &val);
        if (rc)
                return rc;
 
@@ -465,7 +461,7 @@ lod_qos_priofree_seq_write(struct file *file, const char __user *buffer,
 
        return count;
 }
-LPROC_SEQ_FOPS(lod_qos_priofree);
+LUSTRE_RW_ATTR(qos_prio_free);
 
 /**
  * Show threshold for "same space on all OSTs" rule.
@@ -542,15 +538,14 @@ LPROC_SEQ_FOPS(lod_qos_thresholdrr);
  * \retval 0           on success
  * \retval negative    error code if failed
  */
-static int lod_qos_maxage_seq_show(struct seq_file *m, void *v)
+static ssize_t qos_maxage_show(struct kobject *kobj, struct attribute *attr,
+                              char *buf)
 {
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
 
-       LASSERT(dev != NULL);
-       lod = lu2lod_dev(dev->obd_lu_dev);
-       seq_printf(m, "%u Sec\n", lod->lod_desc.ld_qos_maxage);
-       return 0;
+       return sprintf(buf, "%u Sec\n", lod->lod_desc.ld_qos_maxage);
 }
 
 /**
@@ -564,14 +559,13 @@ static int lod_qos_maxage_seq_show(struct seq_file *m, void *v)
  * \retval @count      on success
  * \retval negative    error code if failed
  */
-static ssize_t
-lod_qos_maxage_seq_write(struct file *file, const char __user *buffer,
-                        size_t count, loff_t *off)
+static ssize_t qos_maxage_store(struct kobject *kobj, struct attribute *attr,
+                               const char *buffer, size_t count)
 {
-       struct seq_file *m = file->private_data;
-       struct obd_device *dev = m->private;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
        struct lustre_cfg_bufs bufs;
-       struct lod_device *lod;
        struct lu_device *next;
        struct lustre_cfg *lcfg;
        char str[32];
@@ -579,10 +573,7 @@ lod_qos_maxage_seq_write(struct file *file, const char __user *buffer,
        int rc;
        u32 val;
 
-       LASSERT(dev != NULL);
-       lod = lu2lod_dev(dev->obd_lu_dev);
-
-       rc = kstrtouint_from_user(buffer, count, 0, &val);
+       rc = kstrtouint(buffer, 0, &val);
        if (rc)
                return rc;
 
@@ -613,7 +604,7 @@ lod_qos_maxage_seq_write(struct file *file, const char __user *buffer,
 
        return count;
 }
-LPROC_SEQ_FOPS(lod_qos_maxage);
+LUSTRE_RW_ATTR(qos_maxage);
 
 static void *lod_osts_seq_start(struct seq_file *p, loff_t *pos)
 {
@@ -723,13 +714,6 @@ static int lod_osts_seq_open(struct inode *inode, struct file *file)
        return 0;
 }
 
-LPROC_SEQ_FOPS_RO_TYPE(lod, dt_blksize);
-LPROC_SEQ_FOPS_RO_TYPE(lod, dt_kbytestotal);
-LPROC_SEQ_FOPS_RO_TYPE(lod, dt_kbytesfree);
-LPROC_SEQ_FOPS_RO_TYPE(lod, dt_kbytesavail);
-LPROC_SEQ_FOPS_RO_TYPE(lod, dt_filestotal);
-LPROC_SEQ_FOPS_RO_TYPE(lod, dt_filesfree);
-
 /**
  * Show whether special failout mode for testing is enabled or not.
  *
@@ -739,16 +723,14 @@ LPROC_SEQ_FOPS_RO_TYPE(lod, dt_filesfree);
  * \retval 0           on success
  * \retval negative    error code if failed
  */
-static int lod_lmv_failout_seq_show(struct seq_file *m, void *v)
+static ssize_t lmv_failout_show(struct kobject *kobj, struct attribute *attr,
+                               char *buf)
 {
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
 
-       LASSERT(dev != NULL);
-       lod = lu2lod_dev(dev->obd_lu_dev);
-
-       seq_printf(m, "%d\n", lod->lod_lmv_failout ? 1 : 0);
-       return 0;
+       return sprintf(buf, "%d\n", lod->lod_lmv_failout ? 1 : 0);
 }
 
 /**
@@ -766,20 +748,16 @@ static int lod_lmv_failout_seq_show(struct seq_file *m, void *v)
  * \retval @count      on success
  * \retval negative    error code if failed
  */
-static ssize_t
-lod_lmv_failout_seq_write(struct file *file, const char __user *buffer,
-                         size_t count, loff_t *off)
+static ssize_t lmv_failout_store(struct kobject *kobj, struct attribute *attr,
+                                const char *buffer, size_t count)
 {
-       struct seq_file *m = file->private_data;
-       struct obd_device *dev = m->private;
-       struct lod_device *lod;
+       struct dt_device *dt = container_of(kobj, struct dt_device,
+                                           dd_kobj);
+       struct lod_device *lod = dt2lod_dev(dt);
        bool val = 0;
        int rc;
 
-       LASSERT(dev != NULL);
-       lod = lu2lod_dev(dev->obd_lu_dev);
-
-       rc = kstrtobool_from_user(buffer, count, &val);
+       rc = kstrtobool(buffer, &val);
        if (rc)
                return rc;
 
@@ -787,48 +765,18 @@ lod_lmv_failout_seq_write(struct file *file, const char __user *buffer,
 
        return count;
 }
-LPROC_SEQ_FOPS(lod_lmv_failout);
+LUSTRE_RW_ATTR(lmv_failout);
 
 static struct lprocfs_vars lprocfs_lod_obd_vars[] = {
        { .name =       "stripesize",
          .fops =       &lod_stripesize_fops    },
-       { .name =       "stripeoffset",
-         .fops =       &lod_stripeoffset_fops  },
-       { .name =       "stripecount",
-         .fops =       &lod_stripecount_fops   },
-       { .name =       "stripetype",
-         .fops =       &lod_stripetype_fops    },
-       { .name =       "numobd",
-         .fops =       &lod_numobd_fops        },
-       { .name =       "activeobd",
-         .fops =       &lod_activeobd_fops     },
-       { .name =       "desc_uuid",
-         .fops =       &lod_desc_uuid_fops     },
-       { .name =       "qos_prio_free",
-         .fops =       &lod_qos_priofree_fops  },
        { .name =       "qos_threshold_rr",
          .fops =       &lod_qos_thresholdrr_fops },
-       { .name =       "qos_maxage",
-         .fops =       &lod_qos_maxage_fops    },
-       { .name =       "lmv_failout",
-         .fops =       &lod_lmv_failout_fops   },
-       {
-         .name = "dom_stripesize",
-         .fops = &lod_dom_stripesize_fops
-       },
+       { .name =       "dom_stripesize",
+         .fops =       &lod_dom_stripesize_fops        },
        { NULL }
 };
 
-static struct lprocfs_vars lprocfs_lod_osd_vars[] = {
-       { .name = "blocksize",   .fops = &lod_dt_blksize_fops },
-       { .name = "kbytestotal", .fops = &lod_dt_kbytestotal_fops },
-       { .name = "kbytesfree",  .fops = &lod_dt_kbytesfree_fops },
-       { .name = "kbytesavail", .fops = &lod_dt_kbytesavail_fops },
-       { .name = "filestotal",  .fops = &lod_dt_filestotal_fops },
-       { .name = "filesfree",   .fops = &lod_dt_filesfree_fops },
-       { .name = NULL }
-};
-
 static const struct file_operations lod_proc_target_fops = {
        .owner   = THIS_MODULE,
        .open    = lod_osts_seq_open,
@@ -837,6 +785,19 @@ static const struct file_operations lod_proc_target_fops = {
        .release = lprocfs_seq_release,
 };
 
+static struct attribute *lod_attrs[] = {
+       &lustre_attr_stripeoffset.attr,
+       &lustre_attr_stripecount.attr,
+       &lustre_attr_stripetype.attr,
+       &lustre_attr_activeobd.attr,
+       &lustre_attr_desc_uuid.attr,
+       &lustre_attr_lmv_failout.attr,
+       &lustre_attr_numobd.attr,
+       &lustre_attr_qos_maxage.attr,
+       &lustre_attr_qos_prio_free.attr,
+       NULL,
+};
+
 /**
  * Initialize procfs entries for LOD.
  *
@@ -847,24 +808,28 @@ static const struct file_operations lod_proc_target_fops = {
  */
 int lod_procfs_init(struct lod_device *lod)
 {
+       struct lprocfs_vars ldebugfs_obd_vars[] = { { NULL } };
        struct obd_device *obd = lod2obd(lod);
-       struct proc_dir_entry *lov_proc_dir;
        struct obd_type *type;
        struct kobject *lov;
        int rc;
 
-       obd->obd_vars = lprocfs_lod_obd_vars;
-       rc = lprocfs_obd_setup(obd, true);
+       lod->lod_dt_dev.dd_ktype.default_attrs = lod_attrs;
+       rc = dt_tunables_init(&lod->lod_dt_dev, obd->obd_type, obd->obd_name,
+                             ldebugfs_obd_vars);
        if (rc) {
-               CERROR("%s: cannot setup procfs entry: %d\n",
+               CERROR("%s: failed to setup DT tunables: %d\n",
                       obd->obd_name, rc);
                RETURN(rc);
        }
 
-       rc = lprocfs_add_vars(obd->obd_proc_entry, lprocfs_lod_osd_vars,
-                             &lod->lod_dt_dev);
-       if (rc) {
-               CERROR("%s: cannot setup procfs entry: %d\n",
+       obd->obd_vars = lprocfs_lod_obd_vars;
+       obd->obd_proc_entry = lprocfs_register(obd->obd_name,
+                                              obd->obd_type->typ_procroot,
+                                              obd->obd_vars, obd);
+       if (IS_ERR(obd->obd_proc_entry)) {
+               rc = PTR_ERR(obd->obd_proc_entry);
+               CERROR("%s: error %d setting up lprocfs\n",
                       obd->obd_name, rc);
                GOTO(out, rc);
        }
@@ -889,32 +854,31 @@ int lod_procfs_init(struct lod_device *lod)
        }
 
        lov = kset_find_obj(lustre_kset, "lov");
-       if (lov) {
-               rc = sysfs_create_link(lov, &obd->obd_kset.kobj,
-                                      obd->obd_name);
-               kobject_put(lov);
+       if (!lov) {
+               CERROR("%s: lov subsystem not found\n", obd->obd_name);
+               GOTO(out, rc = -ENODEV);
        }
 
+       rc = sysfs_create_link(lov, &lod->lod_dt_dev.dd_kobj,
+                              obd->obd_name);
+       if (rc)
+               CERROR("%s: failed to create LOV sysfs symlink\n",
+                      obd->obd_name);
+       kobject_put(lov);
+
        lod->lod_debugfs = ldebugfs_add_symlink(obd->obd_name, "lov",
                                                "../lod/%s", obd->obd_name);
        if (!lod->lod_debugfs)
                CERROR("%s: failed to create LOV debugfs symlink\n",
                       obd->obd_name);
 
-       /* If the real LOV is present which is the case for setups
-        * with both server and clients on the same node then use
-        * the LOV's proc root */
-       type = class_search_type(LUSTRE_LOV_NAME);
-       if (type != NULL && type->typ_procroot != NULL)
-               lov_proc_dir = type->typ_procroot;
-       else
-               lov_proc_dir = obd->obd_type->typ_procsym;
-
-       if (lov_proc_dir == NULL)
+       type = container_of(lov, struct obd_type, typ_kobj);
+       if (!type->typ_procroot)
                RETURN(0);
 
        /* for compatibility we link old procfs's LOV entries to lod ones */
-       lod->lod_symlink = lprocfs_add_symlink(obd->obd_name, lov_proc_dir,
+       lod->lod_symlink = lprocfs_add_symlink(obd->obd_name,
+                                              type->typ_procroot,
                                               "../lod/%s", obd->obd_name);
        if (lod->lod_symlink == NULL)
                CERROR("cannot create LOV symlink for /proc/fs/lustre/lod/%s\n",
@@ -922,7 +886,7 @@ int lod_procfs_init(struct lod_device *lod)
        RETURN(0);
 
 out:
-       lprocfs_obd_cleanup(obd);
+       dt_tunables_fini(&lod->lod_dt_dev);
 
        return rc;
 }
@@ -951,7 +915,12 @@ void lod_procfs_fini(struct lod_device *lod)
        if (!IS_ERR_OR_NULL(lod->lod_debugfs))
                ldebugfs_remove(&lod->lod_debugfs);
 
-       lprocfs_obd_cleanup(obd);
+       if (obd->obd_proc_entry) {
+               lprocfs_remove(&obd->obd_proc_entry);
+               obd->obd_proc_entry = NULL;
+       }
+
+       dt_tunables_fini(&lod->lod_dt_dev);
 }
 
 #endif /* CONFIG_PROC_FS */