linux-kernel - Re: [PATCH 5/5] firmware: google memconsole: Add ARM/ARM64 support

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-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <201703260936.VB0xXnwH%fengguang.wu@intel.com>
Date:   Sun, 26 Mar 2017 09:41:17 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Thierry Escande <thierry.escande@...labora.com>
Cc:     kbuild-all@...org, Rob Herring <robh@...nel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Olof Johansson <olof@...om.net>,
        Stephen Warren <swarren@...dia.com>,
        linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
        Julius Werner <jwerner@...omium.org>,
        Brian Norris <briannorris@...omium.org>
Subject: Re: [PATCH 5/5] firmware: google memconsole: Add ARM/ARM64 support

Hi Thierry,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.11-rc3]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Thierry-Escande/firmware-google-memconsole/20170326-080828
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 6.2.0
reproduce:
        wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=ia64 

All errors (new ones prefixed by >>):

   drivers/firmware/google/gsmi.c: In function 'gsmi_exec.constprop':
>> drivers/firmware/google/gsmi.c:198:3: error: inconsistent operand constraints in an 'asm'
      asm volatile (
      ^~~
   drivers/firmware/google/gsmi.c:213:3: error: inconsistent operand constraints in an 'asm'
      asm volatile (
      ^~~
   drivers/firmware/google/gsmi.c:229:3: error: inconsistent operand constraints in an 'asm'
      asm volatile (
      ^~~

vim +/asm +198 drivers/firmware/google/gsmi.c

74c5b31c Mike Waychison 2011-04-29  182  	/*
74c5b31c Mike Waychison 2011-04-29  183  	 * AH  : Subfunction number
74c5b31c Mike Waychison 2011-04-29  184  	 * AL  : Function number
74c5b31c Mike Waychison 2011-04-29  185  	 * EBX : Parameter block address
74c5b31c Mike Waychison 2011-04-29  186  	 * DX  : SMI command port
74c5b31c Mike Waychison 2011-04-29  187  	 *
74c5b31c Mike Waychison 2011-04-29  188  	 * Three protocols here. See also the comment in gsmi_init().
74c5b31c Mike Waychison 2011-04-29  189  	 */
74c5b31c Mike Waychison 2011-04-29  190  	if (gsmi_dev.handshake_type == GSMI_HANDSHAKE_CF) {
74c5b31c Mike Waychison 2011-04-29  191  		/*
74c5b31c Mike Waychison 2011-04-29  192  		 * If handshake_type == HANDSHAKE_CF then set CF on the
74c5b31c Mike Waychison 2011-04-29  193  		 * way in and wait for the handler to clear it; this avoids
74c5b31c Mike Waychison 2011-04-29  194  		 * corrupting register state on those chipsets which have
74c5b31c Mike Waychison 2011-04-29  195  		 * a delay between writing the SMI trigger register and
74c5b31c Mike Waychison 2011-04-29  196  		 * entering SMM.
74c5b31c Mike Waychison 2011-04-29  197  		 */
74c5b31c Mike Waychison 2011-04-29 @198  		asm volatile (
74c5b31c Mike Waychison 2011-04-29  199  			"stc\n"
74c5b31c Mike Waychison 2011-04-29  200  			"outb %%al, %%dx\n"
74c5b31c Mike Waychison 2011-04-29  201  		"1:      jc 1b\n"
74c5b31c Mike Waychison 2011-04-29  202  			: "=a" (result)
74c5b31c Mike Waychison 2011-04-29  203  			: "0" (cmd),
74c5b31c Mike Waychison 2011-04-29  204  			  "d" (gsmi_dev.smi_cmd),
74c5b31c Mike Waychison 2011-04-29  205  			  "b" (gsmi_dev.param_buf->address)
74c5b31c Mike Waychison 2011-04-29  206  			: "memory", "cc"

:::::: The code at line 198 was first introduced by commit
:::::: 74c5b31c6618f01079212332b2e5f6c42f2d6307 driver: Google EFI SMI

:::::: TO: Mike Waychison <mikew@...gle.com>
:::::: CC: Greg Kroah-Hartman <gregkh@...e.de>

---
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" (46805 bytes)