From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751752AbaKTLBF (ORCPT ); Thu, 20 Nov 2014 06:01:05 -0500 Received: from mga02.intel.com ([134.134.136.20]:50364 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750776AbaKTLBD (ORCPT ); Thu, 20 Nov 2014 06:01:03 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.07,423,1413270000"; d="scan'208";a="640336933" From: Wanpeng Li To: Ingo Molnar , Peter Zijlstra Cc: Juri Lelli , Kirill Tkhai , linux-kernel@vger.kernel.org, Wanpeng Li Subject: [PATCH] sched/deadline: fix dl entity is still mark yield after replenishing Date: Thu, 20 Nov 2014 19:00:48 +0800 Message-Id: <1416481248-38998-1-git-send-email-wanpeng.li@linux.intel.com> X-Mailer: git-send-email 1.7.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Yield task semantic for deadline task is get off from the CPU until our next instance, we mark the task got to sleep until its current deadline by forcing its runtime to zero and update_curr_dl() stops it and the bandwidth timer will wake it up and will give it new scheduling parameters. Bandwidth timer may fail to start in update_curr_dl() and instead replenishing the budget immediately, however, dl_yielded of dl_se is cleared when bandwidth timer fire, the immediate replenishing miss to reset it. This patch fix it by reseting the dl_yielded flag after the immediate replenishing if fail to start bandwidth timer. Signed-off-by: Wanpeng Li --- kernel/sched/deadline.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/kernel/sched/deadline.c b/kernel/sched/deadline.c index 981479a..9d3e31f 100644 --- a/kernel/sched/deadline.c +++ b/kernel/sched/deadline.c @@ -633,8 +633,11 @@ static void update_curr_dl(struct rq *rq) __dequeue_task_dl(rq, curr, 0); if (likely(start_dl_timer(dl_se, curr->dl.dl_boosted))) dl_se->dl_throttled = 1; - else + else { enqueue_task_dl(rq, curr, ENQUEUE_REPLENISH); + if (unlikely(dl_se->dl_yielded)) + dl_se->dl_yielded = 0; + } if (!is_leftmost(curr, &rq->dl)) resched_curr(rq); -- 1.9.1