[PATCH V2 RESEND] PCI/TPH: Skip Root Port completer check for RC_END devices

[PATCH V2 RESEND] PCI/TPH: Skip Root Port completer check for RC_END devices

[George Abraham P]

Root Complex Integrated Endpoint devices (PCI_EXP_TYPE_RC_END) are
directly integrated into the root complex and do not have an
associated Root Port in the traditional PCIe hierarchy. The current
TPH implementation incorrectly attempts to find and check a Root Port's
TPH completer capability for these devices.

Add a check to skip Root Port completer type verification for RC_END
devices, allowing them to use their full TPH requester capability
without being limited by a non-existent Root Port's completer support.

For RC_END devices, the root complex itself acts as the TPH completer,
and this relationship is handled differently than the standard
endpoint-to-Root-Port model.

Fixes: f69767a1ada3 ("PCI: Add TLP Processing Hints (TPH) support")
Signed-off-by: George Abraham P <george.abraham.p@intel.com>
---
v1->v2:
  - Added "Fixes:" tag to link the commit hash that introduced the code
---
 drivers/pci/tph.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/drivers/pci/tph.c b/drivers/pci/tph.c
index ca4f97be7538..e896b3958281 100644
--- a/drivers/pci/tph.c
+++ b/drivers/pci/tph.c
@@ -407,10 +407,13 @@ int pcie_enable_tph(struct pci_dev *pdev, int mode)
 	else
 		pdev->tph_req_type = PCI_TPH_REQ_TPH_ONLY;
 
-	rp_req_type = get_rp_completer_type(pdev);
+	/* Check if the device is behind a Root Port */
+	if (pci_pcie_type(pdev) != PCI_EXP_TYPE_RC_END) {
+		rp_req_type = get_rp_completer_type(pdev);
 
-	/* Final req_type is the smallest value of two */
-	pdev->tph_req_type = min(pdev->tph_req_type, rp_req_type);
+		/* Final req_type is the smallest value of two */
+		pdev->tph_req_type = min(pdev->tph_req_type, rp_req_type);
+	}
 
 	if (pdev->tph_req_type == PCI_TPH_REQ_DISABLE)
 		return -EINVAL;
-- 
2.40.1

Re: [PATCH V2 RESEND] PCI/TPH: Skip Root Port completer check for RC_END devices

[George Abraham P]

[+cc TPH authors that were missed in the previous mail]

On 09-Jan-26 10:59 AM, George Abraham P wrote:
> Root Complex Integrated Endpoint devices (PCI_EXP_TYPE_RC_END) are
> directly integrated into the root complex and do not have an
> associated Root Port in the traditional PCIe hierarchy. The current
> TPH implementation incorrectly attempts to find and check a Root Port's
> TPH completer capability for these devices.
> 
> Add a check to skip Root Port completer type verification for RC_END
> devices, allowing them to use their full TPH requester capability
> without being limited by a non-existent Root Port's completer support.
> 
> For RC_END devices, the root complex itself acts as the TPH completer,
> and this relationship is handled differently than the standard
> endpoint-to-Root-Port model.
> 
> Fixes: f69767a1ada3 ("PCI: Add TLP Processing Hints (TPH) support")
> Signed-off-by: George Abraham P <george.abraham.p@intel.com>
> ---
> v1->v2:
>   - Added "Fixes:" tag to link the commit hash that introduced the code
> ---
>  drivers/pci/tph.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/pci/tph.c b/drivers/pci/tph.c
> index ca4f97be7538..e896b3958281 100644
> --- a/drivers/pci/tph.c
> +++ b/drivers/pci/tph.c
> @@ -407,10 +407,13 @@ int pcie_enable_tph(struct pci_dev *pdev, int mode)
>  	else
>  		pdev->tph_req_type = PCI_TPH_REQ_TPH_ONLY;
>  
> -	rp_req_type = get_rp_completer_type(pdev);
> +	/* Check if the device is behind a Root Port */
> +	if (pci_pcie_type(pdev) != PCI_EXP_TYPE_RC_END) {
> +		rp_req_type = get_rp_completer_type(pdev);
>  
> -	/* Final req_type is the smallest value of two */
> -	pdev->tph_req_type = min(pdev->tph_req_type, rp_req_type);
> +		/* Final req_type is the smallest value of two */
> +		pdev->tph_req_type = min(pdev->tph_req_type, rp_req_type);
> +	}
>  
>  	if (pdev->tph_req_type == PCI_TPH_REQ_DISABLE)
>  		return -EINVAL;

Re: [PATCH V2 RESEND] PCI/TPH: Skip Root Port completer check for RC_END devices

[Leon Romanovsky]

On Wed, Feb 18, 2026 at 10:24:28AM +0530, George Abraham P wrote:
> [+cc TPH authors that were missed in the previous mail]
> 
> On 09-Jan-26 10:59 AM, George Abraham P wrote:
> > Root Complex Integrated Endpoint devices (PCI_EXP_TYPE_RC_END) are
> > directly integrated into the root complex and do not have an
> > associated Root Port in the traditional PCIe hierarchy. The current
> > TPH implementation incorrectly attempts to find and check a Root Port's
> > TPH completer capability for these devices.
> > 
> > Add a check to skip Root Port completer type verification for RC_END
> > devices, allowing them to use their full TPH requester capability
> > without being limited by a non-existent Root Port's completer support.
> > 
> > For RC_END devices, the root complex itself acts as the TPH completer,
> > and this relationship is handled differently than the standard
> > endpoint-to-Root-Port model.
> > 
> > Fixes: f69767a1ada3 ("PCI: Add TLP Processing Hints (TPH) support")
> > Signed-off-by: George Abraham P <george.abraham.p@intel.com>
> > ---
> > v1->v2:
> >   - Added "Fixes:" tag to link the commit hash that introduced the code

I do not have a strong preference; I am fine with adding a Fixes tag or
leaving it out. Up to commit 2961f841b025, pcie_enable_tph() is called
in two drivers: bnxt and mlx5. Which of these devices is the RC_END
device?

Thanks

Re: [PATCH V2 RESEND] PCI/TPH: Skip Root Port completer check for RC_END devices

[George Abraham P]

Hi Leon,

On 18-Feb-26 2:12 PM, Leon Romanovsky wrote:
> On Wed, Feb 18, 2026 at 10:24:28AM +0530, George Abraham P wrote:
>> [+cc TPH authors that were missed in the previous mail]
>>
>> On 09-Jan-26 10:59 AM, George Abraham P wrote:
>>> Root Complex Integrated Endpoint devices (PCI_EXP_TYPE_RC_END) are
>>> directly integrated into the root complex and do not have an
>>> associated Root Port in the traditional PCIe hierarchy. The current
>>> TPH implementation incorrectly attempts to find and check a Root Port's
>>> TPH completer capability for these devices.
>>>
>>> Add a check to skip Root Port completer type verification for RC_END
>>> devices, allowing them to use their full TPH requester capability
>>> without being limited by a non-existent Root Port's completer support.
>>>
>>> For RC_END devices, the root complex itself acts as the TPH completer,
>>> and this relationship is handled differently than the standard
>>> endpoint-to-Root-Port model.
>>>
>>> Fixes: f69767a1ada3 ("PCI: Add TLP Processing Hints (TPH) support")
>>> Signed-off-by: George Abraham P <george.abraham.p@intel.com>
>>> ---
>>> v1->v2:
>>>   - Added "Fixes:" tag to link the commit hash that introduced the code
> 
> I do not have a strong preference; I am fine with adding a Fixes tag or
> leaving it out. Up to commit 2961f841b025, pcie_enable_tph() is called
> in two drivers: bnxt and mlx5. Which of these devices is the RC_END
> device?
> 
> Thanks

qat_6xxx supports TPH and is a RC_END device. The patches to enable TPH on qat_6xxx is yet to be reviewed. However, it requires this fix for the same to work.

Thanks,
George

Re: [PATCH V2 RESEND] PCI/TPH: Skip Root Port completer check for RC_END devices

[Wei Huang]

On Wed, Feb 18, 2026 at 09:00:59PM +0530, George Abraham P wrote:
> Hi Leon,
> 
> On 18-Feb-26 2:12 PM, Leon Romanovsky wrote:
> > On Wed, Feb 18, 2026 at 10:24:28AM +0530, George Abraham P wrote:
> >> [+cc TPH authors that were missed in the previous mail]
> >>
> >> On 09-Jan-26 10:59 AM, George Abraham P wrote:
> >>> Root Complex Integrated Endpoint devices (PCI_EXP_TYPE_RC_END) are
> >>> directly integrated into the root complex and do not have an
> >>> associated Root Port in the traditional PCIe hierarchy. The current
> >>> TPH implementation incorrectly attempts to find and check a Root Port's
> >>> TPH completer capability for these devices.
> >>>
> >>> Add a check to skip Root Port completer type verification for RC_END
> >>> devices, allowing them to use their full TPH requester capability
> >>> without being limited by a non-existent Root Port's completer support.
> >>>
> >>> For RC_END devices, the root complex itself acts as the TPH completer,
> >>> and this relationship is handled differently than the standard
> >>> endpoint-to-Root-Port model.
> >>>
> >>> Fixes: f69767a1ada3 ("PCI: Add TLP Processing Hints (TPH) support")
> >>> Signed-off-by: George Abraham P <george.abraham.p@intel.com>
> >>> ---
> >>> v1->v2:
> >>>   - Added "Fixes:" tag to link the commit hash that introduced the code
> > 
> > I do not have a strong preference; I am fine with adding a Fixes tag or
> > leaving it out. Up to commit 2961f841b025, pcie_enable_tph() is called
> > in two drivers: bnxt and mlx5. Which of these devices is the RC_END
> > device?
> > 
> > Thanks
> 
> qat_6xxx supports TPH and is a RC_END device. The patches to enable TPH on qat_6xxx is yet to be reviewed. However, it requires this fix for the same to work.
> 

The patche looks fine to me - it shouldn't affect regular PCI Express Endpoint (type 0000b) devices.

> Thanks,
> George

Re: [PATCH V2 RESEND] PCI/TPH: Skip Root Port completer check for RC_END devices

[Bjorn Helgaas]

On Fri, Jan 09, 2026 at 10:59:23AM +0530, George Abraham P wrote:
> Root Complex Integrated Endpoint devices (PCI_EXP_TYPE_RC_END) are
> directly integrated into the root complex and do not have an
> associated Root Port in the traditional PCIe hierarchy. The current
> TPH implementation incorrectly attempts to find and check a Root Port's
> TPH completer capability for these devices.
> 
> Add a check to skip Root Port completer type verification for RC_END
> devices, allowing them to use their full TPH requester capability
> without being limited by a non-existent Root Port's completer support.
> 
> For RC_END devices, the root complex itself acts as the TPH completer,
> and this relationship is handled differently than the standard
> endpoint-to-Root-Port model.
> 
> Fixes: f69767a1ada3 ("PCI: Add TLP Processing Hints (TPH) support")
> Signed-off-by: George Abraham P <george.abraham.p@intel.com>

Applied to pci/enumeration with the following commit log for v7.1,
thanks!  This will be rebased after v7.0-rc1.

    PCI/TPH: Allow TPH enable for RCiEPs
    
    Previously, pcie_enable_tph() only enabled TLP Processing Hints (TPH) if
    both the Endpoint and its Root Port advertised TPH support.
    
    Root Complex Integrated Endpoints (RCiEPs) are directly integrated into a
    Root Complex and do not have an associated Root Port, so pcie_enable_tph()
    never enabled TPH for RCiEPs.
    
    PCIe r7.0 doesn't seem to include a way to learn whether a Root Complex
    supports TPH, but sec 2.2.7.1.1 says Functions that lack TPH support should
    ignore TPH, and maybe the same is true for Root Complexes:
    
      A Function that does not support the TPH Completer or Routing capability
      and receives a transaction with the TH bit [which indicates the presence
      of TPH in the TLP header] Set is required to ignore the TH bit and handle
      the Request in the same way as Requests of the same transaction type
      without the TH bit Set.
    
    Allow drivers to enable TPH for any RCiEP with a TPH Requester Capability.
    
> ---
> v1->v2:
>   - Added "Fixes:" tag to link the commit hash that introduced the code
> ---
>  drivers/pci/tph.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/pci/tph.c b/drivers/pci/tph.c
> index ca4f97be7538..e896b3958281 100644
> --- a/drivers/pci/tph.c
> +++ b/drivers/pci/tph.c
> @@ -407,10 +407,13 @@ int pcie_enable_tph(struct pci_dev *pdev, int mode)
>  	else
>  		pdev->tph_req_type = PCI_TPH_REQ_TPH_ONLY;
>  
> -	rp_req_type = get_rp_completer_type(pdev);
> +	/* Check if the device is behind a Root Port */
> +	if (pci_pcie_type(pdev) != PCI_EXP_TYPE_RC_END) {
> +		rp_req_type = get_rp_completer_type(pdev);
>  
> -	/* Final req_type is the smallest value of two */
> -	pdev->tph_req_type = min(pdev->tph_req_type, rp_req_type);
> +		/* Final req_type is the smallest value of two */
> +		pdev->tph_req_type = min(pdev->tph_req_type, rp_req_type);
> +	}
>  
>  	if (pdev->tph_req_type == PCI_TPH_REQ_DISABLE)
>  		return -EINVAL;
> -- 
> 2.40.1
>