Comment 1 for bug 1662763

Revision history for this message
AceLan Kao (acelankao) wrote :

The 2 patches in mainline kernel can fix this issue

commit 8646f7b4cdf2d0557e718c4524a3e31455b92ad7
Author: Jacob Keller <email address hidden>
AuthorDate: Tue May 24 13:56:31 2016 -0700
Commit: Jeff Kirsher <email address hidden>
CommitDate: Wed Jun 29 11:14:31 2016 -0700

    igb: call igb_ptp_suspend during suspend/resume cycle

    Properly stop the extra workqueue items and ensure that we resume
    cleanly. This is better than using igb_ptp_init and igb_ptp_stop since
    these functions destroy the PHC device, which will cause other problems
    if we do so. Since igb_ptp_reset now re-schedules the work-queue item we
    don't need an equivalent igb_ptp_resume in the resume workflow.

    Signed-off-by: Jacob Keller <email address hidden>
    Tested-by: Aaron Brown <email address hidden>
    Signed-off-by: Jeff Kirsher <email address hidden>

commit e3f2350de829eb0c3349f416feed81c0a3ac0732
Author: Jacob Keller <email address hidden>
AuthorDate: Tue May 24 13:56:30 2016 -0700
Commit: Jeff Kirsher <email address hidden>
CommitDate: Wed Jun 29 11:00:22 2016 -0700

    igb: implement igb_ptp_suspend

    Make igb_ptp_stop take advantage of this new function to reduce code
    duplication.

    Signed-off-by: Jacob Keller <email address hidden>
    Tested-by: Aaron Brown <email address hidden>
    Signed-off-by: Jeff Kirsher <email address hidden>