URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a17ac460a22dbafde5bfd8a6f98740b9da761f4f Author: Dylan Baker <dy...@pnwbakers.com> Date: Thu May 9 13:21:34 2019 -0700
Docs: add 19.0.4 release notes URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c045d81c3d29db3ecd49651f0291a6d5c3474a94 Author: Dylan Baker <dy...@pnwbakers.com> Date: Thu May 9 13:18:08 2019 -0700 VERSION: bump for 19.0.4 release URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b5698550dd14582ca1edc235db9f03e7894d317c Author: Samuel Pitoiset <samuel.pitoi...@gmail.com> Date: Thu May 9 10:15:20 2019 +0200 radv: fix setting the number of rectangles when it's dyanmic We need to know the number of rectangles. This fixes new CTS dEQP-VK.draw.discard_rectangles.dynamic_*. Fixes: 5db0bf99944 ("radv: Implement VK_EXT_discard_rectangles.") Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> (cherry picked from commit 53dfff1c4d95ee21661d86256f44eae26b985b50) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1281a2207c63cc8be3c2f15129e9a5aab4b269f9 Author: Dylan Baker <dy...@pnwbakers.com> Date: Tue May 7 09:55:38 2019 -0700 meson: Force the use of config-tool for llvm meson git now has a cmake find method for llvm, but it lacks a couple of features that we use from the config tool version. Until that reaches parity we need to use the config-tool version. CC: 19.0 19.1 <<mesa-sta...@lists.freedesktop.org> Reviewed-by: Eric Engestrom <eric.engest...@intel.com> (cherry picked from commit 0d59459432cf077d768164091318af8fb1612500) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ba47599c7984b8d657744a474defe786b4224ac6 Author: Lionel Landwerlin <lionel.g.landwer...@intel.com> Date: Wed May 1 12:30:41 2019 +0100 anv: rework queries writes to ensure ordering memory writes We use a mix of MI & PIPE_CONTROL commands to write our queries' data (results & availability). Those commands' memory write order is not guaranteed with regard to their order in the command stream, unless CS stalls are inserted between them. This is problematic for 2 reasons : 1. We copy results from the device using MI commands even though the values are generated from PIPE_CONTROL, meaning we could copy unlanded values into the results and then copy the availability that is inconsistent with the values. 2. We allow the user to poll on the availability values of the query pool from the CPU. If the availability lands in memory before the values then we could return invalid values. This change does 2 things to address this problem : - We use either PIPE_CONTROL or MI commands to write both queries values and availability, so that the ordering of the memory writes guarantees that if availability is visible, results are also visible. - For the occlusion & timestamp queries we apply a CS stall before copying the results on the device, to ensure copying with MI commands see the correct values of previous PIPE_CONTROL writes of availability (required by the Vulkan spec). Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> Reported-by: Iago Toral Quiroga <ito...@igalia.com> Cc: mesa-sta...@lists.freedesktop.org Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (cherry picked from commit a07d06f10352fc5fa40db8a723fa5842ebc660db) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=0ebd8e316d129a48b1c0adf0629a02b552e560bb Author: Timothy Arceri <tarc...@itsqueeze.com> Date: Tue May 7 13:55:32 2019 +1000 Revert "glx: Fix synthetic error generation in __glXSendError" This reverts commit e91ee763c378d03883eb88cf0eadd8aa916f7878. This seems to have broken a number of wine games. Lets revert everything for now and try again later. Acked-by: Adam Jackson <a...@redhat.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110632 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110590 (cherry picked from commit a01b393c397c846345f03f76f1167dd667e0ee96) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=529981bc193c130c10bba586776506feb38afe8b Author: Kenneth Graunke <kenn...@whitecape.org> Date: Fri May 3 12:02:41 2019 -0700 i965: leave the top 4Gb of the high heap VMA unused This ports commit 9e7b0988d6e98690eb8902e477b51713a6ef9cae from anv to i965. Thanks to Lionel for noticing that it was missing! Fixes: 01058a55229 i965: Add virtual memory allocator infrastructure to brw_bufmgr. Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (cherry picked from commit d568fcd0a09751cd041cdd46bc585e209e0df394) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=be928454f0cdf54a4118b08c8287eb29491e8535 Author: Kenneth Graunke <kenn...@whitecape.org> Date: Fri Apr 26 18:52:45 2019 -0700 i965: Force VMA alignment to be a multiple of the page size. This should happen regardless, but let's be paranoid. Fixes: 01058a55229 i965: Add virtual memory allocator infrastructure to brw_bufmgr. Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (cherry picked from commit 17210c63a91aaf018813b0d336f5f1d4fd87eafb) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=123e5c142bf8c54636860a51e0209c65caac2e17 Author: Kenneth Graunke <kenn...@whitecape.org> Date: Fri Apr 26 17:09:11 2019 -0700 i965: Fix BRW_MEMZONE_LOW_4G heap size. The STATE_BASE_ADDRESS "Size" fields can only hold 0xfffff in pages, and 0xfffff * 4096 = 4294963200, which is 1 page shy of 4GB. So we can't use the top page. Fixes: 01058a55229 i965: Add virtual memory allocator infrastructure to brw_bufmgr. Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (cherry picked from commit 15f134c62853ed6679435a9e4ae40e3308fc7453) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=f2b085df419cddd156228f279eb340d1c8bee4b2 Author: Dylan Baker <dy...@pnwbakers.com> Date: Wed May 8 09:42:19 2019 -0700 cherry-ignore: Add more patches The radv patch was manually backported by Samuel (thanks!), and the two android build system patches were de-nominated by their submitter, as they would require many additional patches to be pulled in URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=29e6f5c11b8ebb6ecb56b055bd73544def3eb59b Author: Samuel Pitoiset <samuel.pitoi...@gmail.com> Date: Fri May 3 11:45:34 2019 +0200 radv: apply the indexing workaround for atomic buffer operations on GFX9 Because the new raw/struct intrinsics are buggy with LLVM 8 (they weren't marked as source of divergence), we fallback to the old instrinsics for atomic buffer operations only. This means we need to apply the indexing workaround for GFX9. The load/store operations still use the new LLVM 8 intrinsics. The fact that we need another workaround is painful but we should be able to clean up that a bit once LLVM 7 support will be dropped. This fixes a GPU hang with AC Odyssey and some rendering problems with Nioh. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110573 Fixes: 31164cf5f70 ("ac/nir: only use the new raw/struct image atomic intrinsics with LLVM 9+") Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b157a98978003a35d488cfff1a51b67aaaf4cfac Author: Emil Velikov <emil.veli...@collabora.com> Date: Mon May 6 16:32:38 2019 +0100 configure.ac: check for libdrm when using VL with X11 The X11 specific code uses libdrm, yet we are missing the dependency. This has gone unnoticed since all drivers which use VL already mandate the library. Note: this is applicable only for the stable branches. Cc: <mesa-sta...@lists.freedesktop.org> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> Tested-by: Alyssa Ross <h...@alyssa.is> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=7011e21890e199646ed946027d42f8445a6287dd Author: Dylan Baker <dy...@pnwbakers.com> Date: Tue May 7 11:39:56 2019 -0700 cherry-ignore: Add another patch URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=553f29b6c306f4a320bf26bd3f6ec8e96a980618 Author: Timothy Arceri <tarc...@itsqueeze.com> Date: Mon May 6 14:39:44 2019 +1000 radeonsi: add config entry for Counter-Strike Global Offensive This fixes rendering issues with gun scopes which is rather important. Cc: "19.0" "19.1" <mesa-sta...@lists.freedesktop.org> Acked-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100239 (cherry picked from commit 49025292fbbf285d4473d2c20a83b6c533a79d45) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=0fedb0fdc6b509ec7c17b78c464d8cd16045e9a0 Author: Erik Faye-Lund <erik.faye-l...@collabora.com> Date: Wed May 1 15:37:45 2019 +0200 draw: flush when setting stream-out targets We need to re-prepare the middle-end state to pick up changes to this state to react correctly to pausing/resuming stream-out. So let's add a flush here. Signed-off-by: Erik Faye-Lund <erik.faye-l...@collabora.com> Fixes: ec8cbd79ac4 "draw/softpipe: EXT_transform_feedback support (v2)" Reviewed-by: Roland Scheidegger <srol...@vmware.com> (cherry picked from commit d84b85bc28d50182f77f2e42e3c14ccedd70715f) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ced513b6917e973b96a320dd4dfef50fe1ba57ac Author: Timothy Arceri <tarc...@itsqueeze.com> Date: Fri May 3 13:59:05 2019 +1000 util/drirc: add workarounds for bugs in Doom 3: BFG This makes the game playable on radeonsi. Cc: "19.0" "19.1" <mesa-sta...@lists.freedesktop.org> Reviewed-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110143 (cherry picked from commit 1af72fa4d665b9847dff9b22d7a7dea01c0960c7) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=444e93828511189a2003b6cad0a99c32005fb870 Author: Chuck Atkins <chuck.atk...@kitware.com> Date: Fri May 3 12:06:22 2019 -0400 meson: Fix missing glproto dependency for gallium-glx Signed-off-by: Chuck Atkins <chuck.atk...@kitware.com> Cc: mesa-stable <mesa-sta...@lists.freedesktop.org> Reviewed-by: Dylan Baker <dy...@pnwbakers.com> (cherry picked from commit a381dbf2537003c8bf7e32ad314c9266846a168f) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ad15e3049da9ae38729c0ffa48d06976d4264bfc Author: Dave Airlie <airl...@redhat.com> Date: Mon Apr 29 13:17:41 2019 +1000 util/bitset: fix bitset range mask calculations. The MASK macro is used in the RANGE macro, and it should return the pre-bitset word mask for the (b) value. i.e. BITSET_MASK(0) should be undefined since it's meaningless. BITSET_MASK(31) should give 0x7fffffff BITSET_MASK(32) should give 0xffffffff BITSET_MASK(33) should give 0x00000001 BITSET_MASK(64) should give 0xffffffff However then BITSET_RANGE ends up broken for cases where it's (b) value is the 0,32,64 value as in that case the lower mask would be 0 not 0xffffffff. This fixes the unit tests that I've added, and my code that uses bitsets. Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> Fixes: bb38cadb1c5f2 "More GLSL code" Reviewed-by: Kristian H. Kristensen <hoegsb...@google.com> (cherry picked from commit 512a31a41299758f0f60d0452b8c9e8c02238189) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=f767301bba12936ed37a72f6a2a23cd8699ecbb6 Author: Samuel Pitoiset <samuel.pitoi...@gmail.com> Date: Thu May 2 17:44:39 2019 +0200 radv: only need to force emit the TCS regs on Vega10 and Raven1 Other GFX9 chips aren't affected. Cc: "19.0" "19.1" <mesa-sta...@lists.freedesktop.org> Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> (cherry picked from commit 62001f3dff8957005f15bd6401c917abf6cec321) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=50174f01f2698a23a4de066d5b89fd53a1318631 Author: Francisco Jerez <curroje...@riseup.net> Date: Wed Jan 16 18:49:47 2019 -0800 intel/fs: Cap dst-aligned region stride to maximum representable hstride value. This is required in combination with the following commit, because otherwise if a source region with an extended 8+ stride is present in the instruction (which we're about to declare legal) we'll end up emitting code that attempts to write to such a region, even though strides greater than four are still illegal for the destination. Tested-by: Anuj Phogat <anuj.pho...@gmail.com> Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (cherry picked from commit 7f9f6263c1ecc3ab9e1984f5a9814f6820eb85cf) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c3ba3370e615b8dc42290349b0cb8ebed09a172e Author: Francisco Jerez <curroje...@riseup.net> Date: Wed Jan 16 19:01:04 2019 -0800 intel/fs: Lower integer multiply correctly when destination stride equals 4. Because the "low" temporary needs to be accessed with word type and twice the original stride, attempting to preserve the alignment of the original destination can potentially lead to instructions with illegal destination stride greater than four. Because the CHV/BXT alignment restrictions are now being enforced by the regioning lowering pass run after lower_integer_multiplication(), there is no real need to preserve the original strides anymore. Note that this bug can be reproduced on stable branches, but back-porting would be non-trivial, because the fix relies on the regioning lowering pass recently introduced. Tested-by: Anuj Phogat <anuj.pho...@gmail.com> Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (cherry picked from commit e2f475ddffabff5cbe2bcff78ff42ce899869b29) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d6afb4a7c6116e629634fe7899c21ad4e9e83e55 Author: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> Date: Thu May 2 16:03:51 2019 +0200 radv: Disable VK_EXT_descriptor_indexing. We did not implement the required non-uniform indexing features. This patch is to disable the extension on 19.0. For 19.1 the necessary functionality has been implemented. Fixes: 0e10790558b "radv: Enable VK_EXT_descriptor_indexing." Reviewed-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=f861f042b869ca9ec9892fc32c6d505996d7d5be Author: Samuel Pitoiset <samuel.pitoi...@gmail.com> Date: Thu May 2 14:56:46 2019 +0200 radv: set WD_SWITCH_ON_EOP=1 when drawing primitives from a stream output buffer According to RadeonSI, this seems to be required by the hardware to avoid GPU hangs. I think I just forgot to set that bit when I implemented VK_EXT_transform_feedback. This fixes a GPU hang with Space Engineers and DXVK. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110291 Fixes: b4eb029062a ("radv: implement VK_EXT_transform_feedback") Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> (cherry picked from commit 08be23bfdec9fb447c58ae48bf9cc1b91ecba128) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=cbce2f47fa3465dc938c98b6c67926b0ffe33b7e Author: Brian Paul <bri...@vmware.com> Date: Wed May 1 15:49:44 2019 -0600 svga: add SVGA_NO_LOGGING env var (v2) valgrind crashes when we try to initialize host logging. This env var can be used to disable logging. v2: rebase onto "svga: move host logging to winsys". Cc: mesa-sta...@lists.freedesktop.org Reviewed-by: Neha Bhende <bhen...@vmware.com> (cherry picked from commit f0f7c3b03a93f1e2f6c15262585af774013c6031) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=646c77d7185d5fc12a5a3a6e5ee12d59bf5cb4cd Author: Charmaine Lee <charmai...@vmware.com> Date: Wed May 1 15:35:54 2019 -0600 svga: move host logging to winsys This patch adds a host_log interface to svga_winsys and moves the host logging code to the winsys layer. Cc: mesa-sta...@lists.freedesktop.org Reviewed-by: Brian Paul <bri...@vmware.com> Reviewed-by: Neha Bhende <bhen...@vmware.com> (cherry picked from commit 9c5f407b0b44a05a52d55ec34d028a1f2a901064) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=e8a0371d257994f4d92f9f2583dc3449ee3f486a Author: Daniel Stone <dani...@collabora.com> Date: Tue Oct 30 12:56:53 2018 +0000 vulkan/wsi/wayland: Respect non-blocking AcquireNextImage If the client has requested that AcquireNextImage not block at all, with a timeout of 0, then don't make any non-blocking calls. This will still potentially block infinitely given a non-infinte timeout, but the fix for that is much more involved. Signed-off-by: Daniel Stone <dani...@collabora.com> Cc: mesa-sta...@lists.freedesktop.org Cc: Chad Versace <chadvers...@chromium.org> Cc: Jason Ekstrand <ja...@jlekstrand.net> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108540 Acked-by: Jason Ekstrand <ja...@jlekstrand.net> Reviewed-by: Chad Versace <chadvers...@chromium.org> Reviewed-by: Eric Engestrom <eric.engest...@intel.com> (cherry picked from commit 9826e04eca69d04b64258d26983410bd2470f607) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3bd0b970fcc724bbf2bc5a6cf116e6f36a95531d Author: Rhys Perry <pendingchao...@gmail.com> Date: Fri Apr 26 15:39:51 2019 +0100 radv: fix set_output_usage_mask() with composite and 64-bit types It previously used var->type instead of deref_instr->type and didn't handle 64-bit outputs. This fixes lots of transform feedback CTS tests involving transform feedback and geometry shaders (mostly dEQP-VK.transform_feedback.fuzz.random_geometry.*) v2: fix writemask widening when comp != 0 v3: fix 64-bit variables when comp != 0, again Signed-off-by: Rhys Perry <pendingchao...@gmail.com> Cc: 19.0 19.1 <mesa-sta...@lists.freedesktop.org> Reviewed-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> (cherry picked from commit 13c423629ea1c8d5822d2266d0969c75ea036e9c) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3618405c6ada46cb9d4e3ed712d6e06cde917101 Author: Samuel Pitoiset <samuel.pitoi...@gmail.com> Date: Tue Apr 30 17:08:19 2019 +0200 radv: implement a workaround for VK_EXT_conditional_rendering Per the Vulkan spec 1.1.107, the predicate is a 32-bit value. Though the AMD hardware treats it as a 64-bit value which means it might fail to discard. I don't know why this extension has been drafted like that but this definitely not fit with AMD. The hardware doesn't seem to support a 32-bit value for the predicate, so we need to implement a workaround. This fixes an issue when DXVK enables conditional rendering with RADV, this also fixes the Sasha conditionalrender demo. Fixes: e45ba51ea45 ("radv: add support for VK_EXT_conditional_rendering") Reported-by: Philip Rebohle <philip.rebo...@tu-dortmund.de> Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> (cherry picked from commit 6ac10e07c2d6d7740f09a4e53c6b2fc88f4ecadd) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=585c25be1e1a96568829542f2765e97c63d79939 Author: Samuel Pitoiset <samuel.pitoi...@gmail.com> Date: Tue Mar 5 18:17:13 2019 +0100 radv: fix color conversions for normalized uint/sint formats The hardware actually rounds before conversion. This now matches what values are used when performing fast clears vs slow clears. This fixes a rendering issue with Far Cry 3&4. This also fixes a bunch of CTS tests that use a 8-bit UNORM format (only when the 512*512 image size hint is manually disabled). Cc: "19.0" "19.1" <mesa-sta...@lists.freedesktop.org> Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> (cherry picked from commit e03e7c510f571a8867ab7a8604058c075c601a70) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=015e8974abb76da88ee4cab34dc764f23e694c9f Author: Samuel Pitoiset <samuel.pitoi...@gmail.com> Date: Wed May 1 16:10:44 2019 +0200 radv: do not need to force emit the TCS regs on Vega20 This chip doesn't need the fixup. This fixes a bunch of dEQP-VK.tessellation tests and avoid random GPU hangs. Cc: "19.0" "19.1" <mesa-sta...@lists.freedesktop.org> Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> (cherry picked from commit 61625439999e3658dddf58485fc0ef85ecf082fc) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=227df9464e26de9800b0be5da575730560c2dfa5 Author: Dave Airlie <airl...@redhat.com> Date: Wed May 1 14:30:10 2019 +1000 r600: reset tex array override even when no view bound If no view is bound we still should reset the override to 0 and array mode. This should fix misrendering in firefox WebRender since the pbo sampler was removed. Fixes: 1250383e36 (st/mesa: remove sampler associated with buffer texture in pbo logic) (cherry picked from commit e2fecf57e343961cd1db9251cef2c9eb95948d2f) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=5bebe2a00deb4d8463c8ee6df5c86e56c7aa2163 Author: Erik Faye-Lund <erik.faye-l...@collabora.com> Date: Tue Apr 30 13:41:39 2019 +0200 softpipe: setup pixel_offset for all primitive types If we don't update this for all primitive-types, we end up rendering slightly offset points and lines up until the point where the first triangle gets drawn. This is obviously not correct, and violates OpenGL's repeatability rule. Signed-off-by: Erik Faye-Lund <erik.faye-l...@collabora.com> Fixes: ca9c413647b ("softpipe: Respect gl_rasterization_rules in primitive setup.") Reviewed-By: Gert Wollny <gert.wol...@collabora.com> (cherry picked from commit f753ac355e5bbd1129755224c54a73339da97cb1) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=4b0f0599388561da98ddcfd684018f39641061fe Author: Andrii Simiklit <andrii.simik...@globallogic.com> Date: Thu Oct 11 13:53:21 2018 +0300 egl: return correct error code for a case req ver < 3 with forward-compatible The EGL_KHR_create_context spec says: "If an OpenGL context is requested and the values for attributes EGL_CONTEXT_MAJOR_VERSION_KHR and EGL_CONTEXT_MINOR_VERSION_KHR, when considered together with the value for attribute EGL_CONTEXT_OPENGL_FORWARD_COMPATIBLE_BIT_KHR, specify an OpenGL version and feature set that are not defined, than an EGL_BAD_MATCH error is generated." This case is already correctly handled a bit below in the same source file. The correct handling was added by commit: 63beb3df Reported-by: Ian Romanick <i...@freedesktop.org> Here: https://bugzilla.freedesktop.org/show_bug.cgi?id=92552#c9 Fixes: 11cabc45b712 "egl: rework handling EGL_CONTEXT_FLAGS" Reviewed-by: Ian Romanick <ian.d.roman...@intel.com> Signed-off-by: Andrii Simiklit <andrii.simik...@globallogic.com> (cherry picked from commit 5c581b3dd6979b79cb3e3ab8e2e03b442e6ecb0d) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=fabaae4d8d2985f9b247c2e620af198972196d7f Author: Axel Davy <davyax...@gmail.com> Date: Sat Apr 27 15:45:12 2019 +0200 st/nine: Fix D3DWindowBuffer_release for old wine nine support No-one reported bugs for that, but is seems c442dd789066104e5e84cc90d98a7ff5cd6296cf and previous commits used APIs not defined until nine minor version 3. This patch should prevent crash in this case. Also turn off the resize feature in this case, as we won't prevent a buffer leak anymore. Cc: "19.0" mesa-sta...@lists.freedesktop.org Signed-off-by: Axel Davy <davyax...@gmail.com> (cherry picked from commit 64a45ba7f83095a8a44a7b1104f31bab68b259fe) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=6e710c1fb4330d72c9ceb520b254f186708a4561 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Fri Apr 26 23:50:47 2019 -0700 mesa: Add missing display list support for GL_FOG_COORDINATE_SOURCE Fixes: fe5d67d95f3 ("Implement EXT_fog_coord and EXT_secondary_color.") Reviewed-by: Mathias Fröhlich <mathias.froehl...@web.de> Cc: Brian Paul <bri...@vmware.com> (cherry picked from commit bfc6486819505916cfe03a9348d5e9605c300007) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3fe72ca2149e611f9f48a44cc609aa9909f611b5 Author: Alejandro Piñeiro <apinhe...@igalia.com> Date: Thu Apr 18 15:58:35 2019 +0200 docs: document MESA_GLSL=errors keyword Added with commit 0161691f3518, still checked on shaderapi.c _mesa_get_shader_flag method. Fixes: 0161691f3518db310411 "mesa: add GLSL_REPORT_ERRORS debug flag" Reviewed-by: Eric Engestrom <eric.engest...@intel.com> (cherry picked from commit 9b6a00e66e2b9d8c999b53bf7f2dffb548ddf864) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=05c7d8c5513058e5dbca537ea26c4b5ac536d335 Author: Ross Burton <ross.bur...@intel.com> Date: Mon Feb 25 20:06:48 2019 +0000 Revert "meson: drop GLESv1 .so version back to 1.0.0" This patch claimed that the autotools build generates libGLESv1_CM.so.1.0.0, but it doesn't: es1api_libGLESv1_CM_la_LDFLAGS = \ -no-undefined \ -version-number 1:1 \ $(GC_SECTIONS) \ $(LD_NO_UNDEFINED) Revert commit cc15460e182148292be877bec5a8a61cec57377d to ensure that the autotools and meson builds produce the same libraries. Fixes: cc15460e182148292be8 "meson: drop GLESv1 .so version back to 1.0.0" Reviewed-by: Eric Engestrom <eric.engest...@intel.com> Reviewed-by: Dylan Baker <dy...@pnwbakers.com> (cherry picked from commit 1c1efa4ca9a190e21ea555c9b02f69af194dce51) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=96fe97d133e2d52eaea83930073adee31963685d Author: Juan A. Suarez Romero <jasua...@igalia.com> Date: Mon Apr 29 17:05:13 2019 +0200 radv: enable descriptor indexing capabilities This enables the remaining capabilities in SPV_EXT_descriptor_indexing. Fixes: 0e10790558b "radv: Enable VK_EXT_descriptor_indexing." Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> (cherry picked from commit 06c9d7f9f94d9ab44a2b6148d9b5ec3f76c8d3db) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=f807986a383d07c232bba59573c435935dad3afd Author: Juan A. Suarez Romero <jasua...@igalia.com> Date: Mon Apr 29 17:02:45 2019 +0200 spirv: add missing SPV_EXT_descriptor_indexing capabilities Add ShaderNonUniformEXT, UniformBufferArrayNonUniformIndexingEXT, SampledImageArrayNonUniformIndexingEXT, StorageBufferArrayNonUniformIndexingEXT, StorageImageArrayNonUniformIndexingEXT, InputAttachmentArrayNonUniformIndexingEXT, UniformTexelBufferArrayNonUniformIndexingEXT and StorageTexelBufferArrayNonUniformIndexingEXT capabilities. Cc: mesa-sta...@lists.freedesktop.org Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> (cherry picked from commit bbbe00a1016bd7b9e777a94fbe92477dc6109678) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b66205a140d055ba0143b67f56021f6183efedad Author: Hal Gentz <zegen...@protonmail.com> Date: Sun Mar 24 16:52:39 2019 -0600 glx: Fix synthetic error generation in __glXSendError To quote Uli Schlachter, who understands this stuff more than I do: > The function __glXSendError() in mesa's src/glx/glx_error.c invents an X11 > protocol error out of thin air. For the sequence number it uses dpy->request. > This is the sequence number of the last request that was sent. _XError() will > then update dpy->last_request_read based on the sequence number of the error > that just "came in". > > If now another something comes in with a sequence number less than > dpy->last_request_read, since sequence numbers are monotonically increasing, > widen() will incorrectly add 1<<32 to the sequence number and things might go > downhill afterwards. `__glXSendErrorForXcb` was also patched, as that's the function that `glXCreateContextAttribsARB` actually uses. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99781 Cc: mesa-sta...@lists.freedesktop.org Fixes: ad503c41 'apple: Initial import of libGL for OSX from AppleSGLX svn repository' Reviewed-by: Adam Jackson <a...@redhat.com> Reviewed-by: Ian Romanick <ian.d.roman...@intel.com> Signed-off-by: Hal Gentz <zegen...@protonmail.com> (cherry picked from commit e91ee763c378d03883eb88cf0eadd8aa916f7878) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=e416ce522ca79876fb8e3632e20f3642e3976710 Author: Dylan Baker <dy...@pnwbakers.com> Date: Mon Apr 29 09:06:23 2019 -0700 cherry-ignore: Add more backported patches URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c8dbe93b0a35c0cd965f090aff053edb22909f2e Author: Jason Ekstrand <ja...@jlekstrand.net> Date: Tue Apr 23 21:46:32 2019 -0500 anv/descriptor_set: Destroy sets before pool finalization Fixes: 105002bd2d "anv: destroy descriptor sets when pool gets..." Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=fa1bbc00a6cf3df7d47b6d7fd3f12e85e560ed37 Author: Tapani Pälli <tapani.pa...@intel.com> Date: Fri Apr 12 12:52:43 2019 +0300 anv: expose VK_EXT_queue_family_foreign on Android VK_ANDROID_external_memory_android_hardware_buffer requires this extension. It is safe to enable it since currently aux usage is disabled for ahw buffers. Fixes following dEQP extension dependency test on Android: dEQP-VK.api.info.device#extensions Cc: <mesa-sta...@lists.freedesktop.org> Signed-off-by: Tapani Pälli <tapani.pa...@intel.com> Acked-by: Jason Ekstrand <ja...@jlekstrand.net> Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> (cherry picked from commit 376c3e8f87ab0b9bce5182da60c59832f7c1c138) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=cd9e4fbec1c6f8e6ff2a161841a57dec6dcb1bc6 Author: Samuel Pitoiset <samuel.pitoi...@gmail.com> Date: Sat Apr 27 12:07:51 2019 +0200 radv: add missing VEGA20 chip in radv_get_device_name() Otherwise it returns "AMD RADV unknown". Cc: 19.0 <mesa-sta...@lists.freedesktop.org> Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> (cherry picked from commit 07745f94948874c875dac924e055a11ee96beb5b) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d19148b7d9c2f737fbeb76d872265610c0c4b5c7 Author: Emil Velikov <emil.veli...@collabora.com> Date: Fri Apr 19 12:11:00 2019 +0100 vulkan/wsi: don't use DUMB_CLOSE for normal GEM handles Currently we get normal GEM handles from PrimeFDToHandle, yet we close then with DUMB_CLOSE. Use GEM_CLOSE instead. Fixes: da997ebec92 ("vulkan: Add KHR_display extension using DRM [v10]") Cc: Jason Ekstrand <ja...@jlekstrand.net> Cc: Keith Packard <kei...@keithp.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> (cherry picked from commit bd0c4e360d08dc1b1a1433530b389358623783bb) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=37a3a80c2e5e829623d9f10ae6ba6421b9a23358 Author: Emil Velikov <emil.veli...@collabora.com> Date: Wed Apr 17 17:49:09 2019 +0100 vulkan/wsi: check if the display_fd given is master As effectively required by the extension, we need to ensure we're master Currently drivers employ vendor specific solutions, which check if the device behind the fd is capable*, yet none of them do the master check. *In the radv case, if acceleration is available. Instead of duplicating the check in each driver, keep it where it's needed and used. Note this copies libdrm's drmIsMaster() to avoid depending on bleeding edge version of the library. v2: set the fd to -1 if not master (Bas) Fixes: da997ebec92 ("vulkan: Add KHR_display extension using DRM [v10]") Cc: Andres Rodriguez <andre...@gmail.com> Cc: Jason Ekstrand <ja...@jlekstrand.net> Cc: Keith Packard <kei...@keithp.com> Reported-by: Andres Rodriguez <andre...@gmail.com> Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> (cherry picked from commit c962a78f18284e2971301bf68c9c60500ca398e4) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ef1f065f0a51340649f0b8cc3afab3dffb6c50de Author: Dylan Baker <dy...@pnwbakers.com> Date: Mon Apr 29 08:57:57 2019 -0700 cherry-ignore: Add a patch that was manually backported URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d81cf30f953eec217efa62597e22015500728fed Author: Kenneth Graunke <kenn...@whitecape.org> Date: Wed Apr 17 17:25:29 2019 -0700 i965: Ignore uniform storage for samplers or images, use binding info gl_nir_lower_samplers_as_deref creates new top level sampler and image uniforms which have been split from structure uniforms. i965 assumed that it could walk through gl_uniform_storage slots by starting at var->data.location and walking forward based on a simple slot count. This assumed that structure types were walked in a particular order. With samplers and images split out of structures, it becomes impossible to assign meaningful locations. Consider: struct S { sampler2D a; sampler2D b; } s[2]; The gl_uniform_storage locations for these follow this map: 0 => a[0], 1 => b[0], 2 => a[0], 3 => b[0]. But the new split variables look like: sampler2D lowered_a[2]; sampler2D lowered_b[2]; and there is no way to know that there's effectively a stride to get to the location for successive elements of a[] or b[]. So, working with location becomes effectively impossible. Ultimately, the point of looking at uniform storage was to pull out the bindings from the opaque index fields. gl_nir_lower_samplers_as_derefs can obtain this information while doing the splitting, however, and sets up var->data.binding to have the desired values. We move gl_nir_lower_samplers before brw_nir_lower_image_load_store so gl_nir_lower_samplers_as_derefs has the opportunity to set proper image bindings. Then, we make the uniform handling code skip sampler(-array) variables, and handle image param setup based on var->data.binding. Fixes Piglit tests/spec/glsl-1.10/execution/samplers/uniform-struct, this time without regressing dEQP-GLES2.functional.uniform_api.random.3. Fixes: f003859f97c nir: Make gl_nir_lower_samplers use gl_nir_lower_samplers_as_deref Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (backported from commit 6981069fc805da1afc867ca3c905075d146d7ff9, without prog_to_nir using derefs or texture bitfield gathering) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=6b74d3392866e11fa2761a2c181bf8369b6c3d45 Author: Kenneth Graunke <kenn...@whitecape.org> Date: Wed Feb 6 02:05:01 2019 -0800 glsl: Don't look at sampler uniform storage for internal vars Passes like nir_lower_drawpixels add additional sampler variables, and set an explicit binding which never changes. These extra samplers don't have proper uniform storage associated with them, and there is no way to update bindings via the API. So, for any 'hidden' variables, just trust that there's an explicit binding set. Reviewed-by: Eric Anholt <e...@anholt.net> (cherry picked from commit 529a0711c1d07c109f83286583ed359a890f5a75) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=06ca5c0a64c1ab9866b066f2c90f14d08b2810f9 Author: Kenneth Graunke <kenn...@whitecape.org> Date: Tue Feb 5 19:08:04 2019 -0800 glsl: Allow gl_nir_lower_samplers*() without a gl_shader_program I would like to be able to run gl_nir_lower_samplers() to turn texture and sampler variable dereferences into indexes and offsets, even for ARB programs, and built-in shaders. This would make sampler handling more consistent across the various types of shaders. For GLSL programs, the gl_nir_lower_samplers_as_deref() pass looks up the variable bindings in the shader program's uniform storage. But ARB programs and built-in shaders don't have a gl_shader_program, and uniform storage doesn't exist. In this case, we simply skip that lookup, and trust var->data.binding to be set correctly by whoever created the shader. Reviewed-by: Eric Anholt <e...@anholt.net> (cherry picked from commit d34e434989ec8e8ca780421f6909cfd796573520) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1a3478dc56a23f545bde85add944cf3a22a1d665 Author: Marek Olšák <marek.ol...@amd.com> Date: Thu Apr 18 15:19:19 2019 -0400 radeonsi/gfx9: rework the gfx9 scissor bug workaround (v2) Needed to track context rolls caused by streamout and ACQUIRE_MEM. ACQUIRE_MEM can occur outside of draw calls. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110355 v2: squashed patches and done more rework Cc: 19.0 <mesa-sta...@lists.freedesktop.org> (cherry picked from commit 440135e5a0d178c537db3f96e6823bc8220a0f3f) Conflicts resolved by Dylan URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9f1d5d63ee5ead9cc822528beb443d5f562d93ad Author: Nicolai Hähnle <nicolai.haeh...@amd.com> Date: Fri Mar 15 14:56:36 2019 +0100 radeonsi: add si_debug_options for convenient adding/removing of options Move the definition of radeonsi_clear_db_cache_before_clear there, as well as radeonsi_enable_nir. This removes the AMD_DEBUG=nir option. We currently still have two places for options: the driconf machinery and AMD_DEBUG/R600_DEBUG. If we are to have a single place for options, then the driconf machinery should be preferred since it's more flexible. The only downside of the driconf machinery was that adding new options was quite inconvenient. With this change, a simple boolean option can be added with a single line of code, same as for AMD_DEBUG. One technical limitation of this particular implementation is that while almost all driconf features are available, the translation machinery doesn't pick up the description strings for options added in si_debvug_options. In practice, translations haven't been provided anyway, and this is intended for developer options, so I'm not too worried. It could always be added later if anybody really cares. v2: - use bool instead of uint8_t for options - si_debug_options.inc -> si_debug_options.h Reviewed-by: Marek Olšák <marek.ol...@amd.com> (cherry picked from commit 8bef4df196fbb0fad7f3bd6048f71dbc38ebceb3) autotools dist fixed by Dylan (not needed in master) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c15519b882e1696dc8532c2aa3beff119a83e8f6 Author: Marek Olšák <marek.ol...@amd.com> Date: Wed Apr 17 11:43:14 2019 -0400 radeonsi/gfx9: set that window_rectangles always roll the context Cc: 19.0 <mesa-sta...@lists.freedesktop.org> (cherry picked from commit bc0d92450757ed2f583fef0a459c53b1ecd13ae6) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a038a271f94483960f7ba66a27948c313f098029 Author: Timothy Arceri <tarc...@itsqueeze.com> Date: Thu Apr 25 11:17:42 2019 +1000 nir: fix nir_remove_unused_varyings() We were only setting the used mask for the first component of a varying. Since the linking opts split vectors into scalars this has mostly worked ok. However this causes an issue where for example if we split a struct on one side of the interface but not the other, then we can possibly end up removing the first components on the side that was split and then incorrectly remove the whole struct on the other side of the varying. With this change we simply mark all 4 components for each slot used by a struct. We could possibly make this more fine gained but that would require a more complex change. This fixes a bug in Strange Brigade on RADV when tessellation is enabled, all credit goes to Samuel Pitoiset for tracking down the cause of the bug. Fixes: f1eb5e639997 ("nir: add component level support to remove_unused_io_vars()") Reviewed-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> (cherry picked from commit b155f74d7bfc9daec7794074c4f644025fbddf8b) Conflicts resolved by Dylan URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a8aa18aa231e77d18bce76bfb96ef1ee9f89f270 Author: Jon Turney <jon.tur...@dronecode.org.uk> Date: Sun Apr 14 20:46:39 2019 +0100 meson: Force '.so' extension for DRI drivers DRI driver loadable modules are always installed with install_megadriver.py with names ending with '.so', irrespective of platform. Force the name the loadable module is built with to match, so install_megadriver.py doesn't spin trying to remove non-existent symlinks. Fixes: c77acc3c "meson: remove meson-created megadrivers symlinks" (cherry picked from commit 5d310015c57536224600252e4ceadcf964bfc4fa) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=aa340450a06ea4feabf24b3f354a30b11d303376 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Thu Apr 18 12:33:39 2019 -0700 glsl: Silence may unused parameter warnings in glsl/ir.h Every file that included glsl/ir.h had a warning like: src/compiler/glsl/ir.h: In member function ‘virtual bool ir_rvalue::is_lvalue(const _mesa_glsl_parse_state*) const’: src/compiler/glsl/ir.h:236:64: warning: unused parameter ‘state’ [-Wunused-parameter] virtual bool is_lvalue(const struct _mesa_glsl_parse_state *state = NULL) const ^ Cc: Samuel Pitoiset <samuel.pitoi...@gmail.com> Fixes: fa4ebf6b8d9 ("glsl: add _mesa_glsl_parse_state object to is_lvalue()") Reviewed-by: Sagar Ghuge <sagar.gh...@intel.com> Reviewed-by: Matt Turner <matts...@gmail.com> (cherry picked from commit 3b087f668f9d5aeae85df2a4128cbe3b9b25ae6a) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d4bbf9f838f46c0f9ff05e5bc8d680e2887bdc3f Author: Caio Marcelo de Oliveira Filho <caio.olive...@intel.com> Date: Mon Apr 22 16:09:56 2019 -0700 spirv: Handle SpvOpDecorateId This operation decorate with an Id instead of a Literal or String. It is used by HlslCounterBufferGOOGLE (provided by SPV_GOOGLE_hlsl_functionality1). Even if we don't do anything with that decoration, we must be able to parse SPIR-V that uses it. Fixes: 891886da2f9 "spirv: Add no-op support for VK_GOOGLE_hlsl_functionality1" Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (cherry picked from commit 7e2684ce014c3f7813c4f0b2d8424fd588606d5f) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=35970dea11b9c29408f1baaac2010b021c647319 Author: Lionel Landwerlin <lionel.g.landwer...@intel.com> Date: Mon Apr 22 22:09:11 2019 +0100 anv: fix argument name for vkCmdEndQuery Doesn't fix anything but it's not the right function prototype. Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> Fixes: 673f33c77dd765 ("anv: Implement CmdBegin/EndQueryIndexed") Reviewed-by: Caio Marcelo de Oliveira Filho <caio.olive...@intel.com> Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> Reviewed-by: Sagar Ghuge <sagar.gh...@intel.com> (cherry picked from commit 0fb0058f18b24674583770c0c95600675ce8336e) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=08f3ce4c7df427f54bc9711d0194a3f22237730a Author: Lionel Landwerlin <lionel.g.landwer...@intel.com> Date: Fri Apr 12 11:05:33 2019 +0100 anv: leave the top 4Gb of the high heap VMA unused In 628c9ca9089789 I forgot to apply the same -4Gb of the high address of the high heap VMA. This was previously computed in the HIGH_HEAP_MAX_ADDRESS. Many thanks to James for pointing this out. Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> Reported-by: Xiong, James <james.xi...@intel.com> Fixes: 628c9ca9089789 ("anv: store heap address bounds when initializing physical device") Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (cherry picked from commit 9e7b0988d6e98690eb8902e477b51713a6ef9cae) URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3a90f8df502672177375d13f8ec756b7969db51e Author: Lionel Landwerlin <lionel.g.landwer...@intel.com> Date: Thu Apr 11 13:01:46 2019 +0100 anv: store heap address bounds when initializing physical device We can then reuse those bounds to initialize the VMA heaps at logical device creation. This fixes an issue on EHL which has only 36bits of VMA. We were incorrectly using the fixed 48bits upper bound to initialize the logical device heap, resulting in addresses beyong the device's limits. v2: Don't confuse heap size (limited by system memory) and VMA size (limited by number of addressing bits the platform has) v3: Fix low heap vma_size :( (Lionel) Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> Reported-by: James Xiong <james.xi...@intel.com> Reviewed-by: Eric Engestrom <eric.engest...@intel.com> (v1) Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (v2) (cherry picked from commit 628c9ca9089789e0103a7de0cc479ff7a5ed0871) _______________________________________________ mesa-commit mailing list mesa-commit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-commit