From: LEROY Christophe <christophe.leroy2@cs-soprasteria.com>
To: Jakub Kicinski <kuba@kernel.org>, Rosen Penev <rosenp@gmail.com>
Cc: "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
Andrew Lunn <andrew+netdev@lunn.ch>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Paolo Abeni <pabeni@redhat.com>,
"open list:FREESCALE QUICC ENGINE UCC ETHERNET DRIVER"
<linuxppc-dev@lists.ozlabs.org>,
open list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH net-next] net: ucc_geth: Batch RX packets before stack handoff
Date: Thu, 21 May 2026 12:55:23 +0000 [thread overview]
Message-ID: <8bc3dc4e-dab7-4354-a594-95ebda75987a@cs-soprasteria.com> (raw)
In-Reply-To: <20260520174545.69766438@kernel.org>
Hi Jakub,
Le 21/05/2026 à 02:45, Jakub Kicinski a écrit :
> On Wed, 20 May 2026 17:39:41 -0700 Rosen Penev wrote:
>>> On Sun, 17 May 2026 12:28:56 -0700 Rosen Penev wrote:
>>>> Collect received skbs on a local list during RX polling and pass the
>>>> completed batch to netif_receive_skb_list(). This lets the networking
>>>> stack process packets from a poll cycle in bulk instead of handing each
>>>> skb up individually.
>>>
>>> GRO should be even better.
>> GRO will result in slower routing performance because there is no
>> hardware checksum.
>
> Mention this in the commit message too.
> Network adapters without checksum offload are pretty rare these days.
> Speaking of being old, do you know if this driver is used in practice?
> Maybe we can delete it.
>
That's way too early to remove that driver.
UCC is what provides Ethernet connectivity in the powerpc MPC83xx CPU
family. This family has just been declared End Of Life by January this
year with a Last Time Buy Date 30 Jul 2026 and Last Time Delivery Date
by 30 Apr 2027. We have that CPU on hundreds of boards spread all over
Europe and have to maintain those systems for the next 10 to 15 years if
not even more.
So we can maybe reconsider removing that driver by 2040 but unlikely before.
Christophe
next prev parent reply other threads:[~2026-05-21 12:55 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-17 19:28 Rosen Penev
2026-05-17 20:24 ` Andrew Lunn
2026-05-17 20:44 ` Rosen Penev
2026-05-17 21:01 ` Andrew Lunn
2026-05-20 23:57 ` Jakub Kicinski
2026-05-21 0:39 ` Rosen Penev
2026-05-21 0:45 ` Jakub Kicinski
2026-05-21 0:54 ` Rosen Penev
2026-05-21 12:55 ` LEROY Christophe [this message]
2026-05-21 13:41 ` Eric Dumazet
2026-05-21 23:28 ` Rosen Penev
2026-05-22 4:44 ` Eric Dumazet
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8bc3dc4e-dab7-4354-a594-95ebda75987a@cs-soprasteria.com \
--to=christophe.leroy2@cs-soprasteria.com \
--cc=andrew+netdev@lunn.ch \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=rosenp@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®