On Thu, Jan 26, 2023 at 06:21:34PM +0100, Eelco Chaudron wrote:
> 
> 
> On 26 Jan 2023, at 17:58, Simon Horman wrote:
> 
> > On Tue, Jan 24, 2023 at 02:01:19PM +0100, Eelco Chaudron wrote:
> >> With some datapaths, read TC, it takes a bit longer to update the
> >> OpenFlow statistics. Rather than adding an additional delay, try
> >> to read the counters multiple times until we get the desired value.
> >>
> >> Signed-off-by: Eelco Chaudron <echau...@redhat.com>
> >> Acked-by: Roi Dayan <r...@nvidia.com>
> >
> > When, possibly unwisely, running this test on my laptop,
> > with a somewhat old kernel the tests fail with absurd byte counts.
> > I haven't had time to dig into why.
> 
> I noticed this happens sometimes, and I excluded this test from the total set 
> in the last patch :)
> 
> However, I also fixed some counter issues recently which fixed it in my setup:
> 
> https://patchwork.ozlabs.org/project/openvswitch/patch/167361458369.1145547.15786262052264455216.stgit@ebuild/
> 
> Can you try this?

Yes. I made a quick attempt just now and it does change things.

n_bytes now seems to be 0 for both cases (closer to but still not 138).
Perhaps that is explained by a timing issue.

I'll see if I can poke a little more.
But likely not today.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to