[PATCH V14 00/21] Enable SRIOV on Power8
Bjorn Helgaas
bhelgaas at google.com
Sat Mar 21 01:34:04 AEDT 2015
On Fri, Mar 20, 2015 at 12:43 AM, Gavin Shan <gwshan at linux.vnet.ibm.com> wrote:
> On Fri, Mar 20, 2015 at 11:06:16AM +0800, Wei Yang wrote:
>
> [snip]
>
>>---
>>v14:
>> * call ppc_md.pcibios_fixup_sriov() in pcibios_add_device
>> * add more explanation in change log
>> * Following patches have been reordered to the beginning.
>> "EEH refactor to use pci_dn":
>> 8ec20d6 powerpc/powernv: Use pci_dn, not device_node, in PCI config accessor
>> a3460fc powerpc/pci: Refactor pci_dn
>> These two patches will be modified to merge with other patches which are
>> under discussion/review in ppc mail list. Some changes may also be made in
>> other patches, which I didn't include them in this series, so that the
>> auto build robot could work on this.
>
> The comment here isn't precise enough and not the things I suggested before.
> Those 2 patches have been split into 3 patches (A/B/C). Some other EEH
> cleanup/refactor patches depends on A/B and those patches would be merged
> before your SRIOV patches to PowerPC tree. C, which I already sent to you,
> need to be integrated to your patchset right after the following one:
>
> powerpc/pci: Don't unset PCI resources for VFs
>
> I guess you can move the patches around after checking if Bjorn has further
> concerns/comments.
I think we need to move forward on this soon so it has at least a
little time in -next. And it sounds like I cannot merge this v14
as-is because it doesn't include these additional EEH cleanup/refactor
patches? (If they are only cleanup and refactoring, I don't
understand why they would be required, but I haven't seen them, so I'm
taking your word for it.)
So I guess I'm waiting for v15. Don't wait for my comments on v14.
Ideally I'd also like an ack from a powerpc maintainer. This has so
much powerpc impact that I'm not comfortable stuffing this in without
their OK.
Bjorn
More information about the Linuxppc-dev
mailing list