+ if (ACPI_SUCCESS(acpi_get_handle(pr->handle, "_SUN", &handle))) { + status = acpi_evaluate_object(pr->handle, + "_SUN", NULL, &buffer); + if (ACPI_FAILURE(status)) + object.integer.value = -1; + } else { + object.integer.value = -1; + } + arch_set_topology_slot(pr->id, object.integer.value); + if (!object.processor.pblk_address) ACPI_DEBUG_PRINT((ACPI_DB_INFO, "No PBLK (NULL address)\n")); else if (object.processor.pblk_length != 6) I'm still seeing these messages: "ACPI: Invalid PBLK length [0]" for every cpu. Presumably because "object" is a union and when you did the "object.integer.value = -1;" you scribbled on some bits that were needed in the subsequent code. -Tony -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html