Re: [PATCH] Btrfs: make xattr replace operations atomic

2014-11-07 Thread Filipe David Manana
On Fri, Nov 7, 2014 at 8:54 PM, Chris Mason wrote: > > > On Fri, Nov 7, 2014 at 3:39 PM, Filipe Manana wrote: >> >> Replacing a xattr consists of doing a lookup for its existing value, >> delete >> the current value from the respective leaf, release the search path and >> then >> finally insert t

Re: [PATCH] Btrfs: make xattr replace operations atomic

2014-11-07 Thread Chris Mason
On Fri, Nov 7, 2014 at 3:39 PM, Filipe Manana wrote: Replacing a xattr consists of doing a lookup for its existing value, delete the current value from the respective leaf, release the search path and then finally insert the new value. This leaves a time window where readers (getxattr, listx

[PATCH] Btrfs: make xattr replace operations atomic

2014-11-07 Thread Filipe Manana
Replacing a xattr consists of doing a lookup for its existing value, delete the current value from the respective leaf, release the search path and then finally insert the new value. This leaves a time window where readers (getxattr, listxattrs) won't see any value for the xattr. Xattrs are used to