Paths

Table of Contentst

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)
	Mon, Nov 3, 1:18 AM

	Unknown Object (File)
	Sat, Nov 1, 4:59 AM

	Unknown Object (File)
	Sat, Oct 25, 10:02 AM

	Unknown Object (File)
	Sat, Oct 25, 2:25 AM

	Unknown Object (File)
	Sun, Oct 19, 1:44 PM

	Unknown Object (File)
	Sun, Oct 19, 1:44 PM

	Unknown Object (File)
	Sat, Oct 18, 11:05 PM

	Unknown Object (File)
	Sat, Oct 18, 11:05 PM

View All 99 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 ().

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

Path

Size

head/

sys/

amd64/

trap.c

3 lines

Diff 43477

View Options

head/sys/amd64/amd64/trap.c

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