Am 16.02.2019 03:56, schrieb Timothy Arceri:
On 13/2/19 8:28 am, Dieter Nützel wrote:
Hello Marek, Timo, Nicolai,
Timo SOLVED this long-standing NIR corruption on Polaris with his
'nir: rewrite varying component packing' commit.
It was triggered with
commit 86b52d42368ac496fe24bc6674e754c32
https://bugs.freedesktop.org/show_bug.cgi?id=109532
--- Comment #32 from andrii simiklit ---
(In reply to andrii simiklit from comment #31)
> (In reply to Mark Janes from comment #30)
> > (In reply to Mark Janes from comment #28)
> > >
> > > https://android-review.googlesource.com/c/platform/e
On Tue, Feb 12, 2019 at 11:53 AM Iago Toral Quiroga
wrote:
> ---
> src/intel/compiler/brw_eu_validate.c| 64 -
> src/intel/compiler/test_eu_validate.cpp | 122
> 2 files changed, 185 insertions(+), 1 deletion(-)
>
> diff --git a/src/intel/compiler/brw_eu
Reviewed-by: Jason Ekstrand
On Tue, Feb 12, 2019 at 5:57 AM Iago Toral Quiroga
wrote:
> NIR already has these and correctly considers exact/inexact qualification,
> whereas the backend doesn't and can apply the optimizations where it
> shouldn't. This happened to be the case in a handful of Tom
On Thursday, February 14, 2019 4:05:00 AM PST Chris Wilson wrote:
> If we hang the GPU and end up banning our context, we will no longer be
> able to submit and abort with an error (exit(1) no less). As we submit
> minimal incremental batches that rely on the logical context state of
> previous bat
Object handles are local to the device fd, so double check we are not
mixing together objects from multiple screens on execbuf submission.
Cc: Kenneth Graunke
---
src/mesa/drivers/dri/i965/intel_batchbuffer.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/intel_b
Reviewed-by: Jason Ekstrand
On Tue, Feb 12, 2019 at 5:56 AM Iago Toral Quiroga
wrote:
> It is very likely that this optimzation is never useful and we'll probably
> just end up removing it, so let's not bother adding more cases to it for
> now.
> ---
> src/intel/compiler/brw_fs.cpp | 4
>
I believe I've now reviewed everything except some of the validator patches
and the is_partial_write() patch. The validator patches I'm hoping Matt or
Curro can look at. For the is_partial_write() patch, I just need to
convince myself that it doesn't make the compiler significantly more bogus
tha
On Tue, Feb 12, 2019 at 5:57 AM Iago Toral Quiroga
wrote:
> At the very least we need it to handle HF too, since we are doing
> constant propagation for MAD and LRP, which relies on this pass
> to promote the immediates to GRF in the end, but ideally
> we want it to support even more types so we
Reviewed-by: Jason Ekstrand
On Tue, Feb 12, 2019 at 11:51 AM Iago Toral Quiroga
wrote:
> This is set to True only for numeric conversion opcodes.
> ---
> src/compiler/nir/nir.h| 3 ++
> src/compiler/nir/nir_opcodes.py | 73 +--
> src/compiler/nir/nir_
Reviewed-by: Jason Ekstrand
On Tue, Feb 12, 2019 at 5:57 AM Iago Toral Quiroga
wrote:
> This is available since gen8.
>
> v2: restore previously existing assertion.
>
> v3: don't use separate tables for gen7 and gen8, just assert that we
> don't use half-float before gen8 (Matt)
>
> Reviewe
Acked-by: Jason Ekstrand
On Tue, Feb 12, 2019 at 11:52 AM Iago Toral Quiroga
wrote:
> The original SrcType is a 3-bit field that takes a subset of the types
> supported for the hardware for 3-source instructions. Since gen8,
> when the half-float type was added, 3-source floating point operatio
If we hang the GPU and end up banning our context, we will no longer be
able to submit and abort with an error (exit(1) no less). As we submit
minimal incremental batches that rely on the logical context state of
previous batches, we can not rely on the kernel's recovery mechanism
which tries to re
On Tue, Feb 12, 2019 at 5:56 AM Iago Toral Quiroga
wrote:
> ---
> src/intel/compiler/brw_eu_validate.c| 10 +-
> src/intel/compiler/test_eu_validate.cpp | 46 +
> 2 files changed, 55 insertions(+), 1 deletion(-)
>
> diff --git a/src/intel/compiler/brw_eu_validate.
If we hang the GPU and end up banning our context, we will no longer be
able to submit and abort with an error (exit(1) no less). As we submit
minimal incremental batches that rely on the logical context state of
previous batches, we can not rely on the kernel's recovery mechanism
which tries to re
Reviewed-by: Jason Ekstrand
On Tue, Feb 12, 2019 at 5:56 AM Iago Toral Quiroga
wrote:
> So it is right after the checks for the other various Int* capabilities.
> ---
> src/compiler/spirv/spirv_to_nir.c | 7 +++
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/src/compile
Reviewed-by: Jason Ekstrand
On Tue, Feb 12, 2019 at 5:57 AM Iago Toral Quiroga
wrote:
> Now that we have the regioning lowering pass we can just put all of these
> opcodes together in a single block and we can just assert on the few cases
> of conversion instructions that are not supported in h
Matt, Curro,
Could one of you please take a look at this and the other validator patches
in this series? Region restrictions aren't my strongest area.
On Tue, Feb 12, 2019 at 5:56 AM Iago Toral Quiroga
wrote:
> The section 'Execution Data Types' of 3D Media GPGPU volume, which
> describes exec
On Tue, Feb 12, 2019 at 5:56 AM Iago Toral Quiroga
wrote:
> Empirical testing shows that gen8 has a bug where MAD instructions with
> a half-float source starting at a non-zero offset fail to execute
> properly.
>
> This scenario usually happened in SIMD8 executions, where we used to
> pack vecto
Introduce a new debug option to wilfully cause the GPU to hang and for
the kernel to accuse of being neglectful.
---
src/intel/Makefile.sources| 2 +
src/intel/common/gen_debug.c | 1 +
src/intel/common/gen_debug.h | 1 +
src/intel/common
Signed-off-by: Alyssa Rosenzweig
---
.../drivers/panfrost/include/panfrost-job.h | 13 +++
src/gallium/drivers/panfrost/pan_context.c| 23 ---
2 files changed, 19 insertions(+), 17 deletions(-)
diff --git a/src/gallium/drivers/panfrost/include/panfrost-job.h
b/src/
https://bugs.freedesktop.org/show_bug.cgi?id=109656
Bug ID: 109656
Summary: duplicate symbol _compute_shader_video_buffer
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Mac OS X (All)
Status: NEW
https://bugs.freedesktop.org/show_bug.cgi?id=109535
Bug 109535 depends on bug 109581, which changed state.
Bug 109581 Summary: [BISECTED] Nothing is Rendered on Sascha Willem's
"subpasses" demo
https://bugs.freedesktop.org/show_bug.cgi?id=109581
What|Removed |A
23 matches
Mail list logo
