bcache: Revert "bcache: shrink btree node cache after bch_btree_check()"
authorColy Li <colyli@suse.de>
Thu, 13 Feb 2020 14:12:06 +0000 (22:12 +0800)
committerJens Axboe <axboe@kernel.dk>
Thu, 13 Feb 2020 15:53:49 +0000 (08:53 -0700)
commit309cc719a2c869b71a7388209a0a80d4284d98fd
tree48cefb7674129b9e02fc9088a27296e2dc765521
parent0b96da639a4874311e9b5156405f69ef9fc3bef8
bcache: Revert "bcache: shrink btree node cache after bch_btree_check()"

This reverts commit 1df3877ff6a4810054237c3259d900ded4468969.

In my testing, sometimes even all the cached btree nodes are freed,
creating gc and allocator kernel threads may still fail. Finally it
turns out that kthread_run() may fail if there is pending signal for
current task. And the pending signal is sent from OOM killer which
is triggered by memory consuption in bch_btree_check().

Therefore explicitly shrinking bcache btree node here does not help,
and after the shrinker callback is improved, as well as pending signals
are ignored before creating kernel threads, now such operation is
unncessary anymore.

This patch reverts the commit 1df3877ff6a4 ("bcache: shrink btree node
cache after bch_btree_check()") because we have better improvement now.

Signed-off-by: Coly Li <colyli@suse.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/md/bcache/super.c