linux-kernel - Re: [PATCH] Add module aliases for acpi-cpufreq

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3231405.HBLxbMEfo4@vostro.rjw.lan>
Date:	Tue, 22 Jan 2013 22:57:05 +0100
From:	"Rafael J. Wysocki" <rjw@...k.pl>
To:	Borislav Petkov <bp@...en8.de>
Cc:	Matthew Garrett <matthew.garrett@...ula.com>,
	linux-acpi@...r.kernel.org, linux-kernel@...r.kernel.org,
	cpufreq@...r.kernel.org, Leonid Isaev <lisaev@...il.iu.edu>
Subject: Re: [PATCH] Add module aliases for acpi-cpufreq

On Tuesday, January 22, 2013 06:38:36 PM Borislav Petkov wrote:
> On Tue, Jan 22, 2013 at 11:42:24AM -0500, Matthew Garrett wrote:
> > The acpi core will call request_module("acpi-cpufreq") on subsystem init,
> > but this will fail if the module isn't available at that stage of boot.
> > Add some module aliases to ensure that udev can load the module on Intel
> > and AMD systems with the appropriate feature bits - I /think/ that this
> > will also work on VIA systems, but haven't verified that.
> > 
> > Signed-off-by: Matthew Garrett <matthew.garrett@...ula.com>
> 
> Let's add some more tags:
> 
> Link: http://lkml.kernel.org/r/1448223.sdUJnNSRz4@vostro.rjw.lan
> Tested-by: Leonid Isaev <lisaev@...il.iu.edu>
> Acked-by: Borislav Petkov <bp@...e.de>
> 
> Rafael, this is all yours. :-)

Thanks!

Do I think correctly that it should go into 3.7.y too?

Rafael


> > ---
> >  drivers/cpufreq/acpi-cpufreq.c | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > 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");
> 
> 
-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
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/