diff mbox series

btrfs: remove stale comment from btrfs_free_extent()

Message ID 61aee02e43101491f4e83144fd58833f04848e16.1694790065.git.fdmanana@suse.com (mailing list archive)
State New, archived
Headers show
Series btrfs: remove stale comment from btrfs_free_extent() | expand

Commit Message

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

A comment at btrfs_free_extent() mentions the call to btrfs_pin_extent()
unlocks the pinned mutex, however that mutex is long gone, it was removed
in 2009 by commit 04018de5d41e ("Btrfs: kill the pinned_mutex"). So just
delete the comment.

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

Comments

David Sterba Sept. 15, 2023, 3:26 p.m. UTC | #1 
On Fri, Sep 15, 2023 at 04:02:56PM +0100, fdmanana@kernel.org wrote:
> From: Filipe Manana <fdmanana@suse.com>
> 
> A comment at btrfs_free_extent() mentions the call to btrfs_pin_extent()
> unlocks the pinned mutex, however that mutex is long gone, it was removed
> in 2009 by commit 04018de5d41e ("Btrfs: kill the pinned_mutex"). So just
> delete the comment.
> 
> 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 6ef7319bb7ef..1701aadbfea3 100644
--- a/fs/btrfs/extent-tree.c
+++ b/fs/btrfs/extent-tree.c
@@ -3375,7 +3375,6 @@  int btrfs_free_extent(struct btrfs_trans_handle *trans, struct btrfs_ref *ref)
 	     ref->tree_ref.owning_root == BTRFS_TREE_LOG_OBJECTID) ||
 	    (ref->type == BTRFS_REF_DATA &&
 	     ref->data_ref.owning_root == BTRFS_TREE_LOG_OBJECTID)) {
-		/* unlocks the pinned mutex */
 		btrfs_pin_extent(trans, ref->bytenr, ref->len, 1);
 		ret = 0;
 	} else if (ref->type == BTRFS_REF_METADATA) {