> > > + /* MTU range: 68 - 9706 */ > > > + netdev->min_mtu = ETH_MIN_MTU; > > > > http://elixir.free- > > electrons.com/linux/latest/source/net/ethernet/eth.c#L361 > Supported range of Min and Max MTU should be at the discretion > of the driver. Therefore, initialization looks fine to me. > > I could not clearly understand the problem being highlighted > over here. Could you further clarify? This is already setting min_mtu to ETH_MIN_MTU. There is no need for you to set it. > > > #define HNS3_RING_MAX_PENDING 32768 > > > +#define HNS3_MAX_MTU 9728 > > > > It seems odd that it does not already exists somewhere. The core does > > not enforce the MTU. You could be passed a frame which is bigger. So > > you should check before trying to pass something to the hardware which > > the hardware cannot handle. > There is a check already in place for this as well since 4.10-rc1. Yes, the core will check when changing the MTU. But when passing frames to be transmitted, it does not check the frame fits the MTU. DSA actually makes use of this, when passing frames to an Ethernet switch attached to the interface. We need to add an extra header to the frame, which makes the frame bigger than the MTU. Most Ethernet drivers are happy with this, they send the frame. Other reject it, and we have had to make driver changes. And some just explode :-( If 9728 is a hard limit for your device, you should check when passed a frame to make sure it is actually <= 9728 bytes in length. Andrew -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
