Page MenuHomeFreeBSD

dtrace/arm64: properly traverse the symbol table
ClosedPublic

Authored by def on Oct 28 2025, 4:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 2, 12:55 PM
Unknown Object (File)
Mon, Nov 24, 11:41 AM
Unknown Object (File)
Sun, Nov 23, 11:24 AM
Unknown Object (File)
Nov 3 2025, 4:36 PM
Unknown Object (File)
Oct 31 2025, 6:09 AM
Unknown Object (File)
Oct 31 2025, 12:51 AM
Unknown Object (File)
Oct 31 2025, 12:51 AM
Unknown Object (File)
Oct 31 2025, 12:50 AM
Subscribers

Details

Summary

dtrace/arm64: properly traverse the symbol table

LINKER_EACH_FUNCTION_NAMEVAL() stops processing the symbol table if a
callback function returns a non-zero value.

The fbt_provide_module_function() callback should not return 1 when
ignoring symbols. Instead, always return 0, as in dtrace/x86.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

def requested review of this revision.Oct 28 2025, 4:54 PM

It would be nice to add a Fixes tag here.

This revision is now accepted and ready to land.Oct 28 2025, 5:02 PM

Include Fixes in the commit message.

This revision now requires review to proceed.Oct 29 2025, 12:06 AM

It would be nice to add a Fixes tag here.

@markj Does it make sense now?

You need to actually edit the description in Phabricator; arc diff won't do it for you if you just amend the commit locally.

def edited the summary of this revision. (Show Details)EditedOct 29 2025, 7:58 AM

You need to actually edit the description in Phabricator; arc diff won't do it for you if you just amend the commit locally.

When I try to update the review description, I get the error:

The value you have entered in "Summary" can not be parsed unambiguously when rendered in a commit message. Edit the message so that keywords like "Summary:" and "Test Plan:" do not appear at the beginning of lines. Parsed keys: reviewedByPHIDs, revisionID.

I updated it instead not to include any tags from the commit message -- the commit itself is fixed now.

In D53399#1219983, @def wrote:

You need to actually edit the description in Phabricator; arc diff won't do it for you if you just amend the commit locally.

When I try to update the review description, I get the error:

The value you have entered in "Summary" can not be parsed unambiguously when rendered in a commit message. Edit the message so that keywords like "Summary:" and "Test Plan:" do not appear at the beginning of lines. Parsed keys: reviewedByPHIDs, revisionID.

I updated it instead not to include any tags from the commit message -- the commit itself is fixed now.

Hum, ok. Maybe point to the commit on github or similar? If you're confident it's right just go ahead and push.

In D53399#1219983, @def wrote:

You need to actually edit the description in Phabricator; arc diff won't do it for you if you just amend the commit locally.

When I try to update the review description, I get the error:

The value you have entered in "Summary" can not be parsed unambiguously when rendered in a commit message. Edit the message so that keywords like "Summary:" and "Test Plan:" do not appear at the beginning of lines. Parsed keys: reviewedByPHIDs, revisionID.

I updated it instead not to include any tags from the commit message -- the commit itself is fixed now.

Hum, ok. Maybe point to the commit on github or similar? If you're confident it's right just go ahead and push.

If you click the Commits tab in the Revision Contents section in this review, you will see the current commit message. A GitHub page for this commit is:
https://github.com/kwitaszczyk/freebsd-src/commit/374e36799bab1803ae6d0205e06bbc802668c11f

In D53399#1220341, @def wrote:
In D53399#1219983, @def wrote:

You need to actually edit the description in Phabricator; arc diff won't do it for you if you just amend the commit locally.

When I try to update the review description, I get the error:

The value you have entered in "Summary" can not be parsed unambiguously when rendered in a commit message. Edit the message so that keywords like "Summary:" and "Test Plan:" do not appear at the beginning of lines. Parsed keys: reviewedByPHIDs, revisionID.

I updated it instead not to include any tags from the commit message -- the commit itself is fixed now.

Hum, ok. Maybe point to the commit on github or similar? If you're confident it's right just go ahead and push.

If you click the Commits tab in the Revision Contents section in this review, you will see the current commit message. A GitHub page for this commit is:
https://github.com/kwitaszczyk/freebsd-src/commit/374e36799bab1803ae6d0205e06bbc802668c11f

Perfect, thank you.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 29 2025, 5:54 PM
This revision was automatically updated to reflect the committed changes.