diff options
| author | Roland McGrath <roland@hack.frob.com> | 2014-11-19 17:34:25 -0800 |
|---|---|---|
| committer | Roland McGrath <roland@hack.frob.com> | 2014-11-19 17:34:25 -0800 |
| commit | d960211ff52a6ab632eace2f905c4898f187df9e (patch) | |
| tree | f989f556029bd7c3a6fad8fc5461aeb11c516e58 /nptl | |
| parent | cbd463e2cfc94d8cf74d17f17a9c572c272b9376 (diff) | |
| download | glibc-d960211ff52a6ab632eace2f905c4898f187df9e.zip glibc-d960211ff52a6ab632eace2f905c4898f187df9e.tar.gz glibc-d960211ff52a6ab632eace2f905c4898f187df9e.tar.bz2 | |
NPTL: Don't (re)validate sched_priority in pthread_create.
Diffstat (limited to 'nptl')
| -rw-r--r-- | nptl/pthread_create.c | 21 |
1 files changed, 1 insertions, 20 deletions
diff --git a/nptl/pthread_create.c b/nptl/pthread_create.c index da3dc46..1dd1b95 100644 --- a/nptl/pthread_create.c +++ b/nptl/pthread_create.c @@ -605,6 +605,7 @@ __pthread_create_2_1 (newthread, attr, start_routine, arg) } if (iattr->flags & ATTR_FLAG_SCHED_SET) + /* The values were validated in pthread_attr_setschedparam. */ memcpy (&pd->schedparam, &iattr->schedparam, sizeof (struct sched_param)); else if ((pd->flags & ATTR_FLAG_SCHED_SET) == 0) @@ -612,26 +613,6 @@ __pthread_create_2_1 (newthread, attr, start_routine, arg) INTERNAL_SYSCALL (sched_getparam, scerr, 2, 0, &pd->schedparam); pd->flags |= ATTR_FLAG_SCHED_SET; } - - /* Check for valid priorities. */ - int minprio = INTERNAL_SYSCALL (sched_get_priority_min, scerr, 1, - iattr->schedpolicy); - int maxprio = INTERNAL_SYSCALL (sched_get_priority_max, scerr, 1, - iattr->schedpolicy); - if (pd->schedparam.sched_priority < minprio - || pd->schedparam.sched_priority > maxprio) - { - /* Perhaps a thread wants to change the IDs and if waiting - for this stillborn thread. */ - if (__builtin_expect (atomic_exchange_acq (&pd->setxid_futex, 0) - == -2, 0)) - lll_futex_wake (&pd->setxid_futex, 1, LLL_PRIVATE); - - __deallocate_stack (pd); - - retval = EINVAL; - goto out; - } } /* Pass the descriptor to the caller. */ |