diff mbox series

btrfs: remove useless comment from btrfs_pin_extent_for_log_replay()

Message ID 46c6009d05086ab67d5f2d8c0ebe7d749f1d8cea.1694790077.git.fdmanana@suse.com (mailing list archive)
State New, archived
Headers show
Series btrfs: remove useless comment from btrfs_pin_extent_for_log_replay() | expand

Commit Message

Filipe Manana Sept. 15, 2023, 3:03 p.m. UTC 
From: Filipe Manana <fdmanana@suse.com>

The comment on top of btrfs_pin_extent_for_log_replay() mentioning that
the function must be called within a transaction is pointless as of
commit 9fce5704542c ("btrfs: Make btrfs_pin_extent_for_log_replay take
transaction handle"), since the function now takes a transaction handle
as its first argument. So remove the comment because it's completely
useless now.

Signed-off-by: Filipe Manana <fdmanana@suse.com>
---
 fs/btrfs/extent-tree.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

David Sterba Sept. 15, 2023, 3:57 p.m. UTC | #1 
On Fri, Sep 15, 2023 at 04:03:57PM +0100, fdmanana@kernel.org wrote:
> From: Filipe Manana <fdmanana@suse.com>
> 
> The comment on top of btrfs_pin_extent_for_log_replay() mentioning that
> the function must be called within a transaction is pointless as of
> commit 9fce5704542c ("btrfs: Make btrfs_pin_extent_for_log_replay take
> transaction handle"), since the function now takes a transaction handle
> as its first argument. So remove the comment because it's completely
> useless now.
> 
> Signed-off-by: Filipe Manana <fdmanana@suse.com>

Added to misc-next, thanks.
diff mbox series

Patch

diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c
index 1701aadbfea3..adffeae41c0d 100644
--- a/fs/btrfs/extent-tree.c
+++ b/fs/btrfs/extent-tree.c
@@ -2566,9 +2566,6 @@  int btrfs_pin_extent(struct btrfs_trans_handle *trans,
 	return 0;
 }
 
-/*
- * this function must be called within transaction
- */
 int btrfs_pin_extent_for_log_replay(struct btrfs_trans_handle *trans,
 				    const struct extent_buffer *eb)
 {