On Tue, Nov 11, 2025 at 09:44:45AM -0600, Bjorn Helgaas wrote: > On Tue, Nov 11, 2025 at 10:33:43AM +0100, Lukas Wunner wrote: > > I note that a number of drivers call pci_disable_link_state() or > > pci_disable_link_state_locked() to disable ASPM on probe. > > Can we convert (all of) these to quirks which use the new helper > > introduced here? [...] > I definitely agree. I forgot to follow up on all of those cases. > There aren't that many of them, but it looks like probably too many to > address for v6.18, and I *think* it's safe to wait and deal with them > for v6.19.
Yes I agree this isn't necessary for v6.18. It may even be too late for v6.19 given the amount of time to come up with patches, get them reviewed and allow sufficient time to soak in linux-next. And this would be a cleanup, so not really urgent. I note that a lot of material has queued up in patchwork and only few new features have been applied to pci.git this cycle, which I guess has been caused by the unusual number of regressions introduced during the merge window. Given that, I'd down-prioritize conversion of pci_disable_link_state() calls. One feature I'd be keen to get in for v6.19 (if time permits) is this one: https://lore.kernel.org/all/[email protected]/ I'm worried that the "pci_save_state() after pci_restore_state()" anti-pattern gets cargo-culted to more drivers if it's not removed soon. I can split patch [2/2] in that series into smaller patches and funnel them through the individual subsystem trees if you'd rather not apply them wholesale in one large treewide patch. Thanks, Lukas
