Hi, Bjorn Helgaas <helgaas@xxxxxxxxxx> writes: > On Fri, Aug 03, 2018 at 01:25:58PM -0400, Sinan Kaya wrote: >> On 8/3/2018 2:26 AM, Felipe Balbi wrote: >> > Sinan Kaya <okaya@xxxxxxxxxx> writes: >> > >> > > On 8/2/2018 7:36 AM, Felipe Balbi wrote: >> > > > PCIe GEN4 defines a new bit on Status Register which tells us that, if >> > > > Set, a function is immediately ready after a Reset. This means that >> > > > all delays after a Conventional or Function Reset can be skipped. >> > > >> > > Can you give a reference to the section of the specification? or >> > > a pointer to the ECN? >> > >> > Section 7.5.1.1.4 of PCIe GEN4 spec. Table 7-4: >> > >> > Immediate Readiness – This optional bit, when Set, indicates the >> > Function is guaranteed to be ready to successfully complete valid >> > configuration accesses at any time following any reset that the host is >> > capable of issuing Configuration Requests to this Function. >> > >> > When this bit is Set, for accesses to this Function, software is exempt >> > from all requirements to delay configuration accesses following any type >> > of reset, including but not limited to the timing requirements defined >> > in Section 6.6. How this guarantee is established is beyond the scope >> > of this document. >> > >> > It is permitted that system software/firmware provide mechanisms that >> > supersede the indication provided by this bit, however such >> > software/firmware mechanisms are outside the scope of this >> > specification. >> > >> >> Thanks for the spec reference. > > Yes. Please include the reference in the changelog of v2. will do >> I think the patch is touching the wrong places. pci_dev_wait() is there >> to wait for CRS response to finish following reset. >> >> Typical sequences are: >> 1. Do some kind of reset in another routine >> 2. Wait reset specific wait time (1sec for secondary bus reset as an >> example and 100ms for d3-d0 transition) >> 3. call pci_dev_wait() after reset to see if device can accept config >> transactions. >> >> Since this applies to all resets, I think you also need to get rid of >> waits following different reset types in step #2 and return immediately. >> I suggest you review callers of pci_dev_wait() and tap in there. > > I agree; I think we should be able to skip the delays in pcie_flr(), > pci_af_flr(), etc. but that's why I put the code in pci_dev_wait(). Both pcie_flr() and pci_af_flr() call pci_dev_wait(); or are you saying that the msleep(100) call in pci_af_flr() can also be removed if (dev->imm_ready)? -- balbi
Attachment:
signature.asc
Description: PGP signature