DPDK patches and discussions
 help / color / mirror / Atom feed
* Re: [dpdk-dev] [RFC PATCH 0/1] Add l2reflect measurement application
@ 2020-07-03 16:07 Moessbauer, Felix
  0 siblings, 0 replies; 3+ messages in thread
From: Moessbauer, Felix @ 2020-07-03 16:07 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev

> 03/07/2020 17:18, Felix Moessbauer:
> > Dear Maintainers,
> >
> > this patch adds a measurement application to benchmark the latency of
> > transmitting l2 packets via DPDK.
> 
> Are you using a real time system to reduce latency?

Yes, we use a Debian buster with preempt-rt patched kernel.
Additionally, we use isolcpus for the RT relevant threads.
Apart from achieving low latency, we use this tool to measure the maximum latency over a long period of time.
This gives a good estimate that the system behaves deterministically, even under high load.
The tool also has a built-in mechanism to set trace points when a threshold is hit, to later inspect the kernel logs.

> 
> > We add this application to the examples folder, as similar tools were
> > placed there as well.
> 
> No, the examples are not supposed to be tools.
> They demonstrate some APIs.
> 
> If it is a tool to measure performance, and can be used in a CI, I think it is better
> hosted in the directory apps/.
> 
Ok, got it. There were some issues reported by checkpatch. I'll fix them and move to apps.
Thanks for the hint!


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [RFC PATCH 0/1] Add l2reflect measurement application
  2020-07-03 15:18 Felix Moessbauer
@ 2020-07-03 15:51 ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2020-07-03 15:51 UTC (permalink / raw)
  To: Felix Moessbauer; +Cc: dev

03/07/2020 17:18, Felix Moessbauer:
> Dear Maintainers,
> 
> this patch adds a measurement application to benchmark the latency
> of transmitting l2 packets via DPDK.

Are you using a real time system to reduce latency?

> We add this application to the examples folder, as similar tools
> were placed there as well.

No, the examples are not supposed to be tools.
They demonstrate some APIs.

If it is a tool to measure performance, and can be used in a CI,
I think it is better hosted in the directory apps/.



^ permalink raw reply	[flat|nested] 3+ messages in thread

* [dpdk-dev] [RFC PATCH 0/1] Add l2reflect measurement application
@ 2020-07-03 15:18 Felix Moessbauer
  2020-07-03 15:51 ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Felix Moessbauer @ 2020-07-03 15:18 UTC (permalink / raw)
  To: thomas; +Cc: dev, Felix Moessbauer

Dear Maintainers,

this patch adds a measurement application to benchmark the latency
of transmitting l2 packets via DPDK.

We add this application to the examples folder, as similar tools
were placed there as well.
Note: the example has an optional dependency to cjson for creating a
json representation of the latency diagram.

Sending this as RFC as this patch does not modify libdpdk.

Best regards,
Felix Mößbauer

Felix Moessbauer (1):
  Add l2reflect measurement application

 examples/l2reflect/Makefile    |  31 ++
 examples/l2reflect/l2reflect.h |  56 +++
 examples/l2reflect/main.c      | 833 +++++++++++++++++++++++++++++++++
 examples/l2reflect/meson.build |  21 +
 examples/l2reflect/stats.c     | 198 ++++++++
 examples/l2reflect/stats.h     |  67 +++
 examples/meson.build           |   1 +
 7 files changed, 1207 insertions(+)
 create mode 100644 examples/l2reflect/Makefile
 create mode 100644 examples/l2reflect/l2reflect.h
 create mode 100644 examples/l2reflect/main.c
 create mode 100644 examples/l2reflect/meson.build
 create mode 100644 examples/l2reflect/stats.c
 create mode 100644 examples/l2reflect/stats.h

-- 
2.20.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-07-03 16:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-03 16:07 [dpdk-dev] [RFC PATCH 0/1] Add l2reflect measurement application Moessbauer, Felix
  -- strict thread matches above, loose matches on Subject: below --
2020-07-03 15:18 Felix Moessbauer
2020-07-03 15:51 ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).