diff options
| author | Maxime Chevallier <maxime.chevallier@bootlin.com> | 2026-09-17 23:53:37 +0200 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2026-09-22 18:51:17 -0700 |
| commit | b8a26d46c0a4254f8bfe143681adb9d18d020299 (patch) | |
| tree | 545f3338840b3b1f9a0fbc9b327da8309e0e3bab /tools/perf/scripts/python | |
| parent | b42e7012773a0e81e97a2dda6ef907f5147a6658 (diff) | |
| download | linux-b8a26d46c0a4254f8bfe143681adb9d18d020299.tar.gz linux-b8a26d46c0a4254f8bfe143681adb9d18d020299.zip | |
net: stmmac: size the RX buffers from the frame length, not the MTU
When picking the buffsize to use based on the MTU, we shouldn't check
only the MTU value, but also :
- ETH_HLEN for the L2 header,
- up to 2 VLAN tags,
- the FCS,
The default bufsize is 1536 bytes, which is enough to contain all the
above so this hasn't surfaced before, but the addition of NET_IP_ALIGN
to the start of buffer address tripped the Jumbo selftest, leading to
this discovery.
With that, we don't need the '>=' checks on the buffer len, we can use
more consistent comparison operators in stmmac_set_bfsize.
Fixes: 286a83721720 ("stmmac: add CHAINED descriptor mode support (V4)")
Reviewed-by: Nicolai Buchwitz <nb@tipi-net.de>
Signed-off-by: Maxime Chevallier <maxime.chevallier@bootlin.com>
Link: https://patch.msgid.link/20260917215339.2022523-7-maxime.chevallier@bootlin.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions
