From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 15855FD2D9F for ; Tue, 10 Mar 2026 14:08:13 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4ED856B00C5; Tue, 10 Mar 2026 10:08:12 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 49B486B00C6; Tue, 10 Mar 2026 10:08:12 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3C8816B00CA; Tue, 10 Mar 2026 10:08:12 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 28A976B00C5 for ; Tue, 10 Mar 2026 10:08:12 -0400 (EDT) Received: from smtpin02.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id D73B4898AC for ; Tue, 10 Mar 2026 14:08:11 +0000 (UTC) X-FDA: 84530332782.02.4EED714 Received: from relay.hostedemail.com (unirelay01 [10.200.18.64]) by imf29.hostedemail.com (Postfix) with ESMTP id 1B2CE120016 for ; Tue, 10 Mar 2026 14:08:09 +0000 (UTC) Authentication-Results: imf29.hostedemail.com; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1773151690; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=HPL66xu6ufWwmWtPeO/cE7zqEYWvLRdiXRFNaq8HLM4=; b=WhhKejIuWNI1QDvmFpcrfcR+8CICVEFdyrWfEjIma+wY/LtfSyV9uglaky+cTNiLEIxKxO PHZ3f1tUColHj1SIHXyZAKSwOuAlk4ckVlGzuOAR1IVIHDvBp3Y06FtFsvHU1JOPejdAES A6UcZTcQiOG3gOt7mOim2smeHOUVkIs= ARC-Authentication-Results: i=1; imf29.hostedemail.com; none ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1773151690; a=rsa-sha256; cv=none; b=OuSR8MDWBbNyTjhKkoNpa5xBqIHO3pSgDNhQklpegANEa9Pj2TVoiWivC3WbVbGej8n+28 k2Vr3WKxA2tchH9KuxH0l2fu5aSvIzLv65vzvcaXlpDEtPXHMoZKNSz8vf1HcXGdJUpiZb QnkZV5qDJ9rvKy6JaQQcVMJ97wyID9E= Received: from omf14.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 0CEFF1C171; Tue, 10 Mar 2026 14:08:05 +0000 (UTC) Received: from [HIDDEN] (Authenticated sender: rostedt@goodmis.org) by omf14.hostedemail.com (Postfix) with ESMTPA id 847E92D; Tue, 10 Mar 2026 14:07:40 +0000 (UTC) Date: Tue, 10 Mar 2026 10:07:50 -0400 From: Steven Rostedt To: Philipp Hahn Cc: amd-gfx@lists.freedesktop.org, apparmor@lists.ubuntu.com, bpf@vger.kernel.org, ceph-devel@vger.kernel.org, cocci@inria.fr, dm-devel@lists.linux.dev, dri-devel@lists.freedesktop.org, gfs2@lists.linux.dev, intel-gfx@lists.freedesktop.org, intel-wired-lan@lists.osuosl.org, iommu@lists.linux.dev, kvm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-block@vger.kernel.org, linux-bluetooth@vger.kernel.org, linux-btrfs@vger.kernel.org, linux-cifs@vger.kernel.org, linux-clk@vger.kernel.org, linux-erofs@lists.ozlabs.org, linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-gpio@vger.kernel.org, linux-hyperv@vger.kernel.org, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, linux-leds@vger.kernel.org, linux-media@vger.kernel.org, linux-mips@vger.kernel.org, linux-mm@kvack.org, linux-modules@vger.kernel.org, linux-mtd@lists.infradead.org, linux-nfs@vger.kernel.org, linux-omap@vger.kernel.org, linux-phy@lists.infradead.org, linux-pm@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-s390@vger.kernel.org, linux-scsi@vger.kernel.org, linux-sctp@vger.kernel.org, linux-security-module@vger.kernel.org, linux-sh@vger.kernel.org, linux-sound@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-trace-kernel@vger.kernel.org, linux-usb@vger.kernel.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org, ntfs3@lists.linux.dev, samba-technical@lists.samba.org, sched-ext@lists.linux.dev, target-devel@vger.kernel.org, tipc-discussion@lists.sourceforge.net, v9fs@lists.linux.dev, Masami Hiramatsu , Mathieu Desnoyers Subject: Re: [PATCH 15/61] trace: Prefer IS_ERR_OR_NULL over manual NULL check Message-ID: <20260310100750.303af303@gandalf.local.home> In-Reply-To: <20260310-b4-is_err_or_null-v1-15-bd63b656022d@avm.de> References: <20260310-b4-is_err_or_null-v1-0-bd63b656022d@avm.de> <20260310-b4-is_err_or_null-v1-15-bd63b656022d@avm.de> X-Mailer: Claws Mail 3.20.0git84 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Session-Marker: 726F737465647440676F6F646D69732E6F7267 X-Session-ID: U2FsdGVkX19Z3Tx1sMlUDFHS8YYPfSOpiqPw8cg/LfU= X-HE-Meta: U2FsdGVkX1+JWrMXdInzvBx9l2WUqahyoYUezPLaF3HICGuHpiLcos2NJHmqwFPLfM8ZBXZ6/fmTg/vlHLcqcaFGukyEZOsOE3kBaCKhhIkGksjLucUHfFRW8KX39dTkHeOQhZTWX/A0f9Y+1fWRMEQzHGhaJF2wpPshgb4yUxNCAKdeI6QYyukTSlQGbTY7Uxizi79ndrJuznFkrJ0f1Gh3JC2I5gazyCY5QhqLMJ9a9ShPFpKdZ7NDaPeyx31oXfa0p+bWWHR1R5JbGDYWhbllAgJDuEbxQsOff6UlFUV0L3hTo0f3pFVAKw4ZSe/3wUeKTGah6IcaIcnei7ziR9eK5k/ow6hfmOBOZGq2CLBxHUplOJMJzaQEF/IeU48U6bUZ8WiPpl/JDxwK03ucqBa/7oG+YB+etWyAxh+TKdi7NbCLCb6PbeExJ2cO76n2AdjJCB03SaDGNN90kGsLmA== X-Rspam-User: X-HE-Tag-Orig: 1773151660-993833 X-Rspamd-Queue-Id: 1B2CE120016 X-Rspamd-Server: rspam08 X-Stat-Signature: 96g3re8yxfaeoumzj3ttytwhczwetqtf X-HE-Tag: 1773151689-198421 X-HE-Meta: U2FsdGVkX19ycyHu+RR3EtH68Chlf+C+7cppba/jBZtBuQ6p0mc4FAGkbatrdwVXNOTr3QGvHj5FXx8O3CKBewIhBqQHFltfRwwDkyvVi3a7XmfYEL+tC5TPGxBNVgI846nN9whkw1N7WPzIbPbaoIpfUrkh3lTTbOR+RPqXBojWkqYK+l/yaxMsswq2TeyBuIoJ/Skmc9l45yqrK0/ayaJpU/FYrDCzM1MIA5Yq+Cg1Tb/cHqRoFPi+dgWB33i39inQP4jHMvdECgdUc2tmin1fkL/Lkx8mMf/4BEnYlgf/FgceL2jxmQgq5fqObiG6RKPH0af84PqCWz1QhO7Zmt4PC2fNyZTo4rbWuYq/T002SerSKKrg2tdPB7SkmMyP/Wp4+win2AGmjHEI0pLGzCssdaUCj5MvtMe4QThnkxtrX6Mtg3JsSHd4WV8rsY0CooEwQ98sb4bsHEfG/mrU+9GcoUzceRyJ/liw3b5vmP5/83XpzxL/fdFAAQVhBPy5iY4899/zASyM3gRfSeaWU7A1kQ5QHsUeb1G6HXCM8RP+0XC+Qii14mmyZU9SabrIeE7TRP4ycoeczhKyCn4EM+do6ajWj/v3EErsKIiG4krt01Aefv8VNJrazaERcf/Ys2XDlFPsXER9YzQds7Gr5om85kWkvPGX+cK/kIW3r2nrxZawAcb5h3WPi2Kb1PUQUww1xXiaxo72R1Njuzisb5uV7PjVBDLjbLrA+F61XXgrJjnqpaXFLdJF5Q9zZEFnirNwY9wiL7QYEjI3Au614CSBSnO7evUBED1yaTTE34kaloar/RlZp5asBtyMVzCYIruTAWfr62DU7sy97qjMD1WWyXT3flpZa05rjMDQAyWPeLLCIPg8mcGA5y87KJFRFm4yWrud7LvrbX27w6VCAMbz090rHO3tB1FwBeVvSx+oOPM27vrpyXqJYjbnM0CEw3qL4e71ZeH9g8ozvMs fdejXJD9 5BJtiT4p2oraNqBYn4ZhVASbcFyUVjyMxBfMexn9Eul0wjtrN5kSpJq0DIw8wxgX4Vy0I/19zWVmojAmZsLxv2dH2pgRL021Bg1eCpqpgv5dNdBfasJie2fSFPg== Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Tue, 10 Mar 2026 12:48:41 +0100 Philipp Hahn wrote: > Prefer using IS_ERR_OR_NULL() over using IS_ERR() and a manual NULL > check. Why? > > Change generated with coccinelle. > > To: Steven Rostedt > To: Masami Hiramatsu > To: Mathieu Desnoyers > Cc: linux-kernel@vger.kernel.org > Cc: linux-trace-kernel@vger.kernel.org > Signed-off-by: Philipp Hahn > --- > kernel/trace/fprobe.c | 2 +- > kernel/trace/kprobe_event_gen_test.c | 2 +- > kernel/trace/trace_events_hist.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/kernel/trace/fprobe.c b/kernel/trace/fprobe.c > index dcadf1d23b8a31f571392d0c49cbd22df1716b4f..a94ce810d83b90f55d1178a9bd29c78fd068df4c 100644 > --- a/kernel/trace/fprobe.c > +++ b/kernel/trace/fprobe.c > @@ -607,7 +607,7 @@ static int fprobe_module_callback(struct notifier_block *nb, > do { > rhashtable_walk_start(&iter); > > - while ((node = rhashtable_walk_next(&iter)) && !IS_ERR(node)) > + while (!IS_ERR_OR_NULL((node = rhashtable_walk_next(&iter)))) Ug, No! That looks so much worse than the original. -- Steve > fprobe_remove_node_in_module(mod, node, &alist); > > rhashtable_walk_stop(&iter); > diff --git a/kernel/trace/kprobe_event_gen_test.c b/kernel/trace/kprobe_event_gen_test.c > index 5a4b722b50451bfdee42769a6d3be39c055690d1..a1735ca273f0b756aa1fcfcdab30ddad9bc51c5f 100644 > --- a/kernel/trace/kprobe_event_gen_test.c > +++ b/kernel/trace/kprobe_event_gen_test.c > @@ -75,7 +75,7 @@ static struct trace_event_file *gen_kretprobe_test; > > static bool trace_event_file_is_valid(struct trace_event_file *input) > { > - return input && !IS_ERR(input); > + return !IS_ERR_OR_NULL(input); > } > > /* > diff --git a/kernel/trace/trace_events_hist.c b/kernel/trace/trace_events_hist.c > index 73ea180cad555898693e92ee397a1c9493c7c167..59df215e1dfd9349eca1c0823ed709ec7285f766 100644 > --- a/kernel/trace/trace_events_hist.c > +++ b/kernel/trace/trace_events_hist.c > @@ -3973,7 +3973,7 @@ trace_action_create_field_var(struct hist_trigger_data *hist_data, > */ > field_var = create_target_field_var(hist_data, system, event, var); > > - if (field_var && !IS_ERR(field_var)) { > + if (!IS_ERR_OR_NULL(field_var)) { > save_field_var(hist_data, field_var); > hist_field = field_var->var; > } else { >