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
Unknown Object (File)
Oct 21 2024, 7:03 AM
Unknown Object (File)
Sep 29 2024, 8:33 AM
Unknown Object (File)
Sep 22 2024, 10:45 PM
Unknown Object (File)
Sep 21 2024, 7:15 AM
Unknown Object (File)
Sep 19 2024, 8:50 PM
Unknown Object (File)
Sep 19 2024, 6:08 PM
Unknown Object (File)
Sep 19 2024, 5:23 AM
Unknown Object (File)
Sep 18 2024, 10:04 PM
View All 51 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...