<div dir="ltr"><div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>I do not see the actual results of the DTS perf test in the links you
posted, only percentage of degradation or improvement, unless I miss
something.</div></blockquote><div><br></div><div>This was done on purpose as requested by the Members participating in the effort. This CI is meant for the DPDK maintainers to make sure that a patch does not introduce significant performance regressions on various hardware platforms, or to show how a patch may improve the performance of DPDK on these platforms (such as driver updates or something in the core of DPDK itself). It is not to compare the performance between different devices.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div> I believe it can be helpful if the baseline of the actual results will also be shown <div dir="auto">to enable comparing to other vendors besides Intel and Mellanox.</div></div></blockquote><div><br></div><div>There is not a single baseline for all devices. The baselines are generated per device. Knowing the baseline would allow creating an absolute result, which we are trying to avoid.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Do you plan to add such URL in the report sent to patchwork?<span class="m_1793572375996440537gmail-m_4722669703508491933gmail-im"><br></span></div></blockquote><div dir="ltr"><span class="m_1793572375996440537gmail-m_4722669703508491933gmail-im"><br></span></div><div dir="ltr">Created a ticket to add the URL to the emailed reports: <a href="https://bugs.dpdk.org/show_bug.cgi?id=180" target="_blank">https://bugs.dpdk.org/show_bug.cgi?id=180</a></div><div dir="ltr"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Instead of filtering based on the label, you could filter based on<br>
the paths of modified files.<br>
Note that such filter depends on the test you run,<br>
because you could also test the doc syntax in the CI.</div></blockquote><div><br></div><div>Okay, so if a series only modified the doc folder, then don't include it for performance tests. Later on, we can introduce syntax checking for documentation when we introduce more unit testing / functional testing. <a href="https://bugs.dpdk.org/show_bug.cgi?id=181" target="_blank">https://bugs.dpdk.org/show_bug.cgi?id=181</a><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 4, 2019 at 10:34 AM Thomas Monjalon <<a href="mailto:thomas@monjalon.net" target="_blank">thomas@monjalon.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">04/01/2019 15:44, Jeremy Plsek:<br>
> Hi Rami,<br>
> <br>
> I'm the current maintainer of the DPDK Performance CI. I realize that the<br>
> performance results don't point to the website, so it's not obvious on<br>
> where to find this information. You can find an overview of these tests<br>
> here: <a href="https://lab.dpdk.org" rel="noreferrer" target="_blank">https://lab.dpdk.org</a><br>
> <br>
> Most of this information can be either found on the detailed results of a<br>
> test (such as <a href="https://lab.dpdk.org/results/dashboard/patchsets/4157/" rel="noreferrer" target="_blank">https://lab.dpdk.org/results/dashboard/patchsets/4157/</a>) or on<br>
> the about page (<a href="https://lab.dpdk.org/results/dashboard/about/" rel="noreferrer" target="_blank">https://lab.dpdk.org/results/dashboard/about/</a>).<br>
<br>
Do you plan to add such URL in the report sent to patchwork?<br>
<br>
> We don't apply the doc folder when applying the series, in case a patch<br>
> included code unrelated to documentation. If others in the group feel that<br>
> it's still unnecessary to include "doc" labeled series, I can look into<br>
> filtering them out.<br>
<br>
Instead of filtering based on the label, you could filter based on<br>
the paths of modified files.<br>
Note that such filter depends on the test you run,<br>
because you could also test the doc syntax in the CI.<br>
<br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail-m_1793572375996440537gmail-m_4722669703508491933gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div></div>Jeremy Plsek<br></div><div>UNH InterOperability Laboratory<br></div></div></div></div></div></div></div></div></div></div>