Received: by 2002:a05:7412:8d10:b0:f3:1519:9f41 with SMTP id bj16csp5170553rdb; Wed, 13 Dec 2023 00:25:00 -0800 (PST) X-Google-Smtp-Source: AGHT+IEf5qS29fSHeblPcrxut3GTSeUefxml7hHfrBrvkT+k7/Kmq1LQK0Pl2/8bbQmjp219NVSm X-Received: by 2002:a05:6808:1390:b0:3af:9838:8494 with SMTP id c16-20020a056808139000b003af98388494mr9585340oiw.20.1702455900615; Wed, 13 Dec 2023 00:25:00 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1702455900; cv=none; d=google.com; s=arc-20160816; b=g+FP8xzaqfAh+rMUY5umH/z2SykgayHtD8cmXjCtvPniHceqZgpzi3htwTHgl/tpWw uUjEEj0kZhQEjfKbtaPN0qHcS6Lcn2R5XhobxkNCxE3/LLSsGMFsFqGfIx6YQvPh12eY sObSIu/TtZJ47o3yYCn613/nwuDHxegX+EeB45IuleGjq8KbCbyKndvv2WvKMXHw6xoL pq/sYY872ij+fw7U4JvRLVuv5UbJRmuZVzpbNL9xlCQJr1MY35JKaWSDxESh9wk1asuT 1iRpjSjTuDTeyagf1RTwquwRkF+X6BArKIKisUg5fh2L2ZXmP4crZWSH8zm/CdNY7KCm u4WA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=6v1Ba8YaX+pMQZF3in4IJSoafdVQFQX6Sy/KwMfRTv8=; fh=vUaF4fVu5z/SZWzFFr8Beq+3MOHTFGFXBBLRlolRwlg=; b=tBl7CU8FnQcHeHb03PJ4rzShNAclucQUA5n6WAJxQWyrnkxglxkUQK71EdEXxECJQ9 glPJ2FR831h5FlhPLi9j2Gnh6XL+Ss+gLpnkz1e8hL1KlqxQoXedPACuJXRCEdxqbBAc ddC36LukIcHNCz45pgSUURDnWHx+QaNgI5OLkr3Ms4S2LVsGbYxNQ2eMfhEKJHCpTtt0 OmslzUTSRo2RHGlj2bjekBmUwr3V3suaMYP8zlOojJtxtz00aopl5X9pJaFZ7rh440pK t0Rdt5De2mXN23s2A5uP2qCU4+5ZATDlujs0QsN/A5JXoWzStJmk1GAbx0I/6j330oXM wDYg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=xemkWMut; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from fry.vger.email (fry.vger.email. [2620:137:e000::3:8]) by mx.google.com with ESMTPS id c13-20020a631c4d000000b005c65fe4373fsi8966415pgm.315.2023.12.13.00.25.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 13 Dec 2023 00:25:00 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=xemkWMut; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by fry.vger.email (Postfix) with ESMTP id 1F50E80AE228; Wed, 13 Dec 2023 00:24:58 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at fry.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231226AbjLMIYm (ORCPT + 99 others); Wed, 13 Dec 2023 03:24:42 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42080 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229480AbjLMIYm (ORCPT ); Wed, 13 Dec 2023 03:24:42 -0500 Received: from mail-oi1-x22a.google.com (mail-oi1-x22a.google.com [IPv6:2607:f8b0:4864:20::22a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 87CCB91 for ; Wed, 13 Dec 2023 00:24:48 -0800 (PST) Received: by mail-oi1-x22a.google.com with SMTP id 5614622812f47-3b9e7f4a0d7so4337958b6e.1 for ; Wed, 13 Dec 2023 00:24:48 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1702455888; x=1703060688; darn=vger.kernel.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=6v1Ba8YaX+pMQZF3in4IJSoafdVQFQX6Sy/KwMfRTv8=; b=xemkWMutT0QK9U0bFNB+eksWPDcKo1TAUmpdsFCLMf+hUhzI0fL6+Pdiio6fhB7fvb W1wvl5ILf/x2IeP73f85+updOI8QFudUgy6Evv4yNrNLvW39UHZCJqmMjLrFphG/A5hD I0dyHSmTU4dia02DWuLJIyhD07iiEiA+9A1IEuLh0ZYa0TxhdKJUqbQUDca2eD/y7XtW K/Ho+pfJetquSQgHQKNrk9l8NLCxuOcXtlRuFcmkfdWx0PXGPglm7vwUGuPvY1onPyPD L/m1ueHAG7BDL/iNtbUt0FrUYalIMbjQcmFSnINTAFZDLMuhFjPymPSEOq7uzxvpJ4us ubYw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702455888; x=1703060688; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=6v1Ba8YaX+pMQZF3in4IJSoafdVQFQX6Sy/KwMfRTv8=; b=qBrZOvE9NS1OlGpuGTC2gTEkqog6ajyrxcgkPppfAXof/12v8uHkhtTFrlTh2gBpys YAXoX/35DxtQneRJurjc0IyphIqfszsFCA/oLnyjcSOd+8FfqCGGGCOa+AjTBK1eYpGK BV/CLmWc46JU2lwHwW3xRXmpPq+n6FgAEO+uE0Pm5oB5BClH4yVTQpmsJ0SPUZysAUfe Z+m6+9P4JNvut92ZC+uZ14K1D1rdWq3O2MorgKr0TBawhnkO8H0BREwrM1vzOdYGjxS9 bENq+WFsyCleESoKjUng11XblojplXUzM9lQ793OM6mfTBiVoPrGCmZ6kdeWQPrCcp18 CY6A== X-Gm-Message-State: AOJu0YzyvFY9lRbLeOKbS0IAsoQO7XlbSEjyEidwbHUExfJ1A9AY15Me +z/WMhnwCocEtHtqzS3F427YSSMDCJtqTj88IimB7Q== X-Received: by 2002:a05:6808:1983:b0:3b9:da2c:e5a9 with SMTP id bj3-20020a056808198300b003b9da2ce5a9mr9598920oib.62.1702455887866; Wed, 13 Dec 2023 00:24:47 -0800 (PST) MIME-Version: 1.0 References: <202312131510+0800-wangjinchao@xfusion.com> In-Reply-To: From: Vincent Guittot Date: Wed, 13 Dec 2023 09:24:36 +0100 Message-ID: Subject: Re: [PATCH v2] sched/fair: merge same code in enqueue_task_fair To: Abel Wu Cc: WangJinchao , 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 Content-Type: text/plain; charset="UTF-8" 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 fry.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 (fry.vger.email [0.0.0.0]); Wed, 13 Dec 2023 00:24:58 -0800 (PST) On Wed, 13 Dec 2023 at 09:19, Abel Wu wrote: > > Hi Jinchao, > > On 12/13/23 3:12 PM, WangJinchao Wrote: > > The code below is duplicated in two for loops and need to be > > consolidated > > It doesn't need to, but it can actually bring some benefit from > the point of view of text size, especially in warehouse-scale > computers where icache is extremely contended. > > add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-56 (-56) > Function old new delta > enqueue_task_fair 936 880 -56 > Total: Before=64899, After=64843, chg -0.09% > > > > > Signed-off-by: WangJinchao > > --- > > kernel/sched/fair.c | 31 ++++++++----------------------- > > 1 file changed, 8 insertions(+), 23 deletions(-) > > > > diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c > > index d7a3c63a2171..e1373bfd4f2e 100644 > > --- a/kernel/sched/fair.c > > +++ b/kernel/sched/fair.c > > @@ -6681,30 +6681,15 @@ enqueue_task_fair(struct rq *rq, struct task_struct *p, int flags) > > cpufreq_update_util(rq, SCHED_CPUFREQ_IOWAIT); > > > > for_each_sched_entity(se) { > > - if (se->on_rq) > > - break; > > cfs_rq = cfs_rq_of(se); > > - enqueue_entity(cfs_rq, se, flags); > > - > > - cfs_rq->h_nr_running++; > > - cfs_rq->idle_h_nr_running += idle_h_nr_running; > > - > > - if (cfs_rq_is_idle(cfs_rq)) > > - idle_h_nr_running = 1; > > - > > - /* end evaluation on encountering a throttled cfs_rq */ > > - if (cfs_rq_throttled(cfs_rq)) > > - goto enqueue_throttle; > > - > > - flags = ENQUEUE_WAKEUP; > > - } > > - > > - for_each_sched_entity(se) { > > - cfs_rq = cfs_rq_of(se); > > - > > - update_load_avg(cfs_rq, se, UPDATE_TG); > > - se_update_runnable(se); > > - update_cfs_group(se); > > + if (se->on_rq) { > > + update_load_avg(cfs_rq, se, UPDATE_TG); > > + se_update_runnable(se); > > + update_cfs_group(se); > > + } else { > > + enqueue_entity(cfs_rq, se, flags); > > + flags = ENQUEUE_WAKEUP; > > + } > > > > cfs_rq->h_nr_running++; > > cfs_rq->idle_h_nr_running += idle_h_nr_running; > > I have no strong opinon about this 'cleanup', but the same pattern > can also be found in dequeue_task_fair() and I think it would be > better get them synchronized. I agree, I don't see any benefit from this change > > Thanks, > Abel