Bug 766571 - add 'fremovexattr()' fop into glusterfs fop list
Summary: add 'fremovexattr()' fop into glusterfs fop list
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: core
Version: mainline
Hardware: Unspecified
OS: Unspecified
high
unspecified
Target Milestone: ---
Assignee: Amar Tumballi
QA Contact: Raghavendra Bhat
URL:
Whiteboard:
Depends On:
Blocks: 817967
TreeView+ depends on / blocked
 
Reported: 2011-12-12 11:07 UTC by Amar Tumballi
Modified: 2013-12-19 00:07 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.4.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-24 17:41:38 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions: glusterfs-3.3.0qa43
Embargoed:

Attachments (Terms of Use)

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;
Note You need to log in before you can comment on or make changes to this bug.