[PATCH 2/3] ethdev: fix memory leak when telemetry xstats
David Marchand
david.marchand at redhat.com
Fri Apr 22 10:14:38 CEST 2022
On Thu, Apr 21, 2022 at 11:04 AM Bruce Richardson
<bruce.richardson at intel.com> wrote:
> > We need some minimal testing for telemetry commands.
> >
> > It could be a test automatically calling all available /ethdev/
> > commands on a running testpmd.
> > This test could be really simple, not even checking what is returned.
> > It would just try every command sequentially with no parameter first,
> > then with port 0 and finally with port 1.
> >
>
> That seems reasonable. However, I'd go a little further and have all
> available commands called as an initial sanity check. Then we can use some
> heuristics to go further, with the *dev/stats commands or xstats commands
> all being called with numeric parameters as you suggest.
Ok, lgtm too.
Just to be clear, I don't have the time to work on this, so this is
open to volunteers.
--
David Marchand
More information about the dev
mailing list