Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp372864pxb; Wed, 27 Oct 2021 04:42:17 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyRuErNGoH6thpqhgiyPq05kURDZoJzLsJNLQ8fuZ4ja0TcKHagPLk5VHsWn9rfJ4onMgXg X-Received: by 2002:a17:906:6a09:: with SMTP id qw9mr22701191ejc.212.1635334937036; Wed, 27 Oct 2021 04:42:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1635334937; cv=none; d=google.com; s=arc-20160816; b=z0DP8za83rwpGLBqJKyB57fiOANphE1Z5ep9jIuM+3s9Rz8BGI6maMUYcyfzMVS8eQ TkzcqYH+GogQyMLNNpCkIuLHsUSLwsrExo8DjwvS739D6/S8aNHXKAU1bqRW9sTfUChP AOGFfXZrkscTZ+RS6iPN9JXDYlrOsJgJ/WqZ+Qd6DpspwXvF7LNDSrOXNnDPwqgrOgE9 i0J0oKxFfl9fsTppl3VS9mm45xxG4C0OYpNrk0LNRF9X7t7WLXhXDT8VPEgjS3kfO5rU G6AIZSZgn0WgMBPXoLAQBNgB7biw4YFPKC//3gdJ1ilOaWhCkGAm02GKqDl9VO47a39W +YNQ== 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; bh=6Q6gUx8/AuiRBg+UWjhYQW/jSwriHb/G7WNOt4GUuhE=; b=IGVl1JpvqcXfbwcyWv509QP6dvmZC5kTNEEkqqxPfsN+zPC/lN/unXKjdbXAin7TmF 4O1hw3/YpIrfuRYG1LkgpFe3NcPh6R1HAQucaPQznxz5rn9v3gUogFjlA3Z0bVrXyT8E qbJZ9R5Jih1aSgs+6x3NXDoaYxGZMkIvvYajaqH1przfd2OwF8GQmGA8MO9LNb90xSce EMClxGkAY222KOTZtEwepXsu1zpSdDlyXTXsx4JiU7/2muyQNNmyGh0/IPaz7a9/4Rb+ dlaDFOZDIPRzr0UfoMQVw7zXucbZr966wmewjdZ1VVegy9YW0VGeR0xCOeLUO2wCAOBu MNpA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id q4si3870550eds.61.2021.10.27.04.41.51; Wed, 27 Oct 2021 04:42:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 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 S237680AbhJZWJg convert rfc822-to-8bit (ORCPT + 99 others); Tue, 26 Oct 2021 18:09:36 -0400 Received: from us-smtp-delivery-44.mimecast.com ([205.139.111.44]:23084 "EHLO us-smtp-delivery-44.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237396AbhJZWJd (ORCPT ); Tue, 26 Oct 2021 18:09:33 -0400 Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-588-N8sjzlchMXas3WNbjyqc4g-1; Tue, 26 Oct 2021 18:07:05 -0400 X-MC-Unique: N8sjzlchMXas3WNbjyqc4g-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 65D9C36303; Tue, 26 Oct 2021 22:07:03 +0000 (UTC) Received: from x1.com (unknown [10.22.32.3]) by smtp.corp.redhat.com (Postfix) with ESMTP id B48C760BF1; Tue, 26 Oct 2021 22:06:54 +0000 (UTC) From: Daniel Bristot de Oliveira To: Steven Rostedt Cc: Tao Zhou , Daniel Bristot de Oliveira , Ingo Molnar , Tom Zanussi , Masami Hiramatsu , Juri Lelli , Clark Williams , John Kacur , Peter Zijlstra , Thomas Gleixner , Sebastian Andrzej Siewior , linux-rt-users@vger.kernel.org, linux-trace-devel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH V6 01/20] trace/osnoise: Do not follow tracing_cpumask Date: Wed, 27 Oct 2021 00:06:12 +0200 Message-Id: In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=bristot@kernel.org X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: kernel.org Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset=WINDOWS-1252 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org In preparation to support multiple instances, decouple the osnoise/timelat workload from instance-specific tracing_cpumask. Different instances can have conflicting cpumasks, making osnoise workload management needlessly complex. Osnoise already has its global cpumask. I also thought about using the first instance mask, but the "first" instance could be removed before the others. This also fixes the problem that changing the tracing_mask was not re-starting the trace. Cc: Steven Rostedt Cc: Ingo Molnar Cc: Tom Zanussi Cc: Masami Hiramatsu Cc: Juri Lelli Cc: Clark Williams Cc: John Kacur Cc: Peter Zijlstra Cc: Thomas Gleixner Cc: Sebastian Andrzej Siewior Cc: Daniel Bristot de Oliveira Cc: linux-rt-users@vger.kernel.org Cc: linux-trace-devel@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Daniel Bristot de Oliveira --- kernel/trace/trace_osnoise.c | 24 ++++++------------------ 1 file changed, 6 insertions(+), 18 deletions(-) diff --git a/kernel/trace/trace_osnoise.c b/kernel/trace/trace_osnoise.c index ce053619f289..73f9609cba3d 100644 --- a/kernel/trace/trace_osnoise.c +++ b/kernel/trace/trace_osnoise.c @@ -1553,13 +1553,9 @@ static int start_per_cpu_kthreads(struct trace_array *tr) cpus_read_lock(); /* - * Run only on CPUs in which trace and osnoise are allowed to run. + * Run only on online CPUs in which osnoise is allowed to run. */ - cpumask_and(current_mask, tr->tracing_cpumask, &osnoise_cpumask); - /* - * And the CPU is online. - */ - cpumask_and(current_mask, cpu_online_mask, current_mask); + cpumask_and(current_mask, cpu_online_mask, &osnoise_cpumask); for_each_possible_cpu(cpu) per_cpu(per_cpu_osnoise_var, cpu).kthread = NULL; @@ -1580,10 +1576,8 @@ static int start_per_cpu_kthreads(struct trace_array *tr) #ifdef CONFIG_HOTPLUG_CPU static void osnoise_hotplug_workfn(struct work_struct *dummy) { - struct trace_array *tr = osnoise_trace; unsigned int cpu = smp_processor_id(); - mutex_lock(&trace_types_lock); if (!osnoise_busy) @@ -1595,9 +1589,6 @@ static void osnoise_hotplug_workfn(struct work_struct *dummy) if (!cpumask_test_cpu(cpu, &osnoise_cpumask)) goto out_unlock; - if (!cpumask_test_cpu(cpu, tr->tracing_cpumask)) - goto out_unlock; - start_kthread(cpu); out_unlock: @@ -1700,13 +1691,10 @@ static void osnoise_tracer_stop(struct trace_array *tr); * interface to the osnoise trace. By default, it lists all CPUs, * in this way, allowing osnoise threads to run on any online CPU * of the system. It serves to restrict the execution of osnoise to the - * set of CPUs writing via this interface. Note that osnoise also - * respects the "tracing_cpumask." Hence, osnoise threads will run only - * on the set of CPUs allowed here AND on "tracing_cpumask." Why not - * have just "tracing_cpumask?" Because the user might be interested - * in tracing what is running on other CPUs. For instance, one might - * run osnoise in one HT CPU while observing what is running on the - * sibling HT CPU. + * set of CPUs writing via this interface. Why not use "tracing_cpumask"? + * Because the user might be interested in tracing what is running on + * other CPUs. For instance, one might run osnoise in one HT CPU + * while observing what is running on the sibling HT CPU. */ static ssize_t osnoise_cpus_write(struct file *filp, const char __user *ubuf, size_t count, -- 2.31.1