On Sat, 31 Jul 2010 20:32:29 +0200, Marek Olšák <mar...@gmail.com> wrote:
> Also program_parse.tab.c has been regenerated.
> 
> This fixes the parser error:
> 
>   ARB_vp: error: relative address offset too large
> 
> See also: https://bugs.freedesktop.org/show_bug.cgi?id=28628
> 
> 4096 * sizeof(vec4) is the maximum size of the constant buffer on NV50,
> so it is a reasonable limit, at least for now.
> (should there be any limit at all?)
> 
> Piglit: vp-arl-constant-array-huge-relative-offset

The limit comes from:

    (26) What limits should be imposed on the constants that can be added to
    or subtracted from the address register for relative addressing?  Negative
    offsets are sometimes useful for shifting down in an array.

      RESOLVED:  -64 to +63 should be sufficient for the time being.  Offset
      sizes are limited to allow offsets to be baked into device-dependent
      instruction encodings.

so wine is really being nonportable here and should handle it itself,
but I'm fine with just removing the limits in the core and letting the
drivers complain if they can't be supported.  4096 doesn't make sense as
a limit, as constant buffer size limits should already be expressed
through MAX_PROGRAM_PARAMETERS_ARB.

Attachment: pgpQulwb3DgAT.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to