diff options
| author | Sabrina Dubroca <sd@queasysnail.net> | 2024-10-11 18:16:37 +0300 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2024-10-15 19:57:03 +0300 |
| commit | cf58aefb1332db322060cad4a330d5f9292b0f41 (patch) | |
| tree | 2ecc27c085e049e9239f0b783a70a48c01ce027f /tools/perf/scripts/python/export-to-postgresql.py | |
| parent | 637c4f6fe40befa04f19c38b5d15429cbb9191d9 (diff) | |
| download | linux-cf58aefb1332db322060cad4a330d5f9292b0f41.tar.xz | |
macsec: don't increment counters for an unrelated SA
On RX, we shouldn't be incrementing the stats for an arbitrary SA in
case the actual SA hasn't been set up. Those counters are intended to
track packets for their respective AN when the SA isn't currently
configured. Due to the way MACsec is implemented, we don't keep
counters unless the SA is configured, so we can't track those packets,
and those counters will remain at 0.
The RXSC's stats keeps track of those packets without telling us which
AN they belonged to. We could add counters for non-existent SAs, and
then find a way to integrate them in the dump to userspace, but I
don't think it's worth the effort.
Fixes: 91ec9bd57f35 ("macsec: Fix traffic counters/statistics")
Reported-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: Sabrina Dubroca <sd@queasysnail.net>
Link: https://patch.msgid.link/f5ac92aaa5b89343232615f4c03f9f95042c6aa0.1728657709.git.sd@queasysnail.net
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/export-to-postgresql.py')
0 files changed, 0 insertions, 0 deletions