Re: [PATCH v2 0/3] powerpc/pci: PCI slot unified reset

[Gavin Shan <gwshan@linux.vnet.ibm.com>]

On Thu, Dec 04, 2014 at 04:50:50PM +1100, Gavin Shan wrote:

Please ignore this one. I'll rebase and send out a new revision
shortly.

Thanks,
Gavin

>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.
>
>The kernel changes have been split into 2 parts: (A) Use the unified
>PCI slot reset OPAL API - this patchset; (B) powernv-php driver to
>support PCI hotplug for PowerNV platform, which will be sent separately.
>
>This patchset affects EEH, PCI reset logics, which is used mainly by
>VFIO. A couple of cases have been tested out on P7/P8 boxes and looks
>good: EEH on PowerNV, EEH support for guest, PCI passthrou.
>
>Changelog
>=========
>v1 -> v2
>        * Keep opal_pci_reinit(). In case the slot is resetted by kernel,
>          instead of skiboot, this API should be called to restore states
>          for those affected devices.
>        * Reworked slot ID scheme so that old/new kernel can work with
>          skiboot with or without unified PCI slot management support.
>        * Code cleanup here and there.
>
>Gavin Shan (3):
>  powerpc/powernv: Use PCI slot reset infrastructure
>  powerpc/powernv: Refactor ioda_eeh_reset()
>  powerpc/powernv: Issue fundamental reset if required
>
> arch/powerpc/include/asm/eeh.h            |   1 +
> arch/powerpc/include/asm/opal.h           |   9 +-
> arch/powerpc/platforms/powernv/eeh-ioda.c | 240 ++++++++++++++++--------------
> 3 files changed, 132 insertions(+), 118 deletions(-)
>
>Thanks,
>Gavin
>