[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20240102-new-gemini-ethernet-regression-v5-0-cf61ab3aa8cd@linaro.org>
Date: Tue, 02 Jan 2024 21:34:24 +0100
From: Linus Walleij <linus.walleij@...aro.org>
To: Hans Ulli Kroll <ulli.kroll@...glemail.com>,
"David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
Vladimir Oltean <olteanv@...il.com>, Household Cang <canghousehold@....com>,
Romain Gantois <romain.gantois@...tlin.com>
Cc: netdev@...r.kernel.org, Linus Walleij <linus.walleij@...aro.org>
Subject: [PATCH net v5 0/2] Fix a regression in the Gemini ethernet
controller.
These fixes were developed on top of the earlier fixes.
Finding the right solution is hard because the Gemini checksumming
engine is completely undocumented in the datasheets.
Signed-off-by: Linus Walleij <linus.walleij@...aro.org>
---
Changes in v5:
- Drop the patch re-implementing eth_header_parse_protocol()
- Link to v4: https://lore.kernel.org/r/20231222-new-gemini-ethernet-regression-v4-0-a36e71b0f32b@linaro.org
Changes in v4:
- Properly drop all MTU/TSO muckery in the TX function, the
whole approach is bogus.
- Make the raw etherype retrieveal return __be16, it is the
callers job to deal with endianness (as per the pattern
from if_vlan.h)
- Use __vlan_get_protocol() instead of vlan_get_protocol()
- Only actively bypass the TSS if the frame is over a certain
size.
- Drop comment that no longer applies.
- Link to v3: https://lore.kernel.org/r/20231221-new-gemini-ethernet-regression-v3-0-a96b4374bfe8@linaro.org
Changes in v3:
- Fix a whitespace bug in the first patch.
- Add generic accessors to obtain the raw ethertype of an
ethernet frame. VLAN already have the right accessors.
- Link to v2: https://lore.kernel.org/r/20231216-new-gemini-ethernet-regression-v2-0-64c269413dfa@linaro.org
Changes in v2:
- Drop the TSO and length checks altogether, this was never
working properly.
- Plan to make a proper TSO implementation in the next kernel
cycle.
- Link to v1: https://lore.kernel.org/r/20231215-new-gemini-ethernet-regression-v1-0-93033544be23@linaro.org
---
Linus Walleij (2):
net: ethernet: cortina: Drop software checksum and TSO
net: ethernet: cortina: Bypass checksumming engine of alien ethertypes
drivers/net/ethernet/cortina/gemini.c | 62 +++++++++++++++--------------------
1 file changed, 26 insertions(+), 36 deletions(-)
---
base-commit: 33cc938e65a98f1d29d0a18403dbbee050dcad9a
change-id: 20231203-new-gemini-ethernet-regression-3c672de9cfd9
Best regards,
--
Linus Walleij <linus.walleij@...aro.org>
Powered by blists - more mailing lists