Re: [PATCH RFC v9 14/51] x86/sev: Add helper functions for RMPUPDATE and PSMASH instruction

Dave Hansen
2023-06-12T17:00:12Z

On 6/11/23 21:25, Michael Roth wrote:
> +/*
> + * Assign a page to guest using the RMPUPDATE instruction.
> + */
> +int rmp_make_private(u64 pfn, u64 gpa, enum pg_level level, int asid, bool immutable)
> +{
> + struct rmp_state val;
> +
> + pr_debug("%s: GPA: 0x%llx, PFN: 0x%llx, level: %d, immutable: %d\n",
> + __func__, gpa, pfn, level, immutable);

Is this needed *EVERY* time a page is assigned to a guest? As in, if I
create a 4GB guest, I'll see a literal million of these pr_debug()s in
dmesg?