On Thu, Apr 06, 2023 at 11:55:56AM -0600, Jane Chu wrote:
>  static vm_fault_t dax_fault_return(int error)
>  {
>       if (error == 0)
>               return VM_FAULT_NOPAGE;
> -     return vmf_error(error);
> +     else if (error == -ENOMEM)
> +             return VM_FAULT_OOM;
> +     else if (error == -EHWPOISON)
> +             return VM_FAULT_HWPOISON;
> +     return VM_FAULT_SIGBUS;
>  }

Why would we want to handle it here instead of changing vmf_error()?

Reply via email to