[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20091005160839.GA13626@kroah.com>
Date: Mon, 5 Oct 2009 09:08:40 -0700
From: Greg KH <greg@...ah.com>
To: Herton Ronaldo Krzesinski <herton@...driva.com.br>
Cc: Greg KH <gregkh@...e.de>, linux-kernel@...r.kernel.org,
stable@...nel.org, Dave Jones <davej@...hat.com>,
akpm@...ux-foundation.org, torvalds@...ux-foundation.org,
stable-review@...nel.org, alan@...rguk.ukuu.org.uk
Subject: Re: [Stable-review] [079/136] [CPUFREQ] Fix NULL ptr regression in
powernow-k8
On Sat, Oct 03, 2009 at 12:19:08PM -0300, Herton Ronaldo Krzesinski wrote:
> Em Qui 01 Out 2009, às 22:17:07, Greg KH escreveu:
> > 2.6.31-stable review patch. If anyone has any objections, please let us know.
> >
> > ------------------
> > From: Kurt Roeckx <kurt@...ckx.be>
> >
> > commit f0adb134d8dc9993a9998dc50845ec4f6ff4fadc upstream.
> >
> > Fixes bugzilla #13780
> >
> > From: Kurt Roeckx <kurt@...ckx.be>
> > Signed-off-by: Dave Jones <davej@...hat.com>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@...e.de>
> >
> > ---
> > arch/x86/kernel/cpu/cpufreq/powernow-k8.c | 15 ++++++++-------
> > 1 file changed, 8 insertions(+), 7 deletions(-)
> >
> > --- a/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
> > +++ b/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
> > @@ -605,9 +605,10 @@ static int check_pst_table(struct powern
> > return 0;
> > }
> >
> > -static void invalidate_entry(struct powernow_k8_data *data, unsigned int entry)
> > +static void invalidate_entry(struct cpufreq_frequency_table *powernow_table,
> > + unsigned int entry)
> > {
> > - data->powernow_table[entry].frequency = CPUFREQ_ENTRY_INVALID;
> > + powernow_table[entry].frequency = CPUFREQ_ENTRY_INVALID;
> > }
> >
> > static void print_basics(struct powernow_k8_data *data)
> > @@ -914,13 +915,13 @@ static int fill_powernow_table_pstate(st
> > "bad value %d.\n", i, index);
> > printk(KERN_ERR PFX "Please report to BIOS "
> > "manufacturer\n");
> > - invalidate_entry(data, i);
> > + invalidate_entry(powernow_table, i);
> > continue;
> > }
> > rdmsr(MSR_PSTATE_DEF_BASE + index, lo, hi);
> > if (!(hi & HW_PSTATE_VALID_MASK)) {
> > dprintk("invalid pstate %d, ignoring\n", index);
> > - invalidate_entry(data, i);
> > + invalidate_entry(powernow_table, i);
> > continue;
> > }
> >
> > @@ -970,7 +971,7 @@ static int fill_powernow_table_fidvid(st
> > /* verify frequency is OK */
> > if ((freq > (MAX_FREQ * 1000)) || (freq < (MIN_FREQ * 1000))) {
> > dprintk("invalid freq %u kHz, ignoring\n", freq);
> > - invalidate_entry(data, i);
> > + invalidate_entry(powernow_table, i);
> > continue;
> > }
> >
> > @@ -978,7 +979,7 @@ static int fill_powernow_table_fidvid(st
> > * BIOSs are using "off" to indicate invalid */
> > if (vid == VID_OFF) {
> > dprintk("invalid vid %u, ignoring\n", vid);
> > - invalidate_entry(data, i);
> > + invalidate_entry(powernow_table, i);
> > continue;
> > }
> >
> > @@ -1009,7 +1010,7 @@ static int fill_powernow_table_fidvid(st
> > (unsigned int)
> > (data->acpi_data.states[i].core_frequency
> > * 1000));
> > - invalidate_entry(data, i);
> > + invalidate_entry(powernow_table, i);
> > continue;
> > }
> > }
> >
>
> With this applied I get following warning:
> arch/x86/kernel/cpu/cpufreq/powernow-k8.c:1001: warning: passing argument 1 of 'invalidate_entry' from incompatible pointer type
>
> There is a missing data->powernow_table change, following diff on top fixes it:
>
> diff -p -up linux-2.6.31/arch/x86/kernel/cpu/cpufreq/powernow-k8.c.orig linux-2.6.31/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
> --- linux-2.6.31/arch/x86/kernel/cpu/cpufreq/powernow-k8.c.orig 2009-10-03 12:06:38.000000000 -0300
> +++ linux-2.6.31/arch/x86/kernel/cpu/cpufreq/powernow-k8.c 2009-10-03 12:08:34.000000000 -0300
> @@ -998,7 +998,7 @@ static int fill_powernow_table_fidvid(st
>
> dprintk("double low frequency table entry, "
> "ignoring it.\n");
> - invalidate_entry(data, i);
> + invalidate_entry(powernow_table, i);
> continue;
> } else
> cntlofreq = i;
>
Thanks, I've merged this into the original patch. For some reason my
build isn't letting me build this file so I didn't see the warning.
thanks again,
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists