Bug 1158621 - syncops do not pass xdata for many calls
Summary: syncops do not pass xdata for many calls
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: core
Version: mainline
Hardware: All
OS: All
unspecified
medium
Target Milestone: ---
Assignee: Raghavendra Talur
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-10-29 18:03 UTC by Jeff Darcy
Modified: 2017-08-11 12:39 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-08-11 12:39:30 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Jeff Darcy 2014-10-29 18:03:21 UTC
More missing code.  Examples (needed by NSR reconciliation):

	creat
	fsetattr
	truncate
	link
	mkdir
	mknod
	open
	opendir
	symlink
	unlink
	(p)write(v)

Comment 1 Anand Avati 2014-10-29 18:03:44 UTC
REVIEW: http://review.gluster.org/8769 (core/api: add xdata support) posted (#2) for review on master by Jeff Darcy (jdarcy)

Comment 2 Anand Avati 2014-10-29 18:51:39 UTC
REVIEW: http://review.gluster.org/8769 (core/api: add xdata support) posted (#3) for review on master by Jeff Darcy (jdarcy)

Comment 3 Anand Avati 2015-03-31 16:01:51 UTC
REVIEW: http://review.gluster.org/9859 (libglusterfs/syncop: Add xdata to all syncop calls) posted (#3) for review on master by Raghavendra Talur (rtalur)

Comment 4 Anand Avati 2015-04-02 11:34:35 UTC
REVIEW: http://review.gluster.org/9859 (libglusterfs/syncop: Add xdata to all syncop calls) posted (#4) for review on master by Raghavendra Talur (rtalur)

Comment 5 Anand Avati 2015-04-06 06:41:43 UTC
REVIEW: http://review.gluster.org/9859 (libglusterfs/syncop: Add xdata to all syncop calls) posted (#5) for review on master by Raghavendra Talur (rtalur)

Comment 6 Anand Avati 2015-04-06 06:51:46 UTC
REVIEW: http://review.gluster.org/9859 (libglusterfs/syncop: Add xdata to all syncop calls) posted (#6) for review on master by Raghavendra Talur (rtalur)

Comment 7 Anand Avati 2015-04-07 18:17:57 UTC
REVIEW: http://review.gluster.org/9859 (libglusterfs/syncop: Add xdata to all syncop calls) posted (#7) for review on master by Raghavendra Talur (rtalur)

Comment 8 Anand Avati 2015-04-07 19:02:30 UTC
REVIEW: http://review.gluster.org/9859 (libglusterfs/syncop: Add xdata to all syncop calls) posted (#8) for review on master by Raghavendra Talur (rtalur)

Comment 9 Anand Avati 2015-04-08 12:04:36 UTC
REVIEW: http://review.gluster.org/9859 (libglusterfs/syncop: Add xdata to all syncop calls) posted (#9) for review on master by Raghavendra Talur (rtalur)

Comment 10 Anand Avati 2015-04-08 15:16:35 UTC
COMMIT: http://review.gluster.org/9859 committed in master by Vijay Bellur (vbellur) 
------
commit 346e64e578573296028efa516cd93cfaf2b17b8f
Author: Raghavendra Talur <rtalur>
Date:   Wed Mar 11 18:36:01 2015 +0530

    libglusterfs/syncop: Add xdata to all syncop calls
    
    This patch adds support for xdata in both the
    request and response path of syncops.
    
    Few calls like lookup already had the support;
    have renamed variables in few places to maintain
    uniformity.
    
    xdata passed downwards is known as xdata_in
    and xdata passed upwards is known as xdata_out.
    
    There is an old patch by Jeff Darcy at
    http://review.gluster.org/#/c/8769/3 which does the
    same for some selected calls. It also brings in
    xdata support at gfapi level.
    
    xdata support at gfapi level would be introduced
    in subsequent patches.
    
    Change-Id: I340e94ebaf2a38e160e65bc30732e8fe1c532dcc
    BUG: 1158621
    Signed-off-by: Raghavendra Talur <rtalur>
    Reviewed-on: http://review.gluster.org/9859
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 11 Raghavendra Talur 2015-04-10 14:24:35 UTC
A new bug has been filed for tracking xdata support in gfapi.
Changing summary of this bug to limit request for xdata support in syncop.


Note You need to log in before you can comment on or make changes to this bug.