Commit a698e08c authored by Kent Overstreet's avatar Kent Overstreet Committed by Linus Torvalds

bcache: Fix a shrinker deadlock

GFP_NOIO means we could be getting called recursively - mca_alloc() ->
mca_data_alloc() - definitely can't use mutex_lock(bucket_lock) then.
Whoops.
Signed-off-by: default avatarKent Overstreet <kmo@daterainc.com>
Cc: linux-stable <stable@vger.kernel.org> # >= v3.10
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 79e3dab9
...@@ -612,7 +612,7 @@ static unsigned long bch_mca_scan(struct shrinker *shrink, ...@@ -612,7 +612,7 @@ static unsigned long bch_mca_scan(struct shrinker *shrink,
return SHRINK_STOP; return SHRINK_STOP;
/* Return -1 if we can't do anything right now */ /* Return -1 if we can't do anything right now */
if (sc->gfp_mask & __GFP_WAIT) if (sc->gfp_mask & __GFP_IO)
mutex_lock(&c->bucket_lock); mutex_lock(&c->bucket_lock);
else if (!mutex_trylock(&c->bucket_lock)) else if (!mutex_trylock(&c->bucket_lock))
return -1; return -1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment