Whamcloud - gitweb
LU-6142 llite: Fix style issues for llite_nfs.c
[fs/lustre-release.git] / lustre / llite / llite_nfs.c
index 7f3f56b..cf24560 100644 (file)
@@ -1,6 +1,4 @@
-/* -*- mode: c; c-basic-offset: 8; indent-tabs-mode: nil; -*-
- * vim:expandtab:shiftwidth=8:tabstop=8:
- *
+/*
  * GPL HEADER START
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * 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
  */
@@ -29,8 +23,7 @@
  * Copyright (c) 2003, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2011 Whamcloud, Inc.
- *
+ * Copyright (c) 2011, 2017, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
  */
 
 #define DEBUG_SUBSYSTEM S_LLITE
-#include <lustre_lite.h>
 #include "llite_internal.h"
-#ifdef HAVE_LINUX_EXPORTFS_H
 #include <linux/exportfs.h>
-#endif
 
-__u32 get_uuid2int(const char *name, int len)
+u32 get_uuid2int(const char *name, int len)
 {
-        __u32 key0 = 0x12a3fe2d, key1 = 0x37abe8f9;
-        while (len--) {
-                __u32 key = key1 + (key0 ^ (*name++ * 7152373));
-                if (key & 0x80000000) key -= 0x7fffffff;
-                key1 = key0;
-                key0 = key;
-        }
-        return (key0 << 1);
-}
+       u32 key0 = 0x12a3fe2d, key1 = 0x37abe8f9;
 
-static int ll_nfs_test_inode(struct inode *inode, void *opaque)
-{
-        return lu_fid_eq(&ll_i2info(inode)->lli_fid,
-                         (struct lu_fid *)opaque);
+       while (len--) {
+               u32 key = key1 + (key0 ^ (*name++ * 7152373));
+
+               if (key & 0x80000000)
+                       key -= 0x7fffffff;
+
+               key1 = key0;
+               key0 = key;
+       }
+       return (key0 << 1);
 }
 
-static struct inode *search_inode_for_lustre(struct super_block *sb,
-                                             const struct lu_fid *fid)
+struct inode *search_inode_for_lustre(struct super_block *sb,
+                                     const struct lu_fid *fid)
 {
-        struct ll_sb_info     *sbi = ll_s2sbi(sb);
-        struct ptlrpc_request *req = NULL;
-        struct inode          *inode = NULL;
-        int                   eadatalen = 0;
-        unsigned long         hash = (unsigned long) cl_fid_build_ino(fid, 0);
-        struct  md_op_data    *op_data;
-        int                   rc;
-        ENTRY;
-
-        CDEBUG(D_INFO, "searching inode for:(%lu,"DFID")\n", hash, PFID(fid));
-
-        inode = ilookup5(sb, hash, ll_nfs_test_inode, (void *)fid);
-        if (inode)
-                RETURN(inode);
-
-        rc = ll_get_max_mdsize(sbi, &eadatalen);
-        if (rc)
-                RETURN(ERR_PTR(rc));
-
-        /* Because inode is NULL, ll_prep_md_op_data can not
-         * be used here. So we allocate op_data ourselves */
-        OBD_ALLOC_PTR(op_data);
-        if (op_data == NULL)
-                return ERR_PTR(-ENOMEM);
-
-        op_data->op_fid1 = *fid;
-        op_data->op_mode = eadatalen;
-        op_data->op_valid = OBD_MD_FLEASIZE;
-
-        /* mds_fid2dentry ignores f_type */
-        rc = md_getattr(sbi->ll_md_exp, op_data, &req);
-        OBD_FREE_PTR(op_data);
-        if (rc) {
-                CERROR("can't get object attrs, fid "DFID", rc %d\n",
-                       PFID(fid), rc);
-                RETURN(ERR_PTR(rc));
-        }
-        rc = ll_prep_inode(&inode, req, sb);
-        ptlrpc_req_finished(req);
-        if (rc)
-                RETURN(ERR_PTR(rc));
-
-        RETURN(inode);
+       struct ll_sb_info *sbi = ll_s2sbi(sb);
+       struct ptlrpc_request *req = NULL;
+       struct inode *inode = NULL;
+       int eadatalen = 0;
+       unsigned long hash = cl_fid_build_ino(fid, ll_need_32bit_api(sbi));
+       struct md_op_data *op_data;
+       int rc;
+
+       ENTRY;
+
+       CDEBUG(D_INFO, "searching inode for:(%lu,"DFID")\n", hash, PFID(fid));
+
+       inode = ilookup5(sb, hash, ll_test_inode_by_fid, (void *)fid);
+       if (inode)
+               RETURN(inode);
+
+       rc = ll_get_default_mdsize(sbi, &eadatalen);
+       if (rc)
+               RETURN(ERR_PTR(rc));
+
+       /*
+        * Because inode is NULL, ll_prep_md_op_data can not
+        * be used here. So we allocate op_data ourselves
+        */
+       OBD_ALLOC_PTR(op_data);
+       if (!op_data)
+               return ERR_PTR(-ENOMEM);
+
+       op_data->op_fid1 = *fid;
+       op_data->op_mode = eadatalen;
+       op_data->op_valid = OBD_MD_FLEASIZE;
+
+       /* mds_fid2dentry ignores f_type */
+       rc = md_getattr(sbi->ll_md_exp, op_data, &req);
+       OBD_FREE_PTR(op_data);
+       if (rc) {
+               /*
+                * Suppress erroneous/confusing messages when NFS
+                * is out of sync and requests old data.
+                */
+               CDEBUG(D_INFO, "can't get object attrs, fid "DFID", rc %d\n",
+                               PFID(fid), rc);
+               RETURN(ERR_PTR(rc));
+       }
+       rc = ll_prep_inode(&inode, req, sb, NULL);
+       ptlrpc_req_finished(req);
+       if (rc)
+               RETURN(ERR_PTR(rc));
+
+       RETURN(inode);
 }
 
 struct lustre_nfs_fid {
-        struct lu_fid   lnf_child;
-        struct lu_fid   lnf_parent;
+       struct lu_fid lnf_child;
+       struct lu_fid lnf_parent;
 };
 
 static struct dentry *
 ll_iget_for_nfs(struct super_block *sb, struct lu_fid *fid, struct lu_fid *parent)
 {
-        struct inode  *inode;
-        struct dentry *result;
-        ENTRY;
-
-        CDEBUG(D_INFO, "Get dentry for fid: "DFID"\n", PFID(fid));
-        if (!fid_is_sane(fid))
-                RETURN(ERR_PTR(-ESTALE));
-
-        inode = search_inode_for_lustre(sb, fid);
-        if (IS_ERR(inode))
-                RETURN(ERR_PTR(PTR_ERR(inode)));
-
-        if (is_bad_inode(inode)) {
-                /* we didn't find the right inode.. */
-                iput(inode);
-                RETURN(ERR_PTR(-ESTALE));
-        }
-
-        /**
-         * It is an anonymous dentry without OST objects created yet.
-         * We have to find the parent to tell MDS how to init lov objects.
-         */
-        if (S_ISREG(inode->i_mode) && ll_i2info(inode)->lli_smd == NULL &&
-            parent != NULL) {
-                struct ll_inode_info *lli = ll_i2info(inode);
-
-                cfs_spin_lock(&lli->lli_lock);
-                lli->lli_pfid = *parent;
-                cfs_spin_unlock(&lli->lli_lock);
-        }
-
-        result = d_obtain_alias(inode);
-        if (IS_ERR(result))
-                RETURN(result);
-
-        ll_dops_init(result, 1, 0);
-
-        RETURN(result);
+       struct inode  *inode;
+       struct dentry *result;
+
+       ENTRY;
+
+       if (!fid_is_sane(fid))
+               RETURN(ERR_PTR(-ESTALE));
+
+       CDEBUG(D_INFO, "Get dentry for fid: "DFID"\n", PFID(fid));
+
+       inode = search_inode_for_lustre(sb, fid);
+       if (IS_ERR(inode))
+               RETURN(ERR_PTR(PTR_ERR(inode)));
+
+       if (is_bad_inode(inode)) {
+               /* we didn't find the right inode.. */
+               iput(inode);
+               RETURN(ERR_PTR(-ESTALE));
+       }
+
+       /* N.B. d_obtain_alias() drops inode ref on error */
+       result = d_obtain_alias(inode);
+       if (!IS_ERR(result)) {
+               int rc;
+
+               rc = ll_d_init(result);
+               if (rc < 0) {
+                       dput(result);
+                       result = ERR_PTR(rc);
+               } else {
+                       struct ll_dentry_data *ldd = ll_d2d(result);
+
+                       /*
+                        * Need to signal to the ll_file_open that
+                        * we came from NFS and so opencache needs to be
+                        * enabled for this one
+                        */
+                       spin_lock(&result->d_lock);
+                       ldd->lld_nfs_dentry = 1;
+                       spin_unlock(&result->d_lock);
+               }
+       }
+
+       RETURN(result);
 }
 
-#define LUSTRE_NFS_FID          0x97
+#ifndef FILEID_INVALID
+#define FILEID_INVALID 0xff
+#endif
+#ifndef FILEID_LUSTRE
+#define FILEID_LUSTRE  0x97
+#endif
 
 /**
  * \a connectable - is nfsd will connect himself or this should be done
@@ -176,197 +182,211 @@ ll_iget_for_nfs(struct super_block *sb, struct lu_fid *fid, struct lu_fid *paren
  * 2 -- contains child file handle and parent file handle;
  * 255 -- error.
  */
-static int ll_encode_fh(struct dentry *de, __u32 *fh, int *plen,
-                        int connectable)
+#ifndef HAVE_ENCODE_FH_PARENT
+static int ll_encode_fh(struct dentry *de, u32 *fh, int *plen,
+                       int connectable)
+{
+       struct inode *inode = de->d_inode;
+       struct inode *parent = de->d_parent->d_inode;
+#else
+static int ll_encode_fh(struct inode *inode, u32 *fh, int *plen,
+                       struct inode *parent)
 {
-        struct inode *inode = de->d_inode;
-        struct inode *parent = de->d_parent->d_inode;
-        struct lustre_nfs_fid *nfs_fid = (void *)fh;
-        ENTRY;
+#endif
+       int fileid_len = sizeof(struct lustre_nfs_fid) / 4;
+       struct lustre_nfs_fid *nfs_fid = (void *)fh;
+
+       ENTRY;
 
-        CDEBUG(D_INFO, "encoding for (%lu,"DFID") maxlen=%d minlen=%d\n",
-              inode->i_ino, PFID(ll_inode2fid(inode)), *plen,
-              (int)sizeof(struct lustre_nfs_fid));
+       CDEBUG(D_INFO, "%s: encoding for ("DFID") maxlen=%d minlen=%d\n",
+              ll_get_fsname(inode->i_sb, NULL, 0),
+              PFID(ll_inode2fid(inode)), *plen, fileid_len);
 
-        if (*plen < sizeof(struct lustre_nfs_fid)/4)
-                RETURN(255);
+       if (*plen < fileid_len) {
+               *plen = fileid_len;
+               RETURN(FILEID_INVALID);
+       }
 
-        nfs_fid->lnf_child = *ll_inode2fid(inode);
-        nfs_fid->lnf_parent = *ll_inode2fid(parent);
-        *plen = sizeof(struct lustre_nfs_fid)/4;
+       nfs_fid->lnf_child = *ll_inode2fid(inode);
+       if (parent)
+               nfs_fid->lnf_parent = *ll_inode2fid(parent);
+       else
+               fid_zero(&nfs_fid->lnf_parent);
+       *plen = fileid_len;
 
-        RETURN(LUSTRE_NFS_FID);
+       RETURN(FILEID_LUSTRE);
 }
 
-static int ll_nfs_get_name_filldir(void *cookie, const char *name, int namelen,
-                                   loff_t hash, u64 ino, unsigned type)
+static int
+#ifndef HAVE_FILLDIR_USE_CTX
+ll_nfs_get_name_filldir(void *cookie, const char *name, int namelen,
+                       loff_t hash, u64 ino, unsigned type)
 {
-        /* It is hack to access lde_fid for comparison with lgd_fid.
-         * So the input 'name' must be part of the 'lu_dirent'. */
-        struct lu_dirent *lde = container_of0(name, struct lu_dirent, lde_name);
-        struct ll_getname_data *lgd = cookie;
-        struct lu_fid fid;
-
-        fid_le_to_cpu(&fid, &lde->lde_fid);
-        if (lu_fid_eq(&fid, &lgd->lgd_fid)) {
-                memcpy(lgd->lgd_name, name, namelen);
-                lgd->lgd_name[namelen] = 0;
-                lgd->lgd_found = 1;
-        }
+       struct ll_getname_data *lgd = cookie;
+#else
+ll_nfs_get_name_filldir(struct dir_context *ctx, const char *name, int namelen,
+                       loff_t hash, u64 ino, unsigned type)
+{
+       struct ll_getname_data *lgd =
+               container_of(ctx, struct ll_getname_data, ctx);
+#endif /* HAVE_FILLDIR_USE_CTX */
+       /*
+        * It is hack to access lde_fid for comparison with lgd_fid.
+        * So the input 'name' must be part of the 'lu_dirent'.
+        */
+       struct lu_dirent *lde = container_of0(name, struct lu_dirent, lde_name);
+       struct lu_fid fid;
+
+       fid_le_to_cpu(&fid, &lde->lde_fid);
+       if (lu_fid_eq(&fid, &lgd->lgd_fid)) {
+               memcpy(lgd->lgd_name, name, namelen);
+               lgd->lgd_name[namelen] = 0;
+               lgd->lgd_found = 1;
+       }
         return lgd->lgd_found;
 }
 
 static int ll_get_name(struct dentry *dentry, char *name,
                        struct dentry *child)
 {
-        struct inode *dir = dentry->d_inode;
-        struct file *filp;
-        struct ll_getname_data lgd;
-        int rc;
-        ENTRY;
-
-        if (!dir || !S_ISDIR(dir->i_mode))
-                GOTO(out, rc = -ENOTDIR);
-
-        if (!dir->i_fop)
-                GOTO(out, rc = -EINVAL);
-
-        filp = ll_dentry_open(dget(dentry), mntget(ll_i2sbi(dir)->ll_mnt),
-                              O_RDONLY, current_cred());
-        if (IS_ERR(filp))
-                GOTO(out, rc = PTR_ERR(filp));
-
-        if (!filp->f_op->readdir)
-                GOTO(out_close, rc = -EINVAL);
-
-        lgd.lgd_name = name;
-        lgd.lgd_fid = ll_i2info(child->d_inode)->lli_fid;
-        lgd.lgd_found = 0;
-
-        cfs_mutex_lock(&dir->i_mutex);
-        rc = ll_readdir(filp, &lgd, ll_nfs_get_name_filldir);
-        cfs_mutex_unlock(&dir->i_mutex);
-        if (!rc && !lgd.lgd_found)
-                rc = -ENOENT;
-        EXIT;
-
-out_close:
-        fput(filp);
+       struct inode *dir = dentry->d_inode;
+       struct ll_getname_data lgd = {
+               .lgd_name = name,
+               .lgd_fid = ll_i2info(child->d_inode)->lli_fid,
+#ifdef HAVE_DIR_CONTEXT
+               .ctx.actor = ll_nfs_get_name_filldir,
+#endif
+               .lgd_found = 0,
+       };
+       struct md_op_data *op_data;
+       u64 pos = 0;
+       int rc;
+
+       ENTRY;
+
+       if (!dir || !S_ISDIR(dir->i_mode))
+               GOTO(out, rc = -ENOTDIR);
+
+       if (!dir->i_fop)
+               GOTO(out, rc = -EINVAL);
+
+       op_data = ll_prep_md_op_data(NULL, dir, dir, NULL, 0, 0,
+                                    LUSTRE_OPC_ANY, dir);
+       if (IS_ERR(op_data))
+               GOTO(out, rc = PTR_ERR(op_data));
+
+       inode_lock(dir);
+#ifdef HAVE_DIR_CONTEXT
+       rc = ll_dir_read(dir, &pos, op_data, &lgd.ctx);
+#else
+       rc = ll_dir_read(dir, &pos, op_data, &lgd, ll_nfs_get_name_filldir);
+#endif
+       inode_unlock(dir);
+       ll_finish_md_op_data(op_data);
+       if (!rc && !lgd.lgd_found)
+               rc = -ENOENT;
+       EXIT;
 out:
-        return rc;
+       return rc;
 }
 
-#ifdef HAVE_FH_TO_DENTRY
 static struct dentry *ll_fh_to_dentry(struct super_block *sb, struct fid *fid,
-                                      int fh_len, int fh_type)
+                                     int fh_len, int fh_type)
 {
-        struct lustre_nfs_fid *nfs_fid = (struct lustre_nfs_fid *)fid;
+       struct lustre_nfs_fid *nfs_fid = (struct lustre_nfs_fid *)fid;
 
-        if (fh_type != LUSTRE_NFS_FID)
-                RETURN(ERR_PTR(-EPROTO));
+       if (fh_type != FILEID_LUSTRE)
+               RETURN(ERR_PTR(-EPROTO));
 
-        RETURN(ll_iget_for_nfs(sb, &nfs_fid->lnf_child, &nfs_fid->lnf_parent));
+       RETURN(ll_iget_for_nfs(sb, &nfs_fid->lnf_child, &nfs_fid->lnf_parent));
 }
 
 static struct dentry *ll_fh_to_parent(struct super_block *sb, struct fid *fid,
-                                      int fh_len, int fh_type)
+                                     int fh_len, int fh_type)
 {
-        struct lustre_nfs_fid *nfs_fid = (struct lustre_nfs_fid *)fid;
+       struct lustre_nfs_fid *nfs_fid = (struct lustre_nfs_fid *)fid;
 
-        if (fh_type != LUSTRE_NFS_FID)
-                RETURN(ERR_PTR(-EPROTO));
+       if (fh_type != FILEID_LUSTRE)
+               RETURN(ERR_PTR(-EPROTO));
 
-        RETURN(ll_iget_for_nfs(sb, &nfs_fid->lnf_parent, NULL));
+       RETURN(ll_iget_for_nfs(sb, &nfs_fid->lnf_parent, NULL));
 }
 
-#else
-
-/*
- * This length is counted as amount of __u32,
- *  It is composed of a fid and a mode
- */
-static struct dentry *ll_decode_fh(struct super_block *sb, __u32 *fh, int fh_len,
-                                   int fh_type,
-                                   int (*acceptable)(void *, struct dentry *),
-                                   void *context)
+int ll_dir_get_parent_fid(struct inode *dir, struct lu_fid *parent_fid)
 {
-        struct lustre_nfs_fid *nfs_fid = (void *)fh;
-        struct dentry *entry;
-        ENTRY;
+       struct ptlrpc_request   *req = NULL;
+       struct ll_sb_info       *sbi;
+       struct mdt_body         *body;
+       static const char       dotdot[] = "..";
+       struct md_op_data       *op_data;
+       int                     rc;
+       int                     lmmsize;
+
+       ENTRY;
+
+       LASSERT(dir && S_ISDIR(dir->i_mode));
+
+       sbi = ll_s2sbi(dir->i_sb);
+
+       CDEBUG(D_INFO, "%s: getting parent for ("DFID")\n",
+              ll_get_fsname(dir->i_sb, NULL, 0),
+              PFID(ll_inode2fid(dir)));
+
+       rc = ll_get_default_mdsize(sbi, &lmmsize);
+       if (rc != 0)
+               RETURN(rc);
+
+       op_data = ll_prep_md_op_data(NULL, dir, NULL, dotdot,
+                                    strlen(dotdot), lmmsize,
+                                    LUSTRE_OPC_ANY, NULL);
+       if (IS_ERR(op_data))
+               RETURN(PTR_ERR(op_data));
+
+       rc = md_getattr_name(sbi->ll_md_exp, op_data, &req);
+       ll_finish_md_op_data(op_data);
+       if (rc != 0) {
+               CERROR("%s: failure inode "DFID" get parent: rc = %d\n",
+                      ll_get_fsname(dir->i_sb, NULL, 0),
+                      PFID(ll_inode2fid(dir)), rc);
+               RETURN(rc);
+       }
+       body = req_capsule_server_get(&req->rq_pill, &RMF_MDT_BODY);
+
+       /*
+        * LU-3952: MDT may lost the FID of its parent, we should not crash
+        * the NFS server, ll_iget_for_nfs() will handle the error.
+        */
+       if (body->mbo_valid & OBD_MD_FLID) {
+               CDEBUG(D_INFO, "parent for "DFID" is "DFID"\n",
+                      PFID(ll_inode2fid(dir)), PFID(&body->mbo_fid1));
+               *parent_fid = body->mbo_fid1;
+       }
+
+       ptlrpc_req_finished(req);
+       RETURN(0);
+}
 
-        CDEBUG(D_INFO, "decoding for "DFID" fh_len=%d fh_type=%x\n", 
-                PFID(&nfs_fid->lnf_child), fh_len, fh_type);
+static struct dentry *ll_get_parent(struct dentry *dchild)
+{
+       struct lu_fid parent_fid = { 0 };
+       int rc;
+       struct dentry *dentry;
 
-        if (fh_type != LUSTRE_NFS_FID)
-                RETURN(ERR_PTR(-EPROTO));
+       ENTRY;
 
-        entry = sb->s_export_op->find_exported_dentry(sb, &nfs_fid->lnf_child,
-                                                      &nfs_fid->lnf_parent,
-                                                      acceptable, context);
-        RETURN(entry);
-}
+       rc = ll_dir_get_parent_fid(dchild->d_inode, &parent_fid);
+       if (rc != 0)
+               RETURN(ERR_PTR(rc));
 
-static struct dentry *ll_get_dentry(struct super_block *sb, void *data)
-{
-        struct dentry *entry;
-        ENTRY;
+       dentry = ll_iget_for_nfs(dchild->d_inode->i_sb, &parent_fid, NULL);
 
-        entry = ll_iget_for_nfs(sb, data, NULL);
-        RETURN(entry);
-}
-#endif
-static struct dentry *ll_get_parent(struct dentry *dchild)
-{
-        struct ptlrpc_request *req = NULL;
-        struct inode          *dir = dchild->d_inode;
-        struct ll_sb_info     *sbi;
-        struct dentry         *result = NULL;
-        struct mdt_body       *body;
-        static char           dotdot[] = "..";
-        struct md_op_data     *op_data;
-        int                   rc;
-        ENTRY;
-
-        LASSERT(dir && S_ISDIR(dir->i_mode));
-
-        sbi = ll_s2sbi(dir->i_sb);
-
-        CDEBUG(D_INFO, "getting parent for (%lu,"DFID")\n",
-                        dir->i_ino, PFID(ll_inode2fid(dir)));
-
-        op_data = ll_prep_md_op_data(NULL, dir, NULL, dotdot,
-                                     strlen(dotdot), 0,
-                                     LUSTRE_OPC_ANY, NULL);
-        if (op_data == NULL)
-                RETURN(ERR_PTR(-ENOMEM));
-
-        rc = md_getattr_name(sbi->ll_md_exp, op_data, &req);
-        ll_finish_md_op_data(op_data);
-        if (rc) {
-                CERROR("failure %d inode %lu get parent\n", rc, dir->i_ino);
-                RETURN(ERR_PTR(rc));
-        }
-        body = req_capsule_server_get(&req->rq_pill, &RMF_MDT_BODY);
-        LASSERT(body->valid & OBD_MD_FLID);
-
-        CDEBUG(D_INFO, "parent for "DFID" is "DFID"\n",
-                PFID(ll_inode2fid(dir)), PFID(&body->fid1));
-
-        result = ll_iget_for_nfs(dir->i_sb, &body->fid1, NULL);
-
-        ptlrpc_req_finished(req);
-        RETURN(result);
+       RETURN(dentry);
 }
 
 struct export_operations lustre_export_operations = {
-       .get_parent = ll_get_parent,
-       .encode_fh  = ll_encode_fh,
-       .get_name   = ll_get_name,
-#ifdef HAVE_FH_TO_DENTRY
-        .fh_to_dentry = ll_fh_to_dentry,
-        .fh_to_parent = ll_fh_to_parent,
-#else
-       .get_dentry = ll_get_dentry,
-       .decode_fh  = ll_decode_fh,
-#endif
+       .get_parent = ll_get_parent,
+       .encode_fh  = ll_encode_fh,
+       .get_name   = ll_get_name,
+       .fh_to_dentry = ll_fh_to_dentry,
+       .fh_to_parent = ll_fh_to_parent,
 };