amd64 pmap_change_attr(): avoid spurious EINVAL.
ClosedPublic
Actions

Authored by kib on Jun 1 2016, 2:25 AM.

Tags

None

Referenced Files

	F134776232: D6668.id17166.diff
	Tue, Nov 4, 1:44 PM

	F134776231: D6668.id.diff
	Tue, Nov 4, 1:44 PM

	F134776216: D6668.id17332.diff
	Tue, Nov 4, 1:44 PM

	F134754487: D6668.diff
	Tue, Nov 4, 9:58 AM

	Unknown Object (File)
	Mon, Oct 27, 6:05 AM

	Unknown Object (File)
	Sat, Oct 25, 1:15 AM

	Unknown Object (File)
	Mon, Oct 20, 4:17 AM

	Unknown Object (File)
	Sun, Oct 19, 5:27 PM

View All 99 Files

Subscribers

imp

Details

Reviewers

alc
jhb
mav

Commits

rS301457: Avoid spurious EINVAL in amd64 pmap_change_attr().

Summary

Do not try to change attributes for DMAP when working on a mapping which is not covered by the DMAP. This was reported on real system where a BAR of a device (NTB) was mapped outside the PCI window.

Reported and tested by: mav

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

kib updated this revision to Diff 17166.Jun 1 2016, 2:25 AM

kib retitled this revision from to amd64 pmap_change_attr(): avoid spurious EINVAL..

kib updated this object.

kib edited the test plan for this revision. (Show Details)

kib added reviewers: jhb, alc, mav.

kib set the repository for this revision to rS FreeBSD src repository - subversion.

Herald added a subscriber: imp. · View Herald TranscriptJun 1 2016, 2:25 AM

Looks good and fixes EINVAL errors for me.

This revision is now accepted and ready to land.Jun 1 2016, 3:07 AM

Note that ntb(4) should be changed to use bus_map_resource() and stop using pmap_change_attr() directly in 11 but that is orthogonal to this.

Closed by commit rS301457: Avoid spurious EINVAL in amd64 pmap_change_attr(). (authored by kib). · Explain WhyJun 5 2016, 5:11 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

amd64/

pmap.c

11 lines

Diff 17166

View Options

amd64 pmap_change_attr(): avoid spurious EINVAL.ClosedPublicActions