Patchwork [v2,9/11] Re-enable Virtual Private Home Node capabilities

login
register
mail settings
Submitter Nathan Fontenot
Date March 25, 2013, 7 p.m.
Message ID <51509EB3.4020609@linux.vnet.ibm.com>
Download mbox | patch
Permalink /patch/230974/
State Changes Requested, archived
Delegated to: Michael Ellerman
Headers show

Comments

Nathan Fontenot - March 25, 2013, 7 p.m.
From: Jesse Larrew <jlarrew@linux.vnet.ibm.com>

The new PRRN firmware feature provides a more convenient and event-driven
interface than VPHN for notifying Linux of changes to the NUMA affinity of
platform resources. However, for practical reasons, it may not be feasible
for some customers to update to the latest firmware. For these customers,
the VPHN feature supported on previous firmware versions may still be the
best option.

The VPHN feature was previously disabled due to races with the load
balancing code when accessing the NUMA cpu maps, but the new stop_machine()
approach protects the NUMA cpu maps from these concurrent accesses. It
should be safe to re-enable this feature now.

Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com>
---
 arch/powerpc/mm/numa.c |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Patch

Index: powerpc/arch/powerpc/mm/numa.c
===================================================================
--- powerpc.orig/arch/powerpc/mm/numa.c	2013-03-20 12:27:46.000000000 -0500
+++ powerpc/arch/powerpc/mm/numa.c	2013-03-20 12:27:48.000000000 -0500
@@ -1545,9 +1545,8 @@ 
 			vphn_enabled = 0;
 			rc = of_reconfig_notifier_register(&dt_update_nb);
 		}
-	} else if (0 && firmware_has_feature(FW_FEATURE_VPHN) &&
+	} else if (firmware_has_feature(FW_FEATURE_VPHN) &&
 		   get_lppaca()->shared_proc) {
-		/* Disabled until races with load balancing are fixed */
 		if (!vphn_enabled) {
 			prrn_enabled = 0;
 			vphn_enabled = 1;