On 30/01/18 01:24, Roland Scheidegger wrote:
> Am 29.01.2018 um 17:03 schrieb Alejandro Piñeiro:
>> On 29/01/18 16:38, Roland Scheidegger wrote:
>>> Am 29.01.2018 um 09:09 schrieb Alejandro Piñeiro:
On 27/01/18 12:09, Roland Scheidegger wrote:
> Am 27.01.2018 um 09:52 schrieb Alejandro Piñ
On Wed, 2018-02-21 at 09:58 -0800, Nanley Chery wrote:
> On Wed, Feb 21, 2018 at 09:18:49AM +0100, Iago Toral Quiroga wrote:
> > We were only resolving the first.
> >
> > v2:
> > - Do not require that the number of layers on dst and src are an
> > exact match, it is okay if the dst has more
On Thu, Feb 22, 2018 at 7:23 AM, Rob Herring wrote:
>
> On Wed, Feb 21, 2018 at 1:22 PM, Emil Velikov
> wrote:
> > On 21 February 2018 at 18:50, Rob Herring wrote:
> >> On Wed, Feb 21, 2018 at 10:01 AM, Emil Velikov
> >> wrote:
> >>> Hi all,
> >>>
> >>> Pardon for dropping in late. I think yo
From: Mathias Fröhlich
Provided the _DrawVAO and the derived state that is maintained if we have
the _DrawVAO set, implement a method to incrementally update the array of
gl_vertex_array input pointers.
v2: Add some more comments.
Rename _vbo_array_init to _vbo_init_inputs.
Rename vbo_co
From: Mathias Fröhlich
Switch over to use the _DrawVAO for all the array type draws.
The _DrawVAO needs to be set before we enter _mesa_update_state, so move
setting the draw method in front of the first call to _mesa_update_state
which is in turn called from the *validate*Draw* calls. Using the
From: Mathias Fröhlich
We will need the flush_vertices argument later in this series.
Signed-off-by: Mathias Fröhlich
---
src/mesa/drivers/common/meta.c | 18 +++---
src/mesa/main/enable.c | 4 ++--
src/mesa/main/varray.c | 30 +++---
src/me
From: Mathias Fröhlich
Correct VBO_MATERIAL_SHIFT value.
The functions will be used next in this series.
Signed-off-by: Mathias Fröhlich
---
src/mesa/vbo/vbo_attrib.h | 4 ++--
src/mesa/vbo/vbo_private.h | 53 ++
2 files changed, 55 insertions(+),
From: Mathias Fröhlich
To get equivalent information than get_vp_mode(), track the vertex
processing mode in a per context variable at
gl_vertex_program_state::_VPMode.
This aims to replace get_vp_mode() as seen in the vbo module.
But instead of the get_vp_mode() implementation which only gives c
From: Mathias Fröhlich
Finally use an internal VAO to execute display list draws. Avoid
duplicate state validation for display list draws. Remove client arrays
previously used exclusively for display lists.
Signed-off-by: Mathias Fröhlich
---
src/mesa/vbo/vbo_save.c | 37 ++--
sr
From: Mathias Fröhlich
During the patch series this VAO gets populated with either the currently
bound VAO or an internal VAO that will be used for immediate mode and
dlist rendering.
v2: More comments about the _DrawVAO, filter and enabled mask.
Rename _DrawVAOEnabled to _DrawVAOEnabledAttr
From: Mathias Fröhlich
Is now unused.
Signed-off-by: Mathias Fröhlich
---
src/mesa/vbo/vbo_private.h | 27 ---
1 file changed, 27 deletions(-)
diff --git a/src/mesa/vbo/vbo_private.h b/src/mesa/vbo/vbo_private.h
index e44a521b11..49922892e5 100644
--- a/src/mesa/vbo/vb
From: Mathias Fröhlich
We will need the flush_vertices argument later in this series.
Signed-off-by: Mathias Fröhlich
---
src/mesa/drivers/common/meta.c | 16 +---
src/mesa/main/bufferobj.c | 2 +-
src/mesa/main/varray.c | 15 ---
src/mesa/main/varray.h
From: Mathias Fröhlich
Change vertex_attrib_binding() to _mesa_vertex_attrib_binding(), add a
flush_vertices argument, and make it publically available.
The function will be needed later in the series.
Signed-off-by: Mathias Fröhlich
---
src/mesa/main/varray.c | 38
From: Mathias Fröhlich
At those places where we used get_vp_mode() use
gl_vertex_program_state::_VPMode instead.
Signed-off-by: Mathias Fröhlich
---
src/mesa/vbo/vbo_exec.c | 4 ++--
src/mesa/vbo/vbo_exec_array.c | 11 ++-
src/mesa/vbo/vbo_exec_draw.c | 5 +++--
src/mesa/vbo/v
From: Mathias Fröhlich
Finally use an internal VAO to execute immediate mode draws. Avoid
duplicate state validation for immediate mode draws. Remove client arrays
previously used exclusively for immediate mode draws.
Signed-off-by: Mathias Fröhlich
---
src/mesa/vbo/vbo_context.c | 8 +++
From: Mathias Fröhlich
Hi Brian,
Thanks for the review on v1!!
The following series moves over to draw via a VAO in all draw
paths. At the current stage this introduces an additional
indirection for the immediate mode and display list draws.
But in the mid term using a VAO should enable drivers
From: Mathias Fröhlich
VAOs will be used in the next change as immutable object across multiple
contexts. Only reference counting may write concurrently on the VAO. So,
make the reference count thread safe for those and only those VAO objects.
Signed-off-by: Mathias Fröhlich
---
src/mesa/main/
We don't want filtering for integer textures, same as depth/stencil.
Fixes: KHR-GL45.direct_state_access.renderbuffers_storage_multisample
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/nv50/nv50_blit.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium
On Tue, Feb 20, 2018 at 6:28 AM, Kenneth Graunke wrote:
> Both KHR_blend_equation_advanced and ARB_bindless_texture provide
> layout qualifiers, and are exposed in compatibility contexts. We
> need to parse the layout qualifier as a token in order for those
> to work, but forgot to extend this ch
6 is Acked-By: Aaron Watry
7-11 (regardless of my comment in 9) are Reviewed-By: Aaron Watry
--Aaron
On Wed, Feb 21, 2018 at 4:50 PM, Pierre Moreau wrote:
> Reviewed-by: Francisco Jerez
> Signed-off-by: Pierre Moreau
> ---
> src/gallium/state_trackers/clover/api/platform.cpp | 4 ++--
> sr
On 22 February 2018 at 11:58, Dieter Nützel wrote:
> Am 22.02.2018 02:35, schrieb Dave Airlie:
>>
>> 2018-02-22 10:57 GMT+10:00 Dieter Nützel :
>>>
>>> Making all in vulkan
>>> make[3]: Verzeichnis „/opt/mesa/src/vulkan“ wird betreten
>>> make[3]: *** Keine Regel vorhanden, um das Ziel
>>> „/unsta
On Wed, Feb 21, 2018 at 4:50 PM, Pierre Moreau wrote:
> One flag that needs to be tracked is whether a library is allowed to
> received mathematics optimisations or not, as the authorisation is given
> when creating the library while the optimisations are specified when
> creating the executable.
Am 22.02.2018 02:35, schrieb Dave Airlie:
2018-02-22 10:57 GMT+10:00 Dieter Nützel :
Making all in vulkan
make[3]: Verzeichnis „/opt/mesa/src/vulkan“ wird betreten
make[3]: *** Keine Regel vorhanden, um das Ziel
„/unstable/linux-dmabuf/linux-dmabuf-unstable-v1.xml“,
benötigt von „wsi/linux-dma
https://bugs.freedesktop.org/show_bug.cgi?id=105161
Kenneth Graunke changed:
What|Removed |Added
Resolution|--- |NOTABUG
Status|ASSIGNED
Patches 1-5:
Reviewed-By: Aaron Watry
On Wed, Feb 21, 2018 at 4:50 PM, Pierre Moreau wrote:
> Signed-off-by: Pierre Moreau
> ---
> src/gallium/state_trackers/clover/core/device.cpp | 6 +-
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/src/gallium/state_trackers/clover/c
2018-02-22 10:57 GMT+10:00 Dieter Nützel :
> Making all in vulkan
> make[3]: Verzeichnis „/opt/mesa/src/vulkan“ wird betreten
> make[3]: *** Keine Regel vorhanden, um das Ziel
> „/unstable/linux-dmabuf/linux-dmabuf-unstable-v1.xml“,
> benötigt von „wsi/linux-dmabuf-unstable-v1-protocol.c“, zu ers
Making all in vulkan
make[3]: Verzeichnis „/opt/mesa/src/vulkan“ wird betreten
make[3]: *** Keine Regel vorhanden, um das Ziel
„/unstable/linux-dmabuf/linux-dmabuf-unstable-v1.xml“,
benötigt von „wsi/linux-dmabuf-unstable-v1-protocol.c“, zu erstellen.
Schluss.
make[3]: Verzeichnis „/opt/mesa
On Wed, Feb 21, 2018 at 1:14 AM, Chad Versace wrote:
> On Thu 21 Dec 2017, Daniel Vetter wrote:
>> On Thu, Dec 21, 2017 at 12:22 AM, Kristian Kristensen
>> wrote:
>>> On Wed, Dec 20, 2017 at 12:41 PM, Miguel Angel Vico
>>> wrote:
On Wed, 20 Dec 2017 11:54:10 -0800 Kristian Høgsberg
On Wed 21 Feb 2018, Daniel Vetter wrote:
> On Tue, Feb 20, 2018 at 10:14:47PM -0800, Chad Versace wrote:
> > On Thu 21 Dec 2017, Daniel Vetter wrote:
> > > On Thu, Dec 21, 2017 at 12:22 AM, Kristian Kristensen
> > > wrote:
> > >> On Wed, Dec 20, 2017 at 12:41 PM, Miguel Angel Vico
> > >> wrote:
On Wed 24 Jan 2018, Jason Ekstrand wrote:
> Cc: Chad Versace
> ---
> src/intel/isl/isl.h | 24
> 1 file changed, 20 insertions(+), 4 deletions(-)
>
> diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h
> index e3acb0e..cf53b5a 100644
> --- a/src/intel/isl/isl.h
> +++
Kenneth Graunke writes:
> On Thursday, February 8, 2018 8:47:00 PM PST Emil Velikov wrote:
>> Rejected (9)
>>
>> Jason Ekstrand (2):
>> e52a9f18d69c94b7cb7f81361cdb9e2582c3d742 i965: Replace
>> draw_aux_buffer_disabled with draw_aux_usage
>> 20f70ae3858bc213e052a8434f0e63
On Tue 20 Feb 2018, Jason Ekstrand wrote:
> On Mon, Feb 19, 2018 at 10:01 AM, Chad Versace <[1]chadvers...@chromium.org>
> wrote:
>
> On Wed 24 Jan 2018, Jason Ekstrand wrote:
> > We're about to start letting the intel_obj->_Format be the "real"
> > texture format. For depth/stencil t
Signed-off-by: Pierre Moreau
---
Notes:
v3: Remove the const from the length argument of clCreateProgramWithIL
src/gallium/state_trackers/clover/api/dispatch.cpp | 2 +-
src/gallium/state_trackers/clover/api/program.cpp | 8
2 files changed, 9 insertions(+), 1 deletion(-)
diff --
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/device.cpp | 3 +--
src/gallium/state_trackers/clover/api/program.cpp | 2 +-
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/device.cpp
b/src/gallium/state_trackers/clove
Signed-off-by: Pierre Moreau
---
Notes:
v3:
* Remove the const on the length argument to CreateProgramWithILKHR
(Francisco
Jerez);
* Capitalize comment (Francisco Jerez);
* Store the IL as a std::vector instead of a pointer + size (Francisco
Jerez);
* Remove the destru
Signed-off-by: Pierre Moreau
---
Notes:
v3: Advertise cl_khr_il_program if if the device support NATIVE as IR
src/gallium/state_trackers/clover/api/platform.cpp | 2 ++
src/gallium/state_trackers/clover/core/device.cpp | 4 +++-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a
Signed-off-by: Pierre Moreau
---
Notes:
v3:
* Bump the required version of SPIRV-Tools to the latest release;
* Add a dependency on llvm-spirv.
configure.ac | 10 ++
meson.build | 4
2 files changed, 14 insertions(+)
diff --git a/configure.ac b/configure.ac
index 8a9
Signed-off-by: Pierre Moreau
---
Notes:
v3: Switch from using a pointer attribute to a vector (Francisco Jerez)
src/gallium/state_trackers/clover/api/program.cpp | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/gallium/state
Signed-off-by: Pierre Moreau
---
Notes:
v3: Throw an exception if the cl_khr_il_program extension is not supported
(Francisco Jerez)
src/gallium/state_trackers/clover/api/device.cpp | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/state_trackers/clover/api/devi
Signed-off-by: Pierre Moreau
---
.../state_trackers/clover/llvm/invocation.cpp | 26 ++
.../state_trackers/clover/llvm/invocation.hpp | 4
src/gallium/state_trackers/clover/meson.build | 2 +-
3 files changed, 31 insertions(+), 1 deletion(-)
diff --git a
Reviewed-by: Francisco Jerez
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/platform.cpp | 4 ++--
src/gallium/state_trackers/clover/core/platform.cpp | 5 +
src/gallium/state_trackers/clover/core/platform.hpp | 2 ++
3 files changed, 9 insertions(+), 2 deletions(-)
Reviewed-by: Karol Herbst
Signed-off-by: Pierre Moreau
---
include/CL/cl_ext.h | 37 +
1 file changed, 37 insertions(+)
diff --git a/include/CL/cl_ext.h b/include/CL/cl_ext.h
index 5078e8f45f..5ea4968042 100644
--- a/include/CL/cl_ext.h
+++ b/include/CL/cl_ex
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/Makefile.am | 11 +-
src/gallium/state_trackers/clover/Makefile.sources | 4 -
src/gallium/state_trackers/clover/core/program.cpp | 15 +--
src/gallium/state_trackers/clover/meson.build | 9 +-
.../state_trackers/c
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/program.cpp | 39 +-
src/gallium/state_trackers/clover/core/program.cpp | 3 +-
2 files changed, 25 insertions(+), 17 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/ga
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/core/device.cpp | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/device.cpp
b/src/gallium/state_trackers/clover/core/device.cpp
index dac03fccb9..9b15b30944 100644
--
Signed-off-by: Pierre Moreau
---
Notes:
v3:
* Dropped supported_irs() (Francisco Jerez)
* Changed supports_ir() argument type to `enum pipe_shader_ir` (Francisco
Jerez)
src/gallium/state_trackers/clover/core/device.cpp | 6 ++
src/gallium/state_trackers/clover/core/device.hpp |
Reviewed-by: Francisco Jerez
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/device.cpp | 11 +--
src/gallium/state_trackers/clover/core/device.cpp | 14 ++
src/gallium/state_trackers/clover/core/device.hpp | 1 +
3 files changed, 16 insertions(+), 10
From the OpenCL 1.2 Specification, Section 5.6.2 (about clBuildProgram):
> If program is created with clCreateProgramWithBinary, then the
> program binary must be an executable binary (not a compiled binary or
> library).
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/pr
Acked-by: Francisco Jerez
Reviewed-by: Karol Herbst
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/device.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/api/device.cpp
b/src/gallium/state_trackers/clover/api/dev
One flag that needs to be tracked is whether a library is allowed to
received mathematics optimisations or not, as the authorisation is given
when creating the library while the optimisations are specified when
creating the executable.
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/
From: Karol Herbst
v2: add more prototypes
Signed-off-by: Karol Herbst
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/dispatch.cpp | 29 +++-
src/gallium/state_trackers/clover/api/dispatch.hpp | 190 +
2 files changed, 218 insertions(+), 1 deletion
If creating a library, do not allow non-compiled object in it, as
executables are not allowed, and libraries would make it really hard to
enforce the "-enable-link-options" flag.
Reviewed-by: Francisco Jerez
Signed-off-by: Pierre Moreau
---
Notes:
v3: Re-write the explanation as to why libr
Hello,
This updated series contains some additional fixes to clover, updates to the
OpenCL headers shipped with Mesa, the removal of the TGSI backend in clover (as
unused), the introduction of a new dependency, llvm-spirv, and some more
changes summarised further down.
llvm-spirv can be currently
Build mesa 6986 completed
Commit c7e22483fe by Daniel Stone on 2/20/2018 8:56 PM:
vulkan/wsi/x11: Consistently update and return swapchain status\n\nUse a helper function for updating the swapchain status. This will be\nused later to handle VK_SUBOPTIMAL_KHR, w
Build mesa 6985 failed
Commit cdeac00267 by Timothy Arceri on 2/21/2018 3:36 AM:
nir: remove old assert\n\nThis was originally intended to make sure the remap location\nwas not -1. However the code has changed alot since then,\nthe location is now never set to
Assuming you're ok with the squash-in and modifications in my branch, the
whole lot is
Reviewed-by: Jason Ekstrand
except, of course, for the ones written by me. Let's make this happen!
--Jason
On Wed, Feb 21, 2018 at 6:05 AM, Daniel Stone wrote:
> Hi,
> This series adds support for buffer
On Wed, Feb 21, 2018 at 1:22 PM, Emil Velikov wrote:
> On 21 February 2018 at 18:50, Rob Herring wrote:
>> On Wed, Feb 21, 2018 at 10:01 AM, Emil Velikov
>> wrote:
>>> Hi all,
>>>
>>> Pardon for dropping in late. I think you've got nearly everything
>>> settled down, just sharing a couple of id
On Wed, Feb 21, 2018 at 6:05 AM, Daniel Stone wrote:
> From: Louis-Francis Ratté-Boulianne
>
> Add support for DRI3 v1.1, which allows pixmaps to be backed by
> multi-planar buffers, or those with format modifiers. This is both
> for allocating render buffers, as well as EGLImage imports from a
On Wed, Feb 21, 2018 at 11:04 AM, Matt Turner wrote:
> On Wed, Feb 21, 2018 at 10:37 AM, Anuj Phogat wrote:
>> On Wed, Feb 21, 2018 at 9:22 AM, Rafael Antognolli
>> wrote:
>>> My understanding is that this commit is enough to make the driver try to
>>> initialize, at least for i965. If that's th
On Wed, Feb 21, 2018 at 01:08:26PM -0800, Jason Ekstrand wrote:
> It's true for depth HiZ clears because we only have HiZ on single-slice
> images right now. However, for stencil-only clears there is no such
> restriction.
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 9 ++---
> 1 file changed
Similarly to vulkan where we store the clear value in the aux surface,
we can do the same in GL.
v2: Remove unneeded extra function.
v3: Use clear_value_state_size instead of clear_value_size.
Signed-off-by: Rafael Antognolli
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 10 ++
1
This warning showed up after aux_bo started being used inside
if (use_clear_address) {...
But use_clear_address depends on aux_surf being not null, in which case
aux_bo would also be set. Make the compiler happy anyway.
Signed-off-by: Rafael Antognolli
Reviewed-by: Nanley Chery
---
src/mesa/d
On Gen10, whenever the fast clear color changes, update it on the clear
color entry buffer. This allow us to use it directly when emitting the
surface state.
Signed-off-by: Rafael Antognolli
---
src/mesa/drivers/dri/i965/brw_blorp.c | 26 ++
1 file changed, 26 insertions(
On Gen10+, instead of copying the clear color from the state buffer to
the surface state, just use the address of the state buffer in the
surface state directly. This way we can avoid the copy from state buffer
to surface state.
Signed-off-by: Rafael Antognolli
---
src/intel/vulkan/anv_image.c
Store the default clear address for HiZ fast clears on a global bo, and
point to it when needed.
Signed-off-by: Rafael Antognolli
---
src/intel/vulkan/anv_device.c | 19 +++
src/intel/vulkan/anv_image.c | 10 +++---
src/intel/vulkan/anv_private.h | 1 +
3 files changed, 2
On Gen10, when emitting the surface state, use the value stored in the
clear color entry buffer by using a clear color address in the surface
state.
Signed-off-by: Rafael Antognolli
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 16
1 file changed, 16 insertions(+)
diff
On gen10+, if surface->clear_color_addr is present, use it directly
intead of copying it to the surface state.
Signed-off-by: Rafael Antognolli
---
src/intel/blorp/blorp_genX_exec.h | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/src/intel/blorp/blorp_genX_exec.h
On gen10 and newer, there's a struct describing how to use the indirect
clear color buffer, and it seems to be 8 dwords long - 4 for the clear
color and the last 4 used by the hardware, apparently depending on the
generation. So just set aside space for them since gen10.
Optionally we can later ad
Rebase of this series after lots of aux surface changes on anv.
Cc: Jason Ekstrand
Cc: Jordan Justen
Cc: Topi Pohjolainen
Rafael Antognolli (13):
anv/image: Do not override lower bits of dword.
genxml: Preserve fields that share dword space with addresses.
intel/genxml: Use a single fiel
Some instructions contain fields that are either an address or a value
of some type based on the content of other fields, such as clear color
values vs address. That works fine if these fields are in the less
significant dword, the lower 32 bits of the address, because they get
OR'ed with the addre
genxml does not support having two address fields with different names
but same position in the state struct. Both "Clear Color Address"
and "Clear Depth Address Low" mean the same thing, only for different
surface types.
To workaround this genxml limitation, rename "Clear Color Address"
to "Clear
The size of the clear color struct (expected by the hardware) is 8
dwords (isl_dev.ss.clear_value_state_size here). But we still need to
track the size of the clear color, used when memcopying it to/from the
state buffer. For that we keep isl_dev.ss.clear_value_size.
Signed-off-by: Rafael Antognol
The lower bits seem to have extra fields in every platform but gen8
(even though we don't use them in gen9). So just go ahead and avoid
using them for the address.
Signed-off-by: Rafael Antognolli
Reviewed-by: Topi Pohjolainen
---
src/intel/vulkan/anv_image.c | 30 +-
gen10 can emit the clear color by setting it on a buffer somewhere, and
then adding only the address to the surface state.
This commit add support for that on isl_surf_fill_state, and if that is
requested, skip setting the clear value itself.
v2: Add assert to make sure we are at least on gen10.
On 22/02/18 06:09, Ilia Mirkin wrote:
On Tue, Feb 20, 2018 at 8:35 PM, Timothy Arceri wrote:
Delaying unrolling and allowing NIR to do it instead has been shown
to result in better code in drivers such as i965. shader-db results
appear to should the same is true for radeonsi.
The other advanta
On Wed, Feb 21, 2018 at 1:32 PM, Daniel Stone wrote:
> Hi,
>
> On 21 February 2018 at 20:50, Jason Ekstrand wrote:
> > On Wed, Feb 21, 2018 at 6:05 AM, Daniel Stone
> wrote:
> >> @@ -832,7 +832,7 @@ x11_acquire_next_image_from_queue(struct
> x11_swapchain
> >> *chain,
> >> VkResult result =
Hi,
On 21 February 2018 at 21:24, Jason Ekstrand wrote:
> On Wed, Feb 21, 2018 at 1:22 PM, Jason Ekstrand
> wrote:
>>> + uint32_t n = 0;
>>> + uint32_t counts[2];
>>> + uint64_t *modifiers[2];
>>> +
>>> + if (mod_reply->num_drawable_modifiers) {
>>> + counts[n] = mod_reply->num_draw
Hi,
On 21 February 2018 at 20:50, Jason Ekstrand wrote:
> On Wed, Feb 21, 2018 at 6:05 AM, Daniel Stone wrote:
>> @@ -832,7 +832,7 @@ x11_acquire_next_image_from_queue(struct x11_swapchain
>> *chain,
>> VkResult result = wsi_queue_pull(&chain->acquire_queue,
>>
On Wed, Feb 21, 2018 at 1:22 PM, Jason Ekstrand
wrote:
> On Wed, Feb 21, 2018 at 6:05 AM, Daniel Stone
> wrote:
>
>> Adds support for multiple planes and buffer modifiers.
>>
>> v4: Rename "has_dri3_v1_1" to "has_dri3_modifiers"
>> Signed-off-by: Daniel Stone
>> ---
>> src/vulkan/wsi/wsi_commo
Tested-by: Rafael Antognolli
On Wed, Feb 21, 2018 at 01:08:26PM -0800, Jason Ekstrand wrote:
> It's true for depth HiZ clears because we only have HiZ on single-slice
> images right now. However, for stencil-only clears there is no such
> restriction.
> ---
> src/intel/vulkan/genX_cmd_buffer.c
On Wed, Feb 21, 2018 at 6:05 AM, Daniel Stone wrote:
> Adds support for multiple planes and buffer modifiers.
>
> v4: Rename "has_dri3_v1_1" to "has_dri3_modifiers"
> Signed-off-by: Daniel Stone
> ---
> src/vulkan/wsi/wsi_common_x11.c | 185 ++
> ++
> 1 file
On Wed, Feb 21, 2018 at 3:03 PM, Daniel Schürmann
wrote:
>
>> On Wed, Feb 21, 2018 at 1:00 PM,
>> wrote:
>>>
>>> From: Dave Airlie
>>>
>>> Co-authored-by: Daniel Schürmann
>>> Signed-off-by: Daniel Schürmann
>>> ---
>>> src/amd/common/ac_llvm_build.c| 3 +-
>>> src/amd/common/ac_nir_t
On 21 February 2018 at 15:10, Chuck Atkins wrote:
>> Something doesn't look quite right - it seems that xmesa_init_display
>> should be fixed instead.
>>
>> Currently it returns non-NULL when either of the following fail:
>> - driver.create_pipe_screen()
>> - CALLOC_STRUCT
>>
>> I would add an e
It's true for depth HiZ clears because we only have HiZ on single-slice
images right now. However, for stencil-only clears there is no such
restriction.
---
src/intel/vulkan/genX_cmd_buffer.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/src/intel/vulkan/genX_cmd_bu
For the series:
Reviewed-by: Marek Olšák
Marek
On Wed, Feb 21, 2018 at 12:42 AM, Timothy Arceri wrote:
> Fixes the following piglit tests:
>
> tests/spec/arb_tessellation_shader/execution/double-array-vs-tcs-tes.shader_test
> tests/spec/arb_tessellation_shader/execution/double-vs-tcs-tes.shade
On Wed, Feb 21, 2018 at 6:05 AM, Daniel Stone wrote:
> The FIFO thread signals shutdown by pushing UINT32_MAX into the acquire
> queue, and shutdown is signaled to it by pushing UINT32_MAX into the
> present queue.
>
> Explicitly catch a push of UINT32_MAX in either direction and treat this
> as
Reviewed-by: Marek Olšák
Marek
On Wed, Feb 21, 2018 at 7:03 AM, Timothy Arceri wrote:
> Fixes assert in glsl-1.50-gs-max-output-components piglit test.
>
> Note that the double handling will only work for doubles that
> don't take up multiple slots i.e. double and dvec2. However
> dual slot dou
On Wed, Feb 21, 2018 at 9:22 AM, Rafael Antognolli
wrote:
> My understanding is that this commit is enough to make the driver try to
> initialize, at least for i965. If that's the case, how about we add
> something like what was removed by commit
> bf1577fe0972ae910c071743dc89d261a46c2926 for CNL?
On Wed, Feb 21, 2018 at 10:37 AM, Anuj Phogat wrote:
> On Wed, Feb 21, 2018 at 9:22 AM, Rafael Antognolli
> wrote:
>> My understanding is that this commit is enough to make the driver try to
>> initialize, at least for i965. If that's the case, how about we add
>> something like what was removed
For this one, I'm making live edits to see if I like the suggested
changes. I'll send out the v2 with the comments incorporated when I'm done.
On Wed, Feb 21, 2018 at 11:32 AM, Jason Ekstrand
wrote:
> From: Daniel Stone
>
> Use a helper function for updating the swapchain status. This will be
On 21 February 2018 at 18:50, Rob Herring wrote:
> On Wed, Feb 21, 2018 at 10:01 AM, Emil Velikov
> wrote:
>> Hi all,
>>
>> Pardon for dropping in late. I think you've got nearly everything
>> settled down, just sharing a couple of ideas.
>>
>> On 21 February 2018 at 04:19, Tomasz Figa wrote:
>
On Wed, Feb 21, 2018 at 3:13 PM, Connor Abbott wrote:
> On Wed, Feb 21, 2018 at 3:03 PM, Daniel Schürmann
> wrote:
>>
>>> On Wed, Feb 21, 2018 at 1:00 PM,
>>> wrote:
From: Dave Airlie
Co-authored-by: Daniel Schürmann
Signed-off-by: Daniel Schürmann
---
sr
On Wed, Feb 21, 2018 at 10:01 AM, Emil Velikov wrote:
> Hi all,
>
> Pardon for dropping in late. I think you've got nearly everything
> settled down, just sharing a couple of ideas.
>
> On 21 February 2018 at 04:19, Tomasz Figa wrote:
>> On Wed, Feb 21, 2018 at 4:03 AM, Rob Herring wrote:
>>> On
On Wed, Feb 21, 2018 at 11:09 AM, Scott D Phillips
wrote:
> Anuj Phogat writes:
>
>> On Wed, Feb 21, 2018 at 10:00 AM, Scott D Phillips
>> wrote:
>>> Matt Turner writes:
>>>
From: Anuj Phogat
Signed-off-by: Anuj Phogat
---
include/pci_ids/i965_pci_ids.h | 9
Please split this up into 4 patches: 1 for the core nir part, 1 for
the spirv part, 1 for the ac part and finally 1 for the radv.
On Wed, Feb 21, 2018 at 7:00 PM, wrote:
> From: Dave Airlie
>
> Co-authored-by: Daniel Schürmann
> Signed-off-by: Daniel Schürmann
> ---
> src/amd/common/ac_llvm_
On Tue, Feb 20, 2018 at 10:14:47PM -0800, Chad Versace wrote:
> On Thu 21 Dec 2017, Daniel Vetter wrote:
> > On Thu, Dec 21, 2017 at 12:22 AM, Kristian Kristensen
> > wrote:
> >> On Wed, Dec 20, 2017 at 12:41 PM, Miguel Angel Vico
> >> wrote:
> >>> On Wed, 20 Dec 2017 11:54:10 -0800 Kristian Hø
Hi George,
This first showed up on one of our older dashboard machines without AVX
(Nehalem processor). You can easily reproduce it by building with
--with-swr-archs=knl,skx when the machine you're trying to use it on is
neither. Note that dri-glx seems to handle this case just fine but
xlib-gl
I believed I created the same failure by deleting libswr*.so, however in my
case (running glxgears) I hit a SEGV in XMesaCreateVisual (calling
xmdpy->screen->is_format_supported()), because xmesa_init_display() created an
xmdpy with a NULL screen. I never get to the xmesa_close_display().
Were
https://bugs.freedesktop.org/show_bug.cgi?id=105161
--- Comment #6 from Allan Sandfeld Jensen ---
Right. I think NVidia can get away with it because this requirement is new in
the KHR version, they also implement the original NV version which doesnt have
it (and like is OpenGL norm, they share th
v2: fix indentation, prefix with "4.3:" like other sections.
---
src/xdemos/glinfo_common.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/xdemos/glinfo_common.c b/src/xdemos/glinfo_common.c
index efa96e6..3668026 100644
--- a/src/xdemos/glinfo_common.c
+++ b/src/xdemos/gl
1 - 100 of 189 matches
Mail list logo
