Re: Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2015-02-11 Thread Jiong Wang
On 20/01/15 13:26, Maxim Kuvyrkov wrote: On Jan 20, 2015, at 1:24 PM, Richard Earnshaw wrote: ... In general, how should someone tuning the compiler for this parameter select a value that isn't one of (-1, m_i_q_d+1)? From my experiments it seems there are 4 reasonable values for the paramet

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2015-01-20 Thread Richard Earnshaw
On 20/01/15 13:26, Maxim Kuvyrkov wrote: > On Jan 20, 2015, at 1:24 PM, Richard Earnshaw wrote: > ... In general, how should someone tuning the compiler for this parameter select a value that isn't one of (-1, m_i_q_d+1)? >>> >>> From my experiments it seems there are 4 reasonable values

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2015-01-20 Thread Maxim Kuvyrkov
On Jan 20, 2015, at 1:24 PM, Richard Earnshaw wrote: ... >>> In general, how should someone tuning the compiler for this parameter >>> select a value that isn't one of (-1, m_i_q_d+1)? >> >> From my experiments it seems there are 4 reasonable values for the >> parameter: (-1) autopref turned off

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2015-01-20 Thread Richard Earnshaw
On 19/01/15 18:14, Maxim Kuvyrkov wrote: > On Jan 19, 2015, at 6:05 PM, Richard Earnshaw wrote: > >> On 16/01/15 15:06, Maxim Kuvyrkov wrote: >>> @@ -1874,7 +1889,8 @@ const struct tune_params arm_cortex_a15_tune = >>> true, true, /* Prefer 32-bit encodings. >

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2015-01-20 Thread Ramana Radhakrishnan
On 19/01/15 18:14, Maxim Kuvyrkov wrote: On Jan 19, 2015, at 6:05 PM, Richard Earnshaw wrote: On 16/01/15 15:06, Maxim Kuvyrkov wrote: @@ -1874,7 +1889,8 @@ const struct tune_params arm_cortex_a15_tune = true, true, /* Prefer 32-bit encodings. */ tru

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2015-01-19 Thread Maxim Kuvyrkov
On Jan 19, 2015, at 6:05 PM, Richard Earnshaw wrote: > On 16/01/15 15:06, Maxim Kuvyrkov wrote: >> @@ -1874,7 +1889,8 @@ const struct tune_params arm_cortex_a15_tune = >> true, true, /* Prefer 32-bit encodings. >> */ >> true,

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2015-01-19 Thread Richard Earnshaw
On 16/01/15 15:06, Maxim Kuvyrkov wrote: > @@ -1874,7 +1889,8 @@ const struct tune_params arm_cortex_a15_tune = >true, true, /* Prefer 32-bit encodings. > */ >true, /* Prefer Neon for > stringops. */ >8,

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2015-01-16 Thread Ramana Radhakrishnan
On Fri, Jan 16, 2015 at 3:06 PM, Maxim Kuvyrkov wrote: > On Nov 19, 2014, at 12:27 PM, Ramana Radhakrishnan > wrote: > >> > > Hi Ramana, > Hi Vladimir, > > I still don't have SPEC2000/SPEC2006 benchmark numbers for this patch. Since > stage3 is about to finish, I'm going to commit the target i

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2015-01-16 Thread Maxim Kuvyrkov
On Nov 19, 2014, at 12:27 PM, Ramana Radhakrishnan wrote: > Hi Ramana, Hi Vladimir, I still don't have SPEC2000/SPEC2006 benchmark numbers for this patch. Since stage3 is about to finish, I'm going to commit the target independent part of the patch now (it was approved by Vladimir a while

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2014-11-19 Thread Maxim Kuvyrkov
On Nov 19, 2014, at 12:27 PM, Ramana Radhakrishnan wrote: > > > On 14/11/14 15:12, Maxim Kuvyrkov wrote: >> On Nov 14, 2014, at 8:38 AM, Jeff Law wrote: >> >>> On 10/20/14 22:06, Maxim Kuvyrkov wrote: Hi, Ramana, this change requires benchmarking, which I can't easily do at >>

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2014-11-19 Thread Ramana Radhakrishnan
On 14/11/14 15:12, Maxim Kuvyrkov wrote: On Nov 14, 2014, at 8:38 AM, Jeff Law wrote: On 10/20/14 22:06, Maxim Kuvyrkov wrote: Hi, Ramana, this change requires benchmarking, which I can't easily do at the moment. I would appreciate any benchmarking results that you can share. In particular

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2014-11-14 Thread Jeff Law
On 11/14/14 08:12, Maxim Kuvyrkov wrote: ... Can this be built on top of Bin's work for insn fusion? There's a lot of commonality in the structure of the insns you care about. He's already got a nice little priority function that I think you could utilize to to ensure the insns with smaller off

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2014-11-14 Thread Maxim Kuvyrkov
On Nov 14, 2014, at 8:38 AM, Jeff Law wrote: > On 10/20/14 22:06, Maxim Kuvyrkov wrote: >> Hi, >> Ramana, this change requires benchmarking, which I can't easily do >> at > the moment. I would appreciate any benchmarking results that you can > share. In particular, the value of PARAM_SCHED_AUTOPR

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2014-11-14 Thread Maxim Kuvyrkov
On Nov 14, 2014, at 4:57 AM, Vladimir Makarov wrote: > On 2014-10-21 12:06 AM, Maxim Kuvyrkov wrote: > ... > I'd prefer symbolic constants for dont_delay. Also the address can contains > other parts, e.g. index for some targets. It is not necessary to change the > code but a comment would be

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2014-11-13 Thread Jeff Law
On 10/20/14 22:06, Maxim Kuvyrkov wrote: Hi, Ramana, this change requires benchmarking, which I can't easily do at the moment. I would appreciate any benchmarking results that you can share. In particular, the value of PARAM_SCHED_AUTOPREF_QUEUE_DEPTH needs to be tuned/confirmed for Cortex-A15.

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2014-11-13 Thread Vladimir Makarov
On 2014-10-21 12:06 AM, Maxim Kuvyrkov wrote: Hi, This patch adds auto-prefetcher modeling to GCC scheduler. The auto-prefetcher model is currently enabled only for ARM Cortex-A15, since this is the only CPU that I know of to have the hardware auto-prefetcher unit. The documentation on the a

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2014-11-10 Thread Maxim Kuvyrkov
On Oct 21, 2014, at 8:06 AM, Maxim Kuvyrkov wrote: > Hi, > > This patch adds auto-prefetcher modeling to GCC scheduler. The > auto-prefetcher model is currently enabled only for ARM Cortex-A15, since > this is the only CPU that I know of to have the hardware auto-prefetcher unit. > > The doc

Re: [PATCH 7/8] Model cache auto-prefetcher in scheduler

2014-10-20 Thread Andrew Pinski
On Mon, Oct 20, 2014 at 9:06 PM, Maxim Kuvyrkov wrote: > Hi, > > This patch adds auto-prefetcher modeling to GCC scheduler. The > auto-prefetcher model is currently enabled only for ARM Cortex-A15, since > this is the only CPU that I know of to have the hardware auto-prefetcher unit. That migh