All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Renninger <trenn@suse.de>
To: alan-jenkins@tuffmail.co.uk, Kay Sievers <kasievers@suse.de>
Cc: Ingo Molnar <mingo@elte.hu>,
	Matthew Garrett <mjg59@srcf.ucam.org>,
	davej@codemonkey.org.uk, linux-kernel@vger.kernel.org,
	cpufreq@vger.kernel.org
Subject: Re: [PATCH] cpufreq: Change link order of x86 cpufreq modules
Date: Mon, 23 Feb 2009 22:12:23 +0100	[thread overview]
Message-ID: <200902232212.23925.trenn@suse.de> (raw)
In-Reply-To: <9b2b86520902190354m259f85d0n97795da7b1b8afdd@mail.gmail.com>

On Thursday 19 February 2009 12:54:06 pm Alan Jenkins wrote:
> On 2/18/09, Ingo Molnar <mingo@elte.hu> wrote:
> > * Matthew Garrett <mjg59@srcf.ucam.org> wrote:
> >> On Wed, Feb 18, 2009 at 07:31:37PM +0100, Ingo Molnar wrote:
> >> > Nice fix! Where does this information come from? Distro module
> >> > ordering magic? It's rather non-trivial.
> >>
> >> Pretty much. p4-clockmod is never the preferred option because
> >> it does no voltage scaling. speedstep-centrino is now almost
> >> entirely functionally replaced with acpi-cpufreq. The
> >> powernow-k8 issue was a personal communication from davej.
> >
> > I'm wondering whether that priority order should/could be
> > expressed in the module space too - so that distros wouldnt have
> > to replicate this. This is really a piece of information the
> > kernel is best at maintaining.
>
> The latest development version of module-init-tools (in the git tree)
> is designed to preserve the kernel link order when resolving builtin
> aliases.  If you have two modules which provide the alias "pci:123",
> they will be loaded in the same order as if they were builtin drivers.
>
> It should work if all the cpufreq drivers provide an alias
> "cpufreq-driver" and userspace just does "modprobe cpufreq-driver".
> You just need to be sure none of the cpufreq drivers provide *other*
> aliases which cause them to be loaded earlier, by udev or some other
> bootscript.
I wonder whether a "cpu:vendor:fam:model:flags:..." alias makes sense.
This would solve autoloading of most cpufreq drivers, but it never looked
important enough.
Other drivers that could make use of this:
  - k8temp/k10temp (Don't know about this one, but AFAIK there exists an
    MSR to read out temperature on latest AMD for which a hwmon driver
    is/should be written)?
  - Microcode update?
  - more?

     Thomas

  parent reply	other threads:[~2009-02-23 21:12 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-18 18:28 [PATCH] cpufreq: Change link order of x86 cpufreq modules Matthew Garrett
2009-02-18 18:31 ` Ingo Molnar
2009-02-18 18:34   ` Dave Jones
2009-02-18 18:36   ` Matthew Garrett
2009-02-18 19:09     ` Ingo Molnar
2009-02-19 11:54       ` Alan Jenkins
2009-02-19 12:03         ` Ingo Molnar
2009-02-23 21:12         ` Thomas Renninger [this message]
2009-02-23 21:47           ` Kay Sievers
2009-02-20 17:29     ` Scott James Remnant
2009-02-20 17:36       ` Matthew Garrett
2009-02-20 17:39         ` Scott James Remnant
2009-02-20 17:41           ` Dave Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200902232212.23925.trenn@suse.de \
    --to=trenn@suse.de \
    --cc=alan-jenkins@tuffmail.co.uk \
    --cc=cpufreq@vger.kernel.org \
    --cc=davej@codemonkey.org.uk \
    --cc=kasievers@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=mjg59@srcf.ucam.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.