From f182151cf338cad36d1d503f33b65ca40785b797 Mon Sep 17 00:00:00 2001 From: Vijay Bellur Date: Mon, 11 Oct 2010 12:32:04 -0700 Subject: Revert "nfs: re-implement logic to perform fresh lookups when lookup revalidates fail" This reverts commit 563ef3d12f8d7a927b403d04c279d9fc702faad0. --- xlators/nfs/server/src/nfs-fops.c | 25 ------------------ xlators/nfs/server/src/nfs-fops.h | 6 ----- xlators/nfs/server/src/nfs-generics.c | 3 +-- xlators/nfs/server/src/nfs-inodes.c | 50 ----------------------------------- xlators/nfs/server/src/nfs-inodes.h | 2 +- 5 files changed, 2 insertions(+), 84 deletions(-) diff --git a/xlators/nfs/server/src/nfs-fops.c b/xlators/nfs/server/src/nfs-fops.c index f748c1fa34c..4928deb969f 100644 --- a/xlators/nfs/server/src/nfs-fops.c +++ b/xlators/nfs/server/src/nfs-fops.c @@ -74,8 +74,6 @@ nfs_fop_local_wipe (xlator_t *nfsx, struct nfs_fop_local *l) if (l->dictgfid) dict_unref (l->dictgfid); - loc_wipe (&l->revalidate_loc); - mem_put (nfs->foppool, l); return; @@ -302,27 +300,9 @@ nfs_fop_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this, { struct nfs_fop_local *local = NULL; fop_lookup_cbk_t progcbk; - inode_table_t *itable = NULL; - xlator_t *xl = NULL; - xl = cookie; nfl_to_prog_data (local, progcbk, frame); nfs_fop_restore_root_ino (local, buf, NULL, NULL, postparent); - - if (op_ret == -1 && local->is_revalidate == 1) { - /* perform a fresh lookup if revalidate fails */ - itable = local->revalidate_loc.inode->table; - inode_unref (local->revalidate_loc.inode); - local->revalidate_loc.inode = inode_new (itable); - - local->is_revalidate = 2; /* prevent entering revalidate loops */ - - STACK_WIND_COOKIE (frame, nfs_fop_lookup_cbk, xl, xl, - xl->fops->lookup, &local->revalidate_loc, - local->dictgfid); - return 0; - } - if (progcbk) progcbk (frame, cookie, this, op_ret, op_errno, inode, buf, xattr, postparent); @@ -349,11 +329,6 @@ nfs_fop_lookup (xlator_t *nfsx, xlator_t *xl, nfs_user_t *nfu, loc_t *loc, nfs_fop_save_root_ino (nfl, loc); nfs_fop_gfid_setup (nfl, ret, err); - if (!uuid_is_null (loc->inode->gfid)) { - nfl->is_revalidate = 1; - loc_copy (&nfl->revalidate_loc, loc); - } - STACK_WIND_COOKIE (frame, nfs_fop_lookup_cbk, xl, xl, xl->fops->lookup, loc, nfl->dictgfid); diff --git a/xlators/nfs/server/src/nfs-fops.h b/xlators/nfs/server/src/nfs-fops.h index 4feb916e2ea..d010db2829b 100644 --- a/xlators/nfs/server/src/nfs-fops.h +++ b/xlators/nfs/server/src/nfs-fops.h @@ -100,12 +100,6 @@ struct nfs_fop_local { char newpath[NFS_NAME_MAX]; xlator_t *nfsx; dict_t *dictgfid; - - /* Determine whether the call was a lookup revalidate in cases where - * lookup fails. Mangle the copied loc_t to perform a fresh lookup - */ - int is_revalidate; - loc_t revalidate_loc; }; extern struct nfs_fop_local * diff --git a/xlators/nfs/server/src/nfs-generics.c b/xlators/nfs/server/src/nfs-generics.c index eb6dc580be2..0ebba689a35 100644 --- a/xlators/nfs/server/src/nfs-generics.c +++ b/xlators/nfs/server/src/nfs-generics.c @@ -83,8 +83,7 @@ nfs_lookup (xlator_t *nfsx, xlator_t *xl, nfs_user_t *nfu, loc_t *pathloc, if ((!nfsx) || (!xl) || (!pathloc) || (!nfu)) return ret; - ret = nfs_inode_lookup (nfsx, xl, nfu, pathloc, cbk, local); - + ret = nfs_fop_lookup (nfsx, xl, nfu, pathloc, cbk, local); return ret; } diff --git a/xlators/nfs/server/src/nfs-inodes.c b/xlators/nfs/server/src/nfs-inodes.c index 7f421034852..cd334525d2d 100644 --- a/xlators/nfs/server/src/nfs-inodes.c +++ b/xlators/nfs/server/src/nfs-inodes.c @@ -335,56 +335,6 @@ err: } -int32_t -nfs_inode_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this, - int32_t op_ret, int32_t op_errno, inode_t *inode, - struct iatt *buf, dict_t *xattr, - struct iatt *postparent) -{ - struct nfs_fop_local *nfl = NULL; - fop_lookup_cbk_t progcbk = NULL; - inode_t *linked_inode = NULL; - - if (op_ret == -1) - goto do_not_link; - - nfl = frame->local; - linked_inode = inode_link (inode, nfl->newparent, nfl->path, buf); - - if (linked_inode) - inode_unref (linked_inode); - -do_not_link: - inodes_nfl_to_prog_data (nfl, progcbk, frame); - if (progcbk) - progcbk (frame, cookie, this, op_ret, op_errno, inode, buf, - xattr, postparent); - return 0; -} - - -int -nfs_inode_lookup (xlator_t *nfsx, xlator_t *xl, nfs_user_t *nfu, loc_t *loc, - fop_lookup_cbk_t cbk, void *local) -{ - struct nfs_fop_local *nfl = NULL; - int ret = -EFAULT; - - if ((!nfsx) || (!xl) || (!loc) || (!nfu)) - return -EFAULT; - - nfs_fop_handle_local_init (NULL, nfsx, nfl, cbk, local, ret, err); - nfl_inodes_init (nfl, NULL, NULL, loc->parent, loc->name, NULL); - ret = nfs_fop_lookup (nfsx, xl, nfu, loc, nfs_inode_lookup_cbk, nfl); - -err: - if (ret < 0) - nfs_fop_local_wipe (xl, nfl); - - return ret; -} - - int32_t nfs_inode_unlink_cbk (call_frame_t *frame, void *cookie, xlator_t *this, int32_t op_ret, int32_t op_errno, struct iatt *preparent, diff --git a/xlators/nfs/server/src/nfs-inodes.h b/xlators/nfs/server/src/nfs-inodes.h index a34a2673867..ef79a11537d 100644 --- a/xlators/nfs/server/src/nfs-inodes.h +++ b/xlators/nfs/server/src/nfs-inodes.h @@ -80,6 +80,6 @@ nfs_inode_mknod (xlator_t *nfsx, xlator_t *xl, nfs_user_t *nfu, loc_t *pathloc, mode_t mode, dev_t dev, fop_mknod_cbk_t cbk, void *local); extern int -nfs_inode_lookup (xlator_t *nfsx, xlator_t *xl, nfs_user_t *nfu, loc_t *pathloc, +nfs_inode_lookup (xlator_t *xl, nfs_user_t *nfu, loc_t *pathloc, fop_lookup_cbk_t cbk, void *local); #endif -- cgit