Re: [PATCH] fstests: add generic test to verify xattr replace operations are atomic

2014-11-10 Thread Filipe David Manana
On Mon, Nov 10, 2014 at 1:31 AM, Dave Chinner da...@fromorbit.com wrote: On Mon, Nov 10, 2014 at 12:49:12AM +, Filipe David Manana wrote: On Sun, Nov 9, 2014 at 11:45 PM, Dave Chinner da...@fromorbit.com wrote: On Fri, Nov 07, 2014 at 08:40:26PM +, Filipe Manana wrote: This test

Re: [PATCH] fstests: add generic test to verify xattr replace operations are atomic

2014-11-10 Thread Dave Chinner
On Mon, Nov 10, 2014 at 11:53:00AM +, Filipe David Manana wrote: On Mon, Nov 10, 2014 at 1:31 AM, Dave Chinner da...@fromorbit.com wrote: On Mon, Nov 10, 2014 at 12:49:12AM +, Filipe David Manana wrote: On Sun, Nov 9, 2014 at 11:45 PM, Dave Chinner da...@fromorbit.com wrote: On

Re: [PATCH] fstests: add generic test to verify xattr replace operations are atomic

2014-11-09 Thread Dave Chinner
On Fri, Nov 07, 2014 at 08:40:26PM +, Filipe Manana wrote: This test verifies that replacing a xattr's value is an atomic operation. This is motivated by an issue in btrfs where replacing a xattr's value wasn't an atomic operation, it consisted of removing the old value and then inserting

Re: [PATCH] fstests: add generic test to verify xattr replace operations are atomic

2014-11-09 Thread Filipe David Manana
On Sun, Nov 9, 2014 at 11:45 PM, Dave Chinner da...@fromorbit.com wrote: On Fri, Nov 07, 2014 at 08:40:26PM +, Filipe Manana wrote: This test verifies that replacing a xattr's value is an atomic operation. This is motivated by an issue in btrfs where replacing a xattr's value wasn't an

Re: [PATCH] fstests: add generic test to verify xattr replace operations are atomic

2014-11-09 Thread Dave Chinner
On Mon, Nov 10, 2014 at 12:49:12AM +, Filipe David Manana wrote: On Sun, Nov 9, 2014 at 11:45 PM, Dave Chinner da...@fromorbit.com wrote: On Fri, Nov 07, 2014 at 08:40:26PM +, Filipe Manana wrote: This test verifies that replacing a xattr's value is an atomic operation. This is

[PATCH] fstests: add generic test to verify xattr replace operations are atomic

2014-11-07 Thread Filipe Manana
This test verifies that replacing a xattr's value is an atomic operation. This is motivated by an issue in btrfs where replacing a xattr's value wasn't an atomic operation, it consisted of removing the old value and then inserting the new value in a btree. This made readers (getxattr and