On Friday 28 May 2010 08:46:27 pm Len Brown wrote:
> On Wed, 26 May 2010, Thomas Renninger wrote:
> > Patch is against latest Linus master branch and is expected to be
> > safe bug fix.
> >
> > You get:
> > ACPI: HARDWARE addr space,NOT supported yet
> > for each ACPI defined CPU which status is active, but exceeds
> > maxcpus= count.
> >
> > As these "not booted" CPUs do not run an idle routine
> > and echo X >/proc/acpi/processor/*/throttling did not work
> > I couldn't find a way to really access not onlined/booted
> > machines. Still this should get fixed and
> > /proc/acpi/processor/X dirs of cores exceeding maxcpus
> > should not show up.
>
> Can we delete the /proc/acpi/processor/ dirs instead of maintaining them?
This is not about "not creating proc files", but about setting up
throttling/c-states for not booted CPUs.
The message:
ACPI: HARDWARE addr space,NOT supported yet
comes from throttling init code. It goes an undefined path because
struct cpuinfo_x86 *c = &cpu_data(cpu);
is uninitialized (all is zero...):
if ((c->x86_vendor != X86_VENDOR_INTEL) ||
!cpu_has(c, X86_FEATURE_ACPI)) {
printk(KERN_ERR PREFIX
"HARDWARE addr space,NOT supported yet\n");
Similar to what happens (happened?) on acpi processor (real) hotplug.
The proc files should vanish in a separate patch, but I am not sure what
could break.
I saw powertop still reading battery info from /proc, I thought hal is also
still accessing it, but I cannot say for sure.
Thomas
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
