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
F169843299: D20751.id58982.diff
Wed, Sep 2, 5:52 PM
Unknown Object (File)
Sun, Aug 30, 2:11 AM
Unknown Object (File)
Sat, Aug 29, 8:52 AM
Unknown Object (File)
Sun, Aug 23, 2:32 PM
Unknown Object (File)
Sat, Aug 22, 11:12 PM
Unknown Object (File)
Fri, Aug 21, 6:44 PM
Unknown Object (File)
Fri, Aug 21, 6:40 PM
Unknown Object (File)
Wed, Aug 12, 1:04 AM
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...