Re: [PATCH net-next v2] atl1c: add 4 RX/TX queue support for Mikrotik 10/25G NIC

[David Miller]

From: Gatis Peisenieks <gatis@xxxxxxxxxxxx>
Date: Tue, 25 May 2021 11:19:33 +0300

> More RX/TX queues on a network card help spread the CPU load among
> cores and achieve higher overall networking performance. The new
> Mikrotik 10/25G NIC supports 4 RX and 4 TX queues. TX queues are
> treated with equal priority. RX queue balancing is fixed based on
> L2/L3/L4 hash.
> 
> This adds support for 4 RX/TX queues while maintaining backwards
> compatibility with older hardware.
> 
> Simultaneous TX + RX performance on AMD Threadripper 3960X
> with Mikrotik 10/25G NIC improved from 1.6Mpps to 3.2Mpps per port.
> 
> Backwards compatiblitiy was verified with AR8151 and AR8131 based
> NICs.
> 
> Signed-off-by: Gatis Peisenieks <gatis@xxxxxxxxxxxx>
> ---
> v2:
>         - rebase on net-next master as requested by David Miller

Please fix these warnings:
drivers/net/ethernet/atheros/atl1c/atl1c_main.c:891: warning: Function parameter or member 'queue' not described in 'atl1c_clean_tx_ring'
drivers/net/ethernet/atheros/atl1c/atl1c_main.c:891: warning: Excess function parameter 'type' description in 'atl1c_clean_tx_ring'
drivers/net/ethernet/atheros/atl1c/atl1c_main.c:917: warning: Function parameter or member 'queue' not described in 'atl1c_clean_rx_ring'
New warnings added
0a1,3
> drivers/net/ethernet/atheros/atl1c/atl1c_main.c:891: warning: Function parameter or member 'queue' not described in 'atl1c_clean_tx_ring'
> drivers/net/ethernet/atheros/atl1c/atl1c_main.c:891: warning: Excess function parameter 'type' description in 'atl1c_clean_tx_ring'
> drivers/net/ethernet/atheros/atl1c/atl1c_main.c:917: warning: Function parameter or member 'queue' not described in 'atl1c_clean_rx_ring'
Per-file breakdown

Thank you.