On Fri, Jun 19, 2020 at 09:24:19AM -0400, Pavel Tatashin wrote:
From: Alexander Duyck alexander.h.duyck@linux.intel.com
From: Alexander Duyck alexander.h.duyck@linux.intel.com
commit 56ec43d8b02719402c9fcf984feb52ec2300f8a5 upstream.
As best as I can tell the meminit_pfn_in_nid call is completely redundant. The deferred memory initialization is already making use of for_each_free_mem_range which in turn will call into __next_mem_range which will only return a memory range if it matches the node ID provided assuming it is not NUMA_NO_NODE.
I am operating on the assumption that there are no zones or pgdata_t structures that have a NUMA node of NUMA_NO_NODE associated with them. If that is the case then __next_mem_range will never return a memory range that doesn't match the zone's node ID and as such the check is redundant.
So one piece I would like to verify on this is if this works for ia64. Technically it was using a different approach to get the node ID, but it seems to have the node ID also encoded into the memblock. So I am assuming this is okay, but would like to get confirmation on that.
On my x86_64 test system with 384GB of memory per node I saw a reduction in initialization time from 2.80s to 1.85s as a result of this patch.
Link: http://lkml.kernel.org/r/20190405221219.12227.93957.stgit@localhost.localdom... Signed-off-by: Alexander Duyck alexander.h.duyck@linux.intel.com Reviewed-by: Pavel Tatashin pavel.tatashin@microsoft.com Acked-by: Michal Hocko mhocko@suse.com Cc: Mike Rapoport rppt@linux.ibm.com Cc: Dan Williams dan.j.williams@intel.com Cc: Dave Jiang dave.jiang@intel.com Cc: David S. Miller davem@davemloft.net Cc: Ingo Molnar mingo@kernel.org Cc: Khalid Aziz khalid.aziz@oracle.com Cc: "Kirill A. Shutemov" kirill.shutemov@linux.intel.com Cc: Laurent Dufour ldufour@linux.vnet.ibm.com Cc: Matthew Wilcox willy@infradead.org Cc: Mel Gorman mgorman@techsingularity.net Cc: Mike Rapoport rppt@linux.vnet.ibm.com Cc: Pavel Tatashin pasha.tatashin@soleen.com Cc: Vlastimil Babka vbabka@suse.cz Cc: yi.z.zhang@linux.intel.com Signed-off-by: Andrew Morton akpm@linux-foundation.org Signed-off-by: Linus Torvalds torvalds@linux-foundation.org Signed-off-by: Pavel Tatashin pasha.tatashin@soleen.com
mm/page_alloc.c | 51 ++++++++++++++----------------------------------- 1 file changed, 14 insertions(+), 37 deletions(-)
Given the recent changes backported to 4.19.y, is this series still needed?
If so, can you please regenerate it and resend as it does not apply to the current 4.19.y tree.
thanks,
greg k-h