[Bug 109241] [CI][BAT] all suspend tests - dmesg-warn - called from state HALTED

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Jan 7 23:13:02 UTC 2019


https://bugs.freedesktop.org/show_bug.cgi?id=109241

--- Comment #3 from Chris Wilson <chris at chris-wilson.co.uk> ---
Fwiw,

commit 2b3e88ea65287ba738a798622405b15344871085
Author: Heiner Kallweit <hkallweit1 at gmail.com>
Date:   Sun Dec 16 18:30:14 2018 +0100

    net: phy: improve phy state checking

    Add helpers phy_is_started() and __phy_is_started() to avoid open-coded
    checks whether PHY has been started. To make the check easier move
    PHY_HALTED before PHY_UP in enum phy_state. Further improvements:

    phy_start_aneg():
    Return -EBUSY and print warning if function is called from a non-started
    state (DOWN, READY, HALTED). Better check because function is exported
    and drivers may use it incorrectly.

    phy_interrupt():
    Return IRQ_NONE also if state is DOWN or READY. We should never receive
    an interrupt in one of these states, but better play safe.

    phy_stop():
    Just return and print a warning if PHY is in a non-started state.
    This warning should help to identify drivers with unbalanced calls to
    phy_start() / phy_stop().

    phy_state_machine():
    Schedule state machine run only if PHY is in a started state.
    E.g. if state is READY we don't need the state machine, it will be
    started by phy_start().

    v2:
    - don't use __func__ within phy_warn_state
    v3:
    - use WARN() instead of printing error message to facilitate debugging

    Signed-off-by: Heiner Kallweit <hkallweit1 at gmail.com>
    Reviewed-by: Florian Fainelli <f.fainelli at gmail.com>
    Signed-off-by: David S. Miller <davem at davemloft.net>

deemed to turn phy_start_aneg() while HALTED into an error (previously it still
did the phy_config_aneg()).

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20190107/a15d3512/attachment.html>


More information about the intel-gfx-bugs mailing list