diff options
| author | Ben Hutchings <bhutchings@solarflare.com> | 2012-07-27 21:50:57 +0200 |
|---|---|---|
| committer | Ben Hutchings <bhutchings@solarflare.com> | 2012-08-24 21:10:23 +0200 |
| commit | 3f65ea5b2aa0f76a463c85ed39955c26faf4f5a7 (patch) | |
| tree | dd07c3cbd93ca44d132d0d86be1f3d400d874b7c /drivers/net/ethernet/sfc | |
| parent | sfc: Remove overly paranoid locking assertions from netdev operations (diff) | |
| download | linux-3f65ea5b2aa0f76a463c85ed39955c26faf4f5a7.tar.xz linux-3f65ea5b2aa0f76a463c85ed39955c26faf4f5a7.zip | |
sfc: Remove bogus comment about MTU change and RX buffer overrun
RX DMA is limited by the length specified in each descriptor and not
by the MAC. Over-length frames may get into the RX FIFO regardless of
the MAC settings, due to a hardware bug, but they will be truncated by
the packet DMA engine and reported as such in the completion event.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Diffstat (limited to 'drivers/net/ethernet/sfc')
| -rw-r--r-- | drivers/net/ethernet/sfc/efx.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/ethernet/sfc/efx.c b/drivers/net/ethernet/sfc/efx.c index b29db12756f9..4105a6664765 100644 --- a/drivers/net/ethernet/sfc/efx.c +++ b/drivers/net/ethernet/sfc/efx.c @@ -1953,8 +1953,6 @@ static int efx_change_mtu(struct net_device *net_dev, int new_mtu) netif_dbg(efx, drv, efx->net_dev, "changing MTU to %d\n", new_mtu); mutex_lock(&efx->mac_lock); - /* Reconfigure the MAC before enabling the dma queues so that - * the RX buffers don't overflow */ net_dev->mtu = new_mtu; efx->type->reconfigure_mac(efx); mutex_unlock(&efx->mac_lock); |