mm-remove-legacy-install_special_mapping-code-checkpatch-fixes

WARNING: nested (un)?likely() calls, IS_ERR already uses unlikely() internally
torvalds#83: FILE: arch/csky/kernel/vdso.c:81:
+	if (unlikely(IS_ERR(vma))) {

WARNING: nested (un)?likely() calls, IS_ERR already uses unlikely() internally
torvalds#97: FILE: arch/csky/kernel/vdso.c:92:
+	if (unlikely(IS_ERR(vma))) {

WARNING: nested (un)?likely() calls, IS_ERR already uses unlikely() internally
torvalds#213: FILE: arch/sh/kernel/vsyscall/vsyscall.c:84:
+	if (unlikely(IS_ERR(vma)))

total: 0 errors, 3 warnings, 249 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

./patches/mm-remove-legacy-install_special_mapping-code.patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.

Please run checkpatch prior to sending patches

Cc: Anton Ivanov <[email protected]>
Cc: Brian Cain <[email protected]>
Cc: Christophe Leroy <[email protected]>
Cc: Dinh Nguyen <[email protected]>
Cc: Guo Ren <[email protected]>
Cc: Jeff Xu <[email protected]>
Cc: Johannes Berg <[email protected]>
Cc: John Paul Adrian Glaubitz <[email protected]>
Cc: Liam R. Howlett <[email protected]>
Cc: Linus Torvalds <[email protected]>
Cc: Michael Ellerman <[email protected]>
Cc: Nathan Chancellor <[email protected]>
Cc: Nicholas Piggin <[email protected]>
Cc: Pedro Falcato <[email protected]>
Cc: Richard Weinberger <[email protected]>
Cc: Rich Felker <[email protected]>
Cc: Rob Landley <[email protected]>
Cc: Yoshinori Sato <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>

arch/csky/kernel/vdso.c

@@ -78,7 +78,7 @@ int arch_setup_additional_pages(struct linux_binprm *bprm,
  (VM_READ | VM_EXEC | VM_MAYREAD | VM_MAYWRITE | VM_MAYEXEC),
  &vdso_mapping);
 
- if (unlikely(IS_ERR(vma))) {
+ if (IS_ERR(vma)) {
  ret = PTR_ERR(vma);
  mm->context.vdso = NULL;
  goto end;
@@ -89,7 +89,7 @@ int arch_setup_additional_pages(struct linux_binprm *bprm,
  vma = _install_special_mapping(mm, vdso_base, PAGE_SIZE,
  (VM_READ | VM_MAYREAD), &vvar_mapping);
 
- if (unlikely(IS_ERR(vma))) {
+ if (IS_ERR(vma)) {
  ret = PTR_ERR(vma);
  mm->context.vdso = NULL;
  goto end;

arch/sh/kernel/vsyscall/vsyscall.c

@@ -81,7 +81,7 @@ int arch_setup_additional_pages(struct linux_binprm *bprm, int uses_interp)
  VM_MAYREAD | VM_MAYWRITE | VM_MAYEXEC,
  &vdso_mapping);
  ret = PTR_ERR(vma);
- if (unlikely(IS_ERR(vma)))
+ if (IS_ERR(vma))
  goto up_fail;
 
  current->mm->context.vdso = (void *)addr;