From: David Rientjes <rientjes@google.com>
To: Christoph Lameter <cl@linux.com>
Cc: Vlastimil Babka <vbabka@suse.cz>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org,
Andrew Morton <akpm@linux-foundation.org>,
Mel Gorman <mgorman@suse.de>, Greg Thelen <gthelen@google.com>,
"Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>,
Pekka Enberg <penberg@kernel.org>,
Joonsoo Kim <iamjoonsoo.kim@lge.com>,
Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
Subject: Re: [RFC v2 4/4] mm: fallback for offline nodes in alloc_pages_node
Date: Fri, 24 Jul 2015 12:54:32 -0700 (PDT) [thread overview]
Message-ID: <alpine.DEB.2.10.1507241251460.5215@chino.kir.corp.google.com> (raw)
In-Reply-To: <alpine.DEB.2.11.1507241047110.6461@east.gentwo.org>
On Fri, 24 Jul 2015, Christoph Lameter wrote:
> On Fri, 24 Jul 2015, Vlastimil Babka wrote:
>
> > diff --git a/include/linux/gfp.h b/include/linux/gfp.h
> > index 531c72d..104a027 100644
> > --- a/include/linux/gfp.h
> > +++ b/include/linux/gfp.h
> > @@ -321,8 +321,12 @@ static inline struct page *alloc_pages_node(int nid, gfp_t gfp_mask,
> > unsigned int order)
> > {
> > /* Unknown node is current (or closest) node */
> > - if (nid == NUMA_NO_NODE)
> > + if (nid == NUMA_NO_NODE) {
> > nid = numa_mem_id();
> > + } else if (!node_online(nid)) {
> > + VM_WARN_ON(!node_online(nid));
> > + nid = numa_mem_id();
> > + }
>
> I would think you would only want this for debugging purposes. The
> overwhelming majority of hardware out there has no memory
> onlining/offlining capability after all and this adds the overhead to each
> call to alloc_pages_node.
>
> Make this dependo n CONFIG_VM_DEBUG or some such thing?
>
Yeah, the suggestion was for VM_WARN_ON() in the conditional, but the
placement has changed somewhat because of the new __alloc_pages_node(). I
think
else if (VM_WARN_ON(!node_online(nid)))
nid = numa_mem_id();
should be fine since it only triggers for CONFIG_DEBUG_VM.
WARNING: multiple messages have this Message-ID (diff)
From: David Rientjes <rientjes@google.com>
To: Christoph Lameter <cl@linux.com>
Cc: Vlastimil Babka <vbabka@suse.cz>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org,
Andrew Morton <akpm@linux-foundation.org>,
Mel Gorman <mgorman@suse.de>, Greg Thelen <gthelen@google.com>,
"Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>,
Pekka Enberg <penberg@kernel.org>,
Joonsoo Kim <iamjoonsoo.kim@lge.com>,
Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
Subject: Re: [RFC v2 4/4] mm: fallback for offline nodes in alloc_pages_node
Date: Fri, 24 Jul 2015 12:54:32 -0700 (PDT) [thread overview]
Message-ID: <alpine.DEB.2.10.1507241251460.5215@chino.kir.corp.google.com> (raw)
In-Reply-To: <alpine.DEB.2.11.1507241047110.6461@east.gentwo.org>
On Fri, 24 Jul 2015, Christoph Lameter wrote:
> On Fri, 24 Jul 2015, Vlastimil Babka wrote:
>
> > diff --git a/include/linux/gfp.h b/include/linux/gfp.h
> > index 531c72d..104a027 100644
> > --- a/include/linux/gfp.h
> > +++ b/include/linux/gfp.h
> > @@ -321,8 +321,12 @@ static inline struct page *alloc_pages_node(int nid, gfp_t gfp_mask,
> > unsigned int order)
> > {
> > /* Unknown node is current (or closest) node */
> > - if (nid == NUMA_NO_NODE)
> > + if (nid == NUMA_NO_NODE) {
> > nid = numa_mem_id();
> > + } else if (!node_online(nid)) {
> > + VM_WARN_ON(!node_online(nid));
> > + nid = numa_mem_id();
> > + }
>
> I would think you would only want this for debugging purposes. The
> overwhelming majority of hardware out there has no memory
> onlining/offlining capability after all and this adds the overhead to each
> call to alloc_pages_node.
>
> Make this dependo n CONFIG_VM_DEBUG or some such thing?
>
Yeah, the suggestion was for VM_WARN_ON() in the conditional, but the
placement has changed somewhat because of the new __alloc_pages_node(). I
think
else if (VM_WARN_ON(!node_online(nid)))
nid = numa_mem_id();
should be fine since it only triggers for CONFIG_DEBUG_VM.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2015-07-24 19:54 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-24 14:45 [RFC v2 1/4] mm: make alloc_pages_exact_node pass __GFP_THISNODE Vlastimil Babka
2015-07-24 14:45 ` Vlastimil Babka
2015-07-24 14:45 ` [RFC v2 2/4] mm: unify checks in alloc_pages_node family of functions Vlastimil Babka
2015-07-24 14:45 ` Vlastimil Babka
2015-07-24 20:09 ` David Rientjes
2015-07-24 20:09 ` David Rientjes
2015-07-24 14:45 ` [RFC v2 3/4] mm: use numa_mem_id in alloc_pages_node() Vlastimil Babka
2015-07-24 14:45 ` Vlastimil Babka
2015-07-24 20:09 ` David Rientjes
2015-07-24 20:09 ` David Rientjes
2015-07-29 13:31 ` Mel Gorman
2015-07-29 13:31 ` Mel Gorman
2015-07-24 14:45 ` [RFC v2 4/4] mm: fallback for offline nodes in alloc_pages_node Vlastimil Babka
2015-07-24 14:45 ` Vlastimil Babka
2015-07-24 15:48 ` Christoph Lameter
2015-07-24 15:48 ` Christoph Lameter
2015-07-24 19:54 ` David Rientjes [this message]
2015-07-24 19:54 ` David Rientjes
2015-07-24 20:39 ` Vlastimil Babka
2015-07-24 20:39 ` Vlastimil Babka
2015-07-24 23:06 ` David Rientjes
2015-07-24 23:06 ` David Rientjes
2015-07-27 11:29 ` Vlastimil Babka
2015-07-27 11:29 ` Vlastimil Babka
2015-07-24 20:08 ` [RFC v2 1/4] mm: make alloc_pages_exact_node pass __GFP_THISNODE David Rientjes
2015-07-24 20:08 ` David Rientjes
2015-07-24 20:52 ` Vlastimil Babka
2015-07-24 20:52 ` Vlastimil Babka
2015-07-24 23:09 ` David Rientjes
2015-07-24 23:09 ` David Rientjes
2015-07-27 15:39 ` Johannes Weiner
2015-07-27 15:39 ` Johannes Weiner
2015-07-27 15:47 ` Vlastimil Babka
2015-07-27 15:47 ` Vlastimil Babka
2015-07-29 13:30 ` Mel Gorman
2015-07-29 13:30 ` Mel Gorman
2015-07-30 14:33 ` Christoph Lameter
2015-07-30 14:33 ` Christoph Lameter
2015-07-30 15:14 ` Johannes Weiner
2015-07-30 15:14 ` Johannes Weiner
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=alpine.DEB.2.10.1507241251460.5215@chino.kir.corp.google.com \
--to=rientjes@google.com \
--cc=akpm@linux-foundation.org \
--cc=aneesh.kumar@linux.vnet.ibm.com \
--cc=cl@linux.com \
--cc=gthelen@google.com \
--cc=iamjoonsoo.kim@lge.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mgorman@suse.de \
--cc=n-horiguchi@ah.jp.nec.com \
--cc=penberg@kernel.org \
--cc=vbabka@suse.cz \
/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.