diff options
| author |   Jarod Wilson <jarod@redhat.com> | 2016-10-20 23:25:27 -0400 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-10-21 13:57:50 -0400 |
| commit | 8b1efc0f83f1f75b8f85c70d2211007de8fd7633 (patch) | |
| tree | 80b313286f4f191d241a803d5f8690b74187a4e9 /net/dsa | |
| parent | hv_netvsc: fix a race between netvsc_send() and netvsc_init_buf() (diff) | |
| download | linux-dev-8b1efc0f83f1f75b8f85c70d2211007de8fd7633.tar.xz linux-dev-8b1efc0f83f1f75b8f85c70d2211007de8fd7633.zip | |
net: remove MTU limits on a few ether_setup callers
These few drivers call ether_setup(), but have no ndo_change_mtu, and thus
were overlooked for changes to MTU range checking behavior. They
previously had no range checks, so for feature-parity, set their min_mtu
to 0 and max_mtu to ETH_MAX_MTU (65535), instead of the 68 and 1500
inherited from the ether_setup() changes. Fine-tuning can come after we get
back to full feature-parity here.
CC: netdev@vger.kernel.org
Reported-by: Asbjoern Sloth Toennesen <asbjorn@asbjorn.st>
CC: Asbjoern Sloth Toennesen <asbjorn@asbjorn.st>
CC: R Parameswaran <parameswaran.r7@gmail.com>
Signed-off-by: Jarod Wilson <jarod@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dsa')
| -rw-r--r-- | net/dsa/slave.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/dsa/slave.c b/net/dsa/slave.c index 68714a5f4a99..d0c7bce88743 100644 --- a/net/dsa/slave.c +++ b/net/dsa/slave.c @@ -1247,6 +1247,8 @@ int dsa_slave_create(struct dsa_switch *ds, struct device *parent, slave_dev->priv_flags |= IFF_NO_QUEUE; slave_dev->netdev_ops = &dsa_slave_netdev_ops; slave_dev->switchdev_ops = &dsa_slave_switchdev_ops; + slave_dev->min_mtu = 0; + slave_dev->max_mtu = ETH_MAX_MTU; SET_NETDEV_DEVTYPE(slave_dev, &dsa_type); netdev_for_each_tx_queue(slave_dev, dsa_slave_set_lockdep_class_one, |