On Thu, Aug 22, 2024 at 04:10:59PM +0100, Joey Gouly wrote:

> +static bool fault_from_pkey(unsigned long esr, struct vm_area_struct *vma,
> +                     unsigned int mm_flags)
> +{
> +     unsigned long iss2 = ESR_ELx_ISS2(esr);
> +
> +     if (!system_supports_poe())
> +             return false;
> +
> +     if (iss2 & ESR_ELx_Overlay)
> +             return true;

Does this need an is_data_abort() && is_instruction_abort() check?
Overlay doesn't appear to be defined for all exception types and it
wasn't clear enough to me that the callers have done this check.

Attachment: signature.asc
Description: PGP signature

Reply via email to