[ovs-dev] [PATCH] Avoid indeterminate statistics in offload implementations.
Ben Pfaff
blp at ovn.org
Thu Oct 31 17:07:53 UTC 2019
On Wed, Oct 30, 2019 at 08:40:22PM +0100, Ilya Maximets wrote:
> On 25.10.2019 20:46, Ben Pfaff wrote:
> > A lot of the offload implementations didn't bother to initialize the
> > statistics they were supposed to return. I don't know whether any of
> > the callers actually use them, but it looked wrong.
> >
> > Found by inspection.
> >
> > Signed-off-by: Ben Pfaff <blp at ovn.org>
> > ---
> > lib/netdev-dummy.c | 10 ++++++++--
> > lib/netdev-offload-dpdk.c | 10 ++++++++--
> > lib/netdev-offload-tc.c | 5 ++++-
> > 3 files changed, 20 insertions(+), 5 deletions(-)
>
> This looks correct.
> 'dummy' and 'dpdk' implementations doesn't support stats at all, so
> these should be cleared. 'tc' needs some more work to return stats
> on flow replacing, but this should be a separate change.
>
> Acked-by: Ilya Maximets <i.maximets at ovn.org>
Thanks, applied to master and branch-2.12.
More information about the dev
mailing list