Age | Commit message (Collapse) | Author | Files | Lines |
|
Change-Id: Iebd62fd6b0c798f7b4dd1f3b093c156e533b3900
Signed-off-by: Jan Gelety <jgelety@cisco.com>
(cherry picked from commit a5b3a8b91d8e6c9baa4361d70b96769a98bfc454)
|
|
Signed-off-by: pmikus <pmikus@cisco.com>
Change-Id: Ie987d5c6ce43ce925418f61c4b14dced602b4a3b
|
|
Signed-off-by: pmikus <pmikus@cisco.com>
Change-Id: I7d83ba048e0609d6b8623fab5c2960e48a37c023
(cherry picked from commit 3c3930b9f6f9a40d6b30f6e56e8c40279e34f650)
|
|
Signed-off-by: pmikus <pmikus@cisco.com>
Change-Id: Ieff60a44d42d66acee8ba1680e7e285d6cd01bc9
(cherry picked from commit e62ade12b7c9918cda2d363387ed0f517aa9840d)
|
|
+ Reduce time overhead when parsing --include vs --test
+ Input files will remain the same
+ 3n-hsw 150include ~24min, 150test ~5min
+ 2n-clx 489include ~61min, 489test ~9min
Signed-off-by: pmikus <pmikus@cisco.com>
Change-Id: Ia453b1bc1d1862bfc378a7611064a67ee564e2f2
|
|
Signed-off-by: Tibor Frank <tifrank@cisco.com>
Change-Id: Idf4593846f2b241ccaabd38452be49a8a220e54c
|
|
Signed-off-by: Tibor Frank <tifrank@cisco.com>
Change-Id: I00de87ca81d344a153d40a34ca5505316983f076
|
|
Signed-off-by: Tibor Frank <tifrank@cisco.com>
Change-Id: I3dd993fceadce74f7f3280f755231b0c7e32511b
|
|
Signed-off-by: Tibor Frank <tifrank@cisco.com>
Change-Id: If2e200b11f542eae977c64c04b831bc9667c975c
|
|
Signed-off-by: Tibor Frank <tifrank@cisco.com>
Change-Id: I4acd41057a713942604d1a167f54af7e03ab83fc
|
|
Attempt to unbind a driver from a device only if it is bound to a
driver.
Remove the dynamic addition of an existing device ID to a driver. From
the docs [0]:
Writing a device ID to this file will attempt to
dynamically add a new device ID to a PCI device driver.
Since we assume the VFs are bound to the kernel driver when VPP Device
topology creation is done, it implies that the kernel driver supports
the device ID of those VFs, removing the need to add the support.
[0]: https://www.kernel.org/doc/Documentation/ABI/testing/sysfs-bus-pci
Change-Id: I20f3ca071a5a84a06ff358ba514532248a8f9ad0
Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
(cherry picked from commit 71d7150a65a7c006bf46b2c1001dbaa00b5681fb)
|
|
Change-Id: Ia8e71df86d561c1d9bc388fa6b70ef850c94e66e
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit d719a272815348bd9229c5cd280ecf39e862870b)
|
|
This reverts commit 2f4a2fd01579816b7cccb1d640642fba03d96a30.
Reason for revert: Publish 2n-zn2 data
Change-Id: If606d15035abb917d7ad0287086a8219cefb2882
Signed-off-by: Tibor Frank <tifrank@cisco.com>
|
|
This is a follow-up to https://gerrit.fd.io/r/c/csit/+/20394
https://gerrit.fd.io/r/20119
has changed the way archival works,
everything should now go to logs.fd.io
(instead of appearing on run page in jenkins.fd.io).
The glob pattern for archiving is quite eager,
doing recursive search. That is good, as it can find
also misplaced useful outputs.
But it also means our usage of copy_archives function
creates two copies of archived directtories,
usually archives/ and archives/archive/.
This change renames copy_archives to move_archives,
with few workarounds to support multiple calls.
I also renamed ARCHIVE_DIR value from $CSIT_DIR/archive
to $CSIT_DIR/archives to make "move" operation look natural.
Finally, download_builds function is being removed,
as after recent improvements to VPP compilation speed
nobody seems to be using it.
Change-Id: I19a429e1dfdfaab7fcf32a9609963b1aebd33c6c
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
(cherry picked from commit 523c6e6e24101206ff1318ca17c310dff8b3c9d2)
|
|
Signed-off-by: Tibor Frank <tifrank@cisco.com>
Change-Id: I543fa0c23124b8ce98824a74d6a704270bf4a3e8
|
|
+ Move "|| true" to a place that really aviods errors.
+ Attemp to parse decoded string if trigger is not found in plain one.
Change-Id: If3587229ec588f9ad41acb3050add1142032d2d8
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
(cherry picked from commit 4582f0f408616cdff8e606ac3abfe154f8f0514b)
(cherry picked from commit f89bc7a87e0b6015e50de3557a1724c8aaafbf60)
|
|
Change-Id: Ib095788b071c74131351045df80ab373f067f1a7
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit ff4af59376eb2dead3eeeef75620272604840349)
|
|
Change-Id: I38f4054cddf01557e8482adcdc804760ef7a9c4b
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit a5c9ea2d653a7c3cb7bb55256bff8b1c6130fa33)
|
|
Change-Id: If42cacf6c643d35aaa7f65d8e2f87972d23d251b
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 637e9b9ade08d368cce5a239798f50113912207f)
|
|
- packet throughput
- speed-up
Change-Id: I5d49ae2da54170e48c51db47e035d2121471be0d
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 52e3b0f6b6b4f92c5ee5ecf003623cb59c798290)
|
|
This reverts commit 7d6ee11472e8ab675248616fbc90005eaf73128b.
Reason for revert: publish 2n-zn2 results
Change-Id: I3c5716af92a014efa0901244260099b9f1cf1af9
Signed-off-by: Tibor Frank <tifrank@cisco.com>
|
|
Change-Id: I1708ebd24a6a26db53a14192b23fe51569543333
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 610e96a2afae5c8e58bee00d05c45d57d7bb6021)
|
|
Change-Id: I26587c8b987be9c7b6fb3a1d79d77a4e1af417f8
Signed-off-by: Tibor Frank <tifrank@cisco.com>
|
|
- 2n-zn2 iter
Change-Id: I7412963c918b53e44294f4cf213a806c8435e3b5
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 4aa4a9d82e5eb41823b56bda56a0d34d9a3b26b1)
|
|
Change-Id: Id4738b823a77e59d5efd913f770326726b8a81e2
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit b6340be1caf13025b2aaac06ccf6ad3bb64d593f)
|
|
Change-Id: I4e4d156dcb9d93bf0c537a870e6fab6c500f5810
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 379e58a740271d84b15059f7059661fbec13cef8)
|
|
Change-Id: Ib7c4bc51bea43dafa2bd788c709c800caeb75988
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit ef51e552aa2d0e134646d28aa3ada2da0d467f99)
|
|
Change-Id: Ifaba0d689a279ae650ae30531b21533cc6927262
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit f8b3d745368deb92af2b7ff70874e3b0ae80a0c7)
|
|
Change-Id: I65cf505edd346fc5da240be08310064b3755e458
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 4d6dcbbbb9103f15fb55d525f14964c2323c2a03)
|
|
Change-Id: I7efb7a339e248972604b1c8027d21ea32e9f4070
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 3ef4eb87e921a350361a54b0449b6536c49c5794)
|
|
Change-Id: I73a735b3e82aaa01f56c21bf91e5073f144c584f
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit dbae94b3db123a5e86b0a5afb5f5baaa262ce8ad)
|
|
- Better to get it fully aligned then cherry-pick spaghetti.
Signed-off-by: pmikus <pmikus@cisco.com>
Change-Id: If223ef3f0247413d53225eb57f8903a7675632e3
|
|
Change-Id: I3aaf06de4ae5fb2dbeef5920987bc427bf0bb529
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 6b33c89d362623dd46f3a14c497835de6f669101)
|
|
Change-Id: I2027c8e0be76c119d88c966f700f9639ed4c1784
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 68d130f9405ed239e624f9dd016381b70ee57001)
|
|
Change-Id: Ic2d8c66ec44b89354e4dbf4f9605c8e53bf21481
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit f4621bff99ef04d87dad83a752f55e16423324e0)
|
|
- reorder NAT44 graphs
Change-Id: Ied762a85fb1ee69c7d12971e3e15822571a18451
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 6328731a2778c864c6372296449a55dae79cd305)
|
|
The previous code counted full TCP connections,
which need one more packet, leading to worse results.
Change-Id: Ifcf78356b6ed54819ea0bf5aa069d7d9cb951183
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
(cherry picked from commit b71112bc323b55e39d8a9992a46530e1eb7f6f58)
|
|
Change-Id: Id59947b8c502e4bf7b7ce66ad326f481346c98d6
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit c70bc9fce0954dce4353d3f99d9fc7d1a385583f)
|
|
Change-Id: Ic9342f0f3deea3d7d175c437993289c86464bb99
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 494f7e4abf409894ccb3652ded0cb0eedeabf85a)
|
|
Change-Id: I2c247c5362396bcb4c8b42b1b46a2601091ca968
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 4e056959f9e5a01f1c1cc6816d50609a4aa42366)
|
|
Change-Id: I81c3b43407921ed7912b530bb93c725515d3d583
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 44186cb43aaa4131f676d9d18f8f573823ec9512)
|
|
Change-Id: I7886285de8edbf66e03e1b22d85c2e7a4c4d7224
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 20f419f950606f1848a81d815c251f8180867b50)
|
|
Change-Id: I6de094fc0cb2acdfb6eeb22e6d85a65d2e9f1478
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit a1a6ce5505840750a34d86ada800367a183cf74b)
|
|
Change-Id: I800273d04c3f956e58be727cbf5befa61377fa2b
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 09f0b96e5484a24e11415ef7839dbf8f1dcfd9bb)
|
|
Change-Id: I99836894960f74f9dc283b812e1106ac512db3ae
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 380ed1f7a22d960ecf4b6a34c15cf3c69a32b800)
|
|
Change-Id: I1cf61c523d781db3dbeccf89077c50992cb03205
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit 41cdf812ff618809e6181a561afbf86b7fae67f0)
|
|
Change-Id: I463bf61d41484738ba5c614b473bcb5457e6abe1
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit d500d77f542b718b0bd7c1dd5e20e70b9f196329)
|
|
Change-Id: I3920f881286565029f35bcfa700f36c6701db457
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit b0791718d2a3cb7a6c902496f721ecb817013aa5)
|
|
Change-Id: Iaa85aaa5179136281ab05325efcf0e91c234a774
Signed-off-by: Tibor Frank <tifrank@cisco.com>
(cherry picked from commit e2e037d5965dabf8b274ed0fe67fb89d5d324d0b)
|
|
Manual cherry-pick from master [1],
reverting impact of [2] and [3].
[1] https://gerrit.fd.io/r/c/csit/+/28208/176
[2] https://gerrit.fd.io/r/c/csit/+/29077
[3] https://gerrit.fd.io/r/c/csit/+/29529
The heap multipliers are left in suites,
as that simplifies cherry-picking between branches.
Original [0] commit message:
Support existing test types with ASTF
+ Add UDP_CPS, TCP_CPS, UDP_PPS and TCP_PPS suites.
+ Update existing cps traffic profiles.
+ Add missing traffic profiles.
+ UDP:
+ Single burst of 32 packets was confirmed as safe enough for TRex.
+ Maybe 64 could work, but not enough testing for that.
+ Multiple bursts have lead to reduced TRex performance,
as overlaping bursts (from different client instances)
tend to fill up the buffers.
+ TCP:
+ Data size set to 11111 bytes, completely arbitrarily.
+ Results look reasonable, so I have kept that.
- MSS not set at all
- No tested support for frame size other than 64B.
- Frame size does not even factor into TCP profiles.
+ So other frame sizes are skipped in autogen.
+ Update tags in related suites.
- HOSTS_{n} and SRC_USER_{n} should be unified.
- Questionable clarification on difference between IP4BASE and SCALE.
+ Add NAT state resetters to tests that need them.
+ Resetter is called (if set) before each measurement.
+ If ramp-up is detected, resetter is not set.
+ Rename "mult" argument to "multiplier".
+ Abstracted from packets to transactions.
+ Transaction corresponds to profile.
+ TRex multiplier argument sets target rate in transactions per second.
+ The familiar STL traffic:
+ Bidirectional is considered to be 2 packets per transaction.
+ Unidirectional is considered to be 1 packet per transaction.
+ The newer ASTF traffic:
+ 4 subtypes, each has different number of packets per transaction.
+ For max rate computation:
+ Packets in the more numerous direction are considered.
+ Rely on TRex reported traffic duration for ASTF:
+ Use the server side value.
- Client side value is higher by an overhead.
- TRex is not sending traffic during that time.
+ Remove delays from traffic profiles.
- Those delays would increase the reprted traffic time.
+ Support for scale lmited trials.
+ Only for ASTF profiles, each ASTF profile has limited scale.
+ Scale defined in suite variables.
+ For TRex to send all transactions provided duration value is ignored.
+ The appropriate value is computed in TrafficGenerator.
+ An ad-hoc time constant is added to match the TRex client side time overhead.
+ The profile driver receives the computed duration.
+ Measurement for PLRsearch add a sleep if the computed duration is smaller.
+ Alternative argument for search algos if scale is limited.
+ Both need higher timeout to accomodate big scales.
+ MLRsearch can afford fewer phases.
+ Added a parameter to optionally shorten the duration.
+ Use short duration for runtime stats trial and failure stats trial.
+ Use very large keepalive values in udp profiles to avoid ka packets.
+ No polling in ASTF profile driver.
- Polling could eliminate the time overhead value.
+ But polling proved to introduce some loss, affecting the results.
+ Handle duration stretching in ASTF by stopping traffic.
+ The stop has several steps so that:
+ The traffic is really stopped entirely.
+ Late packets do not count (maybe as errors).
+ Stats are preserved to read for results (and cleared afterwards).
+ Several quantities added to ReceiveRateMeasurement:
+ Original target duration is preserved (algos need that).
+ Input estimate (tps) for early search iterations.
+ Output estimate (maybe pps) for MRR output.
+ Strict result (unsent counts as loss) for NDR.
+ Use L2 counters (opackets, ipackets) where possible.
- TRex has trouble processing packets for the L7 ones at high loads.
+ Remove warmup from profile drivers and keywords.
+ Suites should call "Send ramp-up traffic" explicitly if needed.
+ Added parsing for few more counters.
+ Both to use in formulas or just for debug purposes.
- Only 64B cases in autogen, framesize support to be added later.
+ Latency streams during search can be enabled via PERF_USE_LATENCY env var.
+ MLRsearch improvments:
+ Rename argument names to min_rate and max_rate.
+ Use relative receive rate in initial phase.
+ PLRsearch improvements:
+ Careful computation when output (pps) does not match input (tps).
+ Use geometric distribution (instead of Poisson).
+ Helps agains math errors.
+ This should improve estimate stability.
- But in practice big losses still lead to significant jumps.
+ Traffic generator improvements:
+ send_traffic_on_tg now calls the full set_rate_provider_defaults.
+ _send_traffic_on_tg_internal for the logic without provider defaults.
+ As the internal function is re-used by measure() without affecting defaults.
+ Move _parse_traffic_results just before get_measurement_result.
+ As the latter uses fields set bu the former, it is now easier to read.
+ Multiple sources for approximate duration.
+ Tried from more precise to more available.
+ Includes logic for _pps tests (added in later change).
+ Move explicit type conversions to earlier occurences.
+ Profile driver output field uses semicolons to simplify parsing.
+ Performance Robot lib file split to several smaller ones.
+ performance_actions.robot:
+ Hosts Additional Statistics Action For * keywords.
+ performance_display.robot:
+ Hosts keyword for displaying and verifying results.
+ Change test message to use the correct unit (pps or cps).
+ performance_limits.robot renamed to performance_vars.robot
+ Added many keywords, mostly for accessing test variables.
+ Moved variables for Policer into a new keyword there.
+ Some keywords need sophisticated logic.
- Other are basically Get Variable Value.
+ But in future more logic can be added, without editing callers.
+ Documentation for the new keywords acts as a documentation for test variables.
+ performance_utils.robot has the rest.
+ Eliminated arguments if the value is in test variable.
+ Small improvements to documentation.
- Still not enough cleanup with respect to arguments and test variables.
+ Keywords are sorted alphabetically now in each one.
+ Suites:
+ Unified variables table:
+ No colons in comments.
+ ${n_hosts}, ${n_ports} and use them instead hardcoded numbers.
+ Add -cps to existing cps suite names.
+ Remove "trial data overwrite".
+ Compute max rate as in STL suites.
+ Each NAT suite has ip4base suite to compare results to.
- Those act as indirect TRex calibration.
- VPP does not lose packets in those.
+ Latency in ASTF suites is disabled hard.
- As we do not support latency in ASTF profiles yet.
+ Unidirectional tests governed by suite variable, not an argument.
+ Write long argument lists vertically.
+ Prefer to use argument names.
+ In Python, also the last argument is followed by comma.
+ It makes renaming and reordering easier.
+ Similarly applies to prints with long lists of values.
+ A TODO to update api crc file comments.
Change-Id: I84729355edbec051298a9de1162107f88ff5737d
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
|