From: mark.rutland@arm.com (Mark Rutland)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/8] arm64: kernel: don't export local symbols from head.S
Date: Mon, 18 Apr 2016 16:35:41 +0100 [thread overview]
Message-ID: <20160418153540.GA7521@leverpostej> (raw)
In-Reply-To: <1460992188-23295-2-git-send-email-ard.biesheuvel@linaro.org>
On Mon, Apr 18, 2016 at 05:09:41PM +0200, Ard Biesheuvel wrote:
> This unexports some symbols from head.S that are only used locally.
It might be worth s/some/all/, as that makes this sound less arbitrary
(and AFAICS this caters for all symbols only used locally).
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Acked-by: Mark Rutland <mark.rutland@arm.com>
Mark.
> ---
> arch/arm64/kernel/head.S | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm64/kernel/head.S b/arch/arm64/kernel/head.S
> index b43417618847..ac27d8d937b2 100644
> --- a/arch/arm64/kernel/head.S
> +++ b/arch/arm64/kernel/head.S
> @@ -638,7 +638,7 @@ ENDPROC(el2_setup)
> * Sets the __boot_cpu_mode flag depending on the CPU boot mode passed
> * in x20. See arch/arm64/include/asm/virt.h for more info.
> */
> -ENTRY(set_cpu_boot_mode_flag)
> +set_cpu_boot_mode_flag:
> adr_l x1, __boot_cpu_mode
> cmp w20, #BOOT_CPU_MODE_EL2
> b.ne 1f
> @@ -691,7 +691,7 @@ ENTRY(secondary_entry)
> b secondary_startup
> ENDPROC(secondary_entry)
>
> -ENTRY(secondary_startup)
> +secondary_startup:
> /*
> * Common entry point for secondary CPUs.
> */
> @@ -706,7 +706,7 @@ ENTRY(secondary_startup)
> ENDPROC(secondary_startup)
> 0: .long (_text - TEXT_OFFSET) - __secondary_switched
>
> -ENTRY(__secondary_switched)
> +__secondary_switched:
> adr_l x5, vectors
> msr vbar_el1, x5
> isb
> --
> 2.5.0
>
WARNING: multiple messages have this Message-ID (diff)
From: Mark Rutland <mark.rutland@arm.com>
To: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, will.deacon@arm.com,
james.morse@arm.com, catalin.marinas@arm.com
Subject: Re: [PATCH 1/8] arm64: kernel: don't export local symbols from head.S
Date: Mon, 18 Apr 2016 16:35:41 +0100 [thread overview]
Message-ID: <20160418153540.GA7521@leverpostej> (raw)
In-Reply-To: <1460992188-23295-2-git-send-email-ard.biesheuvel@linaro.org>
On Mon, Apr 18, 2016 at 05:09:41PM +0200, Ard Biesheuvel wrote:
> This unexports some symbols from head.S that are only used locally.
It might be worth s/some/all/, as that makes this sound less arbitrary
(and AFAICS this caters for all symbols only used locally).
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Acked-by: Mark Rutland <mark.rutland@arm.com>
Mark.
> ---
> arch/arm64/kernel/head.S | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm64/kernel/head.S b/arch/arm64/kernel/head.S
> index b43417618847..ac27d8d937b2 100644
> --- a/arch/arm64/kernel/head.S
> +++ b/arch/arm64/kernel/head.S
> @@ -638,7 +638,7 @@ ENDPROC(el2_setup)
> * Sets the __boot_cpu_mode flag depending on the CPU boot mode passed
> * in x20. See arch/arm64/include/asm/virt.h for more info.
> */
> -ENTRY(set_cpu_boot_mode_flag)
> +set_cpu_boot_mode_flag:
> adr_l x1, __boot_cpu_mode
> cmp w20, #BOOT_CPU_MODE_EL2
> b.ne 1f
> @@ -691,7 +691,7 @@ ENTRY(secondary_entry)
> b secondary_startup
> ENDPROC(secondary_entry)
>
> -ENTRY(secondary_startup)
> +secondary_startup:
> /*
> * Common entry point for secondary CPUs.
> */
> @@ -706,7 +706,7 @@ ENTRY(secondary_startup)
> ENDPROC(secondary_startup)
> 0: .long (_text - TEXT_OFFSET) - __secondary_switched
>
> -ENTRY(__secondary_switched)
> +__secondary_switched:
> adr_l x5, vectors
> msr vbar_el1, x5
> isb
> --
> 2.5.0
>
next prev parent reply other threads:[~2016-04-18 15:35 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-18 15:09 [PATCH 0/8] arm64: kaslr cleanups and improvements Ard Biesheuvel
2016-04-18 15:09 ` Ard Biesheuvel
2016-04-18 15:09 ` [PATCH 1/8] arm64: kernel: don't export local symbols from head.S Ard Biesheuvel
2016-04-18 15:09 ` Ard Biesheuvel
2016-04-18 15:35 ` Mark Rutland [this message]
2016-04-18 15:35 ` Mark Rutland
2016-04-18 15:09 ` [PATCH 2/8] arm64: kernel: use literal for relocated address of __secondary_switched Ard Biesheuvel
2016-04-18 15:09 ` Ard Biesheuvel
2016-04-18 15:57 ` Mark Rutland
2016-04-18 15:57 ` Mark Rutland
2016-04-18 15:09 ` [PATCH 3/8] arm64: kernel: perform relocation processing from ID map Ard Biesheuvel
2016-04-18 15:09 ` Ard Biesheuvel
2016-04-18 15:09 ` [PATCH 4/8] arm64: introduce mov_q macro to move a constant into a 64-bit register Ard Biesheuvel
2016-04-18 15:09 ` Ard Biesheuvel
2016-04-18 15:09 ` [PATCH 5/8] arm64: kernel: replace early 64-bit literal loads with move-immediates Ard Biesheuvel
2016-04-18 15:09 ` Ard Biesheuvel
2016-04-19 14:29 ` Laurentiu Tudor
2016-04-19 14:29 ` Laurentiu Tudor
2016-04-18 15:09 ` [PATCH 6/8] arm64: don't map TEXT_OFFSET bytes below the kernel if we can avoid it Ard Biesheuvel
2016-04-18 15:09 ` Ard Biesheuvel
2016-04-18 15:09 ` [PATCH 7/8] arm64: relocatable: deal with physically misaligned kernel images Ard Biesheuvel
2016-04-18 15:09 ` Ard Biesheuvel
2016-04-18 15:09 ` [PATCH 8/8] arm64: kaslr: increase randomization granularity Ard Biesheuvel
2016-04-18 15:09 ` Ard Biesheuvel
2016-04-26 11:27 ` Will Deacon
2016-04-26 11:27 ` Will Deacon
2016-04-26 15:27 ` Matt Fleming
2016-04-26 15:27 ` Matt Fleming
2016-04-25 16:38 ` [PATCH 0/8] arm64: kaslr cleanups and improvements Catalin Marinas
2016-04-25 16:38 ` Catalin Marinas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160418153540.GA7521@leverpostej \
--to=mark.rutland@arm.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.