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

[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 enabled ARB_half_float_pixel and
ARB/APPLE_vertex_array_object.


Marek

On Sun, Apr 17, 2011 at 7:08 PM, Jerome Glisse <j.glisse at gmail.com> wrote:

> On Sun, Apr 17, 2011 at 7:59 AM, Nicolas Kaiser <nikai at nikai.net> wrote:
> > * Nicolas Kaiser <nikai at nikai.net>:
> >> * Marek Olšák <maraeo at gmail.com>:
> >> > 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.c, where it's listed
> >> in the card_extensions as well.
> >>
> >> I'll fix the patch.
> >
> > Well, when I move it to the ARB_vp_extension array,
> > it doesn't work:
> >
> > .src/tests/prog_parameter
> > GL_RENDERER = Mesa DRI R200 (R200 514D)  TCL DRI2
> > GL_VERSION = 1.3 Mesa 7.11-devel (git-9dacbe2)
> >
> > GL_EXT_gpu_program_parameters not available.
> > Testing glProgramEnvParameter4fvARB (count = 256)...
> > Testing glProgramLocalParameter4fvARB (count = 1024)...
> > PASS!
> >
> > I guess I'm doing something wrong.
> >
> > Best regards,
> > Nicolas Kaiser
> >
>
> You missing the point this extension is useless on r200, because r200
> hardware doesn't the capabilities to use this extension, the test you
> are using is just querying info not doing anythings usefull. Sure we
> can report wrong informations but what's the point ?
>
> Cheers,
> Jerome
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20110418/e64a0721/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-r200-enable-some-extensions.patch
Type: text/x-diff
Size: 3658 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20110418/e64a0721/attachment.patch>