From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.7 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, UNPARSEABLE_RELAY,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6A779C2D0DB for ; Wed, 22 Jan 2020 19:40:04 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id CCBDA24655 for ; Wed, 22 Jan 2020 19:40:03 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CCBDA24655 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.alibaba.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 618526B0005; Wed, 22 Jan 2020 14:40:03 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 5EF686B0006; Wed, 22 Jan 2020 14:40:03 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 4DF966B0007; Wed, 22 Jan 2020 14:40:03 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0238.hostedemail.com [216.40.44.238]) by kanga.kvack.org (Postfix) with ESMTP id 352E96B0005 for ; Wed, 22 Jan 2020 14:40:03 -0500 (EST) Received: from smtpin27.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with SMTP id C09DC824805A for ; Wed, 22 Jan 2020 19:40:02 +0000 (UTC) X-FDA: 76406285844.27.note95_475badb6c26 X-HE-Tag: note95_475badb6c26 X-Filterd-Recvd-Size: 4020 Received: from out30-131.freemail.mail.aliyun.com (out30-131.freemail.mail.aliyun.com [115.124.30.131]) by imf01.hostedemail.com (Postfix) with ESMTP for ; Wed, 22 Jan 2020 19:40:01 +0000 (UTC) X-Alimail-AntiSpam:AC=PASS;BC=-1|-1;BR=01201311R181e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=e01e07484;MF=yang.shi@linux.alibaba.com;NM=1;PH=DS;RN=7;SR=0;TI=SMTPD_---0ToMZIoB_1579721990; Received: from localhost(mailfrom:yang.shi@linux.alibaba.com fp:SMTPD_---0ToMZIoB_1579721990) by smtp.aliyun-inc.com(127.0.0.1); Thu, 23 Jan 2020 03:39:58 +0800 From: Yang Shi To: mhocko@suse.com, richardw.yang@linux.intel.com, akpm@linux-foundation.org Cc: yang.shi@linux.alibaba.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: [PATCH] mm: move_pages: report the number of non-attempted pages Date: Thu, 23 Jan 2020 03:39:50 +0800 Message-Id: <1579721990-18672-1-git-send-email-yang.shi@linux.alibaba.com> X-Mailer: git-send-email 1.8.3.1 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Since commit a49bd4d71637 ("mm, numa: rework do_pages_move"), the semantic of move_pages() was changed to return the number of non-migrated pages (failed to migration) and the call would be aborted immediately if migrate_pages() returns positive value. But it didn't report the number of pages that we even haven't attempted to migrate. So, fix it by including non-attempted pages in the return value. Fixes: a49bd4d71637 ("mm, numa: rework do_pages_move") Suggested-by: Michal Hocko Cc: Wei Yang Cc: [4.17+] Signed-off-by: Yang Shi --- The patch is based off Wei Yang's cleanup patchset: https://lore.kernel.org/linux-mm/20200122011647.13636-1-richardw.yang@linux.intel.com/T/#t mm/migrate.c | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/mm/migrate.c b/mm/migrate.c index 591f2e5..51b1b76 100644 --- a/mm/migrate.c +++ b/mm/migrate.c @@ -1582,7 +1582,7 @@ static int add_page_for_migration(struct mm_struct *mm, unsigned long addr, static int move_pages_and_store_status(struct mm_struct *mm, int node, struct list_head *pagelist, int __user *status, - int start, int nr) + int start, int nr, unsigned long total) { int err; @@ -1590,8 +1590,17 @@ static int move_pages_and_store_status(struct mm_struct *mm, int node, return 0; err = do_move_pages_to_node(mm, pagelist, node); - if (err) + if (err) { + /* + * Possitive err means the number of failed pages to + * migrate. Since we are going to abort and return the + * number of non-migrated pages, so need incude the rest + * of the nr_pages that are not attempted as well. + */ + if (err > 0) + err += total - start - nr - 1; return err; + } err = store_status(status, start, node, nr); return err; } @@ -1640,7 +1649,8 @@ static int do_pages_move(struct mm_struct *mm, nodemask_t task_nodes, start = i; } else if (node != current_node) { err = move_pages_and_store_status(mm, current_node, - &pagelist, status, start, i - start); + &pagelist, status, start, i - start, + nr_pages); if (err) goto out; start = i; @@ -1670,7 +1680,7 @@ static int do_pages_move(struct mm_struct *mm, nodemask_t task_nodes, goto out_flush; err = move_pages_and_store_status(mm, current_node, &pagelist, - status, start, i - start); + status, start, i - start, nr_pages); if (err) goto out; current_node = NUMA_NO_NODE; @@ -1678,7 +1688,7 @@ static int do_pages_move(struct mm_struct *mm, nodemask_t task_nodes, out_flush: /* Make sure we do not overwrite the existing error */ err1 = move_pages_and_store_status(mm, current_node, &pagelist, - status, start, i - start); + status, start, i - start, nr_pages); if (err >= 0) err = err1; out: -- 1.8.3.1