]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
cxgb4vf: Revert force link up behaviour
authorArjun Vynipadath <arjun@chelsio.com>
Fri, 1 Mar 2019 10:14:48 +0000 (15:44 +0530)
committerDavid S. Miller <davem@davemloft.net>
Sat, 2 Mar 2019 07:20:20 +0000 (23:20 -0800)
Reverting force link up changes since this behaviour can be
achieved using VF link state feature.

Reverts:
commit 0913667ab3ad ("cxgb4vf: Forcefully link up virtual interfaces")

Signed-off-by: Arjun Vynipadath <arjun@chelsio.com>
Signed-off-by: Vishal Kulkarni <vishal@chelsio.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/chelsio/cxgb4vf/cxgb4vf_main.c

index 6b2754a574d63157456148798065acc83e367e17..62fd1250ff7ecfd7e27addbf9907554988426325 100644 (file)
@@ -155,6 +155,8 @@ void t4vf_os_link_changed(struct adapter *adapter, int pidx, int link_ok)
                const char *fc;
                const struct port_info *pi = netdev_priv(dev);
 
+               netif_carrier_on(dev);
+
                switch (pi->link_cfg.speed) {
                case 100:
                        s = "100Mbps";
@@ -200,6 +202,7 @@ void t4vf_os_link_changed(struct adapter *adapter, int pidx, int link_ok)
 
                netdev_info(dev, "link up, %s, full-duplex, %s PAUSE\n", s, fc);
        } else {
+               netif_carrier_off(dev);
                netdev_info(dev, "link down\n");
        }
 }
@@ -339,16 +342,6 @@ static int link_start(struct net_device *dev)
        if (ret == 0)
                ret = t4vf_enable_pi(pi->adapter, pi, true, true);
 
-       /* The Virtual Interfaces are connected to an internal switch on the
-        * chip which allows VIs attached to the same port to talk to each
-        * other even when the port link is down.  As a result, we generally
-        * want to always report a VI's link as being "up", provided there are
-        * no errors in enabling vi.
-        */
-
-       if (ret == 0)
-               netif_carrier_on(dev);
-
        return ret;
 }