From: Gavin Shan <gwshan@linux.vnet.ibm.com>
To: Stewart Smith <stewart@linux.vnet.ibm.com>
Cc: Gavin Shan <gwshan@linux.vnet.ibm.com>,
linuxppc-dev@ozlabs.org, bhelgaas@google.com,
linux-pci@vger.kernel.org
Subject: Re: [PATCH RESEND v2 0/7] powerpc/powernv: Unified PCI slot reset and hotplug
Date: Wed, 18 Feb 2015 09:26:15 +1100 [thread overview]
Message-ID: <20150217222615.GA6454@shangw> (raw)
In-Reply-To: <m3vbj0gqmk.fsf@oc8180480414.ibm.com>
On Wed, Feb 18, 2015 at 08:44:19AM +1100, Stewart Smith wrote:
>Gavin Shan <gwshan@linux.vnet.ibm.com> writes:
>
>> The patchset was built based on patchset "powerpc/powernv: Simplify EEH
>> implementation", which can be found from:
>>
>> https://patchwork.ozlabs.org/patch/439956/
>>
>> The patchset corresponds to skiboot changes, which manages PCI slots
>> in a unified way: OPAL APIs used to do slot reset, power management,
>> presence status retrival. The patchset shouldn't be merged before
>> the OPAL firmware counterpart is merged:
>>
>> https://patchwork.ozlabs.org/patch/440463/
>
>But this patchset will work with old OPAL, right? We're not breaking
>compatibility with newer kernels (with this patchset) and older
>firmware?
Yes. With old OPAL, the reset stuff in Linux works as before and we
shouldn't see hotpluggable slots from /sys/bus/pci/slots.
The compatibility issue was pointed by Ben. It's something fixed by
this revision (v2).
Thanks,
Gavin
WARNING: multiple messages have this Message-ID (diff)
From: Gavin Shan <gwshan@linux.vnet.ibm.com>
To: Stewart Smith <stewart@linux.vnet.ibm.com>
Cc: bhelgaas@google.com, linuxppc-dev@ozlabs.org,
Gavin Shan <gwshan@linux.vnet.ibm.com>,
linux-pci@vger.kernel.org
Subject: Re: [PATCH RESEND v2 0/7] powerpc/powernv: Unified PCI slot reset and hotplug
Date: Wed, 18 Feb 2015 09:26:15 +1100 [thread overview]
Message-ID: <20150217222615.GA6454@shangw> (raw)
In-Reply-To: <m3vbj0gqmk.fsf@oc8180480414.ibm.com>
On Wed, Feb 18, 2015 at 08:44:19AM +1100, Stewart Smith wrote:
>Gavin Shan <gwshan@linux.vnet.ibm.com> writes:
>
>> The patchset was built based on patchset "powerpc/powernv: Simplify EEH
>> implementation", which can be found from:
>>
>> https://patchwork.ozlabs.org/patch/439956/
>>
>> The patchset corresponds to skiboot changes, which manages PCI slots
>> in a unified way: OPAL APIs used to do slot reset, power management,
>> presence status retrival. The patchset shouldn't be merged before
>> the OPAL firmware counterpart is merged:
>>
>> https://patchwork.ozlabs.org/patch/440463/
>
>But this patchset will work with old OPAL, right? We're not breaking
>compatibility with newer kernels (with this patchset) and older
>firmware?
Yes. With old OPAL, the reset stuff in Linux works as before and we
shouldn't see hotpluggable slots from /sys/bus/pci/slots.
The compatibility issue was pointed by Ben. It's something fixed by
this revision (v2).
Thanks,
Gavin
next prev parent reply other threads:[~2015-02-17 22:27 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-17 7:13 [PATCH RESEND v2 0/7] powerpc/powernv: Unified PCI slot reset and hotplug Gavin Shan
2015-02-17 7:13 ` Gavin Shan
2015-02-17 7:13 ` [PATCH RESEND v2 1/7] powerpc/powernv: Use PCI slot reset infrastructure Gavin Shan
2015-02-17 7:13 ` Gavin Shan
2015-02-17 7:13 ` [PATCH RESEND v2 2/7] powerpc/powernv: Issue fundamental reset if required Gavin Shan
2015-02-17 7:13 ` Gavin Shan
2015-02-17 7:13 ` [PATCH RESEND v2 3/7] powerpc/pci: Move pcibios_find_pci_bus() around Gavin Shan
2015-02-17 7:13 ` Gavin Shan
2015-02-17 7:13 ` [PATCH RESEND v2 4/7] powerpc/pci: Don't scan empty slot Gavin Shan
2015-02-17 7:13 ` Gavin Shan
2015-02-17 7:13 ` [PATCH RESEND v2 5/7] powerpc/powernv: Introduce pnv_pci_poll() Gavin Shan
2015-02-17 7:13 ` Gavin Shan
2015-02-17 7:13 ` [PATCH RESEND v2 6/7] powerpc/powernv: Functions to retrieve PCI slot status Gavin Shan
2015-02-17 7:13 ` Gavin Shan
2015-02-17 7:13 ` [PATCH RESEND v2 7/7] PCI/hotplug: PowerPC PowerNV PCI hotplug driver Gavin Shan
2015-02-17 7:13 ` Gavin Shan
2015-02-17 22:09 ` Bjorn Helgaas
2015-02-17 22:09 ` Bjorn Helgaas
2015-02-18 0:16 ` Gavin Shan
2015-02-18 0:16 ` Gavin Shan
2015-02-18 0:30 ` Benjamin Herrenschmidt
2015-02-18 14:30 ` Bjorn Helgaas
2015-02-18 14:30 ` Bjorn Helgaas
2015-02-18 14:30 ` Bjorn Helgaas
2015-02-18 21:03 ` Benjamin Herrenschmidt
2015-02-18 21:03 ` Benjamin Herrenschmidt
2015-02-18 21:03 ` Benjamin Herrenschmidt
2015-02-17 21:44 ` [PATCH RESEND v2 0/7] powerpc/powernv: Unified PCI slot reset and hotplug Stewart Smith
2015-02-17 22:26 ` Gavin Shan [this message]
2015-02-17 22:26 ` Gavin Shan
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=20150217222615.GA6454@shangw \
--to=gwshan@linux.vnet.ibm.com \
--cc=bhelgaas@google.com \
--cc=linux-pci@vger.kernel.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=stewart@linux.vnet.ibm.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.