gfapi: send lookup if inode_ctx is not set

During resolving of an entry or inode, if inode ctx
was not set, we will send a lookup to pupulate inode
ctx for every xlators

This patch also make sure that inode_ctx will be created
after every inode_link. We will store inode_ctx value as
LOOKUP_NEEDED if the inode is liked via readdirp, in all
other case we will store inode_ctx value as LOOKUP_NOT_NEEDED.

Back port of>
>Change-Id: I3a10c298944200fa3862127187ae8988e582d352
>BUG: 1297311
>Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com>
>Reviewed-on: http://review.gluster.org/13226
>Reviewed-by: Poornima G <pgurusid@redhat.com>
>Tested-by: NetBSD Build System <jenkins@build.gluster.org>
>Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
>Reviewed-by: Dan Lambright <dlambrig@redhat.com>
>Tested-by: Dan Lambright <dlambrig@redhat.com>

Change-Id: Ifeabb5611fcaa3c41404f0cdc48a680a16600e68
BUG: 1306131
Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com>
Reviewed-on: http://review.gluster.org/13414
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Poornima G <pgurusid@redhat.com>
Reviewed-by: Dan Lambright <dlambrig@redhat.com>

3 files changed, 18 insertions, 5 deletions

diff --git a/api/src/glfs-fops.c b/api/src/glfs-fops.c
index dd9becea3a4..ff55bf3d673 100644
--- a/api/src/glfs-fops.c
+++ b/api/src/glfs-fops.c
@@ -77,6 +77,7 @@ glfs_loc_link (loc_t *loc, struct iatt *iatt)
 {
 	int ret = -1;
         inode_t *old_inode = NULL;
+        uint64_t ctx_value = LOOKUP_NOT_NEEDED;
 
 	if (!loc->inode) {
 		errno = EINVAL;
@@ -91,6 +92,7 @@ glfs_loc_link (loc_t *loc, struct iatt *iatt)
          */
 	loc->inode = inode_link (loc->inode, loc->parent, loc->name, iatt);
 	if (loc->inode) {
+                inode_ctx_set (loc->inode, THIS, &ctx_value);
 		inode_lookup (loc->inode);
                 inode_unref (old_inode);
 		ret = 0;
diff --git a/api/src/glfs-handleops.c b/api/src/glfs-handleops.c
index ac648141eb0..bc57221055f 100644
--- a/api/src/glfs-handleops.c
+++ b/api/src/glfs-handleops.c
@@ -1272,6 +1272,7 @@ pub_glfs_h_create_from_handle (struct glfs *fs, unsigned char *handle, int len,
         inode_t            *newinode = NULL;
         xlator_t           *subvol = NULL;
         struct glfs_object *object = NULL;
+        uint64_t            ctx_value = LOOKUP_NOT_NEEDED;
 
         /* validate in args */
         if ((fs == NULL) || (handle == NULL) || (len != GFAPI_HANDLE_LENGTH)) {
@@ -1320,9 +1321,12 @@ pub_glfs_h_create_from_handle (struct glfs *fs, unsigned char *handle, int len,
         }
 
         newinode = inode_link (loc.inode, 0, 0, &iatt);
-        if (newinode)
+        if (newinode) {
+                if (newinode == loc.inode) {
+                        inode_ctx_set (newinode, THIS, &ctx_value);
+                }
                 inode_lookup (newinode);
-        else {
+        } else {
                 gf_msg (subvol->name, GF_LOG_WARNING, EINVAL,
                         API_MSG_INVALID_ENTRY,
                         "inode linking of %s failed: %s",
diff --git a/api/src/glfs-resolve.c b/api/src/glfs-resolve.c
index b5efcbae0e7..f8933326433 100644
--- a/api/src/glfs-resolve.c
+++ b/api/src/glfs-resolve.c
@@ -91,6 +91,7 @@ glfs_refresh_inode_safe (xlator_t *subvol, inode_t *oldinode,
 	struct iatt  iatt = {0, };
 	inode_t     *newinode = NULL;
         gf_boolean_t lookup_needed = _gf_false;
+        uint64_t     ctx_value = LOOKUP_NOT_NEEDED;
 
 
 	if (!oldinode)
@@ -129,8 +130,11 @@ glfs_refresh_inode_safe (xlator_t *subvol, inode_t *oldinode,
 	}
 
 	newinode = inode_link (loc.inode, 0, 0, &iatt);
-	if (newinode)
-		inode_lookup (newinode);
+        if (newinode) {
+                if (newinode == loc.inode)
+                        inode_ctx_set (newinode, THIS, &ctx_value);
+                inode_lookup (newinode);
+        }
 
 	loc_wipe (&loc);
 
@@ -240,6 +244,7 @@ glfs_resolve_component (struct glfs *fs, xlator_t *subvol, inode_t *parent,
 	struct iatt  ciatt = {0, };
 	uuid_t       gfid;
 	dict_t      *xattr_req = NULL;
+        uint64_t     ctx_value = LOOKUP_NOT_NEEDED;
 
 	loc.name = component;
 
@@ -270,7 +275,7 @@ glfs_resolve_component (struct glfs *fs, xlator_t *subvol, inode_t *parent,
 		gf_uuid_copy (loc.gfid, loc.inode->gfid);
 		reval = 1;
 
-		if (!force_lookup) {
+                if (!(force_lookup || inode_needs_lookup (loc.inode, THIS))) {
 			inode = inode_ref (loc.inode);
 			ciatt.ia_type = inode->ia_type;
 			goto found;
@@ -341,6 +346,8 @@ glfs_resolve_component (struct glfs *fs, xlator_t *subvol, inode_t *parent,
 		goto out;
 
 	inode = inode_link (loc.inode, loc.parent, component, &ciatt);
+        if (inode == loc.inode)
+                inode_ctx_set (inode, THIS, &ctx_value);
 found:
 	if (inode)
 		inode_lookup (inode);