Re: [PATCH net] net: stmmac: Use hrtimer for TX coalescing

[Vincent Whitchurch <Vincent.Whitchurch@axis.com>]

On Wed, 2023-08-30 at 21:05 +0300, Maxim Mikityanskiy wrote:
> On Wed, 30 Aug 2023 at 14:55:37 +0000, Vincent Whitchurch wrote:
> > Any test results with this patch on the hardware with the performance
> > problems would be appreciated.
> 
> TL/DR: it's definitely better than without the patch, but still worse
> than fully reverting hrtimer [1].

Thank you for testing this.

Have you also had the chance to try out Felix's suggestion of completely
disabling coalescing in the driver?  For that you will need to apply the
patch at [0] (the command below may appear to work without the patch but
the timer will continue to be programmed and expire immediately if the
patch is not applied) and then run something like:

  ethtool -C eth0 tx-frame 1 tx-usecs 0

[0] https://lore.kernel.org/all/20230907-stmmac-coaloff-v2-1-38ccfac548b9@axis.com/

With coalescing disabled in the driver, there is also the option of
playing with the generic software IRQ coalescing options available in
NAPI in newer kernels (eg. gro_flush_timeout), which may work better for
you than the one in the driver.