[PATCH][vandrove@vc.cvut.cz: Re: 2.5.39-bk crashes here when P4 clock modulation enabled]

From: Dominik Brodowski (linux@brodo.de)
Date: Mon Sep 30 2002 - 15:37:33 EST


Hi Linus, Alan,

In two drivers a wrong size of memory was allocated for cpufreq_driver: as
it must include NR_CPUS times a struct cpufreq_policy (and not struct
cpufreq_freqs). Thanks to Petr Vandrovec for this patch.

        Dominik

diff -urdN linux/arch/i386/kernel/cpu/cpufreq/p4-clockmod.c linux/arch/i386/kernel/cpu/cpufreq/p4-clockmod.c
--- linux/arch/i386/kernel/cpu/cpufreq/p4-clockmod.c 2002-09-30 11:47:34.000000000 +0000
+++ linux/arch/i386/kernel/cpu/cpufreq/p4-clockmod.c 2002-09-30 19:52:33.000000000 +0000
@@ -221,7 +221,7 @@
 
         printk(KERN_INFO PFX "P4/Xeon(TM) CPU On-Demand Clock Modulation available\n");
         driver = kmalloc(sizeof(struct cpufreq_driver) +
- NR_CPUS * sizeof(struct cpufreq_freqs), GFP_KERNEL);
+ NR_CPUS * sizeof(struct cpufreq_policy), GFP_KERNEL);
         if (!driver)
                 return -ENOMEM;
 
diff -urdN linux/arch/i386/kernel/cpu/cpufreq/powernow-k6.c linux/arch/i386/kernel/cpu/cpufreq/powernow-k6.c
--- linux/arch/i386/kernel/cpu/cpufreq/powernow-k6.c 2002-09-30 11:47:18.000000000 +0000
+++ linux/arch/i386/kernel/cpu/cpufreq/powernow-k6.c 2002-09-30 19:53:01.000000000 +0000
@@ -234,7 +234,7 @@
 
         /* initialization of main "cpufreq" code*/
         driver = kmalloc(sizeof(struct cpufreq_driver) +
- NR_CPUS * sizeof(struct cpufreq_freqs), GFP_KERNEL);
+ NR_CPUS * sizeof(struct cpufreq_policy), GFP_KERNEL);
         if (!driver) {
                 release_region (POWERNOW_IOPORT, 16);
                 return -ENOMEM;

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Mon Sep 30 2002 - 22:00:47 EST