* Re: [PATCH 7/7] Enable KASan for arm
@ 2018-03-24 13:55 Liuwenliang (Abbott Liu)
0 siblings, 0 replies; 3+ messages in thread
From: Liuwenliang (Abbott Liu) @ 2018-03-24 13:55 UTC (permalink / raw)
To: kbuild test robot
Cc: kbuild-all, linux, aryabinin, marc.zyngier, kstewart, gregkh,
f.fainelli, akpm, afzal.mohd.ma, alexander.levin, glider,
dvyukov, christoffer.dall, linux, mawilcox, pombredanne,
ard.biesheuvel, vladimir.murzin, nicolas.pitre, tglx, thgarnie,
dhowells, keescook, arnd, geert, tixy, mark.rutland, james.morse,
zhichao.huang, jinb.park7, labbott, philip, grygorii.strashko,
catalin.marinas, opendmb, kirill.shutemov, linux-arm-kernel,
linux-kernel, kasan-dev, kvmarm, linux-mm
On 03/20/2018 2:30 AM, kbuild test robot wrote:
>All errors (new ones prefixed by >>):
>
> arch/arm/kernel/entry-common.S: Assembler messages:
>>> arch/arm/kernel/entry-common.S:85: Error: invalid constant (ffffffffb6e00000) after fixup
I'm sorry!
We need to add the fellowing code to solve the upper error:
> git diff
diff --git a/arch/arm/kernel/entry-common.S b/arch/arm/kernel/entry-common.S
index b7d0c6c..9b728c5 100644
--- a/arch/arm/kernel/entry-common.S
+++ b/arch/arm/kernel/entry-common.S
@@ -82,7 +82,8 @@ ret_fast_syscall:
str r0, [sp, #S_R0 + S_OFF]! @ save returned r0
disable_irq_notrace @ disable interrupts
ldr r2, [tsk, #TI_ADDR_LIMIT]
- cmp r2, #TASK_SIZE
+ ldr r1, =TASK_SIZE
+ cmp r2, r1
blne addr_limit_check_failed
ldr r1, [tsk, #TI_FLAGS] @ re-check for syscall tracing
tst r1, #_TIF_SYSCALL_WORK | _TIF_WORK_MASK
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 7/7] Enable KASan for arm
2018-03-18 12:53 ` [PATCH 7/7] Enable " Abbott Liu
@ 2018-03-19 20:43 ` kbuild test robot
0 siblings, 0 replies; 3+ messages in thread
From: kbuild test robot @ 2018-03-19 20:43 UTC (permalink / raw)
To: Abbott Liu
Cc: kbuild-all, linux, aryabinin, marc.zyngier, kstewart, gregkh,
f.fainelli, liuwenliang, akpm, afzal.mohd.ma, alexander.levin,
glider, dvyukov, christoffer.dall, linux, mawilcox, pombredanne,
ard.biesheuvel, vladimir.murzin, nicolas.pitre, tglx, thgarnie,
dhowells, keescook, arnd, geert, tixy, mark.rutland, james.morse,
zhichao.huang, jinb.park7, labbott, philip, grygorii.strashko,
catalin.marinas, opendmb, kirill.shutemov, linux-arm-kernel,
linux-kernel, kasan-dev, kvmarm, linux-mm
[-- Attachment #1: Type: text/plain, Size: 4310 bytes --]
Hi Andrey,
I love your patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v4.16-rc6]
[cannot apply to next-20180319]
[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/Abbott-Liu/KASan-for-arm/20180319-120138
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-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
# save the attached .config to linux build tree
make.cross ARCH=arm
All errors (new ones prefixed by >>):
arch/arm/kernel/entry-common.S: Assembler messages:
>> arch/arm/kernel/entry-common.S:85: Error: invalid constant (ffffffffb6e00000) after fixup
vim +85 arch/arm/kernel/entry-common.S
^1da177e4 Linus Torvalds 2005-04-16 68
3302caddf Russell King 2015-08-20 69 /* Ok, we need to do extra processing, enter the slow path. */
^1da177e4 Linus Torvalds 2005-04-16 70 fast_work_pending:
^1da177e4 Linus Torvalds 2005-04-16 71 str r0, [sp, #S_R0+S_OFF]! @ returned r0
3302caddf Russell King 2015-08-20 72 /* fall through to work_pending */
3302caddf Russell King 2015-08-20 73 #else
3302caddf Russell King 2015-08-20 74 /*
3302caddf Russell King 2015-08-20 75 * The "replacement" ret_fast_syscall for when tracing or context tracking
3302caddf Russell King 2015-08-20 76 * is enabled. As we will need to call out to some C functions, we save
3302caddf Russell King 2015-08-20 77 * r0 first to avoid needing to save registers around each C function call.
3302caddf Russell King 2015-08-20 78 */
3302caddf Russell King 2015-08-20 79 ret_fast_syscall:
3302caddf Russell King 2015-08-20 80 UNWIND(.fnstart )
3302caddf Russell King 2015-08-20 81 UNWIND(.cantunwind )
3302caddf Russell King 2015-08-20 82 str r0, [sp, #S_R0 + S_OFF]! @ save returned r0
3302caddf Russell King 2015-08-20 83 disable_irq_notrace @ disable interrupts
e33f8d326 Thomas Garnier 2017-09-07 84 ldr r2, [tsk, #TI_ADDR_LIMIT]
e33f8d326 Thomas Garnier 2017-09-07 @85 cmp r2, #TASK_SIZE
e33f8d326 Thomas Garnier 2017-09-07 86 blne addr_limit_check_failed
3302caddf Russell King 2015-08-20 87 ldr r1, [tsk, #TI_FLAGS] @ re-check for syscall tracing
2404269bc Thomas Garnier 2017-09-07 88 tst r1, #_TIF_SYSCALL_WORK | _TIF_WORK_MASK
3302caddf Russell King 2015-08-20 89 beq no_work_pending
3302caddf Russell King 2015-08-20 90 UNWIND(.fnend )
3302caddf Russell King 2015-08-20 91 ENDPROC(ret_fast_syscall)
3302caddf Russell King 2015-08-20 92
3302caddf Russell King 2015-08-20 93 /* Slower path - fall through to work_pending */
3302caddf Russell King 2015-08-20 94 #endif
3302caddf Russell King 2015-08-20 95
3302caddf Russell King 2015-08-20 96 tst r1, #_TIF_SYSCALL_WORK
3302caddf Russell King 2015-08-20 97 bne __sys_trace_return_nosave
3302caddf Russell King 2015-08-20 98 slow_work_pending:
^1da177e4 Linus Torvalds 2005-04-16 99 mov r0, sp @ 'regs'
^1da177e4 Linus Torvalds 2005-04-16 100 mov r2, why @ 'syscall'
0a267fa6a Al Viro 2012-07-19 101 bl do_work_pending
662852178 Al Viro 2012-07-19 102 cmp r0, #0
81783786d Al Viro 2012-07-19 103 beq no_work_pending
662852178 Al Viro 2012-07-19 104 movlt scno, #(__NR_restart_syscall - __NR_SYSCALL_BASE)
81783786d Al Viro 2012-07-19 105 ldmia sp, {r0 - r6} @ have to reload r0 - r6
81783786d Al Viro 2012-07-19 106 b local_restart @ ... and off we go
e83dd3770 Drew Richardson 2015-08-06 107 ENDPROC(ret_fast_syscall)
81783786d Al Viro 2012-07-19 108
:::::: The code at line 85 was first introduced by commit
:::::: e33f8d32677fa4f4f8996ef46748f86aac81ccff arm/syscalls: Optimize address limit check
:::::: TO: Thomas Garnier <thgarnie@google.com>
:::::: CC: Thomas Gleixner <tglx@linutronix.de>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 65135 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 7/7] Enable KASan for arm
2018-03-18 12:53 [PATCH v2 0/7] " Abbott Liu
@ 2018-03-18 12:53 ` Abbott Liu
2018-03-19 20:43 ` kbuild test robot
0 siblings, 1 reply; 3+ messages in thread
From: Abbott Liu @ 2018-03-18 12:53 UTC (permalink / raw)
To: linux, aryabinin, marc.zyngier, kstewart, gregkh, f.fainelli,
liuwenliang, akpm, afzal.mohd.ma, alexander.levin
Cc: glider, dvyukov, christoffer.dall, linux, mawilcox, pombredanne,
ard.biesheuvel, vladimir.murzin, nicolas.pitre, tglx, thgarnie,
dhowells, keescook, arnd, geert, tixy, mark.rutland, james.morse,
zhichao.huang, jinb.park7, labbott, philip, grygorii.strashko,
catalin.marinas, opendmb, kirill.shutemov, linux-arm-kernel,
linux-kernel, kasan-dev, kvmarm, linux-mm
From: Andrey Ryabinin <a.ryabinin@samsung.com>
This patch enable kernel address sanitizer for arm.
Cc: Andrey Ryabinin <a.ryabinin@samsung.com>
Tested-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Abbott Liu <liuwenliang@huawei.com>
---
arch/arm/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 7e3d535..ac2287b 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -49,6 +49,7 @@ config ARM
select HAVE_ARCH_BITREVERSE if (CPU_32v7M || CPU_32v7) && !CPU_32v6
select HAVE_ARCH_JUMP_LABEL if !XIP_KERNEL && !CPU_ENDIAN_BE32 && MMU
select HAVE_ARCH_KGDB if !CPU_ENDIAN_BE32 && MMU
+ select HAVE_ARCH_KASAN if MMU
select HAVE_ARCH_MMAP_RND_BITS if MMU
select HAVE_ARCH_SECCOMP_FILTER if (AEABI && !OABI_COMPAT)
select HAVE_ARCH_THREAD_STRUCT_WHITELIST
--
2.9.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-03-24 13:55 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-24 13:55 [PATCH 7/7] Enable KASan for arm Liuwenliang (Abbott Liu)
-- strict thread matches above, loose matches on Subject: below --
2018-03-18 12:53 [PATCH v2 0/7] " Abbott Liu
2018-03-18 12:53 ` [PATCH 7/7] Enable " Abbott Liu
2018-03-19 20:43 ` kbuild test robot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).