diff for duplicates of <20150922193620.GA6942@linux.vnet.ibm.com> diff --git a/a/1.txt b/N1/1.txt index e325e52..6963b17 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -19,46 +19,3 @@ resending the patches with the changelog. Please note that the patch is in -mm tree already. ---8<--- ->From 86ead2520662c362d7b9ebd452ce1c33e156016f Mon Sep 17 00:00:00 2001 -From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com> -Date: Sun, 6 Sep 2015 12:54:40 +0530 -Subject: [PATCH V2 2/2] powerpc:numa Do not allocate bootmem memory for non - existing nodes - -With the setup_nr_nodes(), we have already initialized -node_possible_map. So it is safe to use for_each_node here. - -There are many places in the kernel that use hardcoded 'for' loop with -nr_node_ids, because all other architectures have numa nodes populated -serially. That should be reason we had maintained the same for powerpc. - -But, since sparse numa node ids possible on powerpc, we -unnecessarily allocate memory for non existent numa nodes. - -For e.g., on a system with 0,1,16,17 as numa nodes nr_node_ids=18 -and we allocate memory for nodes 2-14. This patch we allocate -memory for only existing numa nodes. - -The patch is boot tested on a 4 node tuleta [ confirming with printks ]. -that it works as expected. - -Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com> ---- - arch/powerpc/mm/numa.c | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - -diff --git a/arch/powerpc/mm/numa.c b/arch/powerpc/mm/numa.c -index 8b9502a..8d8a541 100644 ---- a/arch/powerpc/mm/numa.c -+++ b/arch/powerpc/mm/numa.c -@@ -80,7 +80,7 @@ static void __init setup_node_to_cpumask_map(void) - setup_nr_node_ids(); - - /* allocate the map */ -- for (node = 0; node < nr_node_ids; node++) -+ for_each_node(node) - alloc_bootmem_cpumask_var(&node_to_cpumask_map[node]); - - /* cpumask_of_node() will now work */ --- -1.7.11.7 diff --git a/a/content_digest b/N1/content_digest index 6f9d107..21cb168 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -40,49 +40,6 @@ "\n" "Please note that the patch is in -mm tree already.\n" "\n" - "---8<---\n" - ">From 86ead2520662c362d7b9ebd452ce1c33e156016f Mon Sep 17 00:00:00 2001\n" - "From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>\n" - "Date: Sun, 6 Sep 2015 12:54:40 +0530\n" - "Subject: [PATCH V2 2/2] powerpc:numa Do not allocate bootmem memory for non\n" - " existing nodes\n" - "\n" - "With the setup_nr_nodes(), we have already initialized\n" - "node_possible_map. So it is safe to use for_each_node here.\n" - "\n" - "There are many places in the kernel that use hardcoded 'for' loop with\n" - "nr_node_ids, because all other architectures have numa nodes populated\n" - "serially. That should be reason we had maintained the same for powerpc.\n" - "\n" - "But, since sparse numa node ids possible on powerpc, we\n" - "unnecessarily allocate memory for non existent numa nodes.\n" - "\n" - "For e.g., on a system with 0,1,16,17 as numa nodes nr_node_ids=18\n" - "and we allocate memory for nodes 2-14. This patch we allocate\n" - "memory for only existing numa nodes.\n" - "\n" - "The patch is boot tested on a 4 node tuleta [ confirming with printks ].\n" - "that it works as expected.\n" - "\n" - "Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>\n" - "---\n" - " arch/powerpc/mm/numa.c | 2 +-\n" - " 1 file changed, 1 insertion(+), 1 deletion(-)\n" - "\n" - "diff --git a/arch/powerpc/mm/numa.c b/arch/powerpc/mm/numa.c\n" - "index 8b9502a..8d8a541 100644\n" - "--- a/arch/powerpc/mm/numa.c\n" - "+++ b/arch/powerpc/mm/numa.c\n" - "@@ -80,7 +80,7 @@ static void __init setup_node_to_cpumask_map(void)\n" - " \t\tsetup_nr_node_ids();\n" - " \n" - " \t/* allocate the map */\n" - "-\tfor (node = 0; node < nr_node_ids; node++)\n" - "+\tfor_each_node(node)\n" - " \t\talloc_bootmem_cpumask_var(&node_to_cpumask_map[node]);\n" - " \n" - " \t/* cpumask_of_node() will now work */\n" - "-- \n" - 1.7.11.7 + ---8<--- -6ade2968c7ea51b99b998ae52231dfd111b351d5d77e82aac4e5925ef113a913 +ab5be7cbd4b83d9056136a0fa426d5db797dbf15a73d74206804d29978e1c7cb
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.