Page MenuHomeFreeBSD
Differential D55307

stddef.h: centralize definition of offsetof()
ClosedPublic
Actions

Authored by brooks on Feb 16 2026, 1:49 PM.
Tags
None
Referenced Files
F163022656: D55307.diff
Sun, Jul 19, 7:24 AM
Unknown Object (File)
Wed, Jul 15, 7:08 AM
Unknown Object (File)
Wed, Jul 15, 7:08 AM
Unknown Object (File)
Tue, Jul 14, 5:02 PM
Unknown Object (File)
Tue, Jul 14, 3:46 PM
Unknown Object (File)
Sun, Jul 12, 5:45 AM
Unknown Object (File)
Mon, Jul 6, 8:15 AM
Unknown Object (File)
Sun, Jul 5, 10:06 PM
View All 73 Files
Subscribers
imp

Details

Reviewers
kib
imp
Group Reviewers
cheri
Commits
rG2adc3f0db187: stddef.h: centralize definition of offsetof()
Summary

Move to sys/_offsetof.h and use builtin_offsetof() instead of
offsetof to avoid reintroducing sys/cdefs.h polution in stddef.h.




This has the side effect of allowing sys/stddef.h to be included after

stddef.h which can happen in compatability headers.



Effort:         CHERI upstreaming

Sponsored by:   DARPA, AFRL
Diff Detail
Repository 
rG FreeBSD src repository 
Lint 
Lint Passed
 
Unit 
No Test Coverage
 
Build Status 
Buildable 70696
Build 67579: arc lint + arc unit
 
Event Timeline
brooks created this revision.Feb 16 2026, 1:49 PM
Herald added a subscriber: imp.  ·  View Herald TranscriptFeb 16 2026, 1:49 PM
brooks requested review of this revision.Feb 16 2026, 1:49 PM
brooks added a child revision: D55308: build: add stddef.h to define ptraddr_t as required.
Harbormaster completed remote builds in B70696: Diff 171969.Feb 16 2026, 1:50 PM
brooks added a parent revision: D55305: stddef.h: add ptraddr_t.Feb 16 2026, 1:50 PM
kib accepted this revision.Feb 16 2026, 2:32 PM
The change by itself is fine.



This is a related comment.  sys/stddef.h definition for offsetof() uses offsetof() instead of builtin_offsetof().  Then, should we have e.g. sys/_offsetof.h with include guard, then this issue would be handled automatically?
This revision is now accepted and ready to land.Feb 16 2026, 2:32 PM
brooks retitled this revision from sys/stddef.h: guard offsetof definition to stddef.h: centralize definition of offsetof().Feb 18 2026, 2:39 PM
brooks edited the summary of this revision. (Show Details)
Herald added a reviewer: cheri.  ·  View Herald TranscriptFeb 18 2026, 2:39 PM
brooks updated this revision to Diff 172151.Feb 18 2026, 2:40 PM
centralize definition
This revision now requires review to proceed.Feb 18 2026, 2:40 PM
Harbormaster completed remote builds in B70749: Diff 172151.Feb 18 2026, 2:40 PM
kib accepted this revision.Feb 18 2026, 2:58 PM
This revision is now accepted and ready to land.Feb 18 2026, 2:58 PM
imp accepted this revision.Feb 18 2026, 4:34 PM
Closed by commit rG2adc3f0db187: stddef.h: centralize definition of offsetof() (authored by brooks).  ·  Explain WhyFeb 19 2026, 3:11 PM
This revision was automatically updated to reflect the committed changes.
brooks added a commit: rG2adc3f0db187: stddef.h: centralize definition of offsetof().
Revision Contents
Changeset List
PathSize
sys/
sys/
2 lines
Diff 171969
View Options
sys/sys/stddef.h
Loading...