From nobody Tue Dec 2 02:30:42 2025 Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com [209.85.221.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D7B802EBB8F for ; Wed, 19 Nov 2025 15:46:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763567171; cv=none; b=N/k1T731gQl44sU/Lz2oTPrYx6SiUPEKaDoVHssHsJeIBzicNApWAoCb4bhMSSM0j932Ulx+Bf0ZZdUTFe1vdjslrRuv9okO0E0G+4FKUdHclT470XcKma2Oo80vuqTJbDxZHBmDK3fG+z2lcut1YqxzyMSCDzYVpIHWusHOctY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763567171; c=relaxed/simple; bh=wT8jmHzcNzOn5Qz+3QDhit2hy9V6GOAqUfnO1Z91iT4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Yul5Fhv+R4SJeP0fGl2khc95af1YNzPYoCHaJDyCiRAqz345eEqYlHdUmQKJrNSaJugFoG/RohoB3Jz0qlGlabkJunADBdeEncYkp7zktEGoIZ8+uGcbd2Q5g7QW2jSmV0yGa66yJNQgS61FSjuUlgwUHc9FdmHHJqX38Bz2IBw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=iKEE9S1x; arc=none smtp.client-ip=209.85.221.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="iKEE9S1x" Received: by mail-wr1-f46.google.com with SMTP id ffacd0b85a97d-42b2dc17965so5618676f8f.3 for ; Wed, 19 Nov 2025 07:46:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1763567162; x=1764171962; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=vjdG1/a5AFb8CurP464/OyK+i/niRA2OFjF7IXQ6EqU=; b=iKEE9S1x9QWbGHB/ePXFxV+MOYG1FQjEFKLupkcUbkIDj1MwKSWWC80B5kbXz527Uu +XrrAonfGs9qGc3eyA1TzIeotXf7kj+BVjN4suIrHDlzwVkBEz6MTYOwS0pnaQ4TwDjW np02CTwRAZTGAX4P5BJIGbgELcQkvyKUPNtmsnrychGRuzXzPfutsHnE2bDaRz7Qw3UL Hy2sn1q6aGUhuFRRo7QjslJ0yDKLJsUbaYSNAuGxCiVYaCgg6reHJpLV9v33MftCVQ8R R79z8CbM+8gYja96FcgAPvOTrPUBeYykCrc9acmjDKxn15o9cVyWZ6ZV7q64VV9jB6St nW2g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763567162; x=1764171962; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=vjdG1/a5AFb8CurP464/OyK+i/niRA2OFjF7IXQ6EqU=; b=X8J2cfCbRSFPHG5hGejHLg47hl004Mfg/EebmO1FJZqxETk770uLP7otllDjn7B7oT Nt9HZkzasMmVRCIgX6jJuq774zIm5Cuo6BPoqctUwgSzE0pag34duF9wPBUJRX0qEtby xT2f6WaHJr7HlYyUjs9V7Jot7VbFujg7pe4oGpObT2iQ/Txa6TIJU7jzR59g4B/3HR2X DM7Ko1yYNcPSR5o3HXvokjoJYZCNvkLCORMe+lU/U8TrkR72p0uS3UXnYE5K/yOVIXk0 j68dRYPKP80tl+onlxpbSpN0Lk9x5+KMG3jYUXrtHEPmk9lf9cz7ma9EBlHz7nheE9t0 Y3+Q== X-Forwarded-Encrypted: i=1; AJvYcCUA5E9wtbrEx1GPH212j4SS6zIze0H4OzOyUr6ww9zcv1W1CL7T9UYBU3jJLDZwHAdfbHSseNwINt/R9Lk=@vger.kernel.org X-Gm-Message-State: AOJu0YydS0S2NUYODuo8iA/+7oLzmGGq4mRKLSpgPuFjx0kbJfE3cxOq Y62c2yKhNcCZCYPYM51g4IyFipd8nHmmAgD8O01vRQ1lnWXqSUKW24d2y7OQjGz1lSs= X-Gm-Gg: ASbGnctyRX48L7wIuNMBOkObKtkYGXeSbkk4pF5FhVjJZ+QidiLgmVmmhprnprVYxN7 cDEKuPuXKgeciuGJBLb2U3hqaH1qEm0e9WIlyDeth3sjEPtRLpsyO1EvQvroybmeVyLd2/IHVTC nrp1Y24kNvJDhOTtiWvPOTqdOwdVaMavSbb4elRwMLvAEAf2sYPiJ2jOMqeVwbhFDWIos7/ZplM wcvhqarM4tlW21WIIUdIC3CQEDb5VzDozOiXRswr6g4kn+OLZO+amiy/HNBNcYRjQbG6ULysb74 nGjA/LliQJgOxFETvZL+SIPyTMSGkGx7LbrW85DJQRPPL35yzcyeUowEhwJG5wkyFBMuRpSP0R6 Bes9aEc/nbFAvh1O22UeVxc2PfoQWfNF6jSl/nLtFkbfj+V5Xr4CfdZwpp8BhrYf2Wr8817QhxW k9xOx3jnoUEWdEkktt+ycp4OinIvt3dA== X-Google-Smtp-Source: AGHT+IEYehjZzwQP4QY9mT+/5fGr+NBXUC5zHCL5MMz466EKwRwnbbkUf34pwbvo8KJX34cdlYI+Ug== X-Received: by 2002:a05:6000:657:b0:400:7e60:7ee0 with SMTP id ffacd0b85a97d-42b592d8549mr20248098f8f.0.1763567162304; Wed, 19 Nov 2025 07:46:02 -0800 (PST) Received: from eugen-station.. ([82.76.24.202]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-42b53dea1c9sm38765632f8f.0.2025.11.19.07.46.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 19 Nov 2025 07:46:01 -0800 (PST) From: Eugen Hristev To: linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, tglx@linutronix.de, andersson@kernel.org, pmladek@suse.com, rdunlap@infradead.org, corbet@lwn.net, david@redhat.com, mhocko@suse.com Cc: tudor.ambarus@linaro.org, mukesh.ojha@oss.qualcomm.com, linux-arm-kernel@lists.infradead.org, linux-hardening@vger.kernel.org, jonechou@google.com, rostedt@goodmis.org, linux-doc@vger.kernel.org, devicetree@vger.kernel.org, linux-remoteproc@vger.kernel.org, linux-arch@vger.kernel.org, tony.luck@intel.com, kees@kernel.org, Eugen Hristev Subject: [PATCH 26/26] meminspect: Add Kinfo compatible driver Date: Wed, 19 Nov 2025 17:44:27 +0200 Message-ID: <20251119154427.1033475-27-eugen.hristev@linaro.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20251119154427.1033475-1-eugen.hristev@linaro.org> References: <20251119154427.1033475-1-eugen.hristev@linaro.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" With this driver, the registered regions are copied to a shared memory zone at register time. The shared memory zone is supplied via OF. This driver will select only regions that are of interest, and keep only addresses. The format of the list is Kinfo compatible, with devices like Google Pixel phone. The firmware is only interested in some symbols' addresses. Signed-off-by: Eugen Hristev --- MAINTAINERS | 1 + kernel/meminspect/Kconfig | 10 ++ kernel/meminspect/Makefile | 1 + kernel/meminspect/kinfo.c | 289 +++++++++++++++++++++++++++++++++++++ 4 files changed, 301 insertions(+) create mode 100644 kernel/meminspect/kinfo.c diff --git a/MAINTAINERS b/MAINTAINERS index 8034940d0b1e..9cba0e472e01 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -16168,6 +16168,7 @@ MEMINSPECT KINFO DRIVER M: Eugen Hristev S: Maintained F: Documentation/devicetree/bindings/misc/google,kinfo.yaml +F: kernel/meminspect/kinfo.c =20 MEMBLOCK AND MEMORY MANAGEMENT INITIALIZATION M: Mike Rapoport diff --git a/kernel/meminspect/Kconfig b/kernel/meminspect/Kconfig index 8680fbf0e285..396510908e47 100644 --- a/kernel/meminspect/Kconfig +++ b/kernel/meminspect/Kconfig @@ -18,3 +18,13 @@ config MEMINSPECT Note that modules using this feature must be rebuilt if option changes. =20 +config MEMINSPECT_KINFO + tristate "Shared memory KInfo compatible driver" + depends on MEMINSPECT + help + Say y here to enable the Shared memory KInfo compatible driver + With this driver, the registered regions are copied to a shared + memory zone at register time. + The shared memory zone is supplied via OF. + This driver will select only regions that are of interest, + and keep only addresses. The format of the list is Kinfo compatible. diff --git a/kernel/meminspect/Makefile b/kernel/meminspect/Makefile index 09fd55e6d9cf..283604d892e5 100644 --- a/kernel/meminspect/Makefile +++ b/kernel/meminspect/Makefile @@ -1,3 +1,4 @@ # SPDX-License-Identifier: GPL-2.0 =20 obj-$(CONFIG_MEMINSPECT) +=3D meminspect.o +obj-$(CONFIG_MEMINSPECT_KINFO) +=3D kinfo.o diff --git a/kernel/meminspect/kinfo.c b/kernel/meminspect/kinfo.c new file mode 100644 index 000000000000..62f8ee7a66a9 --- /dev/null +++ b/kernel/meminspect/kinfo.c @@ -0,0 +1,289 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * + * Copyright 2002 Rusty Russell IBM Corporation + * Copyright 2021 Google LLC + * Copyright 2025 Linaro Ltd. Eugen Hristev + */ +#include +#include +#include +#include +#include +#include +#include +#include + +#define BUILD_INFO_LEN 256 +#define DEBUG_KINFO_MAGIC 0xcceeddff + +/* + * Header structure must be byte-packed, since the table is provided to + * bootloader. + */ +struct kernel_info { + /* For kallsyms */ + u8 enabled_all; + u8 enabled_base_relative; + u8 enabled_absolute_percpu; + u8 enabled_cfi_clang; + u32 num_syms; + u16 name_len; + u16 bit_per_long; + u16 module_name_len; + u16 symbol_len; + u64 _relative_pa; + u64 _text_pa; + u64 _stext_pa; + u64 _etext_pa; + u64 _sinittext_pa; + u64 _einittext_pa; + u64 _end_pa; + u64 _offsets_pa; + u64 _names_pa; + u64 _token_table_pa; + u64 _token_index_pa; + u64 _markers_pa; + u64 _seqs_of_names_pa; + + /* For frame pointer */ + u32 thread_size; + + /* For virt_to_phys */ + u64 swapper_pg_dir_pa; + + /* For linux banner */ + u8 last_uts_release[__NEW_UTS_LEN]; + + /* Info of running build */ + u8 build_info[BUILD_INFO_LEN]; + + /* For module kallsyms */ + u32 enabled_modules_tree_lookup; + u32 mod_mem_offset; + u32 mod_kallsyms_offset; +} __packed; + +struct kernel_all_info { + u32 magic_number; + u32 combined_checksum; + struct kernel_info info; +} __packed; + +struct debug_kinfo { + struct device *dev; + void *all_info_addr; + size_t all_info_size; + struct notifier_block nb; +}; + +static void update_kernel_all_info(struct kernel_all_info *all_info) +{ + struct kernel_info *info; + u32 *checksum_info; + int index; + + all_info->magic_number =3D DEBUG_KINFO_MAGIC; + all_info->combined_checksum =3D 0; + + info =3D &all_info->info; + checksum_info =3D (u32 *)info; + for (index =3D 0; index < sizeof(*info) / sizeof(u32); index++) + all_info->combined_checksum ^=3D checksum_info[index]; +} + +static u8 global_build_info[BUILD_INFO_LEN]; + +static int build_info_set(const char *str, const struct kernel_param *kp) +{ + size_t build_info_size =3D sizeof(global_build_info); + + if (strlen(str) > build_info_size) + return -ENOMEM; + memcpy(global_build_info, str, min(build_info_size - 1, strlen(str))); + return 0; +} + +static const struct kernel_param_ops build_info_op =3D { + .set =3D build_info_set, +}; + +module_param_cb(build_info, &build_info_op, NULL, 0200); +MODULE_PARM_DESC(build_info, "Write build info to field 'build_info' of de= bug kinfo."); + +static void __maybe_unused register_kinfo_region(void *priv, + const struct inspect_entry *e) +{ + struct debug_kinfo *kinfo =3D priv; + struct kernel_all_info *all_info =3D kinfo->all_info_addr; + struct kernel_info *info =3D &all_info->info; + struct uts_namespace *uts; + u64 paddr; + + if (e->pa) + paddr =3D e->pa; + else + paddr =3D __pa(e->va); + + switch (e->id) { + case MEMINSPECT_ID__sinittext: + info->_sinittext_pa =3D paddr; + break; + case MEMINSPECT_ID__einittext: + info->_einittext_pa =3D paddr; + break; + case MEMINSPECT_ID__end: + info->_end_pa =3D paddr; + break; + case MEMINSPECT_ID__text: + info->_text_pa =3D paddr; + break; + case MEMINSPECT_ID__stext: + info->_stext_pa =3D paddr; + break; + case MEMINSPECT_ID__etext: + info->_etext_pa =3D paddr; + break; + case MEMINSPECT_ID_kallsyms_num_syms: + info->num_syms =3D *(__u32 *)e->va; + break; + case MEMINSPECT_ID_kallsyms_relative_base: + info->_relative_pa =3D (u64)__pa(*(u64 *)e->va); + break; + case MEMINSPECT_ID_kallsyms_offsets: + info->_offsets_pa =3D paddr; + break; + case MEMINSPECT_ID_kallsyms_names: + info->_names_pa =3D paddr; + break; + case MEMINSPECT_ID_kallsyms_token_table: + info->_token_table_pa =3D paddr; + break; + case MEMINSPECT_ID_kallsyms_token_index: + info->_token_index_pa =3D paddr; + break; + case MEMINSPECT_ID_kallsyms_markers: + info->_markers_pa =3D paddr; + break; + case MEMINSPECT_ID_kallsyms_seqs_of_names: + info->_seqs_of_names_pa =3D paddr; + break; + case MEMINSPECT_ID_swapper_pg_dir: + info->swapper_pg_dir_pa =3D paddr; + break; + case MEMINSPECT_ID_init_uts_ns: + if (!e->va) + return; + uts =3D e->va; + strscpy(info->last_uts_release, uts->name.release, __NEW_UTS_LEN); + break; + default: + break; + }; + + update_kernel_all_info(all_info); +} + +static int kinfo_notifier_cb(struct notifier_block *nb, + unsigned long code, void *entry) +{ + struct debug_kinfo *kinfo =3D container_of(nb, struct debug_kinfo, nb); + + if (code =3D=3D MEMINSPECT_NOTIFIER_ADD) + register_kinfo_region(kinfo, entry); + + return NOTIFY_DONE; +} + +static int debug_kinfo_probe(struct platform_device *pdev) +{ + struct kernel_all_info *all_info; + struct device *dev =3D &pdev->dev; + struct device_node *mem_region; + struct reserved_mem *rmem; + struct debug_kinfo *kinfo; + struct kernel_info *info; + + mem_region =3D of_parse_phandle(dev->of_node, "memory-region", 0); + if (!mem_region) + return dev_err_probe(dev, -ENODEV, "no such memory-region\n"); + + rmem =3D of_reserved_mem_lookup(mem_region); + if (!rmem) + return dev_err_probe(dev, -ENODEV, "no such reserved mem of node name %s= \n", + dev->of_node->name); + + /* Need to wait for reserved memory to be mapped */ + if (!rmem->priv) + return -EPROBE_DEFER; + + if (!rmem->base || !rmem->size) + dev_err_probe(dev, -EINVAL, "unexpected reserved memory\n"); + + if (rmem->size < sizeof(struct kernel_all_info)) + dev_err_probe(dev, -EINVAL, "reserved memory size too small\n"); + + kinfo =3D devm_kzalloc(dev, sizeof(*kinfo), GFP_KERNEL); + if (!kinfo) + return -ENOMEM; + platform_set_drvdata(pdev, kinfo); + + kinfo->dev =3D dev; + + kinfo->all_info_addr =3D rmem->priv; + kinfo->all_info_size =3D rmem->size; + + all_info =3D kinfo->all_info_addr; + + memset(all_info, 0, sizeof(struct kernel_all_info)); + info =3D &all_info->info; + info->enabled_all =3D IS_ENABLED(CONFIG_KALLSYMS_ALL); + info->enabled_absolute_percpu =3D IS_ENABLED(CONFIG_KALLSYMS_ABSOLUTE_PER= CPU); + info->enabled_base_relative =3D IS_ENABLED(CONFIG_KALLSYMS_BASE_RELATIVE); + info->enabled_cfi_clang =3D IS_ENABLED(CONFIG_CFI_CLANG); + info->name_len =3D KSYM_NAME_LEN; + info->bit_per_long =3D BITS_PER_LONG; + info->module_name_len =3D MODULE_NAME_LEN; + info->symbol_len =3D KSYM_SYMBOL_LEN; + info->thread_size =3D THREAD_SIZE; + info->enabled_modules_tree_lookup =3D IS_ENABLED(CONFIG_MODULES_TREE_LOOK= UP); + info->mod_mem_offset =3D offsetof(struct module, mem); + info->mod_kallsyms_offset =3D offsetof(struct module, kallsyms); + + memcpy(info->build_info, global_build_info, strlen(global_build_info)); + + kinfo->nb.notifier_call =3D kinfo_notifier_cb; + + meminspect_notifier_register(&kinfo->nb); + meminspect_lock_traverse(kinfo, register_kinfo_region); + + return 0; +} + +static void debug_kinfo_remove(struct platform_device *pdev) +{ + struct debug_kinfo *kinfo =3D platform_get_drvdata(pdev); + + meminspect_notifier_unregister(&kinfo->nb); +} + +static const struct of_device_id debug_kinfo_of_match[] =3D { + { .compatible =3D "google,debug-kinfo" }, + {}, +}; +MODULE_DEVICE_TABLE(of, debug_kinfo_of_match); + +static struct platform_driver debug_kinfo_driver =3D { + .probe =3D debug_kinfo_probe, + .remove =3D debug_kinfo_remove, + .driver =3D { + .name =3D "debug-kinfo", + .of_match_table =3D of_match_ptr(debug_kinfo_of_match), + }, +}; +module_platform_driver(debug_kinfo_driver); + +MODULE_AUTHOR("Eugen Hristev "); +MODULE_AUTHOR("Jone Chou "); +MODULE_DESCRIPTION("meminspect Kinfo Driver"); +MODULE_LICENSE("GPL"); --=20 2.43.0