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

Improve state emitting for ipers

2004-09-19 Thread 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 off. When we actually see the context get lost, then we "back up"