From patchwork Thu May 11 16:45:20 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Daniel Bristot de Oliveira X-Patchwork-Id: 92739 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp4523538vqo; Thu, 11 May 2023 09:53:29 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6EbWi1Taqmv9PNvDsY595UYk9BYg8+Zx8Jtx5B9wWGUMYhrTS85wVP2C5aw0Ks8oUvrYah X-Received: by 2002:a05:6a20:2584:b0:104:273d:ffc1 with SMTP id k4-20020a056a20258400b00104273dffc1mr1635878pzd.28.1683824008972; Thu, 11 May 2023 09:53:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1683824008; cv=none; d=google.com; s=arc-20160816; b=UWsXxyYXWC+Oq3TAug21XXaelt3w0aMIcQmkJ+K/s/F36ZV0tz3R3qxW1CZE1xYyoT w2Udyy9LF2DzM+XZzBY0WS3prCdgvvYGYZNkNHdEvhsZTMJtJQFyoWpVpy/lR6epEjB3 WdXHKdiKsex7UouPyLcmHmFlJRaJZbVSsJMYOzc+CiFdJ2LtbBQoLumaIHscf0bE5HEh T1SCSxpiwMsNkvrXZWIUBbDRkR1HvrvkUNtHf7gXPfz0mE5a3ILj8xgi4n9JwgnxT32W ctqc7fR4qxIv9d+FmK1iT3Rg9XDBjRxtReOckdv/qdDl3uPawsnwEi5LGvvsur7TS4OU ZDpA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=6U4AywFJN4f1V0TTb323bAi9wLgmhzWIECd6Nugy/DU=; b=KbUKf+KXRXBA6rRLrAz5UqHs7IuiJvrCr1ujPJBuceT0i/PhBxFXHvCpYGD58mUx9j WU23lA0+WjKYK0CFc7ybCbv+tLPkMf2ZvXabrIFI3vvBjIZ+u6G0QIH2tAoTdQFJE+LY cJGFSeBhSIXAP8snpwzNV0gmJybJQCtMuPThjUiBhRYZalb3Z4qqrgXZVW0j3EfMbqIf jWFciAQrV+oDNBHj2PIRAkiK/c6tB704t3OaHUvL8TaS2uV1LgiXCj9ci+yU7R93Kxtt pUAUYm/nIp0hLiRb2jEFpENlvZg5w6k2Jt73n9Dx2YACtdJ8HOx9nqjSH1DzXFIhZlHx zz+A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=qlWBJQzL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id t20-20020aa79474000000b00640f04eb325si8012690pfq.229.2023.05.11.09.53.15; Thu, 11 May 2023 09:53:28 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=qlWBJQzL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238266AbjEKQpv (ORCPT + 99 others); Thu, 11 May 2023 12:45:51 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58180 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238393AbjEKQpj (ORCPT ); Thu, 11 May 2023 12:45:39 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 82D887DBC; Thu, 11 May 2023 09:45:33 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 1F3B164FCE; Thu, 11 May 2023 16:45:33 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D379FC4339E; Thu, 11 May 2023 16:45:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1683823532; bh=QN31lfcfV4O5vC4vDBplSjYRBUdS3kX3MOlPv3M48gA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=qlWBJQzL3XbMbfNHEK7VwFmY+piyLj0vKqtx5D3w8PA1M74Sg+Q7yveaaxCmZii0l 8gEU22I8FauCQ0j+cnmMrfVCSA6A/k7EkImr8QgHbpaTIN6MU75D45JHWF5YmRggAN OzdxxczcXuMIPDuuWc0vpwPlTj8SHw0k6tayK6bfykq96TO7BPimV+iO7gd+W840eq +gDjtLmD4eofrvzT67REcg2TsMvoXg+Jv4nBXVn0/WjM0ZGghoCdcVW02UDrBGZw1z tvJSq+9wTO2VMeCJ+OVmguVdxxvPdC7zl2oyjFyiPlZHWpKofFFlKESSEWaBhKwFgi /Kn/i5cgrv38w== From: Daniel Bristot de Oliveira To: linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org, Steven Rostedt Cc: Daniel Bristot de Oliveira , Juri Lelli , William White , Masami Hiramatsu Subject: [PATCH 2/2] tracing/osnoise: Skip running osnoise if all instances are off Date: Thu, 11 May 2023 18:45:20 +0200 Message-Id: X-Mailer: git-send-email 2.38.1 In-Reply-To: References: MIME-Version: 1.0 X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1765617443953665828?= X-GMAIL-MSGID: =?utf-8?q?1765617443953665828?= In the case of all tracing instances being off, sleep for the entire period. Q: Why not kill all threads so? A: It is valid and useful to start the threads with tracing off. For example, rtla disables tracing, starts the tracer, applies the scheduling setup to the threads, e.g., sched priority and cgroup, and then begin tracing with all set. Skipping the period helps to speed up rtla setup and save the trace after a stop tracing. Cc: Steven Rostedt Cc: Daniel Bristot de Oliveira Cc: Masami Hiramatsu Signed-off-by: Daniel Bristot de Oliveira --- kernel/trace/trace_osnoise.c | 31 ++++++++++++++++++++++++++++--- 1 file changed, 28 insertions(+), 3 deletions(-) diff --git a/kernel/trace/trace_osnoise.c b/kernel/trace/trace_osnoise.c index c265ec5f1726..220172cb874d 100644 --- a/kernel/trace/trace_osnoise.c +++ b/kernel/trace/trace_osnoise.c @@ -1285,6 +1285,22 @@ static __always_inline void osnoise_stop_tracing(void) rcu_read_unlock(); } +/* + * osnoise_has_tracing_on - Check if there is at least one instance on + */ +static __always_inline int osnoise_has_tracing_on(void) +{ + struct osnoise_instance *inst; + int trace_is_on = 0; + + rcu_read_lock(); + list_for_each_entry_rcu(inst, &osnoise_instances, list) + trace_is_on += tracer_tracing_is_on(inst->tr); + rcu_read_unlock(); + + return trace_is_on; +} + /* * notify_new_max_latency - Notify a new max latency via fsnotify interface. */ @@ -1517,13 +1533,16 @@ static struct cpumask save_cpumask; /* * osnoise_sleep - sleep until the next period */ -static void osnoise_sleep(void) +static void osnoise_sleep(bool skip_period) { u64 interval; ktime_t wake_time; mutex_lock(&interface_lock); - interval = osnoise_data.sample_period - osnoise_data.sample_runtime; + if (skip_period) + interval = osnoise_data.sample_period; + else + interval = osnoise_data.sample_period - osnoise_data.sample_runtime; mutex_unlock(&interface_lock); /* @@ -1604,8 +1623,14 @@ static int osnoise_main(void *data) if (osnoise_migration_pending()) break; + /* skip a period if tracing is off on all instances */ + if (!osnoise_has_tracing_on()) { + osnoise_sleep(true); + continue; + } + run_osnoise(); - osnoise_sleep(); + osnoise_sleep(false); } migrate_enable();