Re: [PATCH v3 0/2] cxl: Region bandwidth calculation for targets with shared upstream link

[Dave Jiang <dave.jiang@intel.com>]

On 6/6/24 4:27 PM, Dave Jiang wrote:
> 
> 
> On 6/5/24 11:38 AM, Alison Schofield wrote:
>> On Wed, May 29, 2024 at 02:38:56PM -0700, Dave Jiang wrote:
>>> This series provides recalculation of the CXL region bandwidth when the targets have
>>> shared upstream link by walking the toplogy from bottom up and clamp the bandwdith
>>> as the code trasverses up the tree. An example topology:
>>>
>>>  An example topology from Jonathan:
>>>
>>>                  CFMWS 0
>>>                    |
>>>           _________|_________
>>>          |                   |
>>>    GP0/HB0/ACPI0017-0  GP1/HB1/ACPI0017-1
>>>      |          |        |           |
>>>     RP0        RP1      RP2         RP3
>>>      |          |        |           |
>>>    SW 0       SW 1     SW 2        SW 3
>>>    |   |      |   |    |   |       |   |
>>>   EP0 EP1    EP2 EP3  EP4  EP5    EP6 EP7
>>
>> Are the ACPI0017 labels a typo?
>> Expected host bridges to be labelled as ACPI0016's.
>>
>> That's all, just did a drive-by on the art today.
> 
> It would be something like below in this case:
>         GP0/ACPI0017-0
>         HB0/ACPI0016-0
>            |     |
>           RP0   RP1

Actually it should be:
	ACPI0017-0
    HB0/GP0/ACPI0016-0
	|	|
       RP0     RP1

Generic Port is identified by device handle that is off of ACPI0016 HID. 

> 
> or
> 
>         GP0/ACPI0017-0
> HB0/ACPI0016-0  HB1/ACPI0016-1
>       |               |
>      RP0             RP1
> 
> 
> 
> 
>>
>> -- Alison
>