Page MenuHomeFreeBSD
Differential D20751

vm_map_protect(9) fails to document KERN_RESOURCE_SHORTAGE return value.
ClosedPublic
Actions

Authored by dougm on Jun 25 2019, 6:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 26, 12:31 PM
Unknown Object (File)
Tue, Nov 26, 10:32 AM
Unknown Object (File)
Tue, Nov 19, 8:58 PM
Unknown Object (File)
Thu, Nov 14, 2:17 PM
Unknown Object (File)
Thu, Nov 7, 6:35 PM
Unknown Object (File)
Sun, Nov 3, 8:53 AM
Unknown Object (File)
Oct 28 2024, 11:50 PM
Unknown Object (File)
Oct 28 2024, 8:50 PM
View All Files
Subscribers
alc
imp

Details

Reviewers
kib
markj
Group Reviewers
manpages
Commits
rS349379: Document the KERN_PROTECTION_FAILURE return value from vm_map_protect().
Summary

This patch is a meager attempt to correct that.

Diff Detail

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

Event Timeline

dougm created this revision.Jun 25 2019, 6:31 AM
dougm added a subscriber: alc.Jun 25 2019, 6:47 AM
alc added inline comments.Jun 25 2019, 7:17 AM
vm_map_protect.9
99 ↗(On Diff #58982)

The range isn't being allocated. It is copy-on-write, and it is transitioning from read-only to read/write.

dougm updated this revision to Diff 58983.Jun 25 2019, 7:24 AM

Take Alan's suggestion.

kib added inline comments.Jun 25 2019, 10:33 AM
vm_map_protect.9
100 ↗(On Diff #58983)

I think it is somewhat too cryptic. 'If copy-on-write mapping is transitioned from ro to rw, too little swap space is available for backing the copied pages'.

dougm updated this revision to Diff 59005.Jun 25 2019, 3:37 PM

Accept Kostik's suggestion.

dougm updated this revision to Diff 59006.Jun 25 2019, 3:40 PM
markj accepted this revision.Jun 25 2019, 4:41 PM
markj added inline comments.
vm_map_protect.9
99 ↗(On Diff #59006)

"If a copy-on-write mapping is transitioned from read-only to read-write..."

This revision is now accepted and ready to land.Jun 25 2019, 4:41 PM
kib accepted this revision.Jun 25 2019, 4:46 PM

With Mark' note, accepted.

dougm updated this revision to Diff 59016.Jun 25 2019, 4:47 PM

Accept reviewer suggestion.

This revision now requires review to proceed.Jun 25 2019, 4:47 PM
markj added inline comments.Jun 25 2019, 4:49 PM
vm_map_protect.9
99 ↗(On Diff #59016)

Still missing an article: "If a copy-on-write..."

dougm updated this revision to Diff 59017.Jun 25 2019, 4:51 PM

Articulate.

markj accepted this revision.Jun 25 2019, 5:15 PM
This revision is now accepted and ready to land.Jun 25 2019, 5:15 PM
Closed by commit rS349379: Document the KERN_PROTECTION_FAILURE return value from vm_map_protect(). (authored by dougm). · Explain WhyJun 25 2019, 5:27 PM
This revision was automatically updated to reflect the committed changes.
dougm added a commit: rS349379: Document the KERN_PROTECTION_FAILURE return value from vm_map_protect()..
Herald added a reviewer: manpages. · View Herald TranscriptJun 25 2019, 5:27 PM
Herald added a subscriber: imp. · View Herald Transcript

Revision Contents
Changeset List

PathSize
head/
share/
man/
man9/
5 lines

Diff 59023

View Options

head/share/man/man9/vm_map_protect.9

Loading...