[patch 112/178] arm64: fix add kasan bug

zhong jiang zhongjiang at huawei.com
Sat Jan 16 06:26:43 PST 2016


On 2016/1/15 18:35, Catalin Marinas wrote:
> On Fri, Jan 15, 2016 at 10:39:04AM +0300, Andrey Ryabinin wrote:
>> 2016-01-15 2:22 GMT+03:00  <akpm at linux-foundation.org>:
>>> From: zhong jiang <zhongjiang at huawei.com>
>>> Subject: arm64: fix add kasan bug
>>>
>>> In general, each process have 16kb stack space to use, but stack need
>>> extra space to store red_zone when kasan enable.  the patch fix above
>>> question.
>>>
>>> Signed-off-by: zhong jiang <zhongjiang at huawei.com>
>>> Cc: Andrey Ryabinin <ryabinin.a.a at gmail.com>
>>> Cc: Catalin Marinas <catalin.marinas at arm.com>
>>> Cc: Xishi Qiu <qiuxishi at huawei.com>
>>> Cc: Wang Long <long.wanglong at huawei.com>
>>> Signed-off-by: Andrew Morton <akpm at linux-foundation.org>
>>> ---
>>>
>>>  arch/arm64/include/asm/thread_info.h |   15 +++++++++++++--
>>>  1 file changed, 13 insertions(+), 2 deletions(-)
>>
>> This has been nacked by Catalin:
>> http://lkml.kernel.org/g/<20160108182744.GQ16432@e104818-lin.cambridge.arm.com>
> 
> As above, the patch doesn't make much sense for 4.5 where we have
> separate IRQ stacks. Arguably, it could be included in 4.4 *if* there
> was a regression but the patch author failed to reply.
> 
> Thanks.
> 

I'm so sorry that I don't respond to this email in time, Because I thought
it has been recognized without any discussion. and I am busy with other
things during the time. at present, I have looked at all of the discussion,
and I have some words want to say.

when kasan enables, compiler need to insert an additional 32-byte redzon to
a local variable in the front and behind. as a result, The actual usable stack
space is less than half of the total. The main purpose of this patch is whether
kasan enable or not, the actual usable stack space always satisfied the demand
to avoid the risk of the stack overflow.

Thanks
zhongjiang






More information about the linux-arm-kernel mailing list