Re: [PATCH Part2 v5 08/45] x86/fault: Add support to handle the RMP fault for user address

From: Dave Hansen
Date: Mon Aug 23 2021 - 10:20:50 EST


On 8/20/21 8:58 AM, Brijesh Singh wrote:
> +static int handle_split_page_fault(struct vm_fault *vmf)
> +{
> + if (!IS_ENABLED(CONFIG_AMD_MEM_ENCRYPT))
> + return VM_FAULT_SIGBUS;
> +
> + __split_huge_pmd(vmf->vma, vmf->pmd, vmf->address, false, NULL);
> + return 0;
> +}

We had a whole conversation the last time this was posted about huge
page types *other* than THP. I don't see any comprehension of those
types or what would happen if one of those was used with SEV-SNP.

What was the result of those review comments?

I'm still worried that hugetlbfs (and others) are not properly handled
by this series.