Re: [PATCH 1/4] system_data_types.7: wfix

["Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com>]

Hello Alex,

On 9/18/20 2:45 PM, Alejandro Colomar wrote:
> Signed-off-by: Alejandro Colomar <colomar.6.4.3@gmail.com>
> ---
>  man7/system_data_types.7 | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/man7/system_data_types.7 b/man7/system_data_types.7
> index dba80cf6f..8969ee966 100644
> --- a/man7/system_data_types.7
> +++ b/man7/system_data_types.7
> @@ -350,7 +350,7 @@ struct timespec {
>  };
>  .EE
>  .IP
> -Describes times in seconds and nanoseconds.
> +Describes elapsed time in seconds and nanoseconds.
>  .IP
>  Conforming to: C11 and later; POSIX.1-2001 and later.
>  .IP
> @@ -379,7 +379,7 @@ struct timeval {
>  };
>  .EE
>  .IP
> -Describes times in seconds and microseconds.
> +Describes elapsed time in seconds and microseconds.
>  .IP
>  Conforming to: POSIX.1-2001 and later.
>  .IP

I don;t think this patch is correct. At the least, it needs 
more explanation. As far as I can see, 'timespec' is not just 
about elapsed time. It really is just time, expressed as secs + 
nanosecs. For example, see utimensat(2), which set file timestamps
with nanosec precision.

For the moment, I'm not applying this (and I hope not doing so
doesn't break any following patch).

Thanks,

Michael

-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/