Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 251564f6 authored by Veerasenareddy Burru's avatar Veerasenareddy Burru Committed by David S. Miller
Browse files

liquidio: update VF's netdev->max_mtu if there's a change in PF's MTU 



A VF's MTU is capped at the parent PF's MTU.  So if there's a change in the
PF's MTU, then update the VF's netdev->max_mtu.

Also remove duplicate log messages for MTU change.

Signed-off-by: default avatarVeerasenareddy Burru <veerasenareddy.burru@cavium.com>
Signed-off-by: default avatarRaghu Vatsavayi <raghu.vatsavayi@cavium.com>
Signed-off-by: default avatarFelix Manlunas <felix.manlunas@cavium.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 5022111e

drivers/net/ethernet/cavium/liquidio/lio_core.c

+0 −3
Original line number Diff line number Diff line
@@ -165,9 +165,6 @@ void liquidio_link_ctrl_cmd_completion(void *nctrl_ptr) 
		/* If command is successful, change the MTU. */

		netif_info(lio, probe, lio->netdev, "MTU Changed from %d to %d\n",

			   netdev->mtu, nctrl->ncmd.s.param1);

		dev_info(&oct->pci_dev->dev, "%s MTU Changed from %d to %d\n",

			 netdev->name, netdev->mtu,

			 nctrl->ncmd.s.param1);

		netdev->mtu = nctrl->ncmd.s.param1;

		queue_delayed_work(lio->link_status_wq.wq,

				   &lio->link_status_wq.wk.work, 0);

drivers/net/ethernet/cavium/liquidio/lio_vf_main.c

+6 −3
Original line number Diff line number Diff line
@@ -630,6 +630,12 @@ static void update_link_status(struct net_device *netdev, 
			txqs_stop(netdev);

		}



		if (lio->linfo.link.s.mtu != netdev->max_mtu) {

			dev_info(&oct->pci_dev->dev, "Max MTU Changed from %d to %d\n",

				 netdev->max_mtu, lio->linfo.link.s.mtu);

			netdev->max_mtu = lio->linfo.link.s.mtu;

		}



		if (lio->linfo.link.s.mtu < netdev->mtu) {

			dev_warn(&oct->pci_dev->dev,

				 "PF has changed the MTU for gmx port. Reducing the mtu from %d to %d\n",
@@ -1539,14 +1545,11 @@ static struct net_device_stats *liquidio_get_stats(struct net_device *netdev) 
static int liquidio_change_mtu(struct net_device *netdev, int new_mtu)

{

	struct lio *lio = GET_LIO(netdev);

	struct octeon_device *oct = lio->oct_dev;



	lio->mtu = new_mtu;



	netif_info(lio, probe, lio->netdev, "MTU Changed from %d to %d\n",

		   netdev->mtu, new_mtu);

	dev_info(&oct->pci_dev->dev, "%s MTU Changed from %d to %d\n",

		 netdev->name, netdev->mtu, new_mtu);



	netdev->mtu = new_mtu;