From patchwork Mon Oct 28 08:58:38 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jan Kiszka X-Patchwork-Id: 3101301 Return-Path: X-Original-To: patchwork-linux-kbuild@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 250829F431 for ; Mon, 28 Oct 2013 09:43:36 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id E3597202C8 for ; Mon, 28 Oct 2013 09:43:34 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id AEACB202C7 for ; Mon, 28 Oct 2013 09:43:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752698Ab3J1Jnd (ORCPT ); Mon, 28 Oct 2013 05:43:33 -0400 Received: from david.siemens.de ([192.35.17.14]:33970 "EHLO david.siemens.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753542Ab3J1Jnc (ORCPT ); Mon, 28 Oct 2013 05:43:32 -0400 X-Greylist: delayed 2637 seconds by postgrey-1.27 at vger.kernel.org; Mon, 28 Oct 2013 05:43:31 EDT Received: from mail1.siemens.de (localhost [127.0.0.1]) by david.siemens.de (8.13.6/8.13.6) with ESMTP id r9S8x2gM016529; Mon, 28 Oct 2013 09:59:03 +0100 Received: from mchn199C.mchp.siemens.de.com ([146.254.78.28]) by mail1.siemens.de (8.13.6/8.13.6) with SMTP id r9S8wxGu032049; Mon, 28 Oct 2013 09:59:02 +0100 From: Jan Kiszka To: Andrew Morton , linux-kernel@vger.kernel.org Cc: Jason Wessel , kgdb-bugreport@lists.sourceforge.net, Andi Kleen , Tom Tromey , Ben Widawsky , Borislav Petkov , Tatiana Al-Chueyr Martins , Michal Marek , linux-kbuild@vger.kernel.org Subject: [PATCH v6 01/21] scripts/gdb: Add infrastructure Date: Mon, 28 Oct 2013 09:58:38 +0100 Message-Id: <74596df6935455908a146314b9b42ae2d79f6712.1382950737.git.jan.kiszka@siemens.com> X-Mailer: git-send-email 1.8.1.1.298.ge7eed54 In-Reply-To: References: In-Reply-To: References: Sender: linux-kbuild-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kbuild@vger.kernel.org X-Spam-Status: No, score=-7.4 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP This provides the basic infrastructure to load kernel-specific python helper scripts when debugging the kernel in gdb. The loading mechanism is based on gdb loading for -gdb.py when opening . Therefore, this places a corresponding link to the main helper script into the output directory that contains vmlinux. The main scripts will pull in submodules containing Linux specific gdb commands and functions. To avoid polluting the source directory with compiled python modules, we link to them from the object directory. Due to gdb.parse_and_eval and string redirection for gdb.execute, we depend on gdb >= 7.2. This feature depends on CONFIG_DEBUG_INFO. CC: Michal Marek CC: linux-kbuild@vger.kernel.org Signed-off-by: Jan Kiszka --- Makefile | 5 ++++- scripts/Makefile | 3 ++- scripts/gdb/Makefile | 11 +++++++++++ scripts/gdb/linux/Makefile | 9 +++++++++ scripts/gdb/linux/__init__.py | 0 scripts/gdb/vmlinux-gdb.py | 23 +++++++++++++++++++++++ 6 files changed, 49 insertions(+), 2 deletions(-) create mode 100644 scripts/gdb/Makefile create mode 100644 scripts/gdb/linux/Makefile create mode 100644 scripts/gdb/linux/__init__.py create mode 100644 scripts/gdb/vmlinux-gdb.py diff --git a/Makefile b/Makefile index 868c0eb..0e222d7 100644 --- a/Makefile +++ b/Makefile @@ -778,6 +778,9 @@ endif ifdef CONFIG_BUILD_DOCSRC $(Q)$(MAKE) $(build)=Documentation endif +ifdef CONFIG_DEBUG_INFO + $(Q)ln -fsn $(srctree)/scripts/gdb/vmlinux-gdb.py +endif +$(call if_changed,link-vmlinux) # The actual objects are generated when descending, @@ -1026,7 +1029,7 @@ MRPROPER_FILES += .config .config.old .version .old_version $(version_h) \ Module.symvers tags TAGS cscope* GPATH GTAGS GRTAGS GSYMS \ signing_key.priv signing_key.x509 x509.genkey \ extra_certificates signing_key.x509.keyid \ - signing_key.x509.signer + signing_key.x509.signer vmlinux-gdb.py # clean - Delete most, but leave enough to build external modules # diff --git a/scripts/Makefile b/scripts/Makefile index 01e7adb..3204b91 100644 --- a/scripts/Makefile +++ b/scripts/Makefile @@ -37,6 +37,7 @@ subdir-$(CONFIG_MODVERSIONS) += genksyms subdir-y += mod subdir-$(CONFIG_SECURITY_SELINUX) += selinux subdir-$(CONFIG_DTC) += dtc +subdir-$(CONFIG_DEBUG_INFO) += gdb # Let clean descend into subdirs -subdir- += basic kconfig package selinux +subdir- += basic kconfig package selinux gdb diff --git a/scripts/gdb/Makefile b/scripts/gdb/Makefile new file mode 100644 index 0000000..c0e2fd3 --- /dev/null +++ b/scripts/gdb/Makefile @@ -0,0 +1,11 @@ +subdir-y := linux + +always := gdb-scripts + +$(obj)/gdb-scripts: +ifneq ($(KBUILD_SRC),) + $(Q)ln -fsn $(srctree)/$(obj)/*.py $(objtree)/$(obj) +endif + @: + +clean-files := *.pyc $(if $(KBUILD_SRC),*.py) diff --git a/scripts/gdb/linux/Makefile b/scripts/gdb/linux/Makefile new file mode 100644 index 0000000..34ccd06 --- /dev/null +++ b/scripts/gdb/linux/Makefile @@ -0,0 +1,9 @@ +always := gdb-scripts + +$(obj)/gdb-scripts: +ifneq ($(KBUILD_SRC),) + $(Q)ln -fsn $(srctree)/$(obj)/*.py $(objtree)/$(obj) +endif + @: + +clean-files := *.pyc $(if $(KBUILD_SRC),*.py) diff --git a/scripts/gdb/linux/__init__.py b/scripts/gdb/linux/__init__.py new file mode 100644 index 0000000..e69de29 diff --git a/scripts/gdb/vmlinux-gdb.py b/scripts/gdb/vmlinux-gdb.py new file mode 100644 index 0000000..c1d90ce --- /dev/null +++ b/scripts/gdb/vmlinux-gdb.py @@ -0,0 +1,23 @@ +# +# gdb helper commands and functions for Linux kernel debugging +# +# loader module +# +# Copyright (c) Siemens AG, 2012, 2013 +# +# Authors: +# Jan Kiszka +# +# This work is licensed under the terms of the GNU GPL version 2. +# + +import os + +sys.path.insert(0, os.path.dirname(__file__) + "/scripts/gdb") + +try: + gdb.parse_and_eval("0") + gdb.execute("", to_string=True) +except: + gdb.write("NOTE: gdb 7.2 or later required for Linux helper scripts to " + "work.\n")