Mesa (13.0): 50 new commits
Emil Velikov
evelikov at kemper.freedesktop.org
Fri Nov 11 18:13:51 UTC 2016
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3bb0415ab96f74183f7aa58c1a543448653ccb3e
Author: Emil Velikov <emil.velikov at collabora.com>
Date: Wed Nov 9 18:11:27 2016 +0000
radv: Suffix the radeon_icd file with the host CPU
Port of the anv commit d96345de989 ("anv: Suffix the intel_icd file with
the host CPU").
v2: s/intel_icd/radeon_icd/ in commit summary (Gražvydas)
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
Reviewed-by: Dave Airlie <airlied at redhat.com> (IRC)
(cherry picked from commit 0f434a68a39f391795dff92c317349d28dd2e638)
Squashed with commit:
radv: automake: list correct file in the EXTRA_DIST
Earlier commit renamed the file radeon_icd.json{,.in} but missed one
reference of the file - in EXTRA_DIST.
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Fixes: 0f434a68a ("radv: Suffix the radeon_icd file with the host CPU")
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
(cherry picked from commit b359f62456211c2162109064cb504ad7de2ee799)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a4bc03fdfe998fd09aaefef4964bf9359c4c5399
Author: Emil Velikov <emil.velikov at collabora.com>
Date: Wed Nov 9 18:10:47 2016 +0000
radv: use correct .specVersion for extensions
Analogous to previous commit.
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
Reviewed-by: Dave Airlie <airlied at redhat.com> (IRC)
(cherry picked from commit abe110df019af230bd3d29fb73c046fc404b5525)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d8eea6312119090a460865558fed511aef3e00da
Author: Emil Velikov <emil.velikov at collabora.com>
Date: Wed Nov 9 18:10:46 2016 +0000
anv: use correct .specVersion for extensions
Vulkan has introduced the consept of .specVersion which can be used to
attribute changes of the said extension.
The current loader does not check the value, thus it have gone unnoticed
that the driver exposes an old version of the following extensions:
VK_KHR_xcb_surface (Rev 6)
VK_KHR_xlib_surface (Rev 6)
VK_KHR_wayland_surface (Rev 5)
- Updated the surface create function to take a pCreateInfo structure
VK_KHR_swapchain (Rev 68)
- Moved the "validity" include for vkAcquireNextImage to be in its proper
place, after the prototype and list of parameters.
...
According to the documentation:
* pname:specVersion is the version of this extension.
It is an integer, incremented with backward compatible changes.
Based on the history of vk.xml the above (latest) revision has been
available since Vulkan 1.0 so even if they were any backwards
incompatible change(s) [as hinted by the revision log] those should be
safe.
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
(cherry picked from commit f373a91a5264bb93b31b5be9f9714da856383ed1)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3e616f77bd284360dc793c0d5c5affd76ed28003
Author: Emil Velikov <emil.velikov at collabora.com>
Date: Wed Nov 2 15:32:22 2016 +0000
amd/addrlib: limit fastcall/regparm to GCC i386
The use of regparm causes an error on arm/arm64 builds with clang.
fastcall is allowed, but still throws a warning. As both options only
have effect on 32-bit x86 builds, limit them to that case.
v2: keep the __i386__ within GCC (Nicolai)
Cc: 13.0 <mesa-stable at lists.freedesktop.org>
Cc: Rob Herring <robh at kernel.org>
Cc: Nicolai Hähnle <nhaehnle at gmail.com>
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
Reviewed-by: Rob Herring <robh at kernel.org>
(cherry picked from commit 190bae7685e230d8fb4e9a0f9ce4a6aa3c7643f5)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=49e093a2f58650374c2efe9cc6a5ab03cf52a48c
Author: Dave Airlie <airlied at redhat.com>
Date: Wed Nov 9 01:21:30 2016 +0000
radv: fix GetFenceStatus for signaled fences
if a fence is created pre-signaled we should return that
in GetFenceStatus even if it hasn't been submitted.
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Reviewed-by: Gustaw Smolarczyk <wielkiegie at gmail.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit fb50245ac1e1eb60b2451d9d6dd05388cb89634a)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=2bbf964af84c3220245c1534951341fff4b4ee44
Author: Dave Airlie <airlied at redhat.com>
Date: Wed Nov 2 01:23:11 2016 +0000
radv: enable conditional discard optimisation on radv.
This fixes a bunch of GPU hangs introduced in some CTS
tests like
dEQP-VK.memory.pipeline_barrier.host_write_uniform_buffer.65536
It works around an issue seen in the LLVM backend, but
also makes the radv code work more like the radeonsi stack.
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit 3c9af7578fcd62fc55e0443733f56b2aaa50ba9c)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=fa6c02787e1c4d650b7aaa528a7c4e5e129e2906
Author: Dave Airlie <airlied at redhat.com>
Date: Wed Nov 2 01:22:07 2016 +0000
nir: add conditional discard optimisation (v4)
This is ported from GLSL and converts
if (cond)
discard;
into
discard_if(cond);
This removes a block, but also is needed by radv
to workaround a bug in the LLVM backend.
v2: handle if (a) discard_if(b) (nha)
cleanup and drop pointless loop (Matt)
make sure there are no dependent phis (Eric)
v3: make sure only one instruction in the then block.
v4: remove sneaky tabs, add cursor init (Eric)
Reviewed-by: Eric Anholt <eric at anholt.net>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit b16dff2d88302e5113598a818d2f92f8af02cd79)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a65b6e12f357c9a3022f9d3954002254c7f3ab6e
Author: Dave Airlie <airlied at redhat.com>
Date: Wed Nov 2 01:21:15 2016 +0000
ac/nir: add support for discard_if intrinsic (v2)
We are going to start lowering to this in NIR code,
so prepare radv for it.
v2: handle conversion to kilp properly (nha)
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit dd77faeca2819fc78a72f71ca9b996c209378cde)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ce555a7d1ff436267d4606cd48d2d17d36335311
Author: Kristian Høgsberg Kristensen <kristian.h.kristensen at intel.com>
Date: Tue Mar 8 15:31:47 2016 -0800
anv: Do relocations in userspace before execbuf ioctl
Since our surface state buffer is shared by all batches, the kernel does a
full stall and sync with the CPU between batches every time we call
execbuf2 because it refuses to do relocations on an active buffer. Doing
them in userspace and passing the NO_RELOC flag to the kernel allows us to
perform the relocations without stalling.
This improves the performance of Dota 2 by around 30% on a Sky Lake GT2.
v2 (Jason Ekstrand):
- Better comments (Chris Wilson)
- Fixed write_reloc for correct canonical form (Chris Wilson)
v3 (Jason Ekstrand):
- Skip relocations which aren't needed
- Provide an environment variable to always use the kernel
- More comments about correctness (Chris Wilson)
v4 (Jason Ekstrand):
- More comments (Chris Wilson)
v5 (Jason Ekstrand):
- Rebase on top of moving execbuf2 setup go QueueSubmit
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit b3a29f2e9eb528a16ceec0fd88aad9f0c3c3b6d4)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=621b04873436e5875394cbe4d8733873e0c01d8e
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Sat Nov 5 19:47:33 2016 -0700
anv: Move relocation handling from EndCommandBuffer to QueueSubmit
Ever since the early days of the Vulkan driver, we've been setting up the
lists of relocations at EndCommandBuffer time. The idea behind this was to
move some of the CPU load out of QueueSubmit which the client is required
to lock around and into command buffer building which could be done in
parallel. Then QueueSubmit basically just becomes a bunch of execbuf2
calls.
Technically, this works. However, when you start to do more in QueueSubmit
than just execbuf2, you start to run into problems. In particular, if a
block pool is resized between EndCommandBuffer and QueueSubmit, the list of
anv_bo's and the execbuf2 object list can get out of sync. This can cause
problems if, for instance, you wanted to do relocations in userspace.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 8b61c57049ff75766715ad4f7b1ad2d3657b9b4d)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=039a03d8d2073d2f4234c02376e082e7f107d284
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Mon Nov 7 08:07:43 2016 -0800
anv/batch: Move last_ss_pool_bo_offset to the command buffer
The original reason for putting it in the batch_bo was to allow primaries
to share it across secondaries or something like that. However, the
relocation lists in secondary command buffers are are always left alone and
copied into the primary command buffer's relocation list. This means that
the offset really applies at the command buffer level and putting it in the
batch_bo doesn't make sense. This fixes a couple of potential bugs around
re-submission of command buffers that are not likely to be hit but are bugs
none the less.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 595400d57745fba198b42d95f3c4f5d855023c33)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c64f655408504892c7df63995ba56904bcd6623e
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Sat Nov 5 19:01:44 2016 -0700
anv: Add an anv_execbuf helper struct
This commit adds a little helper struct for storing everything we use to
build an execbuf2 call. Since the add_bo function really has nothing to do
with a command buffer, it makes sense to break it out a bit. This also
reduces some of the churn in the next commit.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 0fe68294275a4ab1d9f71bb48d21a5e6b830e1ca)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d22958eecb09e1abce115b959314640c23c39df0
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Wed Nov 2 10:42:45 2016 -0700
anv/batch_chain: Improve write_reloc
The old version wasn't properly handling large addresses where we have to
sign-extend to get it into the "canonical form" expected by the hardware.
Also, the new version is capable of doing a clflush of the newly written
reloc if requested.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 095c48a496fccdd95821ee426d70674bc75dc6af)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ab3aeab2972cb4f36ec99e42ca7dec80f0d27043
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Mon Oct 31 20:25:08 2016 -0700
anv: Initialize anv_bo::offset to -1
Since -1 is an invalid GPU address, this lets us know whether or not we
have a valid address for a buffer. We don't get a valid address until the
first time that buffer is used in an execbuf2 ioctl.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit d46bfb629725a5b8c327f3bd7f76d04f5ae262aa)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=5bdd4fc273176bbbfb9adfe6cc397a6b0525c1d2
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 13:10:11 2016 -0700
anv/allocator: Simplify anv_scratch_pool
The previous implementation was being overly clever and using the
anv_bo::size field as its mutex. Scratch pool allocations don't happen
often, will happen at most a fixed number of times, and never happen in the
critical path (they only happen in shader compilation). We can make this
much simpler by just using the device mutex. This also means that we can
start using anv_bo_init_new directly on the bo and avoid setting fields
one-at-a-time.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit bd0f8d50706fce400ff0768c659acc90696aadb6)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c4643f5f1eb431e6780ec263dbdfe21796b0bf5d
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 13:09:36 2016 -0700
anv: Add a new bo_pool_init helper
This ensures that we're always setting all of the fields in anv_bo
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 6283b6d56a2bb731cfcb4c876566901075f9bd34)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ceefe979c6b8c37d7166da5beefc9334fdabaa43
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 07:21:00 2016 -0700
anv: Don't presume to know what address is in a surface relocation
Because our relocation processing happens at EndCommandBuffer time and
because RENDER_SURFACE_STATE objects may be shared by batches, we really
have no clue whatsoever what address is actually written to the relocation
offset in the BO. We need to stop making such claims to the kernel and
just let it relocate for us.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit ba1eea4f957ca068eceea121bc3a70e2fe07873d)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9eca84e052b9c5080c54b083fc637428b53338ec
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Wed Nov 2 10:33:54 2016 -0700
anv: Add a cmd_buffer_execbuf helper
This puts the actual execbuf2 call in anv_batch_chain.c along with the
other relocation stuff.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit db9f4b2a2bbf1aff3c6c878735495fc7accbb11e)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=6f55a66ad325f2484687f0bee7d1db7d7e499ebc
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Mon Oct 31 20:36:26 2016 -0700
anv/device: Add an execbuf wrapper
This wrapper ensures that we always update all anv_bo::offset fields based
on the offsets returned by the kernel.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 07798c9c3ed29fd162ebc6b6d6beb8448218487f)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=678b4f6372effa1710fd2387de65d29cfc180796
Author: Anuj Phogat <anuj.phogat at gmail.com>
Date: Mon Oct 24 16:03:00 2016 -0700
i965: Fix GPU hang related to multiple render targets and alpha testing
This patch should have been the part of commit e592f7df.
In a situation when there are multiple render targets with alpha testing
enabled, if fragment shader doesn't write to draw buffer zero, it causes
the GPU hang on SKL. No GPU hang is seen on HSW. Simulator gives a
warning for all gen6+ h/w:
"Illegal render target write message length 0xa expected 0xc"
This patch fixes the GPU hang as well as the simulator warning with
new piglit test fbo-mrt-alphatest-no-buffer-zero-write:
https://patchwork.freedesktop.org/patch/118212
No regressions in Jenkins CI system.
Cc: "12.0 13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
Reviewed-by: Ben Widawsky <ben at bwidawsk.net>
(cherry picked from commit b9df2251c17e3ce52fa55c81f492591e08c3ee04)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=8ab9842d2e258147a2e84367fdd1d8a846df8d08
Author: Dave Airlie <airlied at redhat.com>
Date: Tue Nov 8 16:22:39 2016 +1000
radv: emit correct last export when Z/stencil export is enabled
I was getting a random GPU hang in the renderpass simple tests,
it turns out sometimes radv emitted the wrong thing "last".
This fixes the logic to emit Z/stencil last if they occur,
and not mark a color output as last. Also this relies on the
Z/STENCIL being the first two fragment outputs, which they are
so yay.
Fixes: dEQP-VK.renderpass.simple.color_depth (random hangs)
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit bafc75b4370bfbec0c91ff6bb4d4972fb37bb22a)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=dba0abdc91b0f1a2c871a60272df577410eda78e
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Sat Oct 22 22:27:23 2016 -0700
intel/blorp: Emit all the binding tables
At least on Sky Lake, after emitting 3DSTATE_CONSTANT_*, you are required
to re-emit the 3DSTATE_BINDING_TABLE_POINTERS packet for the corresponding
stage. If you don't, double-buffering may fail and you may get the wrong
constants. It turns out that you need to do this even if you have no push
constants to speak of or else the next 3DSTATE_CONSTANT packet you emit for
that stage may not work correctly.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 406cd9d1261996ba53e3301b2a5a01b5c5c56f00)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a31947fbf95ee79c7617cf74ebb24a6849e0c9c3
Author: Eric Anholt <eric at anholt.net>
Date: Fri Nov 4 13:41:20 2016 -0700
vc4: Use Newton-Raphson on the 1/W write to fix glmark2 terrain.
The 1/W was apparently not accurate enough, and we were getting sparklies
in the distance. The closed driver also did a N-R step here.
Cc: <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 283d4d18e598793bbff7d9ba5a601bced9b36542)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=f7efc0f0fcc6920b46daa8ce73fcba6d76415411
Author: Nicolai Hähnle <nicolai.haehnle at amd.com>
Date: Thu Nov 3 21:49:40 2016 +0100
st/mesa: fix the layer of VDPAU surface samplers
A (latent) bug in VDPAU interop was exposed by commit
e5cc84dd43be066c1dd418e32f5ad258e31a150a.
Before that commit, the st_vdpau code created samplers with
first_layer == last_layer == 1 that the general texture handling code
would immediately delete and re-create, because the layer does not match
the information in the GL texture object.
This was correct behavior at least in the DMABUF case, because the imported
resource is supposed to have the correct offset already applied. In the
non-DMABUF case, this was just plain wrong but apparently nobody noticed.
After that commit, the state tracker assumes that an existing sampler is
correct at all times. Existing samplers are supposed to be deleted when
they may become invalid, and they will be created on-demand. This meant
that the sampler with first_layer == last_layer == 1 stuck around, leading
to rendering artefacts (on radeonsi), command stream failures (on r600), and
assertions (in debug builds everywhere).
This patch fixes the problem by simply not creating a sampler at all in
st_vdpau_map_surface. We rely on the generic texture code to do the right
thing, adding the layer_override to make the non-DMABUF case work.
v2: add the layer_override
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98512
Cc: 13.0 <mesa-stable at lists.freedesktop.org>
Cc: Christian König <deathsimple at vodafone.de>
Cc: Ilia Mirkin <imirkin at alum.mit.edu>
Reviewed-by: Marek Olšák <marek.olsak at amd.com> (v1)
Reviewed-by: Christian König <christian.koenig at amd.com>
(cherry picked from commit 322483f71b068b3bbf69e5434e888f3fd3f4589e)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9c297c5487bf5464f75ffd5e11ad6aaad92915a9
Author: Dave Airlie <airlied at redhat.com>
Date: Thu Sep 15 13:58:33 2016 +1000
Revert "st/vdpau: use linear layout for output surfaces"
This reverts commit d180de35320eafa3df3d76f0e82b332656530126.
This is a radeon specific hack that causes problems on nouveau
when combined with the SHARED flag later. If radeonsi needs a fix
for this, please fix it in the driver.
[chk]
Using linear surfaces for this makes sense because tilling isn't
beneficial and the surfaces can potentially be shared with other GPUs
using the VDPAU OpenGL interop.
[airlied]
I think we need a flag that isn't SHARED/LINEAR that is more
SHARED_OTHER_GPU.
[mareko]
Does radeonsi need PIPE_BIND_VIDEO_DECODE_OUTPUT that it would translate
into linear ?
[mareko]
My only concern is decoding performance. If the decoder works in 64x1
blocks, tiling will hurt. That's the theory. I don't know how the
decoder works.
Cc: 12.0 13.0 <mesa-stable at lists.freedesktop.org>
Acked-by: Christian König <christian.koenig at amd.com>
Signed-off-by: Dave Airlie <airlied at redhat.com>
Tested-by: Ilia Mirkin <imirkin at alum.mit.edu>
Tested-by: Nayan Deshmukh <nayan26deshmukh at gmail.com> (I+A)
(cherry picked from commit d0d5f7600c2e8ab8d0c153787185f7a534753edd)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=aa947e7a6352916a19e6d8bdd8e23c8a55c793ed
Author: Marek Olšák <marek.olsak at amd.com>
Date: Thu Nov 3 19:16:51 2016 +0100
radeonsi: fix an assertion failure in si_decompress_sampler_color_textures
This fixes a crash in Deus Ex: Mankind Divided. Release builds were
unaffected, so it's not too serious.
Cc: 11.2 12.0 13.0 <mesa-stable at lists.freedesktop.org>
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
(cherry picked from commit 00baaa4752ab7e721218a2840cf0952d8c7c6eca)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d54699135f1f2c69206107ba161c1bfd9ca16b78
Author: Marek Olšák <marek.olsak at amd.com>
Date: Wed Nov 2 18:59:22 2016 +0100
glx: make interop ABI visible again
This was broken when the GLAPI use was removed from mesa_glinterop.h.
Cc: 12.0 13.0 <mesa-stable at lists.freedesktop.org>
Acked-by: Alex Deucher <alexander.deucher at amd.com>
Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
(cherry picked from commit 64c2593a5c33a98d880d141793360b44d07d1366)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a0d11b190ade4279c0798844c3a3c962f1adeefd
Author: Marek Olšák <marek.olsak at amd.com>
Date: Wed Nov 2 18:59:22 2016 +0100
egl: make interop ABI visible again
This was broken when the GLAPI use was removed from mesa_glinterop.h.
Cc: 12.0 13.0 <mesa-stable at lists.freedesktop.org>
Acked-by: Alex Deucher <alexander.deucher at amd.com>
Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
(cherry picked from commit ee39d4456e7551b257343551d59e7c6a3388fdc0)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=aa60c7b1c1eb86b457b3717fa2edaec77281eccf
Author: Marek Olšák <marek.olsak at amd.com>
Date: Wed Nov 2 18:56:39 2016 +0100
egl: use util/macros.h
I need the definition of PUBLIC.
Cc: 12.0 13.0 <mesa-stable at lists.freedesktop.org>
Acked-by: Alex Deucher <alexander.deucher at amd.com>
Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
(cherry picked from commit bf51b45313c7cc5ca792401f0cc29574aa9122cf)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b8f99c6b2f9065167cd1db7090a9d5ec7ec5f67b
Author: Nicolai Hähnle <nicolai.haehnle at amd.com>
Date: Thu Nov 3 11:00:36 2016 +0100
st/glsl_to_tgsi: fix dvec[34] loads from SSBO
When splitting up loads, we have to add 16 bytes to the offset for
the high components, just like already happens for stores.
Fixes arb_gpu_shader_fp64 at shader_storage@layout-std140-fp64-shader.
Cc: 13.0 <mesa-stable at lists.freedesktop.org>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
(cherry picked from commit e4b378800eff13752dcfe1f5c6b640444208d543)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=2789bfdbb59c4528a248dd8c02124dd384caefe6
Author: Francisco Jerez <currojerez at riseup.net>
Date: Tue Nov 1 11:56:13 2016 -0700
nir: Flip gl_SamplePosition in nir_lower_wpos_ytransform().
Assuming the hardware is set up to use a screen coordinate system
flipped vertically with respect to the GL's window coordinate system,
the SYSTEM_VALUE_SAMPLE_POS vector will also be flipped vertically
with respect to the value expected by the GL, so we need to give it
the same treatment as gl_FragCoord. Fixes the following CTS tests on
i965:
ES31-CTS.functional.shaders.multisample_interpolation.interpolate_at_offset.at_sample_position.default_framebuffer
ES31-CTS.functional.shaders.sample_variables.sample_pos.correctness.default_framebuffer
when run with any multisample configuration, e.g. rgba8888d24s8ms4.
Cc: <mesa-stable at lists.freedesktop.org>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>
(cherry picked from commit f3d387867f74ae758b41168f23992671f7dce254)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=da1ac6bc4642f0c2ab10f557d7749fc6d41dc4b7
Author: Andreas Boll <andreas.boll.dev at gmail.com>
Date: Wed Nov 2 12:11:54 2016 +0100
glx/windows: Add wgl.h to the sources list
Otherwise it won't be picked in the tarball and the build will fail.
Fixes: 533b3530c12 ("direct-to-native-GL for GLX clients on Cygwin
("Windows-DRI")")
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Andreas Boll <andreas.boll.dev at gmail.com>
Reviewed-by: Jon Turney <jon.turney at dronecode.org.uk>
(cherry picked from commit f792f0687f60e789fe9e0e2550c4eded95253fa1)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=996c20208fdbf50e64480ebe9d87c256b939be4e
Author: Nicolai Hähnle <nicolai.haehnle at amd.com>
Date: Mon Oct 31 14:03:10 2016 +0100
glsl: fix lowering of UBO references of named blocks
When a UBO reference has the form block_name.foo where block_name refers
to a block where the first member has a non-zero offset, the base offset
was incorrectly added to the reference.
Fixes an assertion triggered in debug builds by
GL45-CTS.enhanced_layouts.uniform_block_layout_qualifier_conflict. That test
doesn't properly check for correct execution in this case, so I am also
going to send out a piglit test.
Cc: 13.0 <mesa-stable at lists.freedesktop.org>
Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>
(cherry picked from commit 37d646c1b3626ad54ed93a784824af7b5abe8a99)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9397899aedeec92ec678377785dd78441a74a5fb
Author: Kenneth Graunke <kenneth at whitecape.org>
Date: Wed Nov 2 13:35:30 2016 -0700
glsl: Update deref types when resizing implicitly sized arrays.
At link time, we resolve the size of implicitly sized arrays.
When doing so, we update the type of the ir_variables. However,
we neglected to update the type of ir_dereference nodes which
reference those variables.
It turns out array_resize_visitor (for GS/TCS/TES interface array
handling) already did 2/3 of the cases for this, so we can simply
refactor the code and reuse it.
This fixes:
GL45-CTS.shader_storage_buffer_object.basic-syntax
GL45-CTS.shader_storage_buffer_object.basic-syntaxSSO
which have an SSBO containing an implicitly sized array, followed
by some other members. setup_buffer_access uses the dereference
types to compute offsets to fields, and it had a stale type where
the implicitly sized array's length was still 0 instead of the
actual length.
While we're here, we can also fix update_array_sizes to properly
update deref types as well, fixing a FINISHME from 2010.
Cc: mesa-stable at lists.freedesktop.org
Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>
(cherry picked from commit 8df4aebc94337983194cc72c817c08ee938117a1)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=bd3fde40681171d91b058e016847f4b69470ba8e
Author: Timothy Arceri <timothy.arceri at collabora.com>
Date: Wed Nov 2 09:49:58 2016 +1100
mesa/glsl: delete previously linked shaders earlier when linking
This moves the delete linked shaders call to
_mesa_clear_shader_program_data() which makes sure we delete them
before returning due to any validation problems.
It also reduces some code duplication.
From the OpenGL 4.5 Core spec:
"If LinkProgram failed, any information about a previous link of
that program object is lost. Thus, a failed link does not restore
the old state of program.
...
If one of these commands is called with a program for which
LinkProgram failed, no error is generated unless otherwise noted.
Implementations may return information on variables and interface
blocks that would have been active had the program been linked
successfully. In cases where the link failed because the program
required too many resources, these commands may help applications
determine why limits were exceeded."
Therefore it's expected that we shouldn't be able to query the
program that failed to link and retrieve information about a
previously successful link.
Before this change the linker was doing validation before freeing
the previously linked shaders and therefore could exit on failure
before they were freed.
This change also fixes an issue in compat profile where a program
with no shaders attached is expect to fall back to fixed function
but was instead trying to relink IR from a previous link.
Reviewed-by: Tapani Pälli <tapani.palli at intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97715
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit d2861d682a235993844989f7742c9539c3e10245)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=2478cfe41db41b5ab9b2ed2c0da9b1dcfc1cfe50
Author: Fredrik Höglund <fredrik at kde.org>
Date: Thu Oct 27 03:02:14 2016 +0200
radv: add support for anisotropic filtering on VI+
Ported from radeonsi.
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit e7b9c5eb74619dbda3f24928196ab8c8d9408c0b)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=4514ce8bc7d7aedcb2ecbf366245f0eb5b037663
Author: Dave Airlie <airlied at redhat.com>
Date: Wed Nov 2 00:33:33 2016 +0000
radv: fix dual source blending
Dolphin tried to use this, but we hadn't had any tests for it properly.
All that is required is the shader output format needs to be set
for 0 and 1 exports.
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit 73592b92844af3c1298ef5e66e3055793443546d)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=bc1d7a6ac4d917726cdaa7882cd25233dee50285
Author: Adam Jackson <ajax at redhat.com>
Date: Wed Sep 14 14:01:13 2016 -0400
glx/glvnd: Fix dispatch function names and indices
As this array was not actually sorted, FindGLXFunction's binary search
would only sometimes work.
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
Signed-off-by: Adam Jackson <ajax at redhat.com>
(cherry picked from commit 8bca8d89ef3babbecc4f3160cbbcb93d33b7a6ee)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c08a62a0b1b481e1c0bf02228ef03442bf1b0031
Author: Adam Jackson <ajax at redhat.com>
Date: Wed Sep 14 13:56:50 2016 -0400
glx/glvnd: Don't modify the dummy slot in the dispatch table
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
Signed-off-by: Adam Jackson <ajax at redhat.com>
(cherry picked from commit deb0eb1660f612862c77d225582e102e7ab717ee)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=81df3f63cb581f28caf3286100774ab96b34cff0
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 15:10:29 2016 -0700
anv/pipeline: Properly cache prog_data::param
Before we were caching the prog data but we weren't doing anything with
brw_stage_prog_data::param so anything with push constants wasn't getting
cached properly. This commit fixes that.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98012
Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 71cc1e188d52bebe86a1ac72fe90f4e2a7e76778)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=e016945bdd739811cde46b640e07627386361343
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 16:03:12 2016 -0700
anv/pipeline: Put actual pointers in anv_shader_bin
While we can simply calculate offsets to get to things such as the
prog_data and the key, it's much more user-friendly if there are just
pointers. Also, it's a bit more fool-proof.
While we're at it, we rework the pipeline cache API to use the
brw_stage_prog_data type directly.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98012
Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit ff3185e3ba85b3635a1f645e8e951954f4022afe)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=78fbafedf1bd619161e3f58cff2c1d412ae87bd1
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 14:16:34 2016 -0700
intel/blorp: Pass a brw_stage_prog_data to upload_shader
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98012
Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 4306c10a885bdd833328a2a4eeb0079aa4f4bae3)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=5be463694bfbeed3066601cbdb70825fe7e97859
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 14:03:43 2016 -0700
intel/blorp: Use wm_prog_data instead of hand-rolling our own
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98012
Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 058304f0819c96fd18eb17911022a410540aa295)
[Emil Velikov: brw_compile_fs() has different signature]
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
Conflicts:
src/intel/blorp/blorp.c
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=88ebff8e2555a2572c483b1e2c224480482cbbac
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 17:51:56 2016 -0700
anv: Better handle return codes from anv_physical_device_init
The case where we just want the loop to continue is INCOMPATIBLE_DRIVER
because that simply means that whatever FD we opened isn't a supported
Intel chip. Other error codes such as OUT_OF_HOST_MEMORY are actual errors
and we should be returning early in that case.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Dave Airlie <airlied at redhat.com>
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit a5f8ff6ca18c38a7c4b5b37a4d5b14ca01e71b1e)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9c722e8a2ea914e0693cefc49f7849a8eee12e1d
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 17:18:30 2016 -0700
vulkan/wsi/x11: Clean up connections in finish_wsi
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Dave Airlie <airlied at redhat.com>
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit daeb21e47845795d0320811db1e202540fba356b)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=f622d3334733e0a1349e0283b6d03ded7f864e1e
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Tue Nov 1 17:16:14 2016 -0700
vulkan/wsi/x11: Better handle wsi_x11_connection_create failure
Without this fix, the function would still end up returning NULL but it
would put that NULL connection in the hash table which would be bad.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Dave Airlie <airlied at redhat.com>
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit fc0e9e3e40e4b044ce1b62c1b757941f4ed4c820)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=dd5e802d338250ed9c9f2382cf67ed09a5a0f139
Author: Chih-Wei Huang <cwhuang at android-x86.org>
Date: Fri Oct 28 17:54:17 2016 +0800
android: avoid using libdrm with host modules
Note LOCAL_CFLAGS and LOCAL_SHARED_LIBRARIES in Android.common.mk
are used by both host and target modules. However, commit 112e988
moved libdrm related flags to common. It causes the errors like:
error: 'out/host/linux-x86/obj32/SHARED_LIBRARIES/libdrm_intermediates/export_includes',
needed by 'out/host/linux-x86/obj32/EXECUTABLES/mesa_gen_matypes_intermediates/import_includes',
missing and no known rule to make it
No reason to use libdrm with host modules.
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Fixes: 112e988329b ("Android: move libdrm settings to top-level
Android.common.mk")
Signed-off-by: Chih-Wei Huang <cwhuang at linux.org.tw>
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
(cherry picked from commit e3e5b1a4884666ec806d4eb0188826578580d666)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ea07a57fc0092ae1111df49f76f48f61a27544b9
Author: Nicolai Hähnle <nicolai.haehnle at amd.com>
Date: Thu Oct 27 10:59:37 2016 +0200
radeonsi: fix BFE/BFI lowering for GLSL semantics
Fixes spec/arb_gpu_shader5/execution/built-in-functions/*-bitfield{Extract,Insert}
Cc: 13.0 <mesa-stable at lists.freedesktop.org>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
(cherry picked from commit 5aef14932ac047dc5f1af311a26b7f41b140d79f)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=620ef8e7429e47d243db2dfb477567f14520bcb8
Author: Dave Airlie <airlied at redhat.com>
Date: Tue Nov 1 23:51:43 2016 +0000
radv: expose xlib platform extension
I missed this when I added the xlib code, this allows
dolphin emu to start and crash later.
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit 9f0726f3e509c80c78ddb5e7411fa34f676de71d)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=2f8b48d274bd51613e95a5ca09fd767e431d85a2
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Mon Oct 31 16:33:43 2016 -0700
anv/device: Return DEVICE_LOST if execbuf2 fails
This makes more sense than OUT_OF_HOST_MEMORY. Technically, you can
recover from a failed execbuf2 but the batch you just submitted didn't
fully execute so things are in an ill-defined state. The app doesn't want
to continue from that point anyway.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
Cc: "13.0" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
(cherry picked from commit c41ec1679f02d6705ceda49d9ad966d0d0833824)
More information about the mesa-commit
mailing list