From: "Jan Beulich" <JBeulich@novell.com>
To: Christian.Limpach@cl.cam.ac.uk
Cc: xen-devel@lists.xensource.com
Subject: Re: CONFIG_CPU_FREQ change
Date: Tue, 21 Feb 2006 08:31:20 +0100 [thread overview]
Message-ID: <43FACFD8.76F0.0078.0@novell.com> (raw)
In-Reply-To: <3d8eece20602201012t2aeea205xd496828f0706ec61@mail.gmail.com>
>>> christian.limpach@gmail.com 20.02.06 19:12:30 >>>
>On 2/20/06, Jan Beulich <JBeulich@novell.com> wrote:
>> To my surprise, c/s 8888 enables CPU_FREQ for x86-64 rather than disabling it for i386. Did anyone at your end
actually
>> test that if enabled this at least builds properly now? Not to mention that of course this also should work... If I
>> remember right, the main reason for posting a patch to disable it on 32-bits (similar to how it was on 64-bits
before)
>> was that there were some missing symbols, and I don't think I saw any changesets addressing this. Also, from
previous
>> discussion I seem to recall that it was generally agreed that there is little point in allowing a single domain
(even
>> dom0) to decide whether/what power management actions should be taken without knowing about the requirements of the
rest
>> of the system...
>
>I went with the final statement in the thread where you posted the
>patch, from Jeremy Katz stating, that it was working for dom0 and that
>disabling it would remove functionality. It is disabled for
>unprivileged guests. I'm happy to disable it entirely, if it doesn't
>build or if it doesn't work.
p4-clockmod.c and speedstep-ich.c reference cpu_sibling_map, which doesn't exist in Xen kernels. There was one other
symbol missing, but I don't recall which one (nor which module it was referenced from).
As far as 'working' goes, I would assume that any respective statements refer to a dom0-only scenario only.
Jan
prev parent reply other threads:[~2006-02-21 7:31 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-02-20 15:43 CONFIG_CPU_FREQ change Jan Beulich
2006-02-20 18:12 ` Christian Limpach
2006-02-20 18:21 ` Jeremy Katz
2006-02-21 7:31 ` Jan Beulich [this message]
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=43FACFD8.76F0.0078.0@novell.com \
--to=jbeulich@novell.com \
--cc=Christian.Limpach@cl.cam.ac.uk \
--cc=xen-devel@lists.xensource.com \
/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.