Skip to content

xen: sched: fix locking of {insert, remove}_vcpu()

Dario Faggioli xen-devel 4 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest6 patches 10 repliesIn review
  2. v36 patches 14 repliesSuperseded
  3. v26 patches 7 repliesSuperseded
Earlier versions · 1
VersionPostedPatchesRepliesStatus
v17 patches21 repliesSuperseded

Patches in v4 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6xen: sched: fix locking of remove_vcpu() in credit1xen/common/sched_credit.c+4 -71 reply
2/6xen: sched: fix locking for insert_vcpu() in credit1 and RTDSxen/common/sched_credit.c xen/common/sched_rt.c xen/common/schedule.c+11 -41 reply
3/6xen: sched: clarify use cases of schedule_cpu_switch()xen/common/cpupool.c xen/common/schedule.c+30 -80 replies
4/6xen: sched: better handle (not) inserting idle vCPUs in runqueuesxen/common/sched_credit.c xen/common/sched_credit2.c xen/common/sched_rt.c +1 more+26 -300 replies
5/6xen: sched: get rid of the per domain vCPU list in RTDSxen/common/sched_rt.c+26 -280 replies
6/6xen: sched: get rid of the per domain vCPU list in Credit2xen/common/sched_credit2.c+11 -230 replies

Reviews trailers per patch

All 6 patches have a Reviewed-by or Acked-by.

PatchMeng XuJuergen GrossGeorge Dunlap
1/6R-b
2/6R-b R-b
3/6A-b R-b
4/6R-b A-b
5/6R-b
6/6A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v4 thread

Cover letter [PATCH v4 0/6] xen: sched: fix locking of {insert, remove}_vcpu()

Hi,

This series, improves how inserting vCPUs in schedulers runqueues is done, including fixing a couple of bugs, and paving the way for more improvement in Credit2 runqueue handling (will be submitted as a separate series).

v3 is here:
http://lists.xenproject.org/archives/html/xen-devel/2015-10/msg03278.html
Message-Id: <[email protected]>

Only patch 1 really changed from v3, and patches 1 and 2 are the only one that seem to me to be missing suitable Ack-s.

There is a git branch for this series here:
git://xenbits.xen.org/people/dariof/xen.git rel/sched/fix-vcpu-ins-rem-v4

Regards,
Dario
---
Dario Faggioli (6):
xen: sched: fix locking of remove_vcpu() in credit1
xen: sched: fix locking for insert_vcpu() in credit1 and RTDS
xen: sched: clarify use cases of schedule_cpu_switch()
xen: sched: better handle (not) inserting idle vCPUs in runqueues
xen: sched: get rid of the per domain vCPU list in RTDS
xen: sched: get rid of the per domain vCPU list in Credit2

 xen/common/cpupool.c       |    7 -----
 xen/common/sched_credit.c  |   18 ++++++++-----
 xen/common/sched_credit2.c |   55 ++++++++++++++--------------------------
 xen/common/sched_rt.c      |   61 ++++++++++++++++++++++----------------------
 xen/common/schedule.c      |   57 +++++++++++++++++++++++++++++++----------
 5 files changed, 103 insertions(+), 95 deletions(-)

--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)