From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BD50C25DB12 for ; Fri, 25 Jul 2025 20:34:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1753475660; cv=none; b=gdx2ZhTo6FedYhSgUe3spP/XKvp80ZjSsmuOcEsmfokhDhooLdd5rDdMlDPNT2Ev/zm0Qi/u72SJ0SzdD33eT9yg6fYyfAlm4qJHQFhNFrwP8H8d/wFzU0ikj7YL7QU62/V1fTjB7a1zFsR1VFKZrmS3G//NtalzNhaiZd1axhc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1753475660; c=relaxed/simple; bh=KOt0VqG5p6PfjLU6UfHkWtrmGh01+J8PpffJ4qd+E5g=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=pHsuGJuDoaU7XCbcJJaohrICTsvyScQgIUHNmw4Dfsni2qX/6oWn1BS7joRmue5y36Xxo7op8h04R9AbWhZvz7i+TE194qP6/3x9KsmY2aGVuPgWj2ixHirtUZ7s5hCuBEPG8qDTy38v5EN5pb8JCIyMYmpCSiw94hWTs+ZvCjc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=KBaWFaZo; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="KBaWFaZo" Received: by smtp.kernel.org (Postfix) with ESMTPSA id A0AC4C4CEF8; Fri, 25 Jul 2025 20:34:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1753475660; bh=KOt0VqG5p6PfjLU6UfHkWtrmGh01+J8PpffJ4qd+E5g=; h=Date:From:To:Cc:Subject:References:From; b=KBaWFaZoylqva5wCDWTPDCqFkW0gY+b3UQ789xXleHE7hh1d03oZK/TpxI4NL3q4N 8XB7RLFpKkPgeA5U46Yrs8HPWExXo5NhSr70svLV5et3r8jXvvQyQlbqsfj5nhhb0k 1t2WyaMT/6bRg/TwAAuVikf3kBlMY9A3qbDAG6mt/6NVaLyYqs3ZJbirJqhTbrkKVs ZRvqCa/e3L9jeMJRorsYlaW0JAralNKJBcVs2Op5pGru8bqSi4bXelU9deE7AlVPb/ M+x5gLy816Ea8NlEvC7cSsJ9F5EYP5KBlJT7st/uqsjcutAIuQpbu8MuumDGBgM/wn Kr35sr0e7VkXg== Received: from rostedt by gandalf with local (Exim 4.98.2) (envelope-from ) id 1ufP7a-00000001Qjq-3EII; Fri, 25 Jul 2025 16:34:26 -0400 Message-ID: <20250725203426.624183928@kernel.org> User-Agent: quilt/0.68 Date: Fri, 25 Jul 2025 16:34:13 -0400 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Tomas Glozar , John Kacur , Nam Cao , Juri Lelli , Clark Williams , Gabriele Monaco Subject: [for-next][PATCH 16/25] tools/rv: Do not skip idle in trace References: <20250725203357.087558746@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 From: Gabriele Monaco Currently, the userspace RV tool skips trace events triggered by the RV tool itself, this can be changed by passing the parameter -s, which sets the variable config_my_pid to 0 (instead of the tool's PID). This has the side effect of skipping events generated by idle (PID 0). Set config_my_pid to -1 (an invalid pid) to avoid skipping idle. Cc: Nam Cao Cc: Tomas Glozar Cc: Juri Lelli Cc: Clark Williams Cc: John Kacur Link: https://lore.kernel.org/20250723161240.194860-2-gmonaco@redhat.com Fixes: 6d60f89691fc ("tools/rv: Add in-kernel monitor interface") Signed-off-by: Gabriele Monaco Signed-off-by: Steven Rostedt (Google) --- tools/verification/rv/src/in_kernel.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/verification/rv/src/in_kernel.c b/tools/verification/rv/src/in_kernel.c index c0dcee795c0d..4bb746ea6e17 100644 --- a/tools/verification/rv/src/in_kernel.c +++ b/tools/verification/rv/src/in_kernel.c @@ -431,7 +431,7 @@ ikm_event_handler(struct trace_seq *s, struct tep_record *record, if (config_has_id && (config_my_pid == id)) return 0; - else if (config_my_pid && (config_my_pid == pid)) + else if (config_my_pid == pid) return 0; tep_print_event(trace_event->tep, s, record, "%16s-%-8d [%.3d] ", @@ -734,7 +734,7 @@ static int parse_arguments(char *monitor_name, int argc, char **argv) config_reactor = optarg; break; case 's': - config_my_pid = 0; + config_my_pid = -1; break; case 't': config_trace = 1; -- 2.47.2