Re: Improve state emitting for ipers

2004-09-21 Thread Eric Anholt
On Tue, 2004-09-21 at 22:42, Eric Anholt wrote: > On Tue, 2004-09-21 at 10:41, Dieter Nützel wrote: > > Am Montag, 20. September 2004 23:14 schrieb Eric Anholt: > > > On Mon, 2004-09-20 at 12:58, Dieter Nützel wrote: > > > > But got > > > > > > > > progs/demos> IperS V1.0 > > > > Written by David B

Re: Improve state emitting for ipers

2004-09-21 Thread Eric Anholt
On Tue, 2004-09-21 at 10:41, Dieter Nützel wrote: > Am Montag, 20. September 2004 23:14 schrieb Eric Anholt: > > On Mon, 2004-09-20 at 12:58, Dieter Nützel wrote: > > > Am Montag, 20. September 2004 21:52 schrieb Dieter Nützel: > > > > Am Sonntag, 19. September 2004 11:21 schrieb Eric Anholt: > > >

Re: Improve state emitting for ipers

2004-09-21 Thread Roland Scheidegger
Eric Anholt wrote: Yeah, it was clear that we used to emit in whatever order, and that would have been nicer. At this point I'm seeing about 5% CPU time in EmitState for ipers, which seems pretty hefty for such a small bit of code, but I didn't see much obvious for improvement. That's indeed qu

Re: Improve state emitting for ipers

2004-09-21 Thread Dieter Nützel
Am Montag, 20. September 2004 22:13 schrieb Ian Romanick: > Eric Anholt wrote: > > This gets about a 5% speedup for me in ipers (which I wish was more > > accurate in its reporting), and doesn't touch glxgears. I didn't have > > any interesting apps besides glxgears handy to benchmark with. > > I

Re: Improve state emitting for ipers

2004-09-21 Thread Stephane Marchesin
Dieter Nützel wrote: This gets about a 5% speedup for me in ipers (which I wish was more accurate in its reporting), and doesn't touch glxgears. I didn't have any interesting apps besides glxgears handy to benchmark with. Any thoughts on this? If people think it's a good idea, I'll do it for rad

Re: Improve state emitting for ipers

2004-09-21 Thread Dieter Nützel
Am Dienstag, 21. September 2004 00:00 schrieb Roland Scheidegger: > Eric Anholt wrote: > > The attached patch removes the mandatory emits of all state which were > > happening after each cmdbuf flush. Instead, we set a flag after a > > cmdbuf flush saying "save the state at the next unlock," which

Re: Improve state emitting for ipers

2004-09-21 Thread Dieter Nützel
Am Montag, 20. September 2004 23:14 schrieb Eric Anholt: > On Mon, 2004-09-20 at 12:58, Dieter NÃtzel wrote: > > Am Montag, 20. September 2004 21:52 schrieb Dieter NÃtzel: > > > Am Sonntag, 19. September 2004 11:21 schrieb Eric Anholt: > > > > The attached patch removes the mandatory emits of all s

Re: Improve state emitting for ipers

2004-09-20 Thread Eric Anholt
On Mon, 2004-09-20 at 15:00, Roland Scheidegger wrote: > Eric Anholt wrote: > > The attached patch removes the mandatory emits of all state which were > > happening after each cmdbuf flush. Instead, we set a flag after a > > cmdbuf flush saying "save the state at the next unlock," which means > >

Re: Improve state emitting for ipers

2004-09-20 Thread Roland Scheidegger
Eric Anholt wrote: The attached patch removes the mandatory emits of all state which were happening after each cmdbuf flush. Instead, we set a flag after a cmdbuf flush saying "save the state at the next unlock," which means memcpying the state atoms off. When we actually see the context get lost

Re: Improve state emitting for ipers

2004-09-20 Thread Ian Romanick
Eric Anholt wrote: This gets about a 5% speedup for me in ipers (which I wish was more accurate in its reporting), and doesn't touch glxgears. I didn't have any interesting apps besides glxgears handy to benchmark with. I should be able to give it a spin on viewperf sometime this week... -

Re: Improve state emitting for ipers

2004-09-20 Thread Dieter Nützel
Am Montag, 20. September 2004 21:52 schrieb Dieter Nützel: > Am Sonntag, 19. September 2004 11:21 schrieb Eric Anholt: > > The attached patch removes the mandatory emits of all state which were > > happening after each cmdbuf flush. Instead, we set a flag after a > > cmdbuf flush saying "save the

Re: Improve state emitting for ipers

2004-09-20 Thread Dieter Nützel
Am Sonntag, 19. September 2004 11:21 schrieb Eric Anholt: > The attached patch removes the mandatory emits of all state which were > happening after each cmdbuf flush. Instead, we set a flag after a > cmdbuf flush saying "save the state at the next unlock," which means > memcpying the state atoms

Re: Improve state emitting for ipers

2004-09-20 Thread Keith Whitwell
Eric Anholt wrote: The attached patch removes the mandatory emits of all state which were happening after each cmdbuf flush. Instead, we set a flag after a cmdbuf flush saying "save the state at the next unlock," which means memcpying the state atoms off. When we actually see the context get lost