On Tue, May 9, 2023 at 11:34 AM Zhirun Yan <[email protected]> wrote:
>
> Rename rte_graph_work.h to rte_graph_work_common.h for supporting
> multiple graph worker model.
I have requested to check the performance with dpdk-test and l3fwd
graph in last series.
Have you checked the performance? In my testing, there is regression.
Please check the performance with dpdk-test and l3fwd graph, there
should not be any regression in RTC mode.
There is around -300% regression arm64 and x86.
Command to mesure:
./build/app/test/dpdk-test -c 0xf00000 -- graph_perf_autotest
There is around ~-2% regression in l3fwd-graph. I dont think, there
should not be any reason for regression as it is model are separate
header file.
Please check the common header file in fastpath and fix the regression
to accept this series.
./build/examples/dpdk-l3fwd-graph -a 0002:02:00.0 -c 0xc00000 -- -p
0x1 --config="(0, 0, 23)" -P (edited)
Old
+-------------------------------+---------------+---------------+---------------+---------------+---------------+-----------+
|Node |calls |objs
|realloc_count |objs/call |objs/sec(10E6) |cycles/call|
+-------------------------------+---------------+---------------+---------------+---------------+---------------+-----------+
|ip4_lookup |7282757 |1864385584 |1
|256.000 |38.704896 |1770.0000 |
|ip4_rewrite |7282758 |1864385840 |1
|256.000 |38.704896 |1431.0000 |
|ethdev_tx-0 |7282758 |1864385840 |1
|256.000 |38.704896 |922.0000 |
|ethdev_rx-0-0 |14882133 |1864386096 |2
|256.000 |38.704896 |2015.0000 |
|pkt_cls |7282760 |1864386352 |1
|256.000 |38.704896 |392.0000 |
+-------------------------------+---------------+---------------+---------------+---------------+---------------+-----------+
New
+-------------------------------+---------------+---------------+---------------+---------------+---------------+-----------+
|Node |calls |objs
|realloc_count |objs/call |objs/sec(10E6) |cycles/call|
+-------------------------------+---------------+---------------+---------------+---------------+---------------+-----------+
|ip4_lookup |3002135 |768546560 |2
|256.000 |38.402048 |1770.0000 |
|ip4_rewrite |3002136 |768546816 |1
|256.000 |38.402048 |1425.0000 |
|ethdev_tx-0 |3002137 |768547072 |2
|256.000 |38.402048 |949.0000 |
|ethdev_rx-0-0 |3002138 |768547328 |2
|256.000 |38.402048 |1966.0000 |
|pkt_cls |3002138 |768547328 |1
|256.000 |38.402048 |408.0000 |
+-------------------------------+---------------+---------------+---------------+---------------+---------------+-----------+
NAK for this series till the performance issues fixed.
>
> Signed-off-by: Haiyue Wang <[email protected]>
> Signed-off-by: Cunming Liang <[email protected]>
> Signed-off-by: Zhirun Yan <[email protected]>
> ---
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 8df23e5099..cc11328242 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1714,6 +1714,7 @@ F: doc/guides/prog_guide/bpf_lib.rst
> Graph - EXPERIMENTAL
> M: Jerin Jacob <[email protected]>
> M: Kiran Kumar K <[email protected]>
> +M: Zhirun Yan <[email protected]>
Thanks for adding as maintainer.
Since you are at this change. Could you move up "Nithin Dabilpuram
<[email protected]>" two lines below and group all together?
> F: lib/graph/
> F: doc/guides/prog_guide/graph_lib.rst
> F: app/test/test_graph*
