Re: [Mesa-dev] About the current EGL GLES 1.x implementation

2011-08-26 Thread Emil Velikov
On Fri, 26 Aug 2011 16:36:50 +0100, Ian Romanick i...@freedesktop.org wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/26/2011 05:37 AM, Alex Deucher wrote: On Fri, Aug 26, 2011 at 4:11 AM, Chia-I Wu olva...@gmail.com wrote: On Fri, Aug 26, 2011 at 3:52 PM, Ilyes Gouta

[Mesa-dev] Restructure documents directory layout

2013-04-11 Thread Emil Velikov
As suggested by Jose, here is a short patchset that handles the initial stage of beating some structure into the docs directory * All the release notes (both text and html) have been moved to relnotes/, with filenames indicating the version only * Mesa/WL extensions have been moved to specs/,

[Mesa-dev] [PATCH 3/4] docs: Add some missing release notes

2013-04-11 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- docs/relnotes.html | 3 +++ 1 file changed, 3 insertions(+) diff --git a/docs/relnotes.html b/docs/relnotes.html index 049a979..819d2ce 100644 --- a/docs/relnotes.html +++ b/docs/relnotes.html @@ -22,6 +22,7 @@ The release notes summarize

[Mesa-dev] [PATCH 4/4] docs: Update 'Making new mesa release'

2013-04-11 Thread Emil Velikov
Add a note to update PACKAGE_VERSION for Android and scons builds Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- docs/devinfo.html | 2 ++ 1 file changed, 2 insertions(+) diff --git a/docs/devinfo.html b/docs/devinfo.html index 5c03344..7176824 100644 --- a/docs/devinfo.html +++ b

[Mesa-dev] Compile warnings cleanup

2013-04-11 Thread Emil Velikov
Here is a short patchset that handles most of the compile warnings for my gallium/nouveau 'release' build. The commit messages may be rather silly but I was running out of ideas what to put :) Anyway the first five patches are nv50,nvc0 related, whereas the last one changes a egl/wayland

[Mesa-dev] [PATCH 1/6] nv50/codegen: do not emitATOM() if the subOp is unknown

2013-04-11 Thread Emil Velikov
: warning: 'subOp' may be used uninitialized in this function [-Wmaybe-uninitialized] uint8_t subOp; ^ Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/nv50/codegen/nv50_ir_emit_nv50.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src

[Mesa-dev] [PATCH 2/6] nvc0: compile nve4_cache_name() only in debug build

2013-04-11 Thread Emil Velikov
As otherwise it is unused - pointed out by gcc nve4_compute.c:586:20: warning: 'nve4_cache_split_name' defined but not used [-Wunused-function] static const char *nve4_cache_split_name(unsigned value) ^ Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium

[Mesa-dev] [PATCH 3/6] nvc0: bail out early during nve4_compute_setup()

2013-04-11 Thread Emil Velikov
Exit gracefully rather than trying to create a random object, whenever the chipset is unknown Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/nvc0/nve4_compute.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/nvc0

[Mesa-dev] [PATCH 4/6] nvc0: initialize variable

2013-04-11 Thread Emil Velikov
Pointed out by gcc nve4_compute.c: In function 'nve4_launch_grid': nve4_compute.c:511:7: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized] if (ret) ^ Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/nvc0/nve4_compute.c | 2

[Mesa-dev] [PATCH 5/6] nv50, nvc0: make nv{50, c0} lookalike, kind of

2013-04-11 Thread Emil Velikov
Remove extra chipset check during nvc0_screen_create Set the class_3d after the object is created Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/nv50/nv50_screen.c | 2 +- src/gallium/drivers/nvc0/nvc0_screen.c | 12 +--- 2 files changed, 2 insertions

[Mesa-dev] [PATCH 6/6] gallium-egl: Use native types rather than EGL ones

2013-04-11 Thread Emil Velikov
by default] Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- .../egl/common/native_wayland_drm_bufmgr_helper.c| 16 .../egl/common/native_wayland_drm_bufmgr_helper.h| 4 ++-- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/gallium

[Mesa-dev] [RESENT] Restructure documents directory

2013-04-12 Thread Emil Velikov
No code change since previous version - just git format-patch was being silly With --find-renames, git produces a more sensible patches (max 90k for patch1) Feel free to commit as I do not have access ___ mesa-dev mailing list

[Mesa-dev] [PATCHv2 2/4] docs: move specs to a separate folder

2013-04-12 Thread Emil Velikov
Handle legacy/obsolete specs as well List all specs in extensions.html Mark 'OLD' extensions as obsolete in extensions.html Update the spec location in old relnotes Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- docs/extensions.html | 34

[Mesa-dev] [PATCHv2 3/4] docs: Add some missing release notes

2013-04-12 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- docs/relnotes.html | 3 +++ 1 file changed, 3 insertions(+) diff --git a/docs/relnotes.html b/docs/relnotes.html index 049a979..819d2ce 100644 --- a/docs/relnotes.html +++ b/docs/relnotes.html @@ -22,6 +22,7 @@ The release notes summarize

[Mesa-dev] [PATCHv2 4/4] docs: Update 'Making new mesa release'

2013-04-12 Thread Emil Velikov
Add a note to update PACKAGE_VERSION for Android and scons builds Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- docs/devinfo.html | 2 ++ 1 file changed, 2 insertions(+) diff --git a/docs/devinfo.html b/docs/devinfo.html index 5c03344..7176824 100644 --- a/docs/devinfo.html +++ b

Re: [Mesa-dev] [PATCH] autoconf: enable detection of vdpau and xvmc by default

2013-04-17 Thread Emil Velikov
://lists.freedesktop.org/mailman/listinfo/mesa-dev From 0b90d5bff8c131930760866c45e01ab70598e48f Mon Sep 17 00:00:00 2001 From: Emil Velikov emil.l.veli...@gmail.com Date: Thu, 18 Apr 2013 01:40:40 +0100 Subject: [PATCH] configure: enable vdpau and xvmc detection, when gallium is present

Re: [Mesa-dev] [PATCH] autoconf: enable detection of vdpau and xvmc by default

2013-04-17 Thread Emil Velikov
On 18/04/13 01:55, Emil Velikov wrote: On 18/04/13 01:21, Paul Berry wrote: On 12 April 2013 01:26, Christian König deathsim...@vodafone.de wrote: From: Christian König christian.koe...@amd.com Since we now have UVD support we should enable them by default. Signed-off-by: Christian König

Re: [Mesa-dev] [PATCH] build: Unify PACKAGE_VERSION on autotools, scons and Android

2013-06-04 Thread Emil Velikov
On 04/06/13 21:42, Andreas Boll wrote: 2013/6/4 Brian Paul bri...@vmware.com On 06/04/2013 11:40 AM, Andreas Boll wrote: This patch unifies mesa's PACKAGE_VERSION on autotools, scons and Android build systems. Current behaviour is: - Autotools uses 9.2.0 as PACKAGE_VERSION - Scons

Re: [Mesa-dev] [PATCH 2/2] intel: Use the CHIPSET macro in the PCI ID tables for the device name.

2013-06-05 Thread Emil Velikov
On 05/06/13 20:05, Kenneth Graunke wrote: Putting the human readable device names directly in the PCI ID list consolidates things in one place. It also makes it easy to customize the name on a per-PCI ID basis without a huge code explosion. Based on a patch by Kristian Høgsberg. Hi Kenneth

[Mesa-dev] Build warnings cleanup

2013-06-22 Thread Emil Velikov
Cleanup series addressing a few warnings spotted by gcc Note: the first patch is a recent from previously, which despite it's evasive looks is quite trivial Feel free to commit, as I do not have access [PATCH 1/4] gallium-egl: Use native types rather than EGL ones [PATCH 2/4]

[Mesa-dev] [PATCH 1/4] gallium-egl: Use native types rather than EGL ones

2013-06-22 Thread Emil Velikov
by default] Signed-off-by: Emil Velikov emil.l.veli...@gmail.com CC: Kristian Høgsberg k...@bitplanet.net --- .../egl/common/native_wayland_drm_bufmgr_helper.c| 16 .../egl/common/native_wayland_drm_bufmgr_helper.h| 4 ++-- 2 files changed, 10 insertions(+), 10

[Mesa-dev] [PATCH 2/4] nvc0: Silence uninitialized variable warning in the release build

2013-06-22 Thread Emil Velikov
Resolves the following gcc warning warning: 'ptr' may be used uninitialized in this function xy[0] = ptr[sample_index][0] * 0.0625f; ^ Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/nvc0/nvc0_context.c | 1 + 1 file changed, 1 insertion(+) diff

[Mesa-dev] [PATCH 3/4] glsl ast_to_hir: Silence uninitialized variable warnings in the release build

2013-06-22 Thread Emil Velikov
Resolves the following gcc warnings warning: 'iface_type_name' may be used uninitialized in this function warning: 'var_mode' may be used uninitialized in this function Note: The variables are initialised to UNKNOWN and ir_var_auto Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src

[Mesa-dev] [PATCH 4/4] glsl opt_flip_matrices: Silence unused variable warning in the release build

2013-06-22 Thread Emil Velikov
Resolves the following gcc warning opt_flip_matrices.cpp:84:32: warning: unused variable 'deref' Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/glsl/opt_flip_matrices.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/glsl/opt_flip_matrices.cpp b/src

Re: [Mesa-dev] Build warnings cleanup

2013-06-22 Thread Emil Velikov
On 22/06/13 16:43, Emil Velikov wrote: Cleanup series addressing a few warnings spotted by gcc Note: the first patch is a recent from previously, which despite s/recent/resend English is hard :) Emil it's evasive looks is quite trivial Feel free to commit, as I do not have

Re: [Mesa-dev] [Nouveau] [PATCH] nv50: H.264/MPEG2 decoding support via VP2, available on NV84-NV96, NVA0

2013-06-29 Thread Emil Velikov
Hi Ilia, On 27/06/13 12:26, Ilia Mirkin wrote: Adds H.264 and MPEG2 codec support via VP2, using firmware from the blob. Acceleration is supported at the bitstream level for H.264 and IDCT level for MPEG2. Known issues: - H.264 interlaced doesn't render properly - H.264 shows very

Re: [Mesa-dev] [Nouveau] [PATCH] nv50: H.264/MPEG2 decoding support via VP2, available on NV84-NV96, NVA0

2013-06-29 Thread Emil Velikov
On 29/06/13 21:21, Ilia Mirkin wrote: On Sat, Jun 29, 2013 at 2:07 PM, Emil Velikov emil.l.veli...@gmail.com wrote: Hi Ilia, On 27/06/13 12:26, Ilia Mirkin wrote: Adds H.264 and MPEG2 codec support via VP2, using firmware from the blob. Acceleration is supported at the bitstream level

[Mesa-dev] [PATCHv2 2/4] nvc0: Silence uninitialized variable warning in the release build

2013-07-08 Thread Emil Velikov
Resolves the following gcc warning warning: 'ptr' may be used uninitialized in this function xy[0] = ptr[sample_index][0] * 0.0625f; ^ v2: Bail out when using non-conformant sample_count (spotted by calim) Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium

[Mesa-dev] [PATCH 3/4] glsl ast_to_hir: Silence uninitialized variable warnings in the release build

2013-07-08 Thread Emil Velikov
Resolves the following gcc warnings warning: 'iface_type_name' may be used uninitialized in this function warning: 'var_mode' may be used uninitialized in this function Note: The variables are initialised to UNKNOWN and ir_var_auto Signed-off-by: Emil Velikov emil.l.veli...@gmail.com Reviewed

[Mesa-dev] [PATCHv2 4/4] glsl opt_flip_matrices: Silence unused variable warning in the release build

2013-07-08 Thread Emil Velikov
Resolves the following gcc warning opt_flip_matrices.cpp:84:32: warning: unused variable 'deref' v2: keep the variable, but wrap it in a ifndef NDEBUG block(suggested by Ian) Signed-off-by: Emil Velikov emil.l.veli...@gmail.com Reviewed-by: Kenneth Graunke kenn...@whitecape.org --- Feel free

Re: [Mesa-dev] [PATCH 1/4] gallium-egl: Use native types rather than EGL ones

2013-07-08 Thread Emil Velikov
ping Does anyone have any objections on this patch? Feel free to commit, it looks ok Cheers Emil On 22/06/13 16:43, Emil Velikov wrote: Change egl_g3d_wl_drm_common_query_buffer() to use boolean/int rather than EGLBoolean/EGLint, based on the interface in native_wayland_bufmgr.h, Resolves

[Mesa-dev] [PATCH] gallium/util: use explicily sized types for {un, }pack_rgba_{s, u}int

2013-07-08 Thread Emil Velikov
Every function but the above four uses explicitly sized types for their src and dst arguments. Even fetch_rgba_{s,u}int follows the convention. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/util/u_format.c | 8 src/gallium/auxiliary/util/u_format.h | 8

[Mesa-dev] [PATCH] nv30, nv50: handle pure integer vertex attributes

2013-07-10 Thread Emil Velikov
using channel[0] rather than first-non-void. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/nv30/nv30_vbo.c | 13 - src/gallium/drivers/nv50/nv50_vbo.c | 13 - src/gallium/drivers/nvc0/nvc0_vbo.c | 2 ++ 3 files changed, 26 insertions(+), 2

Re: [Mesa-dev] [PATCH] gallium/util: use explicily sized types for {un, }pack_rgba_{s, u}int

2013-07-16 Thread Emil Velikov
a rather pedantic change, but I always believed that consistency is a good thing. For what it's worth I have been using this patch for approximately three weeks with no side effects. Cheers Emil Marek On Mon, Jul 8, 2013 at 8:56 PM, Emil Velikov emil.l.veli...@gmail.com wrote: Every function

Re: [Mesa-dev] [PATCH mesa 1/3] gallium-egl: Simplify native_wayland_drm_bufmgr_helper interface

2013-07-18 Thread Emil Velikov
On 18/07/13 13:11, Ander Conselvan de Oliveira wrote: The helper provides a series of functions to easy the implementation of the WL_bind_wayland_display extension on different platforms. But even with the helpers there was still a bit of duplicated code between platforms, with the drm

[Mesa-dev] [PATCH] build: unify mesa version by using a VERSION file

2013-07-19 Thread Emil Velikov
Rather than having to keep track of all the build systems and their respecitve definition of the mesa version, use a single top file VERSION. Every build system is responsible for reading/parsing the file and using it Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- Hi guys So here

Re: [Mesa-dev] [PATCH mesa 1/3] gallium-egl: Simplify native_wayland_drm_bufmgr_helper interface

2013-07-19 Thread Emil Velikov
On 19/07/13 09:17, Ander Conselvan de Oliveira wrote: On 07/18/2013 10:04 PM, Emil Velikov wrote: On 18/07/13 13:11, Ander Conselvan de Oliveira wrote: [...] +static EGLBoolean +wayland_drm_bufmgr_query_buffer(struct native_display *ndpy, +struct wl_buffer

[Mesa-dev] [PATCH v2] build: unify mesa version by using a VERSION file

2013-07-25 Thread Emil Velikov
* Androing is Linux. Use '/' in stead of '\', spotted by Chad V * use cleaner code to get the version in scons, suggested by Chad V Reviewed-by: Matt Turner matts...@gmail.com Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- Thanks for the tips and review guys. Feel free to commit if it's ok

Re: [Mesa-dev] [PATCH v2] build: unify mesa version by using a VERSION file

2013-07-25 Thread Emil Velikov
On 25/07/13 21:28, Kenneth Graunke wrote: On 07/25/2013 12:39 PM, Emil Velikov wrote: Rather than having to keep track of all the build systems and their respecitve definition of the mesa version, use a single top file VERSION. Every build system is responsible for reading/parsing the file

[Mesa-dev] [PATCH v3] build: unify mesa version by using a VERSION file

2013-07-25 Thread Emil Velikov
...@gmail.com Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- An interesting point while looking for ways to strip the newline character - python considers carrage return as whitespace whereas m4 does not. AFAICS no-one has used those nasty symbols in mesa, and I hope no-one will. Just

Re: [Mesa-dev] [PATCH] nvc0: Initialize ptr for unexpected sample_count on release builds.

2013-08-06 Thread Emil Velikov
On 04/08/13 08:51, Vinson Lee wrote: Fixes Uninitialized pointer read defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org Hi Vinson This patch looks remarkably similar to the one I've sent a few weeks ago - with yours having a lot better commit message :) The way I

[Mesa-dev] [PATCH] nouveau/video: use correct parameter name

2013-08-14 Thread Emil Velikov
Fix a typo introduced with commit d1ba1055d9 - vl: Add support for max level query v2 Cc: Rico Schüller kgbric...@web.de Cc: Christian König christian.koe...@amd.com Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68126 Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- FWIW

Re: [Mesa-dev] [PATCH] nouveau/video: use correct parameter name

2013-08-15 Thread Emil Velikov
On 15/08/13 13:41, Rico Schüller wrote: On 15.08.2013 02:10, Emil Velikov wrote: Fix a typo introduced with commit d1ba1055d9 - vl: Add support for max level query v2 Cc: Rico Schüller kgbric...@web.de Cc: Christian König christian.koe...@amd.com Bugzilla: https://bugs.freedesktop.org

[Mesa-dev] [RFC] Consolidate the remaining source files to Makefile.sources

2013-08-15 Thread Emil Velikov
Hello list Feeling inspired by the automake work done in mesa, I felt like finishing a few things that did not receive too much attention * use Makefile.sources wherever possible * cleanup the duplicated C{,PP,XX}FLAGS and factor out the the common ones into Automake.inc If anyone is

[Mesa-dev] [PATCH] st/vdpau: drop unnecessary variable prof

2013-08-16 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/state_trackers/vdpau/mixer.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/vdpau/mixer.c b/src/gallium/state_trackers/vdpau/mixer.c index 26db5c8..8c20d05 100644 --- a/src

[Mesa-dev] auxiliary/vl and st/vdpau, xvmc trivial fixes and cleanups

2013-08-17 Thread Emil Velikov
Hello list, This patch-set sets the proper width/height for 442 chroma formats(Andy Furniss), prevents all sort of chaos if create_video_buffer() fails, adds a check after memory allocation and a few trivial oneliners to improve code readability. Not sure how many of those are suiteable

[Mesa-dev] [PATCH 1/7] vdpau/vl 422 chroma width/height mix up

2013-08-17 Thread Emil Velikov
From: Andy Furniss adf.li...@gmail.com I was looking into some minor 422 issues/discrepencies I noticed long ago using vdpau on my rv790. I noticed that there is code that is halving height rather than width - 422 is full height AFAIK. Making the changes below doesn't actually make any

[Mesa-dev] [PATCH 2/7] st/vdpau: exit gracefully if we fail to create video buffer

2013-08-17 Thread Emil Velikov
Otherwise we risk causing memory corruption. v2: forgot the mutex_unlock() Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/state_trackers/vdpau/surface.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/gallium/state_trackers/vdpau/surface.c b/src/gallium

[Mesa-dev] [PATCH 3/7] st/xvmc: exit gracefully if we fail to create video buffer

2013-08-17 Thread Emil Velikov
Free any allocated memory and return BadAlloc if create_video_buffer() has failed to create a buffer. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/state_trackers/xvmc/surface.c | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/state_trackers/xvmc

[Mesa-dev] [PATCH 4/7] vl/buffer: add sanity check after CALLOC_STRUCT

2013-08-17 Thread Emil Velikov
Check if we have successfully allocated memory. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/vl/vl_video_buffer.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/auxiliary/vl/vl_video_buffer.c b/src/gallium/auxiliary/vl/vl_video_buffer.c index

[Mesa-dev] [PATCH 5/7] vl/idct: cleanup all idct buffers

2013-08-17 Thread Emil Velikov
Code should loop through and cleanup the three (VL_NUM_COMPONENTS) idct buffers, rather than doing the first one three times. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 6/7] st/vdpau: drop unnecessary variable prof

2013-08-17 Thread Emil Velikov
Any decent compiler will do this for us, although doing this will make grepping through the code alot easier. v2: In both mixer and query interface Reviewed-by: Christian König christian.koe...@amd.com [v1] Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/state_trackers

[Mesa-dev] [PATCH 7/7] vl/buffers: consistent use on VL_MAX_SURFACES

2013-08-17 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/vl/vl_video_buffer.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/auxiliary/vl/vl_video_buffer.c b/src/gallium/auxiliary/vl/vl_video_buffer.c index 22e618a..9264194 100644

Re: [Mesa-dev] [PATCH 2/7] st/vdpau: exit gracefully if we fail to create video buffer

2013-08-18 Thread Emil Velikov
On 18/08/13 12:31, Christian König wrote: Am 17.08.2013 23:51, schrieb Emil Velikov: Otherwise we risk causing memory corruption. v2: forgot the mutex_unlock() Signed-off-by: Emil Velikov emil.l.veli...@gmail.com NAK. Failing is actually not correct here, cause we can still make it work

Re: [Mesa-dev] [PATCH 2/7] st/vdpau: exit gracefully if we fail to create video buffer

2013-08-19 Thread Emil Velikov
On 19/08/13 09:08, Christian König wrote: Am 18.08.2013 14:20, schrieb Emil Velikov: On 18/08/13 12:31, Christian König wrote: Am 17.08.2013 23:51, schrieb Emil Velikov: Otherwise we risk causing memory corruption. v2: forgot the mutex_unlock() Signed-off-by: Emil Velikov emil.l.veli

[Mesa-dev] auxiliary/vl and st/vdpau, xvmc trivial fixes and cleanups v2

2013-08-19 Thread Emil Velikov
Diff to previous version patch 1,3,4,5,7 - annotate patches with Reviewed-by tags patch 2 - rework completely as per Christian's comments patch 6 - resolve merge conflicts on top of master Patches are also available at guthub for those interested

[Mesa-dev] [PATCH 1/7] vdpau/vl 422 chroma width/height mix up

2013-08-19 Thread Emil Velikov
From: Andy Furniss adf.li...@gmail.com I was looking into some minor 422 issues/discrepencies I noticed long ago using vdpau on my rv790. I noticed that there is code that is halving height rather than width - 422 is full height AFAIK. Making the changes below doesn't actually make any

[Mesa-dev] [PATCH 2/7] st/vdpau: don't try to create video buffer when the format is FORMAT_NONE

2013-08-19 Thread Emil Velikov
Not seen in the wild yet, but seems like a reasonable thing to do. [suggested by Christian] Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/state_trackers/vdpau/surface.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/vdpau

[Mesa-dev] [PATCH 3/7] st/xvmc: exit gracefully if we fail to create video buffer

2013-08-19 Thread Emil Velikov
Free any allocated memory and return BadAlloc if create_video_buffer() has failed to create a buffer. Reviewed-by: Christian König christian.koe...@amd.com Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/state_trackers/xvmc/surface.c | 4 1 file changed, 4 insertions

[Mesa-dev] [PATCH 4/7] vl/buffer: add sanity check after CALLOC_STRUCT

2013-08-19 Thread Emil Velikov
Check if we have successfully allocated memory. Reviewed-by: Christian König christian.koe...@amd.com Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/vl/vl_video_buffer.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/auxiliary/vl

[Mesa-dev] [PATCH 5/7] vl/idct: cleanup all idct buffers

2013-08-19 Thread Emil Velikov
Code should loop through and cleanup the three (VL_NUM_COMPONENTS) idct buffers, rather than doing the first one three times. Reviewed-by: Christian König christian.koe...@amd.com Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 2 +- 1 file

[Mesa-dev] [PATCH 6/7] st/vdpau: drop unnecessary variable prof

2013-08-19 Thread Emil Velikov
Any decent compiler will do this for us, although doing this will make grepping through the code alot easier. v2: In both mixer and query interface v3: rebase Reviewed-by: Christian König christian.koe...@amd.com [v1] Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium

[Mesa-dev] [PATCH 7/7] vl/buffers: consistent use on VL_MAX_SURFACES

2013-08-19 Thread Emil Velikov
Reviewed-by: Christian König christian.koe...@amd.com Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/vl/vl_video_buffer.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/auxiliary/vl/vl_video_buffer.c b/src/gallium/auxiliary

Re: [Mesa-dev] [PATCH 2/7] st/vdpau: don't try to create video buffer when the format is FORMAT_NONE

2013-08-19 Thread Emil Velikov
On 19/08/13 17:09, Christian König wrote: Am 19.08.2013 18:00, schrieb Emil Velikov: Not seen in the wild yet, but seems like a reasonable thing to do. [suggested by Christian] Signed-off-by: Emil Velikov emil.l.veli...@gmail.com Reviewed-by: Christian König christian.koe...@amd.com Do

[Mesa-dev] [PATCH] Bump version to 8.0.0

2012-05-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index da4d445..3f49773 100644 --- a/configure.ac +++ b/configure.ac @@ -22,7 +22,7 @@ dnl Process this file with autoconf

Re: [Mesa-dev] Please test the automake-gallium4 branch

2012-12-02 Thread Emil Velikov
On 23/11/12 21:23, Andreas Boll wrote: Hi mesa-list, I'm happy to announce that I've finished Matt's automake-gallium branch [0]. Some of the independent patches are already in the master branch. I've fixed the missing symbols in vdpau and xvmc and installed lots of compatibility links to

Re: [Mesa-dev] Please test the automake-gallium4 branch

2012-12-02 Thread Emil Velikov
On 03/12/12 00:15, Marek Olšák wrote: On Sun, Dec 2, 2012 at 9:48 PM, Emil Velikov emil.l.veli...@gmail.com wrote: On 23/11/12 21:23, Andreas Boll wrote: Hi mesa-list, I'm happy to announce that I've finished Matt's automake-gallium branch [0]. Some of the independent patches are already

Re: [Mesa-dev] Please test the automake-gallium4 branch

2012-12-03 Thread Emil Velikov
On 03/12/12 01:42, Marek Olšák wrote: On Mon, Dec 3, 2012 at 2:30 AM, Emil Velikov emil.l.veli...@gmail.com wrote: On 03/12/12 00:15, Marek Olšák wrote: On Sun, Dec 2, 2012 at 9:48 PM, Emil Velikov emil.l.veli...@gmail.com wrote: On 23/11/12 21:23, Andreas Boll wrote: Hi mesa-list, I'm

[Mesa-dev] [PATCH] configure.ac: Disable compiler optimizations when --enable-debug is set

2012-12-12 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index a133902..1294740 100644 --- a/configure.ac +++ b/configure.ac @@ -341,18 +341,18 @@ AC_ARG_ENABLE([debug], if test x

Re: [Mesa-dev] [PATCH] Revert configure.ac: Disable compiler optimizations when --enable-debug is set

2013-01-08 Thread Emil Velikov
On 08/01/13 17:27, Marek Olšák wrote: On Tue, Jan 8, 2013 at 6:11 PM, Matt Turner matts...@gmail.com wrote: On Tue, Jan 8, 2013 at 8:38 AM, Marek Olšák mar...@gmail.com wrote: This reverts commit 122345876479cf5cf553e38162ab105658614ab7. Some distributions use --enable-debug for testing

[Mesa-dev] [PATCH] configure.ac: Introduce --disable-optimizations

2013-01-08 Thread Emil Velikov
Allows us to easily disable compiler optimizations without directly dealing with C{,XX}FLAGS Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac | 22 ++ 1 file changed, 22 insertions(+) diff --git a/configure.ac b/configure.ac index 1414ce8..4d96ba0 100644

Re: [Mesa-dev] [PATCH] configure.ac: Introduce --disable-optimizations

2013-01-08 Thread Emil Velikov
On 08/01/13 20:05, Matt Turner wrote: On Tue, Jan 8, 2013 at 11:26 AM, Emil Velikov emil.l.veli...@gmail.com wrote: Allows us to easily disable compiler optimizations without directly dealing with C{,XX}FLAGS Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- This is getting silly

Re: [Mesa-dev] [PATCH 1/9] st/vdpau: implement support for extra mixer layers

2012-03-05 Thread Emil Velikov
On Sun, 04 Mar 2012 11:52:36 -, Christian König deathsim...@vodafone.de wrote: Signed-off-by: Christian König deathsim...@vodafone.de --- src/gallium/state_trackers/vdpau/mixer.c | 27 +-- 1 files changed, 21 insertions(+), 6 deletions(-) Hello Christian

Re: [Mesa-dev] [PATCH 1/2] automake: convert es1api

2012-08-23 Thread Emil Velikov
Hi Matt I believe that this patch would break the es2 build, as they both use the same Makefile. Can you change their commir order ? A few notes in-line as well On Tue, 21 Aug 2012 23:47:03 +0100, Matt Turner matts...@gmail.com wrote: --- configure.ac|2 +

[Mesa-dev] [PATCH] st/mesa: Use correct internal target

2011-06-29 Thread Emil Velikov
Commit 1a339b6c(st/mesa: prefer native texture formats when possible) introduced two new arguments to the st_choose_format() functions. This patch fixes the order and passes the correct internal_target rather than GL_NONE NOTE: This is a candidate for the 7.11 branch Signed-off-by: Emil Velikov

[Mesa-dev] [PATCH] st/mesa: Check after _mesa_create_context()

2011-07-11 Thread Emil Velikov
-by: Emil Velikov emil.l.veli...@gmail.com --- src/mesa/state_tracker/st_context.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/mesa/state_tracker/st_context.c b/src/mesa/state_tracker/st_context.c index 6eddbfc..d021d64 100644 --- a/src/mesa/state_tracker

[Mesa-dev] [PATCH] autoconf: Do not select Xlib when building DRI

2011-07-11 Thread Emil Velikov
As Chia-I Wu said 'There are two libGL providers, Xlib and DRI based they cannot coexist' Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index dde13c9..19cc531 100644

[Mesa-dev] [PATCH] st/mesa: Check after _mesa_create_context()

2011-07-12 Thread Emil Velikov
In some cases _mesa_create_context() can return NULL an in the mesa state tracker, we do not concider the case, which may cause issues within st_create_context_priv() This patch adds a simple check (similar to the one in the dri drivers) v2 Do not zero-out the funcs table Signed-off-by: Emil

[Mesa-dev] [PATCH] utils: Add missing parentheses

2011-07-13 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/util/u_video.h |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/gallium/auxiliary/util/u_video.h b/src/gallium/auxiliary/util/u_video.h index 78cceb6..46da136 100644 --- a/src/gallium

[Mesa-dev] [PATCH] configure.ac: Do not check for LIBDRM_RADEON at startup

2011-07-14 Thread Emil Velikov
Is it still possible to build radeon,r300,r600 without LIBDRM_RADEON? Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/configure.ac b/configure.ac index c186240..d156979 100644 --- a/configure.ac

[Mesa-dev] [PATCH] configure.ac: When built with galluim check for the respective LIBDRM_*

2011-07-14 Thread Emil Velikov
In a rare case of building gallium only, we need to check if the required packages are available libdrm_[intel|radeon|nouveau] - gallium[i915 i965|r300 r600|nouveau] Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac |5 + 1 files changed, 5 insertions(+), 0 deletions

Re: [Mesa-dev] [PATCH] configure.ac: When built with galluim check for the respective LIBDRM_*

2011-07-14 Thread Emil Velikov
14, 2011 at 10:35 PM, Emil Velikov emil.l.veli...@gmail.com wrote: In a rare case of building gallium only, we need to check if the required packages are available libdrm_[intel|radeon|nouveau] - gallium[i915 i965|r300 r600|nouveau] Signed-off-by: Emil Velikov emil.l.veli...@gmail.com

[Mesa-dev] [PATCHv2] configure.ac: Check for the respective libdrm_* when building gallium drivers

2011-07-14 Thread Emil Velikov
In a rare case of building gallium only, we need to check if the required packages are available libdrm_[intel|nouveau] - gallium[i915 i965|nouveau] v2: r300g and r600g do not need libdrm_radeon Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac |3 +++ 1 files changed

[Mesa-dev] Minor st/xorg related cleanups

2013-09-21 Thread Emil Velikov
A couple of small things that I've noticed while going through. Bit curious if there would be anyone to review and/or commit the nouveau patch so I've decided to send it along the st/xorg one :P Cheers Emil [PATCH 1/2] targets/xorg-nouveau: drop usage of dri1 function [PATCH 2/2] st/xorg: add

[Mesa-dev] [PATCH 1/2] targets/xorg-nouveau: drop usage of dri1 function DRICreatePCIBusID

2013-09-21 Thread Emil Velikov
card. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- As there is not much testing of xorg-nouveau one could be rather cautious with such commits, although this is a direct port from the nouveau ddx. If anyone believes that if would be beneficial to split the changes into separate patches

[Mesa-dev] [PATCH 2/2] st/xorg: add sanity checks after malloc

2013-09-21 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/state_trackers/xorg/xorg_driver.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/gallium/state_trackers/xorg/xorg_driver.c b/src/gallium/state_trackers/xorg/xorg_driver.c index 9d7713c..49e726b 100644 --- a/src

Re: [Mesa-dev] [RFC] Consolidate the remaining source files to Makefile.sources

2013-09-21 Thread Emil Velikov
On 12/09/13 18:04, Jakob Bornecrantz wrote: On Thu, Aug 15, 2013 at 7:04 PM, Emil Velikov emil.l.veli...@gmail.comwrote: Hello list Feeling inspired by the automake work done in mesa, I felt like finishing a few things that did not receive too much attention * use Makefile.sources

[Mesa-dev] Consolidate the source files to Makefile.sources v2

2013-09-22 Thread Emil Velikov
This is a first part of my planned automake rework - moving source files list(s) into Makefile.sources. * The first four patches are rather trivial cleanups (first one being an update of Johannes Obermayr's patch from his ). * The next 24 provides Makefile.sources for the following

[Mesa-dev] [PATCH 01/29] gallium/targets: Make use of prebuilt libdricommon.la.

2013-09-22 Thread Emil Velikov
HAVE_COMMON_DRI thus do not set in during configure. v2: [Emil] Add commit message and drop HAVE_COMMON_DRI from configure.ac Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac | 1 - src/gallium/targets/dri-freedreno/Makefile.am | 10 +++--- src

[Mesa-dev] [PATCH 02/29] mesa/drivers: drop HAVE_*_DRI from individual makefiles

2013-09-22 Thread Emil Velikov
The mesa/drivers/dri/Makefile.am already guards the individual targets/subdirs with HAVE_*_DRI before including them. Thus making the additional check within each Makefile.am unnecessary. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/mesa/drivers/dri/i915/Makefile.am| 3

[Mesa-dev] [PATCH 03/29] gallium/radeon: drop unused variable LIBGALLIUM_LIBS

2013-09-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/radeon/Makefile.am | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/gallium/drivers/radeon/Makefile.am b/src/gallium/drivers/radeon/Makefile.am index ac8cbd3..d5a6ff2 100644 --- a/src/gallium/drivers/radeon

[Mesa-dev] [PATCH 04/29] r600: use NEED_RADEON_LLVM over R600_NEED_RADEON_GALLIUM

2013-09-22 Thread Emil Velikov
libllvmradeon.la is available whenever NEED_RADEON_LLVM is set, using R600_NEED_RADEON_GALLIUM is rather ambiguous and unnecessary. Drop it in favour of NEED_RADEON_LLVM. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac | 2 -- src/gallium/drivers

[Mesa-dev] [PATCH 05/29] softpipe: consolidate C sources list into Makefile.sources

2013-09-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/softpipe/Android.mk | 33 ++ src/gallium/drivers/softpipe/Makefile.am | 32 ++--- src/gallium/drivers/softpipe/Makefile.sources | 30 +++ src

[Mesa-dev] [PATCH 06/29] rbug: consolidate C sources list into Makefile.sources

2013-09-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/rbug/Makefile.am | 7 ++- src/gallium/drivers/rbug/Makefile.sources | 5 + src/gallium/drivers/rbug/SConscript | 8 ++-- 3 files changed, 9 insertions(+), 11 deletions(-) create mode 100644 src

[Mesa-dev] [PATCH 07/29] llvmpipe: consolidate C sources list into Makefile.sources

2013-09-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/llvmpipe/Makefile.am | 44 ++ src/gallium/drivers/llvmpipe/Makefile.sources | 42 + src/gallium/drivers/llvmpipe/SConscript | 45 ++- 3

[Mesa-dev] [PATCH 08/29] trace: consolidate C sources list into Makefile.sources

2013-09-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/trace/Makefile.am | 8 ++-- src/gallium/drivers/trace/Makefile.sources | 6 ++ src/gallium/drivers/trace/SConscript | 9 ++--- 3 files changed, 10 insertions(+), 13 deletions(-) create mode 100644

[Mesa-dev] [PATCH 09/29] freedreno: consolidate C sources list into Makefile.sources

2013-09-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/drivers/freedreno/Makefile.am | 13 ++--- src/gallium/drivers/freedreno/Makefile.sources | 11 +++ src/gallium/drivers/freedreno/a2xx/Makefile.am | 17 ++--- src/gallium/drivers

[Mesa-dev] [PATCH 10/29] identity: consolidate C sources list into Makefile.sources

2013-09-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac | 1 + src/gallium/drivers/Makefile.am | 11 +-- src/gallium/drivers/identity/Makefile.am | 11 +++ src/gallium/drivers/identity/Makefile.sources | 4

[Mesa-dev] [PATCH 11/29] noop: consolidate C sources list into Makefile.sources

2013-09-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac | 1 + src/gallium/drivers/Makefile.am | 13 + src/gallium/drivers/noop/Makefile.am | 16 src/gallium/drivers/noop/Makefile.sources | 3 +++ src/gallium

[Mesa-dev] [PATCH 12/29] galahad: consolidate C sources list into Makefile.sources

2013-09-22 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- configure.ac | 1 + src/gallium/drivers/Makefile.am | 21 + src/gallium/drivers/galahad/Makefile.am | 16 src/gallium/drivers/galahad/Makefile.sources

  1   2   3   4   5   6   7   8   9   10   >