[Mesa-dev] [PATCH] nv50/ir: add missing compute system values for GM107

[Ilia Mirkin]

On Sat, Feb 13, 2016 at 12:32 PM, Samuel Pitoiset
<samuel.pitoiset at gmail.com> wrote:
> I don't see the necessity to add SV_LBASE and SV_SBASE for now
> because they are currently not used and I need to double-check
> the ids first.
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>  src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
> index dee2622..8a01f46 100644
> --- a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
> +++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
> @@ -248,6 +248,11 @@ CodeEmitterGM107::emitSYS(int pos, const Value *val)
>     case SV_INVOCATION_ID  : id = 0x11; break;
>     case SV_THREAD_KILL    : id = 0x13; break;
>     case SV_INVOCATION_INFO: id = 0x1d; break;
> +   case SV_TID            : id = 0x21 + val->reg.data.sv.index; break;
> +   case SV_CTAID          : id = 0x25 + val->reg.data.sv.index; break;
> +   case SV_NTID           : id = 0x29 + val->reg.data.sv.index; break;

Are you sure this works? 0x29+ are documented as something else in envydis.

> +   case SV_GRIDID         : id = 0x2c; break;
> +   case SV_NCTAID         : id = 0x2d + val->reg.data.sv.index; break;

These aren't documented either... please double-check them and add to
envydis if these are correct.

>     default:
>        assert(!"invalid system value");
>        id = 0;
> --
> 2.6.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev