x86/mm/fault.c: Convert to use vm_fault_t

Souptick Joarder · sfrothwell · commit 8f9a99aab084 · 2019-02-09T15:47:19.000+11:00
Convert to use vm_fault_t type for last argument of do_sigbus(). kbuild reported a warning during testing of final vm_fault_t patch integrated in mm tree. [auto build test WARNING on linus/master] [also build test WARNING on v5.0-rc1 next-20190109] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] https://github.com/0day-ci/linux/commits/Souptick-Joarder/ mm-Create-the-new-vm_fault_t-type/20190109-154216 All warnings: arch/x86/mm/fault.c:1051:39: warning: restricted vm_fault_t degrades to integer arch/x86/mm/fault.c:1057:29: warning: restricted vm_fault_t degrades to integer arch/x86/mm/fault.c:1059:29: warning: restricted vm_fault_t degrades to integer arch/x86/mm/fault.c:1094:62: warning: incorrect type in argument 4 (different base types) arch/x86/mm/fault.c:1094:62: expected unsigned int fault arch/x86/mm/fault.c:1094:62: got restricted vm_fault_t [usertype] fault This patch has fixed the warnings. Link: http://lkml.kernel.org/r/20190109183742.GA24326@jordon-HP-15-Notebook-PC Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c
@@ -1031,7 +1031,7 @@ bad_area_access_error(struct pt_regs *regs, unsigned long error_code,
 
 static void
 do_sigbus(struct pt_regs *regs, unsigned long error_code, unsigned long address,
-	  unsigned int fault)
+	  vm_fault_t fault)
 {
 	struct task_struct *tsk = current;
 

Original file line number	Diff line number	Diff line change
`@@ -1031,7 +1031,7 @@ bad_area_access_error(struct pt_regs *regs, unsigned long error_code,`
`1031`	`1031`
`1032`	`1032`	`static void`
`1033`	`1033`	`do_sigbus(struct pt_regs *regs, unsigned long error_code, unsigned long address,`
`1034`		`- unsigned int fault)`
	`1034`	`+ vm_fault_t fault)`
`1035`	`1035`	`{`
`1036`	`1036`	`struct task_struct *tsk = current;`
`1037`	`1037`