http://bugzilla.kernel.org/show_bug.cgi?id=15174





--- Comment #8 from Zhang Rui <rui.zh...@intel.com>  2010-03-12 07:46:27 ---
> -     pr->performance->state_count = pss->package.count;
> +     pr->performance->state_count = 0;
>       pr->performance->states =
>           kmalloc(sizeof(struct acpi_processor_px) * pss->package.count,
>                   GFP_KERNEL);
> @@ -286,7 +286,7 @@
> 
>       for (i = 0; i < pr->performance->state_count; i++) {

shouldn't we change this to
        for (i = 0; i < pss->package.count; i++) {
as pr->performance->state_count is 0 at this time.

> -             struct acpi_processor_px *px = &(pr->performance->states[i]);
> +             struct acpi_processor_px *px =
> &(pr->performance->states[pss->package.count]);
...

Okay, I'll refresh your patch and see if we can push it to upstream kernel.
But can you test it for me before sending it out?

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to