Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2015-02-08 Thread Richard Earnshaw
On 17/11/14 17:03, pins...@gmail.com wrote: > > > > >> On Nov 17, 2014, at 8:59 AM, Ramana Radhakrishnan >> wrote: >> >>> On Mon, Nov 17, 2014 at 2:48 PM, Kyrill Tkachov >>> wrote: >>> Hi all, >>> >>> Some configurations of Cortex-A53 and Cortex-A57 don't ship with crypto, >>> so enabling i

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-26 Thread Gerald Pfeifer
On Tuesday 2014-11-25 16:08, Kyrill Tkachov wrote: The change is to the behaviour of -mcpu, not -march. -march is only mentioned as a way of getting the previous behaviour if the user so wishes. Ah, okay. How about this amendment? Where you have "add the +crypto extension to the value" I s

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-25 Thread Kyrill Tkachov
On 25/11/14 01:36, Gerald Pfeifer wrote: On Tuesday 2014-11-18 09:38, Kyrill Tkachov wrote: Here's what I propose. + The cryptographic extensions to the ARMv8-A architecture are no + longer enabled by default when specifying the + -mcpu=cortex-a53, -mcpu=cortex-a57 or +

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-24 Thread Gerald Pfeifer
On Tuesday 2014-11-18 09:38, Kyrill Tkachov wrote: Here's what I propose. + The cryptographic extensions to the ARMv8-A architecture are no + longer enabled by default when specifying the + -mcpu=cortex-a53, -mcpu=cortex-a57 or + -mcpu=cortex-a57.cortex-a53 options. To e

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-18 Thread Richard Earnshaw
On 18/11/14 09:38, Kyrill Tkachov wrote: > > On 17/11/14 16:59, Ramana Radhakrishnan wrote: >> On Mon, Nov 17, 2014 at 2:48 PM, Kyrill Tkachov >> wrote: >>> Hi all, >>> >>> Some configurations of Cortex-A53 and Cortex-A57 don't ship with crypto, >>> so enabling it by default for -mcpu=cortex-a53

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-18 Thread Kyrill Tkachov
On 17/11/14 16:59, Ramana Radhakrishnan wrote: On Mon, Nov 17, 2014 at 2:48 PM, Kyrill Tkachov wrote: Hi all, Some configurations of Cortex-A53 and Cortex-A57 don't ship with crypto, so enabling it by default for -mcpu=cortex-a53 and cortex-a57 is inappropriate. Tested aarch64-none-elf. Remi

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-17 Thread Marcus Shawcroft
On 17 November 2014 14:48, Kyrill Tkachov wrote: > Hi all, > > Some configurations of Cortex-A53 and Cortex-A57 don't ship with crypto, > so enabling it by default for -mcpu=cortex-a53 and cortex-a57 is > inappropriate. > > Tested aarch64-none-elf. Reminder that at the moment all the crypto > exte

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-17 Thread pinskia
> On Nov 17, 2014, at 8:59 AM, Ramana Radhakrishnan > wrote: > >> On Mon, Nov 17, 2014 at 2:48 PM, Kyrill Tkachov >> wrote: >> Hi all, >> >> Some configurations of Cortex-A53 and Cortex-A57 don't ship with crypto, >> so enabling it by default for -mcpu=cortex-a53 and cortex-a57 is >> inap

Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-17 Thread Ramana Radhakrishnan
On Mon, Nov 17, 2014 at 2:48 PM, Kyrill Tkachov wrote: > Hi all, > > Some configurations of Cortex-A53 and Cortex-A57 don't ship with crypto, > so enabling it by default for -mcpu=cortex-a53 and cortex-a57 is > inappropriate. > > Tested aarch64-none-elf. Reminder that at the moment all the crypto

[PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57

2014-11-17 Thread Kyrill Tkachov
Hi all, Some configurations of Cortex-A53 and Cortex-A57 don't ship with crypto, so enabling it by default for -mcpu=cortex-a53 and cortex-a57 is inappropriate. Tested aarch64-none-elf. Reminder that at the moment all the crypto extension does is enable the use of the ACLE crypto intrinsics in