On Tue, 2017-12-12 at 08:20 +0100, Iago Toral wrote:
> On Mon, 2017-12-11 at 08:01 -0800, Jason Ekstrand wrote:
> > On Mon, Dec 11, 2017 at 12:55 AM, Iago Toral
> > wrote:
> > > This didn't get any reviews yet. Any takers?
> > >
> > >
> > >
> > > On Fri, 2017-12-01 at 13:46 +0100, Iago Toral Qu
Thanks Ken!
Reviewed-by: Iago Toral Quiroga
On Wed, 2017-12-13 at 12:25 -0800, Kenneth Graunke wrote:
> The hardware doesn't support this, and isl_surf_get_mcs_surf will
> fail.
>
> I feel a bit bad replicating this logic, but we want to decide up
> front.
>
> This fixes the following test whe
On 12/12/2017 07:00 PM, Ian Romanick wrote:
On 11/19/2017 11:18 PM, Tapani Pälli wrote:
Add GL_GPU_DISJOINT_EXT and enable extension when ARB_timer_query
is supported by the driver.
Following dEQP cases pass:
dEQP-EGL.functional.get_proc_address.extension.gl_ext_disjoint_timer_query
d
Fixes 56 crashes in radeonsi.
---
src/mesa/state_tracker/st_glsl_to_nir.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/state_tracker/st_glsl_to_nir.cpp
b/src/mesa/state_tracker/st_glsl_to_nir.cpp
index 7c9e76a2dce..5683dfe 100644
--- a/src/mesa/state_tracker/st_glsl_to_nir.c
On Wed, Dec 13, 2017 at 10:45 PM, Mario Kleiner
wrote:
> On 12/13/2017 05:27 PM, Marek Olšák wrote:
>>
>> Mario, can we push these patches?
>>
>> Marek
>>
>
> Sorry for the late response and thanks for the reviews! Was a bit sick the
> last days, so couldn't think or do anything about the recent c
We need to move this to a separate loop because
nir_compact_varyings() can alter the IR of a previous stage.
Fixes: 6648bd68fd27 "st/glsl_to_nir: enable NIR link time opts"
---
src/mesa/state_tracker/st_glsl_to_nir.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/state_track
On 12/13/2017 05:27 PM, Marek Olšák wrote:
Mario, can we push these patches?
Marek
Sorry for the late response and thanks for the reviews! Was a bit sick
the last days, so couldn't think or do anything about the recent
comments so far.
I think all patches have a r-b by you and/or Tapani,
On Wed, Dec 13, 2017 at 4:43 AM, Roland Scheidegger wrote:
> Am 13.12.2017 um 04:34 schrieb Brian Paul:
>> The sample mask is used even if msaa is not explicity enabled when we
>> have a framebuffer with multisampled surfaces. That's DX behavior and
>> what the Radeon drivers do. Not sure about
---
src/mesa/drivers/dri/i965/brw_draw.c | 41
1 file changed, 41 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_draw.c
b/src/mesa/drivers/dri/i965/brw_draw.c
index 4945dec..9fd44e4 100644
--- a/src/mesa/drivers/dri/i965/brw_draw.c
+++ b/src/mesa/dr
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
index b87d356..e21e1da 100644
--- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.
Better commit message:
Re-enable regular fast-clears (CCS_D) on gen9+
This reverts commit ee57b15ec764736e2d5360beaef9fb2045ed0f68, "i965:
Disable regular fast-clears (CCS_D) on gen9+". How taht we've fixed the
issue with too many different aux usages in the render cache, it shou
This makes sure we flush things out such as depth writes prior to using
a surface through the render cache. Besides the overhead of a hash
table look-up, this function is harmless to call repeatedly with the
same arguments.
Cc: "17.3"
---
src/mesa/drivers/dri/i965/brw_draw.c | 2 ++
1 file chan
This reverts commit ee57b15ec764736e2d5360beaef9fb2045ed0f68.
Cc: "17.3"
---
src/mesa/drivers/dri/i965/brw_meta_util.c | 10 -
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 57 ---
2 files changed, 25 insertions(+), 42 deletions(-)
diff --git a/src/mesa/drivers
This lets us perform render cache flushes whenever a surface goes from
being used with one aux+format to a different aux+format.
This is the "proper" fix for https://bugs.freedesktop.org/102435.
ee57b15ec764736e2d5360beaef9fb2045ed0f68 which was really just a partial
revert of 3e57e9494c2279580ad6
On Mon, Nov 27, 2017 at 07:06:01PM -0800, Jason Ekstrand wrote:
> Currently, this helper does nothing but we call it every place where an
> image is written through the render pipeline. This will allow us to
> properly mark the aux state so that we can handle resolves correctly.
> ---
> src/intel
https://bugs.freedesktop.org/show_bug.cgi?id=103852
Ben Clapp changed:
What|Removed |Added
Version|17.2|17.3
--
You are receiving this mail becaus
https://bugs.freedesktop.org/show_bug.cgi?id=103852
--- Comment #1 from Ben Clapp ---
I've done some testing with mesa 17.3.0 on my computer with the RX 580 (using
the mesa 17.3.0-1 package available in debian unstable).
All of the previously mentioned bugs are still present on 17.3.0, so I'm
upd
---
src/mesa/state_tracker/st_glsl_to_nir.cpp | 22 +-
1 file changed, 17 insertions(+), 5 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_nir.cpp
b/src/mesa/state_tracker/st_glsl_to_nir.cpp
index 70c5daaa225..be34031bfb5 100644
--- a/src/mesa/state_tracker/st_gls
https://bugs.freedesktop.org/show_bug.cgi?id=104251
Bug ID: 104251
Summary: st_update_single_texture: Assertion `texObj' failed.
Product: Mesa
Version: 17.3
Hardware: Other
OS: All
Status: NEW
Severity: no
On Wed, 2017-12-13 at 17:23 -0500, Jan Vesely wrote:
> On Wed, 2017-12-13 at 13:54 -0800, Dylan Baker wrote:
> > Quoting Jan Vesely (2017-12-13 12:53:25)
> > > On Wed, 2017-12-13 at 09:47 -0800, Dylan Baker wrote:
> > > > +if (with_gallium_va or with_gallium_vdpau or with_gallium_omx or
> > > > +
Hi Robert,
Thanks for looking into this! We need to decide if we want:
(1) A common struct that implementations can subclass, i.e:
struct blah_gralloc_handle {
alloc_handle_t alloc_handle;
int x, y, z;
}
(2) An accessor library that vendors can implement, i.e:
struct drmAndro
Quoting Jan Vesely (2017-12-13 14:23:21)
> On Wed, 2017-12-13 at 13:54 -0800, Dylan Baker wrote:
> > Quoting Jan Vesely (2017-12-13 12:53:25)
> > > On Wed, 2017-12-13 at 09:47 -0800, Dylan Baker wrote:
> > > > +if (with_gallium_va or with_gallium_vdpau or with_gallium_omx or
> > > > +with_galli
On Wed, 2017-12-13 at 13:54 -0800, Dylan Baker wrote:
> Quoting Jan Vesely (2017-12-13 12:53:25)
> > On Wed, 2017-12-13 at 09:47 -0800, Dylan Baker wrote:
> > > +if (with_gallium_va or with_gallium_vdpau or with_gallium_omx or
> > > +with_gallium_xvmc or with_dri)
> > > + pipe_loader_link_with
Quoting Eric Engestrom (2017-12-13 13:43:24)
> On Wednesday, 2017-12-13 10:33:05 -0800, Dylan Baker wrote:
> > cc: Jason Ekstrand
> > Signed-off-by: Dylan Baker
> > ---
> > meson.build | 9 +
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/meson.build b/meson.build
> > index 6e
Quoting Jan Vesely (2017-12-13 13:36:25)
> On Wed, 2017-12-13 at 13:01 -0800, Francisco Jerez wrote:
> > Jan Vesely writes:
> >
>
> autotools build works OK with no-rtti build of llvm, I've been running
> like that for some time. LLVM_CXXFLAGS which include fno-rtti are only
> used to build libc
Dylan Baker writes:
> Quoting Francisco Jerez (2017-12-13 13:01:29)
>> Jan Vesely writes:
>> > meson asked for libdrm_amdgpu dependency even though I'm only building
>> > clover+r600g driver.
>> > after a bit of fiddling with PATH and PK_CONFIG_PATH to pick up the latest
>> > llvm/liblclc
>> >
Quoting Francisco Jerez (2017-12-13 13:01:29)
> Jan Vesely writes:
> > meson asked for libdrm_amdgpu dependency even though I'm only building
> > clover+r600g driver.
> > after a bit of fiddling with PATH and PK_CONFIG_PATH to pick up the latest
> > llvm/liblclc
> > linking failed with:
> > src/
Quoting Jan Vesely (2017-12-13 12:53:25)
> On Wed, 2017-12-13 at 09:47 -0800, Dylan Baker wrote:
> > +if (with_gallium_va or with_gallium_vdpau or with_gallium_omx or
> > +with_gallium_xvmc or with_dri)
> > + pipe_loader_link_with += libgalliumvl
> > +else
> > + pipe_loader_link_with += libga
On Wed, Dec 13, 2017 at 12:25 PM, Kenneth Graunke
wrote:
> The hardware doesn't support this, and isl_surf_get_mcs_surf will fail.
>
> I feel a bit bad replicating this logic, but we want to decide up front.
>
I do too. One of these days, I'd like to move things around so that we
just call isl_
Rb
On Wed, Dec 13, 2017 at 1:08 PM, Rob Herring wrote:
> Commit bb1e6ff161c9 ("spirv: Add a prepass to set types on vtn_values")
> added generation of vtn_gather_types.c, but forgot to add it to the
> Android build files.
>
> Fixes: bb1e6ff161c9 ("spirv: Add a prepass to set types on vtn_values
On Wednesday, 2017-12-13 10:33:05 -0800, Dylan Baker wrote:
> cc: Jason Ekstrand
> Signed-off-by: Dylan Baker
> ---
> meson.build | 9 +
> 1 file changed, 9 insertions(+)
>
> diff --git a/meson.build b/meson.build
> index 6e5ae4d24e9..b00c1ffc3dc 100644
> --- a/meson.build
> +++ b/meson
Jan Vesely writes:
> On Wed, 2017-12-13 at 13:01 -0800, Francisco Jerez wrote:
>> Jan Vesely writes:
>>
>> > On Wed, 2017-12-13 at 09:47 -0800, Dylan Baker wrote:
>> > > This has only been compile tested.
>> > >
>> > > v2: - Have a single option for opencl (Eric E)
>> > > - fix typo "tgis"
On Wed, 2017-12-13 at 13:01 -0800, Francisco Jerez wrote:
> Jan Vesely writes:
>
> > On Wed, 2017-12-13 at 09:47 -0800, Dylan Baker wrote:
> > > This has only been compile tested.
> > >
> > > v2: - Have a single option for opencl (Eric E)
> > > - fix typo "tgis" -> "tgsi" (Curro)
> > > -
https://bugs.freedesktop.org/show_bug.cgi?id=104231
Dylan Baker changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Jan Vesely writes:
> On Wed, 2017-12-13 at 09:47 -0800, Dylan Baker wrote:
>> This has only been compile tested.
>>
>> v2: - Have a single option for opencl (Eric E)
>> - fix typo "tgis" -> "tgsi" (Curro)
>> - Don't add "lib" prefix to pipe loader libraries, which matches the
>> au
On Wed, Dec 13, 2017 at 11:58 AM, Robert Foss wrote:
> Hey Rob&Emil,
>
> I've chatted to both of you about this patch, and it seemed acceptable
> to both of you.
>
> Can I have a s-o-b for either of you?
I've applied it and pushed to master.
Rob
___
me
Commit bb1e6ff161c9 ("spirv: Add a prepass to set types on vtn_values")
added generation of vtn_gather_types.c, but forgot to add it to the
Android build files.
Fixes: bb1e6ff161c9 ("spirv: Add a prepass to set types on vtn_values")
Cc: Jason Ekstrand
Signed-off-by: Rob Herring
---
src/compiler
Thank you for the perfectly timed fix. :)
Reviewed-by: Ian Romanick
On 12/12/2017 11:50 AM, Dylan Baker wrote:
> CC: Nicolai Hähnle
> CC: Reviewed-by: Ian Romanick
> CC: Mark Janes
> Fixes: 46b21b8f906 ("mesa: add GL_ARB_gl_spirv boilerplate")
> Signed-off-by: Dylan Baker
> ---
> src/mesa/m
On Tuesday, 2017-12-12 11:58:14 -0800, Dylan Baker wrote:
> Quoting Eric Engestrom (2017-12-12 06:15:58)
> > On Monday, 2017-12-11 11:50:01 -0800, Dylan Baker wrote:
> > > Quoting Eric Engestrom (2017-12-11 07:55:30)
> > > > On Friday, 2017-12-08 16:27:21 -0800, Dylan Baker wrote:
> > > > > Signed-
On Wed, 2017-12-13 at 09:47 -0800, Dylan Baker wrote:
> This has only been compile tested.
>
> v2: - Have a single option for opencl (Eric E)
> - fix typo "tgis" -> "tgsi" (Curro)
> - Don't add "lib" prefix to pipe loader libraries, which matches the
> autotools behavior
>
> cc: Cur
On Wed, Dec 13, 2017 at 11:30 AM, Robert Foss wrote:
> Mark magic member of alloc_handle_t as const.
> Also bump the version of alloc_handle_t.
>
> Sign-off-by: Robert Foss
> ---
> android/alloc_handle.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/android/alloc_h
On Wed, Dec 13, 2017 at 11:30 AM, Robert Foss wrote:
> This struct is used in mesa and drm_hwcomposer.
> Versions of if have been implemented in several grallocs:
> drm_gralloc, gbm_gralloc, minigbm and intel-minigbm.
>
> Other than the 1:1 move of the struct a new generic name
> has been chosen a
The hardware doesn't support this, and isl_surf_get_mcs_surf will fail.
I feel a bit bad replicating this logic, but we want to decide up front.
This fixes the following test when run with --deqp-surface-width=16384:
-
GTF-GL46.gtf30.GL3Tests.framebuffer_blit.framebuffer_blit_error_blitframebuff
On Tue, Dec 12, 2017 at 6:08 PM, Samuel Pitoiset
wrote:
> Use 16_ABGR instead of 32_ABGR if Z isn't written.
>
> Ported from RadeonSI.
>
> No CTS regressions on Polaris.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/common/ac_nir_to_llvm.c | 65
> ++---
>
On 30/11/2017 23:57, Ian Romanick wrote:
On 11/30/2017 09:28 AM, Eduardo Lima Mitev wrote:
From: Nicolai Hähnle
v2: * Add a gl_shader_spirv_data member to gl_shader, which already
encapsulates a gl_spirv_module where the binary will be saved.
(Eduardo Lima)
* Just use the 'spirv_
From: Alejandro Piñeiro
---
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 4d17393948a..39680ed194a 100644
--- a/src/mesa/drivers/dri/i965/intel_ex
From: Alejandro Piñeiro
v2: Rebase update after changes on previous patches.
---
src/mesa/drivers/dri/i965/brw_context.c | 9 -
src/mesa/main/context.c | 2 ++
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/me
From: Alejandro Piñeiro
We can use it to get real values for ARB_spirv_extensions methods.
v2: Rebase update after changes on previous patches.
---
src/mesa/main/mtypes.h | 3 +++
src/mesa/main/spirv_extensions.c | 20 +++-
2 files changed, 22 insertions(+), 1 deletio
From: Alejandro Piñeiro
Add a struct to maintain which SPIR-V extensions are supported, and an
utility method to initialize it based on
nir_spirv_supported_capabilities.
v2:
* Fixing code style (Ian Romanick)
* Adding a prefix (spirv) to fill_supported_spirv_extensions (Ian Romanick)
v3: re
From: Alejandro Piñeiro
Ideally this should be generated somehow. One option would be gather
all the extension dependencies listed on the core grammar, but there
would be the possibility of not including some of the extensions.
Note that spirv-tools is doing it just slightly better, as it has a
From: Alejandro Piñeiro
v2:
* Mention extension gap at gl_API.xml (Emil Velikov)
* Bail with INVALID_ENUM if extension not available on getStringi (Emil
Velikov)
* Use EXTRA_EXT macro when defining the extension at
get.c/get_hash_params.py (Emil Velikov)
* Rename source files (spirve
Hi,
This series is the latest version of the support for ARB_spirv_extensions on
i965.
It was split from a previous series that included also support for gl_spirv,
but since this is an independent extension, we have chosen to send it
separately to streamline review of the two series.
This ext
---
src/intel/tools/gen_batch_decoder.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/src/intel/tools/gen_batch_decoder.c
b/src/intel/tools/gen_batch_decoder.c
index ab566aa..782e9fd 100644
--- a/src/intel/tools/gen_batch_decoder.c
+++ b/src/intel/tools/gen_batch_decoder.c
@@
Unfortunately, in aubinator and aubinator_error_decode we don't always
know how many of a given state we have, so we must guess. One day,
we'll come up with a way to annotate the batch to solve this problem.
---
src/intel/tools/gen_batch_decoder.c | 91 +
1 fil
---
src/intel/Makefile.tools.am | 1 +
src/intel/tools/aubinator.c | 719 ++--
src/intel/tools/meson.build | 3 +-
3 files changed, 33 insertions(+), 690 deletions(-)
diff --git a/src/intel/Makefile.tools.am b/src/intel/Makefile.tools.am
index 9332ae3..
---
src/intel/common/gen_decoder.h | 2 +
src/intel/tools/gen_batch_decoder.c | 159
2 files changed, 161 insertions(+)
diff --git a/src/intel/common/gen_decoder.h b/src/intel/common/gen_decoder.h
index d401929..2532a36 100644
--- a/src/intel/common/gen
---
src/intel/common/gen_decoder.c | 17 ++---
src/intel/common/gen_decoder.h | 5 +
2 files changed, 19 insertions(+), 3 deletions(-)
diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c
index 1f6b984..c6ddae0 100644
--- a/src/intel/common/gen_decoder.c
+
---
src/intel/tools/gen_batch_decoder.c | 145
1 file changed, 145 insertions(+)
diff --git a/src/intel/tools/gen_batch_decoder.c
b/src/intel/tools/gen_batch_decoder.c
index 8d9f15e..b8c1350 100644
--- a/src/intel/tools/gen_batch_decoder.c
+++ b/src/intel/too
---
src/intel/tools/gen_batch_decoder.c | 73 +
1 file changed, 73 insertions(+)
diff --git a/src/intel/tools/gen_batch_decoder.c
b/src/intel/tools/gen_batch_decoder.c
index f7064b1..5b01879 100644
--- a/src/intel/tools/gen_batch_decoder.c
+++ b/src/intel/tool
---
src/intel/tools/gen_batch_decoder.c | 95 +
1 file changed, 95 insertions(+)
diff --git a/src/intel/tools/gen_batch_decoder.c
b/src/intel/tools/gen_batch_decoder.c
index 5f46714..f7064b1 100644
--- a/src/intel/tools/gen_batch_decoder.c
+++ b/src/intel/tool
The shared framework can now do everything that aubinator_error_decode
ever did and more. It's time to make the switch.
---
src/intel/Makefile.tools.am | 1 +
src/intel/tools/aubinator_error_decode.c | 238 +--
src/intel/tools/meson.build |
---
src/intel/common/gen_decoder.h | 48 +++
src/intel/tools/gen_batch_decoder.c | 258
2 files changed, 306 insertions(+)
create mode 100644 src/intel/tools/gen_batch_decoder.c
diff --git a/src/intel/common/gen_decoder.h b/src/intel/common/gen_deco
---
src/intel/common/gen_decoder.c | 3 ++-
src/intel/common/gen_decoder.h | 1 +
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c
index 333b5ab..1f6b984 100644
--- a/src/intel/common/gen_decoder.c
+++ b/src/intel/common/
Both aubinator and aubinator_error_decode try and do the same task of
decoding batches. They both have code to try and decode various things
such as shaders from the batch. All of that code is completely different
between the two.
This little series reworks the two to use a common gen_print_batc
I'm actually going to drop these patches for now. It seems that
implicitly enabling MSAA when the sample mask != ~0 is expected behavior
with D3D and some (most?) hardware, including our virtual GPU (I'm
fixing a bug there).
-Brian
On 12/13/2017 12:35 PM, Marek Olšák wrote:
Reviewed-by: Mar
On Mon, Nov 27, 2017 at 07:06:00PM -0800, Jason Ekstrand wrote:
> This is copied and pasted from the similar macro we added to ISL.
> ---
> src/intel/vulkan/anv_cmd_buffer.c | 40
> ---
> 1 file changed, 25 insertions(+), 15 deletions(-)
>
This patch is
Revie
Reviewed-by: Marek Olšák
Marek
On Mon, Dec 11, 2017 at 9:09 PM, Brian Paul wrote:
> If we're doing a Z -> Z MSAA blit (for example) we need to enable
> msaa rasterization when drawing the quads so that we can properly
> write the per-sample values.
>
> This fixes a number of Piglit ext_framebuf
On Mon, Dec 11, 2017 at 9:09 PM, Brian Paul wrote:
> Will be used by the next patch.
> ---
> src/gallium/auxiliary/util/u_blitter.c | 8
> src/gallium/auxiliary/util/u_blitter.h | 11 +++
> 2 files changed, 19 insertions(+)
>
> diff --git a/src/gallium/auxiliary/util/u_blitter.c
---
src/mesa/program/ir_to_mesa.cpp | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/program/ir_to_mesa.cpp
index 29198509a6c..5d56c2ef44a 100644
--- a/src/mesa/program/ir_to_mesa.cpp
+++ b/src/mesa/program/ir_to_mesa.cpp
@@ -36,6
This is the main fork of the shader compilation code-path, where a NIR
shader is obtained by calling spirv_to_nir() or glsl_to_nir(),
depending on its nature..
v2: Use 'spirv_data' member from gl_linked_shader to know which method
to call. (Timothy Arceri)
Reviewed-by: Timothy Arceri
---
src
This is the equivalent to link_shaders() from
src/compiler/glsl/linker.cpp, but for SPIR-V programs. It just
creates the program and its gl_linked_shader objects, giving drivers
the opportunity to implement any linking of SPIR-V shaders they choose,
at a later stage.
v2: Bail out if we see more th
This is a reference to the spirv_data object stored in gl_shader, which
stores shader SPIR-V data that is needed during linking too.
Reviewed-by: Timothy Arceri
---
src/mesa/main/mtypes.h| 8
src/mesa/main/shaderobj.c | 1 +
2 files changed, 9 insertions(+)
diff --git a/src/mesa/ma
v2: Use 'spirv_data' from gl_linked_shader instead, to check if shader
is SPIR-V. (Timothy Arceri)
Reviewed-by: Timothy Arceri
---
src/mesa/drivers/dri/i965/brw_link.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_link.cpp
b/src/mesa/dr
From: Alejandro Piñeiro
ARB_gl_spirv adds the ability to use SPIR-V binaries, and a new
method, glSpecializeShader. From OpenGL 4.6 spec, section 7.2.1
"Shader Specialization", error table:
INVALID_VALUE is generated if does not name a valid
entry point for .
INVALID_VALUE is generate
From: Nicolai Hähnle
v2: * Use gl_spirv_validation instead of spirv_to_nir.
This method just validates the shader. The conversion to NIR will
happen later, during linking. (Alejandro Piñeiro)
* Use gl_shader_spirv_data struct to store the SPIR-V data.
(Eduardo Lima)
* Use the '
From: Nicolai Hähnle
For drivers to declare which SPIR-V features they support.
v2: Don't use a pointer (Ian Romanick)
---
src/mesa/main/mtypes.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index b372921e9f0..2cc844da1d0 100644
--- a/sr
From: Alejandro Piñeiro
Needed for ARB_gl_spirv. Right now those are the same that the intel
vulkan driver, but those are not shared. From the ARB_spirv_extensions
spec:
"3. If a new GL extension is added that includes SPIR-V support via
a new SPIR-V extension does it's SPIR-V extension al
This is basically a wrapper around spirv_to_nir() that includes
arguments setup and post-conversion validation.
v2: * Rebase update (SpirVCapabilities not a pointer anymore,
spirv_to_nir_options added, and others).
* Code-style improvements and remove debug hunk. (Timothy Arceri)
Reviewed
Hi,
This is the 3rd version of the series adding initial support for ARB_gl_spirv.
Previous versions of this series included also support for
ARB_spirv_extensions, but we have decided to split the two to ease review. So I
will be sending a second series with only the patches for spirv_extension
On Wed, 2017-12-13 at 09:47 -0800, Dylan Baker wrote:
> This enables the SWR driver, but doesn't actually hook it up to any of
> the targets yet. I felt like this patch was big and complicated enough
> without adding that.
>
> v2: - Fix typo 'delemeited' -> 'delimited' (Eric E)
> - Fix type 'e
https://bugs.freedesktop.org/show_bug.cgi?id=104229
mercuriete changed:
What|Removed |Added
Version|unspecified |17.3
OS|All
On Mon, Nov 27, 2017 at 07:05:56PM -0800, Jason Ekstrand wrote:
> ---
> src/intel/vulkan/anv_image.c | 58
> ++
> src/intel/vulkan/anv_private.h | 5
> 2 files changed, 63 insertions(+)
>
> diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulk
On Mon, Nov 27, 2017 at 07:05:59PM -0800, Jason Ekstrand wrote:
> This moves it to being based on layout_to_aux_usage instead of being
> hard-coded based on bits of a priori knowledge of how transitions
> interact with layouts. This conceptually simplifies things because
> we're now using layout_t
From: Boyuan Zhang
Remove is_idr flag, and use idr_pic_flag provided by vaapi directly
Signed-off-by: Boyuan Zhang
Reviewed-by: Christian König
---
src/gallium/state_trackers/va/picture.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/va
From: Boyuan Zhang
Remove is_idr flag since not being used anymore.
Signed-off-by: Boyuan Zhang
Reviewed-by: Christian König
---
src/gallium/include/pipe/p_video_state.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/include/pipe/p_video_state.h
b/src/gallium/include/pipe/p_v
From: Boyuan Zhang
Vaapi encode interface provides idr frame flags, where omx interface doesn't.
Therefore, change to use picture type to determine idr frame, which will
work for both interfaces.
Signed-off-by: Boyuan Zhang
Reviewed-by: Leo Liu
Reviewed-by: Christian König
---
src/gallium/dr
From: Boyuan Zhang
Signed-off-by: Boyuan Zhang
---
src/gallium/drivers/radeon/radeon_vce_52.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeon/radeon_vce_52.c
b/src/gallium/drivers/radeon/radeon_vce_52.c
index 10bf718..a941c47 100644
--- a/src/gall
On 13/12/17 02:34, srol...@vmware.com wrote:
From: Roland Scheidegger
Cube texture wrapping is a bit special since the values (post face
projection) always are within [0,1], so we took advantage of that and
omitted some clamps.
However, we can still get NaNs (either because the coords already h
cc: Jason Ekstrand
Signed-off-by: Dylan Baker
---
meson.build | 9 +
1 file changed, 9 insertions(+)
diff --git a/meson.build b/meson.build
index 6e5ae4d24e9..b00c1ffc3dc 100644
--- a/meson.build
+++ b/meson.build
@@ -29,6 +29,15 @@ project(
default_options : ['buildtype=debugoptimiz
Hey Rob&Emil,
I've chatted to both of you about this patch, and it seemed acceptable
to both of you.
Can I have a s-o-b for either of you?
Rob.
On Mon, 2017-12-04 at 19:22 +0100, Robert Foss wrote:
> From: Tomasz Figa
>
> There is no API available to properly query the
> IMPLEMENTATION_DEFIN
Signed-off-by: Dylan Baker
---
src/gallium/drivers/r600/meson.build | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/r600/meson.build
b/src/gallium/drivers/r600/meson.build
index 2132dbb33ad..5899518a2e8 100644
--- a/src/gallium/drivers/r600/meson.bu
This has only been compile tested.
v2: - Have a single option for opencl (Eric E)
- fix typo "tgis" -> "tgsi" (Curro)
- Don't add "lib" prefix to pipe loader libraries, which matches the
autotools behavior
cc: Curro Jerez
cc: Jan Vesely
cc: Aaron Watry
Signed-off-by: Dylan Baker
Currently that's dri, libgl-xlib, and osmesa.
v2: - put drivers on a separate line from normal dependencies (Eric E)
Signed-off-by: Dylan Baker
Reviewed-by: Eric Engestrom
---
src/gallium/meson.build| 1 -
src/gallium/targets/dri/meson.build| 4 ++--
src/gallium/tar
This enables the SWR driver, but doesn't actually hook it up to any of
the targets yet. I felt like this patch was big and complicated enough
without adding that.
v2: - Fix typo 'delemeited' -> 'delimited' (Eric E)
- Fix type 'errror' -> 'error' (Eric E)
- Use variables to hold files inste
Change the alloc_handle_t format to signify the fourcc pixel format
code instead of the Android pixel format definition.
This is desirable since the fourcc code is what is used within the DRM
subsystem. Naturally translation will still have to happen somewhere.
Also bump the alloc_handle_t versio
This series moves {gbm,drm,cros}_gralloc_handle_t struct to libdrm,
since at least 4 implementations exist, and share a lot of contents.
The idea is to keep the common stuff defined in one place, and libdrm
is the common codebase to all of these platforms.
Additionally, having this struct defined
This struct is used in mesa and drm_hwcomposer.
Versions of if have been implemented in several grallocs:
drm_gralloc, gbm_gralloc, minigbm and intel-minigbm.
Other than the 1:1 move of the struct a new generic name
has been chosen and variables have had comments added to them.
Sign-off-by: Rober
Mark magic member of alloc_handle_t as const.
Also bump the version of alloc_handle_t.
Sign-off-by: Robert Foss
---
android/alloc_handle.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/android/alloc_handle.h b/android/alloc_handle.h
index b722615638e7..fe93ad7e6567 1006
The version variable will be used for versioning of this
struct and the corresponding accessor functions.
Signed-off-by: Robert Foss
---
android/alloc_handle.h | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/android/alloc_handle.h b/android/alloc_handle.h
index 75c
The name member of alloc_handle_t is no longer needed and has been removed.
The version field has also been bumped.
Signed-off-by: Robert Foss
---
android/alloc_handle.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/android/alloc_handle.h b/android/alloc_handle.h
index fe
1 - 100 of 122 matches
Mail list logo