Messages in this thread | | | Subject | Re: [PATCH v3] mm: Create the new vm_fault_t type | From | William Kucharski <> | Date | Wed, 14 Nov 2018 02:59:11 -0700 |
| |
> On Nov 13, 2018, at 10:13 PM, Souptick Joarder <jrdr.linux@gmail.com> wrote: > > On Tue, Nov 6, 2018 at 5:33 PM Souptick Joarder <jrdr.linux@gmail.com> wrote: >> >> Page fault handlers are supposed to return VM_FAULT codes, >> but some drivers/file systems mistakenly return error >> numbers. Now that all drivers/file systems have been converted >> to use the vm_fault_t return type, change the type definition >> to no longer be compatible with 'int'. By making it an unsigned >> int, the function prototype becomes incompatible with a function >> which returns int. Sparse will detect any attempts to return a >> value which is not a VM_FAULT code. >> >> VM_FAULT_SET_HINDEX and VM_FAULT_GET_HINDEX values are changed >> to avoid conflict with other VM_FAULT codes. >> >> Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com> > > Any further comment on this patch ?
Reviewed-by: William Kucharski <william.kucharski@oracle.com>
| |