- 23 Jan, 2020 40 commits
-
-
Kenneth Graunke authored
This changes the default driver for Intel Gen8-11 hardware to be the newer 'iris' driver rather than the older 'i965' driver. To continue using i965, pass -Dprefer-iris=false when building. Reviewed-by:
Jordan Justen <jordan.l.justen@intel.com> Tested-by: Marge Bot <mesa/mesa!3540> Part-of: <mesa/mesa!3540>
-
Adam Jackson authored
This is not always ->rgbBits, because there are cases where that could be 32 but we're (legally) bound to a depth-24 pixmap. The important thing to have match here is the actual server-side notion of depth. You can look this up (at modest expense) from the xlib visual info if the fbconfig has a visual. But it might not, so if not, fetch it (at slightly greater expense) from XGetGeometry. Do this at GLX drawable creation so you don't have to do it on the SwapBuffers path. Apparently this fixes glx/glx-swap-singlebuffer, which is unintentional but quite pleasant. Fixes: mesa/mesa#2291 Fixes: 90d58286 ("drisw: Fix and simplify drawable setup") Reviewed-by:
Eric Anholt <eric@anholt.net> Tested-by: Marge Bot <mesa/mesa!3305> Part-of: <mesa/mesa!3305>
-
Emma Anholt authored
There are only a couple of hard cases left using pkt4, where the register number to write is computed. Tested-by: Marge Bot <mesa/mesa!3455> Part-of: <mesa/mesa!3455>
-
Emma Anholt authored
This gets a lot of the hard code converted over to the new macros, resulting in (I feel) much more readable code with LESS_SHOUTING_ABOUT_THE_REG(). I decided to consistently put the reg on its own line, so that all the register names line up. Part-of: <mesa/mesa!3455>
-
Emma Anholt authored
This introduces some minor unpacking of the temporary fd_reg_pair structs to code that previously was packing a whole register field. In the pack wrapper in tu_cs.h, I added some explanatory docs, dropped the relocs handling since we don't need it, and removed the extra regs[] in the __ONE_REG() macro (which was causing gcc's optimizer to fall on its face in my release build). Part-of: <mesa/mesa!3455>
-
Emma Anholt authored
Sometimes you want to zero out an address by supplying a NULL BO, but without this we would end up only emitting one dword. Increases size of fd6_gmem.o by .8%, though it's not clear to me why (no obvious terrible codegen happening) Part-of: <mesa/mesa!3455>
-
Emma Anholt authored
Part-of: <mesa/mesa!3455>
-
Ian Romanick authored
Suggested-by:
Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
-
Vasily Khoruzhick authored
It's source is supposed to be int, so we have to use integer multiplication otherwise we'll get undefined result. Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> Reviewed-by:
Erico Nunes <nunes.erico@gmail.com> Signed-off-by:
Vasily Khoruzhick <anarsoul@gmail.com> Tested-by: Marge Bot <mesa/mesa!3529> Part-of: <mesa/mesa!3529>
-
Vasily Khoruzhick authored
Otherwise we'll have to lower it later. Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> Reviewed-by:
Erico Nunes <nunes.erico@gmail.com> Signed-off-by:
Vasily Khoruzhick <anarsoul@gmail.com> Part-of: <mesa/mesa!3529>
-
Icecream95 authored
Reviewed-by:
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Tested-by: Marge Bot <mesa/mesa!3525> Part-of: <mesa/mesa!3525>
-
Icecream95 authored
The file name is taken from the environment variable PANDECODE_DUMP_FILE, defaulting to pandecode.dump if it is not set. Reviewed-by:
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Part-of: <mesa/mesa!3525>
-
Icecream95 authored
Reviewed-by:
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Part-of: <mesa/mesa!3525>
-
Icecream95 authored
Reviewed-by:
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Part-of: <mesa/mesa!3525>
-
Icecream95 authored
Reviewed-by:
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Part-of: <mesa/mesa!3525>
-
Icecream95 authored
Reviewed-by:
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Part-of: <mesa/mesa!3525>
-
Emma Anholt authored
We want to finish off cmd emission in the primary CS and add its entry to the IB, but regardless of whether there had been anything in the primary CS to emit, we still need a reserved CS entry for the loop below. Fixes crashes in dEQP-VK.binding_model.shader_access.secondary_cmd_buf.* and many more in dEQP-VK.renderpass* Reviewed-by:
Jonathan Marek <jonathan@marek.ca> Tested-by: Marge Bot <mesa/mesa!3524> Part-of: <mesa/mesa!3524>
-
Alyssa Rosenzweig authored
It doesn't seem to affect any results and it's not at all clear if/why the blob sometimes(?) sets it? So let's clean this up since this solution isn't correct anyway. Signed-off-by:
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Tested-by: Marge Bot <mesa/mesa!3513> Part-of: <mesa/mesa!3513>
-
Icecream95 authored
Reviewed-by:
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Tested-by: Marge Bot <mesa/mesa!3496> Part-of: <mesa/mesa!3496>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Tested-by: Marge Bot <mesa/mesa!3421> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
There was also some redundant code in si_shader_nir.c Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Marek Olšák authored
Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> Part-of: <mesa/mesa!3421>
-
Jonathan Marek authored
Fixes the "bad VkImageTiling" error when tiling is VK_IMAGE_TILING_DRM_FORMAT_MODIFIER_EXT. Signed-off-by:
Jonathan Marek <jonathan@marek.ca> Acked-by:
Eric Anholt <eric@anholt.net> Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> Tested-by: Marge Bot <mesa/mesa!3485> Part-of: <mesa/mesa!3485>
-
Jonathan Marek authored
Signed-off-by:
Jonathan Marek <jonathan@marek.ca> Acked-by:
Eric Anholt <eric@anholt.net> Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> Part-of: <mesa/mesa!3485>
-
Jonathan Marek authored
Fixes: 210e6887 "vulkan/wsi: Use the interface from the real modifiers extension" Signed-off-by:
Jonathan Marek <jonathan@marek.ca> Acked-by:
Eric Anholt <eric@anholt.net> Part-of: <mesa/mesa!3485>
-
Jonathan Marek authored
Fixes: 210e6887 "vulkan/wsi: Use the interface from the real modifiers extension" Signed-off-by:
Jonathan Marek <jonathan@marek.ca> Acked-by:
Eric Anholt <eric@anholt.net> Part-of: <mesa/mesa!3485>
-
Guido Günther authored
The current code overflows (s * 1000000000) for s >= 5 but that is e.g. used in msm_bo_cpu_prep. Signed-off-by:
Guido Günther <agx@sigxcpu.org> Tested-by: Marge Bot <mesa/mesa!3514> Part-of: <mesa/mesa!3514>
-
Emma Anholt authored
This does appear to be the required instruction sequence (dsxpp_1 dst src; dsxpp_1.p dst src) as dropping either instruction fails the testsuite. Fixes dEQP-VK.glsl.derivate.* Reviewed-by:
Jonathan Marek <jonathan@marek.ca> Reviewed-by:
Rob Clark <robdclark@chromium.org> Tested-by: Marge Bot <mesa/mesa!3494> Part-of: <mesa/mesa!3494>
-
Emma Anholt authored
legalize is computing a lot of state that goes in the variant, let's just store it directly instead of passing pointers around. This leaves max_bary in place, which is doing some surprising work (overwriting the original total_in in some cases). Reviewed-by:
Rob Clark <robdclark@chromium.org> Part-of: <mesa/mesa!3494>
-
Anthony Pesch authored
Reviewed-by:
Eric Anholt <eric@anholt.net> Reviewed-by:
Iago Toral Quiroga <itoral@igalia.com> Tested-by: Marge Bot <mesa/mesa!3475> Part-of: <mesa/mesa!3475>
-
Anthony Pesch authored
A few hash_table users roll their own integer hash functions which call _mesa_hash_data to perform the hashing which ultimately calls into XXH32 with a dynamic key length. When using small keys with a constant size the hash rate can be greatly improved by inlining XXH32 and providing it a constant key length, see: https://fastcompression.blogspot.com/2018/03/xxhash-for-small-keys-impressive-power.html Additionally, this patch removes calls to _mesa_key_hash_string and makes them instead call _mesa_has_string directly, matching the new integer hash functions. Reviewed-by:
Eric Anholt <eric@anholt.net> Reviewed-by:
Iago Toral Quiroga <itoral@igalia.com> Part-of: <!3475>
-