From: Juergen Gross <jgross@suse.com>
Date: Tue, 11 Feb 2020 09:44:18 +0000 (+0100)
Subject: xen/sched: remove pointless BUG_ON() in credit2
X-Git-Tag: archive/raspbian/4.14.0+80-gd101b417b7-1+rpi1^2~63^2~716
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=6bb06bb0eb1528319998de602f969c474396c306;p=xen.git

xen/sched: remove pointless BUG_ON() in credit2

The BUG_ON() at the top of csched2_context_saved() is completely
pointless, as the ASSERT() just following it catches the same problem
already.

Signed-off-by: Juergen Gross <jgross@suse.com>
Reviewed-by: Dario Faggioli <dfaggioli@suse.com>
---

diff --git a/xen/common/sched/credit2.c b/xen/common/sched/credit2.c
index 231f87d960..f2752f27e2 100644
--- a/xen/common/sched/credit2.c
+++ b/xen/common/sched/credit2.c
@@ -2167,8 +2167,6 @@ csched2_context_saved(const struct scheduler *ops, struct sched_unit *unit)
     s_time_t now = NOW();
     LIST_HEAD(were_parked);
 
-    BUG_ON( !is_idle_unit(unit) &&
-            svc->rqd != c2rqd(ops, sched_unit_master(unit)));
     ASSERT(is_idle_unit(unit) ||
            svc->rqd == c2rqd(ops, sched_unit_master(unit)));