Page MenuHomeFreeBSD

libdtrace: get rid of illumos ifdefs in dt_module_update(), fix dm_file and dm_modid
ClosedPublic

Authored by christos on Mar 20 2023, 10:38 PM.
Tags
Referenced Files
F103007157: D39177.diff
Tue, Nov 19, 6:20 PM
F103000117: D39177.id119168.diff
Tue, Nov 19, 4:22 PM
Unknown Object (File)
Wed, Nov 13, 6:48 PM
Unknown Object (File)
Mon, Nov 11, 11:43 AM
Unknown Object (File)
Thu, Nov 7, 2:13 AM
Unknown Object (File)
Wed, Nov 6, 9:35 PM
Unknown Object (File)
Tue, Nov 5, 1:53 PM
Unknown Object (File)
Tue, Nov 5, 11:10 AM
Subscribers

Diff Detail

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

Event Timeline

christos retitled this revision from libdtrace: fix module file name to libdtrace: get rid of illumos ifdefs in dt_module_update().
christos edited the summary of this revision. (Show Details)
christos retitled this revision from libdtrace: get rid of illumos ifdefs in dt_module_update() to libdtrace: get rid of illumos ifdefs in dt_module_update(), fix dm_file and dm_modid.Mar 20 2023, 10:58 PM

Approved, but the log message should explain the motivation in some detail: this function is highly OS-specific, and the ifdefs make it hard to read and follow what's going on. The removal of ".filename" section handling in particular should be mentioned.

This revision is now accepted and ready to land.May 22 2023, 8:05 PM