https://bugs.freedesktop.org/show_bug.cgi?id=99481
Bug ID: 99481
Summary: glsl_parser_extras.cpp:1253:18: error: expected ')'
before 'PRId64'
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (Al
Gallium drivers have had this for a while. It makes sense to support
it consistently across drivers, so expose it in i965 as well.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_context.c | 3 +++
src/mesa/drivers/dri/i965/intel_screen.c | 1 +
2 files changed, 4 insertions(+
https://bugs.freedesktop.org/show_bug.cgi?id=93089
--- Comment #11 from Jonathan Gray ---
Previously 64 bit atomics were contained to i965. The on disk shader cache
introduced in Mesa 13.0 in 87ab26b2ab35a29d446ae66f1795d40c184c0739 now
requires 64 bit atomics as well which breaks Mesa on 32 bit
The "drawbuffer-modes -auto" piglit test was also failing on llvmpipe
approx. 1/5 times, and the glarea sample app had the same issue as the
softpipe. The PIPE_BIND_DISPLAY_SYNC fix seems to apply well on both
drivers.
Signed-off-by: Karl Lessard
---
src/gallium/drivers/llvmpipe/lp_texture.c | 2
I think that we do not have to force a display/drawable refresh right
after unmapping the front buffer, this could be handled later on when
the pipe is being flushed or buffers are swapped (for optimisation).
Signed-off-by: Karl Lessard
---
src/gallium/winsys/sw/dri/dri_sw_winsys.c | 13 ++--
Rename pipe_screen "resource_create_front" method to "resource_create2"
since it is not only invoked for front buffer creation (I'm not a big
fan of the '2' suffix but nothing else came up to my mind...)
Signed-off-by: Karl Lessard
---
src/gallium/drivers/llvmpipe/lp_texture.c | 19 ++---
The "drawbuffer-modes -auto" piglit test was also failing on llvmpipe
approx. 1/5 times, and the glarea sample app had the same issue as the
softpipe. The PIPE_BIND_DISPLAY_SYNC fix seems to apply well on both
drivers.
Signed-off-by: Karl Lessard
---
src/gallium/drivers/llvmpipe/lp_texture.c | 2
Rename pipe_screen "resource_create_front" method to "resource_create2"
since it is not only invoked for front buffer creation (I'm not a big
fan of the '2' suffix but nothing else came up to my mind...)
Signed-off-by: Karl Lessard
---
src/gallium/drivers/llvmpipe/lp_texture.c | 19 ++---
I think that we do not have to force a display/drawable refresh right
after unmapping the front buffer, this could be handled later on when
the pipe is being flushed or buffers are swapped.
Signed-off-by: Karl Lessard
---
src/gallium/winsys/sw/dri/dri_sw_winsys.c | 13 ++---
1 file chang
This patch is mainly a workaround of commit
2b676570960277d47477822ffeccc672613f914 and fixes many problems with
framebuffer blitting in software drivers when the front buffer is
involved. It resolves regression issues listed above but also problems
with gtk-gl applications (like
https://github.com
I think that we do not have to force a display/drawable refresh right
after unmapping the front buffer, this could be handled later on when
the pipe is being flushed or buffers are swapped.
Signed-off-by: Karl Lessard
---
src/gallium/winsys/sw/dri/dri_sw_winsys.c | 13 ++---
1 file chang
The "drawbuffer-modes -auto" piglit test was also failing on llvmpipe
approx. 1/5 times, and the glarea sample app had the same issue as the
softpipe. The PIPE_BIND_DISPLAY_SYNC fix seems to apply well on both
drivers.
Signed-off-by: Karl Lessard
---
src/gallium/drivers/llvmpipe/lp_texture.c | 2
Rename pipe_screen "resource_create_front" method to "resource_create2"
since it is not only invoked for front buffer creation (I'm not a big
fan of the '2' suffix but nothing else came up to my mind...)
Signed-off-by: Karl Lessard
---
src/gallium/drivers/llvmpipe/lp_texture.c | 19 ++---
This patch is mainly a workaround of commit
2b676570960277d47477822ffeccc672613f914 and fixes many problems with
framebuffer blitting in software drivers when the front buffer is
involved. It resolves regression issues listed above but also problems
with gtk-gl applications (like
https://github.com
Build mesa 3219 completed
Commit bb5db5564f by Kenneth Graunke on 1/21/2017 3:14 AM:
glsl: Rename [u]int64_t tokens.\n\nbasetsd.h on Windows defines INT64 and UINT64 typedefs which conflict\nwith these. Append "_TOK" to avoid conflicts.\n\nShould fix the Windo
Elsewhere we use _TOK. Not sure if you want to move to _TOKEN everywhere.
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
basetsd.h on Windows defines INT64 and UINT64 typedefs which conflict
with these. Append "_TOKEN" to avoid conflicts.
Should fix the Windows build.
Signed-off-by: Kenneth Graunke
---
src/compiler/glsl/glsl_lexer.ll | 4 ++--
src/compiler/glsl/glsl_parser.yy | 6 +++---
2 files changed, 5 inse
On Friday, January 20, 2017 7:07:31 PM PST Matt Turner wrote:
> Gen8 adds Q/UQ types. We attempted to change the types back to DF in the
> generator (commit c95380c40), but an assertion added in the FP64 series
> (commit e481dcc3) triggers before that code has a chance to execute.
>
> In fact, usi
Gen8 adds Q/UQ types. We attempted to change the types back to DF in the
generator (commit c95380c40), but an assertion added in the FP64 series
(commit e481dcc3) triggers before that code has a chance to execute.
In fact, using Q/UQ in the IR and then changing to DF in the generator
would not wor
This reverts commit c95380c4044237d73fb537511667c3c8f658fcee.
---
src/mesa/drivers/dri/i965/brw_eu_emit.c | 8
1 file changed, 8 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c
b/src/mesa/drivers/dri/i965/brw_eu_emit.c
index 4b557d9..de6a1d9 100644
--- a/src/mesa/drive
https://bugs.freedesktop.org/show_bug.cgi?id=99467
--- Comment #6 from Pierre-Loup A. Griffais ---
[re-posting with proper account]
The developers are already aware of the problem; it'll probably get addressed
in the next major update for DOOM.
--
You are receiving this mail because:
You are t
https://bugs.freedesktop.org/show_bug.cgi?id=99467
--- Comment #5 from Pierre-Loup A. Griffais ---
The developers are already aware of the problem; it'll probably get addressed
in the next major update for DOOM.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are
https://bugs.freedesktop.org/show_bug.cgi?id=99467
--- Comment #4 from Grazvydas Ignotas ---
Good luck getting in touch with any of the devs of an AAA game company.
And I can't imagine why they'd want to fix it and have to redo the whole QA
just for some obscure platform the game is not supposed
On Fri, Jan 20, 2017 at 4:46 PM, Ian Romanick wrote:
> On 12/23/2016 06:46 AM, Jason Ekstrand wrote:
> > Bah... It's definitely illegal. I had fix glslang myself. I have some
> > NIR reworks planned that would make this work "properly" but it's a bit
> > more complicated than this hack. That s
On 12/23/2016 06:46 AM, Jason Ekstrand wrote:
> Bah... It's definitely illegal. I had fix glslang myself. I have some
> NIR reworks planned that would make this work "properly" but it's a bit
> more complicated than this hack. That said, I'm very reluctant to fix
> app bugs in our driver. There
https://bugs.freedesktop.org/show_bug.cgi?id=99467
--- Comment #3 from Edward O'Callaghan ---
(In reply to Grazvydas Ignotas from comment #2)
> Supposedly it's because the game is doing something illegal:
> https://lists.freedesktop.org/archives/mesa-dev/2016-December/138955.html
>
> However bec
On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen <
topi.pohjolai...@gmail.com> wrote:
> This is kept on purpose in i965. It can be moved to ISL if it
> is needed in vulkan.
>
> Pointers to miptrees are given solely for verification purposes.
> These will be dropped in following patches.
>
> Signe
On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen <
topi.pohjolai...@gmail.com> wrote:
> Gen8 and Gen9 are already more heavily constrained as one
> applies arrayed/mipmapped alignment even for non-arrayed and
> non-mipmapped.
>
> Signed-off-by: Topi Pohjolainen
> ---
> src/intel/isl/isl_gen7.c
For HiZ, we did this directly in isl_choose_image_alignment_el in isl.c.
Maybe it's better to do it there given that they're all 4x4.
On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen <
topi.pohjolai...@gmail.com> wrote:
> Otherwise i965 driver will start experiencing failures with
> "texelFetch
Build mesa 3217 failed
Commit db6d23cfd2 by Ian Romanick on 9/1/2016 7:06 PM:
i965: Enable ARB_gpu_shader_int64 on Gen8+\n\nSigned-off-by: Ian Romanick \nReviewed-by: Matt Turner
Configure your notification preferences
__
Thanks!
In practice, on nv50, it checks the vertex shader for the flag, and on
nvc0 it's per-stage (since they killed the global override on
kepler+).
On Fri, Jan 20, 2017 at 5:48 PM, Axel Davy wrote:
> With this patch, there is one case where we hit the "undefined case" of the
> spec (mismatch
https://bugs.freedesktop.org/show_bug.cgi?id=99477
Andreas Schultes changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #4 from Andreas
With this patch, there is one case where we hit the "undefined case" of
the spec (mismatch between different shaders): With streamout we use a
dummy pixel created by utils instead of nine_shader.
As we use software renderers for the feature doing the streamout, I
guess that's ok for now.
Tha
https://bugs.freedesktop.org/show_bug.cgi?id=99477
--- Comment #3 from Kenneth Graunke ---
Sorry, nevermind that - it exists in 1.30, it's just deprecated. Should work
though.
--
You are receiving this mail because:
You are the assignee for the bug._
https://bugs.freedesktop.org/show_bug.cgi?id=99477
--- Comment #2 from Kenneth Graunke ---
GLSL 1.30 does not include texture2DLod(). The function is called textureLod()
in #version 130.
--
You are receiving this mail because:
You are the assignee for the bug.__
Matt Turner writes:
> In commit c35fa7a, we changed the "width" of DF source registers to 2,
> which is conceptually fine. Unfortunately a VertStride of 2 is not
> allowed by align16 instructions on IVB/BYT, and the regular VertStride
> of 4 works fine in any case.
>
I'll try to throw some light
size_t is defined in the C Standard section 4.1.5 as an
'unsigned integral type'.
Fixes the following clang compile warning:
glsl/blob.c:110:15: warning: comparison of unsigned expression < 0 is
always false [-Wtautological-compare]
if (offset < 0 || blob->size - offset < to_write)
https://bugs.freedesktop.org/show_bug.cgi?id=97879
m...@pepelino.de changed:
What|Removed |Added
CC||m...@pepelino.de
--
You are receiving
Hi Rhys,
2017-01-19 7:02 GMT+01:00 Rhys Kidd :
> Use of unsigned loop control variable with '>= 0' would lead to infinite loop.
>
> Reported by clang:
>
> etnaviv_compiler.c:1024:39: warning: comparison of unsigned expression >= 0 is
> always true [-Wtautological-compare]
>for (unsigned
I have tested this series with the branches that you mentioned, and with
piglit with the patches from my own branch:
https://github.com/rantogno/piglit/tree/review/fences-v02
Everything seems to work fine. You can add:
Tested-by: Rafael Antognolli
I also have gone through these patches several
https://bugs.freedesktop.org/show_bug.cgi?id=99477
--- Comment #1 from Ilia Mirkin ---
Are you sure that the GLSL version is 1.30? i.e. do those shaders have a
#version 130 at the top? If not, they get GLSL version 1.10, which does not
have those functions.
You can force it globally with force_g
On 20/01/17 20:40, Dave Airlie wrote:
On 21 January 2017 at 03:08, Jason Ekstrand wrote:
On Fri, Jan 20, 2017 at 8:45 AM, Lionel Landwerlin
wrote:
Some applications might add location decoration to samplers. Rather
than raising an error it seems it would make more sense to just
discard these
Build mesa 3216 completed
Commit 74c23bde5b by Lionel Landwerlin on 1/19/2017 4:20 PM:
anv: don't require render target isl bit for depth/stencil surfaces\n\nBlorp can deal with depth/stencil surfaces blits/copies without the\nrender target requirement. Also ha
On Tue, Jan 17, 2017 at 1:49 AM, Samuel Iglesias Gonsálvez
wrote:
> It is tested empirically that IVB/BYT don't support indirect addressing
> with doubles but it is not documented in the PRM.
>
> This patch applies the same solution than for Cherryview/Broxton and
> takes into account that we cann
https://bugs.freedesktop.org/show_bug.cgi?id=99477
Bug ID: 99477
Summary: [Master of Orion] Shader compile error: no matching
function for call to `texture2DLod(sampler2D, vec2,
float)';
Product: Mesa
Version: git
Otherwise for a pack_double_2x32_split opcode, we emit:
vec1 64 ssa_135 = pack_double_2x32_split ssa_133, ssa_134
mov(8) g5<1>UD g5<4>.xUD { align16 1Q
compacted };
mov(8) g7<2>UD g5<4,4,1>UD { align1 1Q };
ERR
On IVB/BYT, region parameters and execution size for DF are in terms of
32-bit elements, so they are doubled. For evaluating the validity of an
instruction, we halve them.
---
src/mesa/drivers/dri/i965/brw_eu_validate.c | 12
1 file changed, 12 insertions(+)
diff --git a/src/mesa/dri
I committed my EU validator earlier today. It's caught three bugs in the IVB
fp64 series. Patches 2 and 3 fix two of them. I'll respond directly to the
patch in Igalia's series that introduces the other bug.
These should go in with Igalia's series, before the extension is turned on.
_
In commit c35fa7a, we changed the "width" of DF source registers to 2,
which is conceptually fine. Unfortunately a VertStride of 2 is not
allowed by align16 instructions on IVB/BYT, and the regular VertStride
of 4 works fine in any case.
See
generated_tests/spec/arb_gpu_shader_fp64/execution/buil
In order to handle CCS_E, we stomp the image format to a UINT format and
then do some bitcasting logic in the shader. This works fine since SKL
render compression only considers the channel layout of the format and
not the format itself. In order for this to work on images that have
been fast-cle
https://bugs.freedesktop.org/show_bug.cgi?id=99413
--- Comment #2 from Gustaw Smolarczyk ---
IIRC Rust has undefined behavior in its shader(s) or something like that.
>From what I remember, running Rust without steam overlay (I think just running
it from command line instead of Steam GUI did the
On 21 January 2017 at 03:08, Jason Ekstrand wrote:
> On Fri, Jan 20, 2017 at 8:45 AM, Lionel Landwerlin
> wrote:
>>
>> Some applications might add location decoration to samplers. Rather
>> than raising an error it seems it would make more sense to just
>> discard these decorations.
>
>
> That's
https://bugs.freedesktop.org/show_bug.cgi?id=99413
--- Comment #1 from gtc...@gmail.com ---
Same here. Ubuntu 16.10, kernel 4.8, Mesa 17-dev padoka ppa. GPU Radeon HD 7870
That said: the driver performance is great if compared to the closed source
driver. Absolutely superior! But the colors are w
On Wed, Jan 18, 2017 at 8:27 AM, Jason Ekstrand
wrote:
> Reviewed-by: Jason Ekstrand
>
>
pushed.
> On Jan 18, 2017 3:23 AM, "Elie Tournier" wrote:
>
> Add the following optimisations:
>
> min(x, -x) = -abs(x)
> min(x, -abs(x)) = -abs(x)
> min(x, abs(x)) = x
> max(x, -abs(x)) = x
> max(x, abs(
Build mesa 3215 failed
Commit 0ae1f4a5ea by Matt Turner on 11/22/2016 6:40 PM:
i965: Split gen_device_info_i965 between brw and cl.\n\ncl (Crestline) supports a few more things than brw (Broadwater). Sort\nand sync PCI IDs from the kernel.\n\nThe G45 PRMs use t
Reviewed-by: Bruce Cherniak
> On Jan 20, 2017, at 1:44 PM, George Kyriazis
> wrote:
>
> Some query results struct contents are declared as cache line aligned.
> Use aligned malloc, and align the whole struct, to be safe.
>
> Fixes crash when compiling with clang.
>
> CC:
>
> ---
> src/gal
Some query results struct contents are declared as cache line aligned.
Use aligned malloc, and align the whole struct, to be safe.
Fixes crash when compiling with clang.
CC:
---
src/gallium/drivers/swr/swr_query.cpp | 7 ---
src/gallium/drivers/swr/swr_query.h | 2 +-
2 files changed, 5
Reviewed-by: Jason Ekstrand
On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen <
topi.pohjolai...@gmail.com> wrote:
> which is not applicable for "all slices at each lod". Current
> logic makes one to believe it has some purpose. When miptree
> layout is calculated brw_miptree_layout_texture_arra
On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen <
topi.pohjolai...@gmail.com> wrote:
> Hardware state setup only needs offset and pitch and ignores the
> rest.
>
True... But it seems a a bit sketchy to lie to blorp even if the lie is
ok. This isn't a NAK, just a comment for now. I need to rea
6-9 are
Reviewed-by: Jason Ekstrand
On Mon, Jan 16, 2017 at 1:13 AM, Topi Pohjolainen <
topi.pohjolai...@gmail.com> wrote:
> In intel_hiz_miptree_buf_create() the miptree is unconditionally
> created with MIPTREE_LAYOUT_FORCE_ALL_SLICE_AT_LOD.
>
> Signed-off-by: Topi Pohjolainen
> ---
> src/m
This will allow to expose more queries in order to know which
blocks are busy/idle.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeon/r600_gpu_load.c| 55 ++-
src/gallium/drivers/radeon/r600_pipe_common.h | 18 +
src/gallium/drivers/radeon/r600_que
It's also possible to monitor them via performance counters but
the hardware can only use two counters simultaneously. It seems
easier to re-use the existing code which reads from MMIO instead
of writing a multi-pass approach.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeon/r600_gp
Il lunedì 9 gennaio 2017, Zhen Wu > ha scritto:
> Thanks for your review, Rob. Using kms-dri would mean writing a new
> gralloc to basically the same thing as
> gralloc.default and moving to grm_gralloc seems to be a bigger task meant
> for another day. But I agree we would
> want to avoid introdu
From: Marek Olšák
The corruption is inevitable and hangs are possible too.
---
src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
index d63ff36
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index cba4e7d..0b77c82 100644
--- a/src/gallium/drivers/radeon/r600_textur
From: Marek Olšák
---
src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 11 +++
src/gallium/winsys/amdgpu/drm/amdgpu_cs.h | 2 ++
src/gallium/winsys/amdgpu/drm/amdgpu_winsys.h | 1 +
3 files changed, 14 insertions(+)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
b/src/gall
From: Marek Olšák
Cc: 17.0
---
src/gallium/drivers/radeonsi/si_state.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/radeonsi/si_state.c
index 876cbf6..01edff9 100644
--- a/src/gallium/drivers/radeonsi/si
From: Marek Olšák
so that most shaders can get lower VGPR usage thanks to lazy input loading.
I think this is a more accurate constraint that prevents the black transitions
in Witcher 2.
Affected shaders (7758):
Max Waves: 57437 -> 58231 (1.38 %)
---
src/gallium/drivers/radeonsi/si_shader_tgsi_
From: Marek Olšák
not used since we started using the offchip tess ring
---
src/gallium/drivers/radeonsi/si_shader.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 10f40a9..5ca974e 1
From: Marek Olšák
Some CIK-VI docs say this is the default behavior on SI. That doesn't
answer whether it's also the default behavior on CIK-VI.
Cc: 17.0 13.0
---
src/gallium/drivers/radeonsi/si_state_draw.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/driv
From: Marek Olšák
If radeonsi starts compiling an optimized shader variant for asynchronously
with a GL debug callback set and the application destroys the GL context,
radeonsi crashes when trying to write shader stats into the debug output
of a non-existent context after compilation, because st/
On Fri, Jan 20, 2017 at 10:26 AM, Ilia Mirkin wrote:
> On Fri, Jan 20, 2017 at 1:25 PM, Jason Ekstrand
> wrote:
> > On Fri, Jan 20, 2017 at 10:16 AM, Ilia Mirkin
> wrote:
> >>
> >> What level of support would a driver need to provide? Can this just be
> >> enabled for all drivers? [This seems l
On Fri, Jan 20, 2017 at 1:25 PM, Jason Ekstrand wrote:
> On Fri, Jan 20, 2017 at 10:16 AM, Ilia Mirkin wrote:
>>
>> What level of support would a driver need to provide? Can this just be
>> enabled for all drivers? [This seems like largely a driver-side
>> feature rather than hardware-based.]
>
>
On Fri, Jan 20, 2017 at 10:16 AM, Ilia Mirkin wrote:
> What level of support would a driver need to provide? Can this just be
> enabled for all drivers? [This seems like largely a driver-side
> feature rather than hardware-based.]
>
My understanding is that we should just expose this extension o
What level of support would a driver need to provide? Can this just be
enabled for all drivers? [This seems like largely a driver-side
feature rather than hardware-based.]
On Fri, Jan 20, 2017 at 1:12 PM, Plamena Manolova
wrote:
> Since we already have the functionality in place and games
> like
Since we already have the functionality in place and games
like Game of Thrones seem to depend on this extension, I
think it makes sense to enable it by making it part of
the extension string even though its still a draft:
https://www.khronos.org/registry/gles/extensions/EXT/EXT_compressed_ETC1_RG
Enable getting the results of a transform feedback overflow query with a
buffer object.
v4:
- hsw_overflow_result_to_gpr0 a public function, so it can be used
by conditional render. (Kenneth)
- fix typo grp0/gpr0 (Kenneth)
- rename load_gen_written_data_to_regs to
load_over
On Fri, Jan 20, 2017 at 09:53:27AM -0800, Rafael Antognolli wrote:
> This extension adds new query types which can be used to detect overflow
> of transform feedback buffers. The new query types are also accepted by
> conditional rendering commands.
>
> v3:
> - s/gen7+/gen6+/ in the relnotes (
This extension adds new query types which can be used to detect overflow
of transform feedback buffers. The new query types are also accepted by
conditional rendering commands.
v3:
- s/gen7+/gen6+/ in the relnotes (Jordan Justen)
Signed-off-by: Rafael Antognolli
---
docs/features.txt
Add some basic types and storage for the queries of this extension.
v2:
- update date of extension (Kenneth)
Signed-off-by: Rafael Antognolli
---
src/mesa/main/extensions_table.h | 1 +
src/mesa/main/mtypes.h | 5 +
2 files changed, 6 insertions(+)
diff --git a/src/mesa/main/
Also update checks on conditional rendering.
Signed-off-by: Rafael Antognolli
---
src/mesa/main/condrender.c | 4 +++-
src/mesa/main/queryobj.c| 21 +
src/mesa/state_tracker/st_cb_queryobj.c | 6 ++
3 files changed, 30 insertions(+), 1 delet
When querying for transform feedback overflow on one or all of the
streams, store information about number of generated and written
primitives. Then check whether generated == written.
v2:
- use only SO_PRIM_STORAGE_NEEDED, do not fallback to
CL_INVOCATION_COUNT. (Kenneth)
Signed-off-by
This patch series implements the ARB_transform_feedback_overflow_query
extension for i965.
Changes for v4:
- Reuse of MI_MATH calcs from hsw_queryobj.c in brw_conditional_render.c
- Renamed a couple functions as suggested by Kenneth
- Fallback to CPU-side conditional rendering if MI_MA
Enable the use of a transform feedback overflow query with
glBeginConditionalRender. The render commands will only execute if the
query is true (i.e. if there was an overflow).
Use ARB_conditional_render_inverted to change this behavior.
v4:
- reuse MI_MATH calcs from hsw_queryob (Kenneth)
Cc: "13.0 17.0"
Reviewed-by: Jason Ekstrand
On Fri, Jan 20, 2017 at 9:31 AM, Lionel Landwerlin <
lionel.g.landwer...@intel.com> wrote:
> Blorp can deal with depth/stencil surfaces blits/copies without the
> render target requirement. Also having both render target and
> depth/stencil requiremen
Blorp can deal with depth/stencil surfaces blits/copies without the
render target requirement. Also having both render target and
depth/stencil requirement is incompatible from isl's point of view.
This fixes an image creation issue in the high level quality settings
of the Unity3D player, which r
On Fri, Jan 20, 2017 at 8:45 AM, Lionel Landwerlin <
lionel.g.landwer...@intel.com> wrote:
> Some applications might add location decoration to samplers. Rather
> than raising an error it seems it would make more sense to just
> discard these decorations.
>
That's annoying but not the end of the
On Fri, Jan 20, 2017 at 8:45 AM, Lionel Landwerlin <
lionel.g.landwer...@intel.com> wrote:
> Blorp can deal with depth/stencil surfaces blits/copies without the
> render target requirement. Also having both render target and
> depth/stencil requirement is incompatible from isl's point of view.
>
>
Hi,
While testing Unity3D demos, I've noticed a couple of issues with our
driver.
Cheers,
Lionel Landwerlin (2):
spirv: don't assert with location decorations on non i/o variables
anv: don't require render target isl bit for depth/stencil surfaces
src/compiler/spirv/vtn_variables.c | 3 ++-
Some applications might add location decoration to samplers. Rather
than raising an error it seems it would make more sense to just
discard these decorations.
Signed-off-by: Lionel Landwerlin
---
src/compiler/spirv/vtn_variables.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --g
Blorp can deal with depth/stencil surfaces blits/copies without the
render target requirement. Also having both render target and
depth/stencil requirement is incompatible from isl's point of view.
This fixes an image creation issue in the high level quality settings
of the Unity3D player, which r
On Fri, Jan 20, 2017 at 2:15 AM, Nicolai Hähnle wrote:
> Hi Rob,
>
> On 19.01.2017 23:32, Rob Clark wrote:
>
>> Just a friendly reminder that now would be a good time to update the
>> wiki page for GSoC/EVoC ideas:
>>
>> https://www.x.org/wiki/SummerOfCodeIdeas/
>>
>> There are currently still
All three are
Reviewed-by: Jason Ekstrand
On Fri, Jan 20, 2017 at 8:38 AM, Lionel Landwerlin <
lionel.g.landwer...@intel.com> wrote:
> Signed-off-by: Lionel Landwerlin
> ---
> src/compiler/spirv/vtn_variables.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/compi
On Thu, Jan 19, 2017 at 11:48 PM, Pohjolainen, Topi <
topi.pohjolai...@gmail.com> wrote:
> On Thu, Jan 19, 2017 at 01:39:49PM -0800, Jason Ekstrand wrote:
> >On Thu, Jan 19, 2017 at 12:40 PM, Francisco Jerez
> ><[1]curroje...@riseup.net> wrote:
> >
> > "Pohjolainen, Topi" <[2]topi.poh
Signed-off-by: Lionel Landwerlin
---
src/intel/blorp/blorp.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/intel/blorp/blorp.h b/src/intel/blorp/blorp.h
index ff60567fc4..e0687e9261 100644
--- a/src/intel/blorp/blorp.h
+++ b/src/intel/blorp/blorp.h
@@ -29,7 +29,6 @@
#include "isl/isl.
Signed-off-by: Lionel Landwerlin
---
src/compiler/spirv/vtn_variables.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/compiler/spirv/vtn_variables.c
b/src/compiler/spirv/vtn_variables.c
index 61a3701e43..a543a52dd0 100644
--- a/src/compiler/spirv/vtn_variables.c
+++ b/s
Signed-off-by: Lionel Landwerlin
---
src/intel/vulkan/anv_descriptor_set.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/anv_descriptor_set.c
b/src/intel/vulkan/anv_descriptor_set.c
index 2926e7a259..4367c0ebe0 100644
--- a/src/intel/vulkan/anv_descriptor_s
https://bugs.freedesktop.org/show_bug.cgi?id=99446
yann changed:
What|Removed |Added
Status|RESOLVED|CLOSED
--
You are receiving this mail because:
Y
https://bugs.freedesktop.org/show_bug.cgi?id=99446
Jason Ekstrand changed:
What|Removed |Added
Resolution|--- |NOTOURBUG
Status|NEEDINFO
On 16 January 2017 at 18:30, Kenneth Graunke wrote:
> On Monday, January 16, 2017 3:45:32 PM PST Emil Velikov wrote:
>> From: Emil Velikov
>>
>> Otherwise we might end up w/o the respective folder (depending on
>> autotools version) and fail at build time.
>>
>> Fixes: bfd17c76c12 "i965: Port INT
1 - 100 of 116 matches
Mail list logo