diff mbox series

[07/10] btrfs: remove irq disabling for leak_lock

Message ID 20230314165910.373347-8-hch@lst.de (mailing list archive)
State New, archived
Headers show
Series [01/10] btrfs: use a plain workqueue for ordered_extent processing | expand

Commit Message

Christoph Hellwig March 14, 2023, 4:59 p.m. UTC 
None of the extent state leak tracking is called from irq context,
so remove the irq disabling.

Signed-off-by: Christoph Hellwig <hch@lst.de>
---
 fs/btrfs/extent-io-tree.c | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

Comments

Johannes Thumshirn March 17, 2023, 10:35 a.m. UTC | #1 
Looks good,
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
diff mbox series

Patch

diff --git a/fs/btrfs/extent-io-tree.c b/fs/btrfs/extent-io-tree.c
index 29a225836e286e..caf11a10da71a0 100644
--- a/fs/btrfs/extent-io-tree.c
+++ b/fs/btrfs/extent-io-tree.c
@@ -21,20 +21,16 @@  static DEFINE_SPINLOCK(leak_lock);
 
 static inline void btrfs_leak_debug_add_state(struct extent_state *state)
 {
-	unsigned long flags;
-
-	spin_lock_irqsave(&leak_lock, flags);
+	spin_lock(&leak_lock);
 	list_add(&state->leak_list, &states);
-	spin_unlock_irqrestore(&leak_lock, flags);
+	spin_unlock(&leak_lock);
 }
 
 static inline void btrfs_leak_debug_del_state(struct extent_state *state)
 {
-	unsigned long flags;
-
-	spin_lock_irqsave(&leak_lock, flags);
+	spin_lock(&leak_lock);
 	list_del(&state->leak_list);
-	spin_unlock_irqrestore(&leak_lock, flags);
+	spin_unlock(&leak_lock);
 }
 
 static inline void btrfs_extent_state_leak_debug_check(void)