[Mesa-dev] [Bug 96684] [swrast] piglit glsl-array-bounds-01 regression

https://bugs.freedesktop.org/show_bug.cgi?id=96684 --- Comment #1 from Kenneth Graunke --- I can't reproduce this. I tried both classic swrast and llvmpipe and it seems to be working fine. -- You are receiving this mail because: You are the assignee for the bug. You are

[Mesa-dev] [PATCH RFC 1/1] r600, compute: Use vtx #3 for kernel arguments

Both explicit and implicit. Using vtx 0 (as existing llvm code implies) does not work for dynamic offsets. Signed-off-by: Jan Vesely --- Hi, I ran into problem when using VTX_READ from constant buffer would work only for 0 index. The LLVM code implied that it should

Re: [Mesa-dev] [PATCH] vc4: add hash table look-up for exported dmabufs

Rob Clark writes: > On Sat, Jun 25, 2016 at 11:33 PM, Eric Anholt wrote: >> Rob Herring writes: >> >>> It is necessary to reuse existing BOs when dmabufs are imported. There >>> are 2 cases that need to be handled. dmabufs can be

Re: [Mesa-dev] [PATCH 2/2] gm107/ir: make use of IADD32I for all immediates

On 06/26/2016 11:55 PM, Ilia Mirkin wrote: On Sun, Jun 26, 2016 at 5:49 PM, Samuel Pitoiset wrote: On 06/26/2016 11:46 PM, Ilia Mirkin wrote: We don't appear to handle src(0) having a neg modifier in the immediate case. Does the insnCanLoad logic account for

Re: [Mesa-dev] [PATCH 2/2] gm107/ir: make use of IADD32I for all immediates

On Sun, Jun 26, 2016 at 5:49 PM, Samuel Pitoiset wrote: > > > On 06/26/2016 11:46 PM, Ilia Mirkin wrote: >> >> We don't appear to handle src(0) having a neg modifier in the >> immediate case. Does the insnCanLoad logic account for that? (Perhaps >> the bit is there and

Re: [Mesa-dev] [PATCH 2/2] gm107/ir: make use of IADD32I for all immediates

On 06/26/2016 11:46 PM, Ilia Mirkin wrote: We don't appear to handle src(0) having a neg modifier in the immediate case. Does the insnCanLoad logic account for that? (Perhaps the bit is there and we just forgot about it?) I don't see any neg modifier in envydis for IADD32I. On Sun, Jun

Re: [Mesa-dev] [PATCH 2/2] gm107/ir: make use of IADD32I for all immediates

We don't appear to handle src(0) having a neg modifier in the immediate case. Does the insnCanLoad logic account for that? (Perhaps the bit is there and we just forgot about it?) On Sun, Jun 26, 2016 at 5:40 PM, Samuel Pitoiset wrote: > IADD only allows to emit 19-bits

[Mesa-dev] [PATCH 1/2] gm107/ir: make use of MOV32I for all immediates

MOV only allows to emit 19-bits immediates. This is similar to the previous fix I did for IMUL. Signed-off-by: Samuel Pitoiset Cc: --- src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 3 +-- 1 file changed, 1

[Mesa-dev] [PATCH 2/2] gm107/ir: make use of IADD32I for all immediates

IADD only allows to emit 19-bits immediates. This is similar to the previous fix I did for MOV. Signed-off-by: Samuel Pitoiset Cc: --- src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 2 +- 1 file changed, 1

Re: [Mesa-dev] Fwd: [PATCH] st/vdpau: use bicubic filter for scaling

Nayan Deshmukh wrote: Hi Andy, On Sun, Jun 26, 2016 at 12:25 AM, Andy Furniss wrote: Nayan Deshmukh wrote: Hi Andy, Thanks for testing the patches. Please send me the videos and ratios with which there is corruption.

[Mesa-dev] [PATCH] swrast: fix active attribs with atifragshader

Only include the ones that can be used by the shader. This fixes texture coordinates, which were completely wrong, because WPOS was included in the list of attribs. It also increases performance noticeably. Signed-off-by: Miklós Máté --- src/mesa/swrast/s_context.c | 3 ++- 1

[Mesa-dev] [Bug 96684] [swrast] piglit glsl-array-bounds-01 regression

https://bugs.freedesktop.org/show_bug.cgi?id=96684 Bug ID: 96684 Summary: [swrast] piglit glsl-array-bounds-01 regression Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

Re: [Mesa-dev] [PATCH] vc4: add hash table look-up for exported dmabufs

On Sat, Jun 25, 2016 at 11:33 PM, Eric Anholt wrote: > Rob Herring writes: > >> It is necessary to reuse existing BOs when dmabufs are imported. There >> are 2 cases that need to be handled. dmabufs can be created/exported and >> imported by the same process and

Re: [Mesa-dev] Fwd: [PATCH] st/vdpau: use bicubic filter for scaling

Hi Andy, On Sun, Jun 26, 2016 at 12:25 AM, Andy Furniss wrote: > Nayan Deshmukh wrote: > >> Hi Andy, >> >> Thanks for testing the patches. >> >> Please send me the videos and ratios with which there is corruption. >> > > >

Re: [Mesa-dev] [PATCH] vc4: add hash table look-up for exported dmabufs

Rob Herring writes: > It is necessary to reuse existing BOs when dmabufs are imported. There > are 2 cases that need to be handled. dmabufs can be created/exported and > imported by the same process and can be imported multiple times. > Copying other drivers, add a hash table to

Re: [Mesa-dev] [PATCH] nir: Fix copy_prop_src when src is an indirect access on a reg.

Rb On Jun 25, 2016 9:30 PM, "Eric Anholt" wrote: > The intent was to continue down the indirect chain, not to call ourselves > with unchanged input arguments. Found by code inspection, and comparison > to copy_prop_alu_src(). > > We haven't hit this because callers of NIR's

Re: [Mesa-dev] [PATCH] gm107/ir: make use of IMUL32I for all immediates

Reviewed-by: Ilia Mirkin Might be good to make a note of why, and/or reference the GK110 commit that did the same thing. On Sun, Jun 26, 2016 at 11:23 AM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > Cc:

[Mesa-dev] [PATCH] gm107/ir: make use of IMUL32I for all immediates

Signed-off-by: Samuel Pitoiset Cc: --- src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp

Re: [Mesa-dev] V3 On disk shader cache for i965 (Now with real world results!)

On Sun, 2016-06-26 at 16:15 +0300, Grazvydas Ignotas wrote: > Tried this while playing with apitrace and am getting segfaults when > running any trace with a cached (second) run. Not sure if it's > "wrong" > traces I've chosen or what, you can take one example from this bug: >

Re: [Mesa-dev] [PATCH] nir: Fix copy_prop_src when src is an indirect access on a reg.

On Sat, Jun 25, 2016 at 8:54 PM, Eric Anholt wrote: > The intent was to continue down the indirect chain, not to call ourselves > with unchanged input arguments. Found by code inspection, and comparison > to copy_prop_alu_src(). > > We haven't hit this because callers of NIR's

Re: [Mesa-dev] V3 On disk shader cache for i965 (Now with real world results!)

Tried this while playing with apitrace and am getting segfaults when running any trace with a cached (second) run. Not sure if it's "wrong" traces I've chosen or what, you can take one example from this bug: https://bugs.freedesktop.org/show_bug.cgi?id=96425 It would also be good idea to hide the

[Mesa-dev] [PATCH 2/2] i965: Print EOT in fs_visitor::dump_instruction().

This was useful when debugging the previous commit's issue. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs.cpp | 4 1 file changed, 4 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index

[Mesa-dev] [PATCH 1/2] i965: Make emit_urb_writes() not produce an EOT message for GS.

emit_urb_writes() contains code to emit an EOT write with no actual data when there are no output varyings. This makes sense for the VS and TES stages, where it's called once at the end of the program. However, in the geometry shader stage, emit_urb_writes() is called once for every

Re: [Mesa-dev] [PATCH] st/va: Check NULL pointer

Thx for the patch: Reviewed-by: Julien Isorce I'll push it today. On 26 June 2016 at 08:02, Gurkirpal Singh wrote: > Call to handle_table_get in vlVaDestroySurfaces can > return NULL on failure. > > CID: 1243522 > > Signed-off-by: Gurkirpal Singh

[Mesa-dev] [PATCH] st/va: Check NULL pointer

Call to handle_table_get in vlVaDestroySurfaces can return NULL on failure. CID: 1243522 Signed-off-by: Gurkirpal Singh --- src/gallium/state_trackers/va/surface.c | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/state_trackers/va/surface.c