On 10 November 2017 at 18:38, Dylan Baker wrote:
> Quoting Jordan Justen (2017-11-10 10:24:36)
>> Cc: Emil Velikov
>> Cc: Dylan Baker
>> Signed-off-by: Jordan Justen
>> ---
>> configure.ac | 6 +-
>> meson.build | 6 +-
>> 2 files changed, 10 insertions(+), 2 deletions(-)
>>
>> diff -
On 10 November 2017 at 19:34, Emil Velikov wrote:
> On 10 November 2017 at 18:38, Dylan Baker wrote:
>> Quoting Jordan Justen (2017-11-10 10:24:36)
>>> Cc: Emil Velikov
>>> Cc: Dylan Baker
>>> Signed-off-by: Jordan Justen
>>> ---
>>> configure.ac | 6 +-
>>> meson.build | 6 +-
>>> 2
On 2017-11-10 10:38:19, Dylan Baker wrote:
> Quoting Jordan Justen (2017-11-10 10:24:36)
> > Cc: Emil Velikov
> > Cc: Dylan Baker
> > Signed-off-by: Jordan Justen
> > ---
> > configure.ac | 6 +-
> > meson.build | 6 +-
> > 2 files changed, 10 insertions(+), 2 deletions(-)
> >
> > dif
Quoting Jordan Justen (2017-11-10 11:43:45)
> On 2017-11-10 10:38:19, Dylan Baker wrote:
> > Quoting Jordan Justen (2017-11-10 10:24:36)
> > > Cc: Emil Velikov
> > > Cc: Dylan Baker
> > > Signed-off-by: Jordan Justen
> > > ---
> > > configure.ac | 6 +-
> > > meson.build | 6 +-
> > >
Dylan Baker writes:
> [ Unknown signature status ]
> Quoting Eric Anholt (2017-11-08 13:26:12)
>> We shouldn't have to manually specify most of these deps, I think, since
>> they should be transitively pulled in by the static libraries using
>> them, right? It's fine either way, though.
>>
>> >
Kenneth Graunke writes:
> The L3 configuration code already considers the TCS and TES programs,
> but failed to listen for TCS/TES program changes.
>
> This was somehow missing.
>
> Cc: mesa-sta...@lists.freedesktop.org
Fixes: e9644cb1f96ccf7e155af09e90f6a7888f609a2d
Reviewed-by: Francisco Jerez
On 2017-11-10 11:38:08, Emil Velikov wrote:
> On 10 November 2017 at 19:34, Emil Velikov wrote:
> > On 10 November 2017 at 18:38, Dylan Baker wrote:
> >> Quoting Jordan Justen (2017-11-10 10:24:36)
> >>> Cc: Emil Velikov
> >>> Cc: Dylan Baker
> >>> Signed-off-by: Jordan Justen
> >>> ---
> >>>
On 10.11.2017 19:10, Marek Olšák wrote:
From: Marek Olšák
Android fences can't be deferred, because st/dri calls fence_finish
with ctx = NULL, so the driver can't flush u_threaded_context.
Reviewed-by: Nicolai Hähnle
---
src/gallium/include/state_tracker/st_api.h | 3 +--
src/gallium
On 10.11.2017 19:20, Kyriazis, George wrote:
Hello Nicolai,
Your commit e3a8013 (util/u_queue: add util_queue_fence_wait_timeout), breaks
the centos6 build.
Specifically, I am getting:
CC libmesautil_la-u_queue.lo
../../../src/util/u_queue.c: In function '_util_queue_fence_wait_timeout
On 10.11.2017 19:24, Connor Abbott wrote:
On Fri, Nov 10, 2017 at 1:19 PM, Marek Olšák wrote:
On Fri, Nov 10, 2017 at 6:55 PM, Nicolai Hähnle wrote:
On 10.11.2017 18:43, Marek Olšák wrote:
On Fri, Nov 10, 2017 at 2:09 AM, Connor Abbott
wrote:
On Thu, Nov 9, 2017 at 7:17 PM, Marek Olšák
Why?
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Found by inspection
Cc: Anuj Phogat
---
src/mesa/drivers/dri/i965/brw_pipe_control.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_pipe_control.c
b/src/mesa/drivers/dri/i965/brw_pipe_control.c
index f5a9915..bae4ba7 100644
--- a/src/mesa/d
Found by inspection
Fixes: d3d0fe4572f62474b86ef3a68405046c68b54062
Cc: Anuj Phogat
---
src/mesa/drivers/dri/i965/brw_pipe_control.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_pipe_control.c
b/src/mesa/drivers/dri/i965/brw_pipe_control.
It seems I missed that one when reviewing before, thank you.
Reviewed-by: Rafael Antognolli
On Fri, Nov 10, 2017 at 01:39:22PM -0800, Jason Ekstrand wrote:
> Found by inspection
>
> Fixes: d3d0fe4572f62474b86ef3a68405046c68b54062
> Cc: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/brw_pipe_co
On 2017-11-10 13:15:10, Matt Turner wrote:
> Why?
For patch 4. Patch 4 can't happen unless zlib is optional because it's
a pain to enable zlib on windows.
No surprise, but the disk cache actually works without zlib. (Of
course, Tim originally implemented it without compression.)
-Jordan
On Fri, Nov 10, 2017 at 1:39 PM, Jason Ekstrand wrote:
> Found by inspection
>
> Fixes: d3d0fe4572f62474b86ef3a68405046c68b54062
> Cc: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/brw_pipe_control.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dr
v2:
* Keep zlib required for autotools & meson (Emil)
Cc: Emil Velikov
Cc: Dylan Baker
Signed-off-by: Jordan Justen
---
configure.ac | 8 +++-
meson.build | 6 +-
2 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/configure.ac b/configure.ac
index 411c4f6b3e0..5db39e5a68
Hmm..
Looking at /usr/include/time.h on ubuntu, I see the following:
#if (!defined __timespec_defined\
&& ((defined _TIME_H \
&& (defined __USE_POSIX199309 \
On Fri, Nov 10, 2017 at 8:42 AM, Rafael Antognolli
wrote:
> On Thu, Nov 09, 2017 at 11:14:43AM -0800, Anuj Phogat wrote:
>> On CNL this bit has been moved to CACHE_MODE_SS register.
>> We already have this enabled in OpenGL driver.
>> See Mesa commit 6c681b4cc1
>>
>> Signed-off-by: Anuj Phogat
>>
https://bugs.freedesktop.org/show_bug.cgi?id=102891
--- Comment #9 from jdr...@gmail.com ---
Created attachment 135388
--> https://bugs.freedesktop.org/attachment.cgi?id=135388&action=edit
new qrenderdoc capture
I did an
export RADV_DEBUG=zerovram
before running qrenderdoc, but as I'm on mesa 1
This is the first block of patches to enable generating SPIR-V from
Mesa's GLSL compiler. The initial use of this is for testing
GL_ARB_spirv, but it may eventually be useful in its own right. A lot
more work will be necessary for that to happen, though.
Most of the work is done, but there are s
From: Ian Romanick
compiler/brw_inst.h: In function ‘brw_reg_type brw_inst_dst_type(const
gen_device_info*, const brw_inst*)’:
compiler/brw_inst.h:801:55: warning: enumeral and non-enumeral type in
conditional expression [-Wextra]
unsigned file = __builtin_strcmp("dst", #reg) == 0 ?
From: Ian Romanick
Previously bitset.h would include u_math.h to get bitscan.h. u_math.h
lives in src/gallium/auxiliary/util while both bitset.h and bitscan.h
live in src/util. Having the one file directly include another file
that lives in the same directory makes much more sense.
As a side-e
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/compiler/spirv/spirv.core.grammar.json | 417 -
src/compiler/spirv/spirv.h | 56 ++--
2 files changed, 201 insertions(+), 272 deletions(-)
diff --git a/src/compiler/spirv/spirv.core.grammar.json
From: Ian Romanick
Let the lowering in NIR handle it instead.
This hurts one shader that occurs twice in shader-db (SynMark GSCloth)
on IVB and HSW. No other shaders or platforms were affected.
total cycles in shared programs: 253438422 -> 253438426 (0.00%)
cycles in affected programs: 412 ->
From: Ian Romanick
The SPV_KHR_shader_ballot spec says:
(Add the SubgroupBallotKHR capability to SubgroupSize.)
(Add the SubgroupBallotKHR capability to SubgroupLocalInvocationId.)
Yet the annotations are missing from the JSON. See also
https://github.com/KhronosGroup/SPIRV-Headers/is
From: Ian Romanick
This prevents the next change from breaking the build, and I felt that
it was different enough from the rest of that patch to stand on its own.
Signed-off-by: Ian Romanick
---
src/compiler/nir/nir_validate.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --
From: Ian Romanick
This slightly simplifies later changes that add more Makefile.*.am
files.
Signed-off-by: Ian Romanick
---
src/compiler/Makefile.am | 1 +
src/compiler/Makefile.glsl.am | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/compiler/Makefile.am b/src/c
From: Ian Romanick
Future changes will add generated files used only from
src/compiler/glsl. These can't be built from Makefile.nir.am, and we
can't move all the rules from Makefile.nir.am to Makefile.spirv.am (and
it would be silly anyway).
v2: Do it for meson too.
Signed-off-by: Ian Romanick
From: Ian Romanick
v2: Clean ups. Remove some functions that never ended up being used.
v3: After updating spirv.core.grammar.json, fix the handling of
ShaderViewportMaskNV. See the comment around line 71 of
spirv_capabilities_h.py.
Signed-off-by: Ian Romanick
---
src/compiler/Makefile.sour
From: Ian Romanick
v2: Don't try to automatically set SpvCapabilityGeometry or
SpvCapabilityTessellation.
---
src/compiler/Makefile.sources | 2 +
src/compiler/Makefile.spirv.am| 4 +
src/compiler/spirv/.gitignore | 1 +
src/compiler/spirv/meson.build| 7 +
On 10 November 2017 at 22:18, Jordan Justen wrote:
> v2:
> * Keep zlib required for autotools & meson (Emil)
>
> Cc: Emil Velikov
> Cc: Dylan Baker
> Signed-off-by: Jordan Justen
> ---
> configure.ac | 8 +++-
> meson.build | 6 +-
> 2 files changed, 12 insertions(+), 2 deletions(-)
On Fri, Nov 10, 2017 at 2:32 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> This prevents the next change from breaking the build, and I felt that
> it was different enough from the rest of that patch to stand on its own.
>
> Signed-off-by: Ian Romanick
> ---
> src/compiler/nir/nir_validate.c
Mesa 17.2.5 is now available.
In this release we have:
In Mesa Core a GL error related to the ARB_ES3_1_compatibility spec
noticed with the GFXBench 5 Aztec Ruins has been corrected.
The GLSL compiler is not giving a linker error for mismatching uniform
precision with GLSL ES 1.00 any more. Thi
On 2017-11-10 08:19:38, Emil Velikov wrote:
> On 7 November 2017 at 11:54, Emil Velikov wrote:
> > From: Emil Velikov
> >
> > Checking the override was useful in the early stages of developing the
> > extension.
> >
> > Now that everything is wired, where possible, we can drop the check.
> > Doin
On 11/10/2017 04:54 PM, Matt Turner wrote:
> On Fri, Nov 10, 2017 at 2:32 PM, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> This prevents the next change from breaking the build, and I felt that
>> it was different enough from the rest of that patch to stand on its own.
>>
>> Signed-off-by: Ian
Split off some cleanup patches from my GL_EXT_semaphore series.
There are no functional changes in this series.
Andres Rodriguez (3):
broadcom/vc4: fix indentation in vc4_screen.c
mesa: minor tidy up for memory object error strings
mesa/st: add missing copyright headers to memoryobjects fil
Signed-off-by: Andres Rodriguez
---
src/mesa/state_tracker/st_cb_memoryobjects.c | 24
src/mesa/state_tracker/st_cb_memoryobjects.h | 24
2 files changed, 48 insertions(+)
diff --git a/src/mesa/state_tracker/st_cb_memoryobjects.c
b/src/mesa/stat
Signed-off-by: Andres Rodriguez
---
src/mesa/main/externalobjects.c | 30 ++
1 file changed, 14 insertions(+), 16 deletions(-)
diff --git a/src/mesa/main/externalobjects.c b/src/mesa/main/externalobjects.c
index e70280c..4e9f8f8 100644
--- a/src/mesa/main/externalobje
Stumbled into this when adding a new PIPE_CAP.
Signed-off-by: Andres Rodriguez
---
src/gallium/drivers/vc4/vc4_screen.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/vc4/vc4_screen.c
b/src/gallium/drivers/vc4/vc4_screen.c
index 1a9eaad.
On Fri, Nov 10, 2017 at 4:02 PM, Ian Romanick wrote:
> On 11/10/2017 04:54 PM, Matt Turner wrote:
>> On Fri, Nov 10, 2017 at 2:32 PM, Ian Romanick wrote:
>>> From: Ian Romanick
>>>
>>> This prevents the next change from breaking the build, and I felt that
>>> it was different enough from the res
Jason, having this into account, I'll leave this patch out of 17.2 so
far we don't have another one that fixes this regression (?)
I noticed that the patch bisected by Mark is a different one so I'm not
sure I'm understanding the status, though.
Let me know what you think.
On Thu, 2017-11-09 at
On Fri, Nov 10, 2017 at 4:12 PM, Andres Gomez wrote:
> Jason, having this into account, I'll leave this patch out of 17.2 so
> far we don't have another one that fixes this regression (?)
>
This patch doesn't regress anything, it just isn't sufficient to fix the
bug on little-core.
--Jason
>
From: Boyuan Zhang
Implement encoding of sps, pps, and silce headers using the newly added h.264
header coding descriptors functions based on h.264 specs.
Signed-off-by: Boyuan Zhang
---
src/gallium/drivers/radeon/radeon_vcn_enc.h | 1 +
src/gallium/drivers/radeon/radeon_vcn_enc_1_2.c |
-Original Message-
From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of
Christian König
Sent: November-09-17 12:19 PM
To: Mark Thompson; mesa-dev@lists.freedesktop.org
Subject: Re: [Mesa-dev] [PATCH 10/18] radeon/vcn: add encode header
implementations
Am 09.11.20
On 11/11/17 08:49, Jordan Justen wrote:
On 2017-11-10 13:15:10, Matt Turner wrote:
Why?
For patch 4. Patch 4 can't happen unless zlib is optional because it's
a pain to enable zlib on windows.
No surprise, but the disk cache actually works without zlib. (Of
course, Tim originally implemented
On Fri, Nov 10, 2017 at 4:44 PM, Timothy Arceri wrote:
> On 11/11/17 08:49, Jordan Justen wrote:
>>
>> On 2017-11-10 13:15:10, Matt Turner wrote:
>>>
>>> Why?
>>
>>
>> For patch 4. Patch 4 can't happen unless zlib is optional because it's
>> a pain to enable zlib on windows.
>>
>> No surprise, but
We renamed "Function Enable" to "Enable", which broke our detection
of whether shaders are enabled or not. So, we'd see a bunch of HS/DS
packets with program offsets of 0, and think that was a valid TCS/TES.
Fixes: c032cae9ff77e (genxml: Rename "Function Enable" to "Enable".)
---
src/intel/tools
This indexing is bogus. idx_program is the offset of the next program.
At this point, we've walked through the entire batch, and accumulated
all the programs. So adding it again simply skips over 100% of the
programs.
---
src/intel/tools/aubinator_error_decode.c | 13 ++---
1 file change
On Fri, 2017-11-10 at 16:26 -0800, Jason Ekstrand wrote:
> On Fri, Nov 10, 2017 at 4:12 PM, Andres Gomez
> wrote:
> > Jason, having this into account, I'll leave this patch out of 17.2
> > so
> > far we don't have another one that fixes this regression (?)
>
> This patch doesn't regress anything,
On 2017-11-10 16:46:32, Matt Turner wrote:
> On Fri, Nov 10, 2017 at 4:44 PM, Timothy Arceri wrote:
> > On 11/11/17 08:49, Jordan Justen wrote:
> >>
> >> On 2017-11-10 13:15:10, Matt Turner wrote:
> >>>
> >>> Why?
> >>
> >>
> >> For patch 4. Patch 4 can't happen unless zlib is optional because it'
Reviewed-by: Timothy Arceri
On 11/11/17 00:46, Rob Clark wrote:
In case the IR is NIR, the driver takes reference to the nir_shader.
Also, because there are no variants, we need to clone the shader,
instead of sharing the reference with gl_program, which would result
in a double free in _mesa_d
Jason, this series of nominated commits landed without mentioning any
specific stable queue.
From what I'm seeing, they depend on 379b24a40d3 which didn't make it
for 17.2 so I'm dropping the 3 of them for that queue:
7364f080f9a272323ed3491f278a1eed3eb9b1a7 (intel/nir: Add a helper for getting
On 11/11/17 12:28, Andres Gomez wrote:
Jason, this series of nominated commits landed without mentioning any
specific stable queue.
From what I'm seeing, they depend on 379b24a40d3 which didn't make it
for 17.2 so I'm dropping the 3 of them for that queue:
7364f080f9a272323ed3491f278a1eed3eb9b
On 2017-11-09 19:40:40, Jordan Justen wrote:
> On 2017-11-09 08:07:57, Jose Fonseca wrote:
> > On 09/11/17 13:19, Emil Velikov wrote:
> > > On 9 November 2017 at 06:42, Jordan Justen
> > > wrote:
> > >> Signed-off-by: Jordan Justen
> > > Currently zlib is a dependency for !WIndows platforms.
> >
To avoid forward declaration.
---
src/mesa/state_tracker/st_manager.c | 28 +++-
1 file changed, 15 insertions(+), 13 deletions(-)
diff --git a/src/mesa/state_tracker/st_manager.c
b/src/mesa/state_tracker/st_manager.c
index aedbc13..8a933d4 100644
--- a/src/mesa/state_tra
To get rid of forward declaration.
---
src/mesa/state_tracker/st_context.c | 123 ++--
1 file changed, 62 insertions(+), 61 deletions(-)
diff --git a/src/mesa/state_tracker/st_context.c
b/src/mesa/state_tracker/st_context.c
index 8abf879..c52f42b 100644
--- a/src/
---
docs/llvmpipe.html | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/docs/llvmpipe.html b/docs/llvmpipe.html
index 2efbbd4..e467692 100644
--- a/docs/llvmpipe.html
+++ b/docs/llvmpipe.html
@@ -108,10 +108,10 @@ To build everything on Linux invoke scons as:
scons buil
We only have a single stencil read mask and write mask. Issue a
warning if different front/back values are used. The Piglit
gl-2.0-two-sided-stencil test hits this.
---
src/gallium/drivers/svga/svga_pipe_depthstencil.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/gall
On Fri, Nov 10, 2017 at 9:58 PM, Nicolai Hähnle wrote:
> On 10.11.2017 19:24, Connor Abbott wrote:
>>
>> On Fri, Nov 10, 2017 at 1:19 PM, Marek Olšák wrote:
>>>
>>> On Fri, Nov 10, 2017 at 6:55 PM, Nicolai Hähnle
>>> wrote:
On 10.11.2017 18:43, Marek Olšák wrote:
>
>
> On F
It's common to use signed int modulo in GLSL. As it happens, the GLSL
specs allow the result to be undefined, but that seems fairly
surprising. It's not that much more effort to get it right, at least for
positive modulo operators.
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/codeg
We can still use the optimized division methods which make use of
multiplication with overflow.
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir
Setting build_by_default to true makes it suddenly pull in freedreno and
all of gallium unconditionally.
Cc: Rob Clark
Cc: Dylan Baker
---
src/gallium/drivers/freedreno/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/freedreno/meson.build
b/s
On 09/11/17 21:46, Timothy Arceri wrote:
On 09/11/17 17:42, Jordan Justen wrote:
git://people.freedesktop.org/~jljusten/mesa i965-get-program-binary-v1
This series adds i965 support for ARB_get_program_binary with greater
than 0 supported formats. Today we support this extension, but
advertise
101 - 164 of 164 matches
Mail list logo
