Re: [PATCH v10 08/18] PCI/P2PDMA: Route Relaxed Ordering Completions directly

[sashiko-bot@kernel.org]

> From: Leon Romanovsky <leonro@nvidia.com>
> 
> ACS P2P Completion Redirect leaves Completions carrying the Relaxed
> Ordering attribute alone. PCIe r7.0 sec 6.12.1.1 redirects only those "that
> do not have the Relaxed Ordering Attribute bit set", and sec 7.7.12.5
> describes the enable bit as "applicable only to Completions whose Relaxed
> Ordering Attribute is clear". P2PDMA reports one answer for every kind of
> TLP, so a client that sets Relaxed Ordering on its Non-Posted Requests,
> and therefore receives such Completions (sec 2.2.9.1), is sent through
> the host bridge for a redirect that never happens to it.
> 
> Route Completions directly for the classes that set
> PCI_P2PDMA_TLP_RELAXED_CPL.
> 
> Reviewed-by: Logan Gunthorpe <logang@deltatee.com>
> Tested-by: Tushar Dave <tdave@nvidia.com>
> Signed-off-by: Leon Romanovsky <leonro@nvidia.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261012-fix-p2p-acs-v4-0-v10-0-6ffe7b7a4517@nvidia.com?part=8