From: Linus Torvalds <torvalds@linux-foundation.org>
To: Andrea Arcangeli <aarcange@redhat.com>
Cc: rong.a.chen@intel.com, Michal Hocko <mhocko@suse.com>,
s.priebe@profihost.ag, alex.williamson@redhat.com,
mgorman@techsingularity.net, zi.yan@cs.rutgers.edu,
Vlastimil Babka <vbabka@suse.cz>,
rientjes@google.com, kirill@shutemov.name,
Andrew Morton <akpm@linux-foundation.org>,
Linux List Kernel Mailing <linux-kernel@vger.kernel.org>,
lkp@01.org
Subject: Re: [LKP] [mm] ac5b2c1891: vm-scalability.throughput -61.3% regression
Date: Tue, 27 Nov 2018 14:50:05 -0800 [thread overview]
Message-ID: <CAHk-=wiEUPBzAOPkH1aHypnxW6+vVtmCw_vsOPgNZtOvQnrG8w@mail.gmail.com> (raw)
In-Reply-To: <20181127205737.GI16136@redhat.com>
On Tue, Nov 27, 2018 at 12:57 PM Andrea Arcangeli <aarcange@redhat.com> wrote:
>
> This difference can only happen with defrag=always, and that's not the
> current upstream default.
Ok, thanks. That makes it a bit less critical.
> That MADV_HUGEPAGE causes flights with NUMA balancing is not great
> indeed, qemu needs NUMA locality too, but then the badness caused by
> __GFP_THISNODE was a larger regression in the worst case for qemu.
[...]
> So the short term alternative again would be the alternate patch that
> does __GFP_THISNODE|GFP_ONLY_COMPACT appended below.
Sounds like we should probably do this. Particularly since Vlastimil
pointed out that we'd otherwise have issues with the back-port for 4.4
where that "defrag=always" was the default.
The patch doesn't look horrible, and it directly addresses this
particular issue.
Is there some reason we wouldn't want to do it?
Linus
next prev parent reply other threads:[~2018-11-27 22:50 UTC|newest]
Thread overview: 77+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-27 6:25 [LKP] [mm] ac5b2c1891: vm-scalability.throughput -61.3% regression kernel test robot
2018-11-27 17:08 ` Linus Torvalds
2018-11-27 18:17 ` Michal Hocko
2018-11-27 18:21 ` Michal Hocko
2018-11-27 19:05 ` Vlastimil Babka
2018-11-27 19:16 ` Vlastimil Babka
2018-11-27 20:57 ` Andrea Arcangeli
2018-11-27 22:50 ` Linus Torvalds [this message]
2018-11-28 6:30 ` Michal Hocko
2018-11-28 3:20 ` Huang, Ying
2018-11-28 16:48 ` Linus Torvalds
2018-11-28 18:39 ` Andrea Arcangeli
2018-11-28 23:10 ` David Rientjes
2018-12-03 18:01 ` Linus Torvalds
2018-12-03 18:14 ` Michal Hocko
2018-12-03 18:19 ` Linus Torvalds
2018-12-03 18:30 ` Michal Hocko
2018-12-03 18:45 ` Linus Torvalds
2018-12-03 18:59 ` Michal Hocko
2018-12-03 19:23 ` Andrea Arcangeli
2018-12-03 20:26 ` David Rientjes
2018-12-03 19:28 ` Linus Torvalds
2018-12-03 20:12 ` Andrea Arcangeli
2018-12-03 20:36 ` David Rientjes
2018-12-03 22:04 ` Linus Torvalds
2018-12-03 22:27 ` Linus Torvalds
2018-12-03 22:57 ` David Rientjes
2018-12-04 9:22 ` Vlastimil Babka
2018-12-04 10:45 ` Mel Gorman
2018-12-05 0:47 ` David Rientjes
2018-12-05 9:08 ` Michal Hocko
2018-12-05 10:43 ` Mel Gorman
2018-12-05 11:43 ` Michal Hocko
2018-12-05 10:06 ` Mel Gorman
2018-12-05 20:40 ` Andrea Arcangeli
2018-12-05 21:59 ` David Rientjes
2018-12-06 0:00 ` Andrea Arcangeli
2018-12-05 22:03 ` Linus Torvalds
2018-12-05 22:12 ` David Rientjes
2018-12-05 23:36 ` Andrea Arcangeli
2018-12-05 23:51 ` Linus Torvalds
2018-12-06 0:58 ` Linus Torvalds
2018-12-06 9:14 ` MADV_HUGEPAGE vs. NUMA semantic (was: Re: [LKP] [mm] ac5b2c1891: vm-scalability.throughput -61.3% regression) Michal Hocko
2018-12-06 23:49 ` David Rientjes
2018-12-07 7:34 ` Michal Hocko
2018-12-07 4:31 ` Linus Torvalds
2018-12-07 7:49 ` Michal Hocko
2018-12-07 9:06 ` Vlastimil Babka
2018-12-07 23:15 ` David Rientjes
2018-12-06 23:43 ` [LKP] [mm] ac5b2c1891: vm-scalability.throughput -61.3% regression David Rientjes
2018-12-07 4:01 ` Linus Torvalds
2018-12-10 0:29 ` David Rientjes
2018-12-10 4:49 ` Andrea Arcangeli
2018-12-12 0:37 ` David Rientjes
2018-12-12 9:50 ` Michal Hocko
2018-12-12 17:00 ` Andrea Arcangeli
2018-12-14 11:32 ` Michal Hocko
2018-12-12 10:14 ` Vlastimil Babka
2018-12-14 21:04 ` David Rientjes
2018-12-14 21:33 ` Vlastimil Babka
2018-12-21 22:18 ` David Rientjes
2018-12-22 12:08 ` Mel Gorman
2018-12-14 23:11 ` Mel Gorman
2018-12-21 22:15 ` David Rientjes
2018-12-12 10:44 ` Andrea Arcangeli
2019-04-15 11:48 ` Michal Hocko
2018-12-06 0:18 ` David Rientjes
2018-12-06 0:54 ` Andrea Arcangeli
2018-12-06 9:23 ` Vlastimil Babka
2018-12-03 20:39 ` David Rientjes
2018-12-03 21:25 ` Michal Hocko
2018-12-03 21:53 ` David Rientjes
2018-12-04 8:48 ` Michal Hocko
2018-12-05 0:07 ` David Rientjes
2018-12-05 10:18 ` Michal Hocko
2018-12-05 19:16 ` David Rientjes
2018-11-27 7:23 kernel test robot
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='CAHk-=wiEUPBzAOPkH1aHypnxW6+vVtmCw_vsOPgNZtOvQnrG8w@mail.gmail.com' \
--to=torvalds@linux-foundation.org \
--cc=aarcange@redhat.com \
--cc=akpm@linux-foundation.org \
--cc=alex.williamson@redhat.com \
--cc=kirill@shutemov.name \
--cc=linux-kernel@vger.kernel.org \
--cc=lkp@01.org \
--cc=mgorman@techsingularity.net \
--cc=mhocko@suse.com \
--cc=rientjes@google.com \
--cc=rong.a.chen@intel.com \
--cc=s.priebe@profihost.ag \
--cc=vbabka@suse.cz \
--cc=zi.yan@cs.rutgers.edu \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).