Re: [PATCH v6 0/6] ACPI: Add _OST support for ACPI hotplug

[Toshi Kani <toshi.kani@hp.com>]

On Mon, 2012-07-16 at 08:52 -0400, Prarit Bhargava wrote:
> On 01/-10/-28163 02:59 PM, Kani, Toshimitsu wrote:
> >  * This version updates to use pr_<level> over printk to accommodate
> >    checkpatch update since this patchset was accepted in -next.
> > 
> > This patchset supports ACPI OSPM Status Indication (_OST) method for
> > ACPI CPU/memory/container hotplug operations and sysfs eject. After
> > an ACPI hotplug operation has completed, OSPM calls _OST to indicate
> > the result of the operation to the platform. If a platform does not
> > support _OST, this patchset has no effect on the platform.
> > 
> > This _OST support is enabled when all relevant ACPI hotplug operations,
> > such as CPU, memory and container hotplug, are enabled. This assures
> > consistent behavior among the hotplug operations with regarding the
> > _OST support.
> > 
> > Some platforms may require the OS to support _OST in order to support
> > ACPI hotplug operations. For example, if a platform has the management
> > console where user can request a hotplug operation from, this _OST
> > support would be required for the management console to show the result
> > of the hotplug request to user.
> > 
> > The _OST definition can be found in section 6.3.5 of ACPI 5.0 spec.
> > The HPPF spec below also describes hotplug flows with _OST.
> > 
> >   DIG64 Hot-Plug & Partitioning Flow (HPPF) Specification R1.0
> >   http://www.dig64.org/home/DIG64_HPPF_R1_0.pdf
> > 
> > The changes have been tested with simulated _OST methods.
> > 
> > v6:
> >  - Updated to use pr_<level> over printk.
> > 
> > v5:
> >  - Updated change log in patch 5/6.
> > 
> > v4:
> >  - Removed CONFIG_ACPI_HOTPLUG_OST option.
> > 
> > v3:
> >  - Added more descriptions to the change logs.
> > 
> > v2:
> >  - Added CONFIG_ACPI_HOTPLUG_OST option.
> >  - Added _OST support for container hotplug and sysfs eject.
> >  - Reordered patchset to enable _OST support bit of _OSC in the
> >    last patch.
> > 
> > ---
> > Toshi Kani (6):
> >  ACPI: Add an interface to evaluate _OST
> >  ACPI: Add _OST support for sysfs eject
> >  ACPI: Add _OST support for ACPI CPU hotplug
> >  ACPI: Add _OST support for ACPI memory hotplug
> >  ACPI: Add _OST support for ACPI container hotplug
> >  ACPI: Set hotplug _OST support bit to _OSC
> 
> I have a few systems on which I can do some _OST CPU and Memory hotplug.  These
> systems are newer Intel systems in which ACPI events can be triggered via
> switches to add CPUs.
> 
> I tested this patchset with CPU _add_ and haven't seen any issues.
> 
> So...
> 
> Tested-by: Prarit Bhargava <prarit@redhat.com>

Hi Prarit,

That's great!  Thanks a lot!

-Toshi


> 
> P.