Re: [PATCH 12/15] powerpc/powernv/sriov: De-indent setup and teardown

On 15/07/2020 14:21, Oliver O'Halloran wrote:

On Wed, Jul 15, 2020 at 2:00 PM Alexey Kardashevskiy [off-list ref] wrote:

quoted

or we could just skip setting

ppc_md.pcibios_sriov_enable = pnv_pcibios_sriov_enable;

for uninteresting platforms in pnv_pci_init_ioda_phb().

I don't think so. ppc_md is per-platform, not per-PHB andw e still
have to deal with a mixture of IODA/NVLink/OpenCAPI PHBs on a single
system.

NVLink/OpenCAPI won't have SRIOV devices. Other types won't appear on
the same platform simultaneously. It is not too clean, yes.

We could make it a callback in pnv_phb, but  it seemed like
more indirection than it's worth.

I genuinely dislike how we use ppc_md so removing things from it is
definitely a good thing.


-- 
Alexey