Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-14 Thread Courtney Goeltzenleuchter
On Mon, Oct 14, 2013 at 12:43 PM, Chad Versace wrote: > On 10/14/2013 10:54 AM, Eric Anholt wrote: > >> Courtney Goeltzenleuchter writes: >> >> Does anyone know of a test that measures frame 0 time? Or texture upload >>> speed? >>> >>> For Smokin' Guns, I tried measuring the overall time, but a

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-14 Thread Chad Versace
On 10/14/2013 10:54 AM, Eric Anholt wrote: Courtney Goeltzenleuchter writes: Does anyone know of a test that measures frame 0 time? Or texture upload speed? For Smokin' Guns, I tried measuring the overall time, but an improved frame 0 time has difficulty standing out of a 2607 frame test. I

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-14 Thread Eric Anholt
Courtney Goeltzenleuchter writes: > Does anyone know of a test that measures frame 0 time? Or texture upload > speed? > > For Smokin' Guns, I tried measuring the overall time, but an improved frame > 0 time has difficulty standing out of a 2607 frame test. > > I may have to create something. Sugg

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-14 Thread Eric Anholt
Courtney Goeltzenleuchter writes: > Does anyone know of a test that measures frame 0 time? Or texture upload > speed? > > For Smokin' Guns, I tried measuring the overall time, but an improved frame > 0 time has difficulty standing out of a 2607 frame test. > > I may have to create something. Sugg

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-14 Thread Mark Mueller
Hi Courtney, I've been doing similar work but using Xenotic as the benchmark. Here is how I've been estimating upload times: First I made a rough determination of a texture count, like 300, as a metric. In intelTexImage I use clock_gettime to determine the elapsed time between the loading of textur

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-14 Thread Courtney Goeltzenleuchter
Does anyone know of a test that measures frame 0 time? Or texture upload speed? For Smokin' Guns, I tried measuring the overall time, but an improved frame 0 time has difficulty standing out of a 2607 frame test. I may have to create something. Suggestions for an appropriate framework? Thanks,

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-14 Thread Chad Versace
On 10/13/2013 08:33 PM, Ian Romanick wrote: On 10/13/2013 01:50 PM, Frank Henigman wrote: On Fri, Oct 11, 2013 at 10:00 PM, Chad Versace wrote: On 10/11/2013 10:17 AM, Courtney Goeltzenleuchter wrote: Support all levels of a supported texture format. --- src/mesa/drivers/dri/i965/intel_te

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-13 Thread Ian Romanick
On 10/13/2013 01:50 PM, Frank Henigman wrote: > On Fri, Oct 11, 2013 at 10:00 PM, Chad Versace > wrote: >> On 10/11/2013 10:17 AM, Courtney Goeltzenleuchter wrote: >>> >>> Support all levels of a supported texture format. >>> --- >>> src/mesa/drivers/dri/i965/intel_tex_subimage.c | 13 ++

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-13 Thread Courtney Goeltzenleuchter
On Sun, Oct 13, 2013 at 4:39 PM, Courtney Goeltzenleuchter < court...@lunarg.com> wrote: > > On Sun, Oct 13, 2013 at 2:50 PM, Frank Henigman wrote: > >> On Fri, Oct 11, 2013 at 10:00 PM, Chad Versace >> wrote: >> > On 10/11/2013 10:17 AM, Courtney Goeltzenleuchter wrote: >> >> >> >> Support all l

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-13 Thread Courtney Goeltzenleuchter
On Sun, Oct 13, 2013 at 2:50 PM, Frank Henigman wrote: > On Fri, Oct 11, 2013 at 10:00 PM, Chad Versace > wrote: > > On 10/11/2013 10:17 AM, Courtney Goeltzenleuchter wrote: > >> > >> Support all levels of a supported texture format. > >> --- > >> src/mesa/drivers/dri/i965/intel_tex_subimage.c

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-13 Thread Ian Romanick
Same question about tests here as on the XRGB patch. On 10/11/2013 10:17 AM, Courtney Goeltzenleuchter wrote: > Support all levels of a supported texture format. > --- > src/mesa/drivers/dri/i965/intel_tex_subimage.c | 13 +++-- > 1 file changed, 11 insertions(+), 2 deletions(-) > > diff

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-11 Thread Chad Versace
On 10/11/2013 10:17 AM, Courtney Goeltzenleuchter wrote: Support all levels of a supported texture format. --- src/mesa/drivers/dri/i965/intel_tex_subimage.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_tex_subimage.c b/src

Re: [Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-11 Thread Kenneth Graunke
On 10/11/2013 10:17 AM, Courtney Goeltzenleuchter wrote: > Support all levels of a supported texture format. > --- > src/mesa/drivers/dri/i965/intel_tex_subimage.c | 13 +++-- > 1 file changed, 11 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/intel_tex_subimage.

[Mesa-dev] [PATCH] i965: Fast texture upload now supports all levels

2013-10-11 Thread Courtney Goeltzenleuchter
Support all levels of a supported texture format. --- src/mesa/drivers/dri/i965/intel_tex_subimage.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_tex_subimage.c b/src/mesa/drivers/dri/i965/intel_tex_subimage.c index 4aec05d..5e