Page MenuHomeFreeBSD

amd64 pmap: two trivial changes
ClosedPublic

Authored by kib on Oct 10 2022, 12:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 3, 11:45 PM
Unknown Object (File)
Mon, Dec 2, 2:13 PM
Unknown Object (File)
Mon, Nov 18, 3:46 PM
Unknown Object (File)
Mon, Nov 18, 3:28 PM
Unknown Object (File)
Mon, Nov 18, 1:45 PM
Unknown Object (File)
Fri, Nov 15, 2:10 PM
Unknown Object (File)
Nov 1 2024, 4:56 AM
Unknown Object (File)
Oct 18 2024, 2:06 AM
Subscribers

Details

Summary
add comment explaining why INVLPG is functional for PCID config
remove useless addr2 variables in page range invalidation handlers

Diff Detail

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

Event Timeline

kib requested review of this revision.Oct 10 2022, 12:40 AM
This revision is now accepted and ready to land.Oct 10 2022, 1:51 PM
sys/amd64/amd64/pmap.c
2909–2913 ↗(On Diff #111597)

I would suggest:

"When PCID is enabled, the INVLPG instruction invalidates all TLB entries for the given page that either match the current PCID or are global. Since TLB entries for the same page under different PCIDs are unaffected, kernel pages which reside in all address spaces could be problematic. We avoid ...

kib marked an inline comment as done.

Alan' wording

This revision now requires review to proceed.Oct 10 2022, 8:42 PM
This revision is now accepted and ready to land.Oct 10 2022, 9:30 PM