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
Unknown Object (File)
Sun, Mar 22, 4:36 AM
Unknown Object (File)
Sun, Mar 22, 4:36 AM
Unknown Object (File)
Fri, Mar 20, 7:28 PM
Unknown Object (File)
Wed, Mar 18, 3:42 AM
Unknown Object (File)
Fri, Mar 13, 3:32 PM
Unknown Object (File)
Fri, Mar 13, 3:32 PM
Unknown Object (File)
Wed, Mar 11, 11:24 AM
Unknown Object (File)
Wed, Mar 11, 9:55 AM
View All 26 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 70749
Build 67632: 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
include/
5 lines
sys/
sys/
19 lines
3 lines
Diff 172151
View Options
include/stddef.h
Loading...
View Options
sys/sys/_offsetof.h
Loading...
View Options
sys/sys/stddef.h
Loading...