Re: [PATCH 4.20 237/352] igb: Fix an issue that PME is not enabled during runtime suspend

[niveditas98@gmail.com]

Bugzilla https://bugzilla.kernel.org/show_bug.cgi?id=202571

The igb commit is commit fb29f76cc5668d8fc9e2b04ebfaf4de4f62e1866


On Tue, Feb 12, 2019 at 7:21 PM Arvind Sankar <nivedita@alum.mit.edu> wrote:
>
> After upgrading to 4.20.8, I got a WARN in my dmesg and I suspect this
> change. Reverting it removes this warning.
>
> I am not subscribed to the list, please Cc me if any follup questions.
>
> [   12.457238] ------------[ cut here ]------------
> [   12.457251] PCI PM: State of device not saved by igb_runtime_suspend+0x0/0x80
> [   12.457275] WARNING: CPU: 1 PID: 175 at
> drivers/pci/pci-driver.c:1280 pci_pm_runtime_suspend+0x127/0x130
> [   12.457277] Modules linked in: 8021q zfs(PO) zunicode(PO) zlua(PO)
> zcommon(PO) znvpair(PO) zavl(PO) icp(PO) spl(O) zlib_inflate
> x86_pkg_temp_thermal kvm_intel kvm coretemp hwmon irqbypass
> crc32_pclmul crc32c_intel ghash_clmulni_intel aesni_intel aes_x86_64
> crypto_simd cryptd glue_helper intel_cstate intel_uncore
> intel_rapl_perf intel_pch_thermal i2c_i801 ipmi_si ipmi_devintf
> ipmi_msghandler
> [   12.457318] CPU: 1 PID: 175 Comm: kworker/1:1 Tainted: P
> O    T 4.20.8 #16
> [   12.457320] Hardware name: Supermicro Super Server/X10SDV-7TP8F,
> BIOS 2.0 06/13/2018
> [   12.457329] Workqueue: pm pm_runtime_work
> [   12.457337] RIP: 0010:pci_pm_runtime_suspend+0x127/0x130
> [   12.457340] Code: f7 0f 00 8b 44 24 04 eb a1 44 39 e8 74 9a 80 3d
> cb 7b d0 00 00 75 91 48 c7 c7 d8 b3 75 a2 c6 05 bb 7b d0 00 01 e8 fe
> 37 cf ff <0f> 0b 31 c0 e9 77 ff ff ff 41 55 41 54 45 31 e4 55 53 48 89
> fb 48
> [   12.457343] RSP: 0018:ffffb300406abd88 EFLAGS: 00010282
> [   12.457347] RAX: 0000000000000000 RBX: ffff965fc6ff20b0 RCX: ffffffffa2829cb8
> [   12.457350] RDX: 0000000000000001 RSI: 0000000000000082 RDI: ffffffffa3755d88
> [   12.457353] RBP: ffffffffa24a7980 R08: 0000000000000001 R09: 0000000000000442
> [   12.457355] R10: 0000000000000003 R11: 0000000000000000 R12: ffff965fc6ff2000
> [   12.457358] R13: 0000000000000000 R14: ffffffffa1b62550 R15: 0000000000000000
> [   12.457361] FS:  0000000000000000(0000) GS:ffff96671f440000(0000)
> knlGS:0000000000000000
> [   12.457365] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [   12.457368] CR2: 00007fff7670c11c CR3: 00000002c7c0a004 CR4: 00000000003606e0
> [   12.457370] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [   12.457373] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
> [   12.457374] Call Trace:
> [   12.457387]  ? __switch_to_asm+0x34/0x70
> [   12.457395]  __rpm_callback+0x70/0x1b0
> [   12.457401]  ? __switch_to_asm+0x34/0x70
> [   12.457406]  ? pci_dev_put+0x20/0x20
> [   12.457412]  rpm_callback+0x66/0x90
> [   12.457418]  ? pci_dev_put+0x20/0x20
> [   12.457424]  rpm_suspend+0x15a/0x570
> [   12.457431]  pm_runtime_work+0x8c/0xa0
> [   12.457439]  process_one_work+0x1d2/0x350
> [   12.457446]  worker_thread+0x28/0x3d0
> [   12.457453]  ? wq_calc_node_cpumask.constprop.50+0x20/0x20
> [   12.457457]  kthread+0x103/0x120
> [   12.457462]  ? kthread_create_on_node+0x90/0x90
> [   12.457468]  ret_from_fork+0x35/0x40
> [   12.457472] ---[ end trace aa262aac85ec967a ]---