I225 6 first steps - #1677
Open
DawidWesierski4 wants to merge 16 commits into
Open
Conversation
DawidWesierski4
requested review from
Sakoram,
awilczyns,
moleksy and
soopel
as code owners
August 11, 2026 13:42
Document the supported I226-V build path and provide a helper script for the required DPDK configuration. Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Make the helper fail clearly on invalid environments and document the IGC-specific build requirements. Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
IGC needs timesync enabled after queue setup to reserve its RX timestamp prefix. Port start resets the timestamp registers, so restore timesync afterward. Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Expose the remaining MTL device port and source-IP pairs without changing the existing primary and redundant options. Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Expose built-in MTL PTP controls through the libavdevice plugin. Select PTP pacing when the service is enabled. Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Compact sparse FFmpeg ports and validate shared PTP options. Remove duplicate logging and protect dirty DPDK source trees. Fail IGC timesync safely and preserve port cleanup state. Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
DawidWesierski4
force-pushed
the
i225_6_first_steps
branch
from
August 11, 2026 14:27
2141c69 to
ded09c1
Compare
Exercise IGC timesync startup failures and FFmpeg multi-port and PTP translation through hardware-free unit seams. Repair the PTP AVOption macro so the tested options remain part of the TX option list. Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
This was referenced Aug 11, 2026
Sakoram
reviewed
Aug 12, 2026
|
|
||
| if (args->ptp_enable) { | ||
| p.flags |= MTL_FLAG_PTP_ENABLE; | ||
| p.pacing = ST21_TX_PACING_WAY_PTP; |
Collaborator
|
so we want to enable timesync for i225 or not? you remove enablement in some places to add it in different. I don't understand |
DawidWesierski4
force-pushed
the
i225_6_first_steps
branch
2 times, most recently
from
August 13, 2026 09:58
15b5e9d to
7f79e98
Compare
Add a dedicated opt-in ptp_pacing universal param and have FFmpeg read it and enable_ptp directly, instead of inferring pacing from pacing_way. Map the universal ptp pacing mode to FFmpeg's PTP client and pacing options; keep FFmpeg out of unsupported pacing cases. Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Select ICE or IGC through one build_drivers.sh entry point. Reuse the ICE build-only mode in CI instead of duplicating its download logic. Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
DawidWesierski4
force-pushed
the
i225_6_first_steps
branch
from
August 13, 2026 10:46
7f79e98 to
823c307
Compare
Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
Change CICD_BUILD purpose to be more user-facing. It now downloads all dependencies and runs in an ephemeral, non-interactive mode designed to work in Docker environments. Move Ubuntu-specific environment variables into Ubuntu setup. Fail immediately if the Ice driver kernel headers cannot be installed, rather than prompting for the generic headers package as a fallback. There is some instability in noctx tests that makes the library unable. 3 Tests cases needs fixing still. failed cases: st20p_user_pacing st20p_user_pacing_offset_jitter st20p_exact_user_pacing Signed-off-by: Dawid Wesierski <dawid.wesierski@intel.com>
DawidWesierski4
force-pushed
the
i225_6_first_steps
branch
from
August 13, 2026 15:32
81774f3 to
e9341e5
Compare
Signed-off-by: Wesierski, Dawid <dawid.wesierski@intel.com>
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
4 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.
No description provided.