* [PATCH] ARM: NOMMU: Reload __secondary_data after PROCINFO_INITFUNC
@ 2021-11-25 10:48 Vladimir Murzin
2021-11-25 10:58 ` Russell King (Oracle)
0 siblings, 1 reply; 3+ messages in thread
From: Vladimir Murzin @ 2021-11-25 10:48 UTC (permalink / raw)
To: linux-arm-kernel; +Cc: ardb
__secondary_data used to reside in r7 around call to
PROCINFO_INITFUNC. After commit 95731b8ee63e ("ARM: 9059/1: cache-v7:
get rid of mini-stack") r7 is used as a scratch register, so we have
to reload __secondary_data before we setup the stack pointer.
Fixes: 95731b8ee63ec9419822a51cd9878fa32582fdd2 ("ARM: 9059/1: cache-v7: get rid of mini-stack")
Signed-off-by: Vladimir Murzin <vladimir.murzin@arm.com>
---
arch/arm/kernel/head-nommu.S | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/kernel/head-nommu.S b/arch/arm/kernel/head-nommu.S
index 0fc814b..8796a69 100644
--- a/arch/arm/kernel/head-nommu.S
+++ b/arch/arm/kernel/head-nommu.S
@@ -114,6 +114,7 @@ ENTRY(secondary_startup)
add r12, r12, r10
ret r12
1: bl __after_proc_init
+ ldr r7, __secondary_data @ reload r7
ldr sp, [r7, #12] @ set up the stack pointer
mov fp, #0
b secondary_start_kernel
--
2.7.4
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] ARM: NOMMU: Reload __secondary_data after PROCINFO_INITFUNC
2021-11-25 10:48 [PATCH] ARM: NOMMU: Reload __secondary_data after PROCINFO_INITFUNC Vladimir Murzin
@ 2021-11-25 10:58 ` Russell King (Oracle)
2021-11-25 11:09 ` Vladimir Murzin
0 siblings, 1 reply; 3+ messages in thread
From: Russell King (Oracle) @ 2021-11-25 10:58 UTC (permalink / raw)
To: Vladimir Murzin; +Cc: linux-arm-kernel, ardb
On Thu, Nov 25, 2021 at 10:48:41AM +0000, Vladimir Murzin wrote:
> __secondary_data used to reside in r7 around call to
> PROCINFO_INITFUNC. After commit 95731b8ee63e ("ARM: 9059/1: cache-v7:
> get rid of mini-stack") r7 is used as a scratch register, so we have
> to reload __secondary_data before we setup the stack pointer.
>
> Fixes: 95731b8ee63ec9419822a51cd9878fa32582fdd2 ("ARM: 9059/1: cache-v7: get rid of mini-stack")
Documentation/process/submitting-patches.rst requests only 12 digits for
the sha1 in the fixes line, which makes this:
Fixes: 95731b8ee63e ("ARM: 9059/1: cache-v7: get rid of mini-stack")
and avoids it being unnecessarily long. Please make that change before
sending it to the patch system, thanks.
Other than that, patch looks good, thanks.
>
> Signed-off-by: Vladimir Murzin <vladimir.murzin@arm.com>
> ---
> arch/arm/kernel/head-nommu.S | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/kernel/head-nommu.S b/arch/arm/kernel/head-nommu.S
> index 0fc814b..8796a69 100644
> --- a/arch/arm/kernel/head-nommu.S
> +++ b/arch/arm/kernel/head-nommu.S
> @@ -114,6 +114,7 @@ ENTRY(secondary_startup)
> add r12, r12, r10
> ret r12
> 1: bl __after_proc_init
> + ldr r7, __secondary_data @ reload r7
> ldr sp, [r7, #12] @ set up the stack pointer
> mov fp, #0
> b secondary_start_kernel
> --
> 2.7.4
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] ARM: NOMMU: Reload __secondary_data after PROCINFO_INITFUNC
2021-11-25 10:58 ` Russell King (Oracle)
@ 2021-11-25 11:09 ` Vladimir Murzin
0 siblings, 0 replies; 3+ messages in thread
From: Vladimir Murzin @ 2021-11-25 11:09 UTC (permalink / raw)
To: Russell King (Oracle); +Cc: linux-arm-kernel, ardb
On 11/25/21 10:58 AM, Russell King (Oracle) wrote:
> On Thu, Nov 25, 2021 at 10:48:41AM +0000, Vladimir Murzin wrote:
>> __secondary_data used to reside in r7 around call to
>> PROCINFO_INITFUNC. After commit 95731b8ee63e ("ARM: 9059/1: cache-v7:
>> get rid of mini-stack") r7 is used as a scratch register, so we have
>> to reload __secondary_data before we setup the stack pointer.
>>
>> Fixes: 95731b8ee63ec9419822a51cd9878fa32582fdd2 ("ARM: 9059/1: cache-v7: get rid of mini-stack")
>
> Documentation/process/submitting-patches.rst requests only 12 digits for
> the sha1 in the fixes line, which makes this:
>
> Fixes: 95731b8ee63e ("ARM: 9059/1: cache-v7: get rid of mini-stack")
>
> and avoids it being unnecessarily long. Please make that change before
> sending it to the patch system, thanks.
Done.
>
> Other than that, patch looks good, thanks.
Thanks for prompt feedback!
Cheers
Vladimir
>
>>
>> Signed-off-by: Vladimir Murzin <vladimir.murzin@arm.com>
>> ---
>> arch/arm/kernel/head-nommu.S | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/kernel/head-nommu.S b/arch/arm/kernel/head-nommu.S
>> index 0fc814b..8796a69 100644
>> --- a/arch/arm/kernel/head-nommu.S
>> +++ b/arch/arm/kernel/head-nommu.S
>> @@ -114,6 +114,7 @@ ENTRY(secondary_startup)
>> add r12, r12, r10
>> ret r12
>> 1: bl __after_proc_init
>> + ldr r7, __secondary_data @ reload r7
>> ldr sp, [r7, #12] @ set up the stack pointer
>> mov fp, #0
>> b secondary_start_kernel
>> --
>> 2.7.4
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>>
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-11-25 11:13 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-11-25 10:48 [PATCH] ARM: NOMMU: Reload __secondary_data after PROCINFO_INITFUNC Vladimir Murzin
2021-11-25 10:58 ` Russell King (Oracle)
2021-11-25 11:09 ` Vladimir Murzin
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).