lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <57B482A7.1090104@arm.com>
Date:	Wed, 17 Aug 2016 16:28:39 +0100
From:	Vladimir Murzin <vladimir.murzin@....com>
To:	Alexandre TORGUE <alexandre.torgue@...com>,
	Russell King <linux@...linux.org.uk>,
	Ezequiel Garcia <ezequiel@...guardiasur.com.ar>,
	u.kleine-koenig@...gutronix.de,
	Maxime Coquelin <mcoquelin.stm32@...il.com>
Cc:	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH] ARM: V7M: Add dsb before jumping in handler mode

On 15/06/16 16:36, Vladimir Murzin wrote:
> On 13/06/16 15:12, Alexandre TORGUE wrote:
>> According to ARM AN321 (section 4.12):
>>
>> "If the vector table is in writable memory such as SRAM, either relocated
>> by VTOR or a device dependent memory remapping mechanism, then
>> architecturally a memory barrier instruction is required after the vector
>> table entry is updated, and if the exception is to be activated
>> immediately"
>>
>> Signed-off-by: Maxime Coquelin <mcoquelin.stm32@...il.com>
>> Signed-off-by: Alexandre TORGUE <alexandre.torgue@...com>
>>
> 
> Reviewed-by: Vladimir Murzin <vladimir.murzin@....com>
> 
>> diff --git a/arch/arm/mm/proc-v7m.S b/arch/arm/mm/proc-v7m.S
>> index 7229d8d..2ddc435 100644
>> --- a/arch/arm/mm/proc-v7m.S
>> +++ b/arch/arm/mm/proc-v7m.S
>> @@ -104,6 +104,7 @@ __v7m_setup:
>>  	badr	r1, 1f
>>  	ldr	r5, [r12, #11 * 4]	@ read the SVC vector entry
>>  	str	r1, [r12, #11 * 4]	@ write the temporary SVC vector entry
>> +	dsb
>>  	mov	r6, lr			@ save LR
>>  	ldr	sp, =init_thread_union + THREAD_START_SP
>>  	cpsie	i
>>
> 

Alex, xan you drop it into RMK's Patch system [1] please?

[1] http://www.armlinux.org.uk/developer/patches/

Thanks
Vladimir

> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@...ts.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 
> 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ