Page MenuHomeFreeBSD
Differential D42241

amd64: Zero-fill AP PCPU pages
ClosedPublic
Actions

Authored by markj on Oct 16 2023, 10:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 29, 4:06 PM
Unknown Object (File)
Sat, Sep 26, 10:28 PM
Unknown Object (File)
Sat, Sep 26, 12:21 PM
Unknown Object (File)
Fri, Sep 25, 11:51 PM
Unknown Object (File)
Sat, Sep 19, 7:24 PM
Unknown Object (File)
Sep 4 2026, 1:37 PM
Unknown Object (File)
Sep 2 2026, 9:30 PM
Unknown Object (File)
Sep 2 2026, 5:27 AM
View All Files
Subscribers
imp

Details

Reviewers
kib
Commits
rGe4d86dda3413: amd64: Zero-fill AP PCPU pages
rG4533fa42ad91: amd64: Zero-fill AP PCPU pages
rGa37e484d0497: amd64: Zero-fill AP PCPU pages
Summary

At least KMSAN relies on zero-initialization of AP PCPU regions, see
commit 4b136ef259ce ("amd64: Set GS.base before calling init_secondary() on APs").
Prior to commit af1c6d3f3013 these were allocated with allocpages() in
the amd64 pmap, which always returns zero-initialized memory.

Fixes: af1c6d3f3013 ("amd64: do not leak pcpu pages")
MFC after: 3 days

Diff Detail

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

Event Timeline

markj created this revision.Oct 16 2023, 10:51 PM
Herald added a subscriber: imp. · View Herald TranscriptOct 16 2023, 10:51 PM
markj requested review of this revision.Oct 16 2023, 10:51 PM
Harbormaster completed remote builds in B54019: Diff 128871.Oct 16 2023, 10:51 PM
kib accepted this revision.Oct 17 2023, 9:06 AM
kib added inline comments.
sys/amd64/amd64/mp_machdep.c
297

Why do you need a local var?

This revision is now accepted and ready to land.Oct 17 2023, 9:06 AM
markj added inline comments.Oct 17 2023, 1:04 PM
sys/amd64/amd64/mp_machdep.c
297

I just disliked the duplication.

Closed by commit rGa37e484d0497: amd64: Zero-fill AP PCPU pages (authored by markj). · Explain WhyOct 17 2023, 1:38 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rGa37e484d0497: amd64: Zero-fill AP PCPU pages.
markj added a commit: rG4533fa42ad91: amd64: Zero-fill AP PCPU pages.Oct 20 2023, 4:02 PM
markj added a commit: rGe4d86dda3413: amd64: Zero-fill AP PCPU pages.Oct 22 2023, 3:06 PM

Revision Contents
Changeset List

PathSize
sys/
amd64/
amd64/
9 lines

Diff 128871

View Options

sys/amd64/amd64/mp_machdep.c

Loading...