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)
	Fri, Aug 28, 10:58 PM

	Unknown Object (File)
	Sun, Aug 23, 12:17 AM

	Unknown Object (File)
	Sun, Aug 23, 12:16 AM

	Unknown Object (File)
	Sat, Aug 22, 12:34 PM

	Unknown Object (File)
	Sat, Aug 22, 9:03 AM

	Unknown Object (File)
	Fri, Aug 21, 1:48 PM

	Unknown Object (File)
	Fri, Aug 21, 12:12 PM

	Unknown Object (File)
	Thu, Aug 20, 7:52 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 ().

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