[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1337892003.16730.411.camel@misato.fc.hp.com>
Date: Thu, 24 May 2012 14:40:03 -0600
From: Toshi Kani <toshi.kani@...com>
To: shuahkhan@...il.com
Cc: lenb@...nel.org, linux-acpi@...r.kernel.org, bhelgaas@...gle.com,
liuj97@...il.com, andi@...stfloor.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 1/6] ACPI: Add an interface to evaluate _OST
On Thu, 2012-05-24 at 12:09 -0600, Shuah Khan wrote:
> On Wed, 2012-05-23 at 20:25 -0600, Toshi Kani wrote:
:
> ACPI_HOTPLUG_OST is defined 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. When ACPI_HOTPLUG_OST is not defined, this function is
> a no-op.
>
> ACPI PCI hotplug is not enhanced to support _OST at this time since it
> is a legacy method being replaced by PCIe native hotplug. _OST support
> for ACPI PCI hotplug may be added in future if necessary.
:
> > +
> > +/* Enable _OST when all relevant hotplug operations are enabled */
> > +#if defined(CONFIG_ACPI_HOTPLUG_CPU) && \
> > + (defined(CONFIG_ACPI_HOTPLUG_MEMORY) || \
> > + defined(CONFIG_ACPI_HOTPLUG_MEMORY_MODULE)) && \
> > + (defined(CONFIG_ACPI_CONTAINER) || \
> > + defined(CONFIG_ACPI_CONTAINER_MODULE))
> > +#define ACPI_HOTPLUG_OST
> > +#endif
>
> Already covered in my general comments, but just in case:
>
> This is restricted to a few of the possible cases _OST is intended for.
> What happens when a kernel is configed with all of the above and PCI and
> PCIe hotplug configs. _OST will only be evaluated in these cases,
> shouldn't OS evaluate _OST in all cases once it tells firmware it
> supports _OST in the case of Insertion/Ejection?
PCIe native hotplug is a non-ACPI based hotplug, and is not supported by
_OST. ACPI PCI hotplug is being replaced by PCIe native hotplug, so it
is not supported as described in the change log. Please see the email
below on this.
http://marc.info/?l=linux-acpi&m=133669547631220&w=2
Thanks,
-Toshi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists