Re: [Stable-review] [079/136] [CPUFREQ] Fix NULL ptr regression in powernow-k8

[Greg KH <greg@kroah.com>]

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@roeckx.be>
> > 
> > commit f0adb134d8dc9993a9998dc50845ec4f6ff4fadc upstream.
> > 
> > Fixes bugzilla #13780
> > 
> > From: Kurt Roeckx <kurt@roeckx.be>
> > Signed-off-by: Dave Jones <davej@redhat.com>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@suse.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