On 12/22/2011 07:04 PM, Yuanhan Liu wrote: > On Thu, Dec 22, 2011 at 02:33:03PM -0800, Kenneth Graunke wrote: >> On 12/21/2011 01:33 AM, Yuanhan Liu wrote: [snip] >>> -#define BRW_EU_MAX_INSN_STACK 5 >>> -#define BRW_EU_MAX_INSN 10000 >>> +#define BRW_EU_MAX_INSN_STACK 5 >>> +#define BRW_EU_MAX_INSN (1024 * 1024) >> >> I'm actually surprised to see BRW_EU_MAX_INSN at all. As far as I know, >> there isn't an actual hardware limit on the number of instructions, > > Glad to know that. Thanks. > >> so >> I'm not sure why we should cap it at all. Especially not to some >> arbitrary number. (I'm assuming that 1024 * 1024 is just something you >> came up with arbitrarily...) > > Aha, yes, you are right, I made it. :) > > Here is the fixed patch, please help to review it:
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> I'd wait for an ack from Eric before pushing, though. Thanks again! _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
