diff options
author | Keir Fraser <keir@xen.org> | 2010-10-24 13:09:50 +0100 |
---|---|---|
committer | Keir Fraser <keir@xen.org> | 2010-10-24 13:09:50 +0100 |
commit | 7e6b926a17db1d63f1b8fd2745776f5369dacb4e (patch) | |
tree | 753bf45413056dc82c8d309d919a048149ff802b /xen/common/sched_sedf.c | |
parent | 8e5c88afb0fb552ba803fecc1952d01293442d7d (diff) | |
download | xen-7e6b926a17db1d63f1b8fd2745776f5369dacb4e.tar.gz xen-7e6b926a17db1d63f1b8fd2745776f5369dacb4e.tar.bz2 xen-7e6b926a17db1d63f1b8fd2745776f5369dacb4e.zip |
cpupools: Make interface more consistent
The current cpupools code interface is a bit inconsistent. This
patch addresses this by making the interaction for each
vcpu in a pool look like this:
alloc_vdata() -- allocates and sets up vcpu data
insert_vcpu() -- the vcpu is ready to run in this pool
remove_vcpu() -- take the vcpu out of the pool
free_vdata() -- delete allocated vcpu data
(Previously, remove_vcpu and free_vdata were combined into a "destroy
vcpu", and insert_vcpu was only called for idle vcpus.)
This also addresses a bug in credit2 which was caused by a
misunderstanding of the cpupools interface.
Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
Acked-by: Juergen Gross <juergen.gross@ts.fujitsu.com>
Diffstat (limited to 'xen/common/sched_sedf.c')
-rw-r--r-- | xen/common/sched_sedf.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/xen/common/sched_sedf.c b/xen/common/sched_sedf.c index be71432de9..7dde35b912 100644 --- a/xen/common/sched_sedf.c +++ b/xen/common/sched_sedf.c @@ -410,11 +410,6 @@ static void sedf_free_vdata(const struct scheduler *ops, void *priv) xfree(priv); } -static void sedf_destroy_vcpu(const struct scheduler *ops, struct vcpu *v) -{ - sedf_free_vdata(ops, v->sched_priv); -} - static void * sedf_alloc_domdata(const struct scheduler *ops, struct domain *d) { @@ -1504,8 +1499,6 @@ const struct scheduler sched_sedf_def = { .init_domain = sedf_init_domain, .destroy_domain = sedf_destroy_domain, - .destroy_vcpu = sedf_destroy_vcpu, - .alloc_vdata = sedf_alloc_vdata, .free_vdata = sedf_free_vdata, .alloc_pdata = sedf_alloc_pdata, |