Page MenuHomeFreeBSD

Fixup compilation and enable dtrace support for mips64

Authored by lidl on Sep 4 2017, 10:41 PM.



As it stands right now, one cannot compile the dtrace
modules for mips64. With this patch, the modules
compile, and can be loaded into the kernel via
kldload (or just by running 'dtrace -l').

While the dtrace functionality does not currently work,
this is a useful stepping stone to further explore getting
it to work.

Diff Detail

rS FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

lidl created this revision.Sep 4 2017, 10:41 PM
Herald added 1 blocking reviewer(s): gnn. · View Herald TranscriptSep 4 2017, 10:41 PM
Herald added a subscriber: imp. · View Herald Transcript
emaste added inline comments.Sep 4 2017, 10:45 PM
73 ↗(On Diff #32650)

Does __mips64__ actually get defined? AFAIK __mips__ is on both 32- and 64-bit so it wouldn't make a difference.

lidl added inline comments.Sep 4 2017, 10:50 PM
73 ↗(On Diff #32650)

Heh. You are correct, gcc (4.2.1, the in-tree one) only defines
mips64 not mips64__.

I will remove the mips64 code.

lidl updated this revision to Diff 32653.Sep 4 2017, 10:56 PM

Address emaste's comment about mips64

lidl marked 2 inline comments as done.Sep 4 2017, 10:56 PM
markj accepted this revision.Sep 5 2017, 1:39 AM
gnn accepted this revision.Sep 6 2017, 12:17 AM


This revision is now accepted and ready to land.Sep 6 2017, 12:17 AM
This revision was automatically updated to reflect the committed changes.