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)
Thu, Nov 7, 12:30 PM
Unknown Object (File)
Thu, Nov 7, 12:12 PM
Unknown Object (File)
Sep 16 2024, 9:23 AM
Unknown Object (File)
Sep 11 2024, 11:02 PM
Unknown Object (File)
Sep 3 2024, 7:49 AM
Unknown Object (File)
Aug 22 2024, 1:16 AM
Unknown Object (File)
Aug 22 2024, 1:16 AM
Unknown Object (File)
Aug 22 2024, 1:16 AM
View All 27 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 Not Applicable
Unit
Tests Not Applicable

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/
5 lines

Diff 128907

View Options

sys/amd64/amd64/mp_machdep.c

Loading...