LGTM, applied On 01/30/2018 07:30 PM, tang.junhui@xxxxxxxxxx wrote: > From: Tang Junhui <tang.junhui@xxxxxxxxxx> > > In bch_mca_scan(), the return value should not be the number of freed btree > nodes, but the number of pages of freed btree nodes. > > Signed-off-by: Tang Junhui <tang.junhui@xxxxxxxxxx> > --- > drivers/md/bcache/btree.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/md/bcache/btree.c b/drivers/md/bcache/btree.c > index 81e8dc3..a5fbccc 100644 > --- a/drivers/md/bcache/btree.c > +++ b/drivers/md/bcache/btree.c > @@ -718,7 +718,7 @@ static unsigned long bch_mca_scan(struct shrinker *shrink, > } > out: > mutex_unlock(&c->bucket_lock); > - return freed; > + return freed * c->btree_pages; > } > > static unsigned long bch_mca_count(struct shrinker *shrink, >
