Received: by 2002:a05:7412:8d10:b0:f3:1519:9f41 with SMTP id bj16csp5943580rdb; Thu, 14 Dec 2023 04:23:58 -0800 (PST) X-Google-Smtp-Source: AGHT+IFxV/3mEBcVZvp8RbqjdJGI3ZN3mgXP96AsWOTfR1XKROov5jwlhOi3Aa2XTiGzhT3NlTkA X-Received: by 2002:a05:6a20:da92:b0:18c:c37:35d4 with SMTP id iy18-20020a056a20da9200b0018c0c3735d4mr12288996pzb.14.1702556638431; Thu, 14 Dec 2023 04:23:58 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1702556638; cv=none; d=google.com; s=arc-20160816; b=LE9OGOj/DwtxRftQB2syV7LVBoo07HicTAk5cJJbN7lBOSz37ucUcypQIctg7wJaox e8bhOryw55xa5jyZpjZP/BC/jrZ7b/+Lw6xZwgvKATIZq63M4JMeHSjoRmd+rfVzmhmJ FSCsW90RFWFVqqN3Qf5EaW+ma6m6H5m74NfJ4NYP5SD0umShiadZKwnDD/Z3UIrLpOM0 9F7cMCY0ZcsvP60gnCSCjO0bBDVTjm+5OKa6rZyo0ybEUqpU7808eX3PlqjnCWU6JmCO x6++CtX9bzSpua4w6jdU3Tn/FdO29lF/VHuUjTBc/IHCXBFJlZpYQCMcHQC9llk8KO/u f1kQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=rs1Z6ikTJBs80pmW9mo6L5GeJM722tcA3lmeh1E5sKg=; fh=BQm9TdLiXTBNwf6HUFTlt1Y0iCUx6GcoPe6CNqzVA1c=; b=ur3HBKtmJ5/FLRtNs4OHAFj8AVCiF95/LVOYvOXvwDdPmZgtUS2T1d5/G3ETLHCq67 cetMpOhYMR08zkNVW0sdZjpDlIQbt+E3+r/GHJFng5DMaEL74yZu+UF0F1ksfnF8WlIg SnRMvcol5DklmhdXE5wNGkwV/38jUx/jy4jnOA8tp/FHgXIkB+RojjkKLJwu4uQecIpR RdMFmDdvXZ1hJv9CLBSZRBqAAHwjl0veVDUUX+bMKPmhM/rtQvQLQkskfOigz5NqvfEo 6S4ZDfSeTZTa1dO+OVcuS8gE5dM3zBW5odeuKihCe2J43X3Nvy1NBdvNDDV9SilI1EQ0 vQoA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@bytedance.com header.s=google header.b=TTlLgdnO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=bytedance.com Return-Path: Received: from lipwig.vger.email (lipwig.vger.email. [23.128.96.33]) by mx.google.com with ESMTPS id f20-20020a056a00239400b006d0c02f0070si3289447pfc.353.2023.12.14.04.23.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 14 Dec 2023 04:23:58 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33; Authentication-Results: mx.google.com; dkim=pass header.i=@bytedance.com header.s=google header.b=TTlLgdnO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=bytedance.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id 17F1D81552E9; Thu, 14 Dec 2023 04:23:56 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1572993AbjLNMXm (ORCPT + 99 others); Thu, 14 Dec 2023 07:23:42 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56364 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1572978AbjLNMXl (ORCPT ); Thu, 14 Dec 2023 07:23:41 -0500 Received: from mail-pf1-x432.google.com (mail-pf1-x432.google.com [IPv6:2607:f8b0:4864:20::432]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7446711B for ; Thu, 14 Dec 2023 04:23:23 -0800 (PST) Received: by mail-pf1-x432.google.com with SMTP id d2e1a72fcca58-6d26dabc560so63527b3a.1 for ; Thu, 14 Dec 2023 04:23:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bytedance.com; s=google; t=1702556603; x=1703161403; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=rs1Z6ikTJBs80pmW9mo6L5GeJM722tcA3lmeh1E5sKg=; b=TTlLgdnOLMr5vEES3IAN0dP/ZiUkh+4bm19FC9mClvN2aX+ccFP3ORkJ9sHPYkAnq/ DkkZ92/slrv1QjhjBIf6ptPWkcpDQZZ35yw3usnnT0dGEU3pfDrAm8k1zlnr5kc1PaCT 8ZcOyj4Q1EHyTYjZF1aEeX1RoaGazPrsRTI6CJpyHPFx/xbRiTaaRqE3IcQxbmha6Jyr BSDqIaC2OY8x+T07wHPBrE2UhFaLmivmP0KZ1MRh9m4BcCSKma46S5fK8XDX2J4CpHkW RiFAgoro1kjgoYXLLz70naMccwfwaxHjFEWe9d+5mOG8PW44ftlFDsqRA0m2opKKa40H LGyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702556603; x=1703161403; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=rs1Z6ikTJBs80pmW9mo6L5GeJM722tcA3lmeh1E5sKg=; b=flrMbKk1HE96TpvSSTUsJ4UKVe7FYVAmPrHXuxWvQThzAUXxbfpFIHgA9KfHzRkj/4 7nykFSYz9qYB3QtVzo0qZPu0xmnuinQgSZZ95OJ0IsNP06MBhNj1ab772OaMBG8yI4wk 8fxbBcWRtm8yp/+SsQ/LZ+qnRZUVhOMg92lVQQdB9sdqv7FA5j3z+FoN6YheIgU+I5U4 rAtsZ2wZ2kZ7oJuWCw3Js5vd7Yjpu+pRGe46qQVKupabAVx5eS6TpJWI6whAiy3kcmhj JFtsitMGczb1cPwxP5kR/4g7JYgmx/WVT1e1jwHychuv3HXujC0nF+DfySivdoXGN0Lg F6DA== X-Gm-Message-State: AOJu0YwHcOH35D7wVpWwc7K8vOCaGzlEVoZMxC9VBF9Wegjnj7zWfFGl JvkiXwjbeY10k+dmeUKjEKgrxA== X-Received: by 2002:a05:6a20:158b:b0:190:1246:7d23 with SMTP id h11-20020a056a20158b00b0019012467d23mr12853669pzj.55.1702556602835; Thu, 14 Dec 2023 04:23:22 -0800 (PST) Received: from [10.254.92.96] ([139.177.225.240]) by smtp.gmail.com with ESMTPSA id 23-20020a17090a199700b0028afd8b1e0bsm2009452pji.57.2023.12.14.04.23.10 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 14 Dec 2023 04:23:22 -0800 (PST) Message-ID: <83099016-f609-4ac0-94ab-909ac9c96582@bytedance.com> Date: Thu, 14 Dec 2023 20:21:53 +0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: Re: [PATCH] sched/fair: remove next_buddy_marked Content-Language: en-US To: Vincent Guittot , Wang Jinchao Cc: Ingo Molnar , Peter Zijlstra , Juri Lelli , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Daniel Bristot de Oliveira , Valentin Schneider , linux-kernel@vger.kernel.org, stone.xulei@xfusion.com References: <202312141319+0800-wangjinchao@xfusion.com> From: Abel Wu In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Thu, 14 Dec 2023 04:23:56 -0800 (PST) On 12/14/23 4:18 PM, Vincent Guittot Wrote: > On Thu, 14 Dec 2023 at 06:20, Wang Jinchao wrote: >> >> Remove unused `next_buddy_marked` in `check_preempt_wakeup_fair` >> > > Fixes: 5e963f2bd465 ("sched/fair: Commit to EEVDF") After this commit @pse preempts curr without being the NEXT_BUDDY, but IMHO it should be, so how about this? @@ -8259,8 +8259,11 @@ static void check_preempt_wakeup_fair(struct rq *rq, struct task_struct *p, int /* * XXX pick_eevdf(cfs_rq) != se ? */ - if (pick_eevdf(cfs_rq) == pse) + if (pick_eevdf(cfs_rq) == pse) { + if (!next_buddy_marked) + set_next_buddy(pse); goto preempt; + } return; which will align with before.