Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Marathe, Yogesh
> Just some style comments, feel free to ignore them. > Both comments are relevant, will address them in V3. Thanks Lionel. > On 20/07/17 12:35, aravindan.muthuku...@intel.com wrote: > > From: Aravindan Muthukumar > > > > This patch improves CPI Rate(Cycles per Instruction) and branch > > mispr

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Marathe, Yogesh
> -Original Message- > From: Ian Romanick [mailto:i...@freedesktop.org] > Sent: Friday, July 21, 2017 2:24 AM > To: Marathe, Yogesh ; Muthukumar, Aravindan > ; mesa-dev@lists.freedesktop.org > Subject: Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks >

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Lionel Landwerlin
Just some style comments, feel free to ignore them. On 20/07/17 12:35, aravindan.muthuku...@intel.com wrote: From: Aravindan Muthukumar This patch improves CPI Rate(Cycles per Instruction) and branch mispredict for i965. The function check_state() was showing CPI retired rate. Performance sta

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Ian Romanick
mesa- >> d...@lists.freedesktop.org >> Subject: Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks >> >> On 07/20/2017 04:35 AM, aravindan.muthuku...@intel.com wrote: >>> From: Aravindan Muthukumar >>> >>> This patch improves CPI Rate(Cyc

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Marathe, Yogesh
Ian, > -Original Message- > From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf > Of Ian Romanick > Sent: Friday, July 21, 2017 12:33 AM > To: Muthukumar, Aravindan ; mesa- > d...@lists.freedesktop.org > Subject: Re: [Mesa-dev] [PATCH V2] i965

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Marathe, Yogesh
t; >> Sent: Thursday, July 20, 2017 10:51 PM > >> To: Muthukumar, Aravindan ; mesa- > >> d...@lists.freedesktop.org > >> Cc: Muthukumar, Aravindan > >> Subject: Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag > >> checks > >> > &g

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Ian Romanick
: Muthukumar, Aravindan ; mesa- >> d...@lists.freedesktop.org >> Cc: Muthukumar, Aravindan >> Subject: Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks >> >> aravindan.muthuku...@intel.com writes: >> >>> From: Aravindan Muthukumar >>> >>> Th

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Ian Romanick
On 07/20/2017 04:35 AM, aravindan.muthuku...@intel.com wrote: > From: Aravindan Muthukumar > > This patch improves CPI Rate(Cycles per Instruction) > and branch mispredict for i965. The function check_state() > was showing CPI retired rate. > > Performance stats with android: > CPI retired lower

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Francisco Jerez
gt;> d...@lists.freedesktop.org >> Cc: Muthukumar, Aravindan >> Subject: Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks >> >> aravindan.muthuku...@intel.com writes: >> >> > From: Aravindan Muthukumar >> > >> > This

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Marathe, Yogesh
; Subject: Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks > > aravindan.muthuku...@intel.com writes: > > > From: Aravindan Muthukumar > > > > This patch improves CPI Rate(Cycles per Instruction) and branch > > mispredict for i965. Th

Re: [Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread Francisco Jerez
aravindan.muthuku...@intel.com writes: > From: Aravindan Muthukumar > > This patch improves CPI Rate(Cycles per Instruction) > and branch mispredict for i965. The function check_state() > was showing CPI retired rate. > > Performance stats with android: > CPI retired lowered by 28% (lower is bett

[Mesa-dev] [PATCH V2] i965 : Optimize atom state flag checks

2017-07-20 Thread aravindan . muthukumar
From: Aravindan Muthukumar This patch improves CPI Rate(Cycles per Instruction) and branch mispredict for i965. The function check_state() was showing CPI retired rate. Performance stats with android: CPI retired lowered by 28% (lower is better) Branch missprediction lowered by 13% (lower is bet