public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Matthew Garrett <mjg59@srcf.ucam.org>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>,
	linux-pci@vger.kernel.org, Len Brown <lenb@kernel.org>,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
	linux-pm@lists.linux-foundation.org,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Jesse Barnes <jbarnes@virtuousgeek.org>
Subject: Re: [RFC][PATCH] PCI / PCIe: Ask BIOS for control of all native services simultaneously
Date: Tue, 27 Jul 2010 19:56:28 +0100	[thread overview]
Message-ID: <20100727185628.GA17903@srcf.ucam.org> (raw)
In-Reply-To: <201007272042.39269.rjw@sisk.pl>

On Tue, Jul 27, 2010 at 08:42:39PM +0200, Rafael J. Wysocki wrote:
> On Tuesday, July 27, 2010, Matthew Garrett wrote:
> > I've done some more testing of this and found that my intial belief 
> > (supported by Microsoft's documentation...) that all PCIe support had to 
> > be handed over for any to be used is incorrect. It turns out that the 
> > firmware must support native hotplug, native power management and PCI 
> > express capability structure control - ie, SHPC and AER aren't required.
> 
> But the ACPI spec says quite explicitly that PCIe capability control is
> necessary for AER as well.

I may not have been clear. If the firmware doesn't report AER, Windows 
will use PME and hotplug. If the firmware doesn't support hotplug, 
Windows will *not* use PME or AER, and ditto if it doesn't support PME.

> > (2) If any of bits 0, 2 and 4 are unsupported, disable all PCIe support 
> > via _OSC
> 
> I guess you mean "don't request control of that services at all"?

I mean pass 0 as the third dword in our _OSC call.

> > (3) Ask for the set of supported bits & 0x1d
> 
> Really, if we try to treat native PME, native hot-plug and AER separately
> (which is our current approach), we fall into a Catch 22 situation where
> each of them needs PCIe capability control and once we've received the
> control of that, we have no choice but to use the other native sevices as well.

It seems that it's valid to have hotplug and PME without AER. The 
behaviour of Windows for each bit is:

0 Hotplug             Required
1 SHPHC               Will never request
2 PME                 Required
3 AER                 Optional
4 capability control  Required

So firmware can refuse to support SHPHC and AER and still get PCIe 
support, but if any of the required bits aren't available Windows won't 
use *any* of the _OSC-provided functions.

-- 
Matthew Garrett | mjg59@srcf.ucam.org

  reply	other threads:[~2010-07-27 18:56 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-24 23:05 [RFC][PATCH] PCI / PCIe: Ask BIOS for control of all native services simultaneously Rafael J. Wysocki
2010-07-25 12:23 ` [RFC][PATCH] PCI / PCIe: Ask BIOS for control of all native services at once Rafael J. Wysocki
2010-07-27  0:43 ` [RFC][PATCH] PCI / PCIe: Ask BIOS for control of all native services simultaneously Kenji Kaneshige
2010-07-27 17:18   ` Matthew Garrett
2010-07-27 18:42     ` Rafael J. Wysocki
2010-07-27 18:56       ` Matthew Garrett [this message]
2010-07-27 22:55         ` [RFC][PATCH] PCI / PCIe: Ask BIOS for control of all native services at once (v2) Rafael J. Wysocki
2010-07-28 10:59           ` [RFC][PATCH] PCI / PCIe: Ask BIOS for control of all native services at once (v3) Rafael J. Wysocki
2010-07-27 18:31   ` [RFC][PATCH] PCI / PCIe: Ask BIOS for control of all native services simultaneously Rafael J. Wysocki
2010-07-28  3:39 ` Hidetoshi Seto
2010-07-28 10:49   ` Rafael J. Wysocki
2010-07-28 11:55   ` Matthew Garrett

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=20100727185628.GA17903@srcf.ucam.org \
    --to=mjg59@srcf.ucam.org \
    --cc=jbarnes@virtuousgeek.org \
    --cc=kaneshige.kenji@jp.fujitsu.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=rjw@sisk.pl \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox