linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Brendan Jackman <jackmanb@google.com>
To: Patrick Roy <patrick.roy@campus.lmu.de>
Cc: Patrick Roy <roypat@amazon.co.uk>, <pbonzini@redhat.com>,
	<corbet@lwn.net>,  <maz@kernel.org>, <oliver.upton@linux.dev>,
	<joey.gouly@arm.com>,  <suzuki.poulose@arm.com>,
	<yuzenghui@huawei.com>, <catalin.marinas@arm.com>,
	 <will@kernel.org>, <tglx@linutronix.de>, <mingo@redhat.com>,
	<bp@alien8.de>,  <dave.hansen@linux.intel.com>, <x86@kernel.org>,
	<hpa@zytor.com>,  <luto@kernel.org>, <peterz@infradead.org>,
	<willy@infradead.org>,  <akpm@linux-foundation.org>,
	<david@redhat.com>, <lorenzo.stoakes@oracle.com>,
	 <Liam.Howlett@oracle.com>, <vbabka@suse.cz>, <rppt@kernel.org>,
	 <surenb@google.com>, <mhocko@suse.com>, <song@kernel.org>,
	<jolsa@kernel.org>,  <ast@kernel.org>, <daniel@iogearbox.net>,
	<andrii@kernel.org>,  <martin.lau@linux.dev>, <eddyz87@gmail.com>,
	<yonghong.song@linux.dev>,  <john.fastabend@gmail.com>,
	<kpsingh@kernel.org>, <sdf@fomichev.me>,  <haoluo@google.com>,
	<jgg@ziepe.ca>, <jhubbard@nvidia.com>,  <peterx@redhat.com>,
	<jannh@google.com>, <pfalcato@suse.de>,  <shuah@kernel.org>,
	<seanjc@google.com>, <kvm@vger.kernel.org>,
	 <linux-doc@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	 <linux-arm-kernel@lists.infradead.org>, <kvmarm@lists.linux.dev>,
	 <linux-fsdevel@vger.kernel.org>, <linux-mm@kvack.org>,
	<bpf@vger.kernel.org>,  <linux-kselftest@vger.kernel.org>,
	<xmarcalx@amazon.co.uk>,  <kalyazin@amazon.co.uk>,
	<jackabt@amazon.co.uk>, <derekmn@amazon.co.uk>,
	 <tabba@google.com>, <ackerleytng@google.com>
Subject: Re: [PATCH v7 00/12] Direct Map Removal Support for guest_memfd
Date: Fri, 07 Nov 2025 15:54:31 +0000	[thread overview]
Message-ID: <DE2L1SAOC55E.E4JY62WJQ2A8@google.com> (raw)
In-Reply-To: <20250924151101.2225820-1-patrick.roy@campus.lmu.de>

On Wed Sep 24, 2025 at 3:10 PM UTC, Patrick Roy wrote:
> From: Patrick Roy <roypat@amazon.co.uk>
>
> [ based on kvm/next ]
>
> Unmapping virtual machine guest memory from the host kernel's direct map is a
> successful mitigation against Spectre-style transient execution issues: If the
> kernel page tables do not contain entries pointing to guest memory, then any
> attempted speculative read through the direct map will necessarily be blocked
> by the MMU before any observable microarchitectural side-effects happen. This
> means that Spectre-gadgets and similar cannot be used to target virtual machine
> memory. Roughly 60% of speculative execution issues fall into this category [1,
> Table 1].
>
> This patch series extends guest_memfd with the ability to remove its memory
> from the host kernel's direct map, to be able to attain the above protection
> for KVM guests running inside guest_memfd.
>
> Additionally, a Firecracker branch with support for these VMs can be found on
> GitHub [2].
>
> For more details, please refer to the v5 cover letter [v5]. No
> substantial changes in design have taken place since.
>
> === Changes Since v6 ===
>
> - Drop patch for passing struct address_space to ->free_folio(), due to
>   possible races with freeing of the address_space. (Hugh)
> - Stop using PG_uptodate / gmem preparedness tracking to keep track of
>   direct map state.  Instead, use the lowest bit of folio->private. (Mike, David)
> - Do direct map removal when establishing mapping of gmem folio instead
>   of at allocation time, due to impossibility of handling direct map
>   removal errors in kvm_gmem_populate(). (Patrick)
> - Do TLB flushes after direct map removal, and provide a module
>   parameter to opt out from them, and a new patch to export
>   flush_tlb_kernel_range() to KVM. (Will)
>
> [1]: https://download.vusec.net/papers/quarantine_raid23.pdf
> [2]: https://github.com/firecracker-microvm/firecracker/tree/feature/secret-hiding

I just got around to trying this out, I checked out this patchset using
its base-commit and grabbed the Firecracker branch. Things seem OK until
I set the secrets_free flag in the Firecracker config which IIUC makes
it set GUEST_MEMFD_FLAG_NO_DIRECT_MAP.

If I set it, I find the guest doesn't show anything on the console.
Running it in a VM and attaching GDB suggests that it's entering the
guest repeatedly, it doesn't seem like the vCPU thread is stuck or
anything. I'm a bit clueless about how to debug that (so far, whenever
I've broken KVM, things always exploded very dramatically).
 
Anyway, if I then kill the firecracker process, the host sometimes
crashes, I think this is the most suggestive splat I've seen:

[   99.673420][    T2] BUG: unable to handle page fault for address: ffff888012804000
[   99.676216][    T2] #PF: supervisor write access in kernel mode
[   99.678381][    T2] #PF: error_code(0x0002) - not-present page
[   99.680499][    T2] PGD 2e01067 P4D 2e01067 PUD 2e02067 PMD 12801063 PTE 800fffffed7fb020
[   99.683374][    T2] Oops: Oops: 0002 [#1] SMP
[   99.685004][    T2] CPU: 0 UID: 0 PID: 2 Comm: kthreadd Not tainted 6.17.0-rc7-00366-g473c46a3cb2a #106 NONE 
[   99.688514][    T2] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 0.1 11/11/2019
[   99.691547][    T2] RIP: 0010:clear_page_erms+0x7/0x10
[   99.693440][    T2] Code: 48 89 47 18 48 89 47 20 48 89 47 28 48 89 47 30 48 89 47 38 48 8d 7f 40 75 d9 90 c3 0f 1f 80 00 00 00 00 b9 00 10 00 00 31 c0 <f3> aa c3 66 0f 1f 44 00 00 48 83 f9 40 73 2a 83 f9 08 73 0f 85 c9
[   99.700188][    T2] RSP: 0018:ffff88800318fc10 EFLAGS: 00010246
[   99.702321][    T2] RAX: 0000000000000000 RBX: 0000000000400dc0 RCX: 0000000000001000
[   99.705100][    T2] RDX: ffffea00004a0100 RSI: ffffea00004a0200 RDI: ffff888012804000
[   99.707861][    T2] RBP: 0000000000000801 R08: 0000000000000000 R09: 0000000000000000
[   99.710648][    T2] R10: 0000000000000000 R11: 0000000000000000 R12: 0000000000000002
[   99.713412][    T2] R13: 0000000000000801 R14: ffffea00004a0100 R15: ffffffff81f4df80
[   99.716191][    T2] FS:  0000000000000000(0000) GS:ffff8880bbf28000(0000) knlGS:0000000000000000
[   99.719316][    T2] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[   99.721648][    T2] CR2: ffff888012804000 CR3: 0000000007583001 CR4: 0000000000372eb0
[   99.724421][    T2] Call Trace:
[   99.725608][    T2]  <TASK>
[   99.726646][    T2]  get_page_from_freelist+0x6fe/0x14b0
[   99.728583][    T2]  ? fs_reclaim_acquire+0x43/0xe0
[   99.730325][    T2]  ? find_held_lock+0x2b/0x80
[   99.731965][    T2]  __alloc_frozen_pages_noprof+0x147/0x2d0
[   99.734003][    T2]  __alloc_pages_noprof+0x5/0x50
[   99.735766][    T2]  copy_process+0x1b1/0x1b30
[   99.737398][    T2]  ? lock_is_held_type+0x89/0x100
[   99.739157][    T2]  ? kthreadd+0x25/0x190
[   99.740664][    T2]  kernel_clone+0x59/0x390
[   99.742213][    T2]  ? kthreadd+0x25/0x190
[   99.743728][    T2]  kernel_thread+0x55/0x70
[   99.745310][    T2]  ? kthread_complete_and_exit+0x20/0x20
[   99.747265][    T2]  kthreadd+0x117/0x190
[   99.748748][    T2]  ? kthread_is_per_cpu+0x30/0x30
[   99.750509][    T2]  ret_from_fork+0x16b/0x1e0
[   99.752193][    T2]  ? kthread_is_per_cpu+0x30/0x30
[   99.753992][    T2]  ret_from_fork_asm+0x11/0x20
[   99.755717][    T2]  </TASK>
[   99.756861][    T2] CR2: ffff888012804000
[   99.758353][    T2] ---[ end trace 0000000000000000 ]---
[   99.760319][    T2] RIP: 0010:clear_page_erms+0x7/0x10
[   99.762209][    T2] Code: 48 89 47 18 48 89 47 20 48 89 47 28 48 89 47 30 48 89 47 38 48 8d 7f 40 75 d9 90 c3 0f 1f 80 00 00 00 00 b9 00 10 00 00 31 c0 <f3> aa c3 66 0f 1f 44 00 00 48 83 f9 40 73 2a 83 f9 08 73 0f 85 c9
[   99.769129][    T2] RSP: 0018:ffff88800318fc10 EFLAGS: 00010246
[   99.771297][    T2] RAX: 0000000000000000 RBX: 0000000000400dc0 RCX: 0000000000001000
[   99.774126][    T2] RDX: ffffea00004a0100 RSI: ffffea00004a0200 RDI: ffff888012804000
[   99.777013][    T2] RBP: 0000000000000801 R08: 0000000000000000 R09: 0000000000000000
[   99.779827][    T2] R10: 0000000000000000 R11: 0000000000000000 R12: 0000000000000002
[   99.782641][    T2] R13: 0000000000000801 R14: ffffea00004a0100 R15: ffffffff81f4df80
[   99.785487][    T2] FS:  0000000000000000(0000) GS:ffff8880bbf28000(0000) knlGS:0000000000000000
[   99.788671][    T2] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[   99.791012][    T2] CR2: ffff888012804000 CR3: 0000000007583001 CR4: 0000000000372eb0
[   99.793863][    T2] Kernel panic - not syncing: Fatal exception
[   99.796760][    T2] Kernel Offset: disabled
[   99.798296][    T2] ---[ end Kernel panic - not syncing: Fatal exception ]---

This makes me suspect the kvm_gmem_folio_restore_direct_map() path isn't
working or isn't getting called.

If anyone wants help trying to reproduce this let me know.


  parent reply	other threads:[~2025-11-07 15:54 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-24 15:10 Patrick Roy
2025-09-24 15:10 ` [PATCH v7 01/12] arch: export set_direct_map_valid_noflush to KVM module Patrick Roy
2025-09-24 15:10 ` [PATCH v7 02/12] x86/tlb: export flush_tlb_kernel_range " Patrick Roy
2025-09-24 15:10 ` [PATCH v7 03/12] mm: introduce AS_NO_DIRECT_MAP Patrick Roy
2025-09-24 15:22   ` [PATCH v7 04/12] KVM: guest_memfd: Add stub for kvm_arch_gmem_invalidate Roy, Patrick
2025-09-24 15:22     ` [PATCH v7 05/12] KVM: guest_memfd: Add flag to remove from direct map Roy, Patrick
2025-09-25 11:00       ` David Hildenbrand
2025-09-25 15:52         ` Roy, Patrick
2025-09-25 19:28           ` David Hildenbrand
2025-09-26 14:49       ` Patrick Roy
2025-10-31 17:30       ` Brendan Jackman
2025-11-01  9:39         ` Mike Rapoport
2025-11-03 10:35           ` Brendan Jackman
2025-11-03 10:50             ` Mike Rapoport
2025-11-04 11:08               ` Brendan Jackman
2025-11-10 12:34                 ` Mike Rapoport
2025-11-03  7:57       ` Aneesh Kumar K.V
2025-12-05 17:23         ` Nikita Kalyazin
2025-09-24 15:22     ` [PATCH v7 06/12] KVM: guest_memfd: add module param for disabling TLB flushing Roy, Patrick
2025-09-25 11:02       ` David Hildenbrand
2025-09-25 15:50         ` Roy, Patrick
2025-09-25 19:32           ` David Hildenbrand
2025-09-25 18:27       ` Dave Hansen
2025-09-25 19:20         ` David Hildenbrand
2025-09-25 19:59           ` Dave Hansen
2025-09-25 20:13             ` David Hildenbrand
2025-09-26  9:46               ` Patrick Roy
2025-09-26 10:53                 ` Will Deacon
2025-09-26 20:09                   ` David Hildenbrand
2025-09-27  7:38                     ` Patrick Roy
2025-09-29 10:20                       ` David Hildenbrand
2025-10-11 14:32                         ` Patrick Roy
2025-11-07 15:29                           ` Ackerley Tng
2025-11-07 17:22                             ` Nikita Kalyazin
2025-11-07 17:21                           ` Nikita Kalyazin
2025-10-30 16:05         ` Brendan Jackman
2025-10-31 18:31           ` Brendan Jackman
2025-09-24 15:22     ` [PATCH v7 07/12] KVM: selftests: load elf via bounce buffer Roy, Patrick
2025-09-24 15:22     ` [PATCH v7 08/12] KVM: selftests: set KVM_MEM_GUEST_MEMFD in vm_mem_add() if guest_memfd != -1 Roy, Patrick
2025-09-24 15:22     ` [PATCH v7 09/12] KVM: selftests: Add guest_memfd based vm_mem_backing_src_types Roy, Patrick
2025-09-24 15:22     ` [PATCH v7 10/12] KVM: selftests: cover GUEST_MEMFD_FLAG_NO_DIRECT_MAP in existing selftests Roy, Patrick
2025-09-24 15:22     ` [PATCH v7 11/12] KVM: selftests: stuff vm_mem_backing_src_type into vm_shape Roy, Patrick
2025-09-24 15:22     ` [PATCH v7 12/12] KVM: selftests: Test guest execution from direct map removed gmem Roy, Patrick
2025-10-30 17:18       ` Brendan Jackman
2025-12-05 17:24         ` Nikita Kalyazin
2025-09-25 10:26     ` [PATCH v7 04/12] KVM: guest_memfd: Add stub for kvm_arch_gmem_invalidate David Hildenbrand
2025-09-25 10:25   ` [PATCH v7 03/12] mm: introduce AS_NO_DIRECT_MAP David Hildenbrand
2025-09-24 15:29 ` [PATCH v7 00/12] Direct Map Removal Support for guest_memfd Roy, Patrick
2025-09-24 15:38   ` David Hildenbrand
2025-11-07 15:54 ` Brendan Jackman [this message]
2025-11-07 17:23   ` Nikita Kalyazin
2025-11-07 18:04     ` Brendan Jackman
2025-11-07 18:11       ` Nikita Kalyazin
2025-11-10 15:36         ` Brendan Jackman
2025-11-07 17:37   ` Brendan Jackman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DE2L1SAOC55E.E4JY62WJQ2A8@google.com \
    --to=jackmanb@google.com \
    --cc=Liam.Howlett@oracle.com \
    --cc=ackerleytng@google.com \
    --cc=akpm@linux-foundation.org \
    --cc=andrii@kernel.org \
    --cc=ast@kernel.org \
    --cc=bp@alien8.de \
    --cc=bpf@vger.kernel.org \
    --cc=catalin.marinas@arm.com \
    --cc=corbet@lwn.net \
    --cc=daniel@iogearbox.net \
    --cc=dave.hansen@linux.intel.com \
    --cc=david@redhat.com \
    --cc=derekmn@amazon.co.uk \
    --cc=eddyz87@gmail.com \
    --cc=haoluo@google.com \
    --cc=hpa@zytor.com \
    --cc=jackabt@amazon.co.uk \
    --cc=jannh@google.com \
    --cc=jgg@ziepe.ca \
    --cc=jhubbard@nvidia.com \
    --cc=joey.gouly@arm.com \
    --cc=john.fastabend@gmail.com \
    --cc=jolsa@kernel.org \
    --cc=kalyazin@amazon.co.uk \
    --cc=kpsingh@kernel.org \
    --cc=kvm@vger.kernel.org \
    --cc=kvmarm@lists.linux.dev \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=lorenzo.stoakes@oracle.com \
    --cc=luto@kernel.org \
    --cc=martin.lau@linux.dev \
    --cc=maz@kernel.org \
    --cc=mhocko@suse.com \
    --cc=mingo@redhat.com \
    --cc=oliver.upton@linux.dev \
    --cc=patrick.roy@campus.lmu.de \
    --cc=pbonzini@redhat.com \
    --cc=peterx@redhat.com \
    --cc=peterz@infradead.org \
    --cc=pfalcato@suse.de \
    --cc=roypat@amazon.co.uk \
    --cc=rppt@kernel.org \
    --cc=sdf@fomichev.me \
    --cc=seanjc@google.com \
    --cc=shuah@kernel.org \
    --cc=song@kernel.org \
    --cc=surenb@google.com \
    --cc=suzuki.poulose@arm.com \
    --cc=tabba@google.com \
    --cc=tglx@linutronix.de \
    --cc=vbabka@suse.cz \
    --cc=will@kernel.org \
    --cc=willy@infradead.org \
    --cc=x86@kernel.org \
    --cc=xmarcalx@amazon.co.uk \
    --cc=yonghong.song@linux.dev \
    --cc=yuzenghui@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox