On Sun, 17 Jul 2011 16:25:42 -0700, Ben Widawsky <ben@bwidawsk.net> wrote:
> Upload the system routine as part of the invariant state if debugging.
> 
> Remove SIP setting if not debugging to make it more friendly for others
> that may be debugging shaders or media kernels.
> 
> v2: removed comment per Chris

This patch doesn't really make sense to me.  Nothing good will come of
my driver's buffer execution landing inside someone else's debug kernel,
right?  So why should my driver's batchbuffer be leaving their SIP in
place?

Of course, if we're landing in a SIP at all from our !brw->wm.debugging
batchbuffers, we're screwed if we don't have actual SIP instructions in
place, so I guess this doesn't *really* change anything.