fs/ext4/xattr: Fix issues seen while deleting xattr inode(s)

[PATCH v2 0/2] fs/ext4/xattr: Fix issues seen while deleting xattr inode(s)

Posted by Bhupesh 1 year ago

Changes since v1:
----------------
- v1 can be seen here: https://lore.kernel.org/lkml/1dddb237-1460-8122-7caf-f0acd7c91b5c@igalia.com/T/
- As suggested by Cascardo while reviewing v1, there are two
  patches in v2:
  [PATCH 1/2] Ignores xattr entries past the end entry.
  [PATCH 2/2] Hold 'EXT4_I(inode)->xattr_sem' semaphore while deleting the inode.

Bhupesh (2):
  fs/ext4/xattr: Ignore xattrs past end
  fs/ext4/xattr: Check for 'xattr_sem' inside 'ext4_xattr_delete_inode'

 fs/ext4/xattr.c | 27 +++++++++++++++++++++++++--
 1 file changed, 25 insertions(+), 2 deletions(-)

-- 
2.38.1

Re: [PATCH v2 0/2] fs/ext4/xattr: Fix issues seen while deleting xattr inode(s)

Posted by Theodore Ts'o 10 months, 3 weeks ago

On Tue, 28 Jan 2025 13:57:49 +0530, Bhupesh wrote:
> Changes since v1:
> ----------------
> - v1 can be seen here: https://lore.kernel.org/lkml/1dddb237-1460-8122-7caf-f0acd7c91b5c@igalia.com/T/
> - As suggested by Cascardo while reviewing v1, there are two
>   patches in v2:
>   [PATCH 1/2] Ignores xattr entries past the end entry.
>   [PATCH 2/2] Hold 'EXT4_I(inode)->xattr_sem' semaphore while deleting the inode.
> 
> [...]

Applied, thanks!

[1/2] fs/ext4/xattr: Ignore xattrs past end
      commit: c8e008b60492cf6fd31ef127aea6d02fd3d314cd
[2/2] fs/ext4/xattr: Check for 'xattr_sem' inside 'ext4_xattr_delete_inode'
      (dropped because it breaks ext4/026)

Best regards,
-- 
Theodore Ts'o <tytso@mit.edu>