linux-kernel - Re: [PATCH] x86_32: add support for 64 bit __get

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]

Date:	Thu, 10 Mar 2016 02:22:04 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Benjamin LaHaise <bcrl@...ck.org>
Cc:	kbuild-all@...org, Ingo Molnar <mingo@...hat.com>,
	Russell King <rmk@....linux.org.uk>,
	Thomas Gleixner <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...or.com>, x86@...nel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] x86_32: add support for 64 bit __get_user()

Hi Benjamin,

[auto build test ERROR on tip/x86/core]
[also build test ERROR on v4.5-rc7 next-20160309]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Benjamin-LaHaise/x86_32-add-support-for-64-bit-__get_user/20160310-012551
config: x86_64-randconfig-s3-03100205 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   kernel/ptrace.c: Assembler messages:
   kernel/ptrace.c:1039: Error: operand type mismatch for `movq'
>> kernel/ptrace.c:1043: Error: incorrect register `%cx' used with `q' suffix
   kernel/ptrace.c:1040: Error: operand type mismatch for `movq'
   kernel/ptrace.c:1044: Error: incorrect register `%cx' used with `q' suffix

vim +1043 kernel/ptrace.c

9fed81dc Namhyung Kim  2010-10-27  1033  		struct iovec __user *uiov = datavp;
2225a122 Suresh Siddha 2010-02-11  1034  
2225a122 Suresh Siddha 2010-02-11  1035  		if (!access_ok(VERIFY_WRITE, uiov, sizeof(*uiov)))
2225a122 Suresh Siddha 2010-02-11  1036  			return -EFAULT;
2225a122 Suresh Siddha 2010-02-11  1037  
2225a122 Suresh Siddha 2010-02-11  1038  		if (__get_user(kiov.iov_base, &uiov->iov_base) ||
2225a122 Suresh Siddha 2010-02-11 @1039  		    __get_user(kiov.iov_len, &uiov->iov_len))
2225a122 Suresh Siddha 2010-02-11  1040  			return -EFAULT;
2225a122 Suresh Siddha 2010-02-11  1041  
2225a122 Suresh Siddha 2010-02-11  1042  		ret = ptrace_regset(child, request, addr, &kiov);
2225a122 Suresh Siddha 2010-02-11 @1043  		if (!ret)
2225a122 Suresh Siddha 2010-02-11  1044  			ret = __put_user(kiov.iov_len, &uiov->iov_len);
2225a122 Suresh Siddha 2010-02-11  1045  		break;
2225a122 Suresh Siddha 2010-02-11  1046  	}

:::::: The code at line 1043 was first introduced by commit
:::::: 2225a122ae26d542bdce523d9d87a4a7ba10e07b ptrace: Add support for generic PTRACE_GETREGSET/PTRACE_SETREGSET

:::::: TO: Suresh Siddha <suresh.b.siddha@...el.com>
:::::: CC: H. Peter Anvin <hpa@...or.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/octet-stream" (25154 bytes)