Signed-off-by: Ilia Mirkin
Cc: "12.0"
---
src/gallium/drivers/nouveau/nvc0/nvc0_compute.c| 3 +++
src/gallium/drivers/nouveau/nvc0/nvc0_state_validate.c | 3 +++
src/gallium/drivers/nouveau/nvc0/nve4_compute.c| 3 +++
3 files changed, 9 insertions(+)
diff --git a/src/gallium/dri
Signed-off-by: Ilia Mirkin
Cc: "12.0"
---
src/gallium/drivers/nouveau/nvc0/nvc0_context.c | 25 -
1 file changed, 16 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_context.c
b/src/gallium/drivers/nouveau/nvc0/nvc0_context.c
index 98e787
On Thursday, June 2, 2016 5:00:51 PM PDT Ian Romanick wrote:
> On 06/02/2016 02:09 PM, Kenneth Graunke wrote:
> > i965 has no special hardware for this, so the best way to implement
> > this is to pass it in via a uniform.
> >
> > Signed-off-by: Kenneth Graunke
> > Cc: mesa-sta...@lists.freedeskt
ping?
On 10 May 2016 at 13:59, Dave Airlie wrote:
> From: Dave Airlie
>
> The clamping code always clamps to 0..1, which for SNORM is
> incorrect. This adds a bit to say that clamping is for
> an snorm and to use the correct range.
>
> This fixes a number of SNORM cases in:
> GL33-CTS.texture_s
Reviewed-by: Roland Scheidegger
I actually had fixed that at some point. But it looks like it came back...
Roland
Am 03.06.2016 um 04:27 schrieb Dave Airlie:
> From: Dave Airlie
>
> This fixes:
> GL45-CTS.geometry_shader.layered_rendering.layered_rendering
>
> on Skylake.
>
> Signed-off-by:
ChrisF suggested I do the buffer stuff like intel does - based on
usage hints. This prevents a lot of unnecessary revalidation. I tend
to agree - I'll resend this with updates. (Of course there's a missing
USAGE flag for vertex arrays, but I'll fix that.)
On Thu, Jun 2, 2016 at 8:31 PM, Ilia Mirki
From: Dave Airlie
For 3D textures we shouldn't be using NumLayers, we need
to get it from the depth.
This fixes:
GL45-CTS.geometry_shader.layered_framebuffer.clear_call_support
Signed-off-by: Dave Airlie
---
src/mesa/drivers/dri/i965/intel_fbo.c | 2 +-
1 file changed, 1 insertion(+), 1 delet
Reviewed-by: Chris Forbes
On Fri, Jun 3, 2016 at 2:27 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> This fixes:
> GL45-CTS.geometry_shader.layered_rendering.layered_rendering
>
> on Skylake.
>
> Signed-off-by: Dave Airlie
> ---
> src/mesa/main/get_hash_params.py | 2 +-
> 1 file changed, 1
From: Dave Airlie
This fixes:
GL45-CTS.geometry_shader.layered_rendering.layered_rendering
on Skylake.
Signed-off-by: Dave Airlie
---
src/mesa/main/get_hash_params.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/get_hash_params.py b/src/mesa/main/get_hash_
On Thu, Jun 2, 2016 at 5:01 PM, Ian Romanick wrote:
> I believe this is what GL meta blits do also. This patch is
>
> Reviewed-by: Ian Romanick
>
Thanks!
> On 06/02/2016 04:36 PM, Jason Ekstrand wrote:
> > When upscaling you can end up interpolating between the edge pixel and
> one
> > past
From: Dave Airlie
"all geometry shader output vertex count declarations in a
program must declare the same count."
Fixes:
GL45-CTS.geometry_shader.output.conflicted_output_vertices_max
Cc: "11.2 12.0"
Signed-off-by: Dave Airlie
---
src/compiler/glsl/ast.h | 3 ++-
src/compi
Both textures and buffers can have their underlying resources
reallocated with a glTexImage*() or glBufferData(). Atoms that reference
these need to be notified that a change to the underlying resource has
occurred in order to be recreated. This means textures (which can
reference a buffer object),
On Thu, Jun 02, 2016 at 04:27:26PM -0700, Jason Ekstrand wrote:
> This was causing problems if the user tried to copy to/from the stencil
> portion of a combined depth/stencil image.
>
> Signed-off-by: Jason Ekstrand
> Cc: Nanley Chery
> ---
> src/intel/vulkan/anv_meta_copy.c | 34 +
I believe this is what GL meta blits do also. This patch is
Reviewed-by: Ian Romanick
On 06/02/2016 04:36 PM, Jason Ekstrand wrote:
> When upscaling you can end up interpolating between the edge pixel and one
> past the edge. Using CLAMP_TO_EDGE seems like the most reasonable thing to
> do in
On 06/02/2016 02:09 PM, Kenneth Graunke wrote:
> i965 has no special hardware for this, so the best way to implement
> this is to pass it in via a uniform.
>
> Signed-off-by: Kenneth Graunke
> Cc: mesa-sta...@lists.freedesktop.org
> ---
> src/compiler/glsl/builtin_variables.cpp | 13
On Thu, Jun 2, 2016 at 8:25 PM, Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/common/meta_blit.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/common/meta_blit.c
> b/src/mesa/drivers/common/meta_blit.c
> index 20d3215..d
When upscaling you can end up interpolating between the edge pixel and one
past the edge. Using CLAMP_TO_EDGE seems like the most reasonable thing to
do in this case. This fixes two of the new Vulkan CTS tests in
dEQP-VK.api.copy_and_blit.blit_image.*
Jason Ekstrand
Cc: "12.0"
---
src/intel/v
This was causing problems if the user tried to copy to/from the stencil
portion of a combined depth/stencil image.
Signed-off-by: Jason Ekstrand
Cc: Nanley Chery
---
src/intel/vulkan/anv_meta_copy.c | 34 +-
1 file changed, 17 insertions(+), 17 deletions(-)
diff
On Wed, Jun 1, 2016 at 11:59 AM, Anuj Phogat wrote:
>
> On Wed, Jun 1, 2016 at 9:31 AM, Plamena Manolova
> wrote:
> >
> > When validating attributes during surface creation we should account
> > for the default values of texture target and format (EGL_NO_TEXTURE)
> > since the user is not obligat
R-B
On Wed, Jun 1, 2016 at 5:17 PM, Francisco Jerez
wrote:
> The conditional mod of these instructions determines the semantics of
> the comparison itself (rather than being evaluated based on the result
> of the instruction as is usually the case for most other instructions
> that allow conditi
ping?
On 24 May 2016 at 06:43, Dave Airlie wrote:
> From: Dave Airlie
>
> The GL4.5 spec quote seems clear on this:
> "The value -1 will be returned by either command if an error occurs,
> if name does not identify an active variable on programInterface,
> or if name identifies an active variab
On Tue, May 31, 2016 at 9:20 AM, Emil Velikov wrote:
> On 30 May 2016 at 23:20, Mauro Rossi wrote:
>> Fixes the following building error:
>>
>> target C++: libmesa_glsl <= external/mesa/src/compiler/glsl/glsl_to_nir.cpp
>> In file included from external/mesa/src/compiler/glsl/glsl_to_nir.h:28:0,
On Thu, Jun 2, 2016 at 2:15 PM, Mauro Rossi wrote:
> isl library is needed to build i965, libmesa_isl static library is added
> to fix related Android building errors.
>
> Any attempt to build libmesa_genxml as phony package module failed to deliver
> gen{7,75,8,9}_pack.h generated headers, needed
On Thu, Jun 2, 2016 at 2:15 PM, Mauro Rossi wrote:
> Source files should be defined only once in Makefile.source
> As a side note isl_surface.c is not an unneeded duplicate
> and must stay defined in ISL_GEN{7,75,8,9}_FILES
I built against AOSP master.
Tested-by: Rob Herring
> ---
> src/intel
On 02.06.2016 23:54, Rob Herring wrote:
On Thu, Jun 2, 2016 at 4:45 PM, Tobias Klausmann
wrote:
Starting with commit e8b38ca202fbe8c281aeb81a4b64256983f185e0 the dummy_thread
misses the proper intialization for the new member, leading to the error stated
at the bottom. This patch initializes
On Thu, Jun 2, 2016 at 4:45 PM, Tobias Klausmann
wrote:
> Starting with commit e8b38ca202fbe8c281aeb81a4b64256983f185e0 the dummy_thread
> misses the proper intialization for the new member, leading to the error
> stated
> at the bottom. This patch initializes the new member.
Just hit this too (
On Thu, Jun 2, 2016 at 2:41 PM, Kristian Høgsberg wrote:
> On Thu, Jun 2, 2016 at 11:52 AM, Rob Herring wrote:
>> On Thu, Jun 2, 2016 at 1:31 PM, Rob Clark wrote:
>>> On Thu, Jun 2, 2016 at 1:51 PM, Rob Herring wrote:
As discussed on irc yesterday, I've been looking at adding YUV planar
>>
Starting with commit e8b38ca202fbe8c281aeb81a4b64256983f185e0 the dummy_thread
misses the proper intialization for the new member, leading to the error stated
at the bottom. This patch initializes the new member.
Signed-off-by: Tobias Klausmann
CC main/eglcurrent.lo
In file included from
On Thu, Jun 2, 2016 at 11:52 AM, Rob Herring wrote:
> On Thu, Jun 2, 2016 at 1:31 PM, Rob Clark wrote:
>> On Thu, Jun 2, 2016 at 1:51 PM, Rob Herring wrote:
>>> As discussed on irc yesterday, I've been looking at adding YUV planar
>>> (YV12) to GBM as it is a requirement for Android gralloc. It
Signed-off-by: Jason Ekstrand
Cc: "12.0"
---
src/compiler/spirv/spirv_to_nir.c | 16
src/compiler/spirv/vtn_private.h | 3 +++
2 files changed, 19 insertions(+)
diff --git a/src/compiler/spirv/spirv_to_nir.c
b/src/compiler/spirv/spirv_to_nir.c
index d51089f..e8aea73 100644
-
glslang frequently throw bogus decorations into shaders. While we are free
to assert-fail, it's a bit nicer to the application to just warn.
Signed-off-by: Jason Ekstrand
Cc: "12.0"
---
src/compiler/spirv/spirv_to_nir.c | 38 --
src/compiler/spirv/vtn_varia
Previously we supported a subset of capabilities and just left a default
case for the others. It's time to stop being lazy and actually audit the
capabilities. This should bring them up-to-date with reality.
Signed-off-by: Jason Ekstrand
Cc: "12.0"
---
src/compiler/spirv/spirv_to_nir.c | 48 +
Signed-off-by: Jason Ekstrand
Cc: "12.0"
---
src/compiler/Makefile.sources | 2 +
src/compiler/spirv/spirv_info.c | 150
src/compiler/spirv/spirv_info.h | 27
3 files changed, 179 insertions(+)
create mode 100644 src/compiler/spirv/spirv_in
Getting rid of the default case makes the compiler warn if we are missing
cases. While we're here, we also add the one missing case.
Signed-off-by: Jason Ekstrand
Cc: "12.0"
---
src/compiler/spirv/spirv_to_nir.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/compile
On 3 June 2016 at 07:09, Kenneth Graunke wrote:
> We've had some trouble in the past with copying integers around via
> float pointers, as the C compiler sometimes uses x87 floating point
> registers to load values on 32-bit systems. Passing the
> gl_constant_value union should be safer.
>
> To a
We still need to recompile the passthrough shader when this value
changes, as it also affects the output vertex count. But otherwise,
we can eliminate recompiles on Gen8+.
We probably want to do this for Gen7 as well, but that requires
rewriting the input release code to use a loop, which is a tr
We've had some trouble in the past with copying integers around via
float pointers, as the C compiler sometimes uses x87 floating point
registers to load values on 32-bit systems. Passing the
gl_constant_value union should be safer.
To avoid churn, this patch creates a "GLfloat *value" variable s
Fixes GL44-CTS.tessellation_shader.single.max_patch_vertices,
which uses gl_PatchVerticesIn in the TES, but doesn't link against
a TCS (which would allow the linker to lower it to a constant).
Signed-off-by: Kenneth Graunke
Cc: mesa-sta...@lists.freedesktop.org
---
src/mesa/drivers/dri/i965/brw_
i965 has no special hardware for this, so the best way to implement
this is to pass it in via a uniform.
Signed-off-by: Kenneth Graunke
Cc: mesa-sta...@lists.freedesktop.org
---
src/compiler/glsl/builtin_variables.cpp | 13 -
src/mesa/main/mtypes.h | 1 +
src/mesa/p
i965 has no special hardware for this, so we need to pass this value in
as a uniform (unless the TES is linked against a TCS, in which case the
linker can just replace this with a constant).
Signed-off-by: Kenneth Graunke
Cc: mesa-sta...@lists.freedesktop.org
---
src/compiler/glsl/linker.cpp
Reviewed-by: Marek Olšák
Marek
On Thu, Jun 2, 2016 at 10:23 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> When a shader image view into a buffer texture can be written to, the buffer's
> valid range must be updated, or subsequent transfers may incorrectly skip
> synchronization.
>
> Thi
On 02.06.2016 22:53, Ilia Mirkin wrote:
On Thu, Jun 2, 2016 at 4:50 PM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
Found by inspection after seeing
https://bugs.freedesktop.org/show_bug.cgi?id=96343
---
src/mesa/state_tracker/st_cb_readpixels.c | 2 ++
1 file changed, 2 insertions(+)
diff
On Thu, Jun 2, 2016 at 4:50 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Found by inspection after seeing
> https://bugs.freedesktop.org/show_bug.cgi?id=96343
> ---
> src/mesa/state_tracker/st_cb_readpixels.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/mesa/state_track
From: Nicolai Hähnle
Found by inspection after seeing
https://bugs.freedesktop.org/show_bug.cgi?id=96343
---
src/mesa/state_tracker/st_cb_readpixels.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/state_tracker/st_cb_readpixels.c
b/src/mesa/state_tracker/st_cb_readpixels.c
inde
On Thu, Jun 2, 2016 at 2:52 PM, Rob Herring wrote:
> On Thu, Jun 2, 2016 at 1:31 PM, Rob Clark wrote:
>> On Thu, Jun 2, 2016 at 1:51 PM, Rob Herring wrote:
>>> As discussed on irc yesterday, I've been looking at adding YUV planar
>>> (YV12) to GBM as it is a requirement for Android gralloc. It i
From: Nicolai Hähnle
When a shader image view into a buffer texture can be written to, the buffer's
valid range must be updated, or subsequent transfers may incorrectly skip
synchronization.
This fixes a bug that was exposed in Xephyr by PBO acceleration for
glReadPixels,
reported by Michel Dän
This patch is
Reviewed-by: Ian Romanick
On 06/01/2016 11:38 PM, Alejandro Piñeiro wrote:
> For ES 3.0 NUM_SAMPLE_COUNTS spec points that some formats will be
> always zero. But on ES 3.1 can be different to zero.
>
> The current code is correctly checking exactly against version 3.0,
> but the
To avoid blocking other EGL calls, release the display mutex before
we enqueue buffer to android frameworks and re-acquire the mutex
upon return.
v2: moved lock/unlock inside droid_window_enqueue_buffer().
TEST=verify pinch zoom in Photos app no longer causes hangs
Signed-off-by: Haixia Shi
---
As agreed with Emil, I'm sending a respin of isl for Android,
without python3 macro definition in main Android.mk, since not needed anymore,
and with a few optimizations in makefiles.
In order to backport to mesa-stable the patches, which is recommended,
the patches that remove python3 features fr
Source files should be defined only once in Makefile.source
As a side note isl_surface.c is not an unneeded duplicate
and must stay defined in ISL_GEN{7,75,8,9}_FILES
---
src/intel/isl/Makefile.sources | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/intel/isl/Makefile.sources b/src/intel/is
isl library is needed to build i965, libmesa_isl static library is added
to fix related Android building errors.
Any attempt to build libmesa_genxml as phony package module failed to deliver
gen{7,75,8,9}_pack.h generated headers, needed for libmesa_isl_gen{7,75,8,9}
Due to constraints in Android
On Thu, Jun 2, 2016 at 11:34 AM, Anuj Phogat wrote:
> Piglit test ext_framebuffer_multisample_blit_scaled-blit-scaled
> (with added 16x sample support) now passes with this patch.
>
> Signed-off-by: Anuj Phogat
> Cc: Jason Ekstrand
> Cc: "12.0"
>
Reviewed-by: Jason Ekstrand
Thanks for fixin
On Thu, Jun 2, 2016 at 1:31 PM, Rob Clark wrote:
> On Thu, Jun 2, 2016 at 1:51 PM, Rob Herring wrote:
>> As discussed on irc yesterday, I've been looking at adding YUV planar
>> (YV12) to GBM as it is a requirement for Android gralloc. It is
>> possible to disable the requirement as that is what
Acked-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Piglit test ext_framebuffer_multisample_blit_scaled-blit-scaled
(with added 16x sample support) now passes with this patch.
Signed-off-by: Anuj Phogat
Cc: Jason Ekstrand
Cc: "12.0"
---
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 17 ++---
1 file changed, 10 insertions(+), 7 dele
On Thu, Jun 2, 2016 at 1:51 PM, Rob Herring wrote:
> As discussed on irc yesterday, I've been looking at adding YUV planar
> (YV12) to GBM as it is a requirement for Android gralloc. It is
> possible to disable the requirement as that is what the Android
> emulator and android-x86 do. But that res
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/common/meta_blit.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/common/meta_blit.c
b/src/mesa/drivers/common/meta_blit.c
index 20d3215..d6d3a42 100644
--- a/src/mesa/drivers/common/meta_blit.c
+++ b/src/m
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/common/meta_blit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/common/meta_blit.c
b/src/mesa/drivers/common/meta_blit.c
index 20d3215..01abaf4 100644
--- a/src/mesa/drivers/common/meta_blit.c
+++ b/src/mesa/
Hello.
The patch has moved to
http://lists.freedesktop.org/archives/mesa-dev/2016-June/119257.html (
http://patchwork.freedesktop.org/series/8176/)
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/me
As discussed on irc yesterday, I've been looking at adding YUV planar
(YV12) to GBM as it is a requirement for Android gralloc. It is
possible to disable the requirement as that is what the Android
emulator and android-x86 do. But that results in un-optimized s/w CSC.
Given most/all Android target
LTO compilation can sometimes fail with GCC 4.9 and GCC 5.3 because
src/mapi uses unusual mixing of C code and assembly code. The issue
may be present in case of GCC 6.1 as well.
This is a Mesa bug rather than a compiler bug (although in an ideal
world the compilation with -flto should fail if and
Reviewed-by: Nicolai Hähnle
On 02.06.2016 05:42, Dave Airlie wrote:
From: Dave Airlie
This test was only happening for textures, but there is
nothing in the spec to say this, so test it for all cases.
This fixes:
GL45-CTS.copy_image.invalid_target
Signed-off-by: Dave Airlie
---
src/mesa/
On jue, 2016-06-02 at 14:25 +1000, Dave Airlie wrote:
> From: Dave Airlie
>
> The specs says INVALID_VALUE for exceeding dimensions,
> which is really what is happening here.
>
> This fixes:
> GL45-CTS.copy_image.non_existent_mipmap
>
> Signed-off-by: Dave Airlie
> ---
> src/mesa/main/copyima
Hi Michel,
thanks for letting me know. I can reproduce this on my Tonga, will look
into it soon.
Cheers,
Nicolai
On 02.06.2016 15:12, Michel Dänzer wrote:
On 02.06.2016 05:44, Nicolai =?UNKNOWN?Q?H=C3=A4hnle?= wrote:
Module: Mesa
Branch: master
Commit: ed0e9862c56d13381f0414b5a835fd31960edf
Some R9 390 results:
Tomb Raider (normal settings): 80 -> 88 FPS
Talos Principle (custom settings): 23 -> 56 FPS
Metro Last Light Redux (default benchmark settings): 39 -> 40 FPS
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.free
Acked-by: Marek Olšák
Marek
On Thu, Jun 2, 2016 at 4:15 PM, Rob Clark wrote:
> From: Rob Clark
>
> Unused, and fixes a couple of coverity warnings: CID 1362171, 1362170
>
> Signed-off-by: Rob Clark
> ---
> src/gallium/auxiliary/Makefile.sources | 2 -
> src/gallium/auxiliary/util/u_staging
From: Rob Clark
Unused, and fixes a couple of coverity warnings: CID 1362171, 1362170
Signed-off-by: Rob Clark
---
src/gallium/auxiliary/Makefile.sources | 2 -
src/gallium/auxiliary/util/u_staging.c | 136 -
src/gallium/auxiliary/util/u_staging.h | 67 --
Instead of changing the format on the existing template
which makes error handling not nice and confuses coverity.
CoverityID: 1337953
Signed-off-by: Julien Isorce
---
src/gallium/state_trackers/va/image.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/s
Signed-off-by: Julien Isorce
---
src/gallium/state_trackers/va/buffer.c | 8 +---
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/src/gallium/state_trackers/va/buffer.c
b/src/gallium/state_trackers/va/buffer.c
index 2fd8661..7d3167b 100644
--- a/src/gallium/state_trackers/va/buf
In order to do zero-copy between two different devices
the memory should not be tiled.
This is currently no way to set pipe_resource template's flag
from pipe_video_buffer template. So disabled_tiling is added.
Choosed "disable" prefix so that CALLOC keeps tiling enabled
by default.
Tested with
Reviewed-by: Iago Toral Quiroga
On Wed, 2016-06-01 at 21:25 -0700, Kenneth Graunke wrote:
> Signed-off-by: Kenneth Graunke
> Cc: "12.0"
> ---
> src/mesa/drivers/dri/i965/gen6_sf_state.c | 2 +-
> src/mesa/drivers/dri/i965/gen7_sf_state.c | 2 +-
> src/mesa/drivers/dri/i965/gen8_sf_state.c | 2
I don't think that's right. Have a look at my st/Mesa patch which fixed the
same thing. The issue is that es2 always enables point size from shader, so
you need to apply that sort of logic. However on desktop, you still have to
respect the GL program point size setting.
On Jun 2, 2016 12:26 AM, "Ke
On 02.06.2016 05:44, Nicolai =?UNKNOWN?Q?H=C3=A4hnle?= wrote:
> Module: Mesa
> Branch: master
> Commit: ed0e9862c56d13381f0414b5a835fd31960edfe6
> URL:
> http://cgit.freedesktop.org/mesa/mesa/commit/?id=ed0e9862c56d13381f0414b5a835fd31960edfe6
>
> Author: Nicolai Hähnle
> Date: Tue Apr 26 1
On 02.06.2016 00:35, Marek =?UNKNOWN?B?T2zFocOhaw==?= wrote:
> Module: Mesa
> Branch: master
> Commit: fc1479a95432f291623fa5bafe524701e77af3ca
> URL:
> http://cgit.freedesktop.org/mesa/mesa/commit/?id=fc1479a95432f291623fa5bafe524701e77af3ca
>
> Author: Marek Olšák
> Date: Thu May 12 13:33
On 02.06.2016 00:35, Marek =?UNKNOWN?B?T2zFocOhaw==?= wrote:
> Module: Mesa
> Branch: master
> Commit: 95c5bbae66af3ca1f805d94f6fe8d8e4ba2c9c43
> URL:
> http://cgit.freedesktop.org/mesa/mesa/commit/?id=95c5bbae66af3ca1f805d94f6fe8d8e4ba2c9c43
>
> Author: Marek Olšák
> Date: Tue May 17 20:46
no problem, it was actually good, I think that even if current code is
correct, I'll still submit
the new diff for review because it looks better.
On 2 June 2016 at 10:34, Eric Engestrom wrote:
> On Tue, May 31, 2016 at 02:54:06PM +0200, Christian König wrote:
> > On the other hand the original
On Thu, Jun 2, 2016 at 12:44 AM, Jason Ekstrand wrote:
> Signed-off-by: Jason Ekstrand
> Cc: "12.0"
> Cc: Ian Romanick
> ---
> src/intel/vulkan/gen7_pipeline.c | 12 ++--
> src/intel/vulkan/gen8_pipeline.c | 12 ++--
> src/intel/vulkan/genX_pipeline_util.h | 30 ++
On Tue, May 31, 2016 at 02:54:06PM +0200, Christian König wrote:
> On the other hand the original code was correct in the first place.
>
> And as Julien noted Coverity tries to restore the template buffer format
> from a local variable which doesn't exists any more where you wanted to add
> the li
On Tue, May 31, 2016 at 06:33:53AM -0600, Brian Paul wrote:
> On 05/30/2016 07:26 PM, Eric Engestrom wrote:
> > These two lines have been here since the file was created.
> > I'm guessing the second one was just for testing during dev, so it's the
> > one that's going away.
> >
> > CoverityID: 129
Am 02.06.2016 um 08:41 schrieb Nayan Deshmukh:
Check for null pointer before accessing arrays in get/put bits
native/YCbCr/Indexed in VdpOutputSurface and VdpVideoSurface.
Signed-off-by: Nayan Deshmukh
Reviewed and pushed.
Thanks for the help,
Christian.
---
src/gallium/state_trackers/vd
This fix getting the size of a struct arg. vec3 types still work ok.
Only buit-in args need to have power of two alignment, getTypeAllocSize
reports the correct size.
Cc: 12.0
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
81 matches
Mail list logo
