From patchwork Fri Jun 28 10:29:03 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 168052 Delivered-To: patch@linaro.org Received: by 2002:a92:4782:0:0:0:0:0 with SMTP id e2csp3510746ilk; Fri, 28 Jun 2019 03:29:55 -0700 (PDT) X-Google-Smtp-Source: APXvYqwsz76jMlE9GXydRJ3D0QVjPoIsgoR97K/lhfuD0OX2oWm+RHJaCuPpJoPnJNOK9lyg84iy X-Received: by 2002:a17:902:29a7:: with SMTP id h36mr10968438plb.158.1561717795320; Fri, 28 Jun 2019 03:29:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1561717795; cv=none; d=google.com; s=arc-20160816; b=vC0ynYg9gixzxgWu6cwRuoeN932xb95Ntx5WfLS965IEDM2GIKHZ+ndL/xwlftM5Ov ly12+D8JwO8kYhRJE8PyVLUTWlIR6HY7l7mJ8mFd97BCLZmdZaohVbGyhJhd0p8E0KFj v6bQDXVRzPqrXz+9owA4sB3fVlndbWLYgojCEJWYyjqWTwn7TD59wKvWh7Ob0l9sKWQV a+JtEVKjsJCACj5qG1ERAy4Xyn4F26cvGLsLdljpudITKJQ3LPQwHYZrgg51zGCJShRb beu+NVjcPk2uEkhjhV4u0n1WGwzgoeyPvpR2iG2Vwer67mm3LYJ6y2CKve4ekHRmyFTi K5kg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=xJ00Z8l34PZ5p4r/9p6K40MK6k3D9Y0U8H/gEJ0eG/I=; b=j0t50yy5wL3dv5XEc+3RBTFU8rOU2aGKAeNJUuD4UzmOCoOL4U/W9N3ahXxsGn2eg+ ii06KfiXWaZ7SMkSzPOKf1Ebt9RzIPAb4LiHu40PLixUfKYoOvda0LsHu5qVfF0qRSh4 P17n4yF/jTy2pahQ7ILTAyDneZUkC/COF1dK3FgVi29XiYW3Ga2yrMGuzpbYNNBlyuvI oM53mLUD3Xt3HqhNqyVWa8YZbWbSNEa4024TeQzIT6w4kIbrZcM9DTAH5eP9UID5nt3E aCGXwlMDz5D+f+3+FNIHj/5vbvJDYaOj3q96iNNx/spp4smNMuLvMplUziufGSHGcTlS ZrMw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id r11si1865034pjq.108.2019.06.28.03.29.55; Fri, 28 Jun 2019 03:29:55 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726688AbfF1K3x (ORCPT + 30 others); Fri, 28 Jun 2019 06:29:53 -0400 Received: from mout.kundenserver.de ([217.72.192.75]:36897 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726484AbfF1K3x (ORCPT ); Fri, 28 Jun 2019 06:29:53 -0400 Received: from threadripper.lan ([149.172.19.189]) by mrelayeu.kundenserver.de (mreue107 [212.227.15.145]) with ESMTPA (Nemesis) id 1MekGB-1iEUSN2CMH-00aisc; Fri, 28 Jun 2019 12:29:22 +0200 From: Arnd Bergmann To: Peter Zijlstra , Ingo Molnar , Will Deacon Cc: Arnd Bergmann , Andrew Morton , Bart Van Assche , Frederic Weisbecker , Linus Torvalds , "Paul E. McKenney" , Thomas Gleixner , Waiman Long , Will Deacon , Yuyang Du , Ingo Molnar , linux-kernel@vger.kernel.org Subject: [PATCH] locking/lockdep: clean up #ifdef checks Date: Fri, 28 Jun 2019 12:29:03 +0200 Message-Id: <20190628102919.2345242-1-arnd@arndb.de> X-Mailer: git-send-email 2.20.0 MIME-Version: 1.0 X-Provags-ID: V03:K1:8e/xrGp11CLAguE8bdi1+n04EkhlKPOQBlU5GTABjwnScXJ9Eob bHPV+QQKXSFk509jwouOk4slwYQkTXOpAek6xtTQJMJhI+Vr48rs/w9lIqnPPV+6VFtNS9C 6O4DRU45klDso7IAXMdItfb/d7mm0PxsCv2RB+cWHnrC6GAJhJLdEN7ldQAxWl2Xt2hKW/p MZdgOe+PbvBWMj4TT09UQ== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1; V03:K0:oIfgM93huCo=:gXILo7squPk+PHdn2SMEaS YTIJ+GCtelKegLo4qmcuJ3dxiXtIZMeRjMfPeMZXjIqEmP1seFNotQW/yrKx9bKC+SF/2vinu bjkJxjm4b3MNbzNUF4oMhcVIrlPBOLBmvJ+JUDndYdyWgREXszgaKcZaghAyRpIA8wlBGRlnP wmrnJvOMUXgpDEdZGFY5/VXG3hH+SxReJZ2xByLjeG4KyxXrGCOPT+NjdqjPP8SkMTigVItVC NkypHY3VkpJdvmA+K31OI/8j0NiyrKcT33D9ml7BIGfbjdnzPJ5IXd+vN6xhAv9QMgTS7hDgg gw1jEWwjTjmhV4Nux9ltifEKMZnk8MRU6gvL6iNDni6ofkArVD+y9yBC7b7Tcag5LCt6NpRBP 71m+0K9aGmYkaJbnWGowxOdygyQw4/95BXOxEpStUxRcDSqRhF8enYdhLYHOgg38S7143VoQ9 p0giVODXJ161dFMMDJFMwI5AwivkjpcKIZlgpAWts6r5i/IfLIuJqa3pzcw9WDbIXPHdWl/8p osYABE9Ne3jgCVPw3pqwcNHxBCcwDA6zw0jw3QOT+CvA1ctQc2yaNk1JT/gFk52JpqoVHJxnt iKrbuiecblpYYX0kq4+3dL9gSoT44CKzxvQ1CikojAF3te4MjoBMKaYxRVTqbPtS9GycBqwW0 3e4C4nvSaqryMAdy5XMl6ap6G2U6Vy04tcDOJjNvSh4FAh2XUnYcM5RZDw7PY4JDKXOqUj2Jt P3N7qb2ykiuk4zqHg79sBR5hEExDW3c0d1IFFQ== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org As Will Deacon points out, CONFIG_PROVE_LOCKING implies TRACE_IRQFLAGS, so the conditions I added in the previous patch, and some others in the same file can be simplified by only checking for the former. No functional change. Fixes: 886532aee3cd ("locking/lockdep: Move mark_lock() inside CONFIG_TRACE_IRQFLAGS && CONFIG_PROVE_LOCKING") Signed-off-by: Arnd Bergmann --- kernel/locking/lockdep.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) -- 2.20.0 Acked-by: Will Deacon diff --git a/kernel/locking/lockdep.c b/kernel/locking/lockdep.c index 341f52117f88..4861cf8e274b 100644 --- a/kernel/locking/lockdep.c +++ b/kernel/locking/lockdep.c @@ -448,7 +448,7 @@ static void print_lockdep_off(const char *bug_msg) unsigned long nr_stack_trace_entries; -#if defined(CONFIG_TRACE_IRQFLAGS) && defined(CONFIG_PROVE_LOCKING) +#ifdef CONFIG_PROVE_LOCKING /* * Stack-trace: tightly packed array of stack backtrace * addresses. Protected by the graph_lock. @@ -491,7 +491,7 @@ unsigned int max_lockdep_depth; DEFINE_PER_CPU(struct lockdep_stats, lockdep_stats); #endif -#if defined(CONFIG_TRACE_IRQFLAGS) && defined(CONFIG_PROVE_LOCKING) +#ifdef CONFIG_PROVE_LOCKING /* * Locking printouts: */ @@ -2969,7 +2969,7 @@ static void check_chain_key(struct task_struct *curr) #endif } -#if defined(CONFIG_TRACE_IRQFLAGS) && defined(CONFIG_PROVE_LOCKING) +#ifdef CONFIG_PROVE_LOCKING static int mark_lock(struct task_struct *curr, struct held_lock *this, enum lock_usage_bit new_bit); @@ -3608,7 +3608,7 @@ static int mark_lock(struct task_struct *curr, struct held_lock *this, return ret; } -#else /* defined(CONFIG_TRACE_IRQFLAGS) && defined(CONFIG_PROVE_LOCKING) */ +#else /* CONFIG_PROVE_LOCKING */ static inline int mark_usage(struct task_struct *curr, struct held_lock *hlock, int check) @@ -3627,7 +3627,7 @@ static inline int separate_irq_context(struct task_struct *curr, return 0; } -#endif /* defined(CONFIG_TRACE_IRQFLAGS) && defined(CONFIG_PROVE_LOCKING) */ +#endif /* CONFIG_PROVE_LOCKING */ /* * Initialize a lock instance's lock-class mapping info: @@ -4321,8 +4321,7 @@ static void __lock_unpin_lock(struct lockdep_map *lock, struct pin_cookie cookie */ static void check_flags(unsigned long flags) { -#if defined(CONFIG_PROVE_LOCKING) && defined(CONFIG_DEBUG_LOCKDEP) && \ - defined(CONFIG_TRACE_IRQFLAGS) +#if defined(CONFIG_PROVE_LOCKING) && defined(CONFIG_DEBUG_LOCKDEP) if (!debug_locks) return;