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
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
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
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
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
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 +--
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
---
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
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
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
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
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
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
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
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
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(+)
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 ++
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
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
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
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
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
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|
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
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
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
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
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
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
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
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
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
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
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
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
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 +
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
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 +--
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
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
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
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
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
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
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
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
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
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.
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
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
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
---
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
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
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
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
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
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
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
---
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
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
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
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
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
>
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
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
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
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
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
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
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
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
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 :
>
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
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
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
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
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
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
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
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"
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
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
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
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
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
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
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
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
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:
> >
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,
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
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
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(-)
https://bugs.freedesktop.org/show_bug.cgi?id=101110
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
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
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
>>
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
+++
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
> ++
>
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
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
101 - 200 of 200 matches
Mail list logo