Re: [PATCH v5] prctl.2: Document Syscall User Dispatch

["Alejandro Colomar (man-pages)" <alx.manpages@gmail.com>]

Hi Michael & Gabriel,

I disagree on a comment; see below.

On 12/24/20 10:18 AM, Michael Kerrisk (man-pages) wrote:
> Hello Gabriel,
> 
> Thanks for the revisions!
> 
[...]
>> +.IP
>> +When
>> +.I arg2
>> +is set to
>> +.BR PR_SYS_DISPATCH_ON ,
>> +.I arg3
>> +and
>> +.I arg4
>> +respectively identify the
>> +.I offset
>> +and
>> +.I length
>> +of a single contiguous memory region in the process map
>> +from where system calls are always allowed to be executed,
>> +regardless of the switch variable
>> +(Typically, this area would include the area of memory
>> +containing the C library).
> 
> s/)./.)/

That point is ending the full sentence starting at 'When', not just the
sentence in parentheses.

See <https://www.thepunctuationguide.com/terminal-points.html>.

Quoting that, "The main sentence takes a period outside the closing
parenthesis no matter what punctuation is included inside the
parenthetical element.".  So that you should be able to:  s/(.*)//

s/)./.)./  would be more correct, but I think ').' is enough.
See my quotation above, which is a similar example :)

I'd also s/Typically/typically/

Your thoughts?

Thanks,

Alex

> 
>> +.I arg5
>> +points to a char-sized variable
>> +that is a fast switch to enable/disable the mechanism
>> +without the overhead of doing a system call.
>> +The variable pointed by

-- 
Alejandro Colomar
Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/