[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f6324260-f019-4e1b-87c0-b57e862e28b4@lunn.ch>
Date: Mon, 17 Nov 2025 16:01:03 +0100
From: Andrew Lunn <andrew@...n.ch>
To: Wei Fang <wei.fang@....com>
Cc: "linux@...linux.org.uk" <linux@...linux.org.uk>,
"hkallweit1@...il.com" <hkallweit1@...il.com>,
"davem@...emloft.net" <davem@...emloft.net>,
"edumazet@...gle.com" <edumazet@...gle.com>,
"kuba@...nel.org" <kuba@...nel.org>,
"pabeni@...hat.com" <pabeni@...hat.com>,
"eric@...int.com" <eric@...int.com>,
"maxime.chevallier@...tlin.com" <maxime.chevallier@...tlin.com>,
"imx@...ts.linux.dev" <imx@...ts.linux.dev>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] net: phylink: add missing supported link modes for
the fixed-link
> The default TCP block size sent by iperf is 128KB. ENETC then fragments
> the packet via TSO and sends the fragments to the switch. Because the
> link speed between ENETC and the switch CPU port is 2.5Gbps, it takes
> approximately 420 us for the TCP block to be sent to the switch. However,
> the link speed of the switch's user port is only 1Gbps, and the switch takes
> approximately 1050 us to send the packets out. Therefore, packets
> accumulate within the switch. Without flow control enabled, this can
> exhaust the switch's buffer, eventually leading to congestion.
>
> Debugging results from the switch show that many packets are being
> dropped on the CPU port, and the reason of packet loss is precisely
> due to congestion.
BQL might help you. It could break the 128KB burst up into a number of
smaller bursts, helping avoid the congestion.
Are there any parameters you can change with TSO? This is one
stream. Sending it out at 2.5G line rate makes no sense when you know
it is going to hit a 1G egress. You might as well limit TSO to
1G. That will help single stream traffic. If you have multiple streams
it will not help, you will still hit congestion, assuming you can do
multiple TSOs in parallel.
Andrew
Powered by blists - more mailing lists