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
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
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
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
---
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
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/
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
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.
> >
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
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
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
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'
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,
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
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
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
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
-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
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 |
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
>
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: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
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.
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
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
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
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 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
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(-)
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 +
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
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
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
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
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 ?
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
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
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
>>
Hmm..
Looking at /usr/include/time.h on ubuntu, I see the following:
#if (!defined __timespec_defined\
&& ((defined _TIME_H \
&& (defined __USE_POSIX199309 \
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
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
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
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
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.
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
Why?
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
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
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: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 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
> >>> ---
> >>>
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
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.
>>
>> >
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 +-
> > >
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
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 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 -
Reviewed-by: Ian Romanick
On November 10, 2017 12:17:38 PM Jordan Justen
wrote:
On 2017-11-10 08:43:37, Ian Romanick wrote:
The one thing I would add is an issues section with a single issue about
the decision to use a single enum for everything.
How does this sound?
diff --git a/docs
On Fri, Nov 10, 2017 at 10:48 AM, Chad Versace
wrote:
> On Thu 09 Nov 2017, Jason Ekstrand wrote:
> > On Thu, Nov 9, 2017 at 4:23 PM, Chad Versace <[1]
> chadvers...@chromium.org>
> > wrote:
> >
> > On Wed 08 Nov 2017, Jason Ekstrand wrote:
> > > On Wed, Nov 8, 2017 at 1:34 AM, Samuel Igl
https://bugs.freedesktop.org/show_bug.cgi?id=103412
--- Comment #1 from Brian Paul ---
Thanks. Looks good AFAICT. I'll do a little reformatting and testing before
pushing.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
On 11/10/2017 11:08 AM, Emil Velikov wrote:
On 10 November 2017 at 17:53, Dylan Baker wrote:
These variables were removed from autotools in 2008 (sha:
80f68e1b6a0e5bd2da799c), but they have lived on here. The Scons build
meanwhile doesn't set a patch/tiny version at all, just major and minor.
T
On Thu 09 Nov 2017, Jason Ekstrand wrote:
> On Thu, Nov 9, 2017 at 4:23 PM, Chad Versace <[1]chadvers...@chromium.org>
> wrote:
>
> On Wed 08 Nov 2017, Jason Ekstrand wrote:
> > On Wed, Nov 8, 2017 at 1:34 AM, Samuel Iglesias Gonsálvez <[1]
> > [2]sigles...@igalia.com> wrote:
> >
>
On 10/11/2017 15:42, Nicolai Hähnle wrote:
On 10.11.2017 14:00, Jon Turney wrote:
On 09/11/2017 21:41, Nicolai Hähnle wrote:
Sorry for the mess.
I'm going to suggest that the fallback declaration of timespec_get()
also needs to be provided for POSIX systems which don't have it.
Not noticed
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 --git a/configure.ac b/configure.ac
> index 411c4f6b3e0..213c
https://bugs.freedesktop.org/show_bug.cgi?id=103674
Bug ID: 103674
Summary: u_queue.c:173:7: error: implicit declaration of
function 'timespec_get' is invalid in C99
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
https://bugs.freedesktop.org/show_bug.cgi?id=103671
Vinson Lee changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bugs.freedesktop.org/show_bug.cgi?id=103671
Jon Turney changed:
What|Removed |Added
CC||jon.tur...@dronecode.org.uk
--- Comment #1
It appears that we include the shader cache sources into libglsl
regardless.
The Meson build already does this.
Signed-off-by: Jordan Justen
---
src/compiler/Android.glsl.mk | 3 +--
src/compiler/Makefile.glsl.am | 3 +--
src/compiler/Makefile.sources | 6 ++
3 files changed, 4 insertions(
Cc: Timothy Arceri
Signed-off-by: Jordan Justen
---
src/util/disk_cache.c | 129 ++
1 file changed, 15 insertions(+), 114 deletions(-)
diff --git a/src/util/disk_cache.c b/src/util/disk_cache.c
index fde6e2e0974..911ca8f9234 100644
--- a/src/util/
Signed-off-by: Jordan Justen
---
src/util/Makefile.sources | 4 +-
src/util/meson.build | 2 +
src/util/u_zlib.c | 154 ++
src/util/u_zlib.h | 50 +++
4 files changed, 209 insertions(+), 1 deletion(-)
create mode 1
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 wrote:
>
> On F
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 --git a/configure.ac b/configure.ac
index 411c4f6b3e0..213cfe132fb 100644
--- a/configure.ac
+++ b/configure.ac
@@ -900,7 +
Fixes a build issue with gallium headers trying to include llvm ones
(which I don't have installed).
Thanks!
Tested-by: Lionel Landwerlin
On 10/11/17 17:23, Dylan Baker wrote:
Using build_by_default : false is convenient for dependencies that can
be pulled in by various diverse components of
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':
../../../src/util/u_queue.c:173:7: error: imp
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 wrote:
On Fri, Nov 10, 2017 at 12:40 AM, Matt Arsenault
wrote:
>
On 2017-11-10 08:43:37, Ian Romanick wrote:
>
> The one thing I would add is an issues section with a single issue about
> the decision to use a single enum for everything.
How does this sound?
diff --git a/docs/specs/MESA_program_binary_formats.txt
b/docs/specs/MESA_program_binary_formats.txt
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.
---
src/gallium/include/state_tracker/st_api.h | 3 +--
src/gallium/state_trackers/dri/dri_helpers.c | 3 +--
src/mesa/state_tracker/st_manager.c
Just to note, I made some Piglit patches to verify this new behaviour
too:
https://patchwork.freedesktop.org/series/33626/
This might make testing a little easier if someone wants to implement
the behaviour for other drivers too.
- Neil
Antia Puentes writes:
> Hi,
>
> the series sets gl_BaseV
On 10 November 2017 at 17:53, Dylan Baker wrote:
> These variables were removed from autotools in 2008 (sha:
> 80f68e1b6a0e5bd2da799c), but they have lived on here. The Scons build
> meanwhile doesn't set a patch/tiny version at all, just major and minor.
> This patch removes the unused variables
If fixed the typo in the subject locally.
Quoting Dylan Baker (2017-11-10 09:23:37)
> Using build_by_default : false is convenient for dependencies that can
> be pulled in by various diverse components of the build system, the
> gallium hardware/software drivers and state trackers do not fit that
On 11/10/2017 10:53 AM, Dylan Baker wrote:
These variables were removed from autotools in 2008 (sha:
80f68e1b6a0e5bd2da799c), but they have lived on here. The Scons build
meanwhile doesn't set a patch/tiny version at all, just major and minor.
This patch removes the unused variables and simply se
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 wrote:
On Fri, Nov 10, 2017 at 12:40 AM, Matt Arsenault wrote:
On Nov 10, 2017, at 07:41, Marek Olšák wrote:
Hi,
This fixes the TCS gl_ClipDistance
These variables were removed from autotools in 2008 (sha:
80f68e1b6a0e5bd2da799c), but they have lived on here. The Scons build
meanwhile doesn't set a patch/tiny version at all, just major and minor.
This patch removes the unused variables and simply sets the version,
leaving patch/tiny as 0 since
From: Neil Roberts
The base vertex in Vulkan is different from GL in that for non-indexed
primitives the value is taken from the firstVertex parameter instead
of being set to zero. This coincides with the new
SYSTEM_VALUE_BASE_VERTEX_ID instead of BASE_VERTEX.
---
src/compiler/spirv/vtn_variable
From: Neil Roberts
---
src/mesa/drivers/dri/i965/brw_context.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/mesa/drivers/dri/i965/brw_context.c
index 19d5a2e3503..01c8e8cb3cf 100644
--- a/src/mesa/drivers/dri/i965/brw_contex
- From the OpenGL 4.6 (11.1.3.9 Shader Inputs) specification:
"gl_BaseVertex holds the integer value passed to the baseVertex parameter to the
command that resulted in the current shader invocation. In the case where the
command has no baseVertex parameter, the value of gl_BaseVertex is zero."
-
From: Neil Roberts
---
src/intel/compiler/brw_compiler.h | 1 +
src/intel/compiler/brw_vec4.cpp | 4
2 files changed, 5 insertions(+)
diff --git a/src/intel/compiler/brw_compiler.h
b/src/intel/compiler/brw_compiler.h
index df6ee018546..6b5b73a54f0 100644
--- a/src/intel/compiler/brw_com
The new basevertexid will be emitted in the position previously occupied by
gl_BaseVertex. This way we can keep pointing the indirect buffer for indirect
draw calls. The gl_BaseVertex is now emited as part of the draw_id
VERTEX_ELEMENT.
Reviewed-by: Neil Roberts
---
src/mesa/drivers/dri/i965/brw
From: Neil Roberts
base_vertex will be zero for non-indexed calls, but we need it to
include the ‘first’ parameter. This is true for both GL and Vulkan.
I think this patch will also affect freedreno and radeonsi. I believe
if they are relying on this lowering then they are currently already
brok
This VS system value will contain the value passed as
for indexed draw calls or the value passed as for non-indexed
draw calls. It will be used to calculate the gl_VertexID as
SYSTEM_VALUE_VERTEX_ID_ZERO_BASE plus SYSTEM_VALUE_BASE_VERTEX_ID.
Note that the current calculation which uses SYSTEM_VA
Hi,
the series sets gl_BaseVertex to zero for gl*DrawArrays* in i965.
Previously, its value was the value passed as in the non-indexed
draw call. This was convinient because the gl_VertexID was calculated as
gl_VertexIDBaseZero + gl_BaseVertex.
However, as gl_BaseVertex must be zero for non-inde
The gl_BaseVertex is in a new location now, and the new basevertexid occupies
the old gl_BaseVertex place.
Reviewed-by: Neil Roberts
---
src/intel/compiler/brw_nir.c| 12
src/intel/compiler/brw_vec4.cpp | 14 --
2 files changed, 16 insertions(+), 10 deletions(-)
dif
Reviewed-by: Neil Roberts
---
src/compiler/nir/nir.c | 4
src/compiler/nir/nir_gather_info.c | 1 +
src/compiler/nir/nir_intrinsics.h | 1 +
3 files changed, 6 insertions(+)
diff --git a/src/compiler/nir/nir.c b/src/compiler/nir/nir.c
index 7380bf436a8..6f0477b0676 100644
--- a
1 - 100 of 164 matches
Mail list logo
