Skip to content

Add non-default feature flag 'profiling' to xde#1022

Merged
daniel-levin merged 1 commit into
masterfrom
prof
Jul 3, 2026
Merged

Add non-default feature flag 'profiling' to xde#1022
daniel-levin merged 1 commit into
masterfrom
prof

Conversation

@daniel-levin

@daniel-levin daniel-levin commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

This feature flag is off by default, and can only be enabled by manually editing xde/Cargo.toml. It makes it possible to conditionally disable inlining and name mangling on functions of interest. In particular, it has been applied to xde_rx_one and xde_mc_tx_one. While there are probes that serve as proxies for these functions, it is still valuable to be able to probe them directly while working locally.

$ dtrace -l | grep "xde_.*x_one entry"
fbt               xde                        xde_rx_one entry
fbt               xde                     xde_mc_tx_one entry

Should this workflow become frequently used, it can be made available in the build xtask, but this is a larger project for later, if at all.

This feature flag is off by default, and can only be enabled by manually
editing xde/Cargo.toml. It makes it possible to conditionally disable
inlining and name mangling on functions of interest. In particular, it
has been applied to xde_rx_one and xde_mc_tx_one. While there are probes
that serve as proxies for these functions, it is still valuable to be
able to probe them directly while working locally.

$ dtrace -l | grep "xde_.*x_one entry"
fbt               xde                        xde_rx_one entry
fbt               xde                     xde_mc_tx_one entry

Should this workflow become frequently used, it can be made available
in the build xtask, but this is a larger project for later, if at all.

Signed-off-by: Daniel Levin <daniel.levin@oxidecomputer.com>

@FelixMcFelix FelixMcFelix left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@daniel-levin daniel-levin merged commit 78153e8 into master Jul 3, 2026
10 checks passed
@daniel-levin daniel-levin deleted the prof branch July 3, 2026 15:27
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