[Nouveau] "enable ctxprog xfer only when we need it to save power" introduces big performance regression

[Ben Skeggs]

On Sat, 2011-10-29 at 20:00 +0200, Maarten Maathuis wrote:
> On Sat, Oct 29, 2011 at 7:29 PM, Marcin Slusarz
> <marcin.slusarz at gmail.com> wrote:
> > On Sat, Oct 29, 2011 at 07:08:01PM +0200, Marcin Slusarz wrote:
> >> Hi
> >>
> >> I've bisected pretty big performance regression (nv92):
> >>
> >> $ git bisect good
> >> b2737681d5442f05ab6419e05468c3d2511a5ced is the first bad commit
> >> commit b2737681d5442f05ab6419e05468c3d2511a5ced
> >> Author: Martin Peres <martin.peres at ensi-bourges.fr>
> >> Date:   Sat Jul 30 23:08:45 2011 +0200
> >>
> >>     drm/nv50/gr: enable ctxprog xfer only when we need it to save power
> >
> > Weird, reverting it on top of current git does not restore it - in
> > glxgears it's even worse (-~50%).
> >
> > Marcin
> > _______________________________________________
> > Nouveau mailing list
> > Nouveau at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/nouveau
> >
> 
> Before this patch, what ensured that context switching was enabled?
It was enabled by default during graph init, this was removed with the
"drm/nv50/gr: refactor initialisation" commit.

Ben.
>