From 59ff172f8064757e2302f9b4fb7583ec2841d591 Mon Sep 17 00:00:00 2001 From: Roland McGrath Date: Thu, 8 Jan 2015 10:49:40 -0800 Subject: NPTL: Refactor cpu_set_t validation to be sysdeps-controlled This removes Linuxisms related to cpu_set_t validation from the generic NPTL code and puts them into a new sysdeps header file. --- ChangeLog | 8 ++++++ nptl/check-cpuset.h | 32 +++++++++++++++++++++++ nptl/pthreadP.h | 27 ------------------- nptl/pthread_attr_setaffinity.c | 3 +-- nptl/pthread_setattr_default_np.c | 2 ++ sysdeps/unix/sysv/linux/check-cpuset.h | 48 ++++++++++++++++++++++++++++++++++ 6 files changed, 91 insertions(+), 29 deletions(-) create mode 100644 nptl/check-cpuset.h create mode 100644 sysdeps/unix/sysv/linux/check-cpuset.h diff --git a/ChangeLog b/ChangeLog index a979d86d61..fa360a467e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2015-01-08 Roland McGrath + + * nptl/pthreadP.h (check_cpuset_attr): Moved ... + * sysdeps/unix/sysv/linux/check-cpuset.h: ... to this new file. + * nptl/pthread_attr_setaffinity.c: Include it. + * nptl/pthread_setattr_default_np.c: Likewise. + * nptl/check-cpuset.h: New file. + 2015-01-08 Richard Henderson * elf/Makefile (CFLAGS-tst-execstack-mod.c): Add -Wno-trampolines diff --git a/nptl/check-cpuset.h b/nptl/check-cpuset.h new file mode 100644 index 0000000000..315bdf2626 --- /dev/null +++ b/nptl/check-cpuset.h @@ -0,0 +1,32 @@ +/* Validate cpu_set_t values for NPTL. Stub version. + Copyright (C) 2015 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include + +/* Returns 0 if CS and SZ are valid values for the cpuset and cpuset size + respectively. Otherwise it returns an error number. */ +static inline int +check_cpuset_attr (const cpu_set_t *cs, const size_t sz) +{ + if (sz == 0) + return 0; + + /* This means pthread_attr_setaffinity will return ENOSYS, which + is the right thing when the cpu_set_t features are not available. */ + return ENOSYS; +} diff --git a/nptl/pthreadP.h b/nptl/pthreadP.h index 8a3494cb37..84a7105848 100644 --- a/nptl/pthreadP.h +++ b/nptl/pthreadP.h @@ -629,31 +629,4 @@ check_stacksize_attr (size_t st) return EINVAL; } -/* Defined in pthread_setaffinity.c. */ -extern size_t __kernel_cpumask_size attribute_hidden; -extern int __determine_cpumask_size (pid_t tid); - -/* Returns 0 if CS and SZ are valid values for the cpuset and cpuset size - respectively. Otherwise it returns an error number. */ -static inline int -check_cpuset_attr (const cpu_set_t *cs, const size_t sz) -{ - if (__kernel_cpumask_size == 0) - { - int res = __determine_cpumask_size (THREAD_SELF->tid); - if (res) - return res; - } - - /* Check whether the new bitmask has any bit set beyond the - last one the kernel accepts. */ - for (size_t cnt = __kernel_cpumask_size; cnt < sz; ++cnt) - if (((char *) cs)[cnt] != '\0') - /* Found a nonzero byte. This means the user request cannot be - fulfilled. */ - return EINVAL; - - return 0; -} - #endif /* pthreadP.h */ diff --git a/nptl/pthread_attr_setaffinity.c b/nptl/pthread_attr_setaffinity.c index a249b419f1..7a127b80f7 100644 --- a/nptl/pthread_attr_setaffinity.c +++ b/nptl/pthread_attr_setaffinity.c @@ -23,8 +23,7 @@ #include #include #include - - +#include int diff --git a/nptl/pthread_setattr_default_np.c b/nptl/pthread_setattr_default_np.c index 8d5bb710db..457a467df8 100644 --- a/nptl/pthread_setattr_default_np.c +++ b/nptl/pthread_setattr_default_np.c @@ -21,6 +21,8 @@ #include #include #include +#include + int pthread_setattr_default_np (const pthread_attr_t *in) diff --git a/sysdeps/unix/sysv/linux/check-cpuset.h b/sysdeps/unix/sysv/linux/check-cpuset.h new file mode 100644 index 0000000000..1d55e0bb0e --- /dev/null +++ b/sysdeps/unix/sysv/linux/check-cpuset.h @@ -0,0 +1,48 @@ +/* Validate cpu_set_t values for NPTL. Linux version. + Copyright (C) 2002-2015 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include +#include + + +/* Defined in pthread_setaffinity.c. */ +extern size_t __kernel_cpumask_size attribute_hidden; +extern int __determine_cpumask_size (pid_t tid); + +/* Returns 0 if CS and SZ are valid values for the cpuset and cpuset size + respectively. Otherwise it returns an error number. */ +static inline int +check_cpuset_attr (const cpu_set_t *cs, const size_t sz) +{ + if (__kernel_cpumask_size == 0) + { + int res = __determine_cpumask_size (THREAD_SELF->tid); + if (res) + return res; + } + + /* Check whether the new bitmask has any bit set beyond the + last one the kernel accepts. */ + for (size_t cnt = __kernel_cpumask_size; cnt < sz; ++cnt) + if (((char *) cs)[cnt] != '\0') + /* Found a nonzero byte. This means the user request cannot be + fulfilled. */ + return EINVAL; + + return 0; +} -- cgit v1.2.3