Expand ktr_mask to be a 64-bit unsigned integer.
ClosedPublic
Actions

Authored by jhb on May 19 2015, 9:23 PM.

Details

Reviewers

emaste
neel
rpaulo

Commits

rS283283: Expand ktr_mask to be a 64-bit unsigned integer.

Summary

Expand ktr_mask to be a 64-bit unsigned integer.

The mask does not really need to be updated with atomic operations and
the downside of losing races during transitions is not great (it is
not marked volatile, so those races are pretty wide open as it is).

Test Plan

Tested some ktr tracing (albeit still using a 32-bit mask) and it still worked fine.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb updated this revision to Diff 5502.May 19 2015, 9:23 PM

jhb retitled this revision from to Expand ktr_mask to be a 64-bit unsigned integer..

jhb updated this object.

jhb edited the test plan for this revision. (Show Details)

Herald added a subscriber: imp. · View Herald TranscriptMay 19 2015, 9:23 PM

neel accepted this revision.May 19 2015, 9:42 PM

neel added a reviewer: neel.

This revision is now accepted and ready to land.May 19 2015, 9:42 PM

op added a subscriber: op.May 19 2015, 10:15 PM

I like this change, but I missing one small thing: the conversion of KTR_foo classes from 32 bit defines to 64 bit defines in sys/ktr_class.h

op added a comment.May 19 2015, 10:25 PM

This comment was removed by op.

rpaulo accepted this revision.May 20 2015, 6:25 AM

rpaulo added a reviewer: rpaulo.

emaste accepted this revision.May 20 2015, 5:29 PM

emaste added a reviewer: emaste.

Strictly speaking I don't think we need to convert the KTR class macros just yet. Also, we don't need to bump KTR_VERSION since the layout of the logged records hasn't changed.

Closed by commit rS283283: Expand ktr_mask to be a 64-bit unsigned integer. (authored by jhb). · Explain WhyMay 22 2015, 11:09 AM

This revision was automatically updated to reflect the committed changes.