peci: eSPI host stops responding

[crawfxrd]

• Model: CONFIG_PECI_OVER_ESPI=y
• EC version: dd555b9
• OS: Pop!_OS 22.04
• Kernel: 6.2.7-060207-generic

Cases:

• Unplugging a display connected to the TBT port causes PECI GetTemp() timeouts
• Suspend/resume causes PECI GetTemp() timeouts

Steps to reproduce

1. Plug in a display to the TBT port
2. Power on system, booting to OS
3. Unplug the display

Expected behavior

• PECI commands resume normal operation
• CPU temp reporting is accurate

Actual behavior

• PECI commands begin timing out and do not recover
• CPU temp reported is always 0

Additional info

This specifically affects the PECI-over-eSPI implementation (CONFIG_PECI_OVER_ESPI=y).

Acknowledging the completion in #368 fixed it from hanging on shutdown.

Presumably, this could happen because of:

• invalid/insufficient checks in peci_available()
• missing the check to see if a transaction is currently in progress in peci_get_temp()
• an issue specifically with eSPI usage?

Workaround when it gets in this state:

• Completely power off system (not restart) and unplug from AC power

Blocks: #370

[crawfxrd]

PECI over eSPI is now used by

• addw4
• lemp13
• lemp13-b
• oryp11
• oryp12

While GetTemp is the most likely PECI call to fail, #525 shows that WrPkgConfig also can fail because of this.

[nextsux]

Is there something I can do to help this move forward? I can reproduce it with my HW setup pretty easily so I can provide any additional debug logs if needed.

[jaime-ez]

Any chance this will be fixed soon? Something we can help with? It is a really bad experience having a nice and expensive laptop such as the lemp13 that makes more noise than my washing machine.

[nextsux]

@jaime-ez at this point I've workarounded this by implementing dbus service that switches fan to manual and a sliding bar changing fan speed manually for gnome-shell 🙈 far from ideal but I can share this if you're interested after some cleanup

Not nice, not packaged but I've added some instructions:

https://github.com/nextsux/ec/tree/manual-fan-control

[ywwg]

for me, powering down, removing AC, and then powering back up fixes the problem. Annoying to do on a regular basis but not that big a deal.