Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
From:   Stephen Brennan <stephen.s.brennan@oracle.com>
To:     Arnd Bergmann <arnd@arndb.de>
Cc:     Stephen Brennan <stephen.s.brennan@oracle.com>,
        linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-debuggers@vger.kernel.org
Subject: [PATCH 1/1] kernel/config: Introduce CONFIG_DEBUG_INFO_IKCONFIG
Date:   Wed,  4 Oct 2023 09:58:04 -0700
Message-Id: <20231004165804.659482-2-stephen.s.brennan@oracle.com>
In-Reply-To: <20231004165804.659482-1-stephen.s.brennan@oracle.com>
References: <20231004165804.659482-1-stephen.s.brennan@oracle.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?YKwwqJ2YM5JKEVW9j/SpK4lGxn0Bf+pm0lq2wnObW7DP5STVDIKapuuUHnSN?=
 =?us-ascii?Q?9cWt6SajPXIhNYnm67EJyTJnNpm+M7djVRQ7uyVYx9Ga8D3kEY5zyDhz2r0z?=
 =?us-ascii?Q?NloI8Isycd+PJuRmhzdZJuc+v8HR9a5Ga0UotTs8IewAwtedcR2erRgCBqCd?=
 =?us-ascii?Q?Lon+AwTxmfFJcWGR5N/IEyDpqaGepe7Q7KNLOqSM4hkhSH5C1Uw+47wzfXmw?=
 =?us-ascii?Q?zu+1P+fnE7xYShpnwTASkfuU3s0iv5hx0Q9UeWpSHRU4+zmfHs/cTcXWUJmK?=
 =?us-ascii?Q?uK0BHr3I3gNKpRN60dfNgFnt657oChCYAYdNQ27P7mqCPeXEHtQakCptI11g?=
 =?us-ascii?Q?W7Whty8uSekgJk/PXAu7TjKST1lzPwIdPRjozNqD8IxZiXaUSPM0VOxcl16K?=
 =?us-ascii?Q?7JdHujBPCftSrpUKYvoxDPLvYSMXRbM+fK74LtZyj2M0JhyYCH9N3MQ4CL3B?=
 =?us-ascii?Q?P7zx3hbJwErmgZ6sM9SywP7CTrOtv+nbA2gN4ooiOfsTNA9B9CgrKjAG0Iga?=
 =?us-ascii?Q?LT9LkGyD6cFKoTIooPyH4+MeXsvVqb9ELKweo4t8nU+qs0erSoUKt8YZIpK2?=
 =?us-ascii?Q?hcUv8I2bpqZGP+ECoJ7VmUv9MgI8gZ+0oBlZeKeAyNb3LtkWh5HzeKNuNW4A?=
 =?us-ascii?Q?dRx611+cCErAazC22hWLqXH6wM9Gu1qwMxsFgJ/BEhBz8oPHyPJrVxlllTzq?=
 =?us-ascii?Q?VP61HIA+O/ONTXkRjYqkmEh5uPTb3H8Mx3CMlBoC6Pjt7Yg8vY/yKQgddp5C?=
 =?us-ascii?Q?umAEQhsLRRMnlStslYHcIWOvn+c3AnUnp844cbI1OP3c0tBmClx2mENVJbMH?=
 =?us-ascii?Q?8jTUgcunYpif8XmCOvMTzOc8VJxSEmBXf/AZFvZ9jIrPvHj/wjEY6VF9leKW?=
 =?us-ascii?Q?dlTySGdgna9Hd6UiUzOn1F9XU/8zkT6YkTBATA1tnczvqP9ciM7hMbpnTe1U?=
 =?us-ascii?Q?zMYrJJBvv8kAHvNe2ZL72nFDOougRr0nVhMKE73Hayy0x81fcBjIcbbV1tmO?=
 =?us-ascii?Q?+cnPiiViYVfd5cFglDeIj4+pSB82/ipvuN+3j1buM12pTG+b2V0sYK/qsNZE?=
 =?us-ascii?Q?u1HV3DaYFC/wpQv249asAnvkuDOqsT4Ps7pFvYTY58ZhhMzC5PugneCFrBHx?=
 =?us-ascii?Q?xQekO3GcgLJip+0pickV8doTSIoOvVIH5nmlfhSE5sckikgmlePTR6xe+rG8?=
 =?us-ascii?Q?xUYKi3VIHsjGvBtnQvYxJfeCHwzXL4HwpLiw46Muwz+QT+lm/wGNF65MBWz2?=
 =?us-ascii?Q?PT/dKMyisVv0kF4qCPAK6paAMKwJ5jIJhGZb1Hi2kRCDuyXk+YsmxedEIOAK?=
 =?us-ascii?Q?6bVCxFpaUgc7ElHIxsiqpdpzWr1v9BJifcL8tl877xkbQFIhaIh6uKoOP299?=
 =?us-ascii?Q?qBoL5KadQVUSwO8Qf5vbIFjusOXfATBMCChR/TF8qY8W/84mqHgeZLlXyzSF?=
 =?us-ascii?Q?LiNFG8UdRqBoWVc7d/kmdz8QoFyN6+hUz5u3hkFmcQtEEl9Nz6EYY/v8xHat?=
 =?us-ascii?Q?Thzc2uqGw1fQZKgn3cRqONUQZGvFlVjK5v/JaR3mr5XGNO5MYCbJtB1BSzd+?=
 =?us-ascii?Q?Wt9Q8TWzMAXWSw36+IiURLjiohraK6yhp9LrgoIaebZW9PxR8NtE03MqK0hA?=
 =?us-ascii?Q?ghI7MmNjGh+sc82zsW4sxxL3dDujV1Nb9kBqeFbBty9NOr0Se0vsoV25ds+w?=
 =?us-ascii?Q?nFy15zZkAt4gNluoH4YrlWixKXU=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 7pAbHF6vZFnLFhij0Lj4crYsu1aAio0aWo8o4phvVrMmnPnV4cFuC5O3nCDE/EXHuM/aN2tvTWHsYIV5apDIeM7F5uI01euIMRnJy4LxIIfZkiIUkVQEinQaURdZnbE9aYx7aAEnUDos5NdyZsN8Q+LmKMBUlWtzY+bLlhDZG7Qj1lAlacgA1RfL0x9nNoTQ427Su898kVtpZ1GdC8gQWkdqfwM/lUinX+lO1VpkrmjfmGAA7vnESdi1BbMkP63M/rd4xNPPtHpK928daSDl09rDM8tE/UNRvAxu4ItFDxieeHeZ6xpVEKKwKdl2xpIMiLESQLd4EA8JqCcSNyexaErIf+hKvzOD+ejPcAIKhrAX5OcauaV3pqnfJBwvccQsZYsaZUzsRVR5fNFfnyw88MF24fFT6ORveKIQSxql4TQMFriWip/Cqo97uOAJ9FYFnnDhQ2sX0ShocRf5B9izPbE3/yj2C9H3pDAhXa4N7vBNZnAbiQUzbAH31OttA4itq3ykiJEbzrLkR0uBWh9w3Elzi5B4OlbZVewahigysvnp0MlYVn2cVF5b24FmNrE3DOy3zMRXyhuvbUOdP+ajcBAqtROmnZDut87fK3XkBNX/U5sfOnBmumr32UdYj1v3F6XBgY3YHrZCFXyjeKqFApeeFGgbmNTEFWrrdWRVeU4c8JYErbildPeVKCP83He3HU7S4MTCOtYcq9CY9832ulxcGTh3KA7fh9AR0RHEDxtBgehfVw7MIj/615h+f9eUyVDEkDoW8rPNjls6SmmO1qBhgMqcMlY7bOsK7MdzsO0=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 487d72c3-6ec0-4d66-46d2-08dbc4fb1605
X-MS-Exchange-CrossTenant-AuthSource: PH8PR10MB6597.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Oct 2023 16:58:08.9689
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: GoBekCbu3mJgia8peOtycLQr6ITTiZ+UdmzeHgUxMNYbzuhZuoAOoMqAFSliiLiTloRsecpBJjOBsX6UcgAFFdUHNURUHkBMMZqBcmUHgQk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM3PR10MB7912
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.267,Aquarius:18.0.980,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2023-10-04_08,2023-10-02_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 phishscore=0 mlxlogscore=999
 suspectscore=0 malwarescore=0 spamscore=0 bulkscore=0 mlxscore=0
 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2309180000 definitions=main-2310040123
X-Proofpoint-GUID: 4nNXAUgbiYDQAhv2JriJv9uijGQM1X6i
X-Proofpoint-ORIG-GUID: 4nNXAUgbiYDQAhv2JriJv9uijGQM1X6i
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Wed, 04 Oct 2023 09:59:09 -0700 (PDT)

The option CONFIG_IKCONFIG allows the gzip compressed kernel
configuration to be included into vmlinux or a module. In these cases,
debuggers can access the config data and use it to adjust their behavior
according to the configuration. However, distributions rarely enable
this, likely because it uses a fair bit of kernel memory which cannot be
swapped out.

This means that in practice, the kernel configuration is rarely
available to debuggers.

So, introduce an alternative, CONFIG_DEBUG_INFO_IKCONFIG. This strategy,
which is only available if IKCONFIG is not already built-in, adds a
section ".debug_linux_ikconfig", to the vmlinux ELF. It will be stripped
out of the final images, but will remain in the debuginfo files. So
debuggers which rely on vmlinux debuginfo can have access to the kernel
configuration, without incurring a cost to the kernel at runtime.

Signed-off-by: Stephen Brennan <stephen.s.brennan@oracle.com>
---
 include/asm-generic/vmlinux.lds.h |  3 ++-
 kernel/Makefile                   |  1 +
 kernel/configs-debug.S            | 18 ++++++++++++++++++
 lib/Kconfig.debug                 | 14 ++++++++++++++
 4 files changed, 35 insertions(+), 1 deletion(-)
 create mode 100644 kernel/configs-debug.S

diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
index 9c59409104f6..025b0bfe17bf 100644
--- a/include/asm-generic/vmlinux.lds.h
+++ b/include/asm-generic/vmlinux.lds.h
@@ -824,7 +824,8 @@
 		.comment 0 : { *(.comment) }				\
 		.symtab 0 : { *(.symtab) }				\
 		.strtab 0 : { *(.strtab) }				\
-		.shstrtab 0 : { *(.shstrtab) }
+		.shstrtab 0 : { *(.shstrtab) }				\
+		.debug_linux_ikconfig 0 : { *(.debug_linux_ikconfig) }
 
 #ifdef CONFIG_GENERIC_BUG
 #define BUG_TABLE							\
diff --git a/kernel/Makefile b/kernel/Makefile
index 3947122d618b..e2f517a10f04 100644
--- a/kernel/Makefile
+++ b/kernel/Makefile
@@ -138,6 +138,7 @@ KCSAN_SANITIZE_stackleak.o := n
 KCOV_INSTRUMENT_stackleak.o := n
 
 obj-$(CONFIG_SCF_TORTURE_TEST) += scftorture.o
+obj-$(CONFIG_DEBUG_INFO_IKCONFIG) += configs-debug.o
 
 $(obj)/configs.o: $(obj)/config_data.gz
 
diff --git a/kernel/configs-debug.S b/kernel/configs-debug.S
new file mode 100644
index 000000000000..d0dd5c2f7bd5
--- /dev/null
+++ b/kernel/configs-debug.S
@@ -0,0 +1,18 @@
+/* SPDX-License-Identifier: GPL-2.0-only
+ *
+ * Inline kernel configuration for debuginfo files
+ *
+ * Copyright (c) 2023, Oracle and/or its affiliates.
+ */
+
+/*
+ * By using the same "IKCFG_ST" and "IKCFG_ED" markers found in configs.c, we
+ * can ensure that the resulting debuginfo files can be read by
+ * scripts/extract-ikconfig. Unfortunately, this means that the contents of the
+ * section cannot be directly extracted and used. Since debuggers should be able
+ * to trim these markers off trivially, this is a good tradeoff.
+ */
+	.section .debug_linux_ikconfig
+	.ascii "IKCFG_ST"
+	.incbin "kernel/config_data.gz"
+	.ascii "IKCFG_ED"
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index fa307f93fa2e..4ebd1bcd49c7 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -429,6 +429,20 @@ config GDB_SCRIPTS
 	  instance. See Documentation/dev-tools/gdb-kernel-debugging.rst
 	  for further details.
 
+config DEBUG_INFO_IKCONFIG
+	bool "Embed KConfig in debuginfo, if not already present"
+	depends on IKCONFIG!=y
+	help
+	  This provides the gzip-compressed KConfig information in an ELF
+	  section called .ikconfig which will be stripped out of the final
+	  bootable image, but remain in the debuginfo. Debuggers that are aware
+	  of this can use this to customize their behavior to the kernel
+	  configuration, without requiring the configuration information to be
+	  stored in the kernel like CONFIG_IKCONFIG does. This configuration is
+	  unnecessary when CONFIG_IKCONFIG is enabled, since the data can be
+	  found in the .rodata section in that case (see
+	  scripts/extract-ikconfig).
+
 endif # DEBUG_INFO
 
 config FRAME_WARN
-- 
2.39.3