From: Dave Hansen <dave.hansen@intel.com>
To: Oscar Salvador <osalvador@suse.de>, Yang Shi <shy828301@gmail.com>
Cc: Dave Hansen <dave.hansen@linux.intel.com>,
Linux MM <linux-mm@kvack.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Yang Shi <yang.shi@linux.alibaba.com>,
weixugc@google.com, Huang Ying <ying.huang@intel.com>,
Dan Williams <dan.j.williams@intel.com>,
David Hildenbrand <david@redhat.com>
Subject: Re: [PATCH 04/10] mm/migrate: make migrate_pages() return nr_succeeded
Date: Fri, 9 Apr 2021 08:50:49 -0700 [thread overview]
Message-ID: <d200de95-c6be-75fd-7302-e5368894d201@intel.com> (raw)
In-Reply-To: <YG9IthpDC/lri4Qh@localhost.localdomain>
On 4/8/21 11:17 AM, Oscar Salvador wrote:
> --- a/mm/page_alloc.c
> +++ b/mm/page_alloc.c
> @@ -8490,7 +8490,8 @@ static int __alloc_contig_migrate_range(struct compact_control *cc,
> cc->nr_migratepages -= nr_reclaimed;
>
> ret = migrate_pages(&cc->migratepages, alloc_migration_target,
> - NULL, (unsigned long)&mtc, cc->mode, MR_CONTIG_RANGE);
> + NULL, (unsigned long)&mtc, cc->mode, MR_CONTIG_RANGE,
> + NULL);
> }
> if (ret < 0) {
> putback_movable_pages(&cc->migratepages);
I also considered passing NULL to mean "I don't care about
nr_succeeded". I mostly avoided it to reduce churn. But, looking at it
here, it does seem cleaner.
Any objections to moving over to Oscar's suggestion?
next prev parent reply other threads:[~2021-04-09 15:50 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-01 18:32 [PATCH 00/10] [v7][RESEND] Migrate Pages in lieu of discard Dave Hansen
2021-04-01 18:32 ` [PATCH 01/10] mm/numa: node demotion data structure and lookup Dave Hansen
2021-04-08 8:03 ` Oscar Salvador
2021-04-08 21:29 ` Dave Hansen
2021-04-09 5:32 ` Wei Xu
2021-04-01 18:32 ` [PATCH 02/10] mm/numa: automatically generate node migration order Dave Hansen
2021-04-08 8:26 ` Oscar Salvador
2021-04-08 21:51 ` Dave Hansen
2021-04-09 8:17 ` Oscar Salvador
2021-04-10 3:07 ` Wei Xu
2021-04-14 8:08 ` Oscar Salvador
2021-04-14 8:11 ` Oscar Salvador
2021-04-14 8:12 ` David Hildenbrand
2021-04-14 8:14 ` Oscar Salvador
2021-04-14 8:20 ` David Hildenbrand
2021-04-15 4:07 ` Wei Xu
2021-04-15 15:35 ` Dave Hansen
2021-04-15 20:25 ` Wei Xu
2021-04-01 18:32 ` [PATCH 03/10] mm/migrate: update node demotion order during on hotplug events Dave Hansen
2021-04-08 9:52 ` Oscar Salvador
2021-04-09 10:14 ` Oscar Salvador
2021-04-09 10:15 ` Oscar Salvador
2021-04-09 18:59 ` David Hildenbrand
2021-04-12 7:19 ` Oscar Salvador
2021-04-12 9:19 ` David Hildenbrand
2021-04-01 18:32 ` [PATCH 04/10] mm/migrate: make migrate_pages() return nr_succeeded Dave Hansen
2021-04-01 22:35 ` Wei Xu
2021-04-01 23:21 ` Dave Hansen
2021-04-01 22:39 ` Wei Xu
2021-04-08 10:14 ` Oscar Salvador
2021-04-08 17:26 ` Yang Shi
2021-04-08 18:17 ` Oscar Salvador
2021-04-08 18:21 ` Oscar Salvador
2021-04-08 20:40 ` Yang Shi
2021-04-09 5:06 ` Oscar Salvador
2021-04-09 5:43 ` Wei Xu
2021-04-09 15:43 ` Yang Shi
2021-04-09 15:50 ` Dave Hansen [this message]
2021-04-09 18:47 ` Wei Xu
2021-04-09 20:10 ` Yang Shi
2021-04-01 18:32 ` [PATCH 05/10] mm/migrate: demote pages during reclaim Dave Hansen
2021-04-01 20:01 ` Yang Shi
2021-04-01 22:58 ` Dave Hansen
2021-04-08 10:47 ` Oscar Salvador
2021-04-10 3:35 ` Wei Xu
2021-04-01 18:32 ` [PATCH 06/10] mm/vmscan: add page demotion counter Dave Hansen
2021-04-10 3:40 ` Wei Xu
2021-04-01 18:32 ` [PATCH 07/10] mm/vmscan: add helper for querying ability to age anonymous pages Dave Hansen
2021-04-07 18:40 ` Wei Xu
2021-04-09 8:31 ` Oscar Salvador
2021-04-01 18:32 ` [PATCH 08/10] mm/vmscan: Consider anonymous pages without swap Dave Hansen
2021-04-02 0:55 ` Wei Xu
2021-04-01 18:32 ` [PATCH 09/10] mm/vmscan: never demote for memcg reclaim Dave Hansen
2021-04-02 0:18 ` Wei Xu
2021-04-01 18:32 ` [PATCH 10/10] mm/migrate: new zone_reclaim_mode to enable reclaim migration Dave Hansen
2021-04-01 20:06 ` Yang Shi
2021-04-10 4:10 ` Wei Xu
2021-04-16 12:35 ` [PATCH 00/10] [v7][RESEND] Migrate Pages in lieu of discard Michal Hocko
2021-04-16 14:26 ` Dave Hansen
2021-04-16 15:02 ` Michal Hocko
2021-04-21 2:39 ` Huang, Ying
2021-05-07 6:14 ` Huang, Ying
-- strict thread matches above, loose matches on Subject: below --
2021-03-04 23:59 [PATCH 00/10] [v6] " Dave Hansen
2021-03-04 23:59 ` [PATCH 04/10] mm/migrate: make migrate_pages() return nr_succeeded Dave Hansen
2021-03-09 0:05 ` Yang Shi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d200de95-c6be-75fd-7302-e5368894d201@intel.com \
--to=dave.hansen@intel.com \
--cc=dan.j.williams@intel.com \
--cc=dave.hansen@linux.intel.com \
--cc=david@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=osalvador@suse.de \
--cc=shy828301@gmail.com \
--cc=weixugc@google.com \
--cc=yang.shi@linux.alibaba.com \
--cc=ying.huang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.