From: Dario Faggioli <dario.faggioli@citrix.com>
To: xen-devel@lists.xenproject.org
Cc: George Dunlap <george.dunlap@eu.citrix.com>,
Andrew Cooper <andrew.cooper3@citrix.com>
Subject: [PATCH v2 4/7] xen: get rid of the SEDF scheduler
Date: Fri, 03 Jul 2015 12:25:00 +0200 [thread overview]
Message-ID: <20150703102500.5144.61070.stgit@Solace.station> (raw)
In-Reply-To: <20150703101558.5144.46511.stgit@Solace.station>
more specifically, of all the symbols and references
to it.
Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
Reviewed-by: George Dunlap <george.dunlap@eu.citrix.com>
---
Cc: Andrew Cooper <andrew.cooper3@citrix.com>
Changes from v1:
- added markers for removed #defines, to help keep bits
of the history, and reduce the likelyhood of accidental
reuse, as requested during review.
---
docs/misc/xen-command-line.markdown | 2 +-
xen/common/Makefile | 1 -
xen/common/schedule.c | 1 -
xen/include/public/domctl.h | 9 +--------
xen/include/public/trace.h | 2 +-
xen/include/xen/sched-if.h | 1 -
6 files changed, 3 insertions(+), 13 deletions(-)
diff --git a/docs/misc/xen-command-line.markdown b/docs/misc/xen-command-line.markdown
index aa684c0..d292fae 100644
--- a/docs/misc/xen-command-line.markdown
+++ b/docs/misc/xen-command-line.markdown
@@ -1206,7 +1206,7 @@ Map the HPET page as read only in Dom0. If disabled the page will be mapped
with read and write permissions.
### sched
-> `= credit | credit2 | sedf | arinc653`
+> `= credit | credit2 | arinc653`
> Default: `sched=credit`
diff --git a/xen/common/Makefile b/xen/common/Makefile
index 1cddebc..3fdf931 100644
--- a/xen/common/Makefile
+++ b/xen/common/Makefile
@@ -31,7 +31,6 @@ obj-y += rbtree.o
obj-y += rcupdate.o
obj-y += sched_credit.o
obj-y += sched_credit2.o
-obj-y += sched_sedf.o
obj-y += sched_arinc653.o
obj-y += sched_rt.o
obj-y += schedule.o
diff --git a/xen/common/schedule.c b/xen/common/schedule.c
index ecf1545..0dc2788 100644
--- a/xen/common/schedule.c
+++ b/xen/common/schedule.c
@@ -65,7 +65,6 @@ DEFINE_PER_CPU(struct schedule_data, schedule_data);
DEFINE_PER_CPU(struct scheduler *, scheduler);
static const struct scheduler *schedulers[] = {
- &sched_sedf_def,
&sched_credit_def,
&sched_credit2_def,
&sched_arinc653_def,
diff --git a/xen/include/public/domctl.h b/xen/include/public/domctl.h
index bc45ea5..6d28489 100644
--- a/xen/include/public/domctl.h
+++ b/xen/include/public/domctl.h
@@ -324,7 +324,7 @@ DEFINE_XEN_GUEST_HANDLE(xen_domctl_max_vcpus_t);
/* XEN_DOMCTL_scheduler_op */
/* Scheduler types. */
-#define XEN_SCHEDULER_SEDF 4
+/* #define XEN_SCHEDULER_SEDF 4 (Removed) */
#define XEN_SCHEDULER_CREDIT 5
#define XEN_SCHEDULER_CREDIT2 6
#define XEN_SCHEDULER_ARINC653 7
@@ -337,13 +337,6 @@ struct xen_domctl_scheduler_op {
uint32_t sched_id; /* XEN_SCHEDULER_* */
uint32_t cmd; /* XEN_DOMCTL_SCHEDOP_* */
union {
- struct xen_domctl_sched_sedf {
- uint64_aligned_t period;
- uint64_aligned_t slice;
- uint64_aligned_t latency;
- uint32_t extratime;
- uint32_t weight;
- } sedf;
struct xen_domctl_sched_credit {
uint16_t weight;
uint16_t cap;
diff --git a/xen/include/public/trace.h b/xen/include/public/trace.h
index 5211ae7..274f8f6 100644
--- a/xen/include/public/trace.h
+++ b/xen/include/public/trace.h
@@ -75,7 +75,7 @@
/* Per-scheduler IDs, to identify scheduler specific events */
#define TRC_SCHED_CSCHED 0
#define TRC_SCHED_CSCHED2 1
-#define TRC_SCHED_SEDF 2
+/* #define XEN_SCHEDULER_SEDF 2 (Removed) */
#define TRC_SCHED_ARINC653 3
#define TRC_SCHED_RTDS 4
diff --git a/xen/include/xen/sched-if.h b/xen/include/xen/sched-if.h
index 7cc25c6..dbe7cab 100644
--- a/xen/include/xen/sched-if.h
+++ b/xen/include/xen/sched-if.h
@@ -165,7 +165,6 @@ struct scheduler {
void (*tick_resume) (const struct scheduler *, unsigned int);
};
-extern const struct scheduler sched_sedf_def;
extern const struct scheduler sched_credit_def;
extern const struct scheduler sched_credit2_def;
extern const struct scheduler sched_arinc653_def;
next prev parent reply other threads:[~2015-07-03 10:25 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-03 10:24 [PATCH v2 0/7] get rid of the SEDF scheduler Dario Faggioli
2015-07-03 10:24 ` [PATCH v2 1/7] libxl: " Dario Faggioli
2015-07-03 14:23 ` Ian Campbell
2015-07-03 14:33 ` Dario Faggioli
2015-07-03 14:39 ` Ian Campbell
2015-07-03 14:50 ` Ian Jackson
2015-07-03 14:49 ` Ian Jackson
2015-07-03 10:24 ` [PATCH v2 2/7] tools: python: get rid of the SEDF scheduler bindings Dario Faggioli
2015-07-03 14:23 ` Ian Campbell
2015-07-03 10:24 ` [PATCH v2 3/7] libxc: get rid of the SEDF scheduler Dario Faggioli
2015-07-03 14:23 ` Ian Campbell
2015-07-03 10:25 ` Dario Faggioli [this message]
2015-07-03 10:30 ` [PATCH v2 4/7] xen: " Andrew Cooper
2015-07-03 13:26 ` Dario Faggioli
2015-07-03 10:25 ` [PATCH v2 5/7] xen: kill sched_sedf.c Dario Faggioli
2015-07-03 10:25 ` [PATCH v2 6/7] xl: get rid of the SEDF scheduler Dario Faggioli
2015-07-03 14:26 ` Ian Campbell
2015-07-03 14:36 ` Dario Faggioli
2015-07-03 14:42 ` Ian Campbell
2015-07-03 10:25 ` [PATCH v2 7/7] docs: " Dario Faggioli
2015-07-03 14:26 ` Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150703102500.5144.61070.stgit@Solace.station \
--to=dario.faggioli@citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=george.dunlap@eu.citrix.com \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).