[linux-next:master 13875/13946] mm/mempolicy.c:1408:30: error: expected ')' before ';' token

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   d47175169c28eedd2cc2ab8c01f38764cb0269cc
commit: 1842a5a9ba44c8bd174fa124227a31d4b8eafea6 [13875/13946] mm: untag user pointers passed to memory syscalls
config: sparc64-allmodconfig (attached as .config)
compiler: sparc64-linux-gcc (GCC) 7.4.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 1842a5a9ba44c8bd174fa124227a31d4b8eafea6
        # save the attached .config to linux build tree
        GCC_VERSION=7.4.0 make.cross ARCH=sparc64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@xxxxxxxxx>

All error/warnings (new ones prefixed by >>):

   mm/mempolicy.c: In function 'kernel_mbind':
>> mm/mempolicy.c:1408:30: error: expected ')' before ';' token
     start = untagged_addr(start);
                                 ^
>> mm/mempolicy.c:1420:1: error: expected ';' before '}' token
    }
    ^
   mm/mempolicy.c:1406:17: warning: unused variable 'mode_flags' [-Wunused-variable]
     unsigned short mode_flags;
                    ^~~~~~~~~~
   mm/mempolicy.c:1405:6: warning: unused variable 'err' [-Wunused-variable]
     int err;
         ^~~
   mm/mempolicy.c:1404:13: warning: unused variable 'nodes' [-Wunused-variable]
     nodemask_t nodes;
                ^~~~~
>> mm/mempolicy.c:1420:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^
   mm/mempolicy.c: In function 'kernel_get_mempolicy':
   mm/mempolicy.c:1562:28: error: expected ')' before ';' token
     addr = untagged_addr(addr);
                               ^
   mm/mempolicy.c:1579:1: error: expected ';' before '}' token
    }
    ^
   mm/mempolicy.c:1560:13: warning: unused variable 'nodes' [-Wunused-variable]
     nodemask_t nodes;
                ^~~~~
   mm/mempolicy.c:1558:6: warning: unused variable 'err' [-Wunused-variable]
     int err;
         ^~~
   mm/mempolicy.c:1579:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^
   At top level:
   mm/mempolicy.c:1384:12: warning: 'copy_nodes_to_user' defined but not used [-Wunused-function]
    static int copy_nodes_to_user(unsigned long __user *mask, unsigned long maxnode,
               ^~~~~~~~~~~~~~~~~~
   mm/mempolicy.c:1215:13: warning: 'do_mbind' defined but not used [-Wunused-function]
    static long do_mbind(unsigned long start, unsigned long len,
                ^~~~~~~~
   mm/mempolicy.c:869:13: warning: 'do_get_mempolicy' defined but not used [-Wunused-function]
    static long do_get_mempolicy(int *policy, nodemask_t *nmask,
                ^~~~~~~~~~~~~~~~
--
   mm/migrate.c: In function 'do_pages_move':
>> mm/migrate.c:1615:41: error: expected ')' before ';' token
      addr = (unsigned long)untagged_addr(p);
                                            ^
>> mm/migrate.c:1663:2: error: expected ';' before '}' token
     }
     ^
   mm/migrate.c:1674:1: warning: label 'out' defined but not used [-Wunused-label]
    out:
    ^~~
   At top level:
   mm/migrate.c:1523:12: warning: 'add_page_for_migration' defined but not used [-Wunused-function]
    static int add_page_for_migration(struct mm_struct *mm, unsigned long addr,
               ^~~~~~~~~~~~~~~~~~~~~~
--
   mm/mincore.c: In function '__do_sys_mincore':
>> mm/mincore.c:259:30: error: expected ')' before ';' token
     start = untagged_addr(start);
                                 ^
>> mm/mincore.c:303:1: error: expected ';' before '}' token
    }
    ^
   mm/mincore.c:257:17: warning: unused variable 'tmp' [-Wunused-variable]
     unsigned char *tmp;
                    ^~~
   mm/mincore.c:256:16: warning: unused variable 'pages' [-Wunused-variable]
     unsigned long pages;
                   ^~~~~
   mm/mincore.c:255:7: warning: unused variable 'retval' [-Wunused-variable]
     long retval;
          ^~~~~~
>> mm/mincore.c:303:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^
   At top level:
   mm/mincore.c:207:13: warning: 'do_mincore' defined but not used [-Wunused-function]
    static long do_mincore(unsigned long addr, unsigned long pages, unsigned char *vec)
                ^~~~~~~~~~
--
   mm/mlock.c: In function 'do_mlock':
>> mm/mlock.c:677:30: error: expected ')' before ';' token
     start = untagged_addr(start);
                                 ^
>> mm/mlock.c:716:1: error: expected ';' before '}' token
    }
    ^
   mm/mlock.c:675:6: warning: unused variable 'error' [-Wunused-variable]
     int error = -ENOMEM;
         ^~~~~
   mm/mlock.c:674:16: warning: unused variable 'lock_limit' [-Wunused-variable]
     unsigned long lock_limit;
                   ^~~~~~~~~~
   mm/mlock.c:673:16: warning: unused variable 'locked' [-Wunused-variable]
     unsigned long locked;
                   ^~~~~~
>> mm/mlock.c:716:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^
   mm/mlock.c: In function '__do_sys_munlock':
   mm/mlock.c:740:30: error: expected ')' before ';' token
     start = untagged_addr(start);
                                 ^
   mm/mlock.c:751:1: error: expected ';' before '}' token
    }
    ^
   mm/mlock.c:738:6: warning: unused variable 'ret' [-Wunused-variable]
     int ret;
         ^~~
   mm/mlock.c:751:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^
   At top level:
   mm/mlock.c:639:22: warning: 'count_mm_mlocked_page_nr' defined but not used [-Wunused-function]
    static unsigned long count_mm_mlocked_page_nr(struct mm_struct *mm,
                         ^~~~~~~~~~~~~~~~~~~~~~~~
   mm/mlock.c:583:12: warning: 'apply_vma_lock_flags' defined but not used [-Wunused-function]
    static int apply_vma_lock_flags(unsigned long start, size_t len,
               ^~~~~~~~~~~~~~~~~~~~
   mm/mlock.c:225:12: warning: '__mlock_posix_error_return' defined but not used [-Wunused-function]
    static int __mlock_posix_error_return(long retval)
               ^~~~~~~~~~~~~~~~~~~~~~~~~~
--
   mm/mprotect.c: In function 'do_mprotect_pkey':
>> mm/mprotect.c:462:30: error: expected ')' before ';' token
     start = untagged_addr(start);
                                 ^
>> mm/mprotect.c:573:1: error: expected ';' before '}' token
    }
    ^
   mm/mprotect.c:459:13: warning: unused variable 'rier' [-Wunused-variable]
     const bool rier = (current->personality & READ_IMPLIES_EXEC) &&
                ^~~~
   mm/mprotect.c:458:12: warning: unused variable 'grows' [-Wunused-variable]
     const int grows = prot & (PROT_GROWSDOWN|PROT_GROWSUP);
               ^~~~~
   mm/mprotect.c:457:6: warning: unused variable 'error' [-Wunused-variable]
     int error = -EINVAL;
         ^~~~~
   mm/mprotect.c:456:31: warning: unused variable 'prev' [-Wunused-variable]
     struct vm_area_struct *vma, *prev;
                                  ^~~~
   mm/mprotect.c:456:25: warning: unused variable 'vma' [-Wunused-variable]
     struct vm_area_struct *vma, *prev;
                            ^~~
   mm/mprotect.c:455:34: warning: unused variable 'reqprot' [-Wunused-variable]
     unsigned long nstart, end, tmp, reqprot;
                                     ^~~~~~~
   mm/mprotect.c:455:29: warning: unused variable 'tmp' [-Wunused-variable]
     unsigned long nstart, end, tmp, reqprot;
                                ^~~
   mm/mprotect.c:455:24: warning: unused variable 'end' [-Wunused-variable]
     unsigned long nstart, end, tmp, reqprot;
                           ^~~
   mm/mprotect.c:455:16: warning: unused variable 'nstart' [-Wunused-variable]
     unsigned long nstart, end, tmp, reqprot;
                   ^~~~~~
>> mm/mprotect.c:573:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^
--
   mm/mremap.c: In function '__do_sys_mremap':
>> mm/mremap.c:614:28: error: expected ')' before ';' token
     addr = untagged_addr(addr);
                               ^
   mm/mremap.c:736:1: error: expected ';' before '}' token
    }
    ^
   mm/mremap.c:605:28: warning: unused variable 'uf' [-Wunused-variable]
     struct vm_userfaultfd_ctx uf = NULL_VM_UFFD_CTX;
                               ^~
   mm/mremap.c:604:7: warning: unused variable 'downgraded' [-Wunused-variable]
     bool downgraded = false;
          ^~~~~~~~~~
   mm/mremap.c:603:7: warning: unused variable 'locked' [-Wunused-variable]
     bool locked = false;
          ^~~~~~
   mm/mremap.c:602:16: warning: unused variable 'charged' [-Wunused-variable]
     unsigned long charged = 0;
                   ^~~~~~~
   mm/mremap.c:601:16: warning: unused variable 'ret' [-Wunused-variable]
     unsigned long ret = -EINVAL;
                   ^~~
   mm/mremap.c:600:25: warning: unused variable 'vma' [-Wunused-variable]
     struct vm_area_struct *vma;
                            ^~~
   mm/mremap.c:599:20: warning: unused variable 'mm' [-Wunused-variable]
     struct mm_struct *mm = current->mm;
                       ^~
>> mm/mremap.c:736:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^
   At top level:
   mm/mremap.c:575:12: warning: 'vma_expandable' defined but not used [-Wunused-function]
    static int vma_expandable(struct vm_area_struct *vma, unsigned long delta)
               ^~~~~~~~~~~~~~
   mm/mremap.c:498:22: warning: 'mremap_to' defined but not used [-Wunused-function]
    static unsigned long mremap_to(unsigned long addr, unsigned long old_len,
                         ^~~~~~~~~
--
   mm/msync.c: In function '__do_sys_msync':
>> mm/msync.c:40:30: error: expected ')' before ';' token
     start = untagged_addr(start);
                                 ^
>> mm/msync.c:110:1: error: expected ';' before '}' token
    }
    ^
   mm/msync.c:38:6: warning: unused variable 'error' [-Wunused-variable]
     int error = -EINVAL;
         ^~~~~
   mm/msync.c:37:6: warning: unused variable 'unmapped_error' [-Wunused-variable]
     int unmapped_error = 0;
         ^~~~~~~~~~~~~~
   mm/msync.c:36:25: warning: unused variable 'vma' [-Wunused-variable]
     struct vm_area_struct *vma;
                            ^~~
   mm/msync.c:35:20: warning: unused variable 'mm' [-Wunused-variable]
     struct mm_struct *mm = current->mm;
                       ^~
   mm/msync.c:34:16: warning: unused variable 'end' [-Wunused-variable]
     unsigned long end;
                   ^~~
>> mm/msync.c:110:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^
--
   mm/madvise.c: In function '__do_sys_madvise':
>> mm/madvise.c:787:30: error: expected ')' before ';' token
     start = untagged_addr(start);
                                 ^
>> mm/madvise.c:873:1: error: expected ';' before '}' token
    }
    ^
   mm/madvise.c:785:18: warning: unused variable 'plug' [-Wunused-variable]
     struct blk_plug plug;
                     ^~~~
   mm/madvise.c:784:9: warning: unused variable 'len' [-Wunused-variable]
     size_t len;
            ^~~
   mm/madvise.c:783:6: warning: unused variable 'write' [-Wunused-variable]
     int write;
         ^~~~~
   mm/madvise.c:782:6: warning: unused variable 'error' [-Wunused-variable]
     int error = -EINVAL;
         ^~~~~
   mm/madvise.c:781:6: warning: unused variable 'unmapped_error' [-Wunused-variable]
     int unmapped_error = 0;
         ^~~~~~~~~~~~~~
   mm/madvise.c:780:31: warning: unused variable 'prev' [-Wunused-variable]
     struct vm_area_struct *vma, *prev;
                                  ^~~~
   mm/madvise.c:780:25: warning: unused variable 'vma' [-Wunused-variable]
     struct vm_area_struct *vma, *prev;
                            ^~~
   mm/madvise.c:779:21: warning: unused variable 'tmp' [-Wunused-variable]
     unsigned long end, tmp;
                        ^~~
   mm/madvise.c:779:16: warning: unused variable 'end' [-Wunused-variable]
     unsigned long end, tmp;
                   ^~~
>> mm/madvise.c:873:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^
   At top level:
   mm/madvise.c:681:1: warning: 'madvise_behavior_valid' defined but not used [-Wunused-function]
    madvise_behavior_valid(int behavior)
    ^~~~~~~~~~~~~~~~~~~~~~
   mm/madvise.c:664:1: warning: 'madvise_vma' defined but not used [-Wunused-function]
    madvise_vma(struct vm_area_struct *vma, struct vm_area_struct **prev,
    ^~~~~~~~~~~
   mm/madvise.c:39:12: warning: 'madvise_need_mmap_write' defined but not used [-Wunused-function]
    static int madvise_need_mmap_write(int behavior)
               ^~~~~~~~~~~~~~~~~~~~~~~

vim +1408 mm/mempolicy.c

  1399	
  1400	static long kernel_mbind(unsigned long start, unsigned long len,
  1401				 unsigned long mode, const unsigned long __user *nmask,
  1402				 unsigned long maxnode, unsigned int flags)
  1403	{
  1404		nodemask_t nodes;
  1405		int err;
> 1406		unsigned short mode_flags;
  1407	
> 1408		start = untagged_addr(start);
  1409		mode_flags = mode & MPOL_MODE_FLAGS;
  1410		mode &= ~MPOL_MODE_FLAGS;
  1411		if (mode >= MPOL_MAX)
  1412			return -EINVAL;
  1413		if ((mode_flags & MPOL_F_STATIC_NODES) &&
  1414		    (mode_flags & MPOL_F_RELATIVE_NODES))
  1415			return -EINVAL;
  1416		err = get_nodes(&nodes, nmask, maxnode);
  1417		if (err)
  1418			return err;
  1419		return do_mbind(start, len, mode, mode_flags, &nodes, flags);
> 1420	}
  1421	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux