Test: Refactored performance tests on rxtxapp#1306
Merged
Conversation
awilczyns
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
from
99ea65a to
e55efd6
Compare
awilczyns
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
from
b89972a to
33d8e2d
Compare
awilczyns
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
3 times, most recently
from
f569077 to
cfbf02c
Compare
awilczyns
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
8 times, most recently
from
0aa748d to
b2528de
Compare
awilczyns
requested review from
DawidWesierski4,
Falron98 and
Sakoram
as code owners
awilczyns
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
6 times, most recently
from
ad8c956 to
926fae6
Compare
DawidWesierski4
requested changes
Feb 24, 2026
Collaborator
DawidWesierski4
left a comment
DawidWesierski4
left a comment
There was a problem hiding this comment.
i geniully think that the most importat would be the ramdisk fixture workaround fixes
and the genconfig -> i would try to simplify it, we don't wnat to add anything there
we should propably also add copyright everywhere i see a lot of these are missing
awilczyns
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
4 times, most recently
from
82e0fd9 to
fbb0ed7
Compare
awilczyns
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
from
fbb0ed7 to
449ce07
Compare
DawidWesierski4
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
from
449ce07 to
ff09d39
Compare
DawidWesierski4
approved these changes
Mar 2, 2026
awilczyns
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
5 times, most recently
from
6532b8f to
f3502e8
Compare
…approach. Tests are prepared for test performance for multiple session on single lcore. Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
Multi-session performance tests are now available for 25, 30, 50, and 59 FPS with and without DSA usage. Currently, only FPS metrics are measured. Additional KPIs will be enhanced in the future. Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
…s rx redundant single core Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
…port generation Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
awilczyns
force-pushed
the
refactored-performance-tests-on-rxtxapp
branch
from
f3502e8 to
3cced23
Compare
- Add _get_active_vf_pf_indices() to track which PFs host active VFs - Rewrite _select_sniff_interface() with topology-aware heuristic: single-host captures on VF-hosting PF (loopback visibility), multi-host prefers PF without VFs (avoids VFIO contention) - Add 1s delay after VF creation in nicctl for VFIO binding stabilization - Handle empty PCAP streams gracefully in compliance teardown - Skip redundant VF creation only when capture needs the 2nd PF (multi-host) Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
DawidWesierski4
approved these changes
Mar 3, 2026
Collaborator
DawidWesierski4
left a comment
DawidWesierski4
left a comment
There was a problem hiding this comment.
reaprove after force push
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Introduces a dual-host session capacity sweep framework for MTL ST2110-20 performance testing. Uses binary search to find the maximum number of video sessions that maintain target FPS on a single lcore (or multi-core), across resolutions (1080p/4K/8K), framerates (25/29/50/59fps), with optional ST2022-7 redundancy and DSA offloading.
Key Modifications
• conftest.py — Major expansion: topology fixture override for extra fields (dsa_device, build_path, per-host mtl_path), nic_port_list now sets up redundant VFs on 2nd NIC, new collect_platform_config autouse fixture, log_session routes perf tests to dedicated log folder, auto-base_performance marker for 1080p/59fps tests
• rxtxapp.py — Config generation refactored for ST2022-7 redundant mode (dual interfaces, dual IP arrays), queue count overrides (rx_queues_cnt/tx_queues_cnt), netsniff capture_time fix
• application_base.py — start_process changed to background=True (was blocking), get_executable_path handles direct paths
• nicctl.py — Per-host MTL paths, new reset_vfio_bindings() (crash recovery) and ensure_vfio_bound() (inter-iteration cleanup)
• gen_config.py — New --media_path and --dsa_device CLI args, extra_info field in topology config
• execute.py — New helpers: kill_all_rxtxapp(), stop_remote_process(), read_remote_log()
• universal_params.py — New params: redundant, nic_port_r, source_ip_r, destination_ip_r, rx_queues_cnt, tx_queues_cnt, disable_migrate, tx_copy_once
• ip_pools.py — New tx_r/rx_r redundant IP address pools
pytest.ini — Registered performance and base_performance markers