feat(linux): Add Ethernet performance numbers for 12.01 release - #774
feat(linux): Add Ethernet performance numbers for 12.01 release#774danish-ti wants to merge 1 commit into
Conversation
|
New warnings found with rstcheck: |
Add netperf Ethernet performance data for AM62X, AM62PX, AM62LX and AM64X in their respective performance guide RST files. Each section covers CPSW TCP bidirectional throughput (with and without interrupt pacing) and UDP egress/ingress 0-loss throughput across 64/128/256/ 1024/1518B frame sizes. AM64X additionally includes ICSSG results. Signed-off-by: MD Danish Anwar <danishanwar@ti.com>
20b807d to
133887e
Compare
| - CPSW3g: AM62x | ||
|
|
||
| TCP Bidirectional Throughput | ||
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ |
There was a problem hiding this comment.
Hm. Don't like this sudden formatting change. What's proposing it?
There was a problem hiding this comment.
This comes from a heading-level shift introduced by cc2de1e feat(linux): AM62X: Add initial performance guide for 12.01.00 release PR #758 , which rewrote the file and dropped the = heading level. In 12.00 performance doc, the Ethernet section used = (level 3) → - (level 4, CPSW) → ^ (level 5, TCP/UDP sub-headings). After cc2de1e, - became level 3 and ^ became level 4. When restoring the Ethernet section, CPSW/ICSSG now sits at ^ (level 4), so the TCP/UDP sub-headings need a new level 5 — hence ~. The character itself is arbitrary; the level shift was caused by the 12.01 template rewrite, not this PR. Level 5 can be either " or ~.
| | | ||
|
|
||
| System Benchmarks | ||
| ------------------------- |
There was a problem hiding this comment.
Oh. This file has sections all out of order.
|
Ugh, alright. I'll look at fixing those good awful templates again. Change the am64 document to match the others for now, assuming that makes the TOC tree happy. Test it locally and make sure it's not trying to put that section above anything else in the hierarchy. |
@StaticRocket If I change the AM64 document to match the others, rstcheck fails with "Title level inconsistent" erros. |
|
The problem with that is, that requires more changes that go explicitly against the section hierarchy outlined in the contribution guide. |
What do you think should be done here then? The current state of PR is 3. |
|
Eh. If the linter is happy for now then it's fine as is. I have a pending PR reworking that template file again. |
Add netperf Ethernet performance data for AM62X, AM62PX, AM62LX and AM64X in their respective performance guide RST files. Each section covers CPSW TCP bidirectional throughput (with and without interrupt pacing) and UDP egress/ingress 0-loss throughput across 64/128/256/ 1024/1518B frame sizes. AM64X additionally includes ICSSG results.