Re: [arch-general] powernow-k8 fails to load with linux 3.7.2

Leonid Isaev
2013-01-21T05:14:32Z

[-- Attachment #1: Type: text/plain, Size: 1807 bytes --]

On Sat, 19 Jan 2013 13:08:14 +0100
Borislav Petkov <bp@alien8.de> wrote:

> On Fri, Jan 18, 2013 at 03:17:04PM +0000, Matthew Garrett wrote:
> > Can you try this (entirely untested) patch?
> >
> > diff --git a/drivers/cpufreq/acpi-cpufreq.c
> > b/drivers/cpufreq/acpi-cpufreq.c
> > index 0d048f6..7b0d49d 100644
> > --- a/drivers/cpufreq/acpi-cpufreq.c
> > +++ b/drivers/cpufreq/acpi-cpufreq.c
> > @@ -1030,4 +1030,11 @@ MODULE_PARM_DESC(acpi_pstate_strict,
> > late_initcall(acpi_cpufreq_init);
> > module_exit(acpi_cpufreq_exit);
> >
> > +static const struct x86_cpu_id acpi_cpufreq_ids[] = {
> > + X86_FEATURE_MATCH(X86_FEATURE_ACPI),
> > + X86_FEATURE_MATCH(X86_FEATURE_HW_PSTATE),
> > + {}
> > +};
> > +MODULE_DEVICE_TABLE(x86cpu, acpi_cpufreq_ids);
> > +
> > MODULE_ALIAS("acpi");
>
> Leonid, can you try Matthew's patch?
>
> This is adding the additional cpufreatures aliases:
>
> alias: x86cpu:vendor:*:family:*:model:*:feature:*00E8*
> alias: x86cpu:vendor:*:family:*:model:*:feature:*0016*
>
> and since powernow-k8 used to load on 3.6-stable, it probably happened
> because the 0xe8 feature and since 3.7 moved it to acpi-cpufreq, this
> patch should actually fix it.
>
> Thanks.
>

Matthew and Borislav, thanks a lot for the patch and explanation!

I applied the above patch on top of linux 3.7.3 -- it does generate the
aliases and fixes the autoprobing issue, at least on my AMD machine. Should we
expect it in 3.7.4 or 3.8? Otherwise, I'll forward it to archlinux devel ML to
be possibly included in current -stable kernel. Also, Borislav, this patch is
not in your patch series, is it?

Thanks again,
L.

--
Leonid Isaev
GnuPG key: 0x164B5A6D
Fingerprint: C0DF 20D0 C075 C3F1 E1BE 775A A7AE F6CB 164B 5A6D

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 490 bytes --]