Page MenuHomeFreeBSD
Differential D16130

Correct an obscure PV entry leak in the recent change to amd64's pmap_enter()
ClosedPublic
Actions

Authored by alc on Jul 4 2018, 7:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 10, 7:17 AM
Unknown Object (File)
Thu, Mar 5, 7:13 PM
Unknown Object (File)
Fri, Feb 27, 3:59 AM
Unknown Object (File)
Feb 8 2026, 12:04 AM
Unknown Object (File)
Feb 7 2026, 9:44 PM
Unknown Object (File)
Jan 31 2026, 11:19 AM
Unknown Object (File)
Dec 27 2025, 6:10 PM
Unknown Object (File)
Dec 19 2025, 6:37 PM
View All 75 Files
Subscribers
imp

Details

Reviewers
kib
markj
Commits
rS335971: As of r335784, if pmap_enter() replaces a managed mapping by an unmanaged
Summary

Hypothetically, pmap_enter() could replace a managed mapping by an unmanaged mapping, in which case, pmap_enter() would currently leak the unlinked PV entry. This change frees that PV entry when the new mapping is unmanaged.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

alc created this revision.Jul 4 2018, 7:07 PM
markj accepted this revision.Jul 4 2018, 7:30 PM

I noticed this bug while reading the i386 pmap code. :)

This revision is now accepted and ready to land.Jul 4 2018, 7:30 PM
kib accepted this revision.Jul 4 2018, 8:05 PM
Closed by commit rS335971: As of r335784, if pmap_enter() replaces a managed mapping by an unmanaged (authored by alc). · Explain WhyJul 5 2018, 2:04 AM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptJul 5 2018, 2:04 AM
alc added a comment.Jul 5 2018, 2:15 AM
In D16130#342002, @markj wrote:

I noticed this bug while reading the i386 pmap code. :)

I'm working on the i386 version. As you may have noticed, the i386 pmap has the CoW problem, but not the PV entry allocation problem.

Revision Contents
Changeset List

PathSize
head/
sys/
amd64/
amd64/
2 lines

Diff 44884

View Options

head/sys/amd64/amd64/pmap.c

Loading...