summaryrefslogtreecommitdiff
path: root/lib/librt
diff options
context:
space:
mode:
authorrmind <rmind@NetBSD.org>2008-01-26 17:55:29 +0000
committerrmind <rmind@NetBSD.org>2008-01-26 17:55:29 +0000
commitb5e9addd22eabcce2e017947a3b5e8569ca6500a (patch)
tree3dd6e6289a1458aafa08984cba5c0c1f18ae4180 /lib/librt
parentef515ac1dc49cc036e4437a2a8dbf89d538707fe (diff)
sched_setparam: fix the case when incorrect (according to the class)
in-kernel priority is used. Reported by <drochner>. Minor fixes for scheduling calls to conform the POSIX: - If pid is equal to zero, use the calling process; - In case of permission problem, return EPERM instead of EACESS; - sched_setscheduler() should return previously used policy; - pthread_* calls should return the error code or zero; Should fix the namespace problems (and builds of some packages): - Move cpuset_t defintion from pset.h to sched.h; - Remove the #include of pset.h in pthread.h;
Diffstat (limited to 'lib/librt')
-rw-r--r--lib/librt/sched.c48
1 files changed, 37 insertions, 11 deletions
diff --git a/lib/librt/sched.c b/lib/librt/sched.c
index 263bebf3a3a..e2e14bdf6f7 100644
--- a/lib/librt/sched.c
+++ b/lib/librt/sched.c
@@ -1,4 +1,4 @@
-/* $NetBSD: sched.c,v 1.1 2008/01/15 03:37:15 rmind Exp $ */
+/* $NetBSD: sched.c,v 1.2 2008/01/26 17:55:30 rmind Exp $ */
/*
* Copyright (c) 2008, Mindaugas Rasiukevicius <rmind at NetBSD org>
@@ -27,7 +27,7 @@
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: sched.c,v 1.1 2008/01/15 03:37:15 rmind Exp $");
+__RCSID("$NetBSD: sched.c,v 1.2 2008/01/26 17:55:30 rmind Exp $");
#include <string.h>
#include <unistd.h>
@@ -37,6 +37,9 @@ __RCSID("$NetBSD: sched.c,v 1.1 2008/01/15 03:37:15 rmind Exp $");
#include <sys/pset.h>
#include <sys/types.h>
+/* All LWPs in the process */
+#define P_ALL_LWPS 0
+
/*
* Scheduling parameters.
*/
@@ -44,36 +47,51 @@ __RCSID("$NetBSD: sched.c,v 1.1 2008/01/15 03:37:15 rmind Exp $");
int
sched_setparam(pid_t pid, const struct sched_param *param)
{
+ struct sched_param sp;
- return _sched_setparam(pid, 0, param);
+ memset(&sp, 0, sizeof(struct sched_param));
+ sp.sched_priority = param->sched_priority;
+ sp.sched_class = SCHED_NONE;
+ return _sched_setparam(pid, P_ALL_LWPS, &sp);
}
int
sched_getparam(pid_t pid, struct sched_param *param)
{
- return _sched_getparam(pid, 0, param);
+ return _sched_getparam(pid, P_ALL_LWPS, param);
}
int
sched_setscheduler(pid_t pid, int policy, const struct sched_param *param)
{
struct sched_param sp;
+ int ret, old_policy;
- memcpy(&sp, param, sizeof(struct sched_param));
+ ret = _sched_getparam(pid, P_ALL_LWPS, &sp);
+ if (ret < 0)
+ return ret;
+ old_policy = sp.sched_class;
+
+ memset(&sp, 0, sizeof(struct sched_param));
+ sp.sched_priority = param->sched_priority;
sp.sched_class = policy;
- return _sched_setparam(pid, 0, &sp);
+ ret = _sched_setparam(pid, P_ALL_LWPS, &sp);
+ if (ret < 0)
+ return ret;
+
+ return old_policy;
}
int
sched_getscheduler(pid_t pid)
{
struct sched_param sp;
- int error;
+ int ret;
- error = _sched_getparam(pid, 0, &sp);
- if (error)
- return error;
+ ret = _sched_getparam(pid, P_ALL_LWPS, &sp);
+ if (ret < 0)
+ return ret;
return sp.sched_class;
}
@@ -86,6 +104,10 @@ int
sched_get_priority_max(int policy)
{
+ if (policy < SCHED_OTHER || policy > SCHED_RR) {
+ errno = EINVAL;
+ return -1;
+ }
return sysconf(_SC_SCHED_PRI_MAX);
}
@@ -93,6 +115,10 @@ int
sched_get_priority_min(int policy)
{
+ if (policy < SCHED_OTHER || policy > SCHED_RR) {
+ errno = EINVAL;
+ return -1;
+ }
return sysconf(_SC_SCHED_PRI_MIN);
}
@@ -113,5 +139,5 @@ int
pset_bind(psetid_t psid, idtype_t idtype, id_t id, psetid_t *opsid)
{
- return _pset_bind(idtype, id, 0, psid, opsid);
+ return _pset_bind(idtype, id, P_ALL_LWPS, psid, opsid);
}