From: Roland McGrath Date: Thu, 8 Jan 2015 18:49:40 +0000 (-0800) Subject: NPTL: Refactor cpu_set_t validation to be sysdeps-controlled X-Git-Tag: upstream/2.30~6496 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=59ff172f8064757e2302f9b4fb7583ec2841d591;p=external%2Fglibc.git 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. --- diff --git a/ChangeLog b/ChangeLog index a979d86..fa360a4 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 0000000..315bdf2 --- /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 8a3494c..84a7105 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 a249b41..7a127b8 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 8d5bb71..457a467 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 0000000..1d55e0b --- /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; +}