Initially, nr_scanned == nr_to_isolate, but in futher
nr_scanned may decreese faster, i.e., in any time
nr_scanned <= nr_to_isolate.

The check, we do in while (), is:

nr_scanned > 0 && nr_to_isolate > 0

and it may be reduced to nr_scanned > 0 because of above.
So, do that, and rename nr_scanned in nr_to_scan, because
it better fits to its physical sence.

Signed-off-by: Kirill Tkhai <ktk...@virtuozzo.com>
---
 mm/tcache.c |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/mm/tcache.c b/mm/tcache.c
index 458d3c45b6a..aaf906e3dd3 100644
--- a/mm/tcache.c
+++ b/mm/tcache.c
@@ -990,14 +990,13 @@ __tcache_insert_reclaim_node(struct tcache_nodeinfo *ni,
 static noinline_for_stack int
 __tcache_lru_isolate(struct tcache_nodeinfo *ni,
                     struct tcache_pool_nodeinfo *pni,
-                    struct page **pages, int nr_to_isolate)
+                    struct page **pages, int nr_to_scan)
 {
        struct tcache_node *node;
        struct page *page;
        int nr_isolated = 0;
-       int nr_scanned = nr_to_isolate;
 
-       while (nr_to_isolate > 0 && !list_empty(&pni->lru) && nr_scanned--) {
+       while (nr_to_scan-- > 0 && !list_empty(&pni->lru)) {
                page = list_first_entry(&pni->lru, struct page, lru);
 
                if (unlikely(!page_cache_get_speculative(page)))
@@ -1017,7 +1016,6 @@ __tcache_lru_isolate(struct tcache_nodeinfo *ni,
                tcache_hold_pool(node->pool);
 
                pages[nr_isolated++] = page;
-               nr_to_isolate--;
        }
        return nr_isolated;
 }

_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to