On Thu, Nov 24, 2016 at 09:34:37AM +0200, Pohjolainen, Topi wrote:
> On Wed, Nov 23, 2016 at 01:05:31PM -0800, Jason Ekstrand wrote:
> >On Wed, Nov 23, 2016 at 1:16 AM, Topi Pohjolainen
> ><[1]topi.pohjolai...@gmail.com> wrote:
> >
> > v2: Added intel_resolve_map_clear() into intel_mi
https://bugs.freedesktop.org/show_bug.cgi?id=98833
--- Comment #2 from Pekka Paalanen ---
(In reply to Eero Tamminen from comment #0)
> Test suite has options for using single, double or triple buffering and
> toggling Vsync, but these didn't have any effect.
Just curious, how do you actually ch
On Wed, Nov 23, 2016 at 09:26:29AM -0800, Jason Ekstrand wrote:
>General comment: Does it make sense to squash this with the previous
>patch? I'm fine either way.
>On Wed, Nov 23, 2016 at 1:16 AM, Topi Pohjolainen
><[1]topi.pohjolai...@gmail.com> wrote:
>
> Signed-off-by: Top
From: Kalyan Kondapally
- initialize buffer age as 0 (Tapani)
Signed-off-by: Kalyan Kondapally
---
src/egl/drivers/dri2/egl_dri2.h | 2 ++
src/egl/drivers/dri2/platform_android.c | 24 +++-
2 files changed, 25 insertions(+), 1 deletion(-)
diff --git a/src/egl/driv
https://bugs.freedesktop.org/show_bug.cgi?id=98833
--- Comment #3 from Eero Tamminen ---
Before checking other things, I'm going to try whether this happens also with
latest kernel and on something else than SKL.
(In reply to Daniel Stone from comment #1)
> Which compositor is this? I wouldn't
Oops, commit message missed the dEQP testing results. I can add these
when committing:
On Android (running dEQP with *buffer_age*):
102 passing tests, 0 failing and 84 not supported ones
On 11/24/2016 11:38 AM, Tapani Pälli wrote:
From: Kalyan Kondapally
- initialize buffer age as 0 (Tapani
https://bugs.freedesktop.org/show_bug.cgi?id=98833
--- Comment #4 from Pekka Paalanen ---
> (In reply to Pekka Paalanen from comment #2)
> > (In reply to Eero Tamminen from comment #0)
> > > Test suite has options for using single, double or triple buffering and
> > > toggling Vsync, but these di
Hi,
patch is Reviewed-by: Christian König .
Regards,
Christian.
Am 24.11.2016 um 07:57 schrieb Namburu, Chandu-babu:
Hi ,
I have been working on an issue where the for a particular stream
playback is stuttering and there are lot of frame drops.
That particular stream has negative POC valu
This series is :
Reviewed-by: Lionel Landwerlin
On 24/11/16 04:27, Jason Ekstrand wrote:
We were reading from the "comment size" dword and incrementing by that
amount. This never caused a problem because that field was always zero.
However, experimenting with actual aub file comments indicate
Fixes multiple Vulkan CTS tests that combine anisotropy and VK_FILTER_NEAREST
in dEQP-VK.texture.filtering_anisotropy.*
---
src/intel/vulkan/genX_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/genX_state.c b/src/intel/vulkan/genX_state.c
index 4122395.
Fixes warning.
---
src/intel/vulkan/genX_pipeline.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c
index cb164ad..ada7985 100644
--- a/src/intel/vulkan/genX_pipeline.c
+++ b/src/intel/vulkan/genX_pipeline.c
@@
Fixes defined but not used warning.
---
src/intel/vulkan/genX_pipeline.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c
index ada7985..991fbf6 100644
--- a/src/intel/vulkan/genX_pipeline.c
+++ b/src/intel/vulkan/genX_pipeli
I'm sending several unrelated patches that fix different warnings when building
Mesa.
Juan A. Suarez Romero (5):
anv/pipeline: do not discard 'const' modifier
anv/pipeline: define is_dual_src_blend_factor() for gen <= 8
st/va: remove unsed variable
ttn: handle GLSL_SAMPLER_DIM_SUBPASS_MS c
Fixes a warning.
---
src/gallium/auxiliary/nir/tgsi_to_nir.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c
b/src/gallium/auxiliary/nir/tgsi_to_nir.c
index 3f05acd..e90684f 100644
--- a/src/gallium/auxiliary/nir/tgsi_to_nir.c
+++ b/src/gallium/auxiliar
And declare coded_buf in vlVaContext as "vlVaBuffer *" instead of
"struct vlVaBuffer *".
This fixes several warnings later about assignment from incompatible
pointer type.
---
src/gallium/state_trackers/va/va_private.h | 30 +++---
1 file changed, 15 insertions(+), 15 dele
pbuff is defined but not used.
---
src/gallium/state_trackers/va/surface.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/state_trackers/va/surface.c
b/src/gallium/state_trackers/va/surface.c
index f8513d9..357e85e 100644
--- a/src/gallium/state_trackers/va/surface.c
+++ b/src/gal
Hi Iago,
Looking at the history, before ed4fe3e9ba9018e68afe6fdd4f267218a537fdaa
we seem to set min/mag filter to MAPFILTER_ANISOTROPIC if maxAnisotropy
> 1. It seems your patch makes sense in using MAPFILTER_ANISOTROPIC in
the NEAREST case, but I wonder whether we should also check for
maxAn
Reviewed-by: Lionel Landwerlin
On 24/11/16 12:36, Juan A. Suarez Romero wrote:
Fixes warning.
---
src/intel/vulkan/genX_pipeline.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c
index cb164ad..ada7985 10
Reviewed-by: Lionel Landwerlin
On 24/11/16 12:36, Juan A. Suarez Romero wrote:
Fixes defined but not used warning.
---
src/intel/vulkan/genX_pipeline.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c
index ada7985..991
On 23 November 2016 at 19:07, Bas Nieuwenhuizen
wrote:
> Not needed anymore.
>
> Signed-off-by: Bas Nieuwenhuizen
> ---
> src/amd/vulkan/Makefile.am | 6 +-
> src/amd/vulkan/Makefile.sources | 3 +--
> 2 files changed, 2 insertions(+), 7 deletions(-)
>
> diff --git a/src/amd/vulkan/Mak
Hi Lionel,
On Thu, 2016-11-24 at 13:08 +, Lionel Landwerlin wrote:
> Hi Iago,
>
> Looking at the history, before
> ed4fe3e9ba9018e68afe6fdd4f267218a537fdaa
> we seem to set min/mag filter to MAPFILTER_ANISOTROPIC if
> maxAnisotropy
> > 1. It seems your patch makes sense in using MAPFILTER_
https://bugs.freedesktop.org/show_bug.cgi?id=97145
--- Comment #1 from Emil Velikov ---
Most/all of that should be resolved with commit
a89faa2022fd995af2019c886b152b49a01f9392.
Please give it a try and let me know the exact configure line (and the
summary?) if things are still odd.
--
You are
https://bugs.freedesktop.org/show_bug.cgi?id=95035
--- Comment #12 from Emil Velikov ---
Hi Frederic, please send your patches to the ML for review[1].
[1] http://mesa3d.org/devinfo.html#submitting
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact
https://bugs.freedesktop.org/show_bug.cgi?id=91724
--- Comment #5 from Emil Velikov ---
Hi Frederic, sorry for dropping the ball on this one.
Can you please attach (as plain text) the instructions and demo program that
you've mentioned - the gist seems to have been removed.
If you already have
Until now fast clear has been supported only for non-layered and
non-mipmapped buffers. However, from gen8 onwards there is hardware
support also for layered/mipmapped. Once this is enabled, fast clear
operations target specific layer/level and call for the state to be
tracked in the same granulari
Note that RESOLVED is not tracked in the map explicitly. Absence
of item implicitly means RESOLVED state.
v2: Added intel_resolve_map_clear() into intel_miptree_release()
v3 (Jason): Properly handle the assumption of resolve map not
containing any items with state RESOLVED. Removed
One can now also delete intel_get_non_msrt_mcs_alignment().
v2 (Jason): Do not leak aux buf but allocate only after getting
ISL surfaces.
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 138 +++---
src/mesa/drivers/dri/i965/int
v2: Make intel_miptree_resolve_color() take start layer and
layer count.
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/brw_blorp.c | 13 ++-
src/mesa/drivers/dri/i965/brw_blorp.h | 3 ++-
src/mesa/drivers/dri/i965/brw_context.c | 14 +++-
On Wed, Nov 23, 2016 at 11:15:59AM +0200, Topi Pohjolainen wrote:
> This is a rebase on top recent changes by Jason and Lionel. While
> things have changed quite a bit in some of the patches they have
> mostly become clearer.
> 2.5.5
I handpicked and sent v3 for patches that went thru significant
On 24 November 2016 at 06:22, Boyan Ding wrote:
> 2016-11-24 13:29 GMT+08:00 Derek Foreman :
>> On 23/11/16 07:18 PM, Boyan Ding wrote:
>>>
>>> 2016-11-24 7:01 GMT+08:00 Derek Foreman :
This is a copy of commit 536003c11e4cb1172c540932ce3cce06f03bf44e
except for i915.
Orig
On Thu, 2016-11-24 at 14:33 +0100, Iago Toral wrote:
> Hi Lionel,
>
> On Thu, 2016-11-24 at 13:08 +, Lionel Landwerlin wrote:
> >
> > Hi Iago,
> >
> > Looking at the history, before
> > ed4fe3e9ba9018e68afe6fdd4f267218a537fdaa
> > we seem to set min/mag filter to MAPFILTER_ANISOTROPIC if
>
This is the second revision to the EGL_EXT_image_dma_buf_import_modifiers [1]
series at [2], addressing the comments received. This diverges from v1 due to
some reordered/squashed changes, hence the resend.
Major modifications include:
* Both format and modifier queries are now handled in the driv
From: Pekka Paalanen
Commit 0135e5d6c83add5e539492a4899504e33f3f2434 "egl: Add support for
more EGLImage extensions to EGL core." removed an error code for unknown
EGLImage attributes without explaining why.
EGL_KHR_image_base says:
* If an attribute specified in is not one of the
From: Pekka Paalanen
Rather than hardcoding 3, use a #define. Makes it easier to bump this
later to 4.
Signed-off-by: Pekka Paalanen
Signed-off-by: Varad Gautam
Reviewed-by: Eric Engestrom
---
src/egl/drivers/dri2/egl_dri2.c | 8
src/egl/main/eglimage.h | 8 +---
2 files
From: Pekka Paalanen
The EGL_EXT_dma_buf_import_modifiers extension adds support for a
fourth plane, just like DRM KMS API does.
Bump maximum dma_buf plane count to four.
v2: prevent attribute tokens from being parsed if
EXT_image_dma_buf_import_modifiers is not suported. (Emil Velikov)
Si
From: Pekka Paalanen
add createImageFromDmaBufs2 function which accepts per-plane dmabuf
format modifiers.
Signed-off-by: Pekka Paalanen
Signed-off-by: Varad Gautam
---
include/GL/internal/dri_interface.h | 21 -
1 file changed, 20 insertions(+), 1 deletion(-)
diff --git
From: Varad Gautam
update eglext.h to revision 33288 from Khronos
Signed-off-by: Varad Gautam
Reviewed-by: Eric Engestrom
---
include/EGL/eglext.h | 60 ++--
1 file changed, 58 insertions(+), 2 deletions(-)
diff --git a/include/EGL/eglext.h b/i
From: Varad Gautam
these allow querying the driver for supported dmabuf formats and
modifiers.
Signed-off-by: Varad Gautam
---
include/GL/internal/dri_interface.h | 22 +-
1 file changed, 21 insertions(+), 1 deletion(-)
diff --git a/include/GL/internal/dri_interface.h
b/i
From: Varad Gautam
this allows drivers to be queried for supported formats and format
modifiers. drivers that implement these queries must expose these under
PIPE_CAP_QUERY_DMABUF_ATTRIBS.
Signed-off-by: Varad Gautam
---
src/gallium/docs/source/screen.rst | 2 ++
src/gallium/dri
From: Varad Gautam
ask the driver for supported modifiers for a given format.
bump __DRI_IMAGE implementation version to 15.
Signed-off-by: Varad Gautam
---
src/gallium/state_trackers/dri/dri2.c | 21 -
1 file changed, 20 insertions(+), 1 deletion(-)
diff --git a/src/gall
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 assembling and error label name (Eric Engestrom)
v3:
- remove goto jumps within switch-case (Emil Velikov)
- treat zer
From: Varad Gautam
support importing dmabufs into DRIimage taking format modifiers in
account, as per DRIimage extension version 14.
Signed-off-by: Varad Gautam
---
src/gallium/include/state_tracker/drm_driver.h | 2 ++
src/gallium/state_trackers/dri/dri2.c | 45 +
From: Varad Gautam
ask the driver for supported dmabuf formats
Signed-off-by: Varad Gautam
---
src/gallium/state_trackers/dri/dri2.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/gallium/state_trackers/dri/dri2.c
b/src/gallium/state_trackers/dri/dri2.c
index a2b87a7.
From: Varad Gautam
Signed-off-by: Varad Gautam
---
src/egl/drivers/dri2/egl_dri2.c | 5 +
src/egl/main/eglapi.c | 1 +
2 files changed, 6 insertions(+)
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
index d387e28..e155801 100644
--- a/src/egl/drive
From: Varad Gautam
allow egl clients to query the dmabuf formats supported on this platform.
Signed-off-by: Louis-Francis Ratté-Boulianne
Signed-off-by: Varad Gautam
---
src/egl/drivers/dri2/egl_dri2.c | 17 +
src/egl/main/eglapi.c | 19 +++
src/egl/m
From: Varad Gautam
query and return supported dmabuf format modifiers for
EGL_EXT_image_dma_buf_import_modifiers.
Signed-off-by: Varad Gautam
---
src/egl/drivers/dri2/egl_dri2.c | 47 +
src/egl/main/eglapi.c | 21 ++
src/egl/mai
On 24/11/16 08:53 AM, Emil Velikov wrote:
On 24 November 2016 at 06:22, Boyan Ding wrote:
2016-11-24 13:29 GMT+08:00 Derek Foreman :
On 23/11/16 07:18 PM, Boyan Ding wrote:
2016-11-24 7:01 GMT+08:00 Derek Foreman :
This is a copy of commit 536003c11e4cb1172c540932ce3cce06f03bf44e
except fo
https://bugs.freedesktop.org/show_bug.cgi?id=98842
--- Comment #1 from Emil Velikov ---
Can you search through your system for the following files libdrm.so and
libdrm.pc.
For both, provide the exact location. And the contents in for the latter.
Thanks
--
You are receiving this mail because:
Y
On 16-11-23 12:04:02, Jason Ekstrand wrote:
On Wed, Nov 23, 2016 at 10:16 AM, Pohjolainen, Topi <
topi.pohjolai...@gmail.com> wrote:
On Wed, Nov 23, 2016 at 09:05:39AM -0800, Jason Ekstrand wrote:
>On Wed, Nov 23, 2016 at 1:16 AM, Topi Pohjolainen
><[1]topi.pohjolai...@gmail.com> wrote:
https://bugs.freedesktop.org/show_bug.cgi?id=98833
--- Comment #5 from Eero Tamminen ---
Happens both on BDW & SKL, both with Ubuntu 16.04 kernel & latest kernel from
drm nightly git.
Not sure whether I have time to write test program, but at least I can provide
EGL/GL API calls from apitrace du
On Nov 24, 2016 8:42 AM, "Ben Widawsky" wrote:
>
> On 16-11-23 12:04:02, Jason Ekstrand wrote:
>>
>> On Wed, Nov 23, 2016 at 10:16 AM, Pohjolainen, Topi <
>> topi.pohjolai...@gmail.com> wrote:
>>
>>> On Wed, Nov 23, 2016 at 09:05:39AM -0800, Jason Ekstrand wrote:
>>> >On Wed, Nov 23, 2016 at 1
Hello list,
The candidate for the Mesa 13.0.2 is now available. Currently we have:
- 49 queued
- 4 nominated (outstanding)
- and 1 rejected patch(es)
With this series we have - fixes for vc4, i965 and radeon drivers. In addition
to that PCI IDs for Geminilake have been added to the i965 drive
On Nov 24, 2016 6:06 AM, "Topi Pohjolainen"
wrote:
>
> Note that RESOLVED is not tracked in the map explicitly. Absence
> of item implicitly means RESOLVED state.
>
> v2: Added intel_resolve_map_clear() into intel_miptree_release()
> v3 (Jason): Properly handle the assumption of resolve map not
>
2016-11-24 18:25 GMT+01:00 Emil Velikov :
> Hello list,
>
> The candidate for the Mesa 13.0.2 is now available. Currently we have:
> - 49 queued
> - 4 nominated (outstanding)
> - and 1 rejected patch(es)
>
>
> With this series we have - fixes for vc4, i965 and radeon drivers. In
> addition
> to
On 24 November 2016 at 18:09, Gustaw Smolarczyk wrote:
> 2016-11-24 18:25 GMT+01:00 Emil Velikov :
>> The plan is to have 13.0.2 this Saturday (25th of November), around or
>
> Saturday is 26th of November.
>
That's correct. Thank you.
Emil
___
mesa-de
From: Emil Velikov
Cap up-to the number of properties available while copying the data.
Otherwise we might crash and/or leak data.
Cc: Dave Airlie
Cc: "13.0"
Signed-off-by: Emil Velikov
---
src/amd/vulkan/radv_device.c | 18 +++---
1 file changed, 7 insertions(+), 11 deletions(-)
From: Emil Velikov
Signed-off-by: Emil Velikov
---
src/amd/vulkan/radv_private.h | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_private.h b/src/amd/vulkan/radv_private.h
index a5d13a9..def0af2 100644
--- a/src/amd/vulkan/radv_private.h
+++ b/src/a
From: Emil Velikov
Use the generic construct instead of the currect GCC specific one.
Cc: Kenneth Graunke
Suggested-by: Kenneth Graunke
Signed-off-by: Emil Velikov
---
src/intel/vulkan/anv_private.h | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/src/intel
From: Emil Velikov
Signed-off-by: Emil Velikov
---
src/intel/isl/isl_priv.h | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/intel/isl/isl_priv.h b/src/intel/isl/isl_priv.h
index dc3975d..1867d25 100644
--- a/src/intel/isl/isl_priv.h
+++ b/src/intel/isl/isl_priv.h
On Thu, Nov 24, 2016 at 10:01:51AM -0800, Jason Ekstrand wrote:
>On Nov 24, 2016 6:06 AM, "Topi Pohjolainen"
><[1]topi.pohjolai...@gmail.com> wrote:
>>
>> Note that RESOLVED is not tracked in the map explicitly. Absence
>> of item implicitly means RESOLVED state.
>>
>> v
Reviewed-by: Bas Nieuwenhuizen
On Thu, Nov 24, 2016 at 7:14 PM, Emil Velikov wrote:
> From: Emil Velikov
>
> Cap up-to the number of properties available while copying the data.
> Otherwise we might crash and/or leak data.
>
> Cc: Dave Airlie
> Cc: "13.0"
> Signed-off-by: Emil Velikov
> ---
You might want to rename the title of this patch.
- Bas
On Thu, Nov 24, 2016 at 7:18 PM, Emil Velikov wrote:
> From: Emil Velikov
>
> Signed-off-by: Emil Velikov
> ---
> src/intel/isl/isl_priv.h | 8 +++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/src/intel/isl/isl_
On 11/24/2016 06:25 PM, Emil Velikov wrote:
> Hello list,
>
> The candidate for the Mesa 13.0.2 is now available. Currently we have:
> - 49 queued
> - 4 nominated (outstanding)
> - and 1 rejected patch(es)
>
>
> With this series we have - fixes for vc4, i965 and radeon drivers. In addition
>
Making this thread safe is probably overkill I guess?
Reviewed-by: Bas Nieuwenhuizen
On Thu, Nov 24, 2016 at 7:18 PM, Emil Velikov wrote:
> From: Emil Velikov
>
> Signed-off-by: Emil Velikov
> ---
> src/amd/vulkan/radv_private.h | 8 +++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
On 23/11/16 17:37, Emil Velikov wrote:
On 23 November 2016 at 16:14, George Kyriazis wrote:
---
src/gallium/targets/libgl-gdi/SConscript | 1 +
src/gallium/targets/libgl-gdi/libgl_gdi.c | 4
2 files changed, 5 insertions(+)
diff --git a/src/gallium/targets/libgl-gdi/SConscript
b/src/ga
From: Boyuan Zhang
The gop_size in rate control is the budget window for internal rate
control calculation, and shouldn't always equal to idr period. Define
a coefficient to let budget window contains a number of idr period for
proper rate control calculation. Adjust the number of i/p frame remai
https://bugs.freedesktop.org/show_bug.cgi?id=98846
Bug ID: 98846
Summary: triangle clipping causes wrong vertex's attribute to
be broadcasted for flat interpolation
Product: Mesa
Version: git
Hardware: Other
From: Boyuan Zhang
The gop_size in rate control is the budget window for internal rate
control calculation, and shouldn't always equal to idr period. Define
a coefficient to let budget window contains a number of idr period for
proper rate control calculation. Adjust the number of i/p frame remai
Clip distances need to be perspective-divided. This fixes all the
interpolation-*-{distance,vertex} piglits.
Also take this opportunity to fix clip distances for points rasterized
as triangles - the clip distance is not subject to sprite coord
replacement, so there's no interpolation of it. We jus
From: Emil Velikov
Port of an equivalent commit for radv.
Signed-off-by: Emil Velikov
---
src/intel/vulkan/anv_device.c | 18 ++
src/intel/vulkan/anv_pipeline_cache.c | 8
src/intel/vulkan/anv_private.h| 4 ++--
3 files changed, 16 insertions(+), 14 d
From: Emil Velikov
If radv_device_get_cache_uuid() fails result will be VK_SUCCESS as set
by the radv_init_wsi() call above.
Fixes: d943839 (radv: Use library mtime for cache UUID.)
Cc: Bas Nieuwenhuizen
Signed-off-by: Emil Velikov
---
src/amd/vulkan/radv_device.c | 2 ++
1 file changed, 2 in
From: Emil Velikov
brw_compiler_create() rzalloc-ates memory which we forgot to free.
Cc: "13.0"
Signed-off-by: Emil Velikov
---
src/intel/vulkan/anv_device.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.
From: Emil Velikov
Good bye, you shall not be missed.
Signed-off-by: Emil Velikov
---
configure.ac | 2 --
1 file changed, 2 deletions(-)
diff --git a/configure.ac b/configure.ac
index 5f30ae8..e7e5628 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2644,8 +2644,6 @@ AC_SUBST([XA_MINOR], $XA
From: Emil Velikov
radv_amdgpu_winsys_create() does not take ownership of the fd, thus we
end up leaking it as we return with VK_SUCCESS.
Cc: Dave Airlie
Cc: "13.0"
Signed-off-by: Emil Velikov
---
Not 100% sure on this one.
---
src/amd/vulkan/radv_device.c | 1 +
1 file changed, 1 insertion(
From: Emil Velikov
No longer used as of last commit.
Signed-off-by: Emil Velikov
---
src/intel/vulkan/Makefile.am | 6 --
src/intel/vulkan/Makefile.sources | 3 +--
2 files changed, 1 insertion(+), 8 deletions(-)
diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.a
From: Emil Velikov
Cc: "13.0"
Signed-off-by: Emil Velikov
---
src/intel/vulkan/anv_device.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
index 4711501..0e01e28 100644
--- a/src/intel/vulkan/anv_device.c
+++
From: Emil Velikov
Inspired by a similar commit for radv.
Rather than recomputing the timestamp on each make invocation, just
fetch it at runtime.
Thus we no longer get the constant rebuild of anv_device.c and the
follow-up libvulkan_intel.so link, when nothing has changed.
I.e. using make &&
On Nov 24, 2016 9:26 AM, "Emil Velikov" wrote:
>
> Hello list,
>
> The candidate for the Mesa 13.0.2 is now available. Currently we have:
> - 49 queued
> - 4 nominated (outstanding)
> - and 1 rejected patch(es)
>
>
> With this series we have - fixes for vc4, i965 and radeon drivers. In
addition
On Nov 24, 2016 12:42 PM, "Emil Velikov" wrote:
>
> From: Emil Velikov
>
> Port of an equivalent commit for radv.
>
> Signed-off-by: Emil Velikov
> ---
> src/intel/vulkan/anv_device.c | 18 ++
> src/intel/vulkan/anv_pipeline_cache.c | 8
> src/intel/vulkan/anv_
I'm not sure what I think of this... It has some interesting implications.
I'll think about it and write something more detailed on Monday.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
https://bugs.freedesktop.org/show_bug.cgi?id=96993
Ilia Mirkin changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=98166
Emil Velikov changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
Thanks for the explanation, feel free to wait for Jason's rb otherwise
:
Reviewed-by: Lionel Landwerlin
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
I don't see any regressions in the automated test suites (piglit, deqp,
gl cts) with this patch.
-Mark
Emil Velikov writes:
> On 24 November 2016 at 06:22, Boyan Ding wrote:
>> 2016-11-24 13:29 GMT+08:00 Derek Foreman :
>>> On 23/11/16 07:18 PM, Boyan Ding wrote:
2016-11-24 7:01 GMT+
Seems like libdrm_amdgpu does a dup indeed. Patches 7-8 are
Reviewed-by: Bas Nieuwenhuizen
On Thu, Nov 24, 2016 at 9:30 PM, Emil Velikov wrote:
> From: Emil Velikov
>
> radv_amdgpu_winsys_create() does not take ownership of the fd, thus we
> end up leaking it as we return with VK_SUCCESS.
>
>
On 24 November 2016 at 20:44, Jason Ekstrand wrote:
> On Nov 24, 2016 9:26 AM, "Emil Velikov" wrote:
>>
>> Hello list,
>>
>> The candidate for the Mesa 13.0.2 is now available. Currently we have:
>> - 49 queued
>> - 4 nominated (outstanding)
>> - and 1 rejected patch(es)
>>
>>
>> With this ser
On 24 November 2016 at 20:58, Jason Ekstrand wrote:
> I'm not sure what I think of this... It has some interesting implications.
> I'll think about it and write something more detailed on Monday.
But but... it helps up get rid of the lovely timestamp experience
Thanks for having a look - how go
Instead of (incorrectly) biasing the snorm value to make it look like a
unorm, just use signed integer math.
This fixes arb_color_buffer_float-render GL_RGBA8_SNORM
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/swr/rasterizer/jitter/blend_jit.cpp | 17 -
1 file changed, 12
On 24 November 2016 at 18:35, Bas Nieuwenhuizen
wrote:
> You might want to rename the title of this patch.
>
That's correct, thanks. Fixed locally.
Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/list
On 17 November 2016 at 17:27, Emil Velikov wrote:
> Admittedly not a huge C++ expert, but what's wrong with having a
> simple 5 line patch that adds the original code in a if guard ?
> If then the LLVM <= 3.9 codepath is (somewhat) wrong, let's tackle
> that separately.
>
Anyone ?
Emil
_
Thanks.
Patches 7 & 8 are:
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Tue, 2016-11-22 at 20:25 +0100, Vedran Miletić wrote:
> The commit 8e430ff8b060b4e8e922bae24b3c57837da6ea77 support for LLVM
> 3.9 and older versionsin Clover. This patch restores it and refactors
> the support using Clover compatibility layer for LLVM.
>
> v2: merged #ifdef blocks
> v3: added
On Fri, 2016-11-25 at 00:00 +, Emil Velikov wrote:
> On 17 November 2016 at 17:27, Emil Velikov wrote:
>
> > Admittedly not a huge C++ expert, but what's wrong with having a
> > simple 5 line patch that adds the original code in a if guard ?
> > If then the LLVM <= 3.9 codepath is (somewhat)
On Nov 24, 2016 7:12 AM, "Iago Toral" wrote:
>
> On Thu, 2016-11-24 at 14:33 +0100, Iago Toral wrote:
> > Hi Lionel,
> >
> > On Thu, 2016-11-24 at 13:08 +, Lionel Landwerlin wrote:
> > >
> > > Hi Iago,
> > >
> > > Looking at the history, before
> > > ed4fe3e9ba9018e68afe6fdd4f267218a537fdaa
>
Emil Velikov writes:
> On 17 November 2016 at 17:27, Emil Velikov wrote:
>
>> Admittedly not a huge C++ expert, but what's wrong with having a
>> simple 5 line patch that adds the original code in a if guard ?
>> If then the LLVM <= 3.9 codepath is (somewhat) wrong, let's tackle
>> that separate
Vedran Miletić writes:
> The commit 8e430ff8b060b4e8e922bae24b3c57837da6ea77 support for LLVM
> 3.9 and older versionsin Clover. This patch restores it and refactors
> the support using Clover compatibility layer for LLVM.
>
> v2: merged #ifdef blocks
> v3: added support for LLVM 3.6-3.8
> v4: a
Jan Vesely writes:
> On Tue, 2016-11-22 at 20:25 +0100, Vedran Miletić wrote:
>> The commit 8e430ff8b060b4e8e922bae24b3c57837da6ea77 support for LLVM
>> 3.9 and older versionsin Clover. This patch restores it and refactors
>> the support using Clover compatibility layer for LLVM.
>>
>> v2: merg
On Thu, Nov 24, 2016 at 05:25:26PM +, Emil Velikov wrote:
> Hello list,
>
> The candidate for the Mesa 13.0.2 is now available. Currently we have:
> - 49 queued
> - 4 nominated (outstanding)
> - and 1 rejected patch(es)
>
>
> With this series we have - fixes for vc4, i965 and radeon drive
I wrote this code in response to a issue someone on mesa-users had
(https://lists.freedesktop.org/archives/mesa-users/2016-October/001231.html).
They didn't reply so I don't know if the patch resolved the issue. Can
someone please determine its usefulness. Also please check if
drmCommandWrite() is
A different patch to that code has been applied that does the same
thing in a different (better) way.
On 24 November 2016 at 05:25, Albert Freeman wrote:
> I wrote this code in response to a issue someone on mesa-users had
> (https://lists.freedesktop.org/archives/mesa-users/2016-October/001231.h
1 - 100 of 111 matches
Mail list logo
