Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-19 Thread Marek Olšák
OK, thanks for testing. I have pushed the patch. Marek On Tue, Apr 19, 2011 at 11:54 AM, Nicolas Kaiser wrote: > * Ian Romanick : > > > ./src/demos/vao_demo > > > GL_RENDERER = Mesa DRI R200 (R200 514D) TCL DRI2 > > > GL_VERSION = 1.3 Mesa 7.11-devel (git-4ee40c3) > > > > Do we have VAO (eithe

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-19 Thread Nicolas Kaiser
* Ian Romanick : > > ./src/demos/vao_demo > > GL_RENDERER = Mesa DRI R200 (R200 514D) TCL DRI2 > > GL_VERSION = 1.3 Mesa 7.11-devel (git-4ee40c3) > > Do we have VAO (either ARB or APPLE) in piglit? That would actually be > a more interesting test. The Mesa demos exercise extensions pretty >

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-18 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/17/2011 01:28 AM, Nicolas Kaiser wrote: > * Marek Olšák : >> r200 does support ARB_vertex_program. The backend is in r200_vertprog.c. I >> think it has vertex shader 1.1 in D3D. >> >> The patch is wrong though. The extension should be listed in t

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-18 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/16/2011 03:01 AM, Nicolas Kaiser wrote: > Signed-off-by: Nicolas Kaiser > --- > According to Marek Olšák, this extension is quite popular in > the GL community.[1] Yeah, and it's a very trivial extension to support, too. I actually think that

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-18 Thread Roland Scheidegger
Am 18.04.2011 00:29, schrieb Marek Olšák: > Jerome, > > Nicolas is actually right. There is absolutely nothing driver-specific > about the extension. It's a pure API improvement and doesn't have > /anything/ to do with the hardware. There are no capabilities required, > just ARB_vertex_program. Y

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-18 Thread Nicolas Kaiser
* Marek Olšák : > The extension is driver-independent, it's implemented in the Mesa core, and > doesn't need any driver or hardware parts. It's not much useful without > ARB_texture_float though, which the hardware cannot do. > draw-vertices-half-float tests ARB_half_float_vertex only. > ARB_half_

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-18 Thread Marek Olšák
On Mon, Apr 18, 2011 at 1:06 PM, Nicolas Kaiser wrote: > - GL_ARB_half_float_pixel: > > I'm having problems with this one. > > (1a) According to the spec this extension, quote > "will work with the OpenGL 1.5 Specification." > However, glxinfo says it is at 1.3: > > OpenGL vendor string: Tungs

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-18 Thread Nicolas Kaiser
* Marek Olšák : > Please test the attached patch. I have also enabled ARB_half_float_pixel and > ARB/APPLE_vertex_array_object. - GL_EXT_gpu_program_parameters: ./src/tests/prog_parameter GL_RENDERER = Mesa DRI R200 (R200 514D) TCL DRI2 GL_VERSION = 1.3 Mesa 7.11-devel (git-699ecec) GL_EXT_gpu_

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-17 Thread Nicolas Kaiser
* Marek Olšák : > Please test the attached patch. I have also enabled ARB_half_float_pixel and > ARB/APPLE_vertex_array_object. Thanks a lot! I'll do so. > @@ -455,7 +463,7 @@ GLboolean r200CreateContext( gl_api api, > driInitExtensions( ctx, blend_extensions, GL_FALSE ); > } > if

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-17 Thread Marek Olšák
Jerome, Nicolas is actually right. There is absolutely nothing driver-specific about the extension. It's a pure API improvement and doesn't have *anything* to do with the hardware. There are no capabilities required, just ARB_vertex_program. Nicolas, Please test the attached patch. I have also

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-17 Thread Jerome Glisse
On Sun, Apr 17, 2011 at 7:59 AM, Nicolas Kaiser wrote: > * Nicolas Kaiser : >> * Marek Olšák : >> > r200 does support ARB_vertex_program. The backend is in r200_vertprog.c. I >> > think it has vertex shader 1.1 in D3D. >> > >> > The patch is wrong though. The extension should be listed in the >> >

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-17 Thread Nicolas Kaiser
* Nicolas Kaiser : > * Marek Olšák : > > r200 does support ARB_vertex_program. The backend is in r200_vertprog.c. I > > think it has vertex shader 1.1 in D3D. > > > > The patch is wrong though. The extension should be listed in the > > ARB_vp_extension array, not card_extensions. > > Thanks! I

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-17 Thread Nicolas Kaiser
* Marek Olšák : > r200 does support ARB_vertex_program. The backend is in r200_vertprog.c. I > think it has vertex shader 1.1 in D3D. > > The patch is wrong though. The extension should be listed in the > ARB_vp_extension array, not card_extensions. Thanks! I copied it from dri/r300/r300_context.

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-16 Thread Marek Olšák
r200 does support ARB_vertex_program. The backend is in r200_vertprog.c. I think it has vertex shader 1.1 in D3D. The patch is wrong though. The extension should be listed in the ARB_vp_extension array, not card_extensions. Marek On Sun, Apr 17, 2011 at 4:57 AM, Jerome Glisse wrote: > On Sat,

Re: [Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-16 Thread Jerome Glisse
On Sat, Apr 16, 2011 at 6:01 AM, Nicolas Kaiser wrote: > Signed-off-by: Nicolas Kaiser > --- > According to Marek Olšák, this extension is quite popular in > the GL community.[1] > > Tests look good: > ./src/tests/prog_parameter > GL_RENDERER = Mesa DRI R200 (R200 514D)  TCL DRI2 > GL_VERSION = 1

[Mesa-dev] [PATCH] r200: enable EXT_gpu_program_parameters

2011-04-16 Thread Nicolas Kaiser
Signed-off-by: Nicolas Kaiser --- According to Marek Olšák, this extension is quite popular in the GL community.[1] Tests look good: ./src/tests/prog_parameter GL_RENDERER = Mesa DRI R200 (R200 514D) TCL DRI2 GL_VERSION = 1.3 Mesa 7.11-devel (git-4ee40c3) GL_EXT_gpu_program_parameters availabl