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)
	Sat, Apr 11, 2:41 PM

	Unknown Object (File)
	Tue, Apr 7, 12:27 PM

	Unknown Object (File)
	Mon, Apr 6, 3:14 PM

	Unknown Object (File)
	Thu, Apr 2, 12:22 AM

	Unknown Object (File)
	Tue, Mar 31, 1:20 AM

	Unknown Object (File)
	Wed, Mar 25, 3:27 AM

	Unknown Object (File)
	Mon, Mar 23, 12:11 AM

	Unknown Object (File)
	Mar 19 2026, 9:03 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