lkml.org 
[lkml]   [2014]   [Apr]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v4 3/5] acpi_processor: do not mark present at boot but not onlined CPU as onlined

* Rafael J. Wysocki <rjw@rjwysocki.net> wrote:

> On Monday, April 14, 2014 05:11:15 PM Igor Mammedov wrote:
> > acpi_processor_add() assumes that present at boot CPUs
> > are always onlined, it is not so if a CPU failed to become
> > onlined. As result acpi_processor_add() will mark such CPU
> > device as onlined in sysfs and following attempts to
> > online/offline it using /sys/device/system/cpu/cpuX/online
> > attribute will fail.
> >
> > Do not poke into device internals in acpi_processor_add()
> > and touch "struct device { .offline }" attribute, since
> > for CPUs onlined at boot it's set by:
> > topology_init() -> arch_register_cpu() -> register_cpu()
> > before ACPI device tree is parsed, and for hotplugged
> > CPUs it's set when userspace onlines CPU via sysfs.
> >
> > Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> > ---
> > v2:
> > - fix regression in v1 leading to NULL pointer dereference
> > on CPU unplug, do not remove "pr->dev = dev;"
>
> Yeah.
>
> Does this patch depend on any other patches in the series?
>
> I don't think so, but just asking.
>
> If it doesn't, why is it part of this series at all?

I suspect because Igor was rigorously stress-testing CPU hotplug, and
was fixing all the bugs he saw, before adding the one feature he is
interested in.

The feature cannot be guaranteed to be correct, without having a
stable base to work on.

As such this series makes sense, as long as the fixes precede the
feature, and as long as the fixes are correct.

Consider it work in progress, with you being one of the reviewers who
makes sure the fixes are correct.

Thanks,

Ingo


\
 
 \ /
  Last update: 2014-04-15 08:41    [W:0.083 / U:0.408 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site