diff options
| author | Christoph Hellwig <[email protected]> | 2025-03-20 07:52:13 +0000 |
|---|---|---|
| committer | Carlos Maiolino <[email protected]> | 2025-04-14 08:24:30 +0000 |
| commit | b73e05281cd9e37b5525641ca6f4544867372533 (patch) | |
| tree | 975200d666dff92fe453e3b389eb2b4d6204065b /fs/xfs/xfs_zone_alloc.c | |
| parent | Linux 6.15-rc2 (diff) | |
| download | kernel-b73e05281cd9e37b5525641ca6f4544867372533.tar.gz kernel-b73e05281cd9e37b5525641ca6f4544867372533.zip | |
xfs: remove the leftover xfs_{set,clear}_li_failed infrastructure
Marking a log item as failed kept a buffer reference around for
resubmission of inode and dquote items.
For inode items commit 298f7bec503f3 ("xfs: pin inode backing buffer to
the inode log item") started pinning the inode item buffers
unconditionally and removed the need for this. Later commit acc8f8628c37
("xfs: attach dquot buffer to dquot log item buffer") did the same for
dquot items but didn't fully clean up the xfs_clear_li_failed side
for them. Stop adding the extra pin for dquot items and remove the
helpers.
This happens to fix a call to xfs_buf_free with the AIL lock held,
which would be incorrect for the unlikely case freeing the buffer
ends up calling vfree.
Reported-by: Dan Carpenter <[email protected]>
Signed-off-by: Christoph Hellwig <[email protected]>
Reviewed-by: Carlos Maiolino <[email protected]>
Signed-off-by: Carlos Maiolino <[email protected]>
Diffstat (limited to 'fs/xfs/xfs_zone_alloc.c')
0 files changed, 0 insertions, 0 deletions
