e1000e: fix issues with S0ix

[Timo Aaltonen]

[Impact]

1. Cold boot system to desktop

2. Press power button to suspend the system

3. Press power button to resume the system.

4. Check the log, you will only find

     e1000e 0000:00:1f.6 eno2: Hardware Error

5. Press power button to suspend the system again

6. System will *auto resume* from the suspend. Then check the logs, you *will find* follow logs.

    [ 120.895527] PM: pci_pm_suspend(): e1000e_pm_suspend+0x0/0x730 [e1000e] returns -2

    [ 120.895529] PM: dpm_run_callback(): pci_pm_suspend+0x0/0x150 returns -2

    [ 120.895530] PM: Device 0000:00:1f.6 failed to suspend async: error -2

    [ 121.539907] PM: Some devices failed to suspend, or early wake event detected

[Fix]

Four patches from upstream list.

 e1000e: fix S0ix flow to allow S0i3.2 subset entry
 e1000e: allow turning s0ix flows on for systems with ME
 e1000e: Add Dell's Comet Lake systems into s0ix heuristics
 e1000e: Add more Dell CML systems into s0ix heuristics

[Test case]

Repeat above.

Not really an SRU as the target is the unstable/devel kernel and oem-5.10 for focal (-oem-20.04-edge).

[Ubuntu Kernel Bot (ubuntu-kernel-bot)]

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-focal' to 'verification-done-focal'. If the problem still exists, change the tag 'verification-needed-focal' to 'verification-failed-focal'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

[Anthony Wong (anthonywong)]

Mario has submitted a v2 to the above patches: https://www.spinics.net/lists/netdev/msg705210.html

[Timo Aaltonen (tjaalton)]

and now there should be a v3?

[Timo Aaltonen (tjaalton)]

no, v5

[Timo Aaltonen (tjaalton)]

we'll go with the old set for now, and maybe reuse this bug when the final version is upstream