Ilia,

On Friday, August 07, 2015 11:20:42 Ilia Mirkin wrote:
> Yeah, I tend to agree with you -- it seems like _tnl_wakeup is meant for
> "oh hey, I shut you down and haven't been sending you updates, but I want
> to use you again, so please update your stuff to match reality". Does it
> even need to be called at all? nouveau_state.c:nouveau_update_state
> unconditionally calls _tnl_InvalidateState. i965, on the other hand, the
> only other user of tnl_wakeup, calls it at draw time, for RenderMode !=
> GL_RENDER, and never calls _tnl_InvalidateState, so the tnl module is
> completely out-of-date there.
>
> So I'm actually in favor of removing it entirely. Thoughts?
You mean remove _tnl_wakeup/_tnl_InvalidateState entirely from the
nouveau initialzation? That may work - I have the same
impression, but I cannot exactly foresee what we might break.

Greetings

Mathias
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to