public inbox for linux-btrfs@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] btrfs: fix error message in btrfs_delete_delayed_dir_index()
@ 2026-02-17 18:25 Mark Harmstone
  2026-02-17 18:41 ` Filipe Manana
  0 siblings, 1 reply; 2+ messages in thread
From: Mark Harmstone @ 2026-02-17 18:25 UTC (permalink / raw)
  To: linux-btrfs, fdmanana; +Cc: Mark Harmstone

Fix the error message in btrfs_delete_delayed_dir_index() if
__btrfs_add_delayed_item() fails: the message says root, inode, index,
error, but we're actually passing index, root, inode, error.

Signed-off-by: Mark Harmstone <mark@harmstone.com>
Fixes: adc1ef55dc04 ("btrfs: add details to error messages at btrfs_delete_delayed_dir_index()")
---
 fs/btrfs/delayed-inode.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/btrfs/delayed-inode.c b/fs/btrfs/delayed-inode.c
index 1739a0b29c49..2746841c167d 100644
--- a/fs/btrfs/delayed-inode.c
+++ b/fs/btrfs/delayed-inode.c
@@ -1657,7 +1657,7 @@ int btrfs_delete_delayed_dir_index(struct btrfs_trans_handle *trans,
 	if (unlikely(ret)) {
 		btrfs_err(trans->fs_info,
 "failed to add delayed dir index item, root: %llu, inode: %llu, index: %llu, error: %d",
-			  index, btrfs_root_id(node->root), node->inode_id, ret);
+			  btrfs_root_id(node->root), node->inode_id, index, ret);
 		btrfs_delayed_item_release_metadata(dir->root, item);
 		btrfs_release_delayed_item(item);
 	}
-- 
2.52.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] btrfs: fix error message in btrfs_delete_delayed_dir_index()
  2026-02-17 18:25 [PATCH] btrfs: fix error message in btrfs_delete_delayed_dir_index() Mark Harmstone
@ 2026-02-17 18:41 ` Filipe Manana
  0 siblings, 0 replies; 2+ messages in thread
From: Filipe Manana @ 2026-02-17 18:41 UTC (permalink / raw)
  To: Mark Harmstone; +Cc: linux-btrfs

On Tue, Feb 17, 2026 at 6:29 PM Mark Harmstone <mark@harmstone.com> wrote:
>
> Fix the error message in btrfs_delete_delayed_dir_index() if
> __btrfs_add_delayed_item() fails: the message says root, inode, index,
> error, but we're actually passing index, root, inode, error.
>
> Signed-off-by: Mark Harmstone <mark@harmstone.com>
> Fixes: adc1ef55dc04 ("btrfs: add details to error messages at btrfs_delete_delayed_dir_index()")

Reviewed-by: Filipe Manana <fdmanana@suse.com>

Pushed it to for-next, thanks.

> ---
>  fs/btrfs/delayed-inode.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/btrfs/delayed-inode.c b/fs/btrfs/delayed-inode.c
> index 1739a0b29c49..2746841c167d 100644
> --- a/fs/btrfs/delayed-inode.c
> +++ b/fs/btrfs/delayed-inode.c
> @@ -1657,7 +1657,7 @@ int btrfs_delete_delayed_dir_index(struct btrfs_trans_handle *trans,
>         if (unlikely(ret)) {
>                 btrfs_err(trans->fs_info,
>  "failed to add delayed dir index item, root: %llu, inode: %llu, index: %llu, error: %d",
> -                         index, btrfs_root_id(node->root), node->inode_id, ret);
> +                         btrfs_root_id(node->root), node->inode_id, index, ret);
>                 btrfs_delayed_item_release_metadata(dir->root, item);
>                 btrfs_release_delayed_item(item);
>         }
> --
> 2.52.0
>
>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-02-17 18:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-17 18:25 [PATCH] btrfs: fix error message in btrfs_delete_delayed_dir_index() Mark Harmstone
2026-02-17 18:41 ` Filipe Manana

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox