From patchwork Tue May 11 18:50:42 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jim Cromie X-Patchwork-Id: 12251855 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.5 required=3.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED,DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_GIT autolearn=ham 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 99F36C433ED for ; Tue, 11 May 2021 18:51:45 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 1C9FD61185 for ; Tue, 11 May 2021 18:51:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1C9FD61185 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 6AF686B0085; Tue, 11 May 2021 14:51:35 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 685126B0087; Tue, 11 May 2021 14:51:35 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 414F66B0088; Tue, 11 May 2021 14:51:35 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0022.hostedemail.com [216.40.44.22]) by kanga.kvack.org (Postfix) with ESMTP id 20EA66B0085 for ; Tue, 11 May 2021 14:51:35 -0400 (EDT) Received: from smtpin40.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id D8C758249980 for ; Tue, 11 May 2021 18:51:34 +0000 (UTC) X-FDA: 78129843708.40.73F5C21 Received: from mail-il1-f172.google.com (mail-il1-f172.google.com [209.85.166.172]) by imf27.hostedemail.com (Postfix) with ESMTP id 6394F8019389 for ; Tue, 11 May 2021 18:51:33 +0000 (UTC) Received: by mail-il1-f172.google.com with SMTP id r5so18069325ilb.2 for ; Tue, 11 May 2021 11:51:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=aWjz67jruSBa3AfnJK6T0K+rhp52me264VjeXl1/Rrw=; b=tcmZbpv4N7BmdgE/VhMdKYX5VajzQkXZ9HM9JhHaPQegrpCIW6+E8YvZtCfgkdj/1q uCKZCSoPJb84nkdfmWQpXyFsga/opJNb0sc0Lm1lPgBD83G7Ab964HZMQxHXmk9g3NBY Csth1IFzOhyIUsjpkJ6BBIriMaY8S5e4meZjdOoOfCbIKa9/OTa5NImaId8UNpVh1GRq YTjVInNQ7c7romCCF2Pj0ujgveX/KhTfhzZJ8OdPY6ECAo+4PbszxPqjbhlPMQRDAYR8 8C1aQWMs7JJ2qqOmzjb7HRUXiBwTTtzviJ99li2XUuJ7wSQrTgSVTRiaT0hxXNF8zMsD 7Fzw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=aWjz67jruSBa3AfnJK6T0K+rhp52me264VjeXl1/Rrw=; b=IQ7czJXtrFRWDrdkfqpMv0UbEA0J6PHvND/wwommVdroKvMgbLhuiPZ+ZhiJYgCI+P ISXdOWSKlFggTGvxpJRA8uZi2FJxkhjmDY1Eoh4ZXfh5KIlYv5HGqnzi3EW+4vbHBd+C Aez3kpNXmp0qbTQrsg/tUi2Mpn6bdyhKgmLc5rVLVq0KdzDdf62MsYJV4j/iiZluUf0F aT6GCaDVEgGQKVBE+5lVDiKfWkYfGBKxwo5+UTubIL540YVg6fbZPwrqLNHZVl5UPrNN BRdfpbDSyiMeVb2b+O4IRdqiafadY4x55ASybxTz/YdnIS8w9nGtD1kagSd8hcIAlV0U g9lQ== X-Gm-Message-State: AOAM532drpHWRNu4Bk/FAKGQxI6VZKQUDXV9T8ihs9dNJiCSAmeFPjrr 8+lKgFDIIrDjSrxCIzxnRuQ= X-Google-Smtp-Source: ABdhPJyHyeXZ+uUGCw0gMpIlqjjffT+dvi62Suw1jrSBFF3VZEqbivUlNjZAqyX6XfY4bgsVNK6m/g== X-Received: by 2002:a92:c7a9:: with SMTP id f9mr26182241ilk.255.1620759093805; Tue, 11 May 2021 11:51:33 -0700 (PDT) Received: from frodo.mearth (c-24-9-77-57.hsd1.co.comcast.net. [24.9.77.57]) by smtp.googlemail.com with ESMTPSA id t10sm405096ils.36.2021.05.11.11.51.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 11 May 2021 11:51:33 -0700 (PDT) From: Jim Cromie To: Jason Baron , Jessica Yu , "GitAuthor: Jim Cromie" , linux-kernel@vger.kernel.org Cc: linux-mm@kvack.org, kernel test robot Subject: [RFC PATCH v5 13/28] dyndbg+module: expose ddebug_sites to modules Date: Tue, 11 May 2021 12:50:42 -0600 Message-Id: <20210511185057.3815777-14-jim.cromie@gmail.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210511185057.3815777-1-jim.cromie@gmail.com> References: <20210511185057.3815777-1-jim.cromie@gmail.com> MIME-Version: 1.0 X-Rspamd-Queue-Id: 6394F8019389 Authentication-Results: imf27.hostedemail.com; dkim=pass header.d=gmail.com header.s=20161025 header.b=tcmZbpv4; spf=pass (imf27.hostedemail.com: domain of jimcromie@gmail.com designates 209.85.166.172 as permitted sender) smtp.mailfrom=jimcromie@gmail.com; dmarc=pass (policy=none) header.from=gmail.com X-Rspamd-Server: rspam04 X-Stat-Signature: eha8jgwhmrziiij1y8tuyc1h96m8ijnn Received-SPF: none (gmail.com>: No applicable sender policy available) receiver=imf27; identity=mailfrom; envelope-from=""; helo=mail-il1-f172.google.com; client-ip=209.85.166.172 X-HE-DKIM-Result: pass/pass X-HE-Tag: 1620759093-352913 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: Up until now, the "__dyndbg_sites" section was kinda dangling. Unlike the "__dyndbg" section, it was not explicitly loaded by module.c:load_info(); but it came along quietly, perhaps because of the per .site initialization (by DEFINE_DYNAMIC_DEBUG_METADATA) across the section boundary, or maybe because x86 is more permissive. ISTM we need to treat "__dyndbg_sites" section just like "__dyndbg", on basic principles. (And maybe do this with commit that splits the sections, if robot tests indicate problems between). The changes to load_info propagated out to ddebug_add_module's prototype, which needs to manage the 2nd section like the 1st. RFC: Maybe the 2 sections/arrays should be a single item/tuple, but I didn't see any real benefit to adding something; recapitulating the previous pattern was straightforward. -v5 fix !CONFIG_DYNAMIC_DEBUG fn-stub Reported-by: kernel test robot Signed-off-by: Jim Cromie --- include/linux/dynamic_debug.h | 8 ++++---- kernel/module-internal.h | 1 + kernel/module.c | 9 ++++++--- lib/dynamic_debug.c | 20 ++++++++++++-------- 4 files changed, 23 insertions(+), 15 deletions(-) diff --git a/include/linux/dynamic_debug.h b/include/linux/dynamic_debug.h index bc4e778b755c..868e0769b72d 100644 --- a/include/linux/dynamic_debug.h +++ b/include/linux/dynamic_debug.h @@ -66,8 +66,8 @@ struct _ddebug { /* exported for module authors to exercise >control */ int dynamic_debug_exec_queries(const char *query, const char *modname); -int ddebug_add_module(struct _ddebug *tab, unsigned int n, - const char *modname); +int ddebug_add_module(struct _ddebug *tab, struct _ddebug_site *sites, + unsigned int numdbgs, const char *modname); extern int ddebug_remove_module(const char *mod_name); extern __printf(2, 3) void __dynamic_pr_debug(struct _ddebug *descriptor, const char *fmt, ...); @@ -199,8 +199,8 @@ void __dynamic_ibdev_dbg(struct _ddebug *descriptor, #include #include -static inline int ddebug_add_module(struct _ddebug *tab, unsigned int n, - const char *modname) +static inline int ddebug_add_module(struct _ddebug *tab, struct _ddebug_site *sites, + unsigned int numdbgs, const char *modname) { return 0; } diff --git a/kernel/module-internal.h b/kernel/module-internal.h index 33783abc377b..fb61eb2f8032 100644 --- a/kernel/module-internal.h +++ b/kernel/module-internal.h @@ -18,6 +18,7 @@ struct load_info { char *secstrings, *strtab; unsigned long symoffs, stroffs, init_typeoffs, core_typeoffs; struct _ddebug *debug; + struct _ddebug_site *sites; unsigned int num_debug; bool sig_ok; #ifdef CONFIG_KALLSYMS diff --git a/kernel/module.c b/kernel/module.c index 30479355ab85..792903230acd 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -2780,11 +2780,12 @@ static void add_kallsyms(struct module *mod, const struct load_info *info) } #endif /* CONFIG_KALLSYMS */ -static void dynamic_debug_setup(struct module *mod, struct _ddebug *debug, unsigned int num) +static void dynamic_debug_setup(struct module *mod, struct _ddebug *debug, + struct _ddebug_site *sites, unsigned int num) { if (!debug) return; - ddebug_add_module(debug, num, mod->name); + ddebug_add_module(debug, sites, num, mod->name); } static void dynamic_debug_remove(struct module *mod, struct _ddebug *debug) @@ -3333,6 +3334,8 @@ static int find_module_sections(struct module *mod, struct load_info *info) info->debug = section_objs(info, "__dyndbg", sizeof(*info->debug), &info->num_debug); + info->sites = section_objs(info, "__dyndbg_sites", + sizeof(*info->sites), &info->num_debug); return 0; } @@ -4004,7 +4007,7 @@ static int load_module(struct load_info *info, const char __user *uargs, goto free_arch_cleanup; } - dynamic_debug_setup(mod, info->debug, info->num_debug); + dynamic_debug_setup(mod, info->debug, info->sites, info->num_debug); /* Ftrace init must be called in the MODULE_STATE_UNFORMED state */ ftrace_module_init(mod); diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index 76315d20672a..1441b31915e7 100644 --- a/lib/dynamic_debug.c +++ b/lib/dynamic_debug.c @@ -49,6 +49,7 @@ struct ddebug_table { const char *mod_name; unsigned int num_ddebugs; struct _ddebug *ddebugs; + struct _ddebug_site *sites; }; struct ddebug_query { @@ -1008,14 +1009,14 @@ static const struct proc_ops proc_fops = { * Allocate a new ddebug_table for the given module * and add it to the global list. */ -int ddebug_add_module(struct _ddebug *tab, unsigned int n, - const char *name) +int ddebug_add_module(struct _ddebug *tab, struct _ddebug_site *sites, + unsigned int numdbgs, const char *modname) { struct ddebug_table *dt; dt = kzalloc(sizeof(*dt), GFP_KERNEL); if (dt == NULL) { - pr_err("error adding module: %s\n", name); + pr_err("error adding module: %s\n", modname); return -ENOMEM; } /* @@ -1024,15 +1025,16 @@ int ddebug_add_module(struct _ddebug *tab, unsigned int n, * member of struct module, which lives at least as long as * this struct ddebug_table. */ - dt->mod_name = name; - dt->num_ddebugs = n; + dt->mod_name = modname; + dt->num_ddebugs = numdbgs; dt->ddebugs = tab; + dt->sites = sites; mutex_lock(&ddebug_lock); list_add(&dt->link, &ddebug_tables); mutex_unlock(&ddebug_lock); - v2pr_info("%3u debug prints in module %s\n", n, dt->mod_name); + v2pr_info("%3u debug prints in module %s\n", numdbgs, modname); return 0; } @@ -1172,7 +1174,9 @@ static int __init dynamic_debug_init(void) if (strcmp(modname, site->modname)) { modct++; - ret = ddebug_add_module(iter_mod_start, site_ct, modname); + + ret = ddebug_add_module(iter_mod_start, site_mod_start, + site_ct, modname); if (ret) goto out_err; site_ct = 0; @@ -1182,7 +1186,7 @@ static int __init dynamic_debug_init(void) } site_ct++; } - ret = ddebug_add_module(iter_mod_start, site_ct, modname); + ret = ddebug_add_module(iter_mod_start, site_mod_start, site_ct, modname); if (ret) goto out_err;