Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-07-10 Thread Daniel Cederman
On 2017-07-07 18:04, Eric Botcazou wrote: Great! Would you mind to apply the patch for us? The only person here with write access just went on vacation. I have submitted a new version (v4) with the change that applies to both main and 7. OK, will do. Thanks! -- Daniel Cederman Cobham

Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-07-07 Thread Eric Botcazou
> Great! Would you mind to apply the patch for us? The only person here > with write access just went on vacation. I have submitted a new version > (v4) with the change that applies to both main and 7. OK, will do. -- Eric Botcazou

Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-07-07 Thread Daniel Cederman
On 2017-07-07 12:01, Eric Botcazou wrote: We can drop the define if necessary, but we would like to keep the two flags. Would that be OK to apply? Yes, OK to apply on mainline and 7 branch with this change, thanks. Great! Would you mind to apply the patch for us? The only person here with

Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-07-07 Thread Eric Botcazou
> We can drop the define if necessary, but we would like to keep the two > flags. Would that be OK to apply? Yes, OK to apply on mainline and 7 branch with this change, thanks. -- Eric Botcazou

Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-07-04 Thread Sebastian Huber
On 04/07/17 15:38, Daniel Cederman wrote: On 2017-06-30 07:11, Sebastian Huber wrote: On 29/06/17 18:05, David Miller wrote: From: Daniel Cederman Date: Thu, 29 Jun 2017 17:15:43 +0200 I'm not thrilled with this, it's undocumented, the other workaround don't have

Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-07-04 Thread Daniel Cederman
On 2017-06-30 07:11, Sebastian Huber wrote: On 29/06/17 18:05, David Miller wrote: From: Daniel Cederman Date: Thu, 29 Jun 2017 17:15:43 +0200 I'm not thrilled with this, it's undocumented, the other workaround don't have it and I don't think that we really need it.

Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-06-29 Thread Sebastian Huber
On 29/06/17 18:05, David Miller wrote: From: Daniel Cederman Date: Thu, 29 Jun 2017 17:15:43 +0200 I'm not thrilled with this, it's undocumented, the other workaround don't have it and I don't think that we really need it. The B2BST errata workaround requires more

Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-06-29 Thread David Miller
From: Daniel Cederman Date: Thu, 29 Jun 2017 17:15:43 +0200 >> I'm not thrilled with this, it's undocumented, the other workaround >> don't have >> it and I don't think that we really need it. > > The B2BST errata workaround requires more changes to assembler > routines

Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-06-29 Thread Daniel Cederman
Hello Eric, Thank you for reviewing the patch. Let's forget -mfix-gr712rc for now, -mfix-ut700 is enough I think. I think it would be confusing to use the -mfix-ut700 flag when compiling for the GR712RC. Now when we are not using a generic name for the errata workaround we should at least

Re: [PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-06-29 Thread Eric Botcazou
> This patch adds a workaround to the Sparc backend for the LEON3FT > store-store errata. It is enabled when using the -mfix-ut699, > -mfix-ut700, or -mfix-gr712rc flag. Let's forget -mfix-gr712rc for now, -mfix-ut700 is enough I think. > The workaround inserts NOP instructions to prevent the

[PATCH-v3] [SPARC] Add a workaround for the LEON3FT store-store errata

2017-06-27 Thread Daniel Cederman
This patch adds a workaround to the Sparc backend for the LEON3FT store-store errata. It is enabled when using the -mfix-ut699, -mfix-ut700, or -mfix-gr712rc flag. The workaround inserts NOP instructions to prevent the following two instruction sequences from being generated: std ->