On Wed, Sep 10, 2025 at 10:55:24PM +0000, Ashish Kalra wrote:
> From: Ashish Kalra <ashish.kalra@xxxxxxx>
>
> When leaking certain page types, such as Hypervisor Fixed (HV_FIXED)
> pages, it does not make sense to dump RMP contents for the 2MB range of
> the page(s) being leaked. In the case of HV_FIXED pages, this is not an
> error situation where the surrounding 2MB page RMP entries can provide
> debug information.
>
> Add new __snp_leak_pages() API with dump_rmp bool parameter to support
> continue adding pages to the snp_leaked_pages_list but not issue
> dump_rmpentry().
>
> Make snp_leak_pages() a wrapper for the common case which also allows
> existing users to continue to dump RMP entries.
>
> Suggested-by: Thomas Lendacky <Thomas.Lendacky@xxxxxxx>
> Suggested-by: Sean Christopherson <seanjc@xxxxxxxxxx>
> Signed-off-by: Ashish Kalra <ashish.kalra@xxxxxxx>
> ---
> arch/x86/include/asm/sev.h | 8 +++++++-
> arch/x86/virt/svm/sev.c | 7 ++++---
> 2 files changed, 11 insertions(+), 4 deletions(-)
Sean, lemme know if I should carry this through tip.
Or, if you wanna take it:
Acked-by: Borislav Petkov (AMD) <bp@xxxxxxxxx>
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette
