Received: by 2002:ab2:715a:0:b0:1fd:c064:50c with SMTP id l26csp65538lqm; Mon, 10 Jun 2024 12:49:43 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCVTDFNPFZdRu1S1LsjXCxuSxSVmcoOiJ56KuEjOj7HJiEl81ya1o4UR/pwWqVTDGGCPztaR1sPdprSHJerVjiuTYTkU29OT/s9GIaN++A== X-Google-Smtp-Source: AGHT+IEDU2Mwt9kBphPJj7Wvd0oPipyS7W+5gaypDxNKoy1HEWWPsF11i9pC9YXFsspemg7yZ2w6 X-Received: by 2002:a17:902:f54e:b0:1f6:7f8f:65c7 with SMTP id d9443c01a7336-1f6d02e0c84mr114109965ad.26.1718048983328; Mon, 10 Jun 2024 12:49:43 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1718048983; cv=pass; d=google.com; s=arc-20160816; b=HW7/omqAmceEONymFrmjevzyve0JZTiDAAg+IIkG4b6yS/2WQMvyfhsdqmlgVns4zM jEQq0yrXEH1S9edD9El5yaverdRZsAPz/nMxsa3aWb9Oybq3h62e9UCGxyaF6XWrnO0N e2D5IavDVYJt9Aq/9FO+shec32ALimddcsRh7Kt/NpOPf6mMWbCP44vSCuqe+iZzmTxH hKIkfmcKRRaUHyW4MxraEpKv0s16vRFdR9vjD5JZDXHzqS5LFH7b4Ko1ibb/am8rpmb3 Kb0LiysgkfnIaAqZpzRykg2xTxy2d1DghOTEFj33pio3PDNdCJaJzaGufiI5dXMcfn3q YsgA== 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=zk/M6AnvrD9BC9PNl8gXkWi3lZX3+mGPAwXgE5CNlVg=; fh=EgDnBVNFpkahDe3lyTiewyNckmuoGLBC3pOrQoNG6v0=; b=UzC/bcqqKrps4AeWIU0tDm6Q0Q+gGcHqEIHzYVrtjVvzytGH/kmbj2hX1mimRivVxj VgK5A30VMSGeiiBHwKt+VvQPszAv2pnPodT9owR5kHUmMRypvdtDyGyoDqo5FBs8bnjy 5f0mMqdK1fATZhzk/w4IyYV6mMZPVAMWg6+In32qvQ9brDjb6/Xm5th8pTC+DjLn90Vq LlvxRJb+YNCPlFjmUj6nPnmUFXhxKwzg401zffZzyR9ezgo0j9TL69+tWjxRcudmTjm6 bIypWNgbNMT2ygJIGGGkHGTP5It5MgzAXh26AitKBShA6GyjPZv5qQKwFKyXBBRpPCS2 Qvfg==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel+bounces-208813-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-208813-linux.lists.archive=gmail.com@vger.kernel.org" Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id d9443c01a7336-1f72dcaf2d6si291125ad.267.2024.06.10.12.49.43 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 10 Jun 2024 12:49:43 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-208813-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel+bounces-208813-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-208813-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 sv.mirrors.kernel.org (Postfix) with ESMTPS id EA6D4285DD5 for ; Mon, 10 Jun 2024 19:49:42 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id A22D314E2E9; Mon, 10 Jun 2024 19:49:37 +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 2787F249F5; Mon, 10 Jun 2024 19:49:36 +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=1718048977; cv=none; b=o+9nHK7NKjSX/VS0ew0HxACKpr1mfqgBO9N/nxKhbjn2PzPRUBuEqeL3g7uQ6shqWyQgeBA+JynLglsPS1wSi5vujiug/xGwFPYPQatp0TJ0BZL93+OPYw7HckxEdbCspwyvCD5+cayP+s5lNiSr99VVw+QNwlZtEdgvRurhwqg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718048977; c=relaxed/simple; bh=BajpPb9cYuVE2LzaAxXNYoeuNPuKokjgXCrmdP/kO4w=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=eVcIL4V/W7zaYHVgu8Z+2bz0+jY/E4v09cM5mBihW5z1agnOtFHtOWniadJFdtkBDQns7MSXkvewbQCydPBfHJcS6bcmkFmLD8JrwrpUlUOMH4QkpKMjFz+khSdKcySx37AGVzsa7k71s8GjvL+srSKXs7SDEQr1s+WS/hAP4Fs= 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 309AEC32789; Mon, 10 Jun 2024 19:49:36 +0000 (UTC) Date: Mon, 10 Jun 2024 15:49:49 -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: <20240610154949.66f3c700@gandalf.local.home> In-Reply-To: <20240610111001.5ed3e97f6383915fcba87e31@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> <20240610111001.5ed3e97f6383915fcba87e31@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 Mon, 10 Jun 2024 11:10:01 +0900 Masami Hiramatsu (Google) wrote: > > But you don't explain what exactly the conflict is. What about those > > events causes kprobe selftests to fail? > > I also found another problem on these modules. These modules get trace > event file references to prevent removing probes. Therefore, if we > embed these modules, we can not remove these events forever! > > /* > * Now get the gen_kprobe_test event file. We need to prevent > * the instance and event from disappearing from underneath > * us, which trace_get_event_file() does (though in this case > * we're using the top-level instance which never goes away). > */ > gen_kprobe_test = trace_get_event_file(NULL, "kprobes", > "gen_kprobe_test"); > if (IS_ERR(gen_kprobe_test)) { > ret = PTR_ERR(gen_kprobe_test); > goto delete; > } > > This means most ftracetest fails because we can not clean up the > tracing state by removing dynamic events, which are installed while > booting. > Note that these references (locks) will be removed when the module > is unloaded. I'm fine if you want to force these as modules. Just make sure the change log lists all the issues of them not being modules. -- Steve