Re: lib/test_user_copy.c:244: Error: unrecognized keyword/register name `l.lwz ?ap,4(r18)'
From: Kees Cook
Date: Mon Dec 30 2019 - 13:47:37 EST
On Fri, Dec 27, 2019 at 05:09:08PM +0800, kbuild test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 46cf053efec6a3a5f343fead837777efe8252a46
> commit: 6d13de1489b6bf539695f96d945de3860e6d5e17 uaccess: disallow > INT_MAX copy sizes
> date: 3 weeks ago
> config: openrisc-randconfig-a001-20191225 (attached as .config)
> compiler: or1k-linux-gcc (GCC) 9.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 6d13de1489b6bf539695f96d945de3860e6d5e17
> # save the attached .config to linux build tree
> GCC_VERSION=9.2.0 make.cross ARCH=openrisc
>
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@xxxxxxxxx>
>
> All errors (new ones prefixed by >>):
>
> lib/test_user_copy.c: Assembler messages:
> >> lib/test_user_copy.c:244: Error: unrecognized keyword/register name `l.lwz ?ap,4(r18)'
> >> lib/test_user_copy.c:249: Error: unrecognized keyword/register name `l.addi ?ap,r0,0'
Hi!
This looks like something deep in the usercopy code on openrisc?
Something likely around 64-bit copies, I'd guess. Can someone with
openrisc knowledge take a look at this?
Thanks!
-Kees
>
> vim +244 lib/test_user_copy.c
>
> 3e2a4c183ace87 Kees Cook 2014-01-23 193
> 3e2a4c183ace87 Kees Cook 2014-01-23 194 kmem = kmalloc(PAGE_SIZE * 2, GFP_KERNEL);
> 3e2a4c183ace87 Kees Cook 2014-01-23 195 if (!kmem)
> 3e2a4c183ace87 Kees Cook 2014-01-23 196 return -ENOMEM;
> 3e2a4c183ace87 Kees Cook 2014-01-23 197
> 3e2a4c183ace87 Kees Cook 2014-01-23 198 user_addr = vm_mmap(NULL, 0, PAGE_SIZE * 2,
> 3e2a4c183ace87 Kees Cook 2014-01-23 199 PROT_READ | PROT_WRITE | PROT_EXEC,
> 3e2a4c183ace87 Kees Cook 2014-01-23 200 MAP_ANONYMOUS | MAP_PRIVATE, 0);
> 3e2a4c183ace87 Kees Cook 2014-01-23 201 if (user_addr >= (unsigned long)(TASK_SIZE)) {
> 3e2a4c183ace87 Kees Cook 2014-01-23 202 pr_warn("Failed to allocate user memory\n");
> 3e2a4c183ace87 Kees Cook 2014-01-23 203 kfree(kmem);
> 3e2a4c183ace87 Kees Cook 2014-01-23 204 return -ENOMEM;
> 3e2a4c183ace87 Kees Cook 2014-01-23 205 }
> 3e2a4c183ace87 Kees Cook 2014-01-23 206
> 3e2a4c183ace87 Kees Cook 2014-01-23 207 usermem = (char __user *)user_addr;
> 3e2a4c183ace87 Kees Cook 2014-01-23 208 bad_usermem = (char *)user_addr;
> 3e2a4c183ace87 Kees Cook 2014-01-23 209
> f5f893c57e37ca Kees Cook 2017-02-13 210 /*
> f5f893c57e37ca Kees Cook 2017-02-13 211 * Legitimate usage: none of these copies should fail.
> f5f893c57e37ca Kees Cook 2017-02-13 212 */
> 4c5d7bc63775b4 Kees Cook 2017-02-14 213 memset(kmem, 0x3a, PAGE_SIZE * 2);
> 3e2a4c183ace87 Kees Cook 2014-01-23 214 ret |= test(copy_to_user(usermem, kmem, PAGE_SIZE),
> 3e2a4c183ace87 Kees Cook 2014-01-23 215 "legitimate copy_to_user failed");
> 4c5d7bc63775b4 Kees Cook 2017-02-14 216 memset(kmem, 0x0, PAGE_SIZE);
> 4c5d7bc63775b4 Kees Cook 2017-02-14 217 ret |= test(copy_from_user(kmem, usermem, PAGE_SIZE),
> 4c5d7bc63775b4 Kees Cook 2017-02-14 218 "legitimate copy_from_user failed");
> 4c5d7bc63775b4 Kees Cook 2017-02-14 219 ret |= test(memcmp(kmem, kmem + PAGE_SIZE, PAGE_SIZE),
> 4c5d7bc63775b4 Kees Cook 2017-02-14 220 "legitimate usercopy failed to copy data");
> 4c5d7bc63775b4 Kees Cook 2017-02-14 221
> 4c5d7bc63775b4 Kees Cook 2017-02-14 222 #define test_legit(size, check) \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 223 do { \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 224 val_##size = check; \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 225 ret |= test(put_user(val_##size, (size __user *)usermem), \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 226 "legitimate put_user (" #size ") failed"); \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 227 val_##size = 0; \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 228 ret |= test(get_user(val_##size, (size __user *)usermem), \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 229 "legitimate get_user (" #size ") failed"); \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 230 ret |= test(val_##size != check, \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 231 "legitimate get_user (" #size ") failed to do copy"); \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 232 if (val_##size != check) { \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 233 pr_info("0x%llx != 0x%llx\n", \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 234 (unsigned long long)val_##size, \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 235 (unsigned long long)check); \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 236 } \
> 4c5d7bc63775b4 Kees Cook 2017-02-14 237 } while (0)
> 4c5d7bc63775b4 Kees Cook 2017-02-14 238
> 4c5d7bc63775b4 Kees Cook 2017-02-14 239 test_legit(u8, 0x5a);
> 4c5d7bc63775b4 Kees Cook 2017-02-14 240 test_legit(u16, 0x5a5b);
> 4c5d7bc63775b4 Kees Cook 2017-02-14 241 test_legit(u32, 0x5a5b5c5d);
> 4c5d7bc63775b4 Kees Cook 2017-02-14 242 #ifdef TEST_U64
> 4c5d7bc63775b4 Kees Cook 2017-02-14 243 test_legit(u64, 0x5a5b5c5d6a6b6c6d);
> 4c5d7bc63775b4 Kees Cook 2017-02-14 @244 #endif
> 4c5d7bc63775b4 Kees Cook 2017-02-14 245 #undef test_legit
> 3e2a4c183ace87 Kees Cook 2014-01-23 246
> f5a1a536fa1489 Aleksa Sarai 2019-10-01 247 /* Test usage of check_nonzero_user(). */
> f5a1a536fa1489 Aleksa Sarai 2019-10-01 248 ret |= test_check_nonzero_user(kmem, usermem, 2 * PAGE_SIZE);
> f5a1a536fa1489 Aleksa Sarai 2019-10-01 @249 /* Test usage of copy_struct_from_user(). */
> f5a1a536fa1489 Aleksa Sarai 2019-10-01 250 ret |= test_copy_struct_from_user(kmem, usermem, 2 * PAGE_SIZE);
> f5a1a536fa1489 Aleksa Sarai 2019-10-01 251
> f5f893c57e37ca Kees Cook 2017-02-13 252 /*
> f5f893c57e37ca Kees Cook 2017-02-13 253 * Invalid usage: none of these copies should succeed.
> f5f893c57e37ca Kees Cook 2017-02-13 254 */
> f5f893c57e37ca Kees Cook 2017-02-13 255
> f5f893c57e37ca Kees Cook 2017-02-13 256 /* Prepare kernel memory with check values. */
> 4fbfeb8bd684d5 Hoeun Ryu 2017-02-12 257 memset(kmem, 0x5a, PAGE_SIZE);
> 4fbfeb8bd684d5 Hoeun Ryu 2017-02-12 258 memset(kmem + PAGE_SIZE, 0, PAGE_SIZE);
> f5f893c57e37ca Kees Cook 2017-02-13 259
> f5f893c57e37ca Kees Cook 2017-02-13 260 /* Reject kernel-to-kernel copies through copy_from_user(). */
> 3e2a4c183ace87 Kees Cook 2014-01-23 261 ret |= test(!copy_from_user(kmem, (char __user *)(kmem + PAGE_SIZE),
> 3e2a4c183ace87 Kees Cook 2014-01-23 262 PAGE_SIZE),
> 3e2a4c183ace87 Kees Cook 2014-01-23 263 "illegal all-kernel copy_from_user passed");
> f5f893c57e37ca Kees Cook 2017-02-13 264
> f5f893c57e37ca Kees Cook 2017-02-13 265 /* Destination half of buffer should have been zeroed. */
> 4fbfeb8bd684d5 Hoeun Ryu 2017-02-12 266 ret |= test(memcmp(kmem + PAGE_SIZE, kmem, PAGE_SIZE),
> 4fbfeb8bd684d5 Hoeun Ryu 2017-02-12 267 "zeroing failure for illegal all-kernel copy_from_user");
> f5f893c57e37ca Kees Cook 2017-02-13 268
>
> :::::: The code at line 244 was first introduced by commit
> :::::: 4c5d7bc63775b40631b75f6c59a3a3005455262d usercopy: Add tests for all get_user() sizes
>
> :::::: TO: Kees Cook <keescook@xxxxxxxxxxxx>
> :::::: CC: Kees Cook <keescook@xxxxxxxxxxxx>
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx Intel Corporation
--
Kees Cook