diff for duplicates of <488E150F.9050508@windriver.com> diff --git a/a/1.txt b/N1/1.txt index 7e66af4..205fe6c 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -34,3 +34,70 @@ documentation to reflect that it is advised to use frame pointers, but not a requirement. Jason. + + + +>From 329f5bb0071eb2a56f2bcf79838ba91784c8b54d Mon Sep 17 00:00:00 2001 +From: Jason Wessel <jason.wessel@windriver.com> +Date: Mon, 28 Jul 2008 13:48:31 -0500 +Subject: [PATCH] kgdb: remove the requirement for CONFIG_FRAME_POINTER + +There is no technical reason that the kgdb core requires frame +pointers. It is up to the end user of KGDB to decide if they need +them or not. + +[ anemo@mba.ocn.ne.jp: removed frame pointers on mips ] + +Signed-off-by: Jason Wessel <jason.wessel@windriver.com> +--- + Documentation/DocBook/kgdb.tmpl | 8 ++++++++ + lib/Kconfig.kgdb | 11 +++++++---- + 2 files changed, 15 insertions(+), 4 deletions(-) + +diff --git a/Documentation/DocBook/kgdb.tmpl b/Documentation/DocBook/kgdb.tmpl +index e8acd1f..54d3b15 100644 +--- a/Documentation/DocBook/kgdb.tmpl ++++ b/Documentation/DocBook/kgdb.tmpl +@@ -98,6 +98,14 @@ + "Kernel debugging" select "KGDB: kernel debugging with remote gdb". + </para> + <para> ++ It is advised, but not required that you turn on the ++ CONFIG_FRAME_POINTER kernel option. This option inserts code to ++ into the compiled executable which saves the frame information in ++ registers or on the stack at different points which will allow a ++ debugger such as gdb to more accurately construct stack back traces ++ while debugging the kernel. ++ </para> ++ <para> + Next you should choose one of more I/O drivers to interconnect debugging + host and debugged target. Early boot debugging requires a KGDB + I/O driver that supports early debugging and the driver must be +diff --git a/lib/Kconfig.kgdb b/lib/Kconfig.kgdb +index 2cfd272..9b5d1d7 100644 +--- a/lib/Kconfig.kgdb ++++ b/lib/Kconfig.kgdb +@@ -4,14 +4,17 @@ config HAVE_ARCH_KGDB + + menuconfig KGDB + bool "KGDB: kernel debugging with remote gdb" +- select FRAME_POINTER + depends on HAVE_ARCH_KGDB + depends on DEBUG_KERNEL && EXPERIMENTAL + help + If you say Y here, it will be possible to remotely debug the +- kernel using gdb. Documentation of kernel debugger is available +- at http://kgdb.sourceforge.net as well as in DocBook form +- in Documentation/DocBook/. If unsure, say N. ++ kernel using gdb. It is recommended but not required, that ++ you also turn on the kernel config option ++ CONFIG_FRAME_POINTER to aid in producing more reliable stack ++ backtraces in the external debugger. Documentation of ++ kernel debugger is available at http://kgdb.sourceforge.net ++ as well as in DocBook form in Documentation/DocBook/. If ++ unsure, say N. + + if KGDB + +-- +1.5.5.1 diff --git a/a/content_digest b/N1/content_digest index 03ef230..bce00e8 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -46,6 +46,73 @@ "documentation to reflect that it is advised to use frame pointers, but\n" "not a requirement.\n" "\n" - Jason. + "Jason.\n" + "\n" + "\n" + "\n" + ">From 329f5bb0071eb2a56f2bcf79838ba91784c8b54d Mon Sep 17 00:00:00 2001\n" + "From: Jason Wessel <jason.wessel@windriver.com>\n" + "Date: Mon, 28 Jul 2008 13:48:31 -0500\n" + "Subject: [PATCH] kgdb: remove the requirement for CONFIG_FRAME_POINTER\n" + "\n" + "There is no technical reason that the kgdb core requires frame\n" + "pointers. It is up to the end user of KGDB to decide if they need\n" + "them or not.\n" + "\n" + "[ anemo@mba.ocn.ne.jp: removed frame pointers on mips ]\n" + "\n" + "Signed-off-by: Jason Wessel <jason.wessel@windriver.com>\n" + "---\n" + " Documentation/DocBook/kgdb.tmpl | 8 ++++++++\n" + " lib/Kconfig.kgdb | 11 +++++++----\n" + " 2 files changed, 15 insertions(+), 4 deletions(-)\n" + "\n" + "diff --git a/Documentation/DocBook/kgdb.tmpl b/Documentation/DocBook/kgdb.tmpl\n" + "index e8acd1f..54d3b15 100644\n" + "--- a/Documentation/DocBook/kgdb.tmpl\n" + "+++ b/Documentation/DocBook/kgdb.tmpl\n" + "@@ -98,6 +98,14 @@\n" + " \"Kernel debugging\" select \"KGDB: kernel debugging with remote gdb\".\n" + " </para>\n" + " <para>\n" + "+ It is advised, but not required that you turn on the\n" + "+ CONFIG_FRAME_POINTER kernel option. This option inserts code to\n" + "+ into the compiled executable which saves the frame information in\n" + "+ registers or on the stack at different points which will allow a\n" + "+ debugger such as gdb to more accurately construct stack back traces\n" + "+ while debugging the kernel.\n" + "+ </para>\n" + "+ <para>\n" + " Next you should choose one of more I/O drivers to interconnect debugging\n" + " host and debugged target. Early boot debugging requires a KGDB\n" + " I/O driver that supports early debugging and the driver must be\n" + "diff --git a/lib/Kconfig.kgdb b/lib/Kconfig.kgdb\n" + "index 2cfd272..9b5d1d7 100644\n" + "--- a/lib/Kconfig.kgdb\n" + "+++ b/lib/Kconfig.kgdb\n" + "@@ -4,14 +4,17 @@ config HAVE_ARCH_KGDB\n" + " \n" + " menuconfig KGDB\n" + " \tbool \"KGDB: kernel debugging with remote gdb\"\n" + "-\tselect FRAME_POINTER\n" + " \tdepends on HAVE_ARCH_KGDB\n" + " \tdepends on DEBUG_KERNEL && EXPERIMENTAL\n" + " \thelp\n" + " \t If you say Y here, it will be possible to remotely debug the\n" + "-\t kernel using gdb. Documentation of kernel debugger is available\n" + "-\t at http://kgdb.sourceforge.net as well as in DocBook form\n" + "-\t in Documentation/DocBook/. If unsure, say N.\n" + "+\t kernel using gdb. It is recommended but not required, that\n" + "+\t you also turn on the kernel config option\n" + "+\t CONFIG_FRAME_POINTER to aid in producing more reliable stack\n" + "+\t backtraces in the external debugger. Documentation of\n" + "+\t kernel debugger is available at http://kgdb.sourceforge.net\n" + "+\t as well as in DocBook form in Documentation/DocBook/. If\n" + "+\t unsure, say N.\n" + " \n" + " if KGDB\n" + " \n" + "-- \n" + 1.5.5.1 -27a15e41c7ecce7738a3d13296edfa4fb828b8d7523e97f3fdb0048ae9b15fd3 +f01c52ce8ec2f98e2f0a7e470bc07283c85782c94a7c5961daebb78e5ba6cca1
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.