From patchwork Tue May 11 18:50:53 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jim Cromie X-Patchwork-Id: 12251877 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.6 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,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 425B7C433ED for ; Tue, 11 May 2021 18:52:06 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id C9C0761185 for ; Tue, 11 May 2021 18:52:05 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C9C0761185 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 1A46A6B009B; Tue, 11 May 2021 14:51:54 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 12DA66B009C; Tue, 11 May 2021 14:51:54 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E9BC76B009D; Tue, 11 May 2021 14:51:53 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0070.hostedemail.com [216.40.44.70]) by kanga.kvack.org (Postfix) with ESMTP id C70B86B009B for ; Tue, 11 May 2021 14:51:53 -0400 (EDT) Received: from smtpin26.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay02.hostedemail.com (Postfix) with ESMTP id 84CE0B7B2 for ; Tue, 11 May 2021 18:51:53 +0000 (UTC) X-FDA: 78129844506.26.583CD14 Received: from mail-io1-f46.google.com (mail-io1-f46.google.com [209.85.166.46]) by imf03.hostedemail.com (Postfix) with ESMTP id 068CBC0007FF for ; Tue, 11 May 2021 18:51:44 +0000 (UTC) Received: by mail-io1-f46.google.com with SMTP id t3so19128942iol.5 for ; Tue, 11 May 2021 11:51:52 -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=O0BJdtccf2yAr6b2uo1yZDvtVjzorNp/j0upe7RkRXg=; b=in7nJsUIXuQhI9wTQ88x1zOvaDu7RnqR0Q8idpyFRjxJGrqbIzbZtc1fRGAQ8tcJOb Z5YdMYEvZRsqoq0hSkVmr0YjYhyPXqgh/0tp3c/fPeYWiQcloama1HgWn83/bONGOJJX UwXIiLLRaUs54LNNBnqUFGiBzfT3YdXXDTtAR1K9WrkoE+y+C5XQc2VJ2IMPUv8kd1HZ oTMG0EXWQkg9DTT+NsV3+zu/OmyPkcMTrDfOyUin+sXG0vHwFwFsA/mwVHYkZe0rX4rV +9zh3b4H9ATeAllwC/8iT3Dg6siH2zXbAYuTaAWys1VocRPWUbgEIsBGEOVoXs0xiLnT 4Zaw== 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=O0BJdtccf2yAr6b2uo1yZDvtVjzorNp/j0upe7RkRXg=; b=TilKYEiygV3+MQAHa35kJlK5+oN+HMnQaaWVo/pGtB7VAlffEVf8kITQSSpcokdtiA 01tMBt1nkRD5y3irIGwXjD/QdmFJrxFh+mLP+IZVIy2CCN/J7O5y/jtImTZq8OXaSX7p cQ4d2qnk6ZhUwzHgjwRWC8LY5RXKeLVpbgkJrW/MG016mFxWRilt0geYIoTbh3CwYnoV ZVePRdPIpT/oRysh4efy4H4zgMt9RkJNngIhltEkv6qgEJ90ZyT3ZBkQX3hWAgCIBBR4 JTyj2iIt2T4CP9Pz+gHl98ieaO047RdToSyIDTcbT4xedVqYc4HYG+x6o5Lmjo0c8GmN 8PMg== X-Gm-Message-State: AOAM531M2ddX2DxqDXGknZJ0NAj6Cnbx6LgTkDWl66aRd9Le5FVWToxF V2tc4t94rC1reziXWQzeJPU= X-Google-Smtp-Source: ABdhPJz85XfrSMdeQNwF8K/Ub1WdvZ+s3Hf15DIDmS9kxaJYqeQ4vTjfWj1GV0Z3rzimZH5IsK5q3w== X-Received: by 2002:a05:6602:2002:: with SMTP id y2mr8570269iod.70.1620759112554; Tue, 11 May 2021 11:51:52 -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.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 11 May 2021 11:51:52 -0700 (PDT) From: Jim Cromie To: Jason Baron , linux-kernel@vger.kernel.org Cc: linux-mm@kvack.org, Jim Cromie Subject: [RFC PATCH v5 24/28] dyndbg: make site pointer and checks on it optional (not quite) Date: Tue, 11 May 2021 12:50:53 -0600 Message-Id: <20210511185057.3815777-25-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 Authentication-Results: imf03.hostedemail.com; dkim=pass header.d=gmail.com header.s=20161025 header.b=in7nJsUI; spf=pass (imf03.hostedemail.com: domain of jimcromie@gmail.com designates 209.85.166.46 as permitted sender) smtp.mailfrom=jimcromie@gmail.com; dmarc=pass (policy=none) header.from=gmail.com X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: 068CBC0007FF X-Stat-Signature: 5w85tnt6yhb8xtssrbi1jnpcnng961zg Received-SPF: none (gmail.com>: No applicable sender policy available) receiver=imf03; identity=mailfrom; envelope-from=""; helo=mail-io1-f46.google.com; client-ip=209.85.166.46 X-HE-DKIM-Result: pass/pass X-HE-Tag: 1620759104-182400 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: Wrap declaration and uses of _ddebug.site pointer in ifdefs. Mostly, this eliminates all the WARN_ONs in ddebug_site_get(), which seem to be mostly proven (they were BUG_ONs). This adds macro indirection around ifdef SITE_CHK; adding SITE_INIT_ to adjust the .site initializations, and SITE_COND, which abstracts a logic test used by is_dyndbg_header_pair(i,s). TODO: breaks when !defined SITE_CHK - the header, being just a struct _ddebug, also lost the .site member. Need to specialize the header struct, then unionize it back in. Signed-off-by: Jim Cromie --- include/linux/dynamic_debug.h | 20 ++++++++++++++++---- lib/dynamic_debug.c | 13 ++++++++++--- 2 files changed, 26 insertions(+), 7 deletions(-) diff --git a/include/linux/dynamic_debug.h b/include/linux/dynamic_debug.h index 06ae1d8d8c10..46901b5348ee 100644 --- a/include/linux/dynamic_debug.h +++ b/include/linux/dynamic_debug.h @@ -6,6 +6,18 @@ #include #endif +#define SITE_CHK /* include .site pointer and checks on it */ + +#ifdef SITE_CHK +#define SITE_CHK_(...) __VA_ARGS__ +#define SITE_COND(_i, _s) (_s == _i->site) +#define SITE_INIT_(_name) .site = &_name, +#else +#define SITE_CHK_(...) +#define SITE_COND(_i, _s) (true) +#define SITE_INIT_(name) +#endif + /* * A pair of these structs are created in 2 special ELF sections * (__dyndbg, __dyndbg_sites) for every dynamic debug callsite. @@ -25,7 +37,7 @@ struct _ddebug_site { } __aligned(8); struct _ddebug { - struct _ddebug_site *site; + SITE_CHK_(struct _ddebug_site *site;) /* format is always needed, lineno shares word with flags */ const char *format; const unsigned lineno:18; @@ -106,7 +118,7 @@ void __dynamic_ibdev_dbg(struct _ddebug *descriptor, }; \ static struct _ddebug __aligned(8) \ __section("__dyndbg") name = { \ - .site = &name##_site, \ + SITE_INIT_(name##_site) \ .format = (fmt), \ .lineno = __LINE__, \ .flags = _DPRINTK_FLAGS_DEFAULT, \ @@ -138,13 +150,13 @@ void __dynamic_ibdev_dbg(struct _ddebug *descriptor, __weak struct _ddebug __used __aligned(8) \ __section(".gnu.linkonce.dyndbg") \ _LINKONCE_dyndbg_header = { \ - .site = &_LINKONCE_dyndbg_site_header, \ + SITE_INIT_(_LINKONCE_dyndbg_site_header) \ .format = KBUILD_MODNAME \ } /* The header initializations as a distinguishing predicate */ #define is_dyndbg_header_pair(iter, sitep) \ - (sitep == iter->site \ + (SITE_COND(iter, sitep) \ && (!iter->lineno) \ && (iter->format == sitep->modname) \ && (sitep->modname == sitep->function) \ diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index b83187e72ee0..9551c0d406b6 100644 --- a/lib/dynamic_debug.c +++ b/lib/dynamic_debug.c @@ -151,7 +151,7 @@ static struct _ddebug_site *ddebug_site_get(struct _ddebug *dp) struct _ddebug *dh = dp - (dp->_index); WARN_ON(!is_dyndbg_header_pair(dh, dh->site)); - +#ifdef SITE_CHK if (dp >= __start___dyndbg && dp < __stop___dyndbg) { v5pr_info("get: %s is builtin: %d %s:%s:%d\n", @@ -177,6 +177,9 @@ static struct _ddebug_site *ddebug_site_get(struct _ddebug *dp) WARN_ON(&dh->site[dp->_index] != dp->site); return dp->site; +#else + return &dh->site[dp->_index]; +#endif } static inline void ddebug_site_put(struct _ddebug *dp) { @@ -241,6 +244,7 @@ static void ddebug_alter_site(struct _ddebug *dp, } else if (modifiers->flags & _DPRINTK_FLAGS_PRINT) static_branch_enable(&dp->key.dd_key_true); #endif +#ifdef SITE_CHK /* delete site info for this callsite */ if (modifiers->flags & _DPRINTK_FLAGS_DELETE_SITE) { if (dp->site) { @@ -249,6 +253,7 @@ static void ddebug_alter_site(struct _ddebug *dp, dp->site = NULL; } } +#endif } /* @@ -1073,7 +1078,9 @@ static int __ddebug_add_module(struct _ddebug *tab, struct _ddebug_site *sites, v3pr_info(" header: %d %s.%s.%d\n", tab[0]._index, modname, tab[0].site->function, tab[0].lineno); - WARN_ON(tab[0].site != &sites[0]); + + SITE_CHK_(BUG_ON(tab[0].site != &sites[0])); + if (numdbgs <= 1) { v3pr_info(" skip header %s\n", modname); return 0; @@ -1251,7 +1258,7 @@ static int __init dynamic_debug_init(void) for (i = 0; iter < __stop___dyndbg; iter++, site++, i++) { - WARN_ON(site != iter->site); + SITE_CHK_(WARN_ON(site != iter->site)); if (strcmp(modname, site->modname)) { modct++;