* [PATCH 0/4] CRASH_ZEROIZE: Wipe secrets before kdump
@ 2026-07-31 16:27 Jan Sebastian Götte
0 siblings, 0 replies; 3+ messages in thread
From: Jan Sebastian Götte @ 2026-07-31 16:27 UTC (permalink / raw)
To: Jan Sebastian Götte
Cc: devicetree, linux-kernel, kexec, keyrings, linux-mm,
linux-security-module, linux-integrity
I'm using linux on an embedded target in a Hardware Security Module-like
application. One requirement is that I want the system to be able to
quickly erase its memory when it detects physical tampering. I'm
approaching that by using kdump to load into a small payload that
instead of dumping RAM, erases RAM from start to end. However, writing
all of RAM, especially on an embedded target, is rather slow. For this
reason, I propose the mechanism in this patch series:
Add CONFIG_CRASH_ZEROIZE (default off), which when enabled makes various
subsystems handling secret data do a quick, targeted wipe of these
secrets before kdump. This behavior might also be interesting in cases
where you run a normal kdump kernel but you still want to keep things
like fde crypto keys out of these dumps.
CONFIG_CRASH_ZEROIZE is a best effort, defense in depth solution. There
are circumstances, such as when a panic is triggered after memory
corruption, or when a panic interrupts some operation that mutates data
structures under locks, when the kernel cannot safely wipe some memory
areas. The handlers proposed in this series will just print a warning
and skip the affected areas in this case.
This series introduces two handlers as a starting point: One for kernel
keyrings, and one for secretmem. Future places where such handlers could
be added would be for example drivers for crypto accelerators.
The patch series applies on top of linux-next but should work on 7.0.0,
too. I've tested the patches on a Arduino uno Q (Qualcomm QRB2210)
embedded target.
Jan Sebastian Götte (4):
of/kexec: fix typo in comment (usable-memory-range)
kexec: add CRASH_ZEROIZE to wipe secrets before kdump
mm/secretmem: zeroize secret pages before kdump
security/keys: zeroize key payloads before kdump
drivers/of/kexec.c | 2 +-
include/linux/crash_core.h | 5 +++
include/linux/key-type.h | 9 ++++
kernel/Kconfig.kexec | 8 ++++
kernel/crash_core.c | 18 ++++++++
mm/secretmem.c | 50 +++++++++++++++++++++++
security/keys/big_key.c | 15 +++++++
security/keys/encrypted-keys/encrypted.c | 12 ++++++
security/keys/key.c | 44 ++++++++++++++++++++
security/keys/trusted-keys/trusted_core.c | 14 +++++++
security/keys/user_defined.c | 11 +++++
11 files changed, 187 insertions(+), 1 deletion(-)
--
2.53.0
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/4] CRASH_ZEROIZE: Wipe secrets before kdump
[not found] <20260731154608.153258-1-linux@jaseg.de>
@ 2026-08-01 14:03 ` Baoquan He
2026-08-01 16:31 ` Jan Sebastian Götte
0 siblings, 1 reply; 3+ messages in thread
From: Baoquan He @ 2026-08-01 14:03 UTC (permalink / raw)
To: Jan Sebastian Götte
Cc: Andrew Morton, Mike Rapoport, Pasha Tatashin, Pratyush Yadav,
Dave Young, David Howells, Jarkko Sakkinen, Paul Moore,
James Morris, Serge E. Hallyn, Mimi Zohar, James Bottomley,
Rob Herring, Saravana Kannan, Coiby Xu, devicetree, linux-kernel,
kexec, keyrings, linux-mm, linux-security-module, linux-integrity
On 07/31/26 at 05:46pm, Jan Sebastian Götte wrote:
> I'm using linux on an embedded target in a Hardware Security Module-like
> application. One requirement is that I want the system to be able to
> quickly erase its memory when it detects physical tampering. I'm
> approaching that by using kdump to load into a small payload that
> instead of dumping RAM, erases RAM from start to end. However, writing
> all of RAM, especially on an embedded target, is rather slow. For this
> reason, I propose the mechanism in this patch series:
>
> Add CONFIG_CRASH_ZEROIZE (default off), which when enabled makes various
> subsystems handling secret data do a quick, targeted wipe of these
> secrets before kdump. This behavior might also be interesting in cases
> where you run a normal kdump kernel but you still want to keep things
> like fde crypto keys out of these dumps.
Please check below patchset, you both seem to have the similar
requirement. Please go there to discuss.
[RFC PATCH 0/4] panic: a pre kdump notifier list for hypervisor upcalls
Note that we usually dont' want to run a lot of work after panic and
before jumping into kdump kernel.
>
> CONFIG_CRASH_ZEROIZE is a best effort, defense in depth solution. There
> are circumstances, such as when a panic is triggered after memory
> corruption, or when a panic interrupts some operation that mutates data
> structures under locks, when the kernel cannot safely wipe some memory
> areas. The handlers proposed in this series will just print a warning
> and skip the affected areas in this case.
>
> This series introduces two handlers as a starting point: One for kernel
> keyrings, and one for secretmem. Future places where such handlers could
> be added would be for example drivers for crypto accelerators.
>
> The patch series applies on top of linux-next but should work on 7.0.0,
> too. I've tested the patches on a Arduino uno Q (Qualcomm QRB2210)
> embedded target.
>
> Jan Sebastian Götte (4):
> of/kexec: fix typo in comment (usable-memory-range)
> kexec: add CRASH_ZEROIZE to wipe secrets before kdump
> mm/secretmem: zeroize secret pages before kdump
> security/keys: zeroize key payloads before kdump
>
> drivers/of/kexec.c | 2 +-
> include/linux/crash_core.h | 5 +++
> include/linux/key-type.h | 9 ++++
> kernel/Kconfig.kexec | 8 ++++
> kernel/crash_core.c | 18 ++++++++
> mm/secretmem.c | 50 +++++++++++++++++++++++
> security/keys/big_key.c | 15 +++++++
> security/keys/encrypted-keys/encrypted.c | 12 ++++++
> security/keys/key.c | 44 ++++++++++++++++++++
> security/keys/trusted-keys/trusted_core.c | 14 +++++++
> security/keys/user_defined.c | 11 +++++
> 11 files changed, 187 insertions(+), 1 deletion(-)
>
> --
> 2.53.0
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/4] CRASH_ZEROIZE: Wipe secrets before kdump
2026-08-01 14:03 ` [PATCH 0/4] CRASH_ZEROIZE: Wipe secrets before kdump Baoquan He
@ 2026-08-01 16:31 ` Jan Sebastian Götte
0 siblings, 0 replies; 3+ messages in thread
From: Jan Sebastian Götte @ 2026-08-01 16:31 UTC (permalink / raw)
To: Baoquan He
Cc: Andrew Morton, Mike Rapoport, Pasha Tatashin, Pratyush Yadav,
Dave Young, David Howells, Jarkko Sakkinen, Paul Moore,
James Morris, Serge E. Hallyn, Mimi Zohar, James Bottomley,
Rob Herring, Saravana Kannan, Coiby Xu, devicetree, linux-kernel,
kexec, keyrings, linux-mm, linux-security-module, linux-integrity
Hi there,
On 8/1/26 16:03, Baoquan He wrote:
> On 07/31/26 at 05:46pm, Jan Sebastian Götte wrote:
>> Add CONFIG_CRASH_ZEROIZE (default off), which when enabled makes various
>> subsystems handling secret data do a quick, targeted wipe of these
>> secrets before kdump. This behavior might also be interesting in cases
>> where you run a normal kdump kernel but you still want to keep things
>> like fde crypto keys out of these dumps.
>
> Please check below patchset, you both seem to have the similar
> requirement. Please go there to discuss.
>
> [RFC PATCH 0/4] panic: a pre kdump notifier list for hypervisor upcalls
Thank you for the pointer. That's indeed similar, but I think this
patchset here makes sense as an independent patchset.
* The two notifier chains run at different points, and theirs runs
inside panic independent of kdump. This one here has to be run after
machine_crash_shutdown(), and so makes most sense inside
__crash_kexec(). The wipe code in this patchset must be one of the last
things to run before the kexec since the wiped data structures could
easily lead to problems if something tried using them later.
* I think this patch series here is a bit cleaner. I don't think this
needs a custom notifier implementation.
* Since there's lots of places that may need wiping after crash, I think
it makes sense to have an explicit notifier list for that purpose, and
to not mix these with other things like hypercalls. I think ordering is
important here: The wiping should be the last thing before the kexec
jump. Having it on a generic notifier chain risks that later, other
callbacks get added that when interleaved could cause problems.
* Since a good fraction of users may not care about wiping secrets, I
think it should be gated behind an explicit enable setting.
> Note that we usually dont' want to run a lot of work after panic and
> before jumping into kdump kernel.
I understand. For this reason, I think it's best to keep this
default-off. As-is, the notifier list call is timed and on the (slow)
ARM64 target I'm using, it takes about 3-5 ms to run. I took the "try
lock, skip if locked" approach to keep the risk of this code crashing
during panic minimal. In my application, the kdump payload is code that
then does a full wipe, taking a couple hundred milliseconds.
Let me know what you think.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-08-01 16:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20260731154608.153258-1-linux@jaseg.de>
2026-08-01 14:03 ` [PATCH 0/4] CRASH_ZEROIZE: Wipe secrets before kdump Baoquan He
2026-08-01 16:31 ` Jan Sebastian Götte
2026-07-31 16:27 Jan Sebastian Götte
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox