[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