Page MenuHomeFreeBSD
Differential D40917

Don't bother to check COMPAT_32BIT when __LP64__ is defined
ClosedPublic
Actions

Authored by jrtc27 on Jul 7 2023, 11:57 PM.
Tags
None
Referenced Files
F149250679: D40917.id124386.diff
Mon, Mar 23, 7:14 AM
Unknown Object (File)
Sun, Mar 22, 3:15 AM
Unknown Object (File)
Sun, Mar 15, 7:47 AM
Unknown Object (File)
Thu, Mar 5, 3:44 PM
Unknown Object (File)
Feb 20 2026, 3:45 AM
Unknown Object (File)
Feb 7 2026, 8:15 PM
Unknown Object (File)
Nov 25 2025, 5:50 AM
Unknown Object (File)
Nov 20 2025, 2:59 AM
View All 87 Files
Subscribers
delphij
imp

Details

Reviewers
mm
brooks
jhb
imp
Commits
rGc63c4e694cfb: Don't bother to check COMPAT_32BIT when __LP64__ is defined
Summary

Under COMPAT_32BIT we are compiling 32-bit code and so LP64 is not
defined, ILP32 is, and thus the check is completely redundant.

Diff Detail

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

Event Timeline

jrtc27 created this revision.Jul 7 2023, 11:57 PM
Herald added subscribers: delphij, imp. · View Herald TranscriptJul 7 2023, 11:57 PM
jrtc27 requested review of this revision.Jul 7 2023, 11:57 PM
Harbormaster completed remote builds in B52491: Diff 124325.Jul 7 2023, 11:57 PM
jrtc27 added a parent revision: D40932: bsd.compat.mk: Remove obsolete COMPAT_32BIT now it's unused.Jul 8 2023, 12:07 AM
jrtc27 removed a parent revision: D40932: bsd.compat.mk: Remove obsolete COMPAT_32BIT now it's unused.
jrtc27 added a child revision: D40932: bsd.compat.mk: Remove obsolete COMPAT_32BIT now it's unused.
imp accepted this revision.Jul 8 2023, 12:13 AM
This revision is now accepted and ready to land.Jul 8 2023, 12:13 AM
brooks accepted this revision.Jul 8 2023, 12:33 AM

Probably should upstream this

jrtc27 added a comment.Jul 8 2023, 12:47 AM
In D40917#931597, @brooks wrote:

Probably should upstream this

Yeah, I was planning to

jhb accepted this revision.Jul 8 2023, 7:33 PM
Closed by commit rGc63c4e694cfb: Don't bother to check COMPAT_32BIT when __LP64__ is defined (authored by jrtc27). · Explain WhyJul 9 2023, 5:53 PM
This revision was automatically updated to reflect the committed changes.
jrtc27 added a commit: rGc63c4e694cfb: Don't bother to check COMPAT_32BIT when __LP64__ is defined.
des added a reverting change: D41483: openzfs: Revert to last known good state..Aug 16 2023, 6:31 PM

Revision Contents
Changeset List

PathSize
sys/
cddl/
compat/
opensolaris/
sys/
4 lines
contrib/
openzfs/
include/
os/
freebsd/
spl/
sys/
4 lines

Diff 124386

View Options

sys/cddl/compat/opensolaris/sys/atomic.h

Loading...
View Options

sys/contrib/openzfs/include/os/freebsd/spl/sys/atomic.h

Loading...