Bug 765785

Summary: Enhance 'readdirp' fop to send xattrs and inodes in _cbk()
Product: [Community] GlusterFS Reporter: Amar Tumballi <amarts>
Component: coreAssignee: Amar Tumballi <amarts>
Status: CLOSED CURRENTRELEASE QA Contact: Raghavendra Bhat <rabhat>
Severity: low Docs Contact:
Priority: high    
Version: mainlineCC: aavati, gluster-bugs, vraman
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-3.4.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-07-24 17:49:53 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: glusterfs-3.3.0qa43 Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 817967, 850503    

Description Amar Tumballi 2011-12-09 12:17:09 UTC
Description of problem: Without this feature, implementing a proper stat-prefetch/xattr-prefetch will not be complete.

Version-Release number of selected component (if applicable): git master, 3.3.0

How reproducible: Feature not present as of now..

Additional info: Enhancement Request

Comment 1 Anand Avati 2012-01-25 10:03:54 UTC
CHANGE: http://review.gluster.com/771 (core: get xattrs also as part of readdirp) merged in master by Anand Avati (avati)

Comment 2 Anand Avati 2012-02-21 05:14:33 UTC
CHANGE: http://review.gluster.com/2713 (md-cache: meta-data caching translator) merged in master by Anand Avati (avati)

Comment 3 Anand Avati 2012-02-21 05:14:55 UTC
CHANGE: http://review.gluster.com/2714 (md-cache: replace stat-prefetch with md-cache in a backward compatible way) merged in master by Anand Avati (avati)

Comment 4 Anand Avati 2012-03-01 19:43:02 UTC
CHANGE: http://review.gluster.com/2853 (Fix memory leaks found in readdir) merged in master by Vijay Bellur (vijay)

Comment 5 Anand Avati 2012-03-19 12:45:24 UTC
CHANGE: http://review.gluster.com/2974 (libglusterfs/call-stub: ref dict while storing it in readdirp stub) merged in master by Vijay Bellur (vijay)

Comment 6 Raghavendra Bhat 2012-05-24 09:52:13 UTC
Now with readdirp the xattrs of the files are also received. Checked with glusterfs-3.3.0qa43.


default_readdirp (call_frame_t *frame, xlator_t *this, fd_t *fd,
1120	                  size_t size, off_t off)	1120	                  size_t size, off_t off, dict_t *dict)
1121	{	1121	{
1122	        STACK_WIND (frame, default_readdirp_cbk, FIRST_CHILD(this),	1122	        STACK_WIND (frame, default_readdirp_cbk, FIRST_CHILD(this),
1123	                    FIRST_CHILD(this)->fops->readdirp, fd, size, off);	1123	                    FIRST_CHILD(this)->fops->readdirp, fd, size, off, dict);
1124	        return 0;	1124	        return 0;
1125	}	1125	}

Comment 7 Vijay Bellur 2012-07-04 08:15:51 UTC
CHANGE: http://review.gluster.com/3538 (md-cache: cache SELinux and Posix ACL xattrs only if enabled) merged in master by Anand Avati (avati)

Comment 8 Anand Avati 2013-05-13 20:34:52 UTC
REVIEW: http://review.gluster.org/4994 (md-cache: Make options structure NULL terminated) posted (#2) for review on master by Anand Avati (avati)

Comment 9 Anand Avati 2013-05-17 06:38:12 UTC
REVIEW: http://review.gluster.org/5028 (md-cache: Make options structure NULL terminated) posted (#1) for review on release-3.4 by Krishnan Parthasarathi (kparthas)

Comment 10 Anand Avati 2013-05-21 06:30:17 UTC
COMMIT: http://review.gluster.org/5028 committed in release-3.4 by Anand Avati (avati) 
------
commit 7ecc172ade1b8c42086b53084c509d217c82229a
Author: Krishnan Parthasarathi <kparthas>
Date:   Mon May 13 17:50:56 2013 +0530

    md-cache: Make options structure NULL terminated
    
    Change-Id: I8aa4f90ba7e1eecf3f978be04f8550049275464f
    BUG: 765785
    Signed-off-by: Krishnan Parthasarathi <kparthas>
    Reviewed-on: http://review.gluster.org/5028
    Reviewed-by: Amar Tumballi <amarts>
    Tested-by: Gluster Build System <jenkins.com>

Comment 11 Anand Avati 2013-05-21 06:30:54 UTC
REVIEW: http://review.gluster.org/4994 (md-cache: Make options structure NULL terminated.) posted (#3) for review on master by Anand Avati (avati)

Comment 12 Anand Avati 2013-05-21 06:31:23 UTC
COMMIT: http://review.gluster.org/4994 committed in master by Anand Avati (avati) 
------
commit 2eed9c78569311e4fa75735d67e8838cc79109a7
Author: Krishnan Parthasarathi <kparthas>
Date:   Mon May 13 17:50:56 2013 +0530

    md-cache: Make options structure NULL terminated.
    
    Change-Id: I8aa4f90ba7e1eecf3f978be04f8550049275464f
    BUG: 765785
    Signed-off-by: Krishnan Parthasarathi <kparthas>
    Reviewed-on: http://review.gluster.org/4994
    Reviewed-by: Anand Avati <avati>
    Tested-by: Anand Avati <avati>