[Mesa-dev] [RFC PATCH 61/65] radeonsi: upload new descriptors when resident buffers are invalidated

2017-05-19 Thread Samuel Pitoiset
When texture buffers are invalidated the addr in the resident descriptor has to be updated but we can't create a new descriptor because the resident handle has to be the same. Instead, use the WRITE_DATA packet which allows to update memory directly but graphics/compute have to be idle in case

[Mesa-dev] [RFC PATCH 39/65] st/mesa: make update_single_texture() non-static

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_atom_texture.c | 15 +-- src/mesa/state_tracker/st_texture.h | 5 + 2 files changed, 14 insertions(+), 6 deletions(-) diff --git a/src/mesa/state_tracker/st_atom_texture.c

[Mesa-dev] [RFC PATCH 37/65] tgsi/scan: record bindless samplers/images usage

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/auxiliary/tgsi/tgsi_scan.c | 37 ++ src/gallium/auxiliary/tgsi/tgsi_scan.h | 2 ++ 2 files changed, 39 insertions(+) diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c

[Mesa-dev] [RFC PATCH 18/65] mesa: add update_single_program_texture_state() helper

2017-05-19 Thread Samuel Pitoiset
This will also be used for looping over bindless samplers bound to texture units. Signed-off-by: Samuel Pitoiset --- src/mesa/main/texstate.c | 36 +++- 1 file changed, 23 insertions(+), 13 deletions(-) diff --git

[Mesa-dev] [RFC PATCH 44/65] st/mesa: make bindless samplers/images bound to units resident

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_atom_constbuf.c | 6 ++ src/mesa/state_tracker/st_texture.c | 94 +++ src/mesa/state_tracker/st_texture.h | 8 +++ 3 files changed, 108 insertions(+) diff --git

[Mesa-dev] [RFC PATCH 57/65] radeonsi: decompress resident textures/images before graphics/compute

2017-05-19 Thread Samuel Pitoiset
Similar to the existing decompression code path except that it loops over the list of resident textures/images. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_blit.c| 67 +--

[Mesa-dev] [RFC PATCH 36/65] st/glsl_to_tgsi: teach rename_temp_registers() about bindless samplers

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp index ee1afdcd08..c69eefe013 100644 ---

[Mesa-dev] [RFC PATCH 53/65] radeonsi: add all resident buffers to the current CS

2017-05-19 Thread Samuel Pitoiset
Resident buffers have to be added to every new command stream. Though, this could be slightly improved when current shaders don't use any bindless textures/images but usually applications tend to use bindless for almost every draw call, and the winsys thread might help when buffers are added

[Mesa-dev] [RFC PATCH 62/65] radeonsi: invalidate buffers which are made resident if needed

2017-05-19 Thread Samuel Pitoiset
When a buffer becomes resident, check if it has been invalidated, if so update the descriptor and the dirty flag. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_descriptors.c | 34 +++ 1 file changed, 34 insertions(+) diff

[Mesa-dev] [RFC PATCH 48/65] radeonsi: add a slab allocator for resident descriptors

2017-05-19 Thread Samuel Pitoiset
For each texture/image handles, we need to allocate a new buffer for the resident descriptor. But when the number of buffers added to the current CS becomes high, the overhead in the winsys (and in the kernel) is important. To reduce this bottleneck, the idea is to suballocate the resident

[Mesa-dev] [RFC PATCH 46/65] st/mesa: disable per-context seamless cubemap when using texture handles

2017-05-19 Thread Samuel Pitoiset
The ARB_bindless_texture spec say: "If ARB_seamless_cubemap (or OpenGL 4.0, which includes it) is supported, the per-context seamless cubemap enable is ignored and treated as disabled when using texture handles." "If AMD_seamless_cubemap_per_texture is supported, the seamless

[Mesa-dev] [RFC PATCH 63/65] radeonsi: add support for loading bindless samplers

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c

[Mesa-dev] [RFC PATCH 45/65] st/mesa: do not release sampler views for resident textures

2017-05-19 Thread Samuel Pitoiset
When a texture is referenced by one or more texture handles, it might be resident and we shouldn't release the sampler views. Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_sampler_view.c | 6 ++ 1 file changed, 6 insertions(+) diff --git

[Mesa-dev] [RFC PATCH 51/65] radeonsi: add si_set_shader_image_desc() helper

2017-05-19 Thread Samuel Pitoiset
To share some common code between bound and bindless images. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_descriptors.c | 78 --- 1 file changed, 46 insertions(+), 32 deletions(-) diff --git

[Mesa-dev] [RFC PATCH 64/65] radeonsi: add support for loading bindless images

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 31 ++- 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c

[Mesa-dev] [RFC PATCH 54/65] radeonsi: only add descriptors in presence of resident handles

2017-05-19 Thread Samuel Pitoiset
This won't help much except for applications that use a ton of resident handles. Though, this will reduce the winsys overhead a little bit. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_descriptors.c | 5 + 1 file changed, 5 insertions(+)

[Mesa-dev] [RFC PATCH 59/65] radeonsi: track use of bindless samplers/images from tgsi_shader_info

2017-05-19 Thread Samuel Pitoiset
This adds some new helper functions to know if the current draw call (or dispatch compute) is using bindless samplers/images, based on TGSI analysis. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_compute.c | 2 ++

[Mesa-dev] [RFC PATCH 34/65] st/glsl_to_tgsi: add support for bindless pack/unpack operations

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp index

[Mesa-dev] [RFC PATCH 00/65] ARB_bindless_texture for RadeonSI

2017-05-19 Thread Samuel Pitoiset
Hi, This series implements ARB_bindless_texture for RadeonSI. Reminder: the GLSL compiler part is already upstream. This series has been mainly tested with Feral games, here's the list of existing games that use ARB_bindless_texture (though not by default): - DXMD - Hitman - Dirt Rally - Mad

[Mesa-dev] [RFC PATCH 05/65] mesa: refuse to update sampler parameters when a handle is allocated

2017-05-19 Thread Samuel Pitoiset
The ARB_bindless_texture spec says: "The error INVALID_OPERATION is generated by SamplerParameter* if identifies a sampler object referenced by one or more texture handles." Signed-off-by: Samuel Pitoiset --- src/mesa/main/samplerobj.c | 42

[Mesa-dev] [RFC PATCH 02/65] mesa: implement ARB_bindless_texture

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/main/context.c | 3 + src/mesa/main/dd.h | 17 + src/mesa/main/mtypes.h | 34 ++ src/mesa/main/samplerobj.c | 6 + src/mesa/main/shared.c | 12 + src/mesa/main/texobj.c

[Mesa-dev] [RFC PATCH 04/65] mesa: add support for glUniformHandleui64*ARB()

2017-05-19 Thread Samuel Pitoiset
Bindless sampler/image handles are represented using 64-bit unsigned integers. The ARB_bindless_texture spec says: "The error INVALID_OPERATION is generated by UniformHandleui64{v}ARB if the sampler or image uniform being updated has the "bound_sampler" or "bound_image" layout

[Mesa-dev] [RFC PATCH 01/65] mapi: add GL_ARB_bindless_texture entry points

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mapi/glapi/gen/ARB_bindless_texture.xml | 100 src/mapi/glapi/gen/Makefile.am | 1 + src/mapi/glapi/gen/gl_API.xml | 4 +- src/mapi/glapi/gen/gl_genexec.py|

[Mesa-dev] [RFC PATCH 07/65] mesa: refuse to change textures when a handle is allocated

2017-05-19 Thread Samuel Pitoiset
The ARB_bindless_texture spec says: "The error INVALID_OPERATION is generated by TexImage*, CopyTexImage*, CompressedTexImage*, TexBuffer*, TexParameter*, as well as other functions defined in terms of these, if the texture object to be modified is referenced by one or more texture

[Mesa-dev] [RFC PATCH 06/65] mesa: refuse to update tex parameters when a handle is allocated

2017-05-19 Thread Samuel Pitoiset
The ARB_bindless_texture spec says: "The ARB_bindless_texture spec says: "The error INVALID_OPERATION is generated by TexImage*, CopyTexImage*, CompressedTexImage*, TexBuffer*, TexParameter*, as well as other functions defined in terms of these, if the texture object to be modified

[Mesa-dev] [RFC PATCH 12/65] glsl: process uniform samplers declared bindless

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/compiler/glsl/link_uniforms.cpp | 83 - src/compiler/glsl/shader_cache.cpp | 23 ++ src/mesa/program/program.c | 4 ++ 3 files changed, 99 insertions(+), 11 deletions(-) diff

[Mesa-dev] [RFC PATCH 03/65] mesa: add support for unsigned 64-bit vertex attributes

2017-05-19 Thread Samuel Pitoiset
This adds support in the VBO and array code to handle unsigned 64-bit vertex attributes as specified by ARB_bindless_texture. Signed-off-by: Samuel Pitoiset --- src/mapi/glapi/gen/apiexec.py | 3 +++ src/mesa/main/api_loopback.c | 4 src/mesa/main/varray.c

Re: [Mesa-dev] [RFC PATCH 00/65] ARB_bindless_texture for RadeonSI

2017-05-19 Thread Ilia Mirkin
Great work, Samuel! Is this available in a branch somewhere? On Fri, May 19, 2017 at 12:52 PM, Samuel Pitoiset wrote: > Hi, > > This series implements ARB_bindless_texture for RadeonSI. > > Reminder: the GLSL compiler part is already upstream. > > This series has been

[Mesa-dev] [PATCH 1/3] radeonsi: move building llvm.SI.load.const into ac_build_buffer_load

2017-05-19 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 50 src/amd/common/ac_llvm_build.h | 3 +- src/amd/common/ac_nir_to_llvm.c | 2 +- src/gallium/drivers/radeonsi/si_shader.c | 23 +++ 4 files

[Mesa-dev] [PATCH 2/3] radeonsi: use ac_build_buffer_load for shader buffer loads

2017-05-19 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 30 ++- 1 file changed, 7 insertions(+), 23 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c b/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c index

[Mesa-dev] [PATCH 3/4] i965: Use correct vs and gs size for gen6_upload_urb.

2017-05-19 Thread Rafael Antognolli
The documentation for SNB says that it expects a value [0,4] = [1,5], but we don't really check whether the parameters are actually >= 1. Additionally, on blorp we are calling gen6_upload_urb(gs_size = 0), which causes it to be sent as -1, and that shouldn't be valid, but seems to be just

[Mesa-dev] [PATCH 2/4] genxml: Increase the field length by 1 bit.

2017-05-19 Thread Rafael Antognolli
VS, HS, DS and GS URB Starting Address field on the respective 3DSTATE_URB_?S instructions is 5 bits on gen7. However, if VS is the only step enabled, the whole URB gets allocated (except for push constant space) for the VS. But on some configurations, the size of the URB is 32 "chunks" (8kB each

[Mesa-dev] [PATCH 4/4] i965: Move urb state emitting code to genxml.

2017-05-19 Thread Rafael Antognolli
Both gen6 and gen7+ were moved into the same function, but the code is still quite split. In order to make both functions have the same signature, I moved the logic from upload_urb on gen6 that sets gs_size to vs_size into genX(upload_urb), so we don't have to pass gs_size to that function

[Mesa-dev] [PATCH 1/4] genxml: Add macros for exporting genxml functions.

2017-05-19 Thread Rafael Antognolli
The GENX_DECL() macro is used to declare prototypes of a given function to each gen, so we don't have to manually add each of them to headers. The second macro, genX_find(), is used to call the right function for the running gen, but from code that is not gen specific. Signed-off-by: Rafael

[Mesa-dev] [RFC PATCH 49/65] radeonsi: add si_init_descriptor_list() helper

2017-05-19 Thread Samuel Pitoiset
This will be used in order to initialize resident descriptors for bindless textures/images. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_descriptors.c | 15 +++ 1 file changed, 15 insertions(+) diff --git

[Mesa-dev] [RFC PATCH 24/65] gallium: add PIPE_CAP_BINDLESS_TEXTURE

2017-05-19 Thread Samuel Pitoiset
Whether bindless texture operations are supported by the underlying driver. Signed-off-by: Samuel Pitoiset --- src/gallium/docs/source/screen.rst | 2 ++ src/gallium/drivers/etnaviv/etnaviv_screen.c | 1 +

[Mesa-dev] [RFC PATCH 28/65] tc: add ARB_bindless_texture support

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/auxiliary/util/u_threaded_context.c| 147 + .../auxiliary/util/u_threaded_context_calls.h | 4 + 2 files changed, 151 insertions(+) diff --git

[Mesa-dev] [RFC PATCH 20/65] mesa: pass gl_program to _mesa_associate_uniform_storage()

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/program/ir_to_mesa.cpp| 7 --- src/mesa/program/ir_to_mesa.h | 4 ++-- src/mesa/state_tracker/st_glsl_to_nir.cpp | 3 +-- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 3 +--

[Mesa-dev] [RFC PATCH 33/65] st/glsl_to_tgsi: add support for bindless images

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 78 -- 1 file changed, 63 insertions(+), 15 deletions(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp

[Mesa-dev] [RFC PATCH 32/65] st/glsl_to_tgsi: add support for bindless samplers

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 27 +-- 1 file changed, 21 insertions(+), 6 deletions(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp index

[Mesa-dev] [RFC PATCH 22/65] mesa: handle bindless uniforms bound to texture/image units

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/main/uniform_query.cpp | 122 ++-- 1 file changed, 116 insertions(+), 6 deletions(-) diff --git a/src/mesa/main/uniform_query.cpp b/src/mesa/main/uniform_query.cpp index

[Mesa-dev] [RFC PATCH 26/65] ddebug: add ARB_bindless_texture support

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/ddebug/dd_context.c | 61 + 1 file changed, 61 insertions(+) diff --git a/src/gallium/drivers/ddebug/dd_context.c b/src/gallium/drivers/ddebug/dd_context.c index

[Mesa-dev] [RFC PATCH 10/65] mesa: store bindless samplers as PROGRAM_UNIFORM

2017-05-19 Thread Samuel Pitoiset
Old-style samplers (ie. bound samplers) are stored as PROGRAM_SAMPLER, while bindless ones are PROGRAM_UNIFORM. Signed-off-by: Samuel Pitoiset --- src/mesa/program/ir_to_mesa.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Mesa-dev] [RFC PATCH 17/65] mesa: add update_single_shader_texture_used() helper

2017-05-19 Thread Samuel Pitoiset
This will also be used for looping over bindless samplers bound to texture units. Signed-off-by: Samuel Pitoiset --- src/mesa/main/uniforms.c | 67 +++- 1 file changed, 38 insertions(+), 29 deletions(-) diff --git

Re: [Mesa-dev] [RFC PATCH 00/65] ARB_bindless_texture for RadeonSI

2017-05-19 Thread Samuel Pitoiset
On 05/19/2017 07:05 PM, Ilia Mirkin wrote: Great work, Samuel! Is this available in a branch somewhere? Thanks Ilia! Here's the branch: https://cgit.freedesktop.org/~hakzsam/mesa/log/?h=arb_bindless_texture On Fri, May 19, 2017 at 12:52 PM, Samuel Pitoiset

Re: [Mesa-dev] Bug in 17.1.0-rc4 source packaging for swr?

2017-05-19 Thread Emil Velikov
On 19 May 2017 at 13:11, Chuck Atkins wrote: > Would it be feasible for packaging purposes to generate multiple headers, > i.e. gen_builder._llvm38.hpp, gen_builder_llvm39.hpp, > gen_builder_llvm40.hpp, etc. and then have gen_builder.hpp be a stub that > just has

[Mesa-dev] [PATCH 3/3] radeonsi: document why we can't use SMEM for shader buffer loads yet

2017-05-19 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 22 -- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c b/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c index

Re: [Mesa-dev] Bug in 17.1.0-rc4 source packaging for swr?

2017-05-19 Thread Rowley, Timothy O
On May 19, 2017, at 10:26 AM, Emil Velikov > wrote: On 19 May 2017 at 13:11, Chuck Atkins > wrote: Would it be feasible for packaging purposes to generate multiple headers, i.e.

[Mesa-dev] [RFC PATCH 29/65] tgsi: add new Bindless flag to tgsi_instruction_texture

2017-05-19 Thread Samuel Pitoiset
Old-style images are identified using TGSI_FILE_SAMPLER, but bindless samplers can be TGSI_FILE_CONSTANT or TGSI_FILE_TEMPORARY. To avoid backend compilers to be confused, this adds a new flag that will only be set for bindless samplers. Signed-off-by: Samuel Pitoiset

[Mesa-dev] [RFC PATCH 52/65] radeonsi: implement ARB_bindless_texture

2017-05-19 Thread Samuel Pitoiset
This implements the Gallium interface. Decompression of resident textures/images will follow in the next patches. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_descriptors.c | 340 ++ src/gallium/drivers/radeonsi/si_pipe.c

[Mesa-dev] [RFC PATCH 56/65] radeonsi: decompress DCC for resident textures/images

2017-05-19 Thread Samuel Pitoiset
Analogous to bound textures/images. We should also update the resident descriptors and disable COMPRESSION_EN for avoiding useless DCC fetches, but I postpone this optimization for a separate series. Signed-off-by: Samuel Pitoiset ---

[Mesa-dev] [RFC PATCH 55/65] radeonsi: add si_update_check_render_feedback() helper

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_descriptors.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c b/src/gallium/drivers/radeonsi/si_descriptors.c index

[Mesa-dev] [RFC PATCH 58/65] radeonsi: isolate real framebuffer changes from the decompression passes

2017-05-19 Thread Samuel Pitoiset
When a stencil buffer is part of the framebuffer state, it is decompressed but because it's bindles, all draw calls set stencil_dirty_level_mask to 1. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeonsi/si_blit.c | 8

[Mesa-dev] [RFC PATCH 27/65] trace: add ARB_bindless_texture support

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/trace/tr_context.c | 114 + 1 file changed, 114 insertions(+) diff --git a/src/gallium/drivers/trace/tr_context.c b/src/gallium/drivers/trace/tr_context.c index

[Mesa-dev] [RFC PATCH 42/65] st/mesa: add st_create_{texture, image}_handle_from_unit() helper

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_texture.c | 42 + 1 file changed, 42 insertions(+) diff --git a/src/mesa/state_tracker/st_texture.c b/src/mesa/state_tracker/st_texture.c index 2e9856dcdf..65f86f2b4f

[Mesa-dev] [RFC PATCH 41/65] st/mesa: add st_convert_image_from_unit() helper

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_atom_image.c | 33 ++--- src/mesa/state_tracker/st_texture.h| 5 + 2 files changed, 27 insertions(+), 11 deletions(-) diff --git a/src/mesa/state_tracker/st_atom_image.c

[Mesa-dev] [RFC PATCH 38/65] st/mesa: implement ARB_bindless_texture

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_cb_texture.c | 84 ++ 1 file changed, 84 insertions(+) diff --git a/src/mesa/state_tracker/st_cb_texture.c b/src/mesa/state_tracker/st_cb_texture.c index

[Mesa-dev] [RFC PATCH 47/65] st/mesa: enable ARB_bindless_texture

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_extensions.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/state_tracker/st_extensions.c b/src/mesa/state_tracker/st_extensions.c index 2fa7ba7797..80695580cb 100644 ---

[Mesa-dev] [RFC PATCH 43/65] st/mesa: add infrastructure for storing bound texture/image handles

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_context.c | 2 + src/mesa/state_tracker/st_context.h | 11 ++ src/mesa/state_tracker/st_texture.c | 77 + src/mesa/state_tracker/st_texture.h | 5 +++ 4 files

[Mesa-dev] [RFC PATCH 40/65] st/mesa: make convert_sampler_from_unit() non-static

2017-05-19 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/mesa/state_tracker/st_atom_sampler.c | 14 -- src/mesa/state_tracker/st_texture.h | 5 + 2 files changed, 13 insertions(+), 6 deletions(-) diff --git a/src/mesa/state_tracker/st_atom_sampler.c

Re: [Mesa-dev] [PATCH 1/9] mesa: Handle common extension checks with more compact code

2017-05-19 Thread Nanley Chery
On Fri, May 19, 2017 at 06:38:03AM -0700, Ian Romanick wrote: > From: Ian Romanick > > The previous code handled everything with the general case. I noticed > that every time I converted an open-coded check to use a > _mesa_has_EXT_foo() function, the text size of the

Re: [Mesa-dev] Bug in 17.1.0-rc4 source packaging for swr?

2017-05-19 Thread Emil Velikov
On 19 May 2017 at 17:46, Rowley, Timothy O wrote: > > > On May 19, 2017, at 10:26 AM, Emil Velikov wrote: > > The original idea by Tim sounds OK imho and I'm actually giving it a try. > Are you referring to using a llvm-3.9 generated

Re: [Mesa-dev] [PATCH v13 03/36] dri: support DRIimage creation from dmabufs with modifiers

2017-05-19 Thread Jason Ekstrand
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote: > From: Pekka Paalanen > > add createImageFromDmaBufs2 function which accepts per-plane dmabuf > format modifiers. > > Signed-off-by: Pekka Paalanen >

Re: [Mesa-dev] [PATCH v13 04/36] egl/dri2: Create EGLImages with dmabuf modifiers

2017-05-19 Thread Jason Ekstrand
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote: > From: Varad Gautam > > Allow creating EGLImages with dmabuf format modifiers when target is > EGL_LINUX_DMA_BUF_EXT for EGL_EXT_image_dma_buf_import_modifiers. > > v2: > - clear modifier

Re: [Mesa-dev] [PATCH v13 05/36] dri: add queryDmaBufFormats and queryDmaBufModifiers to DRIimage

2017-05-19 Thread Jason Ekstrand
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote: > From: Varad Gautam > > these allow querying the driver for supported dmabuf formats and > modifiers. > > v2: move to __DRIimageExtension version 16. > v3: return GLBoolean for error

Re: [Mesa-dev] [PATCH v13 16/36] i965: Remove same-FD restriction for imports

2017-05-19 Thread Jason Ekstrand
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote: > Intel hardware requires that all planes of an image come from the same > buffer, which is currently implemented by testing that all FDs are > numerically the same. > > However, when going through a winsys (e.g.) or

Re: [Mesa-dev] [PATCH 1/9] mesa: Handle common extension checks with more compact code

2017-05-19 Thread Ilia Mirkin
On Fri, May 19, 2017 at 1:28 PM, Nanley Chery wrote: > The only thing I think may need > addressing is endianness. FYI, I'm happy to provide BE support, both reviewing code and testing. ___ mesa-dev mailing list

Re: [Mesa-dev] [PATCH 2/2] i965: Enable ASTC HDR for Broxton

2017-05-19 Thread Anuj Phogat
On Thu, May 18, 2017 at 3:53 PM, Nanley Chery wrote: > This platform passes the following GLES3 tests: > ES3-CTS.functional.texture.compressed.astc.endpoint_value_hdr_cem_* > We've hdr tests in piglit as well. With those passing, both patches are: Reviewed-by: Anuj Phogat

[Mesa-dev] [PATCH 1/2] anv: automake: list shared libraries after the static ones

2017-05-19 Thread Emil Velikov
From: Emil Velikov The compiler can discard the shared ones from the link chain, since there is no user (the static libraries) before it on the command line. Cc: mesa-sta...@lists.freedesktop.org Reported-by: Laurent Carlier Signed-off-by: Emil

[Mesa-dev] [PATCH 2/2] radv: automake: list shared libraries after the static ones

2017-05-19 Thread Emil Velikov
From: Emil Velikov Analogous to previous commit - the compiler can discard xcb + wayland libs, since there is no user (the static libraries) before it on the command line. Cc: mesa-sta...@lists.freedesktop.org Signed-off-by: Emil Velikov

Re: [Mesa-dev] [PATCH 14/16] anv: Advertise both 32-bit and 48-bit heaps when we have enough memory

2017-05-19 Thread Jason Ekstrand
On Thu, May 18, 2017 at 2:01 PM, Jason Ekstrand wrote: > --- > src/intel/vulkan/anv_device.c | 42 ++ > ++-- > 1 file changed, 36 insertions(+), 6 deletions(-) > > diff --git a/src/intel/vulkan/anv_device.c

Re: [Mesa-dev] [PATCH] i965: Add RGBX8888 and RGBA8888 to EGL configure and reorder the list

2017-05-19 Thread Emil Velikov
On 18 May 2017 at 23:01, Rob Herring wrote: > On Thu, May 18, 2017 at 5:25 AM, Emil Velikov > wrote: >> On 18 May 2017 at 05:10, Chih-Wei Huang wrote: >>> 2017-05-18 12:01 GMT+08:00 Xu, Randy : >

Re: [Mesa-dev] [PATCH] Android: r600: fix build when LLVM is disabled

2017-05-19 Thread Emil Velikov
Jfyi: feel free to add Fixes tag as you see fit. We parse though and automatically nominate patches for -stable :-) On 19 May 2017 at 13:04, Rob Herring wrote: > On Fri, May 12, 2017 at 11:55 AM, Rob Herring wrote: >> There's still an error after my recent

Re: [Mesa-dev] [PATCH v13 07/36] egl: implement eglQueryDmaBufModifiersEXT

2017-05-19 Thread Jason Ekstrand
Just FYI: I'm not checking that you implement the EGL API correctly. I'm mostly checking for whether or not it's using DRI correctly. On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote: > From: Varad Gautam > > query and return supported dmabuf

[Mesa-dev] [PATCH] automake: add SWR LLVM gen_builder.hpp workaround

2017-05-19 Thread Emil Velikov
From: Emil Velikov As gen_builder.hpp file is generated, it contains information that is specific to the LLVM version it originates from. As suggested by Tim, the file seems to be forwards compatible. So in order to produce ship a file which will work everywhere we

Re: [Mesa-dev] [PATCH] android: add -Wl, --build-id=sha1 to LDFLAGS for libvulkan_intel

2017-05-19 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand On Fri, May 19, 2017 at 4:25 AM, Tapani Pälli wrote: > Just like is done on desktop and what is expected by the build-id code. > > Signed-off-by: Tapani Pälli > --- > > Jason, I commented before

Re: [Mesa-dev] [PATCH v13 06/36] egl: implement eglQueryDmaBufFormatsEXT

2017-05-19 Thread Jason Ekstrand
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote: > From: Varad Gautam > > allow egl clients to query the dmabuf formats supported on this platform. > > v2: return EGLBoolean. > v3: Check DRIimageExtension version before querying (Daniel

Re: [Mesa-dev] [PATCH v13 24/36] i965/miptree: Add a return for updating of winsys

2017-05-19 Thread Jason Ekstrand
It's a bit annoying, but I think this would all make more sense if we moved 23 and 24 to before 19. Then we could drop no_aux from 21. Then again, I'm sure Ben has some good reason why that's not practical and I'm fine with leaving the order as-is. It's hard to review but there's a lot of

Re: [Mesa-dev] [PATCH 9/9] nouveau: s/nv04_surface_/nouveau_surface_/

2017-05-19 Thread Francisco Jerez
Ilia Mirkin writes: > I kinda see it both ways - yeah, the functions are the same and it's > all shared, so your patch makes sense. OTOH, all of these functions > (which do anything) have a nv04/nv10/nv20 prefix, which makes it > easier to separate stuff out by generation

Re: [Mesa-dev] Bug in 17.1.0-rc4 source packaging for swr?

2017-05-19 Thread Chuck Atkins
Would it be feasible for packaging purposes to generate multiple headers, i.e. gen_builder._llvm38.hpp, gen_builder_llvm39.hpp, gen_builder_llvm40.hpp, etc. and then have gen_builder.hpp be a stub that just has something like: #include #if llvm_version >= 4.0 #include "gen_builder_llvm40.hpp"

Re: [Mesa-dev] [PATCH v13 21/36] i965: Restructure CCS disabling

2017-05-19 Thread Jason Ekstrand
On Fri, May 19, 2017 at 2:38 AM, Daniel Stone wrote: > From: Ben Widawsky > > Make the code only disable CCS when it has to, unlike before where it > disabled CCS and enabled it when it could. This is much more inline with > how it should work in a few

Re: [Mesa-dev] [PATCH 2/2] i965: Enable ASTC HDR for Broxton

2017-05-19 Thread Nanley Chery
On Fri, May 19, 2017 at 10:41:22AM -0700, Anuj Phogat wrote: > On Thu, May 18, 2017 at 3:53 PM, Nanley Chery wrote: > > This platform passes the following GLES3 tests: > > ES3-CTS.functional.texture.compressed.astc.endpoint_value_hdr_cem_* > > > We've hdr tests in piglit as

Re: [Mesa-dev] [Mesa-stable] [PATCH] automake: add SWR LLVM gen_builder.hpp workaround

2017-05-19 Thread Rowley, Timothy O
Thanks for doing this; I would have been hunting for the dist-hook: magic for a while. Tested “make dist” on llvm-3.9.0 (works) and llvm-4.0/llvm-svn (fails, expected desired behavior). Built result of llvm-3.9.0 “make dist” with llvm-4.0 and llvm-svn and it compiles/works. Reviewed-by: Tim

Re: [Mesa-dev] [PATCH 14/16] anv: Advertise both 32-bit and 48-bit heaps when we have enough memory

2017-05-19 Thread Gustaw Smolarczyk
2017-05-18 23:01 GMT+02:00 Jason Ekstrand : > --- > src/intel/vulkan/anv_device.c | 42 -- > 1 file changed, 36 insertions(+), 6 deletions(-) > > diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c > index

[Mesa-dev] [PATCH] vulkan/wsi/wayland: Fix proxy wrappers for swapchain recreation

2017-05-19 Thread Philipp Zabel
Before the swapchain event queue is destroyed, all proxy objects that reference it must be dropped. Otherwise we risk a use-after-free if a frame callback event or buffer release events are received afterwards. This happens when an application destroys and recreates a swapchain in FIFO mode

Re: [Mesa-dev] [PATCH 2/2] gallium: Add renderonly-based support for pl111+vc4.

2017-05-19 Thread Eric Anholt
Emil Velikov writes: > On 17 May 2017 at 20:13, Emil Velikov wrote: >> On 17 May 2017 at 18:53, Eric Anholt wrote: >>> Emil Velikov writes: >>> Hi Eric, On 11 May 2017 at 00:06, Eric

[Mesa-dev] [Bug 101110] Build failure in GNOME Continuous

2017-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101110 Bug ID: 101110 Summary: Build failure in GNOME Continuous Product: Mesa Version: git Hardware: Other OS: All Status: NEW Severity: normal

Re: [Mesa-dev] [PATCH 2/2] gallium: Add renderonly-based support for pl111+vc4.

2017-05-19 Thread Emil Velikov
On 19 May 2017 at 23:21, Eric Anholt wrote: > Emil Velikov writes: > >> On 17 May 2017 at 20:13, Emil Velikov wrote: >>> On 17 May 2017 at 18:53, Eric Anholt wrote: Emil Velikov

[Mesa-dev] [Bug 98833] [REGRESSION, bisected] Wayland revert commit breaks non-Vsync fullscreen frame updates

2017-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98833 --- Comment #20 from Eero Tamminen --- (In reply to Daniel Stone from comment #19) > (In reply to Daniel Stone from comment #18) > > Right, here we are: > >

Re: [Mesa-dev] [PATCH 2/2] gallium: Add renderonly-based support for pl111+vc4.

2017-05-19 Thread Emil Velikov
On 19 May 2017 at 23:41, Emil Velikov wrote: > On 19 May 2017 at 23:21, Eric Anholt wrote: >> Emil Velikov writes: >> >>> On 17 May 2017 at 20:13, Emil Velikov wrote: On 17 May 2017 at 18:53,

[Mesa-dev] [Bug 101110] Build failure in GNOME Continuous

2017-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101110 --- Comment #1 from Emil Velikov --- Seems like I've missed some corner case - I'm looking into it. -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the

Re: [Mesa-dev] [PATCH v13 27/36] i965: Change resolve flags to enum

2017-05-19 Thread Jason Ekstrand
So, I think I'm going to end up doing a fairly significant rework of resolves over the course of the next couple of weeks. Carry on with the branch as is and I'll figure out how to rebase it on top of whatever changes I do later. But it'll probably be significant. Just a heads up. --Jason On

Re: [Mesa-dev] [PATCH 14/16] anv: Advertise both 32-bit and 48-bit heaps when we have enough memory

2017-05-19 Thread Jason Ekstrand
On Fri, May 19, 2017 at 2:53 PM, Gustaw Smolarczyk wrote: > 2017-05-18 23:01 GMT+02:00 Jason Ekstrand : > > --- > > src/intel/vulkan/anv_device.c | 42 ++ > ++-- > > 1 file changed, 36 insertions(+), 6 deletions(-)

[Mesa-dev] [Bug 101110] Build failure in GNOME Continuous

2017-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101110 Emil Velikov changed: What|Removed |Added Status|NEW |RESOLVED

Re: [Mesa-dev] [Mesa-stable] [PATCH] automake: add SWR LLVM gen_builder.hpp workaround

2017-05-19 Thread Emil Velikov
On 19 May 2017 at 20:50, Rowley, Timothy O wrote: > Thanks for doing this; I would have been hunting for the dist-hook: magic > for a while. > > Tested “make dist” on llvm-3.9.0 (works) and llvm-4.0/llvm-svn (fails, > expected desired behavior). > > Built result of

Re: [Mesa-dev] [PATCH 1/9] mesa: Handle common extension checks with more compact code

2017-05-19 Thread Ian Romanick
On 05/19/2017 10:28 AM, Nanley Chery wrote: > On Fri, May 19, 2017 at 06:38:03AM -0700, Ian Romanick wrote: >> From: Ian Romanick >> >> The previous code handled everything with the general case. I noticed >> that every time I converted an open-coded check to use a >>

[Mesa-dev] [PATCH] docs: Document ASTC extension support for SKL and BXT

2017-05-19 Thread Nanley Chery
Cc: Anuj Phogat Signed-off-by: Nanley Chery --- docs/features.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/features.txt b/docs/features.txt index e18bf54a48..05d776be08 100644 --- a/docs/features.txt +++

Re: [Mesa-dev] [PATCH 03/16] anv: Handle color layout transitions from the UNINITIALIZED layout

2017-05-19 Thread Nanley Chery
On Thu, May 18, 2017 at 02:00:50PM -0700, Jason Ekstrand wrote: > This causes dEQP-VK.api.copy_and_blit.resolve_image.partial.* to start > failing due to test bugs. See CL 1031 for a test fix. > --- > src/intel/vulkan/anv_blorp.c | 40 > ++ >

Re: [Mesa-dev] [PATCH 04/16] anv: Handle transitioning depth from UNDEFINED to other layouts

2017-05-19 Thread Nanley Chery
On Thu, May 18, 2017 at 02:00:51PM -0700, Jason Ekstrand wrote: > --- > src/intel/vulkan/anv_image.c | 12 ++-- > src/intel/vulkan/genX_cmd_buffer.c | 9 + > 2 files changed, 11 insertions(+), 10 deletions(-) > > diff --git a/src/intel/vulkan/anv_image.c

Re: [Mesa-dev] [PATCH] Android: r600: fix build when LLVM is disabled

2017-05-19 Thread Rob Herring
On Fri, May 12, 2017 at 11:55 AM, Rob Herring wrote: > There's still an error after my recent clean-up if LLVM is not patched to > enable AMDGPU target: > > external/mesa3d/src/amd/common/ac_llvm_util.c:38:2: error: implicit > declaration of function

<    1   2   3   >