Skip to content

Make AIE dtrace plugin standalone for bandwidth metrics.#82

Merged
jvillarre merged 4 commits into
Xilinx:masterfrom
garimadhaked:standalone_dtrace_plugin
Jun 17, 2026
Merged

Make AIE dtrace plugin standalone for bandwidth metrics.#82
jvillarre merged 4 commits into
Xilinx:masterfrom
garimadhaked:standalone_dtrace_plugin

Conversation

@garimadhaked

Copy link
Copy Markdown
Collaborator

Add AieDtraceMetadata to parse AIE_dtrace_settings and Debug.profiling_runtime_config without aie_profile. Wire the plugin through AieDtraceMetadata and AieDtraceImpl, limit VE2 to self-contained bandwidth CT generation, and remove dtrace logic from AieProfileMetadata.

Add AieDtraceMetadata to parse AIE_dtrace_settings and
Debug.profiling_runtime_config without aie_profile. Wire the plugin
through AieDtraceMetadata and AieDtraceImpl, limit VE2 to self-contained
bandwidth CT generation, and remove dtrace logic from AieProfileMetadata.

Signed-off-by: Garima Dhaked <garima.dhaked@amd.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
garimadhaked and others added 3 commits June 16, 2026 02:53
Dtrace does not run a counter poll loop; drop threadCtrl/thread and use
empty default virtuals instead of VE2 no-op overrides.

Signed-off-by: Garima Dhaked <garima.dhaked@amd.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Align SPDX copyright lines across the plugin and use the short SPDX
header for callback sources.

Signed-off-by: Garima Dhaked <garima.dhaked@amd.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Resolve merge conflict by keeping aie_dtrace_impl.h from
standalone_dtrace_plugin and the new includes from upstream master.
Remove unnecessary aie_profile includes that are not used by the
dtrace VE2 implementation.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jvillarre jvillarre merged commit 5d44f34 into Xilinx:master Jun 17, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants