From nobody Thu Dec 18 22:23:39 2025 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by smtp.subspace.kernel.org (Postfix) with ESMTP id CA95623E321; Wed, 2 Apr 2025 12:47:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=13.77.154.182 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743598070; cv=none; b=ZceM3prtnm/F/zGqubCmGCGSYys2cx4G6Ll3qzw2MF7WwCRvEoFKOYG9ZSeJA9H5Sk92RRUtbBxhSUscACJo4YOB1Allr6p1ENV+AlRqCNzpHJheGd1AliEIao20uluy7lDlRVWfFI7Bl6XuMKVOOP2BsnSx83FAciISgKB/+nw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743598070; c=relaxed/simple; bh=1kvwnO3H6Jss1uxMPsuR/y6ko+f8O5oM2Dd+ugazDhs=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=td3x415c1uYZfmgnX5oBlvbpHd5ZX7gcmrq8HnL1ZUPQ9jsU4Q9R0S7QIYY0LzZsCj2MXGIZ/RfbQDexDsiI7ratETClV51wkfXAdIucUBQJGMBMdFA9KYR24sLeqFLXJdOeIBe/s1EhDXad0Ttf6LXq5IEvOneZGZveCV26UsE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.microsoft.com; spf=pass smtp.mailfrom=linux.microsoft.com; dkim=pass (1024-bit key) header.d=linux.microsoft.com header.i=@linux.microsoft.com header.b=P5T2QWeJ; arc=none smtp.client-ip=13.77.154.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.microsoft.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.microsoft.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.microsoft.com header.i=@linux.microsoft.com header.b="P5T2QWeJ" Received: from DESKTOP-VOT081N.hsd1.ga.comcast.net (unknown [20.114.144.49]) by linux.microsoft.com (Postfix) with ESMTPSA id 52A4B2041315; Wed, 2 Apr 2025 05:47:46 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 52A4B2041315 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1743598068; bh=c7xlCRrPrh0uSEOIDsBq+I5m+1pDMPQECAnp8rbF0oQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=P5T2QWeJRhqYxLPKlnx5GmyWPp1wX7x9pWcSJBteHGJestyj/PjJimXt+rwAnyEgJ XUsoXOkWgrCzG8b+Lavk7cqtnpzu0QiJuXdWyevgdzYeNF/hEoo84rG90F1YS6mSm/ WQZExhNNptEGi8pirH5PbuJM2sI8Jt857nBESSL0= From: steven chen To: zohar@linux.ibm.com, stefanb@linux.ibm.com, roberto.sassu@huaweicloud.com, roberto.sassu@huawei.com, eric.snowberg@oracle.com, ebiederm@xmission.com, paul@paul-moore.com, code@tyhicks.com, bauermann@kolabnow.com, linux-integrity@vger.kernel.org, kexec@lists.infradead.org, linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org Cc: madvenka@linux.microsoft.com, nramas@linux.microsoft.com, James.Bottomley@HansenPartnership.com, bhe@redhat.com, vgoyal@redhat.com, dyoung@redhat.com Subject: [PATCH v11 5/9] ima: kexec: define functions to copy IMA log at soft boot Date: Wed, 2 Apr 2025 05:47:18 -0700 Message-ID: <20250402124725.5601-6-chenste@linux.microsoft.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250402124725.5601-1-chenste@linux.microsoft.com> References: <20250402124725.5601-1-chenste@linux.microsoft.com> 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" The IMA log is currently copied to the new kernel during kexec 'load'=20 using ima_dump_measurement_list(). However, the log copied at kexec=20 'load' may result in loss of IMA measurements that only occurred after kexec "load'. Therefore, the log needs to be copied during kexec=20 'execute'. Setup the needed infrastructure to move the IMA log copy from kexec 'load' to 'execute'. Define a new IMA hook ima_update_kexec_buffer() as a stub function. It will be used to call ima_dump_measurement_list() during kexec 'execute'. Implement ima_kexec_post_load() function to be invoked after the new=20 Kernel image has been loaded for kexec. ima_kexec_post_load() maps the=20 IMA buffer to a segment in the newly loaded Kernel. It also registers=20 the reboot notifier_block to trigger ima_update_kexec_buffer() at=20 kexec 'execute'. Set the priority of register_reboot_notifier to INT_MIN to ensure that the IMA log copy operation will happen at the end of the operation chain, which is crucial for maintaining the integrity of the logs Signed-off-by: Tushar Sugandhi Cc: Eric Biederman Cc: Baoquan He =20 Cc: Vivek Goyal Cc: Dave Young Signed-off-by: steven chen Reviewed-by: Stefan Berger Reviewed-by: Mimi Zohar --- include/linux/ima.h | 3 ++ security/integrity/ima/ima_kexec.c | 47 ++++++++++++++++++++++++++++++ 2 files changed, 50 insertions(+) diff --git a/include/linux/ima.h b/include/linux/ima.h index 0bae61a15b60..8e29cb4e6a01 100644 --- a/include/linux/ima.h +++ b/include/linux/ima.h @@ -32,6 +32,9 @@ static inline void ima_appraise_parse_cmdline(void) {} =20 #ifdef CONFIG_IMA_KEXEC extern void ima_add_kexec_buffer(struct kimage *image); +extern void ima_kexec_post_load(struct kimage *image); +#else +static inline void ima_kexec_post_load(struct kimage *image) {} #endif =20 #else diff --git a/security/integrity/ima/ima_kexec.c b/security/integrity/ima/im= a_kexec.c index 7e0a19c3483f..e79f6caf895b 100644 --- a/security/integrity/ima/ima_kexec.c +++ b/security/integrity/ima/ima_kexec.c @@ -12,10 +12,14 @@ #include #include #include +#include +#include #include "ima.h" =20 #ifdef CONFIG_IMA_KEXEC +static bool ima_kexec_update_registered; static struct seq_file ima_kexec_file; +static void *ima_kexec_buffer; =20 static void ima_free_kexec_file_buf(struct seq_file *sf) { @@ -162,6 +166,49 @@ void ima_add_kexec_buffer(struct kimage *image) kexec_dprintk("kexec measurement buffer for the loaded kernel at 0x%lx.\n= ", kbuf.mem); } + +/* + * Called during kexec execute so that IMA can update the measurement list. + */ +static int ima_update_kexec_buffer(struct notifier_block *self, + unsigned long action, void *data) +{ + return NOTIFY_OK; +} + +static struct notifier_block update_buffer_nb =3D { + .notifier_call =3D ima_update_kexec_buffer, + .priority =3D INT_MIN +}; + +/* + * Create a mapping for the source pages that contain the IMA buffer + * so we can update it later. + */ +void ima_kexec_post_load(struct kimage *image) +{ + if (ima_kexec_buffer) { + kimage_unmap_segment(ima_kexec_buffer); + ima_kexec_buffer =3D NULL; + } + + if (!image->ima_buffer_addr) + return; + + ima_kexec_buffer =3D kimage_map_segment(image, + image->ima_buffer_addr, + image->ima_buffer_size); + if (!ima_kexec_buffer) { + pr_err("Could not map measurements buffer.\n"); + return; + } + + if (!ima_kexec_update_registered) { + register_reboot_notifier(&update_buffer_nb); + ima_kexec_update_registered =3D true; + } +} + #endif /* IMA_KEXEC */ =20 /* --=20 2.25.1