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 X-Spam-Level: X-Spam-Status: No, score=-13.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI, NICE_REPLY_A,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A44F2C433E1 for ; Tue, 18 Aug 2020 14:50:05 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 5EE8C20897 for ; Tue, 18 Aug 2020 14:50:05 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="JshAvEf5" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5EE8C20897 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 0C44E8D000F; Tue, 18 Aug 2020 10:50:05 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 074998D0003; Tue, 18 Aug 2020 10:50:05 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E80C58D000F; Tue, 18 Aug 2020 10:50:04 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0208.hostedemail.com [216.40.44.208]) by kanga.kvack.org (Postfix) with ESMTP id D29D78D0003 for ; Tue, 18 Aug 2020 10:50:04 -0400 (EDT) Received: from smtpin15.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 931168248D51 for ; Tue, 18 Aug 2020 14:50:04 +0000 (UTC) X-FDA: 77163974328.15.stone36_090998627020 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin15.hostedemail.com (Postfix) with ESMTP id 5BAFD1814B0C1 for ; Tue, 18 Aug 2020 14:50:04 +0000 (UTC) X-HE-Tag: stone36_090998627020 X-Filterd-Recvd-Size: 4204 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by imf09.hostedemail.com (Postfix) with ESMTP for ; Tue, 18 Aug 2020 14:50:03 +0000 (UTC) Received: from devnote2 (NE2965lan1.rev.em-net.ne.jp [210.141.244.193]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 4FFC420866; Tue, 18 Aug 2020 14:49:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1597762203; bh=aC0a+9Dqz+xL6DqOsy65jLPJS026yhiSAzjqQcepjuo=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=JshAvEf5sD0uKtjIkYZf5OeOLR+8ZCAUEzYqTrQmuoKBBkZf1y04mfkcT7FfRYXsc a6XmQbbDaXcMnq1VXll9PBZdC3HdVUTiatqWSpvaaymLU/Dupw+zTw2Q8jwVMf3RGi pwQL+oINPDOHIgkjlQsGrwAOtyckKKbc6rGNcQM4= Date: Tue, 18 Aug 2020 23:49:56 +0900 From: Masami Hiramatsu To: Jarkko Sakkinen Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, Andi Kleen , Ard Biesheuvel , Jessica Yu , Mark Rutland , Masami Hiramatsu , Mike Rapoport , Peter Zijlstra , Russell King , Will Deacon , Steven Rostedt , Ingo Molnar Subject: Re: [PATCH] kprobes: Use module_name() macro Message-Id: <20200818234956.c616371d8766e74aef5e7bd2@kernel.org> In-Reply-To: <20200818050857.117998-1-jarkko.sakkinen@linux.intel.com> References: <20200818050857.117998-1-jarkko.sakkinen@linux.intel.com> X-Mailer: Sylpheed 3.7.0 (GTK+ 2.24.32; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 5BAFD1814B0C1 X-Spamd-Result: default: False [0.00 / 100.00] X-Rspamd-Server: rspam05 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Tue, 18 Aug 2020 08:08:57 +0300 Jarkko Sakkinen wrote: > It is advised to use module_name() macro instead of dereferencing mod->name > directly. This makes sense for consistencys sake and also it prevents a > hard dependency to CONFIG_MODULES. > > Cc: linux-mm@kvack.org > Cc: Andi Kleen > Cc: Ard Biesheuvel > Cc: Jessica Yu > Cc: Mark Rutland , > Cc: Masami Hiramatsu > Cc: Mike Rapoport > Cc: Peter Zijlstra > Cc: Russell King > Cc: Will Deacon > Signed-off-by: Jarkko Sakkinen OK, this looks good to me. Acked-by: Masami Hiramatsu Thank you, > --- > I thought that to get things moving it would make sense to fix this low > hanging fruit issue first. Similarly as Masami's fix kernel/kprobes.c > this will make my patch set less rambling, and thus easier to follow. > kernel/trace/trace_kprobe.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c > index aefb6065b508..19c00ee90945 100644 > --- a/kernel/trace/trace_kprobe.c > +++ b/kernel/trace/trace_kprobe.c > @@ -106,9 +106,10 @@ static nokprobe_inline bool trace_kprobe_has_gone(struct trace_kprobe *tk) > static nokprobe_inline bool trace_kprobe_within_module(struct trace_kprobe *tk, > struct module *mod) > { > - int len = strlen(mod->name); > + int len = strlen(module_name(mod)); > const char *name = trace_kprobe_symbol(tk); > - return strncmp(mod->name, name, len) == 0 && name[len] == ':'; > + > + return strncmp(module_name(mod), name, len) == 0 && name[len] == ':'; > } > > static nokprobe_inline bool trace_kprobe_module_exist(struct trace_kprobe *tk) > @@ -688,7 +689,7 @@ static int trace_kprobe_module_callback(struct notifier_block *nb, > if (ret) > pr_warn("Failed to re-register probe %s on %s: %d\n", > trace_probe_name(&tk->tp), > - mod->name, ret); > + module_name(mod), ret); > } > } > mutex_unlock(&event_mutex); > -- > 2.25.1 > -- Masami Hiramatsu