ext2: fix race between setxattr and write back

[PATCH v2 0/4] ext2: fix race between setxattr and write back

Ye Bin posted 4 patches 2 years, 4 months ago

Download series mbox

fs/ext2/balloc.c | 32 +++++++++++++++++---------------
fs/ext2/ext2.h   |  8 ++++++--
fs/ext2/inode.c  |  2 +-
fs/ext2/xattr.c  |  4 +++-
4 files changed, 27 insertions(+), 19 deletions(-)

Expand all Fold all

[PATCH v2 0/4] ext2: fix race between setxattr and write back

Posted by Ye Bin 2 years, 4 months ago

Diff v2 vs v1:
1. Do not use reservation window when allocate block for xattr.
2. Remove BUG() in ext2_try_to_allocate_with_rsv().

Ye Bin (4):
  ext2: remove ext2_new_block()
  ext2: introduce flag argument for ext2_new_blocks()
  ext2: fix race between setxattr and write back
  ext2: dump current reservation window info

 fs/ext2/balloc.c | 32 +++++++++++++++++---------------
 fs/ext2/ext2.h   |  8 ++++++--
 fs/ext2/inode.c  |  2 +-
 fs/ext2/xattr.c  |  4 +++-
 4 files changed, 27 insertions(+), 19 deletions(-)

-- 
2.31.1

Re: [PATCH v2 0/4] ext2: fix race between setxattr and write back

Posted by Jan Kara 2 years, 4 months ago

On Tue 15-08-23 19:26:08, Ye Bin wrote:
> Diff v2 vs v1:
> 1. Do not use reservation window when allocate block for xattr.
> 2. Remove BUG() in ext2_try_to_allocate_with_rsv().

Thanks! I did't a few smaller spelling fixups on commit but otherwise the
patches look good so I've added them to my tree.

								Honza

> 
> Ye Bin (4):
>   ext2: remove ext2_new_block()
>   ext2: introduce flag argument for ext2_new_blocks()
>   ext2: fix race between setxattr and write back
>   ext2: dump current reservation window info
> 
>  fs/ext2/balloc.c | 32 +++++++++++++++++---------------
>  fs/ext2/ext2.h   |  8 ++++++--
>  fs/ext2/inode.c  |  2 +-
>  fs/ext2/xattr.c  |  4 +++-
>  4 files changed, 27 insertions(+), 19 deletions(-)
> 
> -- 
> 2.31.1
> 
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR