Page MenuHomeFreeBSD

Revert r336326.
ClosedPublic

Authored by markj on Dec 17 2018, 6:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 21 2024, 3:55 AM
Unknown Object (File)
Sep 20 2024, 7:08 PM
Unknown Object (File)
Sep 8 2024, 3:01 PM
Unknown Object (File)
Sep 7 2024, 8:33 PM
Unknown Object (File)
Sep 7 2024, 7:49 AM
Unknown Object (File)
Aug 31 2024, 12:07 PM
Unknown Object (File)
Aug 26 2024, 2:31 AM
Unknown Object (File)
Aug 17 2024, 3:18 PM
Subscribers
None

Details

Summary

On my Dell Latitude, the suspend/resume issue caused by ig4 appears to
be resolved by r342170. In particular, while debugging the issue I
found that ig4iic_pci0 and vgapci0 share an interrupt, and ig4iic_intr() was
being called after the device entered D3, so the read of IG4_REG_I2C_STA
returned 0xffffffff and we would enter an infinite loop as a result.

Test Plan

Suspend/resume on my laptop now works with ig4.ko. I happened to be
debugging the issue at the same time that r342170 was committed, so
based on the symptoms I'm fairly confident that r342170 addressed the
root cause.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj added reviewers: imp, gonzo, grembo.
This revision is now accepted and ready to land.Dec 17 2018, 6:26 PM
This revision was automatically updated to reflect the committed changes.