2015-06-09 12:12:36

by Julian Orth

Subject: [PATCH] sched_setattr.2: Remove a const attribute

The attr argument of sched_setattr was documented as const but the
kernel will modify the size field of this struct if it contains an
invalid value. See the documentation of the size field for details.
---
man2/sched_setattr.2 | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man2/sched_setattr.2 b/man2/sched_setattr.2
index dff8e6d..df14ab4 100644
--- a/man2/sched_setattr.2
+++ b/man2/sched_setattr.2
@@ -31,7 +31,7 @@ set and get scheduling policy and attributes
.nf
.B #include <sched.h>

-.BI "int sched_setattr(pid_t " pid ", const struct sched_attr *" attr ,
+.BI "int sched_setattr(pid_t " pid ", struct sched_attr *" attr ,
.BI " unsigned int " flags );

.BI "int sched_getattr(pid_t " pid ", struct sched_attr *" attr ,
--
2.4.2

2015-07-23 07:40:36

by Michael Kerrisk (man-pages)

Subject: Re: [PATCH] sched_setattr.2: Remove a const attribute

Hello Julian,

On 06/09/2015 02:11 PM, Julian Orth wrote:
> The attr argument of sched_setattr was documented as const but the
> kernel will modify the size field of this struct if it contains an
> invalid value. See the documentation of the size field for details.

Thanks! Patch applied.

Cheers,

Michael


> ---
> man2/sched_setattr.2 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/man2/sched_setattr.2 b/man2/sched_setattr.2
> index dff8e6d..df14ab4 100644
> --- a/man2/sched_setattr.2
> +++ b/man2/sched_setattr.2
> @@ -31,7 +31,7 @@ set and get scheduling policy and attributes
> .nf
> .B #include <sched.h>
>
> -.BI "int sched_setattr(pid_t " pid ", const struct sched_attr *" attr ,
> +.BI "int sched_setattr(pid_t " pid ", struct sched_attr *" attr ,
> .BI " unsigned int " flags );
>
> .BI "int sched_getattr(pid_t " pid ", struct sched_attr *" attr ,



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