ethernet/qlogic: use core min/max MTU checking
authorJarod Wilson <jarod@redhat.com>
Mon, 17 Oct 2016 19:54:08 +0000 (15:54 -0400)
committerDavid S. Miller <davem@davemloft.net>
Tue, 18 Oct 2016 15:34:19 +0000 (11:34 -0400)
qede: min_mtu 46, max_mtu 9600
- Put define for max in qede.h

qlcnic: min_mtu 68, max_mtu 9600

CC: netdev@vger.kernel.org
CC Dept-GELinuxNICDev@qlogic.com
CC: Yuval Mintz <Yuval.Mintz@qlogic.com>
CC: Ariel Elior <Ariel.Elior@qlogic.com>
Signed-off-by: Jarod Wilson <jarod@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/qlogic/qede/qede.h
drivers/net/ethernet/qlogic/qede/qede_ethtool.c
drivers/net/ethernet/qlogic/qede/qede_main.c
drivers/net/ethernet/qlogic/qlcnic/qlcnic_hw.c
drivers/net/ethernet/qlogic/qlcnic/qlcnic_main.c

index f50e5270ad3044c97b387ec9187bac5d651caa81..9135b9d37dfaa46a506ca7d28be1fb6584b98a81 100644 (file)
@@ -362,8 +362,9 @@ void qede_recycle_rx_bd_ring(struct qede_rx_queue *rxq, struct qede_dev *edev,
 #define NUM_TX_BDS_MIN         128
 #define NUM_TX_BDS_DEF         NUM_TX_BDS_MAX
 
-#define QEDE_MIN_PKT_LEN       64
-#define QEDE_RX_HDR_SIZE       256
+#define QEDE_MIN_PKT_LEN               64
+#define QEDE_RX_HDR_SIZE               256
+#define QEDE_MAX_JUMBO_PACKET_SIZE     9600
 #define        for_each_queue(i) for (i = 0; i < edev->num_queues; i++)
 
 #endif /* _QEDE_H_ */
index 25a9b293ee8f8cf2352f8d90c34c816be3edecba..b7dbb4493a645724636c225a79d951eb88225e94 100644 (file)
@@ -723,19 +723,11 @@ static void qede_update_mtu(struct qede_dev *edev, union qede_reload_args *args)
 }
 
 /* Netdevice NDOs */
-#define ETH_MAX_JUMBO_PACKET_SIZE      9600
-#define ETH_MIN_PACKET_SIZE            60
 int qede_change_mtu(struct net_device *ndev, int new_mtu)
 {
        struct qede_dev *edev = netdev_priv(ndev);
        union qede_reload_args args;
 
-       if ((new_mtu > ETH_MAX_JUMBO_PACKET_SIZE) ||
-           ((new_mtu + ETH_HLEN) < ETH_MIN_PACKET_SIZE)) {
-               DP_ERR(edev, "Can't support requested MTU size\n");
-               return -EINVAL;
-       }
-
        DP_VERBOSE(edev, (NETIF_MSG_IFUP | NETIF_MSG_IFDOWN),
                   "Configuring MTU size of %d\n", new_mtu);
 
index 0e483afc2b87019dd1e2c68788618fd77e8bda83..4f298656bf47409e966a0a7cf5377ff089c0efd5 100644 (file)
@@ -2391,6 +2391,10 @@ static void qede_init_ndev(struct qede_dev *edev)
 
        ndev->hw_features = hw_features;
 
+       /* MTU range: 46 - 9600 */
+       ndev->min_mtu = ETH_ZLEN - ETH_HLEN;
+       ndev->max_mtu = QEDE_MAX_JUMBO_PACKET_SIZE;
+
        /* Set network device HW mac */
        ether_addr_copy(edev->ndev->dev_addr, edev->dev_info.common.hw_mac);
 }
index 509b596cf1e8612436af6886dc0e2a6e78acdc1d..838cc0ceafd8d0824495206bf30cdaf53f98bc59 100644 (file)
@@ -1024,12 +1024,6 @@ int qlcnic_change_mtu(struct net_device *netdev, int mtu)
        struct qlcnic_adapter *adapter = netdev_priv(netdev);
        int rc = 0;
 
-       if (mtu < P3P_MIN_MTU || mtu > P3P_MAX_MTU) {
-               dev_err(&adapter->netdev->dev, "%d bytes < mtu < %d bytes"
-                       " not supported\n", P3P_MAX_MTU, P3P_MIN_MTU);
-               return -EINVAL;
-       }
-
        rc = qlcnic_fw_cmd_set_mtu(adapter, mtu);
 
        if (!rc)
index 3ae3968b0edf5ec260f8eaca3255a503ddc74278..4c0cce962585f61b61a43d780b0cbe64217de02e 100644 (file)
@@ -2342,6 +2342,10 @@ qlcnic_setup_netdev(struct qlcnic_adapter *adapter, struct net_device *netdev,
        netdev->priv_flags |= IFF_UNICAST_FLT;
        netdev->irq = adapter->msix_entries[0].vector;
 
+       /* MTU range: 68 - 9600 */
+       netdev->min_mtu = P3P_MIN_MTU;
+       netdev->max_mtu = P3P_MAX_MTU;
+
        err = qlcnic_set_real_num_queues(adapter, adapter->drv_tx_rings,
                                         adapter->drv_sds_rings);
        if (err)