https://bugzilla.kernel.org/show_bug.cgi?id=202055 --- Comment #30 from Alex Williamson (alex.williamson@xxxxxxxxxx) --- (In reply to LimeTech from comment #29) > Hi Alex, > > The "Prefer secondary bus reset over FLR" patch works for devices you added > in pci_dev_reset_methods[]. Will this patch work correctly for a SM2263 > controller as well? One such device (Crucial P1 CT500P1SSD8) has PCI ID > [c0a9:2263], just a matter of adding this ID? > > > Also, should we be using the "Test patch, NVMe shutdown + delay to avoid ACS > violation" patch instead? Hi Tom, The second patch is intended to be a replacement of the original, it at least enables the ADATA drive on a server where the first patch did not, even if that turned out to be not exactly the same issue as Dongli experiences. To add the SM2263 just add a new ID, ex: { 0xc0a9, 0x2263, sm2262_reset }, Add it to the code where the last chunk of the patch includes the known SM2262 variants, in the pci_dev_reset_methods array. Please report back the results. It's really unfortunate that there's such a fundamental bug in a whole family of controllers that's getting rebranded with different PCI IDs by so many vendors. Thanks, Alex -- You are receiving this mail because: You are watching the assignee of the bug.
