Page MenuHomeFreeBSD
Differential D58986

contrib/ck: keep dcas parameters thin under bounds-safety
AcceptedPublic
Actions

Authored by abhijeetsharma2002_gmail.com on Aug 19 2026, 12:03 PM.
Tags
None
Referenced Files
F172469986: D58986.diff
Fri, Sep 18, 4:57 PM
Unknown Object (File)
Thu, Sep 17, 4:21 PM
Unknown Object (File)
Thu, Sep 17, 10:22 AM
Unknown Object (File)
Thu, Sep 17, 6:15 AM
Unknown Object (File)
Wed, Sep 16, 5:33 AM
Unknown Object (File)
Tue, Sep 15, 4:58 AM
Unknown Object (File)
Mon, Sep 14, 11:38 PM
Unknown Object (File)
Sat, Sep 12, 9:02 AM
View All 50 Files
Subscribers
imp

Details

Reviewers
rpaulo
emaste
andrew
Summary

Under -fbounds-safety a uint64_t [2] parameter becomes a bounds-carrying
pointer, which cannot be an inline assembly operand. Plain uint64_t * is
identical in C and carries no bound. Vendor declarations stay in the
#else for the next CK import.

Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 75739
Build 72622: arc lint + arc unit

Event Timeline

abhijeetsharma2002_gmail.com created this revision.Aug 19 2026, 12:03 PM
Herald added a reviewer: andrew. · View Herald TranscriptAug 19 2026, 12:03 PM
Herald added a subscriber: imp. · View Herald Transcript
abhijeetsharma2002_gmail.com requested review of this revision.Aug 19 2026, 12:03 PM
Harbormaster completed remote builds in B75739: Diff 184353.Aug 19 2026, 12:03 PM
abhijeetsharma2002_gmail.com added a parent revision: D58985: sys: build core headers under bounds-safety.Aug 19 2026, 12:03 PM
abhijeetsharma2002_gmail.com added a child revision: D58987: kern: add bounds-safety soft-trap runtime.
rpaulo added inline comments.Thu, Aug 27, 10:41 PM
sys/contrib/ck/include/gcc/aarch64/ck_pr_llsc.h
42

Wouldn’t this work?

abhijeetsharma2002_gmail.com added inline comments.Fri, Sep 4, 3:29 AM
sys/contrib/ck/include/gcc/aarch64/ck_pr_llsc.h
42

Hey, unfortunately not. The issue is that any pointer with bounds cannot be used with inline assembly.

rpaulo accepted this revision.Sat, Sep 5, 12:15 AM
This revision is now accepted and ready to land.Sat, Sep 5, 12:15 AM

Revision Contents
Changeset List

PathSize
sys/
contrib/
ck/
include/
gcc/
aarch64/
16 lines
x86_64/
25 lines

Diff 184353

View Options

sys/contrib/ck/include/gcc/aarch64/ck_pr_llsc.h

Loading...
View Options

sys/contrib/ck/include/gcc/x86_64/ck_pr.h

Loading...