Bug 766571

Summary:	add 'fremovexattr()' fop into glusterfs fop list
Product:	[Community] GlusterFS	Reporter:	Amar Tumballi <amarts>
Component:	core	Assignee:	Amar Tumballi <amarts>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Raghavendra Bhat <rabhat>
Severity:	unspecified	Docs Contact:
Priority:	high
Version:	mainline	CC:	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:41:38 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:
Bug Depends On:
Bug Blocks:	817967

Description Amar Tumballi 2011-12-12 11:07:22 UTC

Description of problem:
With out this, one cannot remove a xattr on a fd. And this is a valid syscall in a system.

Version-Release number of selected component (if applicable): All

How reproducible: Feature Request...


Additional info:

Need this feature to have complete protocol set.

Comment 1 Anand Avati 2012-01-23 17:43:38 UTC

CHANGE: http://review.gluster.com/803 (distribute: utilize new 'fremovexattr()' fop for rebalance) merged in master by Vijay Bellur (vijay)

Comment 2 Anand Avati 2012-01-25 10:24:25 UTC

CHANGE: http://review.gluster.com/778 (core: add 'fremovexattr()' fop) merged in master by Anand Avati (avati)

Comment 3 Anand Avati 2012-01-25 10:26:56 UTC

CHANGE: http://review.gluster.com/802 (complete the implementation of missing 'f**xattr()' fops) merged in master by Anand Avati (avati)

Comment 4 Anand Avati 2012-02-03 03:43:18 UTC

CHANGE: http://review.gluster.com/2691 (distribute: utilize new 'fremovexattr()' fop for rebalance) merged in master by Vijay Bellur (vijay)

Comment 5 Anand Avati 2012-02-03 16:14:05 UTC

CHANGE: http://review.gluster.com/2692 (extras/test: add a test file to test 'f-fops'.) merged in master by Vijay Bellur (vijay)

Comment 6 Raghavendra Bhat 2012-05-24 09:43:39 UTC

checked on glusterfs-3.3.0qa43. Now fremovexattr fop is present.

xlator.c:        SET_DEFAULT_FOP (fremovexattr);
xlator.h:typedef int32_t (*fop_fremovexattr_cbk_t) (call_frame_t *frame,
xlator.h:typedef int32_t (*fop_fremovexattr_t) (call_frame_t *frame,
xlator.h:        fop_fremovexattr_t   fremovexattr;
xlator.h:        fop_fremovexattr_cbk_t   fremovexattr_cbk;