lkml.org 
[lkml]   [2019]   [Mar]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 5.0 167/262] net: phy: consider latched link-down status in polling mode
    Date
    From: Heiner Kallweit <hkallweit1@gmail.com>

    [ Upstream commit 93c0970493c71f264e6c3c7caf1ff24a9e1de786 ]

    The link status value latches link-down events. To get the current
    status we read the register twice in genphy_update_link(). There's
    a potential risk that we miss a link-down event in polling mode.
    This may cause issues if the user e.g. connects his machine to a
    different network.

    On the other hand reading the latched value may cause issues in
    interrupt mode. Following scenario:

    - After boot link goes up
    - phy_start() is called triggering an aneg restart, hence link goes
    down and link-down info is latched.
    - After aneg has finished link goes up and triggers an interrupt.
    Interrupt handler reads link status, means it reads the latched
    "link is down" info. But there won't be another interrupt as long
    as link stays up, therefore phylib will never recognize that link
    is up.

    Deal with both scenarios by reading the register twice in interrupt
    mode only.

    Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/net/phy/phy-c45.c | 10 ++++++++--
    drivers/net/phy/phy_device.c | 13 +++++++++----
    2 files changed, 17 insertions(+), 6 deletions(-)

    diff --git a/drivers/net/phy/phy-c45.c b/drivers/net/phy/phy-c45.c
    index 03af927fa5ad..e39bf0428dd9 100644
    --- a/drivers/net/phy/phy-c45.c
    +++ b/drivers/net/phy/phy-c45.c
    @@ -147,9 +147,15 @@ int genphy_c45_read_link(struct phy_device *phydev, u32 mmd_mask)
    mmd_mask &= ~BIT(devad);

    /* The link state is latched low so that momentary link
    - * drops can be detected. Do not double-read the status
    - * register if the link is down.
    + * drops can be detected. Do not double-read the status
    + * in polling mode to detect such short link drops.
    */
    + if (!phy_polling_mode(phydev)) {
    + val = phy_read_mmd(phydev, devad, MDIO_STAT1);
    + if (val < 0)
    + return val;
    + }
    +
    val = phy_read_mmd(phydev, devad, MDIO_STAT1);
    if (val < 0)
    return val;
    diff --git a/drivers/net/phy/phy_device.c b/drivers/net/phy/phy_device.c
    index 46c86725a693..b3d564592ad4 100644
    --- a/drivers/net/phy/phy_device.c
    +++ b/drivers/net/phy/phy_device.c
    @@ -1683,10 +1683,15 @@ int genphy_update_link(struct phy_device *phydev)
    {
    int status;

    - /* Do a fake read */
    - status = phy_read(phydev, MII_BMSR);
    - if (status < 0)
    - return status;
    + /* The link state is latched low so that momentary link
    + * drops can be detected. Do not double-read the status
    + * in polling mode to detect such short link drops.
    + */
    + if (!phy_polling_mode(phydev)) {
    + status = phy_read(phydev, MII_BMSR);
    + if (status < 0)
    + return status;
    + }

    /* Read link and autonegotiation status */
    status = phy_read(phydev, MII_BMSR);
    --
    2.19.1
    \
     
     \ /
      Last update: 2019-03-27 20:40    [W:3.659 / U:0.200 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site