s390/qeth: use core MTU range checking
authorJulian Wiedmann <jwi@linux.ibm.com>
Thu, 19 Jul 2018 10:43:54 +0000 (12:43 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sat, 21 Jul 2018 17:12:30 +0000 (10:12 -0700)
qeth's ndo_change_mtu() only applies some trivial bounds checking. Set
up dev->min_mtu properly, so that dev_set_mtu() can do this for us.

Signed-off-by: Julian Wiedmann <jwi@linux.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/s390/net/qeth_core.h
drivers/s390/net/qeth_core_main.c
drivers/s390/net/qeth_core_mpc.h
drivers/s390/net/qeth_l2_main.c
drivers/s390/net/qeth_l3_main.c

index 6f02a6cbe59ec81e9d908938fa0d685c592ab2e8..994ac7f434d52e5651faa2665bf0e87c3f6267db 100644 (file)
@@ -993,7 +993,6 @@ void qeth_clear_cmd_buffers(struct qeth_channel *);
 void qeth_clear_qdio_buffers(struct qeth_card *);
 void qeth_setadp_promisc_mode(struct qeth_card *);
 struct net_device_stats *qeth_get_stats(struct net_device *);
-int qeth_change_mtu(struct net_device *, int);
 int qeth_setadpparms_change_macaddr(struct qeth_card *);
 void qeth_tx_timeout(struct net_device *);
 void qeth_prepare_control_data(struct qeth_card *, int,
index ab3d63f987791f7b118713fb60156fe710ba2012..717511c167e7dd1871bc703f65a27c0d03c56429 100644 (file)
@@ -2332,20 +2332,6 @@ static int qeth_get_mtu_outof_framesize(int framesize)
        }
 }
 
-static int qeth_mtu_is_valid(struct qeth_card *card, int mtu)
-{
-       switch (card->info.type) {
-       case QETH_CARD_TYPE_OSD:
-       case QETH_CARD_TYPE_OSM:
-       case QETH_CARD_TYPE_OSX:
-       case QETH_CARD_TYPE_IQD:
-               return ((mtu >= 576) && (mtu <= card->dev->max_mtu));
-       case QETH_CARD_TYPE_OSN:
-       default:
-               return 1;
-       }
-}
-
 static int qeth_ulp_enable_cb(struct qeth_card *card, struct qeth_reply *reply,
                unsigned long data)
 {
@@ -4204,24 +4190,6 @@ void qeth_setadp_promisc_mode(struct qeth_card *card)
 }
 EXPORT_SYMBOL_GPL(qeth_setadp_promisc_mode);
 
-int qeth_change_mtu(struct net_device *dev, int new_mtu)
-{
-       struct qeth_card *card;
-       char dbf_text[15];
-
-       card = dev->ml_priv;
-
-       QETH_CARD_TEXT(card, 4, "chgmtu");
-       sprintf(dbf_text, "%8x", new_mtu);
-       QETH_CARD_TEXT(card, 4, dbf_text);
-
-       if (!qeth_mtu_is_valid(card, new_mtu))
-               return -EINVAL;
-       dev->mtu = new_mtu;
-       return 0;
-}
-EXPORT_SYMBOL_GPL(qeth_change_mtu);
-
 struct net_device_stats *qeth_get_stats(struct net_device *dev)
 {
        struct qeth_card *card;
@@ -5696,7 +5664,7 @@ static struct net_device *qeth_alloc_netdev(struct qeth_card *card)
 
        dev->ml_priv = card;
        dev->watchdog_timeo = QETH_TX_TIMEOUT;
-       dev->min_mtu = 64;
+       dev->min_mtu = IS_OSN(card) ? 64 : 576;
         /* initialized when device first goes online: */
        dev->max_mtu = 0;
        dev->mtu = 0;
index 54c35224262a802394baaa40b0437bd0e3c214ef..cf5ad94e960a0b86ca1d70ff7e69edabc8ba6cb3 100644 (file)
@@ -65,6 +65,7 @@ enum qeth_card_types {
 };
 
 #define IS_IQD(card)   ((card)->info.type == QETH_CARD_TYPE_IQD)
+#define IS_OSN(card)   ((card)->info.type == QETH_CARD_TYPE_OSN)
 
 #define QETH_MPC_DIFINFO_LEN_INDICATES_LINK_TYPE 0x18
 /* only the first two bytes are looked at in qeth_get_cardname_short */
index 00d8bb1d2a41d3c1e489935f06a0226febba5734..668f80680575e9c58f6db5f1fb3070fcd7c391f1 100644 (file)
@@ -928,7 +928,6 @@ static const struct net_device_ops qeth_l2_netdev_ops = {
        .ndo_set_rx_mode        = qeth_l2_set_rx_mode,
        .ndo_do_ioctl           = qeth_do_ioctl,
        .ndo_set_mac_address    = qeth_l2_set_mac_address,
-       .ndo_change_mtu         = qeth_change_mtu,
        .ndo_vlan_rx_add_vid    = qeth_l2_vlan_rx_add_vid,
        .ndo_vlan_rx_kill_vid   = qeth_l2_vlan_rx_kill_vid,
        .ndo_tx_timeout         = qeth_tx_timeout,
index e5fa8e5ac1b36ea9815f05d671465e778b2c5338..078b891a6d249d9b966fd77e3924e6d7c670e209 100644 (file)
@@ -2506,7 +2506,6 @@ static const struct net_device_ops qeth_l3_netdev_ops = {
        .ndo_validate_addr      = eth_validate_addr,
        .ndo_set_rx_mode        = qeth_l3_set_rx_mode,
        .ndo_do_ioctl           = qeth_do_ioctl,
-       .ndo_change_mtu         = qeth_change_mtu,
        .ndo_fix_features       = qeth_fix_features,
        .ndo_set_features       = qeth_set_features,
        .ndo_vlan_rx_add_vid    = qeth_l3_vlan_rx_add_vid,
@@ -2523,7 +2522,6 @@ static const struct net_device_ops qeth_l3_osa_netdev_ops = {
        .ndo_validate_addr      = eth_validate_addr,
        .ndo_set_rx_mode        = qeth_l3_set_rx_mode,
        .ndo_do_ioctl           = qeth_do_ioctl,
-       .ndo_change_mtu         = qeth_change_mtu,
        .ndo_fix_features       = qeth_fix_features,
        .ndo_set_features       = qeth_set_features,
        .ndo_vlan_rx_add_vid    = qeth_l3_vlan_rx_add_vid,