diff options
author | Andrew Cooper <andrew.cooper3@citrix.com> | 2013-09-13 14:30:02 +0200 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2013-09-13 14:30:02 +0200 |
commit | 4c57b3fbdf2706a9441680cdae652c959b40dffe (patch) | |
tree | ac89d429c880b77f5d7715585a9bfcfcfda9a328 /xen/common | |
parent | c71a3e6d83fb6d03eeae8df08eafd4ea02d9e278 (diff) | |
download | xen-4c57b3fbdf2706a9441680cdae652c959b40dffe.tar.gz xen-4c57b3fbdf2706a9441680cdae652c959b40dffe.tar.bz2 xen-4c57b3fbdf2706a9441680cdae652c959b40dffe.zip |
sched/credit: Remove redundant assignments from alloc_* functions
Noticed because Coverity was complaining at the atomic_set(), but because of
the use of xzalloc(), these assignments of 0 are completely redundent.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Acked-by: George Dunlap <george.dunlap@eu.citrix.com>
Diffstat (limited to 'xen/common')
-rw-r--r-- | xen/common/sched_credit.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/xen/common/sched_credit.c b/xen/common/sched_credit.c index 0b816629e6..dbe6de666b 100644 --- a/xen/common/sched_credit.c +++ b/xen/common/sched_credit.c @@ -858,8 +858,6 @@ csched_alloc_vdata(const struct scheduler *ops, struct vcpu *vc, void *dd) INIT_LIST_HEAD(&svc->active_vcpu_elem); svc->sdom = dd; svc->vcpu = vc; - atomic_set(&svc->credit, 0); - svc->flags = 0U; svc->pri = is_idle_domain(vc->domain) ? CSCHED_PRI_IDLE : CSCHED_PRI_TS_UNDER; SCHED_VCPU_STATS_RESET(svc); @@ -1081,11 +1079,9 @@ csched_alloc_domdata(const struct scheduler *ops, struct domain *dom) /* Initialize credit and weight */ INIT_LIST_HEAD(&sdom->active_vcpu); - sdom->active_vcpu_count = 0; INIT_LIST_HEAD(&sdom->active_sdom_elem); sdom->dom = dom; sdom->weight = CSCHED_DEFAULT_WEIGHT; - sdom->cap = 0U; return (void *)sdom; } |