Re: [PATCH] KVM: selftests: Drop stale exact flags check in private_mem_kvm_exits_test

[David Woodhouse]

On 12 November 2026 10:22:38 GMT-07:00, Sean Christopherson <seanjc@xxxxxxxxxx> wrote:
>On Thu, Nov 12, 2026, David Woodhouse wrote:
>> From: David Woodhouse <dwmw@xxxxxxxxxxxx>
>> 
>> Commit 7d0afe669374 ("KVM: selftests: Take into account mixed memory
>> fault flags") relaxed the memory_fault.flags checks so that they only
>> test for KVM_MEMORY_EXIT_FLAG_PRIVATE, since KVM may report the access
>> type alongside it. In test_private_access_memslot_deleted() it added
>> the relaxed check but left the exact match above it, and with the
>> READ flag now reported the test fails for me there:
>
>Ah, this is against kvm/kvm-memprot-v4.
>
>Paolo, what are your plans with that branch?  Sashiko had valid complaints, e.g.
>against patch 1[*], so I assume the branch won't be merged as-is.
>
>[*] https://lore.kernel.org/all/sashiko-outbox-193903@xxxxxxxxxx

Yeah, I have a whole ASCII art diagram for VSM of the various branches it depends on, all of which need to come together.

Not sure how it'll survive being pasted on my phone...

```
  *   e7bf7087c71e (vsm-v2, vsm2/rollup) Merge branch 'vsm2/translate-probe-fix' into vsm2/rollup
  |\
  | * 2041e48469c9 (vsm2/translate-probe-fix) KVM: selftests: Add a test for KVM_TRANSLATE write permission
  | * 11e4bc669f5a KVM: x86: Report the guest's write permission from KVM_TRANSLATE
  | * (base: v7.4-rc2)                                    ← postable №2, [RFC]
  |
  * 711d67d9ee59 (vsm2/vsm-extras) ⋯ 10 EXTRA commits ⋯
  |      TLB-flush-inhibit group [Nikolas×5 + 1 ours], suspended-state,
  |      hugepage selftest [Mushahid], stats, KVM_TRANSLATE allowlist
  |
  * 45e6e98c313e (vsm2/vsm-min, tag: vsm2-min1) ⋯ 32 MIN commits ⋯
  |      per-plane attrs → TSC page → TLB flush → VTL call/return →
  |      forced entry → SynIC/overlays → intercepts → SVM state → selftests
  |      [29 dwmw, 2 Mushahid incl. folds, 1 Sriram]
  |
  *   2dba8e6aa231 Merge branch 'vsm2/hcall-exit' into vsm2/vsm-min
  |\
  | * ab78e5f86ef8 (vsm2/hcall-exit) KVM: selftests: Add Hyper-V hypercall forwarding test
  | * b5db9e2e8a73 KVM: x86: hyper-v: Advertise XMM fast hypercall output
  | * 13c74a93fa3a KVM: x86: hyper-v: Let userspace fault a forwarded hypercall
  | * 365758d0df84 KVM: x86: hyper-v: Add KVM_CAP_HYPERV_FORWARD_HCALL
  | * (base: v7.4-rc2)                                    ← postable №1
  |
  *   d834e6e32d45 Merge branch 'vsm2/memattrs' into vsm2/vsm-min
  |\
  | *   2fd5c2a710a2 (vsm2/memattrs) Merge 'kvm-memprot-v4' of kvm.git
  | |\
  | | * ⋯ 45 commits, Paolo, verbatim ⋯
  | |/
  | * (base: v7.4-rc2)
  |
  * 08140f5dfd94 (vsm2/planes-uapi) ⋯ 9 commits ⋯
  |      KVM_CAP_PLANES opt-in, run-state ioctls, request fan-out,
  |      APICv inhibit, XCR0 sharing, selftests [8 dwmw, 1 Paolo-reworked]
  |
  * 52a6b958f643 (vsm2/planes-fixes) ⋯ 16 commits ⋯
  |      s390 lkp fix first; Hyper-V init, allowlist, current_vcpu/IPIv,
  |      TSC matching, CPUID [13 dwmw, 3 Mushahid]
  |
  * ea638a7ede9e (vsm2/planes) ⋯ 60 commits, Jörg, verbatim ⋯
  |
  * (v7.4-rc2)

  parked, not merged anywhere:
  * de563700e7ed (vsm2/translate2) ours ×1 (squashed probe) on ⋯ Nikolas ×16 ⋯ on vsm2/memattrs


```