[PATCH 06/14] powerpc/powernv: Add support for the cxl kernel api on the real phb
Ian Munsie
imunsie at au1.ibm.com
Thu Jul 7 16:28:32 AEST 2016
Excerpts from Frederic Barrat's message of 2016-07-06 19:38:18 +0200:
>
> > + /* No special handling for cxl function: */
> > + if (PCI_FUNC(dev->devfn) == 0)
> > + return true;
>
> I believe that is the first time we're getting a hint of the black magic
> which is going to occur when the card is switched to cxl mode and the
> appearance of a new pci function. I think a general comment explaining
> it is needed somewhere. In this patch or a later one. Also "peer model"
> is used several times in the commit messages, though it's not clear to
> the novice what it really means.
>
> At this point of the review, I was a bit overwhelmed by all the new
> APIs, wondering how everything would end up working together. By the
> last patch, it's understandable, but a few extra comments would help.
> For the vPHB model, pretty much all the relevant code is in one file,
> which helps grabbing the full picture. But here it's spread between the
> phb platform code and the cxl driver.
>
> Fred
Ok, will see what I can to to clarify this.
-Ian
More information about the Linuxppc-dev
mailing list