[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20231204100811.2708884-1-tobias@waldekranz.com>
Date: Mon, 4 Dec 2023 11:08:08 +0100
From: Tobias Waldekranz <tobias@...dekranz.com>
To: davem@...emloft.net,
kuba@...nel.org
Cc: andrew@...n.ch,
gregory.clement@...tlin.com,
sebastian.hesselbarth@...il.com,
robh+dt@...nel.org,
krzysztof.kozlowski+dt@...aro.org,
conor+dt@...nel.org,
netdev@...r.kernel.org
Subject: [PATCH v2 net-next 0/3] net: mvmdio: Performance related improvements
Observations of the XMDIO bus on a CN9130-based system during a
firmware download showed a very low bus utilization, which stemmed
from the 150us (10x the average access time) sleep which would take
place when the first poll did not succeed.
With this series in place, bus throughput increases by about 10x,
multiplied by whatever gain you are able to extract from running the
MDC at a higher frequency (hardware dependent).
I would really appreciate it if someone with access to hardware using
the IRQ driven path could test that out, since I have not been able to
figure out how to set this up on CN9130.
v1 -> v2:
- Remove dead code
- Simplify IRQ path, now that the polled path is separate.
Tobias Waldekranz (3):
arm64: dts: marvell: cp11x: Provide clock names for MDIO controllers
net: mvmdio: Avoid excessive sleeps in polled mode
net: mvmdio: Support setting the MDC frequency on XSMI controllers
arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 4 +
drivers/net/ethernet/marvell/mvmdio.c | 97 ++++++++++++-------
2 files changed, 64 insertions(+), 37 deletions(-)
--
2.34.1
Powered by blists - more mailing lists