[Xen-devel] [PATCH v2 0/9] xen: scheduler cleanups

Juergen Gross posted 9 patches 4 years, 3 months ago
Failed in applying to current master (apply log)
MAINTAINERS                                        |   8 +-
xen/arch/arm/domain.c                              |   6 +-
xen/arch/x86/acpi/cpu_idle.c                       |  15 +-
xen/arch/x86/cpu/mwait-idle.c                      |   8 +-
xen/arch/x86/dom0_build.c                          |   5 +-
xen/common/Kconfig                                 |  66 +-----
xen/common/Makefile                                |   8 +-
xen/common/domain.c                                |  70 ------
xen/common/domctl.c                                | 135 +----------
xen/common/rcupdate.c                              |   7 +-
xen/common/sched/Kconfig                           |  65 ++++++
xen/common/sched/Makefile                          |   7 +
xen/common/{sched_arinc653.c => sched/arinc653.c}  |  15 +-
xen/common/{compat/schedule.c => sched/compat.c}   |   2 +-
xen/common/{schedule.c => sched/core.c}            | 246 ++++++++++++++++++---
xen/common/{ => sched}/cpupool.c                   |  23 +-
xen/common/{sched_credit.c => sched/credit.c}      |  65 +++---
xen/common/{sched_credit2.c => sched/credit2.c}    |  85 +++----
xen/common/{sched_null.c => sched/null.c}          | 105 ++++++---
.../xen/sched-if.h => common/sched/private.h}      |  18 +-
xen/common/{sched_rt.c => sched/rt.c}              | 109 +++++----
xen/include/xen/domain.h                           |   3 +
xen/include/xen/rcupdate.h                         |   3 -
xen/include/xen/sched.h                            |  39 ++--
24 files changed, 568 insertions(+), 545 deletions(-)
create mode 100644 xen/common/sched/Kconfig
create mode 100644 xen/common/sched/Makefile
rename xen/common/{sched_arinc653.c => sched/arinc653.c} (99%)
rename xen/common/{compat/schedule.c => sched/compat.c} (97%)
rename xen/common/{schedule.c => sched/core.c} (92%)
rename xen/common/{ => sched}/cpupool.c (97%)
rename xen/common/{sched_credit.c => sched/credit.c} (97%)
rename xen/common/{sched_credit2.c => sched/credit2.c} (98%)
rename xen/common/{sched_null.c => sched/null.c} (92%)
rename xen/{include/xen/sched-if.h => common/sched/private.h} (96%)
rename xen/common/{sched_rt.c => sched/rt.c} (94%)
[Xen-devel] [PATCH v2 0/9] xen: scheduler cleanups
Posted by Juergen Gross 4 years, 3 months ago
Move all scheduler related hypervisor code to xen/common/sched/ and
do a lot of cleanups.

Juergen Gross (9):
  xen/sched: move schedulers and cpupool coding to dedicated directory
  xen/sched: make sched-if.h really scheduler private
  xen/sched: cleanup sched.h
  xen/sched: remove special cases for free cpus in schedulers
  xen/sched: use scratch cpumask instead of allocating it on the stack
  xen/sched: replace null scheduler percpu-variable with pdata hook
  xen/sched: switch scheduling to bool where appropriate
  xen/sched: eliminate sched_tick_suspend() and sched_tick_resume()
  xen/sched: add const qualifier where appropriate

 MAINTAINERS                                        |   8 +-
 xen/arch/arm/domain.c                              |   6 +-
 xen/arch/x86/acpi/cpu_idle.c                       |  15 +-
 xen/arch/x86/cpu/mwait-idle.c                      |   8 +-
 xen/arch/x86/dom0_build.c                          |   5 +-
 xen/common/Kconfig                                 |  66 +-----
 xen/common/Makefile                                |   8 +-
 xen/common/domain.c                                |  70 ------
 xen/common/domctl.c                                | 135 +----------
 xen/common/rcupdate.c                              |   7 +-
 xen/common/sched/Kconfig                           |  65 ++++++
 xen/common/sched/Makefile                          |   7 +
 xen/common/{sched_arinc653.c => sched/arinc653.c}  |  15 +-
 xen/common/{compat/schedule.c => sched/compat.c}   |   2 +-
 xen/common/{schedule.c => sched/core.c}            | 246 ++++++++++++++++++---
 xen/common/{ => sched}/cpupool.c                   |  23 +-
 xen/common/{sched_credit.c => sched/credit.c}      |  65 +++---
 xen/common/{sched_credit2.c => sched/credit2.c}    |  85 +++----
 xen/common/{sched_null.c => sched/null.c}          | 105 ++++++---
 .../xen/sched-if.h => common/sched/private.h}      |  18 +-
 xen/common/{sched_rt.c => sched/rt.c}              | 109 +++++----
 xen/include/xen/domain.h                           |   3 +
 xen/include/xen/rcupdate.h                         |   3 -
 xen/include/xen/sched.h                            |  39 ++--
 24 files changed, 568 insertions(+), 545 deletions(-)
 create mode 100644 xen/common/sched/Kconfig
 create mode 100644 xen/common/sched/Makefile
 rename xen/common/{sched_arinc653.c => sched/arinc653.c} (99%)
 rename xen/common/{compat/schedule.c => sched/compat.c} (97%)
 rename xen/common/{schedule.c => sched/core.c} (92%)
 rename xen/common/{ => sched}/cpupool.c (97%)
 rename xen/common/{sched_credit.c => sched/credit.c} (97%)
 rename xen/common/{sched_credit2.c => sched/credit2.c} (98%)
 rename xen/common/{sched_null.c => sched/null.c} (92%)
 rename xen/{include/xen/sched-if.h => common/sched/private.h} (96%)
 rename xen/common/{sched_rt.c => sched/rt.c} (94%)

-- 
2.16.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH v2 0/9] xen: scheduler cleanups
Posted by Dario Faggioli 4 years, 3 months ago
On Wed, 2020-01-08 at 16:23 +0100, Juergen Gross wrote:
> Move all scheduler related hypervisor code to xen/common/sched/ and
> do a lot of cleanups.
> 
> Juergen Gross (9):
>   xen/sched: move schedulers and cpupool coding to dedicated
> directory
>   xen/sched: make sched-if.h really scheduler private
>   xen/sched: cleanup sched.h
>   xen/sched: remove special cases for free cpus in schedulers
>   xen/sched: use scratch cpumask instead of allocating it on the
> stack
>   xen/sched: replace null scheduler percpu-variable with pdata hook
>   xen/sched: switch scheduling to bool where appropriate
>   xen/sched: eliminate sched_tick_suspend() and sched_tick_resume()
>   xen/sched: add const qualifier where appropriate
>
Ok, unless I'm missing something, I think that "scheduling-wise" this
series if fully Rev/Acked-by.

Thanks Juergen for the cleanups. The code looks a lot better with this
patches applied! :-)

Regards
-- 
Dario Faggioli, Ph.D
http://about.me/dario.faggioli
Virtualization Software Engineer
SUSE Labs, SUSE https://www.suse.com/
-------------------------------------------------------------------
<<This happens because _I_ choose it to happen!>> (Raistlin Majere)

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel