Re: arm: Only load TLS values when needed

2013-08-26 Thread André Hentschel
Am 15.08.2013 20:27, schrieb André Hentschel: > Hi Jonathan, > > Am 15.08.2013 19:29, schrieb Jonathan Austin: >> Hi André >> What I've *not* tested: >> * As this CPU does not have the tls register, I haven't run any tests that >> try to read/write it, but we wouldn't expect that to work, right?

Re: arm: Only load TLS values when needed

2013-08-15 Thread André Hentschel
Hi Jonathan, Am 15.08.2013 19:29, schrieb Jonathan Austin: > Hi André > > (I've put RMK and LAKML back on Cc: they got dropped somewhere along the way) > > On 14/08/13 22:21, André Hentschel wrote: >> [...] Now we've only got one instruction between the store and the load a

Re: arm: Only load TLS values when needed

2013-08-15 Thread Jonathan Austin
Hi André (I've put RMK and LAKML back on Cc: they got dropped somewhere along the way) On 14/08/13 22:21, André Hentschel wrote: [...] Now we've only got one instruction between the store and the load and risk stalling the pipeline... Dave M cautiously says "The ancient advice was that one

Re: arm: Only load TLS values when needed

2013-08-14 Thread André Hentschel
Am 14.08.2013 18:20, schrieb Jonathan Austin: > Hi André, > > On 14/08/13 15:07, André Hentschel wrote: >> Hi Jonathan, >> Any updates on this? >> > > I was holding out to see the version with ldrd at the top, as discussed below > - I never saw a version with that change? I'd meant to ping you t

Re: arm: Only load TLS values when needed

2013-08-14 Thread Jonathan Austin
Hi André, On 14/08/13 15:07, André Hentschel wrote: Hi Jonathan, Any updates on this? I was holding out to see the version with ldrd at the top, as discussed below - I never saw a version with that change? I'd meant to ping you to see if that was coming, sorry. [...] [...] Now we've on

Re: arm: Only load TLS values when needed

2013-08-14 Thread André Hentschel
Hi Jonathan, Any updates on this? Am 17.07.2013 21:49, schrieb André Hentschel: > Am 17.07.2013 13:10, schrieb Jonathan Austin: >> Hi André, >> Do you have access to anything v6-NOT-k-ish? If not I can try and test this >> on something appropriate. How does your test-case access tpidrurw? If it

Re: arm: Only load TLS values when needed

2013-07-17 Thread André Hentschel
Am 17.07.2013 13:10, schrieb Jonathan Austin: > Hi André, > > On 16/07/13 20:27, André Hentschel wrote: >> Hi Jonathan, First, thank you for your review. >> >> Am 16.07.2013 19:31, schrieb Jonathan Austin: >>> Hi André, >>> >>> On 15/07/13 18:14, André Hentschel wrote: From: André Hentschel

Re: arm: Only load TLS values when needed

2013-07-17 Thread Jonathan Austin
Hi André, On 16/07/13 20:27, André Hentschel wrote: Hi Jonathan, First, thank you for your review. Am 16.07.2013 19:31, schrieb Jonathan Austin: Hi André, On 15/07/13 18:14, André Hentschel wrote: From: André Hentschel This patch intents to reduce loading instructions when the resulting va

Re: arm: Only load TLS values when needed

2013-07-16 Thread André Hentschel
Hi Jonathan, First, thank you for your review. Am 16.07.2013 19:31, schrieb Jonathan Austin: > Hi André, > > On 15/07/13 18:14, André Hentschel wrote: >> From: André Hentschel >> >> This patch intents to reduce loading instructions when the resulting value >> is not used. >> It's a follow up on

Re: arm: Only load TLS values when needed

2013-07-16 Thread Jonathan Austin
Hi André, On 15/07/13 18:14, André Hentschel wrote: From: André Hentschel This patch intents to reduce loading instructions when the resulting value is not used. It's a follow up on a4780adeefd042482f624f5e0d577bf9cdcbb760 Have you done any benchmarking to see that this has any real impact

arm: Only load TLS values when needed

2013-07-15 Thread André Hentschel
From: André Hentschel This patch intents to reduce loading instructions when the resulting value is not used. It's a follow up on a4780adeefd042482f624f5e0d577bf9cdcbb760 Signed-off-by: André Hentschel --- This patch is against Linux 3.11-rc1 (ad81f0545ef01ea651886dddac4bef6cec930092) Thanks