Whamcloud - gitweb
LU-7243 misc: update Intel copyright messages 2015
[fs/lustre-release.git] / lustre / obdclass / dt_object.c
index 13962cd..65649be 100644 (file)
@@ -27,7 +27,7 @@
  * Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2011, 2012, Intel Corporation.
+ * Copyright (c) 2011, 2015, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -60,85 +60,104 @@ struct lu_context_key dt_key = {
         .lct_init = dt_global_key_init,
         .lct_fini = dt_global_key_fini
 };
-EXPORT_SYMBOL(dt_key);
 
 /* no lock is necessary to protect the list, because call-backs
  * are added during system startup. Please refer to "struct dt_device".
  */
 void dt_txn_callback_add(struct dt_device *dev, struct dt_txn_callback *cb)
 {
-        cfs_list_add(&cb->dtc_linkage, &dev->dd_txn_callbacks);
+       list_add(&cb->dtc_linkage, &dev->dd_txn_callbacks);
 }
 EXPORT_SYMBOL(dt_txn_callback_add);
 
 void dt_txn_callback_del(struct dt_device *dev, struct dt_txn_callback *cb)
 {
-        cfs_list_del_init(&cb->dtc_linkage);
+       list_del_init(&cb->dtc_linkage);
 }
 EXPORT_SYMBOL(dt_txn_callback_del);
 
 int dt_txn_hook_start(const struct lu_env *env,
                       struct dt_device *dev, struct thandle *th)
 {
-        int rc = 0;
-        struct dt_txn_callback *cb;
+       int rc = 0;
+       struct dt_txn_callback *cb;
 
-        if (th->th_local)
-                return 0;
+       if (th->th_local)
+               return 0;
 
-        cfs_list_for_each_entry(cb, &dev->dd_txn_callbacks, dtc_linkage) {
-                if (cb->dtc_txn_start == NULL ||
-                    !(cb->dtc_tag & env->le_ctx.lc_tags))
-                        continue;
-                rc = cb->dtc_txn_start(env, th, cb->dtc_cookie);
-                if (rc < 0)
-                        break;
-        }
-        return rc;
+       list_for_each_entry(cb, &dev->dd_txn_callbacks, dtc_linkage) {
+               struct thandle *dtc_th = th;
+
+               if (cb->dtc_txn_start == NULL ||
+                   !(cb->dtc_tag & env->le_ctx.lc_tags))
+                       continue;
+
+               /* Usually dt_txn_hook_start is called from bottom device,
+                * and if the thandle has th_top, then we need use top
+                * thandle for the callback in the top thandle layer */
+               if (th->th_top != NULL)
+                       dtc_th = th->th_top;
+
+               rc = cb->dtc_txn_start(env, dtc_th, cb->dtc_cookie);
+               if (rc < 0)
+                       break;
+       }
+       return rc;
 }
 EXPORT_SYMBOL(dt_txn_hook_start);
 
-int dt_txn_hook_stop(const struct lu_env *env, struct thandle *txn)
+int dt_txn_hook_stop(const struct lu_env *env, struct thandle *th)
 {
-        struct dt_device       *dev = txn->th_dev;
-        struct dt_txn_callback *cb;
-        int                     rc = 0;
+       struct dt_device       *dev = th->th_dev;
+       struct dt_txn_callback *cb;
+       int                     rc = 0;
 
-        if (txn->th_local)
-                return 0;
+       if (th->th_local)
+               return 0;
 
-        cfs_list_for_each_entry(cb, &dev->dd_txn_callbacks, dtc_linkage) {
-                if (cb->dtc_txn_stop == NULL ||
-                    !(cb->dtc_tag & env->le_ctx.lc_tags))
-                        continue;
-                rc = cb->dtc_txn_stop(env, txn, cb->dtc_cookie);
-                if (rc < 0)
-                        break;
-        }
-        return rc;
+       list_for_each_entry(cb, &dev->dd_txn_callbacks, dtc_linkage) {
+               struct thandle *dtc_th = th;
+
+               if (cb->dtc_txn_stop == NULL ||
+                   !(cb->dtc_tag & env->le_ctx.lc_tags))
+                       continue;
+
+               /* Usually dt_txn_hook_stop is called from bottom device,
+                * and if the thandle has th_top, then we need use top
+                * thandle for the callback in the top thandle layer */
+               if (th->th_top != NULL)
+                       dtc_th = th->th_top;
+
+               rc = cb->dtc_txn_stop(env, dtc_th, cb->dtc_cookie);
+               if (rc < 0)
+                       break;
+       }
+       return rc;
 }
 EXPORT_SYMBOL(dt_txn_hook_stop);
 
-void dt_txn_hook_commit(struct thandle *txn)
+void dt_txn_hook_commit(struct thandle *th)
 {
-        struct dt_txn_callback *cb;
-
-        if (txn->th_local)
-                return;
-
-        cfs_list_for_each_entry(cb, &txn->th_dev->dd_txn_callbacks,
-                                dtc_linkage) {
-                if (cb->dtc_txn_commit)
-                        cb->dtc_txn_commit(txn, cb->dtc_cookie);
-        }
+       struct dt_txn_callback *cb;
+
+       if (th->th_local)
+               return;
+
+       list_for_each_entry(cb, &th->th_dev->dd_txn_callbacks,
+                           dtc_linkage) {
+               /* Right now, the bottom device (OSD) will use this hook
+                * commit to notify OSP, so we do not check and replace
+                * the thandle to top thandle now */
+               if (cb->dtc_txn_commit)
+                       cb->dtc_txn_commit(th, cb->dtc_cookie);
+       }
 }
 EXPORT_SYMBOL(dt_txn_hook_commit);
 
 int dt_device_init(struct dt_device *dev, struct lu_device_type *t)
 {
-
-        CFS_INIT_LIST_HEAD(&dev->dd_txn_callbacks);
-        return lu_device_init(&dev->dd_lu_dev, t);
+       INIT_LIST_HEAD(&dev->dd_txn_callbacks);
+       return lu_device_init(&dev->dd_lu_dev, t);
 }
 EXPORT_SYMBOL(dt_device_init);
 
@@ -205,38 +224,41 @@ EXPORT_SYMBOL(dt_mode_to_dft);
 int dt_lookup_dir(const struct lu_env *env, struct dt_object *dir,
                   const char *name, struct lu_fid *fid)
 {
-        if (dt_try_as_dir(env, dir))
-                return dt_lookup(env, dir, (struct dt_rec *)fid,
-                                 (const struct dt_key *)name, BYPASS_CAPA);
-        return -ENOTDIR;
+       if (dt_try_as_dir(env, dir))
+               return dt_lookup(env, dir, (struct dt_rec *)fid,
+                                (const struct dt_key *)name);
+       return -ENOTDIR;
 }
 EXPORT_SYMBOL(dt_lookup_dir);
 
 /* this differs from dt_locate by top_dev as parameter
  * but not one from lu_site */
 struct dt_object *dt_locate_at(const struct lu_env *env,
-                              struct dt_device *dev, const struct lu_fid *fid,
-                              struct lu_device *top_dev)
+                              struct dt_device *dev,
+                              const struct lu_fid *fid,
+                              struct lu_device *top_dev,
+                              const struct lu_object_conf *conf)
 {
-       struct lu_object *lo, *n;
-       ENTRY;
+       struct lu_object *lo;
+       struct lu_object *n;
 
-       lo = lu_object_find_at(env, top_dev, fid, NULL);
+       lo = lu_object_find_at(env, top_dev, fid, conf);
        if (IS_ERR(lo))
-               return (void *)lo;
+               return ERR_PTR(PTR_ERR(lo));
 
        LASSERT(lo != NULL);
 
-       cfs_list_for_each_entry(n, &lo->lo_header->loh_layers, lo_linkage) {
+       list_for_each_entry(n, &lo->lo_header->loh_layers, lo_linkage) {
                if (n->lo_dev == &dev->dd_lu_dev)
                        return container_of0(n, struct dt_object, do_lu);
        }
+
        return ERR_PTR(-ENOENT);
 }
 EXPORT_SYMBOL(dt_locate_at);
 
 /**
- * find a object named \a entry in given \a dfh->dfh_o directory.
+ * find an object named \a entry in given \a dfh->dfh_o directory.
  */
 static int dt_find_entry(const struct lu_env *env, const char *entry, void *data)
 {
@@ -293,22 +315,21 @@ dt_store_resolve(const struct lu_env *env, struct dt_device *dt,
        struct dt_thread_info *info = dt_info(env);
        struct dt_find_hint   *dfh = &info->dti_dfh;
        struct dt_object      *obj;
-       char                  *local = info->dti_buf;
        int                    result;
 
 
         dfh->dfh_dt = dt;
         dfh->dfh_fid = fid;
 
-        strncpy(local, path, DT_MAX_PATH);
-        local[DT_MAX_PATH - 1] = '\0';
+       strlcpy(info->dti_buf, path, sizeof(info->dti_buf));
 
         result = dt->dd_ops->dt_root_get(env, dt, fid);
         if (result == 0) {
                 obj = dt_locate(env, dt, fid);
                 if (!IS_ERR(obj)) {
                         dfh->dfh_o = obj;
-                        result = dt_path_parser(env, local, dt_find_entry, dfh);
+                       result = dt_path_parser(env, info->dti_buf,
+                                               dt_find_entry, dfh);
                         if (result != 0)
                                 obj = ERR_PTR(result);
                         else
@@ -319,7 +340,6 @@ dt_store_resolve(const struct lu_env *env, struct dt_device *dt,
         }
         return obj;
 }
-EXPORT_SYMBOL(dt_store_resolve);
 
 static struct dt_object *dt_reg_open(const struct lu_env *env,
                                      struct dt_device *dt,
@@ -364,7 +384,6 @@ struct dt_object *dt_store_open(const struct lu_env *env,
         }
         return file;
 }
-EXPORT_SYMBOL(dt_store_open);
 
 struct dt_object *dt_find_or_create(const struct lu_env *env,
                                     struct dt_device *dt,
@@ -450,8 +469,8 @@ void dt_global_fini(void)
 int dt_read(const struct lu_env *env, struct dt_object *dt,
             struct lu_buf *buf, loff_t *pos)
 {
-        LASSERTF(dt != NULL, "dt is NULL when we want to read record\n");
-        return dt->do_body_ops->dbo_read(env, dt, buf, pos, BYPASS_CAPA);
+       LASSERTF(dt != NULL, "dt is NULL when we want to read record\n");
+       return dt->do_body_ops->dbo_read(env, dt, buf, pos);
 }
 EXPORT_SYMBOL(dt_read);
 
@@ -471,35 +490,31 @@ EXPORT_SYMBOL(dt_read);
 int dt_record_read(const struct lu_env *env, struct dt_object *dt,
                    struct lu_buf *buf, loff_t *pos)
 {
-        int rc;
-
-        LASSERTF(dt != NULL, "dt is NULL when we want to read record\n");
+       ssize_t size;
 
-        rc = dt->do_body_ops->dbo_read(env, dt, buf, pos, BYPASS_CAPA);
+       LASSERTF(dt != NULL, "dt is NULL when we want to read record\n");
 
-        if (rc == buf->lb_len)
-                rc = 0;
-        else if (rc >= 0)
-                rc = -EFAULT;
-        return rc;
+       size = dt->do_body_ops->dbo_read(env, dt, buf, pos);
+       if (size < 0)
+               return size;
+       return (size == (ssize_t)buf->lb_len) ? 0 : -EFAULT;
 }
 EXPORT_SYMBOL(dt_record_read);
 
 int dt_record_write(const struct lu_env *env, struct dt_object *dt,
                     const struct lu_buf *buf, loff_t *pos, struct thandle *th)
 {
-        int rc;
+       ssize_t size;
 
-        LASSERTF(dt != NULL, "dt is NULL when we want to write record\n");
-        LASSERT(th != NULL);
-        LASSERT(dt->do_body_ops);
-        LASSERT(dt->do_body_ops->dbo_write);
-        rc = dt->do_body_ops->dbo_write(env, dt, buf, pos, th, BYPASS_CAPA, 1);
-        if (rc == buf->lb_len)
-                rc = 0;
-        else if (rc >= 0)
-                rc = -EFAULT;
-        return rc;
+       LASSERTF(dt != NULL, "dt is NULL when we want to write record\n");
+       LASSERT(th != NULL);
+       LASSERT(dt->do_body_ops);
+       LASSERT(dt->do_body_ops->dbo_write);
+
+       size = dt->do_body_ops->dbo_write(env, dt, buf, pos, th, 1);
+       if (size < 0)
+               return size;
+       return (size == (ssize_t)buf->lb_len) ? 0 : -EFAULT;
 }
 EXPORT_SYMBOL(dt_record_write);
 
@@ -528,7 +543,7 @@ void dt_version_set(const struct lu_env *env, struct dt_object *o,
         vbuf.lb_buf = &version;
         vbuf.lb_len = sizeof(version);
 
-        rc = dt_xattr_set(env, o, &vbuf, xname, 0, th, BYPASS_CAPA);
+       rc = dt_xattr_set(env, o, &vbuf, xname, 0, th);
         if (rc < 0)
                 CDEBUG(D_INODE, "Can't set version, rc %d\n", rc);
         return;
@@ -545,7 +560,7 @@ dt_obj_version_t dt_version_get(const struct lu_env *env, struct dt_object *o)
         LASSERT(o);
         vbuf.lb_buf = &version;
         vbuf.lb_len = sizeof(version);
-        rc = dt_xattr_get(env, o, &vbuf, xname, BYPASS_CAPA);
+       rc = dt_xattr_get(env, o, &vbuf, xname);
         if (rc != sizeof(version)) {
                 CDEBUG(D_INODE, "Can't get version, rc %d\n", rc);
                 version = 0;
@@ -564,6 +579,17 @@ EXPORT_SYMBOL(dt_directory_features);
 const struct dt_index_features dt_otable_features;
 EXPORT_SYMBOL(dt_otable_features);
 
+/* lfsck orphan */
+const struct dt_index_features dt_lfsck_orphan_features = {
+       .dif_flags              = 0,
+       .dif_keysize_min        = sizeof(struct lu_fid),
+       .dif_keysize_max        = sizeof(struct lu_fid),
+       .dif_recsize_min        = sizeof(struct lu_orphan_rec),
+       .dif_recsize_max        = sizeof(struct lu_orphan_rec),
+       .dif_ptrsize            = 4
+};
+EXPORT_SYMBOL(dt_lfsck_orphan_features);
+
 /* lfsck */
 const struct dt_index_features dt_lfsck_features = {
        .dif_flags              = DT_IND_UPDATE,
@@ -627,6 +653,8 @@ static inline const struct dt_index_features *dt_index_feat_select(__u64 seq,
                        /* slave index should be a regular file */
                        return ERR_PTR(-ENOENT);
                return &dt_quota_slv_features;
+       } else if (seq == FID_SEQ_LAYOUT_RBTREE){
+               return &dt_lfsck_orphan_features;
        } else if (seq >= FID_SEQ_NORMAL) {
                /* object is part of the namespace, verify that it is a
                 * directory */
@@ -652,18 +680,18 @@ static inline const struct dt_index_features *dt_index_feat_select(__u64 seq,
  * \param arg  - is a pointer to the idx_info structure
  */
 static int dt_index_page_build(const struct lu_env *env, union lu_page *lp,
-                              int nob, const struct dt_it_ops *iops,
+                              size_t nob, const struct dt_it_ops *iops,
                               struct dt_it *it, __u32 attr, void *arg)
 {
        struct idx_info         *ii = (struct idx_info *)arg;
        struct lu_idxpage       *lip = &lp->lp_idx;
        char                    *entry;
-       int                      rc, size;
+       size_t                   size;
+       int                      rc;
        ENTRY;
 
-       /* no support for variable key & record size for now */
-       LASSERT((ii->ii_flags & II_FL_VARKEY) == 0);
-       LASSERT((ii->ii_flags & II_FL_VARREC) == 0);
+       if (nob < LIP_HDR_SIZE)
+               return -EINVAL;
 
        /* initialize the header of the new container */
        memset(lip, 0, LIP_HDR_SIZE);
@@ -680,7 +708,9 @@ static int dt_index_page_build(const struct lu_env *env, union lu_page *lp,
        do {
                char            *tmp_entry = entry;
                struct dt_key   *key;
-               __u64            hash;
+               __u64           hash;
+               __u16           keysize;
+               __u16           recsize;
 
                /* fetch 64-bit hash value */
                hash = iops->store(env, it);
@@ -697,18 +727,24 @@ static int dt_index_page_build(const struct lu_env *env, union lu_page *lp,
                        GOTO(out, rc = 0);
                }
 
-               if ((ii->ii_flags & II_FL_NOHASH) == 0) {
+               if (!(ii->ii_flags & II_FL_NOHASH)) {
                        /* client wants to the 64-bit hash value associated with
                         * each record */
                        memcpy(tmp_entry, &hash, sizeof(hash));
                        tmp_entry += sizeof(hash);
                }
 
-               /* then the key value */
-               LASSERT(iops->key_size(env, it) == ii->ii_keysize);
-               key = iops->key(env, it);
-               memcpy(tmp_entry, key, ii->ii_keysize);
-               tmp_entry += ii->ii_keysize;
+               if (ii->ii_flags & II_FL_VARKEY)
+                       keysize = iops->key_size(env, it);
+               else
+                       keysize = ii->ii_keysize;
+
+               if (!(ii->ii_flags & II_FL_NOKEY)) {
+                       /* then the key value */
+                       key = iops->key(env, it);
+                       memcpy(tmp_entry, key, keysize);
+                       tmp_entry += keysize;
+               }
 
                /* and finally the record */
                rc = iops->rec(env, it, (struct dt_rec *)tmp_entry, attr);
@@ -720,7 +756,13 @@ static int dt_index_page_build(const struct lu_env *env, union lu_page *lp,
                        lip->lip_nr++;
                        if (unlikely(lip->lip_nr == 1 && ii->ii_count == 0))
                                ii->ii_hash_start = hash;
-                       entry = tmp_entry + ii->ii_recsize;
+
+                       if (ii->ii_flags & II_FL_VARREC)
+                               recsize = iops->rec_size(env, it, attr);
+                       else
+                               recsize = ii->ii_recsize;
+
+                       entry = tmp_entry + recsize;
                        nob -= size;
                }
 
@@ -742,6 +784,7 @@ out:
        return rc;
 }
 
+
 /*
  * Walk index and fill lu_page containers with key/record pairs
  *
@@ -761,7 +804,7 @@ int dt_index_walk(const struct lu_env *env, struct dt_object *obj,
 {
        struct dt_it            *it;
        const struct dt_it_ops  *iops;
-       unsigned int             pageidx, nob, nlupgs = 0;
+       size_t                   pageidx, nob, nlupgs = 0;
        int                      rc;
        ENTRY;
 
@@ -769,13 +812,13 @@ int dt_index_walk(const struct lu_env *env, struct dt_object *obj,
        LASSERT(obj->do_index_ops != NULL);
 
        nob = rdpg->rp_count;
-       if (nob <= 0)
+       if (nob == 0)
                RETURN(-EFAULT);
 
        /* Iterate through index and fill containers from @rdpg */
        iops = &obj->do_index_ops->dio_it;
        LASSERT(iops != NULL);
-       it = iops->init(env, obj, rdpg->rp_attrs, BYPASS_CAPA);
+       it = iops->init(env, obj, rdpg->rp_attrs);
        if (IS_ERR(it))
                RETURN(PTR_ERR(it));
 
@@ -794,6 +837,10 @@ int dt_index_walk(const struct lu_env *env, struct dt_object *obj,
                rc = iops->next(env, it);
        } else if (rc > 0) {
                rc = 0;
+       } else {
+               if (rc == -ENODATA)
+                       rc = 0;
+               GOTO(out, rc);
        }
 
        /* Fill containers one after the other. There might be multiple
@@ -808,11 +855,11 @@ int dt_index_walk(const struct lu_env *env, struct dt_object *obj,
                int              i;
 
                LASSERT(pageidx < rdpg->rp_npages);
-               lp = cfs_kmap(rdpg->rp_pages[pageidx]);
+               lp = kmap(rdpg->rp_pages[pageidx]);
 
                /* fill lu pages */
                for (i = 0; i < LU_PAGE_COUNT; i++, lp++, nob -= LU_PAGE_SIZE) {
-                       rc = filler(env, lp, min_t(int, nob, LU_PAGE_SIZE),
+                       rc = filler(env, lp, min_t(size_t, nob, LU_PAGE_SIZE),
                                    iops, it, rdpg->rp_attrs, arg);
                        if (rc < 0)
                                break;
@@ -822,14 +869,15 @@ int dt_index_walk(const struct lu_env *env, struct dt_object *obj,
                                /* end of index */
                                break;
                }
-               cfs_kunmap(rdpg->rp_pages[i]);
+               kunmap(rdpg->rp_pages[i]);
        }
 
+out:
        iops->put(env, it);
        iops->fini(env, it);
 
        if (rc >= 0)
-               rc = min_t(unsigned int, nlupgs * LU_PAGE_SIZE, rdpg->rp_count);
+               rc = min_t(size_t, nlupgs * LU_PAGE_SIZE, rdpg->rp_count);
 
        RETURN(rc);
 }
@@ -859,18 +907,13 @@ int dt_index_read(const struct lu_env *env, struct dt_device *dev,
 
        /* rp_count shouldn't be null and should be a multiple of the container
         * size */
-       if (rdpg->rp_count <= 0 && (rdpg->rp_count & (LU_PAGE_SIZE - 1)) != 0)
+       if (rdpg->rp_count == 0 || (rdpg->rp_count & (LU_PAGE_SIZE - 1)) != 0)
                RETURN(-EFAULT);
 
-       if (fid_seq(&ii->ii_fid) >= FID_SEQ_NORMAL)
-               /* we don't support directory transfer via OBD_IDX_READ for the
-                * time being */
+       if (!fid_is_quota(&ii->ii_fid) && !fid_is_layout_rbtree(&ii->ii_fid) &&
+           !fid_is_norm(&ii->ii_fid))
                RETURN(-EOPNOTSUPP);
 
-       if (!fid_is_quota(&ii->ii_fid))
-               /* block access to all local files except quota files */
-               RETURN(-EPERM);
-
        /* lookup index object subject to the transfer */
        obj = dt_locate(env, dev, &ii->ii_fid);
        if (IS_ERR(obj))
@@ -892,35 +935,29 @@ int dt_index_read(const struct lu_env *env, struct dt_device *dev,
        }
 
        /* fill ii_flags with supported index features */
-       ii->ii_flags &= II_FL_NOHASH;
-
-       ii->ii_keysize = feat->dif_keysize_max;
-       if ((feat->dif_flags & DT_IND_VARKEY) != 0) {
-               /* key size is variable */
-               ii->ii_flags |= II_FL_VARKEY;
-               /* we don't support variable key size for the time being */
-               GOTO(out, rc = -EOPNOTSUPP);
-       }
+       ii->ii_flags &= (II_FL_NOHASH | II_FL_NOKEY | II_FL_VARKEY |
+                        II_FL_VARREC);
 
-       ii->ii_recsize = feat->dif_recsize_max;
-       if ((feat->dif_flags & DT_IND_VARREC) != 0) {
-               /* record size is variable */
-               ii->ii_flags |= II_FL_VARREC;
-               /* we don't support variable record size for the time being */
-               GOTO(out, rc = -EOPNOTSUPP);
-       }
+       if (!(feat->dif_flags & DT_IND_VARKEY))
+               ii->ii_keysize = feat->dif_keysize_max;
 
-       if ((feat->dif_flags & DT_IND_NONUNQ) != 0)
+       if (!(feat->dif_flags & DT_IND_VARREC))
+               ii->ii_recsize = feat->dif_recsize_max;
+
+       if (feat->dif_flags & DT_IND_NONUNQ)
                /* key isn't necessarily unique */
                ii->ii_flags |= II_FL_NONUNQ;
 
-       dt_read_lock(env, obj, 0);
-       /* fetch object version before walking the index */
-       ii->ii_version = dt_version_get(env, obj);
+       if (!fid_is_layout_rbtree(&ii->ii_fid)) {
+               dt_read_lock(env, obj, 0);
+               /* fetch object version before walking the index */
+               ii->ii_version = dt_version_get(env, obj);
+       }
 
        /* walk the index and fill lu_idxpages with key/record pairs */
-       rc = dt_index_walk(env, obj, rdpg, dt_index_page_build ,ii);
-       dt_read_unlock(env, obj);
+       rc = dt_index_walk(env, obj, rdpg, dt_index_page_build, ii);
+       if (!fid_is_layout_rbtree(&ii->ii_fid))
+               dt_read_unlock(env, obj);
 
        if (rc == 0) {
                /* index is empty */
@@ -935,121 +972,98 @@ out:
 }
 EXPORT_SYMBOL(dt_index_read);
 
-#ifdef LPROCFS
-
-int lprocfs_dt_rd_blksize(char *page, char **start, off_t off,
-                         int count, int *eof, void *data)
+#ifdef CONFIG_PROC_FS
+int lprocfs_dt_blksize_seq_show(struct seq_file *m, void *v)
 {
-       struct dt_device *dt = data;
+       struct dt_device *dt = m->private;
        struct obd_statfs osfs;
 
        int rc = dt_statfs(NULL, dt, &osfs);
-       if (rc != 0) {
-               *eof = 1;
-               rc = snprintf(page, count, "%d\n",
-                               (unsigned) osfs.os_bsize);
-       }
-
+       if (rc == 0)
+               seq_printf(m, "%u\n", (unsigned) osfs.os_bsize);
        return rc;
 }
-EXPORT_SYMBOL(lprocfs_dt_rd_blksize);
+EXPORT_SYMBOL(lprocfs_dt_blksize_seq_show);
 
-int lprocfs_dt_rd_kbytestotal(char *page, char **start, off_t off,
-                             int count, int *eof, void *data)
+int lprocfs_dt_kbytestotal_seq_show(struct seq_file *m, void *v)
 {
-       struct dt_device *dt = data;
+       struct dt_device *dt = m->private;
        struct obd_statfs osfs;
 
        int rc = dt_statfs(NULL, dt, &osfs);
-       if (rc != 0) {
+       if (rc == 0) {
                __u32 blk_size = osfs.os_bsize >> 10;
                __u64 result = osfs.os_blocks;
 
                while (blk_size >>= 1)
                        result <<= 1;
 
-               *eof = 1;
-               rc = snprintf(page, count, LPU64"\n", result);
+               seq_printf(m, LPU64"\n", result);
        }
-
        return rc;
 }
-EXPORT_SYMBOL(lprocfs_dt_rd_kbytestotal);
+EXPORT_SYMBOL(lprocfs_dt_kbytestotal_seq_show);
 
-int lprocfs_dt_rd_kbytesfree(char *page, char **start, off_t off,
-                            int count, int *eof, void *data)
+int lprocfs_dt_kbytesfree_seq_show(struct seq_file *m, void *v)
 {
-       struct dt_device *dt = data;
+       struct dt_device *dt = m->private;
        struct obd_statfs osfs;
 
        int rc = dt_statfs(NULL, dt, &osfs);
-       if (rc != 0) {
+       if (rc == 0) {
                __u32 blk_size = osfs.os_bsize >> 10;
                __u64 result = osfs.os_bfree;
 
                while (blk_size >>= 1)
                        result <<= 1;
 
-               *eof = 1;
-               rc = snprintf(page, count, LPU64"\n", result);
+               seq_printf(m, LPU64"\n", result);
        }
-
        return rc;
 }
-EXPORT_SYMBOL(lprocfs_dt_rd_kbytesfree);
+EXPORT_SYMBOL(lprocfs_dt_kbytesfree_seq_show);
 
-int lprocfs_dt_rd_kbytesavail(char *page, char **start, off_t off,
-                             int count, int *eof, void *data)
+int lprocfs_dt_kbytesavail_seq_show(struct seq_file *m, void *v)
 {
-       struct dt_device *dt = data;
+       struct dt_device *dt = m->private;
        struct obd_statfs osfs;
 
        int rc = dt_statfs(NULL, dt, &osfs);
-       if (rc != 0) {
+       if (rc == 0) {
                __u32 blk_size = osfs.os_bsize >> 10;
                __u64 result = osfs.os_bavail;
 
                while (blk_size >>= 1)
                        result <<= 1;
 
-               *eof = 1;
-               rc = snprintf(page, count, LPU64"\n", result);
+               seq_printf(m, LPU64"\n", result);
        }
-
        return rc;
 }
-EXPORT_SYMBOL(lprocfs_dt_rd_kbytesavail);
+EXPORT_SYMBOL(lprocfs_dt_kbytesavail_seq_show);
 
-int lprocfs_dt_rd_filestotal(char *page, char **start, off_t off,
-                            int count, int *eof, void *data)
+int lprocfs_dt_filestotal_seq_show(struct seq_file *m, void *v)
 {
-       struct dt_device *dt = data;
+       struct dt_device *dt = m->private;
        struct obd_statfs osfs;
 
        int rc = dt_statfs(NULL, dt, &osfs);
-       if (rc != 0) {
-               *eof = 1;
-               rc = snprintf(page, count, LPU64"\n", osfs.os_files);
-       }
-
+       if (rc == 0)
+               seq_printf(m, LPU64"\n", osfs.os_files);
        return rc;
 }
-EXPORT_SYMBOL(lprocfs_dt_rd_filestotal);
+EXPORT_SYMBOL(lprocfs_dt_filestotal_seq_show);
 
-int lprocfs_dt_rd_filesfree(char *page, char **start, off_t off,
-                           int count, int *eof, void *data)
+int lprocfs_dt_filesfree_seq_show(struct seq_file *m, void *v)
 {
-       struct dt_device *dt = data;
+       struct dt_device *dt = m->private;
        struct obd_statfs osfs;
 
        int rc = dt_statfs(NULL, dt, &osfs);
-       if (rc != 0) {
-               *eof = 1;
-               rc = snprintf(page, count, LPU64"\n", osfs.os_ffree);
-       }
-
+       if (rc == 0)
+               seq_printf(m, LPU64"\n", osfs.os_ffree);
        return rc;
 }
-EXPORT_SYMBOL(lprocfs_dt_rd_filesfree);
+EXPORT_SYMBOL(lprocfs_dt_filesfree_seq_show);
 
-#endif /* LPROCFS */
+#endif /* CONFIG_PROC_FS */