Commit ccd8e877 authored by Dimitri Daskalakis's avatar Dimitri Daskalakis Committed by David S. Miller
Browse files

eth: fbnic: Add validation for MTU changes 



Increasing the MTU beyond the HDS threshold causes the hardware to
fragment packets across multiple buffers. If a single-buffer XDP program
is attached, the driver will drop all multi-frag frames. While we can't
prevent a remote sender from sending non-TCP packets larger than the MTU,
this will prevent users from inadvertently breaking new TCP streams.

Traditionally, drivers supported XDP with MTU less than 4Kb
(packet per page). Fbnic currently prevents attaching XDP when MTU is too high.
But it does not prevent increasing MTU after XDP is attached.

Fixes: 1b0a3950 ("eth: fbnic: Add XDP pass, drop, abort support")
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
Signed-off-by: default avatarDimitri Daskalakis <dimitri.daskalakis1@gmail.com>
Reviewed-by: default avatarSimon Horman <horms@kernel.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent be054cc6

drivers/net/ethernet/meta/fbnic/fbnic_netdev.c

+18 −0
Original line number Diff line number Diff line
@@ -262,6 +262,23 @@ static int fbnic_set_mac(struct net_device *netdev, void *p) 
	return 0;

}



static int fbnic_change_mtu(struct net_device *dev, int new_mtu)

{

	struct fbnic_net *fbn = netdev_priv(dev);



	if (fbnic_check_split_frames(fbn->xdp_prog, new_mtu, fbn->hds_thresh)) {

		dev_err(&dev->dev,

			"MTU %d is larger than HDS threshold %d in XDP mode\n",

			new_mtu, fbn->hds_thresh);



		return -EINVAL;

	}



	WRITE_ONCE(dev->mtu, new_mtu);



	return 0;

}



void fbnic_clear_rx_mode(struct fbnic_dev *fbd)

{

	struct net_device *netdev = fbd->netdev;
@@ -533,6 +550,7 @@ static const struct net_device_ops fbnic_netdev_ops = { 
	.ndo_start_xmit		= fbnic_xmit_frame,

	.ndo_features_check	= fbnic_features_check,

	.ndo_set_mac_address	= fbnic_set_mac,

	.ndo_change_mtu		= fbnic_change_mtu,

	.ndo_set_rx_mode	= fbnic_set_rx_mode,

	.ndo_get_stats64	= fbnic_get_stats64,

	.ndo_bpf		= fbnic_bpf,