[Mesa-dev] [PATCH 0/8] i965: dynamic eu instruction store size

Yuanhan Liu yuanhan.liu at linux.intel.com
Thu Dec 22 17:06:45 PST 2011


On Thu, Dec 22, 2011 at 02:37:58PM -0800, Kenneth Graunke wrote:
> On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
> > Hi, this is a new series of patches for dynamic eu instruction store
> > size. The first 4 is from Eric. I just grabed it to make it rebase to
> > current repo. The last 4 patch is from mine which some are based on
> > those patches from Eric.
> > 
> > Please help to review it.
> > 
> > BTW, I checked those patches with all oglc test cases, and found
> > no regression. (Sandybridge only).
> > 
> > Thanks,
> > Yuanhan Liu
> > 
> > 
> > --
> > Eric Anholt (4):
> >   i965: Drop unused do_insn argument from gen6_CONT().
> >   i965: Don't make consumers of brw_DO()/brw_WHILE() track loop start
> >   i965: Don't make consumers of brw_WHILE do pre-gen6 BREAK/CONT
> >     patching
> >   i965: Don't make consumers of brw_CONT/brw_WHILE track if depth in
> >     loop
> > 
> > Yuanhan Liu (4):
> >   i965: let the if_stack just store the instruction index
> >   i965: get the jmp distance by instruction index
> >   i965: call next_insn() before referencing a instruction by index
> 
> 
> Patches 1-7 (v2 of 6 and after changing to bool in 7) are:
> Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

Thanks.

> 
> >   i965: increase the brw eu instruction store size dynamically
> 
> Patch 8 does not get a R-b just yet.
Ok , will fix it.

> 
> Thanks for doing this, Yuanhan, I'm really glad to see the arbitrary
> 10000 limit die.

Welcome and it's my pleasure.

> And Eric, thanks for cleaning up the rest of the
> control flow stack code---it's /so/ much nicer now!


More information about the mesa-dev mailing list