diff options
| author | John Baldwin <jhb@FreeBSD.org> | 2010-12-20 17:08:22 +0000 |
|---|---|---|
| committer | John Baldwin <jhb@FreeBSD.org> | 2010-12-20 17:08:22 +0000 |
| commit | a5ba914f270373007534073941cc48241f02b515 (patch) | |
| tree | 5e04588835f9cc096181a3042ba194907d2a9a13 | |
| parent | e9e9c992e114c0d57076168dfa8637c33b9f4000 (diff) | |
Notes
| -rw-r--r-- | sys/kern/kern_resource.c | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/sys/kern/kern_resource.c b/sys/kern/kern_resource.c index c693d8340b9d..336538218953 100644 --- a/sys/kern/kern_resource.c +++ b/sys/kern/kern_resource.c @@ -472,29 +472,27 @@ rtp_to_pri(struct rtprio *rtp, struct thread *td) u_char newpri; u_char oldpri; - thread_lock(td); switch (RTP_PRIO_BASE(rtp->type)) { case RTP_PRIO_REALTIME: - if (rtp->prio > RTP_PRIO_MAX) { - thread_unlock(td); + if (rtp->prio > RTP_PRIO_MAX) return (EINVAL); - } newpri = PRI_MIN_REALTIME + rtp->prio; break; case RTP_PRIO_NORMAL: - if (rtp->prio > (PRI_MAX_TIMESHARE - PRI_MIN_TIMESHARE)) { - thread_unlock(td); + if (rtp->prio > (PRI_MAX_TIMESHARE - PRI_MIN_TIMESHARE)) return (EINVAL); - } newpri = PRI_MIN_TIMESHARE + rtp->prio; break; case RTP_PRIO_IDLE: + if (rtp->prio > RTP_PRIO_MAX) + return (EINVAL); newpri = PRI_MIN_IDLE + rtp->prio; break; default: - thread_unlock(td); return (EINVAL); } + + thread_lock(td); sched_class(td, rtp->type); /* XXX fix */ oldpri = td->td_user_pri; sched_user_prio(td, newpri); |