linux.git/fs/btrfs/locking.c, branch v4.9

btrfs: cleanup, remove stray return statements

2016-01-07T13:30:52+00:00

Signed-off-by: David Sterba

btrfs: comment the rest of implicit barriers before waitqueue_active

2015-10-10T16:42:00+00:00

There are atomic operations that imply the barrier for waitqueue_active
mixed in an if-condition.

Signed-off-by: David Sterba

btrfs: add comments to barriers before waitqueue_active

2015-10-10T16:40:04+00:00

Reduce number of undocumented barriers out there.

Signed-off-by: David Sterba

btrfs: Add WARN_ON() for double lock in btrfs_tree_lock()

2015-08-09T14:07:14+00:00

When a task trying to double lock a extent buffer, there are no
lockdep warning about it because this lock may be in "blocking_lock"
state, and make us hard to debug.

This patch add a WARN_ON() for above condition, it can not report
all deadlock cases(as lock between tasks), but at least helps us
some.

Signed-off-by: Zhao Lei 
Signed-off-by: Chris Mason

btrfs: fix lockups from btrfs_clear_path_blocking

2014-11-19T18:34:35+00:00

The fair reader/writer locks mean that btrfs_clear_path_blocking needs
to strictly follow lock ordering rules even when we already have
blocking locks on a given path.

Before we can clear a blocking lock on the path, we need to make sure
all of the locks have been converted to blocking.  This will remove lock
inversions against anyone spinning in write_lock() against the buffers
we're trying to get read locks on.  These inversions didn't exist before
the fair read/writer locks, but now we need to be more careful.

We papered over this deadlock in the past by changing
btrfs_try_read_lock() to be a true trylock against both the spinlock and
the blocking lock.  This was slower, and not sufficient to fix all the
deadlocks.  This patch adds a btrfs_tree_read_lock_atomic(), which
basically means get the spinlock but trylock on the blocking lock.

Signed-off-by: Chris Mason 
Signed-off-by: Josef Bacik 
Reported-by: Patrick Schmid 
cc: stable@vger.kernel.org #v3.15+

Btrfs: fix deadlocks with trylock on tree nodes

2014-06-19T21:19:55+00:00

The Btrfs tree trylock function is poorly named.  It always takes
the spinlock and backs off if the blocking lock is held.  This
can lead to surprising lockups because people expect it to really be a
trylock.

This commit makes it a pure trylock, both for the spinlock and the
blocking lock.  It also reworks the nested lock handling slightly to
avoid taking the read lock while a spinning write lock might be held.

Signed-off-by: Chris Mason

btrfs: make static code static & remove dead code

2013-05-06T19:55:23+00:00

Big patch, but all it does is add statics to functions which
are in fact static, then remove the associated dead-code fallout.

removed functions:

btrfs_iref_to_path()
__btrfs_lookup_delayed_deletion_item()
__btrfs_search_delayed_insertion_item()
__btrfs_search_delayed_deletion_item()
find_eb_for_page()
btrfs_find_block_group()
range_straddles_pages()
extent_range_uptodate()
btrfs_file_extent_length()
btrfs_scrub_cancel_devid()
btrfs_start_transaction_lflush()

btrfs_print_tree() is left because it is used for debugging.
btrfs_start_transaction_lflush() and btrfs_reada_detach() are
left for symmetry.

ulist.c functions are left, another patch will take care of those.

Signed-off-by: Eric Sandeen 
Signed-off-by: Josef Bacik

Btrfs: save us a read_lock

2013-02-20T14:37:17+00:00

This does not change the logic of code, but can save us a read_lock.

Signed-off-by: Liu Bo 
Signed-off-by: Josef Bacik

Btrfs: fix a misplaced address operator in a condition

2012-08-28T20:53:23+00:00

This should obviously not be "if (&flag)" but "if (flag)".

Signed-off-by: Stefan Behrens

Btrfs: reduce calls to wake_up on uncontended locks

2012-07-23T19:36:18+00:00

The btrfs locks were unconditionally calling wake_up as the
locks were released.  This lead to extra thrashing on the waitqueue,
especially for locks that were dominated by readers.

Signed-off-by: Chris Mason