From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752176AbcGSAxx (ORCPT ); Mon, 18 Jul 2016 20:53:53 -0400 Received: from mail-qk0-f178.google.com ([209.85.220.178]:36099 "EHLO mail-qk0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751492AbcGSAxu (ORCPT ); Mon, 18 Jul 2016 20:53:50 -0400 Subject: Re: [PATCH v15 06/10] arm64: Treat all entry code as non-kprobe-able To: Catalin Marinas References: <1467995754-32508-1-git-send-email-dave.long@linaro.org> <1467995754-32508-7-git-send-email-dave.long@linaro.org> <20160715164720.GC14792@e104818-lin.cambridge.arm.com> Cc: Huang Shijie , James Morse , Marc Zyngier , Pratyush Anand , Sandeepa Prabhu , Will Deacon , William Cohen , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Steve Capper , Masami Hiramatsu , Li Bin , Jisheng Zhang , Mark Rutland , Daniel Thompson , Vladimir Murzin , Petr Mladek , Ard Biesheuvel , Jens Wiklander , Robin Murphy , Mark Brown , Suzuki K Poulose , Dave P Martin , Andrey Ryabinin , yalin wang , Yang Shi , Zi Shen Lim , John Blackwood , Andrew Morton , =?UTF-8?Q?Alex_Benn=c3=a9e?= , Adam Buchbinder , Christoffer Dall From: David Long Message-ID: <578D7A14.6070008@linaro.org> Date: Mon, 18 Jul 2016 20:53:40 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <20160715164720.GC14792@e104818-lin.cambridge.arm.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/15/2016 12:47 PM, Catalin Marinas wrote: > On Fri, Jul 08, 2016 at 12:35:50PM -0400, David Long wrote: >> --- a/arch/arm64/kernel/entry.S >> +++ b/arch/arm64/kernel/entry.S >> @@ -243,6 +243,7 @@ tsk .req x28 // current thread_info >> * Exception vectors. >> */ >> >> + .pushsection ".entry.text", "ax" >> .align 11 >> ENTRY(vectors) >> ventry el1_sync_invalid // Synchronous EL1t >> @@ -781,3 +782,5 @@ ENTRY(sys_rt_sigreturn_wrapper) >> mov x0, sp >> b sys_rt_sigreturn >> ENDPROC(sys_rt_sigreturn_wrapper) >> + >> + .popsection > > Does the above sigreturn wrapper need to be included in the .entry.text > section? > Apparently not. It wouldn't make sense for that to be in entry.text when sys_rt_sigreturn() isn't. I'll put that in the list of changes. Thanks, -dl