On Fri, Nov 15, 2024 at 10:46:04AM +0800, Mengyuan Lou wrote: > Wangxun FF5xxx NICs are similar to SFxxx, RP1000 and RP2000 NICs. > They may be multi-function devices, but they do not advertise an ACS > capability. > > But the hardware does isolate FF5xxx functions as though it had an > ACS capability and PCI_ACS_RR and PCI_ACS_CR were set in the ACS > Control register, i.e., all peer-to-peer traffic is directed > upstream instead of being routed internally. > > Add ACS quirk for FF5xxx NICs in pci_quirk_wangxun_nic_acs() so the > functions can be in independent IOMMU groups. > > Signed-off-by: Mengyuan Lou <mengyuanlou@xxxxxxxxxxxxx> Applied to pci/virtualization for v6.13, thank you! > --- > > v2: > - Update commit and comment logs. > v1: > https://lore.kernel.org/linux-pci/3D914272-CFAE-4B37-A07B-36CA77210110@xxxxxxxxxxxxx/T/#t > > drivers/pci/quirks.c | 15 +++++++++------ > 1 file changed, 9 insertions(+), 6 deletions(-) > > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c > index dccb60c1d9cc..8103bc24a54e 100644 > --- a/drivers/pci/quirks.c > +++ b/drivers/pci/quirks.c > @@ -4996,18 +4996,21 @@ static int pci_quirk_brcm_acs(struct pci_dev *dev, u16 acs_flags) > } > > /* > - * Wangxun 10G/1G NICs have no ACS capability, and on multi-function > - * devices, peer-to-peer transactions are not be used between the functions. > - * So add an ACS quirk for below devices to isolate functions. > + * Wangxun 40G/25G/10G/1G NICs have no ACS capability, but on > + * multi-function devices, the hardware isolates the functions by > + * directing all peer-to-peer traffic upstream as though PCI_ACS_RR and > + * PCI_ACS_CR were set. > * SFxxx 1G NICs(em). > * RP1000/RP2000 10G NICs(sp). > + * FF5xxx 40G/25G/10G NICs(aml). > */ > static int pci_quirk_wangxun_nic_acs(struct pci_dev *dev, u16 acs_flags) > { > switch (dev->device) { > - case 0x0100 ... 0x010F: > - case 0x1001: > - case 0x2001: > + case 0x0100 ... 0x010F: /* EM */ > + case 0x1001: case 0x2001: /* SP */ > + case 0x5010: case 0x5025: case 0x5040: /* AML */ > + case 0x5110: case 0x5125: case 0x5140: /* AML */ > return pci_acs_ctrl_enabled(acs_flags, > PCI_ACS_SV | PCI_ACS_RR | PCI_ACS_CR | PCI_ACS_UF); > } > -- > 2.43.2 >
