Received: by 2002:a05:6500:1b41:b0:1fb:d597:ff75 with SMTP id cz1csp121988lqb; Tue, 4 Jun 2024 06:58:47 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCVMZFYXizZZNlIelJwBhxVjAmDEc+9AtFiCYN5lE8jLZZvUyKOkN7btmDJoiTF3mqm523JLOWwdTW32HDcK7NLN9ZlYUpTrla92ZN+DKg== X-Google-Smtp-Source: AGHT+IFPiSU1Thlcr5EQtqRcEcuWCP1hhIyzN87gbsaDdX1UNOtJhqJclrg1bBAchEM05pNIBFGy X-Received: by 2002:a50:aa9d:0:b0:56d:f9dc:c82f with SMTP id 4fb4d7f45d1cf-57a36383546mr7258332a12.14.1717509527319; Tue, 04 Jun 2024 06:58:47 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1717509527; cv=pass; d=google.com; s=arc-20160816; b=dqrlzfPQs07jpqfaKqF5eDsxleHD1o9CbC3br619z6gE2m7XtqGhiJwQYwFsTg7Czn EtXRu5dOnr6frBAMg4/ZdhUKKX0zJfFfpZQVEhdYahPF2EHEuiBtuI1q+T1ht0GVkdPS TisNqeFPcswTWU562c3WLHp3b/kBIV02wfIOMW5SvkpqC5Yw0Cs/7FWdobcEOKT24Jjb 9dkgqT6c4//ZeNFOd2KNMedRRrzYaqNwos48smgXzZ0LBG7vpS1tURwRc8pkENA/VtcR EpdmcNI7tSeNEzE5chl9aLjIFq7Z/mPPuQwC1HWcCjOs97pOLcwoOGmiu7fweIU3Hite YB2A== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :subject:cc:to:from:date; bh=xKB6JydT2Sst3Cd1Z7n8/N4HBm4NN5CvPcMSiG9YOnE=; fh=EgDnBVNFpkahDe3lyTiewyNckmuoGLBC3pOrQoNG6v0=; b=AXjMP+9Az6h0npKA1WIc/qfwitNn+IPtJY7iy/qorumj+id1rJefiyEWlHkBcuo45n o7idmClte2wSNbWbPTLmmDtAhFWYjsDgpBoOIunAsCzDQ1JpxRHYeRHGWirAmQoHc052 lo/ku0AMNzLkF2Arln03tyz5fTnKRbTLWF3q1RqBBhoVx7kgFr4F2pmWb5D8y2oSbdD/ MI06av5hclpU5B34uhMYQaKpuFDcBF5Rbsen4o0liTkjkMm4uUaKlSkeVG8yaF+g4QWn g2XLS3nA3ShSy4mRGjGFTMPvVvbcUR+NJtXOgRbP3dobt2wFTdhVesdm16Y7v5tvLbi3 Qidg==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel+bounces-200759-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-200759-linux.lists.archive=gmail.com@vger.kernel.org" Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id 4fb4d7f45d1cf-57a4d464b90si3737059a12.277.2024.06.04.06.58.47 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 04 Jun 2024 06:58:47 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-200759-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel+bounces-200759-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-200759-linux.lists.archive=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 66C191F227DF for ; Tue, 4 Jun 2024 13:58:02 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 796E0F9CF; Tue, 4 Jun 2024 13:57:52 +0000 (UTC) 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 F395818C05; Tue, 4 Jun 2024 13:57:51 +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=1717509472; cv=none; b=GPePnTr99y++P+BPiE+YDJpUz0ad5Wwm3+PW3CCBPTPZlBxBgMfVjx5TzmU4hwipKwZc5gTnYiP4SoO2CzD5cVXxIZkkbBLneVYOpEftmNPiLa0Yz1rUiDJwGpEqcHGEhfw4q8aeG9n4l80Qwr+kB2Tj3aYoNDczh/CIaUNw9v4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1717509472; c=relaxed/simple; bh=aMBCMEacFCAd34LFVoL5DP9nMf0EsmiIUcjcm9yks/c=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=MqtA0b6rjI+1QgJQ23T2AY8U2j7raXGALgdvXxKwLbXi3Wd6mc9nLKuNIZvIxknQ+zsZujG7Uv5p4AwVQesyws9ajAE+P4vb3giub2O1whF4jU2VByqgw0r6w+r8kzcC7pTMc8tEMp2PmSexOjvBz7MRhvDnmeRgv1dwiONRr1M= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0C213C2BBFC; Tue, 4 Jun 2024 13:57:49 +0000 (UTC) Date: Tue, 4 Jun 2024 09:57:46 -0400 From: Steven Rostedt To: "Masami Hiramatsu (Google)" Cc: Tom Zanussi , LKML , Linux Trace Kernel Subject: Re: [PATCH 0/3] tracing: Fix some selftest issues Message-ID: <20240604095746.1bbb003c@gandalf.local.home> In-Reply-To: <20240531232047.fca5712a98e360d4a27fd92c@kernel.org> References: <171671825710.39694.6859036369216249956.stgit@devnote2> <20240527192907.49c9220f@rorschach.local.home> <20240529014640.3a04f7301f12eb44738f9f2d@kernel.org> <20240529083818.b7835de167191d4567405ce6@kernel.org> <3a7e679712fb47b6c75af84163b5d3ea252f4da9.camel@linux.intel.com> <20240531113721.c0314e0cdb3beb70c1a6ba7d@kernel.org> <20240531032425.3635dc93@rorschach.local.home> <20240531232047.fca5712a98e360d4a27fd92c@kernel.org> X-Mailer: Claws Mail 3.20.0git84 (GTK+ 2.24.33; x86_64-pc-linux-gnu) 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=US-ASCII Content-Transfer-Encoding: 7bit On Fri, 31 May 2024 23:20:47 +0900 Masami Hiramatsu (Google) wrote: > The major conflict happens when the boot-time test cleans up the kprobe > events by > > dyn_events_release_all(&trace_kprobe_ops); > > And I removed it by [3/3] patch in this series :) because it does not > needed and not confirmed there is no other kprobe events when the test > starts. Also the warning message are redundant so I removed it by [2/3]. > > So without this [1/3], if we apply [2/3] and [3/3], the problem will be > mitigated, but I think the root cause is that these modules are built-in. I'm OK with making them module only, but I don't see any selftests for sythetic events. I think they should have a boot up test as well. If we remove them, let's add something to test them at boot up. Then the boot up code could clean it up. Or change the test module to be a boot up test that cleans itself up if it is compiled in as not a module? -- Steve