[PATCH RFC 0/2] mlx5/pci: Fix enablement of PCIe AtomicOp Requests

Gerd Bayer posted 2 patches 1 month, 1 week ago
drivers/infiniband/hw/mlx5/data_direct.c       | 6 +++---
drivers/net/ethernet/mellanox/mlx5/core/main.c | 6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)
[PATCH RFC 0/2] mlx5/pci: Fix enablement of PCIe AtomicOp Requests
Posted by Gerd Bayer 1 month, 1 week ago
As I promised in
https://lore.kernel.org/linux-pci/9c7c4217171fb56c505dc90b8c73b2ce079207a9.camel@linux.ibm.com/#t
here's an RFC patch proposing to correct the usage of
pci_enable_atomic_ops_to_root() in the Ethernet and InfiniBand drivers
for Mellanox/NVidia devices.

With this fix, AtomicOp Requests would only be enabled if a root-port
correctly announced its capability to support all AtomicOps including
the optional 128bit CAS - as the drivers intend to.

That said, on the only x86 system that I can test this on, it appears
that UEFI is already enabling AtomicOp Requests for the ConnectX-6 Dx
card in that system.

Overall, both the enablement of PCIe AtomicOps by core PCI code, and
their usage by (very few) device drivers, appear to be in need of some
attention?

Thanks,
Gerd

Signed-off-by: Gerd Bayer <gbayer@linux.ibm.com>
---
Gerd Bayer (2):
      net/mlx5: Request PCIe AtomicOps enabled for all 3 sizes
      ib/mlx5: Request PCIe AtomicOps enabled for all 3 sizes

 drivers/infiniband/hw/mlx5/data_direct.c       | 6 +++---
 drivers/net/ethernet/mellanox/mlx5/core/main.c | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)
---
base-commit: 6146a0f1dfae5d37442a9ddcba012add260bceb0
change-id: 20251024-mlxatomics-5729e8af60b9

Best regards,
-- 
Gerd Bayer <gbayer@linux.ibm.com>