linux-kernel - [tip:WIP.sched/core 19/22] arch/m68k/include/asm/switch_to.h:30:12: error: expected '=', ',', ';', 'asm' or '__attribute_

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <201803041903.WlVM69or%fengguang.wu@intel.com>
Date:   Sun, 4 Mar 2018 19:16:05 +0800
From:   kbuild test robot <fengguang.wu@...el.com>
To:     Ingo Molnar <mingo@...nel.org>
Cc:     kbuild-all@...org, linux-kernel@...r.kernel.org, tipbuild@...or.com
Subject: [tip:WIP.sched/core 19/22] arch/m68k/include/asm/switch_to.h:30:12:
 error: expected '=', ',', ';', 'asm' or '__attribute__' before 'void'

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.sched/core
head:   ffce6fffa0033fa54d1bac1c7494862b71316978
commit: ecaf52c9ec5ad31395e5c06c01e293ebcf0e4350 [19/22] sched/headers: Simplify and clean up header usage in the scheduler
config: m68k-sun3_defconfig (attached as .config)
compiler: m68k-linux-gnu-gcc (Debian 7.2.0-11) 7.2.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 ecaf52c9ec5ad31395e5c06c01e293ebcf0e4350
        # save the attached .config to linux build tree
        make.cross ARCH=m68k 

All errors (new ones prefixed by >>):

   In file included from kernel/sched/core.c:8:0:
>> arch/m68k/include/asm/switch_to.h:30:12: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'void'
    asmlinkage void resume(void);
               ^~~~

vim +30 arch/m68k/include/asm/switch_to.h

803f69144 David Howells 2012-03-28   4  
803f69144 David Howells 2012-03-28   5  /*
803f69144 David Howells 2012-03-28   6   * switch_to(n) should switch tasks to task ptr, first checking that
803f69144 David Howells 2012-03-28   7   * ptr isn't the current task, in which case it does nothing.  This
803f69144 David Howells 2012-03-28   8   * also clears the TS-flag if the task we switched to has used the
803f69144 David Howells 2012-03-28   9   * math co-processor latest.
803f69144 David Howells 2012-03-28  10   */
803f69144 David Howells 2012-03-28  11  /*
803f69144 David Howells 2012-03-28  12   * switch_to() saves the extra registers, that are not saved
803f69144 David Howells 2012-03-28  13   * automatically by SAVE_SWITCH_STACK in resume(), ie. d0-d5 and
803f69144 David Howells 2012-03-28  14   * a0-a1. Some of these are used by schedule() and its predecessors
803f69144 David Howells 2012-03-28  15   * and so we might get see unexpected behaviors when a task returns
803f69144 David Howells 2012-03-28  16   * with unexpected register values.
803f69144 David Howells 2012-03-28  17   *
803f69144 David Howells 2012-03-28  18   * syscall stores these registers itself and none of them are used
803f69144 David Howells 2012-03-28  19   * by syscall after the function in the syscall has been called.
803f69144 David Howells 2012-03-28  20   *
803f69144 David Howells 2012-03-28  21   * Beware that resume now expects *next to be in d1 and the offset of
803f69144 David Howells 2012-03-28  22   * tss to be in a1. This saves a few instructions as we no longer have
803f69144 David Howells 2012-03-28  23   * to push them onto the stack and read them back right after.
803f69144 David Howells 2012-03-28  24   *
803f69144 David Howells 2012-03-28  25   * 02/17/96 - Jes Sorensen (jds@....auc.dk)
803f69144 David Howells 2012-03-28  26   *
803f69144 David Howells 2012-03-28  27   * Changed 96/09/19 by Andreas Schwab
803f69144 David Howells 2012-03-28  28   * pass prev in a0, next in a1
803f69144 David Howells 2012-03-28  29   */
803f69144 David Howells 2012-03-28 @30  asmlinkage void resume(void);
803f69144 David Howells 2012-03-28  31  #define switch_to(prev,next,last) do { \
803f69144 David Howells 2012-03-28  32    register void *_prev __asm__ ("a0") = (prev); \
803f69144 David Howells 2012-03-28  33    register void *_next __asm__ ("a1") = (next); \
803f69144 David Howells 2012-03-28  34    register void *_last __asm__ ("d1"); \
803f69144 David Howells 2012-03-28  35    __asm__ __volatile__("jbsr resume" \
803f69144 David Howells 2012-03-28  36  		       : "=a" (_prev), "=a" (_next), "=d" (_last) \
803f69144 David Howells 2012-03-28  37  		       : "0" (_prev), "1" (_next) \
803f69144 David Howells 2012-03-28  38  		       : "d0", "d2", "d3", "d4", "d5"); \
803f69144 David Howells 2012-03-28  39    (last) = _last; \
803f69144 David Howells 2012-03-28  40  } while (0)
803f69144 David Howells 2012-03-28  41  

:::::: The code at line 30 was first introduced by commit
:::::: 803f69144f0d48863c68f9d111b56849c7cef5bb Disintegrate asm/system.h for M68K

:::::: TO: David Howells <dhowells@...hat.com>
:::::: CC: David Howells <dhowells@...hat.com>

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

Download attachment ".config.gz" of type "application/gzip" (12419 bytes)