On Wed, Oct 14, 2020 at 03:33:17PM +0200, Ian Kumlien wrote: > On Wed, Oct 14, 2020 at 10:34 AM Kai-Heng Feng > <kai.heng.feng@xxxxxxxxxxxxx> wrote: > > > On Oct 12, 2020, at 18:20, Ian Kumlien <ian.kumlien@xxxxxxxxx> wrote: > > > On Thu, Oct 8, 2020 at 6:13 PM Bjorn Helgaas <helgaas@xxxxxxxxxx> wrote: > > >> OK, now we're getting close. We just need to flesh out the > > >> justification. We need: > > >> > > >> - Tidy subject line. Use "git log --oneline drivers/pci/pcie/aspm.c" > > >> and follow the example. > > > > > > Will do > > > > > >> - Description of the problem. I think it's poor bandwidth on your > > >> Intel I211 device, but we don't have the complete picture because > > >> that NIC is 03:00.0, which doesn't appear above at all. > > > > > > I think we'll use Kai-Hengs issue, since it's actually more related to > > > the change itself... > > > > > > Mine is a side effect while Kai-Heng is actually hitting an issue > > > caused by the bug. > > > > I filed a bug here: > > https://bugzilla.kernel.org/show_bug.cgi?id=209671 > > Thanks! Sigh. I feel like I'm just not getting anywhere here. I still do not have a "before" and "after" set of lspci output. Kai-Heng's bugzilla has two sets of output, but one is a working config with CONFIG_PCIEASPM_DEFAULT=y and the other is a working config with Ian's patch applied and CONFIG_PCIEASPM_POWERSAVE=y. Comparing them doesn't show the effect of Ian's patch; it shows the combined effect of Ian's patch and the CONFIG_PCIEASPM_POWERSAVE=y change. I'm not really interested in spending a few hours trying to reverse-engineer the important changes. Can you please, please, collect these on your system, Ian? I assume that you can easily collect it once without your patch, when you see poor I211 NIC performance but the system is otherwise working. And you can collect it again *with* your patch. Same Kconfig, same *everything* except adding your patch. Bjorn
