Re: [PATCH v6 2/2] Add l2reflect measurement application

[Thomas Monjalon <thomas@monjalon.net>]

21/09/2022 16:42, Jerin Jacob:
> On Fri, Sep 2, 2022 at 2:16 PM Felix Moessbauer
> <felix.moessbauer@siemens.com> wrote:
> >
> > The l2reflect application implements a ping-pong benchmark to
> > measure the latency between two instances. For communication,
> > we use raw ethernet and send one packet at a time. The timing data
> > is collected locally and min/max/avg values are displayed in a TUI.
> > Finally, a histogram of the latencies is printed which can be
> 
> One highlevel comment,
> 
> IMO, We don't need to add code for capturing in histogram and analyze
> it in C code.
> We can simply use, rte_trace and so that we can visualize with
> standard trace viewers.
> Also add add python based script parse the CTF emitted by trace and find
> min/max/avg values to simply the code.

+1, let's keep it minimal and simple