On Tue, 2022-03-15 at 09:06 +1100, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
>
> When we release an intent that a whiteout applies to, it will not
> have been committed to the journal and so won't be in the AIL. Hence
> when we drop the last reference to the intent, we do not want to try
> to remove it from the AIL as that will trigger a filesystem
> shutdown. Hence make the removal of intents from the AIL conditional
> on them actually being in the AIL so we do the correct thing.
>
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
Ok, makes sense
Reviewed-by Allison Henderson <allison.henderson@xxxxxxxxxx>
> ---
> fs/xfs/xfs_bmap_item.c | 8 +++++---
> fs/xfs/xfs_extfree_item.c | 8 +++++---
> fs/xfs/xfs_refcount_item.c | 8 +++++---
> fs/xfs/xfs_rmap_item.c | 8 +++++---
> 4 files changed, 20 insertions(+), 12 deletions(-)
>
> diff --git a/fs/xfs/xfs_bmap_item.c b/fs/xfs/xfs_bmap_item.c
> index 2e7abfe35644..c6f47c13da27 100644
> --- a/fs/xfs/xfs_bmap_item.c
> +++ b/fs/xfs/xfs_bmap_item.c
> @@ -54,10 +54,12 @@ xfs_bui_release(
> struct xfs_bui_log_item *buip)
> {
> ASSERT(atomic_read(&buip->bui_refcount) > 0);
> - if (atomic_dec_and_test(&buip->bui_refcount)) {
> + if (!atomic_dec_and_test(&buip->bui_refcount))
> + return;
> +
> + if (test_bit(XFS_LI_IN_AIL, &buip->bui_item.li_flags))
> xfs_trans_ail_delete(&buip->bui_item,
> SHUTDOWN_LOG_IO_ERROR);
> - xfs_bui_item_free(buip);
> - }
> + xfs_bui_item_free(buip);
> }
>
>
> diff --git a/fs/xfs/xfs_extfree_item.c b/fs/xfs/xfs_extfree_item.c
> index 1d0e5cdc15f9..36eeac9413f5 100644
> --- a/fs/xfs/xfs_extfree_item.c
> +++ b/fs/xfs/xfs_extfree_item.c
> @@ -58,10 +58,12 @@ xfs_efi_release(
> struct xfs_efi_log_item *efip)
> {
> ASSERT(atomic_read(&efip->efi_refcount) > 0);
> - if (atomic_dec_and_test(&efip->efi_refcount)) {
> + if (!atomic_dec_and_test(&efip->efi_refcount))
> + return;
> +
> + if (test_bit(XFS_LI_IN_AIL, &efip->efi_item.li_flags))
> xfs_trans_ail_delete(&efip->efi_item,
> SHUTDOWN_LOG_IO_ERROR);
> - xfs_efi_item_free(efip);
> - }
> + xfs_efi_item_free(efip);
> }
>
> /*
> diff --git a/fs/xfs/xfs_refcount_item.c b/fs/xfs/xfs_refcount_item.c
> index ada5793ce550..d4632f2ceb89 100644
> --- a/fs/xfs/xfs_refcount_item.c
> +++ b/fs/xfs/xfs_refcount_item.c
> @@ -53,10 +53,12 @@ xfs_cui_release(
> struct xfs_cui_log_item *cuip)
> {
> ASSERT(atomic_read(&cuip->cui_refcount) > 0);
> - if (atomic_dec_and_test(&cuip->cui_refcount)) {
> + if (!atomic_dec_and_test(&cuip->cui_refcount))
> + return;
> +
> + if (test_bit(XFS_LI_IN_AIL, &cuip->cui_item.li_flags))
> xfs_trans_ail_delete(&cuip->cui_item,
> SHUTDOWN_LOG_IO_ERROR);
> - xfs_cui_item_free(cuip);
> - }
> + xfs_cui_item_free(cuip);
> }
>
>
> diff --git a/fs/xfs/xfs_rmap_item.c b/fs/xfs/xfs_rmap_item.c
> index 6e66e7718902..fa691a6ae737 100644
> --- a/fs/xfs/xfs_rmap_item.c
> +++ b/fs/xfs/xfs_rmap_item.c
> @@ -53,10 +53,12 @@ xfs_rui_release(
> struct xfs_rui_log_item *ruip)
> {
> ASSERT(atomic_read(&ruip->rui_refcount) > 0);
> - if (atomic_dec_and_test(&ruip->rui_refcount)) {
> + if (!atomic_dec_and_test(&ruip->rui_refcount))
> + return;
> +
> + if (test_bit(XFS_LI_IN_AIL, &ruip->rui_item.li_flags))
> xfs_trans_ail_delete(&ruip->rui_item,
> SHUTDOWN_LOG_IO_ERROR);
> - xfs_rui_item_free(ruip);
> - }
> + xfs_rui_item_free(ruip);
> }
>
> STATIC void
