[PATCH] drm: fix the usage after free

2015-08-24 Thread Jammy Zhou
From: Mathias Tillman For readdir_r(), the next directory entry is returned in caller-allocted buffer (pointered by pent here). https://bugs.freedesktop.org/show_bug.cgi?id=91704 Signed-off-by: Jammy Zhou --- xf86drm.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/x

[Bug 91704] Tonga startx fails since mesa/drm 56d8dd6 amdgpu: make vamgr per device v2

2015-08-24 Thread bugzilla-dae...@freedesktop.org
this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/eb479a79/attachment.html>

[Bug 103271] AMD R9 270X Flickering with DPM Enabled on Linux 4.1 with RadeonSI

2015-08-24 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=103271 Alex Deucher changed: What|Removed |Added CC||alexdeucher at gmail.com --- Comment #3 f

[PATCH] drm: fix the usage after free

2015-08-24 Thread Christian König
On 24.08.2015 05:56, Jammy Zhou wrote: > From: Mathias Tillman > > For readdir_r(), the next directory entry is returned in caller-allocted > buffer (pointered by pent here). > > https://bugs.freedesktop.org/show_bug.cgi?id=91704 > > Signed-off-by: Jammy Zhou Would be more convenient if Mathias

[Bug 91704] Tonga startx fails since mesa/drm 56d8dd6 amdgpu: make vamgr per device v2

2015-08-24 Thread bugzilla-dae...@freedesktop.org
this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/d7b50cde/attachment.html>

[PATCH] drm: fix the usage after free

2015-08-24 Thread Zhou, Jammy
> Would be more convenient if Mathias would add his Signed-off-by as well and > send out the patch, cause he is the original author. Agreed. Just was not quite sure if Mathias is working on the libdrm project directly or not based on the comments in the bugzilla "hopefully the fix can be pushed

[PATCH] drm: fix the usage after free

2015-08-24 Thread Christian König
quot;/%s", > > + ent->d_name); > > + > > free(pent); > > closedir(sysdir); > > > > - snprintf(dev_name, sizeof(dev_name), > DRM_DIR_NAME "/%s", > > - ent->d_name); > > return strdup(dev_name); > > } > > } > -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/fb484601/attachment-0001.html>

drm/amdgpu: implement cgs gpu memory callbacks

2015-08-24 Thread Zhou, David(ChunMing)
Hi Dan, Thanks for figuring out that. >274 min_offset = obj->placements[0].fpfn << PAGE_SHIFT; >275 max_offset = obj->placements[0].lpfn << PAGE_SHIFT; Maybe should be: min_offset = obj->placements[0].fpfn; min_offset <<= PAGE_SHIFT; max_offset = o

[PATCH 1/2] amdgpu: remove sequence mutex

2015-08-24 Thread Christian König
From: Christian König It's not used any more. Signed-off-by: Christian König --- amdgpu/amdgpu_cs.c | 10 -- amdgpu/amdgpu_internal.h | 3 --- 2 files changed, 13 deletions(-) diff --git a/amdgpu/amdgpu_cs.c b/amdgpu/amdgpu_cs.c index 41071fd..ae78a4c 100644 --- a/amdgpu/amdg

[PATCH 2/2] amdgpu: serialize drmPrimeFDToHandle

2015-08-24 Thread Christian König
From: Christian König Fixes the same problem as "intel: Serialize drmPrimeFDToHandle with struct_mutex". Signed-off-by: Christian König --- amdgpu/amdgpu_bo.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c index dab3804..

about mmap dma-buf and sync

2015-08-24 Thread Thomas Hellstrom
Hi! On 08/22/2015 01:06 AM, Tiago Vignatti wrote: > Hi back! > > On 08/20/2015 03:48 AM, Thomas Hellstrom wrote: >> Hi, Tiago! > > Something that the Chrome OS folks told me today is whether we could > change the sync API to use a syscall for that instead. Reason for that > is to eventually fit th

about mmap dma-buf and sync

2015-08-24 Thread Thomas Hellstrom
On 08/24/2015 03:41 AM, Jerome Glisse wrote: > On Sat, Aug 22, 2015 at 12:00:21AM +0200, Thomas Hellstrom wrote: >> On 08/21/2015 06:00 PM, Jerome Glisse wrote: >>> On Fri, Aug 21, 2015 at 04:15:53PM +0200, Thomas Hellstrom wrote: On 08/21/2015 03:32 PM, Jerome Glisse wrote: > On Fri, Aug

[Bug 91741] Opengl Version Downgrades to 2.1 from 3.3 in Ubuntu 15.04

2015-08-24 Thread bugzilla-dae...@freedesktop.org
ignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/993b4211/attachment.html>

[Bug 91704] Tonga startx fails since mesa/drm 56d8dd6 amdgpu: make vamgr per device v2

2015-08-24 Thread bugzilla-dae...@freedesktop.org
attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/b7b74d02/attachment.html>

[Bug 91704] Tonga startx fails since mesa/drm 56d8dd6 amdgpu: make vamgr per device v2

2015-08-24 Thread bugzilla-dae...@freedesktop.org
ves/dri-devel/attachments/20150824/77cbd2ab/attachment.html>

[Bug 91741] Opengl Version Downgrades to 2.1 from 3.3 in Ubuntu 15.04

2015-08-24 Thread bugzilla-dae...@freedesktop.org
scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/8cd587cc/attachment.html>

[PATCH 03/11] drm/exynos: add prepare and cleanup phases for planes

2015-08-24 Thread Inki Dae
On 2015년 08월 16일 01:26, Gustavo Padovan wrote: > From: Gustavo Padovan > > .prepare_plane() and .cleanup_plane() allows to perform extra operations > before and after the update of planes. For FIMD for example this will > be used to enable disable the shadow protection bit. > > Signed-off-

[PATCH] DRM - radeon: Don't link train DisplayPort on HPD until we get the dpcd

2015-08-24 Thread Jerome Glisse
On Fri, Aug 21, 2015 at 02:16:12PM -0400, cpaul at redhat.com wrote: > From: Stephen Chandler Paul > > Most of the time this isn't an issue since hotplugging an adaptor will > trigger a crtc mode change which in turn, causes the driver to probe > every DisplayPort for a dpcd. However, in cases wh

[PATCH 2/2] amdgpu: serialize drmPrimeFDToHandle

2015-08-24 Thread Zhou, Jammy
Both patches are Reviewed-by: Jammy Zhou Regards, Jammy -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Monday, August 24, 2015 5:44 PM To: dri-devel at lists.freedesktop.org Subject: [PATCH 2/2] amdgpu: serialize dr

[PATCH 00/14] drm/exynos: rewrite fimg2d error handling

2015-08-24 Thread Tobias Jakobi
Hello, during the discussion about the last patchset touching the fimg2d code, it became apparent that the error handling for the command submission is currently unsatisfactory. This series rewrites the handling. All functions that submit command buffers now first check if enough space is availab

[PATCH 01/14] exynos/fimg2d: fix empty buffer handling in g2d_flush()

2015-08-24 Thread Tobias Jakobi
Empty command buffers are no error, we just don't have anything to do for flushing then. Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/exynos/exynos_fimg2d.c b/exynos/exynos_fimg2d.c index 24a06d0..4a88e0c 100644 --- a

[PATCH 02/14] exynos/fimg2d: simplify base address submission in g2d_scale_and_blend()

2015-08-24 Thread Tobias Jakobi
Use g2d_add_base_addr() for source and destination base address just like all other calls. Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/exynos/exynos_fimg2d.c b/exynos/exynos_fimg2d.c index 4a88e0c..85

[PATCH 03/14] exynos/fimg2d: add g2d_check_space()

2015-08-24 Thread Tobias Jakobi
This is going to be used to check if the command buffers have enough space left prior to actual submission of the commands. Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 17 + 1 file changed, 17 insertions(+) diff --git a/exynos/exynos_fimg2d.c b/exynos/exynos_fimg2d

[PATCH 04/14] exynos/fimg2d: check buffer space in g2d_solid_fill()

2015-08-24 Thread Tobias Jakobi
The amount of commands (regular and GEM) doesn't depend on the input here. Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/exynos/exynos_fimg2d.c b/exynos/exynos_fimg2d.c index 1ae8adf..9b7bcce 100644 --- a/exynos/exynos_fimg2d.c +++

[PATCH 05/14] exynos/fimg2d: check buffer space in g2d_copy()

2015-08-24 Thread Tobias Jakobi
Move the parameter validation before buffer space checking so that we can exit early if it fails. Also don't reset the G2D context anymore in this situation (since the buffers are not partially submitted). Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 26 ++ 1

[PATCH 06/14] exynos/fimg2d: check buffer space in g2d_copy_with_scale()

2015-08-24 Thread Tobias Jakobi
Parameter validation goes to the top. Repeat mode is checked first to make computation of space easier. Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 53 -- 1 file changed, 30 insertions(+), 23 deletions(-) diff --git a/exynos/exynos_f

[PATCH 07/14] exynos/fimg2d: add g2d_validate_xyz() functions

2015-08-24 Thread Tobias Jakobi
The G2D headers define a number of modes through enums (like e.g. color, select, repeat, etc.). This introduces g2d_validate_select_mode() and g2d_validate_blending_op() which validate a select mode or blending operation respectively. Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 44

[PATCH 08/14] exynos/fimg2d: check buffer space in g2d_blend()

2015-08-24 Thread Tobias Jakobi
Move parameter validation to the top and also validate the select mode of the source image and the requested blending operation before starting command submission. Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 66 +- 1 file changed, 39

[PATCH 09/14] exynos/fimg2d: check buffer space in g2d_scale_and_blend()

2015-08-24 Thread Tobias Jakobi
Apply the same transformation as in g2d_blend(). Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 67 +- 1 file changed, 39 insertions(+), 28 deletions(-) diff --git a/exynos/exynos_fimg2d.c b/exynos/exynos_fimg2d.c index 5acccf8..4274a94

[PATCH 10/14] exynos/fimg2d: remove default case from g2d_get_blend_op()

2015-08-24 Thread Tobias Jakobi
We now validate the blending mode via g2d_validate_mode() prior to feeding it to g2d_get_blend_op(). Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 5 - 1 file changed, 5 deletions(-) diff --git a/exynos/exynos_fimg2d.c b/exynos/exynos_fimg2d.c index 4274a94..d708e91 100644 --- a

[PATCH 11/14] exynos/fimg2d: remove superfluous initialization of g2d_point_val

2015-08-24 Thread Tobias Jakobi
The g2d_point_val union consists of two coordinates of 16 bits. Whenever this union is used though, both coordinates are explicitly set. Hence prior initialization is unnecessary. Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 19 --- 1 file changed, 19 deletions(-) d

[PATCH 12/14] exynos/fimg2d: make g2d_add_cmd() less heavy

2015-08-24 Thread Tobias Jakobi
The function currently checks for each added command if an overflow of the corresponding command buffers occurs, but none of the callers ever checks the return value. Since all callers are now converted to use g2d_check_space() simplify the function. (1) The overflow checks become asserts, so the

[PATCH 13/14] exynos/fimg2d: add message prefix

2015-08-24 Thread Tobias Jakobi
Add a prefix to the messages printed to the console via printf() and fprintf() so that one can easily see where the message comes from. Signed-off-by: Tobias Jakobi --- exynos/exynos_fimg2d.c | 30 -- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/exy

[PATCH] DRM - radeon: Don't link train DisplayPort on HPD until we get the dpcd

2015-08-24 Thread Deucher, Alexander
> -Original Message- > From: Jerome Glisse [mailto:j.glisse at gmail.com] > Sent: Monday, August 24, 2015 9:48 AM > To: cpaul at redhat.com > Cc: Deucher, Alexander; Koenig, Christian; David Airlie; dri- > devel at lists.freedesktop.org; linux-kernel at vger.kernel.org; Jerome > Glisse; >

[PATCH] drm: fix the usage after free

2015-08-24 Thread Emil Velikov
Hi guys, On 24 August 2015 at 09:20, Christian König wrote: > Hi Mathias, > > thanks for the good help and no problem with the Signed-off-by line. If you > don't object we will just add a "Signed-off-by: Mathias Tillman > " line when we push this patch to the repository. > > It's just for tracki

drm/msm/dsi: hs_zero timing

2015-08-24 Thread Hai Li
Hi Werner, Yes, the register is to adjust hs_zero. Could you share the panel's video timing and dphy timings (or the panel DT), used by downstream driver? The dphy timing calculations in the phy driver are from the excel sheet as well, I can check if there is any issue inside the calculation co

[PATCH 14/14] exynos/fimg2d: remove g2d_context from public header

2015-08-24 Thread Tobias Jakobi
All functions from the public API only operation on struct g2d_context*, so this shouldn't break too much. Make the context private since we don't want the user to modify its content directly. Also remove the defines that were only used for fields of g2d_context. Signed-off-by: Tobias Jakobi ---

[PATCH 2/2] amdgpu: serialize drmPrimeFDToHandle

2015-08-24 Thread Emil Velikov
[CCing mesa-dev as mesa will need similar fixes] Hi Christian, On 24/08/15 10:43, Christian König wrote: > From: Christian König > > Fixes the same problem as "intel: Serialize drmPrimeFDToHandle with > struct_mutex". > > Signed-off-by: Christian König > --- > amdgpu/amdgpu_bo.c | 9

[Bug 91741] Opengl Version Downgrades to 2.1 from 3.3 in Ubuntu 15.04

2015-08-24 Thread bugzilla-dae...@freedesktop.org
reedesktop.org/archives/dri-devel/attachments/20150824/71b5e51c/attachment.html>

about mmap dma-buf and sync

2015-08-24 Thread Daniel Stone
Hi Thomas, On 24 August 2015 at 10:50, Thomas Hellstrom wrote: > In any case, Ideally I'd want the struct dma_buf_sync look something like > > enum dma_buf_sync_flags { > DMA_BUF_SYNC_READ = (1 << 0), > DMA_BUF_SYNC_WRITE = (2 << 0), > DMA_BUF_SYNC_RW = (3 << 0), >

[Bug 91741] Opengl Version Downgrades to 2.1 from 3.3 in Ubuntu 15.04

2015-08-24 Thread bugzilla-dae...@freedesktop.org
-- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/b21e9f00/attachment.html>

[PATCH libdrm 01/17] vbltest: fix variable shadowing warning

2015-08-24 Thread Emil Velikov
Just remove the second (shadowing) declaration of ret. Signed-off-by: Emil Velikov --- tests/vbltest/vbltest.c | 1 - 1 file changed, 1 deletion(-) diff --git a/tests/vbltest/vbltest.c b/tests/vbltest/vbltest.c index 4200adb..e27f45c 100644 --- a/tests/vbltest/vbltest.c +++ b/tests/vbltest/vblt

[PATCH libdrm 02/17] modetest: fix shadowing warnings

2015-08-24 Thread Emil Velikov
No real issue here, but let's fix these so that real issues don't get lost in the spam. Signed-off-by: Emil Velikov --- tests/modetest/modetest.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c index 3a545c3..3b01918 100644 --- a/tests/

[PATCH libdrm 03/17] intel: resolve shadowing warnings

2015-08-24 Thread Emil Velikov
Cc: intel-gfx at lists.freedesktop.org Signed-off-by: Emil Velikov --- intel/intel_bufmgr_fake.c | 2 +- intel/intel_bufmgr_gem.c | 7 +++ intel/intel_decode.c | 7 ++- 3 files changed, 6 insertions(+), 10 deletions(-) diff --git a/intel/intel_bufmgr_fake.c b/intel/intel_bufmgr_fak

[PATCH libdrm 04/17] intel: error out on has_error in exec2

2015-08-24 Thread Emil Velikov
Just like we do for the original exec() Cc: intel-gfx at lists.freedesktop.org Signed-off-by: Emil Velikov --- intel/intel_bufmgr_gem.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c index cf55a53..5287419 100644 --- a/intel/intel_bufmg

[PATCH libdrm 05/17] configure: enable -Wshadow when available

2015-08-24 Thread Emil Velikov
Signed-off-by: Emil Velikov --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index f07507b..c2a8ed1 100644 --- a/configure.ac +++ b/configure.ac @@ -174,7 +174,7 @@ MAYBE_WARN="-Wall -Wextra \ -Wstrict-aliasing=2 -Winit-self \ -Wde

[PATCH libdrm 07/17] modetest: annotate const data as such for good measure

2015-08-24 Thread Emil Velikov
Signed-off-by: Emil Velikov --- tests/modetest/cursor.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/modetest/cursor.c b/tests/modetest/cursor.c index 62a50ef..d8a19bd 100644 --- a/tests/modetest/cursor.c +++ b/tests/modetest/cursor.c @@ -70,7 +70,7 @@ stat

[PATCH libdrm 08/17] libkms: use static const for good measure

2015-08-24 Thread Emil Velikov
Signed-off-by: Emil Velikov --- libkms/linux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libkms/linux.c b/libkms/linux.c index b735527..6e0da83 100644 --- a/libkms/linux.c +++ b/libkms/linux.c @@ -149,7 +149,7 @@ struct create_record int (*func)(int fd, struct km

[PATCH libdrm 09/17] radeon: cleanup bo/cs func tables

2015-08-24 Thread Emil Velikov
Annotate the data as static const and use C99 designated initializers. Signed-off-by: Emil Velikov --- radeon/radeon_bo_gem.c | 23 --- radeon/radeon_bo_int.h | 2 +- radeon/radeon_cs_gem.c | 20 ++-- radeon/radeon_cs_int.h | 2 +- 4 files changed, 24 insert

[PATCH libdrm 10/17] freedreno: annotate the device/bo/pipe/ringbuffer funcs as const data

2015-08-24 Thread Emil Velikov
Cc: freedreno at lists.freedesktop.org Signed-off-by: Emil Velikov --- freedreno/freedreno_priv.h | 6 +++--- freedreno/freedreno_ringbuffer.h | 2 +- freedreno/kgsl/kgsl_bo.c | 2 +- freedreno/kgsl/kgsl_device.c | 2 +- freedreno/kgsl/kgsl_pipe.c | 2 +- freedreno/kgsl/kg

[PATCH libdrm 11/17] tests/amdgpu: reference the correct variable for memcpy

2015-08-24 Thread Emil Velikov
Cc: Alex Deucher Cc: Leo Liu Signed-off-by: Emil Velikov --- tests/amdgpu/cs_tests.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/amdgpu/cs_tests.c b/tests/amdgpu/cs_tests.c index 416f36b..dfbf5af 100644 --- a/tests/amdgpu/cs_tests.c +++ b/tests/amdgpu/cs_tests.

[PATCH libdrm 12/17] tests/amdgpu: annotate the uvd message templates as const data

2015-08-24 Thread Emil Velikov
Cc: Alex Deucher Cc: Leo Liu Signed-off-by: Emil Velikov --- tests/amdgpu/uvd_messages.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/amdgpu/uvd_messages.h b/tests/amdgpu/uvd_messages.h index 44bcacc..00235cb 100644 --- a/tests/amdgpu/uvd_messages.h +++ b

[PATCH libdrm 13/17] tests/amdgpu: annotate (most of) the vce templates as const data

2015-08-24 Thread Emil Velikov
The remaining two templates are modified on the fly, depending on the type of test to be performed. Cc: Alex Deucher Cc: Leo Liu Signed-off-by: Emil Velikov --- tests/amdgpu/vce_ib.h | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/amdgpu/vce

[PATCH libdrm 14/17] tests/amdgpu: ensure tests work by using c99 initializers

2015-08-24 Thread Emil Velikov
In the latest version of CUnit the fourth parameter of the CU_SuiteInfo struct is pSetUpFunc rather than *pTests. Seems like the CUnit ABI broke at some point, so let's the the robust thing and use c99 designated initializers to correctly populate the struct(s). Cc: Alex Deucher Cc: Leo Liu Sig

[PATCH libdrm 17/17] configure: remove -Wno-missing-field-initializers

2015-08-24 Thread Emil Velikov
Signed-off-by: Emil Velikov --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index c2a8ed1..1d7bd5b 100644 --- a/configure.ac +++ b/configure.ac @@ -173,7 +173,7 @@ MAYBE_WARN="-Wall -Wextra \ -Wpacked -Wswitch-enum -Wmissing-format

[PATCH libdrm 06/17] amdgpu: remove unneeded -Wno-switch-enum flag

2015-08-24 Thread Emil Velikov
Signed-off-by: Emil Velikov --- amdgpu/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/amdgpu/Makefile.am b/amdgpu/Makefile.am index 37f7198..cf7bc1b 100644 --- a/amdgpu/Makefile.am +++ b/amdgpu/Makefile.am @@ -25,7 +25,7 @@ include Makefile.sources AM_CFLAGS =

[PATCH libdrm 15/17] nouveau: use designated initializers

2015-08-24 Thread Emil Velikov
Cc: nouveau at lists.freedesktop.org Signed-off-by: Emil Velikov --- nouveau/abi16.c | 13 ++--- nouveau/nouveau.c | 6 +++--- 2 files changed, 13 insertions(+), 6 deletions(-) diff --git a/nouveau/abi16.c b/nouveau/abi16.c index 4ca0bfb..59bc436 100644 --- a/nouveau/abi16.c +++ b/nou

[PATCH libdrm 16/17] intel: ignore missing-field-initializers warnings

2015-08-24 Thread Emil Velikov
We're about to remove the -Wno flag from configure.ac which will lead to a lot of unnecessary spam. Signed-off-by: Emil Velikov --- intel/intel_decode.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/intel/intel_decode.c b/intel/intel_decode.c index 345d457..e7aef74 100644 --- a/intel/in

about mmap dma-buf and sync

2015-08-24 Thread Thomas Hellstrom
Hi, Daniel, On 08/24/2015 05:52 PM, Daniel Stone wrote: > Hi Thomas, > > On 24 August 2015 at 10:50, Thomas Hellstrom wrote: >> In any case, Ideally I'd want the struct dma_buf_sync look something like >> >> enum dma_buf_sync_flags { >> DMA_BUF_SYNC_READ = (1 << 0), >> DMA_BUF_SYN

about mmap dma-buf and sync

2015-08-24 Thread Daniel Stone
Hi, On 24 August 2015 at 17:56, Thomas Hellstrom wrote: > On 08/24/2015 05:52 PM, Daniel Stone wrote: >> I still don't think this ameliorates the need for batching: consider >> the case where you update two disjoint screen regions and want them >> both flushed. Either you issue two separate sync

[PATCH libdrm 11/17] tests/amdgpu: reference the correct variable for memcpy

2015-08-24 Thread Alex Deucher
On Mon, Aug 24, 2015 at 12:34 PM, Emil Velikov wrote: > Cc: Alex Deucher > Cc: Leo Liu > Signed-off-by: Emil Velikov Reviewed-by: Alex Deucher > --- > tests/amdgpu/cs_tests.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/amdgpu/cs_tests.c b/tests/amdgpu

[PATCH libdrm 09/17] radeon: cleanup bo/cs func tables

2015-08-24 Thread Alex Deucher
On Mon, Aug 24, 2015 at 12:34 PM, Emil Velikov wrote: > Annotate the data as static const and use C99 designated initializers. > > Signed-off-by: Emil Velikov Reviewed-by: Alex Deucher > --- > radeon/radeon_bo_gem.c | 23 --- > radeon/radeon_bo_int.h | 2 +- > radeon/rad

[PATCH libdrm 12/17] tests/amdgpu: annotate the uvd message templates as const data

2015-08-24 Thread Alex Deucher
On Mon, Aug 24, 2015 at 12:34 PM, Emil Velikov wrote: > Cc: Alex Deucher > Cc: Leo Liu > Signed-off-by: Emil Velikov Reviewed-by: Alex Deucher > --- > tests/amdgpu/uvd_messages.h | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/tests/amdgpu/uvd_messages.h

[PATCH libdrm 13/17] tests/amdgpu: annotate (most of) the vce templates as const data

2015-08-24 Thread Alex Deucher
On Mon, Aug 24, 2015 at 12:34 PM, Emil Velikov wrote: > The remaining two templates are modified on the fly, depending on the > type of test to be performed. > > Cc: Alex Deucher > Cc: Leo Liu > Signed-off-by: Emil Velikov Reviewed-by: Alex Deucher > --- > tests/amdgpu/vce_ib.h | 24 ++

[PATCH libdrm 14/17] tests/amdgpu: ensure tests work by using c99 initializers

2015-08-24 Thread Alex Deucher
On Mon, Aug 24, 2015 at 12:34 PM, Emil Velikov wrote: > In the latest version of CUnit the fourth parameter of the CU_SuiteInfo > struct is pSetUpFunc rather than *pTests. > > Seems like the CUnit ABI broke at some point, so let's the the robust > thing and use c99 designated initializers to corr

[PATCH libdrm 06/17] amdgpu: remove unneeded -Wno-switch-enum flag

2015-08-24 Thread Alex Deucher
On Mon, Aug 24, 2015 at 12:34 PM, Emil Velikov wrote: > Signed-off-by: Emil Velikov Reviewed-by: Alex Deucher > --- > amdgpu/Makefile.am | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/amdgpu/Makefile.am b/amdgpu/Makefile.am > index 37f7198..cf7bc1b 100644 > --- a/am

about mmap dma-buf and sync

2015-08-24 Thread Thomas Hellstrom
On 08/24/2015 07:04 PM, Daniel Stone wrote: > Hi, > > On 24 August 2015 at 17:56, Thomas Hellstrom wrote: >> On 08/24/2015 05:52 PM, Daniel Stone wrote: >>> I still don't think this ameliorates the need for batching: consider >>> the case where you update two disjoint screen regions and want them

about mmap dma-buf and sync

2015-08-24 Thread Daniel Stone
Hi, On 24 August 2015 at 18:10, Thomas Hellstrom wrote: > On 08/24/2015 07:04 PM, Daniel Stone wrote: >> On 24 August 2015 at 17:56, Thomas Hellstrom >> wrote: >>> On 08/24/2015 05:52 PM, Daniel Stone wrote: I still don't think this ameliorates the need for batching: consider the case

about mmap dma-buf and sync

2015-08-24 Thread Thomas Hellstrom
On 08/24/2015 07:12 PM, Daniel Stone wrote: > Hi, > > On 24 August 2015 at 18:10, Thomas Hellstrom wrote: >> On 08/24/2015 07:04 PM, Daniel Stone wrote: >>> On 24 August 2015 at 17:56, Thomas Hellstrom >>> wrote: On 08/24/2015 05:52 PM, Daniel Stone wrote: > I still don't think this ame

about mmap dma-buf and sync

2015-08-24 Thread Tiago Vignatti
On 08/24/2015 02:42 PM, Thomas Hellstrom wrote: > On 08/24/2015 07:12 PM, Daniel Stone wrote: >> Hi, >> >> On 24 August 2015 at 18:10, Thomas Hellstrom >> wrote: >>> On 08/24/2015 07:04 PM, Daniel Stone wrote: On 24 August 2015 at 17:56, Thomas Hellstrom wrote: > On 08/24/2015 05:

drm/amdgpu: implement cgs gpu memory callbacks

2015-08-24 Thread Dan Carpenter
On Mon, Aug 24, 2015 at 07:09:15AM +, Zhou, David(ChunMing) wrote: > Hi Dan, > Thanks for figuring out that. > >274 min_offset = obj->placements[0].fpfn << PAGE_SHIFT; > >275 max_offset = obj->placements[0].lpfn << PAGE_SHIFT; > Maybe should be: > min_offset = o

[alsa-devel] [PATCH V5 3/3] ASoC: AMD: add AMD ASoC ACP-I2S driver

2015-08-24 Thread Alex Deucher
On Fri, Aug 21, 2015 at 12:17 PM, Mark Brown wrote: > On Fri, Aug 21, 2015 at 05:21:07PM +0530, maruthi srinivas wrote: >> On Fri, Aug 21, 2015 at 4:48 AM, Mark Brown wrote: > >> > We already have a driver for the DesignWare I2S controller. To repeat >> > the concern I raised in a slightly diffe

[PATCH] rnndb/dsi: Add more bits for DSI PHY

2015-08-24 Thread Samuel Pitoiset
Hi Hai, I don't know what those bits are used for, but I'm a bit curious about how did you find them? Also, the next time don't forget to send patches related to envytools to nouveau at lists.freedesktop.org (or better send a pull request from github). Thanks. On 08/13/2015 11:44 PM, Hai Li w

[PATCH] rnndb/dsi: Add more bits for DSI PHY

2015-08-24 Thread Rob Clark
On Mon, Aug 24, 2015 at 4:20 PM, Samuel Pitoiset wrote: > Hi Hai, > > I don't know what those bits are used for, but I'm a bit curious about how > did you find them? > > Also, the next time don't forget to send patches related to envytools to > nouveau at lists.freedesktop.org > (or better send a

[PATCH] rnndb/dsi: Add more bits for DSI PHY

2015-08-24 Thread Samuel Pitoiset
On 08/24/2015 10:16 PM, Rob Clark wrote: > On Mon, Aug 24, 2015 at 4:20 PM, Samuel Pitoiset > wrote: >> Hi Hai, >> >> I don't know what those bits are used for, but I'm a bit curious about how >> did you find them? >> >> Also, the next time don't forget to send patches related to envytools to >>

[PATCH] rnndb/dsi: Add more bits for DSI PHY

2015-08-24 Thread Ilia Mirkin
On Mon, Aug 24, 2015 at 4:26 PM, Samuel Pitoiset wrote: > On 08/24/2015 10:16 PM, Rob Clark wrote: >> On Mon, Aug 24, 2015 at 4:20 PM, Samuel Pitoiset >> wrote: >>> >>> Hi Hai, >>> >>> I don't know what those bits are used for, but I'm a bit curious about >>> how >>> did you find them? >>> >>> Al

[PATCH] rnndb/dsi: Add more bits for DSI PHY

2015-08-24 Thread Rob Clark
On Mon, Aug 24, 2015 at 4:26 PM, Samuel Pitoiset wrote: > > > On 08/24/2015 10:16 PM, Rob Clark wrote: >> >> On Mon, Aug 24, 2015 at 4:20 PM, Samuel Pitoiset >> wrote: >>> >>> Hi Hai, >>> >>> I don't know what those bits are used for, but I'm a bit curious about >>> how >>> did you find them? >>>

[Bug 103271] AMD R9 270X Flickering with DPM Enabled on Linux 4.1 with RadeonSI

2015-08-24 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=103271 --- Comment #4 from Kevin --- I did not change any hardware. I tried changing the connection to the monitor because I had issues a few months back with DisplayPort but that didn't help here. I don't seem to have an xorg log in /var/log/ but and t

[Bug 103271] AMD R9 270X Flickering with DPM Enabled on Linux 4.1 with RadeonSI

2015-08-24 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=103271 --- Comment #5 from Kevin --- Created attachment 185731 --> https://bugzilla.kernel.org/attachment.cgi?id=185731&action=edit dmesg output while running linux-git-4.2rc8.r0.gc13dcf9-1-x86_64 This is the dmesg output while running 4.2rc8 built f

[Bug 103271] AMD R9 270X Flickering with DPM Enabled on Linux 4.1 with RadeonSI

2015-08-24 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=103271 --- Comment #6 from Kevin --- I just uploaded the dmesg output. I have never done bisecting before. Is this a good place to start? https://wiki.archlinux.org/index.php/Bisecting_bugs -- You are receiving this mail because: You are watching the

[Bug 84614] radeon gpu crash /kernel crash when using dynamic light in borderlands 2

2015-08-24 Thread bugzilla-dae...@freedesktop.org
: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/608ef967/attachment.html>

[Bug 88968] Unit mouseover in World of Warcraft (wine, D3D) freezes up the entire system

2015-08-24 Thread bugzilla-dae...@freedesktop.org
s had an update to > version 6.1. So can I close this? -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/7f2a8bad/attachment.html>

[Bug 88968] Unit mouseover in World of Warcraft (wine, D3D) freezes up the entire system

2015-08-24 Thread bugzilla-dae...@freedesktop.org
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20150824/acf78ce6/attachment.html>

[Bug 89015] [radeonsi] unvanquished & nexuiz crash after switching AA

2015-08-24 Thread bugzilla-dae...@freedesktop.org
rg/archives/dri-devel/attachments/20150824/0625da7c/attachment.html>

about mmap dma-buf and sync

2015-08-24 Thread Tiago Vignatti
On 08/24/2015 03:01 PM, Tiago Vignatti wrote: > yup, I think so. So IIUC the main changes needed for the drivers > implement 2D sync lies in the dma_buf_sync_2d structure only. I.e. > there's nothing really to be changed in the common code, right? Do we have any special requirements in how we want

[PATCH v3 06/14] Documentation: drm/bridge: add document for analogix_dp

2015-08-24 Thread Heiko Stuebner
Hi Yakir, Am Montag, 24. August 2015, 20:48:01 schrieb Yakir Yang: > 在 08/24/2015 12:20 PM, Krzysztof Kozlowski 写道: > > On 24.08.2015 11:42, Yakir Yang wrote: > >> Hi Krzysztof, > >> > >> 在 08/23/2015 07:43 PM, Krzysztof Kozlowski 写道: > >>> 2015-08-24 8:23 GMT+09:00 Rob Herring : > >>

[PATCH v7 3/6] mm: Introduce VM_LOCKONFAULT

2015-08-24 Thread Vlastimil Babka
On 08/24/2015 03:50 PM, Konstantin Khlebnikov wrote: > On Mon, Aug 24, 2015 at 4:30 PM, Vlastimil Babka wrote: >> On 08/24/2015 12:17 PM, Konstantin Khlebnikov wrote: I am in the middle of implementing lock on fault this way, but I cannot see how we will hanlde mremap of a lock

drm/msm/dsi: hs_zero timing

2015-08-24 Thread Werner Johansson
On Mon, Aug 24, 2015 at 7:32 AM, Hai Li wrote: > Hi Werner, > > Yes, the register is to adjust hs_zero. > Could you share the panel's video timing and dphy timings (or the panel DT), > used by downstream driver? > > The dphy timing calculations in the phy driver are from the excel sheet as > wel

[PATCH v3 06/14] Documentation: drm/bridge: add document for analogix_dp

2015-08-24 Thread Krzysztof Kozlowski
2015-08-24 8:23 GMT+09:00 Rob Herring : > On Wed, Aug 19, 2015 at 9:50 AM, Yakir Yang wrote: >> Analogix dp driver is split from exynos dp driver, so we just >> make an copy of exynos_dp.txt, and then simplify exynos_dp.txt >> >> Beside update some exynos dtsi file with the latest change >> accord

[PATCH v3 06/14] Documentation: drm/bridge: add document for analogix_dp

2015-08-24 Thread Krzysztof Kozlowski
On 24.08.2015 11:42, Yakir Yang wrote: > Hi Krzysztof, > > 在 08/23/2015 07:43 PM, Krzysztof Kozlowski 写道: >> 2015-08-24 8:23 GMT+09:00 Rob Herring : >>> On Wed, Aug 19, 2015 at 9:50 AM, Yakir Yang wrote: Analogix dp driver is split from exynos dp driver, so we just make an copy of

[PATCH v3 06/14] Documentation: drm/bridge: add document for analogix_dp

2015-08-24 Thread Jingoo Han
On 2015. 8. 24., at AM 9:43, Krzysztof Kozlowski wrote: > > 2015-08-24 8:23 GMT+09:00 Rob Herring : >>> On Wed, Aug 19, 2015 at 9:50 AM, Yakir Yang wrote: >>> Analogix dp driver is split from exynos dp driver, so we just >>> make an copy of exynos_dp.txt, and then simplify exynos_dp.txt >>> >>

[PATCH] drm: fix the usage after free

2015-08-24 Thread Mathias Tillman
, DRM_DIR_NAME > "/%s", > > + ent->d_name); > > + > > free(pent); > > closedir(sysdir); > > > > - snprintf(dev_name, sizeof(dev_name), DRM_DIR_NAME > "/%s&q

[PATCH] drm: fix the usage after free

2015-08-24 Thread Mathias Tillman
nt != NULL) { >> > if (strncmp(ent->d_name, name, len) == 0) { >> > + snprintf(dev_name, sizeof(dev_name), DRM_DIR_NAME >> "/%s", >> > + ent->d_name); >> > + >> > free(pent); >> > closedir(sysdir); >> > >> > - snprintf(dev_name, sizeof(dev_name), DRM_DIR_NAME >> "/%s", >> > - ent->d_name); >> > return strdup(dev_name); >> > } >> > } >> >> > -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150824/4ab9cd12/attachment-0001.html>

[PATCH v3 06/14] Documentation: drm/bridge: add document for analogix_dp

2015-08-24 Thread Yakir Yang
Hi Jingoo, 在 08/24/2015 03:40 PM, Jingoo Han 写道: > On 2015. 8. 24., at AM 9:43, Krzysztof Kozlowski > wrote: >> 2015-08-24 8:23 GMT+09:00 Rob Herring : On Wed, Aug 19, 2015 at 9:50 AM, Yakir Yang wrote: Analogix dp driver is split from exynos dp driver, so we just make an c

[PATCH v3 06/14] Documentation: drm/bridge: add document for analogix_dp

2015-08-24 Thread Russell King - ARM Linux
On Sun, Aug 23, 2015 at 06:23:14PM -0500, Rob Herring wrote: > On Wed, Aug 19, 2015 at 9:50 AM, Yakir Yang wrote: > > + -analogix,color-depth: > > + number of bits per colour component. > > + COLOR_6 = 0, COLOR_8 = 1, COLOR_10 = 2, COLOR_12 = 3 > > This s

[PATCH v7 3/6] mm: Introduce VM_LOCKONFAULT

2015-08-24 Thread Vlastimil Babka
On 08/24/2015 12:17 PM, Konstantin Khlebnikov wrote: >> >> I am in the middle of implementing lock on fault this way, but I cannot >> see how we will hanlde mremap of a lock on fault region. Say we have >> the following: >> >> addr = mmap(len, MAP_ANONYMOUS, ...); >> mlock(addr, len, MLO

[PATCH 1/7] drm/vc4: Add devicetree bindings for VC4.

2015-08-24 Thread Rob Herring
On Mon, Aug 17, 2015 at 1:30 PM, Eric Anholt wrote: > Stephen Warren writes: > >> On 08/12/2015 06:56 PM, Eric Anholt wrote: >>> Signed-off-by: Eric Anholt >> >> This one definitely needs a patch description, since someone might not >> know what a VC4 is, and "git log" won't show the text from t

[PATCH v7 3/6] mm: Introduce VM_LOCKONFAULT

2015-08-24 Thread Konstantin Khlebnikov
On Mon, Aug 24, 2015 at 4:30 PM, Vlastimil Babka wrote: > On 08/24/2015 12:17 PM, Konstantin Khlebnikov wrote: >>> >>> >>> I am in the middle of implementing lock on fault this way, but I cannot >>> see how we will hanlde mremap of a lock on fault region. Say we have >>> the following: >>> >>>

[PATCH v3 06/14] Documentation: drm/bridge: add document for analogix_dp

2015-08-24 Thread Rob Herring
On Mon, Aug 24, 2015 at 7:57 AM, Russell King - ARM Linux wrote: > On Sun, Aug 23, 2015 at 06:23:14PM -0500, Rob Herring wrote: >> On Wed, Aug 19, 2015 at 9:50 AM, Yakir Yang wrote: >> > + -analogix,color-depth: >> > + number of bits per colour component. >> > +

[PATCH v7 3/6] mm: Introduce VM_LOCKONFAULT

2015-08-24 Thread Konstantin Khlebnikov
On Mon, Aug 24, 2015 at 6:09 PM, Eric B Munson wrote: > On Mon, 24 Aug 2015, Vlastimil Babka wrote: > >> On 08/24/2015 03:50 PM, Konstantin Khlebnikov wrote: >> >On Mon, Aug 24, 2015 at 4:30 PM, Vlastimil Babka wrote: >> >>On 08/24/2015 12:17 PM, Konstantin Khlebnikov wrote: >> >> >> >>

  1   2   >