mbox series

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

Message ID 20230815112612.221145-1-yebin10@huawei.com
Headers show
Series ext2: fix race between setxattr and write back | expand

Message

yebin (H) Aug. 15, 2023, 11:26 a.m. UTC 
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(-)

Comments

Jan Kara Aug. 16, 2023, 3:53 p.m. UTC | #1 
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
>