From: Johannes Thumshirn <johannes.thumsh...@wdc.com>

A following patch will add another caller of
btrfs_lookup_ordered_extent(), but from a bio's endio context.

btrfs_lookup_ordered_extent() uses spin_lock_irq() which unconditionally
disables interrupts. Change this to spin_lock_irqsave() so interrupts
aren't disabled and re-enabled unconditionally.

Signed-off-by: Johannes Thumshirn <johannes.thumsh...@wdc.com>
Reviewed-by: Josef Bacik <jo...@toxicpanda.com>
---
 fs/btrfs/ordered-data.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/fs/btrfs/ordered-data.c b/fs/btrfs/ordered-data.c
index 2dc707f02f00..fe235ab935d3 100644
--- a/fs/btrfs/ordered-data.c
+++ b/fs/btrfs/ordered-data.c
@@ -767,9 +767,10 @@ struct btrfs_ordered_extent 
*btrfs_lookup_ordered_extent(struct btrfs_inode *ino
        struct btrfs_ordered_inode_tree *tree;
        struct rb_node *node;
        struct btrfs_ordered_extent *entry = NULL;
+       unsigned long flags;
 
        tree = &inode->ordered_tree;
-       spin_lock_irq(&tree->lock);
+       spin_lock_irqsave(&tree->lock, flags);
        node = tree_search(tree, file_offset);
        if (!node)
                goto out;
@@ -780,7 +781,7 @@ struct btrfs_ordered_extent 
*btrfs_lookup_ordered_extent(struct btrfs_inode *ino
        if (entry)
                refcount_inc(&entry->refs);
 out:
-       spin_unlock_irq(&tree->lock);
+       spin_unlock_irqrestore(&tree->lock, flags);
        return entry;
 }
 
-- 
2.30.0

Reply via email to