[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen stable-4.5] credit: update timeslice under lock
commit 3fa5fb56a96253f4fe1efdce235bb783001138ea Author: Juergen Gross <jgross@xxxxxxxx> AuthorDate: Wed Feb 17 16:48:16 2016 +0100 Commit: Jan Beulich <jbeulich@xxxxxxxx> CommitDate: Wed Feb 17 16:48:16 2016 +0100 credit: update timeslice under lock When updating the timeslice of the credit scheduler protect the scheduler's private data by it's lock. Today a possible race could result only in some weird scheduling decisions during one timeslice, but further adjustments will need the lock anyway. Signed-off-by: Juergen Gross <jgross@xxxxxxxx> Reviewed-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> master commit: f2c96ac4dedf4976e46de34c69c2cd8b289c4ef2 master date: 2016-02-02 14:03:06 +0100 --- xen/common/sched_credit.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/xen/common/sched_credit.c b/xen/common/sched_credit.c index 2a3a67d..0e61a0e 100644 --- a/xen/common/sched_credit.c +++ b/xen/common/sched_credit.c @@ -1080,6 +1080,7 @@ csched_sys_cntl(const struct scheduler *ops, int rc = -EINVAL; xen_sysctl_credit_schedule_t *params = &sc->u.sched_credit; struct csched_private *prv = CSCHED_PRIV(ops); + unsigned long flags; switch ( sc->cmd ) { @@ -1091,8 +1092,12 @@ csched_sys_cntl(const struct scheduler *ops, || params->ratelimit_us < XEN_SYSCTL_SCHED_RATELIMIT_MIN)) || MICROSECS(params->ratelimit_us) > MILLISECS(params->tslice_ms) ) goto out; + + spin_lock_irqsave(&prv->lock, flags); __csched_set_tslice(prv, params->tslice_ms); prv->ratelimit_us = params->ratelimit_us; + spin_unlock_irqrestore(&prv->lock, flags); + /* FALLTHRU */ case XEN_SYSCTL_SCHEDOP_getinfo: params->tslice_ms = prv->tslice_ms; -- generated by git-patchbot for /home/xen/git/xen.git#stable-4.5 _______________________________________________ Xen-changelog mailing list Xen-changelog@xxxxxxxxxxxxx http://lists.xensource.com/xen-changelog
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |