On Thu, Jan 14, 2016 at 04:24:50PM +0100, Luca Abeni wrote:Ah, thanks! I did not know that WARN_ON() returns a value...
+ if (dl_rq->running_bw > dl_rq->this_bw) {
+ WARN_ON(1);
+ dl_rq->running_bw = dl_rq->this_bw;
+ }
FWIW you can write this as:
if (WARN_ON(dl_rq->running_bw > dl_rq->this_bw))
dl_rq->running_bw = dl_rq->this_bw;