Attachment 294378 Details for Bug 430947 – RHEL5 version 1 fix for this patch

[patch] RHEL5 version 1 fix for this patch

version1.patch (text/plain), 2.50 KB, created by Prarit Bhargava on 2008-02-08 16:33:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Prarit Bhargava

Created: 2008-02-08 16:33:34 UTC

Size: 2.50 KB

patch

obsolete

>--- linux-2.6.18.x86_64.orig/arch/i386/kernel/cpu/cpufreq/powernow-k8.c	2008-02-06 13:36:44.000000000 -0500
>+++ linux-2.6.18.x86_64/arch/i386/kernel/cpu/cpufreq/powernow-k8.c	2008-02-08 11:13:33.000000000 -0500
>@@ -33,6 +33,7 @@
> #include <linux/string.h>
> #include <linux/cpumask.h>
> #include <linux/sched.h>	/* for current / set_cpus_allowed() */
>+#include <linux/dmi.h>
> 
> #include <asm/msr.h>
> #include <asm/io.h>
>@@ -774,12 +775,39 @@ static void powernow_k8_acpi_pst_values(
> 	data->vstable = (data->acpi_data->states[index].control >> VST_SHIFT) & VST_MASK;
> }
> 
>+/* Certain HP boxes have broken ACPI tables that incorrectly map cpus to the
>+   wrong domains.  This results in a very unstable system.  Until a new BIOS
>+   is released with the correct ordering, we have to avoid running the
>+   preregister call on these systems. There are likely other systems that
>+   have this problem.  For now*/
>+
>+static int preregister_acpi_perf = 1;
>+static int powernow_k8_no_preregister(struct dmi_system_id *d)
>+{
>+	preregister_acpi_perf = 0;
>+	printk(KERN_INFO "%s detected: disabling ACPI pre-initialization\n",
>+	       d->ident);
>+	return 0;
>+}
>+
>+static struct dmi_system_id __devinitdata noprereg_dmi_table[] = {
>+	{
>+		.callback = powernow_k8_no_preregister,
>+		.ident = "HP xw9400 Workstation",
>+		.matches = {
>+			DMI_MATCH(DMI_PRODUCT_NAME, "HP xw9400 Workstation"),
>+		},
>+	},
>+
>+	{}
>+};
>+
> static struct acpi_processor_performance *acpi_perf_data[NR_CPUS];
> static int preregister_valid = 0;
> 
>-static int powernow_k8_cpu_preinit_acpi()
>+static int powernow_k8_cpu_preinit_acpi(void)
> {
>-	int i; 
>+	int i;
> 	struct acpi_processor_performance *data;
> 	for_each_possible_cpu(i) {
> 		data = kzalloc(sizeof(struct acpi_processor_performance),
>@@ -795,10 +823,16 @@ static int powernow_k8_cpu_preinit_acpi(
> 		acpi_perf_data[i] = data;
> 	}
> 
>-	if (acpi_processor_preregister_performance(acpi_perf_data))
>-		return -ENODEV;
>-	else 
>-		preregister_valid = 1;
>+	dmi_check_system(noprereg_dmi_table);
>+
>+	if (preregister_acpi_perf) {
>+		if (acpi_processor_preregister_performance(acpi_perf_data))
>+			return -ENODEV;
>+		else
>+			preregister_valid = 1;
>+	} else
>+		printk(KERN_INFO "Disabling ACPI pre-initialization.\n");
>+
> 	return 0;
> }
> 
>@@ -1460,3 +1494,6 @@ module_exit(powernowk8_exit);
> 
> module_param(tscsync, int, 0);
> MODULE_PARM_DESC(tscsync, "enable tsc by synchronizing powernow-k8 changes");
>+module_param(preregister_acpi_perf, int, 1);
>+MODULE_PARM_DESC(preregister_acpi_perf, "allow preregistering of performance"
>+		 " related ACPI data");

Actions: View | Diff

Attachments on bug 430947: 294378 | 294398 | 295438 | 296644 | 297670