diff options
author | Tianyu Yuan <tianyu.yuan@corigine.com> | 2022-08-10 13:47:44 +0800 |
---|---|---|
committer | Simon Horman <simon.horman@corigine.com> | 2022-09-12 18:54:33 +0200 |
commit | dd9881ed55e6bf98b77a8c2347ec6001b0bee3f8 (patch) | |
tree | b743ae8e233c628135a224bfe3a42edbc3829f56 /tests | |
parent | 108a48d6a923025eb2f7649939392fc417bae2f7 (diff) | |
download | openvswitch-dd9881ed55e6bf98b77a8c2347ec6001b0bee3f8.tar.gz |
tc: Fix stats dump when using same meter table
When we apply meter police on both directions of TCP traffic, the
dumped stats is shown same (as shown below). This issue is introduced
by modifying the stats update strategy.
...,in_port(6),eth(),eth_type(0x0800),ipv4(frag=no), packets:1488557,
bytes:2089059644, used:0.040s, actions:meter(0),9
...,in_port(9),eth(),eth_type(0x0800),ipv4(frag=no), packets:1488557,
bytes:2089059644, used:0.040s, actions:meter(0),6
In previous patch, after parsing police action, the flower stats will
be updated by dumped meter table stats, which will result in the issue
above.
Thus, the stats of meter table should not be used when dumping flow
stats. Ignore the stats update when police.index belongs to meter.
Fixes: a9b8cdde69de ("tc: Add support parsing tc police action")
Signed-off-by: Tianyu Yuan <tianyu.yuan@corigine.com>
Reviewed-by: Baowen Zheng <baowen.zheng@corigine.com>
Signed-off-by: Simon Horman <simon.horman@corigine.com>
Acked-by: Eelco Chaudron <echaudro@redhat.com>
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions