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()?