Re: [PATCH net-next 5/6] sfc: ef10: implement ethtool::get_fec_stats

On 14/04/2021 04:44, Jakub Kicinski wrote:

Report what appears to be the standard block counts:
 - 30.5.1.1.17 aFECCorrectedBlocks
 - 30.5.1.1.18 aFECUncorrectableBlocks

Don't report the per-lane symbol counts, if those really
count symbols they are not what the standard calls for
(even if symbols seem like the most useful thing to count.)

Fingers crossed that fec_corrected_errors is not in symbols.

Signed-off-by: Jakub Kicinski <kuba@kernel.org>

I'm trying to find out whether these count the right thing or not.
Some component documentation says that the per-lane _signal_ "asserts at
 maximum once per FEC coding block".  I haven't yet been able to track
 down how the counters aggregate that, but it would seem to suggest that
 they in fact count blocks and not symbols, and are just misnamed.
Investigation continues.

-ed