Page MenuHomeFreeBSD

vm_map_protect.9: update after code changes
ClosedPublic

Authored by kib on Jan 23 2021, 8:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 7 2024, 11:42 PM
Unknown Object (File)
Dec 20 2023, 4:19 AM
Unknown Object (File)
Dec 14 2023, 7:05 PM
Unknown Object (File)
Sep 26 2023, 3:23 AM
Unknown Object (File)
Sep 26 2023, 3:21 AM
Unknown Object (File)
Sep 7 2023, 4:01 AM
Unknown Object (File)
Sep 7 2023, 3:58 AM
Unknown Object (File)
Sep 7 2023, 3:58 AM
Subscribers
None

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kib requested review of this revision.Jan 23 2021, 8:21 PM
kib created this revision.
share/man/man9/vm_map_protect.9
55–56

I think it's fine to leave this as "sets". I assume you changed it because the function does not necessarily set both. Because it's entirely dependent on the function's arguments I think it's still reasonable to say sets. To me "may set" implies that vm_map_protect will itself decide if one/both will be set.

84

Perhaps "are not allowed to"?

As is (and before your change) it seems unclear to me whether this is something the caller must take care not to violate, or that the function disallows.

97

Maybe something like
"Also, any in-progress wiring operation on the map affecting the specified range will cause .Nm to sleep, waiting for completion"

115

"The map does not allow ..." I think is clearer

116

Drop If as above, "Both new protection and..."
Or maybe Both VM_MAP_PROTECT_SET_PROT and VM_MAP_PROTECT_SET_MAXPROT were ..."

kib marked 5 inline comments as done.

Handle Ed notes.

share/man/man9/vm_map_protect.9
87

"The operation"

I would change "is declined" to "will fail" and "the condition" to "this condition".

89

"a shared mapping"

"of a read-only file from read-only to read-write"

126

s/for backing/to back/

129

"... protection updates were requested"

130

"but the specified"

kib marked 5 inline comments as done.

Mark' notes.

This revision is now accepted and ready to land.Jan 25 2021, 9:29 PM
This revision was automatically updated to reflect the committed changes.