Page MenuHomeFreeBSD
Differential D15708

Don’t bother looking for non-executable pages when a process is excluded from PTI.
ClosedPublic
Actions

Authored by tychon on Jun 8 2018, 7:26 PM.
Tags
None
Referenced Files
F156501406: D15708.id43477.diff
Thu, May 14, 4:34 AM
Unknown Object (File)
Sun, May 10, 10:42 AM
Unknown Object (File)
Fri, May 8, 6:49 PM
Unknown Object (File)
Thu, May 7, 7:09 PM
Unknown Object (File)
Fri, May 1, 10:17 PM
Unknown Object (File)
Mon, Apr 27, 8:50 AM
Unknown Object (File)
Apr 11 2026, 2:41 PM
Unknown Object (File)
Apr 7 2026, 12:27 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
Commits
rS334856: Don't bother looking for non-executable pages when a process is

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tychon created this revision.Jun 8 2018, 7:26 PM
kib added inline comments.Jun 8 2018, 8:14 PM
sys/amd64/amd64/trap.c
773 ↗(On Diff #43471)

pti is not needed anymore ?

775 ↗(On Diff #43471)

Excess ().

tychon updated this revision to Diff 43476.Jun 8 2018, 8:22 PM
tychon marked an inline comment as done.

Incorporate feedback.

sys/amd64/amd64/trap.c
773 ↗(On Diff #43471)

You're right. I can replace pti with PCPU_GET(curpmap)->pm_ucr3 != PMAP_NO_CR3.

kib accepted this revision.Jun 8 2018, 8:33 PM
This revision is now accepted and ready to land.Jun 8 2018, 8:33 PM
Closed by commit rS334856: Don't bother looking for non-executable pages when a process is (authored by tychon). · Explain WhyJun 8 2018, 8:36 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptJun 8 2018, 8:36 PM

Revision Contents
Changeset List

PathSize
head/
sys/
amd64/
amd64/
3 lines

Diff 43477

View Options

head/sys/amd64/amd64/trap.c

Loading...