Enhance PCIe Test Rule PCI_IN_19 to consider the DPC #192
Conversation
Include segment number as well while checking for the root port's bdf in the API val_pcie_parent_is_rootport(). Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Add support to disable & re-enable DPC during the execution of rule PCI_IN_19. This avoids unwanted DPC trigger and hence avoids link down event. Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
|
@chetan-rathore Can you please check this? |
|
Hi @ashishsingha, I am on holidays and will return on 4th Jan, adding @gowthamsiddarthd who will help in reviewing the changes. |
|
Hi @tvidyasagar / @ashishsingha Thanks for catching this and for suggesting the change. Based on your suggestion, it looks like this behavior could be applicable to other tests as well, especially those that intentionally generate Unsupported Requests (URs). Disabling DPC centrally would help ensure that UR generation does not inadvertently bring the link down, allowing subsequent tests to continue accessing downstream devices. Given that, my suggestion would be to disable DPC during BDF table creation itself (around Let me know if you’re okay making this change, or if you’d prefer that I take care of it from my side. Regards, |
4 participants
This patch series enhances the PCIe Test Rule PCI_IN_19 to disable the DPC (if enabled already). Disabling the DPC makes sure that the link will not go down because of the Unsupported Request generation and ensures that the tests that get executed after PCI_IN_19 continue to have access to the downstream devices. If the DPC is enabled, UR generation can potentially trigger DPC and the link goes down and any further accesses to the downstream hierarchy will not take place.