Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-14 Thread Michal Hocko
On Fri 14-07-17 11:34:31, Vlastimil Babka wrote: > On 07/14/2017 11:13 AM, Michal Hocko wrote: > > On Fri 07-07-17 14:00:03, Vlastimil Babka wrote: > >> On 07/04/2017 07:17 AM, Joonsoo Kim wrote: > > Still, backporting b8f1a75d61d8 fixes this: > > [1.538379] allocated

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-14 Thread Michal Hocko
On Fri 14-07-17 11:34:31, Vlastimil Babka wrote: > On 07/14/2017 11:13 AM, Michal Hocko wrote: > > On Fri 07-07-17 14:00:03, Vlastimil Babka wrote: > >> On 07/04/2017 07:17 AM, Joonsoo Kim wrote: > > Still, backporting b8f1a75d61d8 fixes this: > > [1.538379] allocated

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-14 Thread Vlastimil Babka
On 07/14/2017 11:13 AM, Michal Hocko wrote: > On Fri 07-07-17 14:00:03, Vlastimil Babka wrote: >> On 07/04/2017 07:17 AM, Joonsoo Kim wrote: Still, backporting b8f1a75d61d8 fixes this: [1.538379] allocated 738197504 bytes of page_ext [1.539340] Node 0, zone

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-14 Thread Vlastimil Babka
On 07/14/2017 11:13 AM, Michal Hocko wrote: > On Fri 07-07-17 14:00:03, Vlastimil Babka wrote: >> On 07/04/2017 07:17 AM, Joonsoo Kim wrote: Still, backporting b8f1a75d61d8 fixes this: [1.538379] allocated 738197504 bytes of page_ext [1.539340] Node 0, zone

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-14 Thread Michal Hocko
On Fri 07-07-17 14:00:03, Vlastimil Babka wrote: > On 07/04/2017 07:17 AM, Joonsoo Kim wrote: > >> > >> Still, backporting b8f1a75d61d8 fixes this: > >> > >> [1.538379] allocated 738197504 bytes of page_ext > >> [1.539340] Node 0, zone DMA: page owner found early allocated 0 > >>

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-14 Thread Michal Hocko
On Fri 07-07-17 14:00:03, Vlastimil Babka wrote: > On 07/04/2017 07:17 AM, Joonsoo Kim wrote: > >> > >> Still, backporting b8f1a75d61d8 fixes this: > >> > >> [1.538379] allocated 738197504 bytes of page_ext > >> [1.539340] Node 0, zone DMA: page owner found early allocated 0 > >>

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-07 Thread Vlastimil Babka
On 07/04/2017 07:17 AM, Joonsoo Kim wrote: >> >> Still, backporting b8f1a75d61d8 fixes this: >> >> [1.538379] allocated 738197504 bytes of page_ext >> [1.539340] Node 0, zone DMA: page owner found early allocated 0 >> pages >> [1.540179] Node 0, zoneDMA32: page owner found

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-07 Thread Vlastimil Babka
On 07/04/2017 07:17 AM, Joonsoo Kim wrote: >> >> Still, backporting b8f1a75d61d8 fixes this: >> >> [1.538379] allocated 738197504 bytes of page_ext >> [1.539340] Node 0, zone DMA: page owner found early allocated 0 >> pages >> [1.540179] Node 0, zoneDMA32: page owner found

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-04 Thread Michal Hocko
On Tue 04-07-17 11:39:57, Vlastimil Babka wrote: > On 07/04/2017 07:23 AM, Joonsoo Kim wrote: > > On Mon, Jul 03, 2017 at 04:18:01PM +0200, Vlastimil Babka wrote: > >> allocated" looks much more sane there. But there's a warning nevertheless. > > > > Warning would comes from the fact that

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-04 Thread Michal Hocko
On Tue 04-07-17 11:39:57, Vlastimil Babka wrote: > On 07/04/2017 07:23 AM, Joonsoo Kim wrote: > > On Mon, Jul 03, 2017 at 04:18:01PM +0200, Vlastimil Babka wrote: > >> allocated" looks much more sane there. But there's a warning nevertheless. > > > > Warning would comes from the fact that

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-04 Thread Vlastimil Babka
On 07/04/2017 07:23 AM, Joonsoo Kim wrote: > On Mon, Jul 03, 2017 at 04:18:01PM +0200, Vlastimil Babka wrote: >> allocated" looks much more sane there. But there's a warning nevertheless. > > Warning would comes from the fact that drain_all_pages() is called > before mm_percpu_wq is initialised.

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-04 Thread Vlastimil Babka
On 07/04/2017 07:23 AM, Joonsoo Kim wrote: > On Mon, Jul 03, 2017 at 04:18:01PM +0200, Vlastimil Babka wrote: >> allocated" looks much more sane there. But there's a warning nevertheless. > > Warning would comes from the fact that drain_all_pages() is called > before mm_percpu_wq is initialised.

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-04 Thread Michal Hocko
On Tue 04-07-17 14:11:41, Joonsoo Kim wrote: > On Fri, Jun 30, 2017 at 05:44:16PM +0200, Michal Hocko wrote: > > On Fri 30-06-17 17:42:24, Michal Hocko wrote: > > [...] > > > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > > > index 16532fa0bb64..894697c1e6f5 100644 > > > ---

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-04 Thread Michal Hocko
On Tue 04-07-17 14:11:41, Joonsoo Kim wrote: > On Fri, Jun 30, 2017 at 05:44:16PM +0200, Michal Hocko wrote: > > On Fri 30-06-17 17:42:24, Michal Hocko wrote: > > [...] > > > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > > > index 16532fa0bb64..894697c1e6f5 100644 > > > ---

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Joonsoo Kim
On Mon, Jul 03, 2017 at 04:18:01PM +0200, Vlastimil Babka wrote: > On 07/03/2017 01:48 PM, Vlastimil Babka wrote: > > On 06/30/2017 04:18 PM, Michal Hocko wrote: > >> fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem > >> to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp.

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Joonsoo Kim
On Mon, Jul 03, 2017 at 04:18:01PM +0200, Vlastimil Babka wrote: > On 07/03/2017 01:48 PM, Vlastimil Babka wrote: > > On 06/30/2017 04:18 PM, Michal Hocko wrote: > >> fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem > >> to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp.

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Joonsoo Kim
On Mon, Jul 03, 2017 at 01:48:05PM +0200, Vlastimil Babka wrote: > On 06/30/2017 04:18 PM, Michal Hocko wrote: > > fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem > > to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. > > CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Joonsoo Kim
On Mon, Jul 03, 2017 at 01:48:05PM +0200, Vlastimil Babka wrote: > On 06/30/2017 04:18 PM, Michal Hocko wrote: > > fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem > > to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. > > CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Joonsoo Kim
On Fri, Jun 30, 2017 at 05:44:16PM +0200, Michal Hocko wrote: > On Fri 30-06-17 17:42:24, Michal Hocko wrote: > [...] > > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > > index 16532fa0bb64..894697c1e6f5 100644 > > --- a/include/linux/mmzone.h > > +++ b/include/linux/mmzone.h > >

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Joonsoo Kim
On Fri, Jun 30, 2017 at 05:44:16PM +0200, Michal Hocko wrote: > On Fri 30-06-17 17:42:24, Michal Hocko wrote: > [...] > > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > > index 16532fa0bb64..894697c1e6f5 100644 > > --- a/include/linux/mmzone.h > > +++ b/include/linux/mmzone.h > >

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Vlastimil Babka
On 07/03/2017 01:48 PM, Vlastimil Babka wrote: > On 06/30/2017 04:18 PM, Michal Hocko wrote: >> fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem >> to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. >> CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Vlastimil Babka
On 07/03/2017 01:48 PM, Vlastimil Babka wrote: > On 06/30/2017 04:18 PM, Michal Hocko wrote: >> fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem >> to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. >> CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Vlastimil Babka
On 06/30/2017 04:18 PM, Michal Hocko wrote: > fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem > to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. > CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with > !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) && >

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-07-03 Thread Vlastimil Babka
On 06/30/2017 04:18 PM, Michal Hocko wrote: > fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem > to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. > CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with > !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) && >

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-06-30 Thread Michal Hocko
On Fri 30-06-17 17:42:24, Michal Hocko wrote: [...] > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > index 16532fa0bb64..894697c1e6f5 100644 > --- a/include/linux/mmzone.h > +++ b/include/linux/mmzone.h > @@ -1055,6 +1055,7 @@ static inline struct zoneref >

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-06-30 Thread Michal Hocko
On Fri 30-06-17 17:42:24, Michal Hocko wrote: [...] > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > index 16532fa0bb64..894697c1e6f5 100644 > --- a/include/linux/mmzone.h > +++ b/include/linux/mmzone.h > @@ -1055,6 +1055,7 @@ static inline struct zoneref >

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-06-30 Thread Michal Hocko
On Fri 30-06-17 16:18:47, Michal Hocko wrote: > fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem > to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. > CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with > !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) &&

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-06-30 Thread Michal Hocko
On Fri 30-06-17 16:18:47, Michal Hocko wrote: > fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem > to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. > CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with > !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) &&

"mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-06-30 Thread Michal Hocko
fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) && !defined(CONFIG_HAVE_MEMBLOCK_NODE_MAP) I am not sure

"mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-06-30 Thread Michal Hocko
fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) && !defined(CONFIG_HAVE_MEMBLOCK_NODE_MAP) I am not sure