Hello Zhuo, > Use pcie_aer_is_native() in place of "host->native_aer || > pcie_ports_native" to judge whether OS has native control of AER > in aer_root_reset() and pcie_do_recovery(). > > Replace "dev->aer_cap && (pcie_ports_native || host->native_aer)" in > get_port_device_capability() with pcie_aer_is_native(), which has no > functional changes. > > Signed-off-by: Zhuo Chen <chenzhu...@bytedance.com> > --- > Changelog: > v4: > - Use pcie_aer_is_native() instead in aer_root_reset(). > v3: > - Simplify why we use pcie_aer_is_native(). > - Revert modification of pci_aer_clear_nonfatal_status() and comments. > v2: > - Add details and note in commit log. [...]
A similar change has been proposed in the past, and back then, Bjorn had a few questions and concerns. Have a look at the entire discussion: https://lore.kernel.org/linux-pci/1612490648-44817-1-git-send-email-tanxiao...@huawei.com/ If you think that the proposed changes are fine and can address some of Bjorn's concerns, then it would be great. Thank you in advance! Krzysztof
