Patch attempt #2 with a comment added. Fix a for-loop in sn_hwperf_geoid_to_cnode(). It needs to loop over num_cnodes to ensure it can still process TIO nodes in addition to compute nodes on systems with many nodes. Signed-off-by: Dean Roe <roe@xxxxxxx> Index: linux-2.6/arch/ia64/sn/kernel/sn2/sn_hwperf.c =================================================================== --- linux-2.6.orig/arch/ia64/sn/kernel/sn2/sn_hwperf.c +++ linux-2.6/arch/ia64/sn/kernel/sn2/sn_hwperf.c @@ -110,7 +110,11 @@ if (sn_hwperf_location_to_bpos(location, &rack, &bay, &slot, &slab)) return -1; - for_each_node(cnode) { + /* + * FIXME: replace with cleaner for_each_XXX macro which addresses + * both compute and IO nodes once ACPI3.0 is available. + */ + for (cnode = 0; cnode < num_cnodes; cnode++) { geoid = cnodeid_get_geoid(cnode); module_id = geo_module(geoid); this_rack = MODULE_GET_RACK(module_id); - : send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |