[Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-09-20 Thread bugzilla-dae...@freedesktop.org
because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/78e57974/attachment-0001.html>

[Bug 172381] New: nouveau kernel module leaks memory on hibernate.

2016-09-20 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=172381 Bug ID: 172381 Summary: nouveau kernel module leaks memory on hibernate. Product: Drivers Version: 2.5 Kernel Version: 4.8rc6 Hardware: All OS: Linux Tree: Ma

[PATCH 5/6] GPU-DRM-GMA500: One error message less for a GCT revision mismatch in mid_get_vbt_data()

2016-09-20 Thread Patrik Jakobsson
On Tue, Sep 20, 2016 at 2:08 PM, Jani Nikula wrote: > On Tue, 20 Sep 2016, Dan Carpenter wrote: >> Don't be a dummy... This is easy to review an it fixes a bug. In this particular case it might not be clear that an unknown GCT version causes a complete GCT failure so both messages are useful.

[Bug 141741] drm:radeon_get_bios [radeon]] *ERROR* Unable to locate a BIOS ROM

2016-09-20 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=141741 --- Comment #14 from Jose Barros --- Created attachment 239331 --> https://bugzilla.kernel.org/attachment.cgi?id=239331&action=edit dmesg | egrep 'drm|radeon' on 4.6.0-kali1-amd64 -- You are receiving this mail because: You are watching the a

[PATCH v12 5/7] drm/i915/skl: Ensure pipes with changed wms get added to the state

2016-09-20 Thread Mike Lothian
Hi I've bisected back to this commit in the drm-intel-nightly branch 05a76d3d6ad1ee9f9814f88949cc9305fc165460 is the first bad commit commit 05a76d3d6ad1ee9f9814f88949cc9305fc165460 Author: Lyude Date: Wed Aug 17 15:55:57 2016 -0400 drm/i915/skl: Ensure pipes with changed wms get added to

[PATCH v12 5/7] drm/i915/skl: Ensure pipes with changed wms get added to the state

2016-09-20 Thread Mike Lothian
lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel > -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/1ee00b89/attachment-0001.html>

[PATCH v3] dma-buf/sync_file: Increment refcount of fence when all are signaled.

2016-09-20 Thread Sumit Semwal
Hi Rafael, On 16 September 2016 at 00:44, Rafael Antognolli wrote: > When we merge several fences, if all of them are signaled already, we > still keep one of them. So instead of using add_fence(), which will not > increase the refcount of signaled fences, we should explicitly call > fence_get()

[PATCH] dma-buf/sync_file: fix documentation error

2016-09-20 Thread Sumit Semwal
Hi Emilio, On 19 September 2016 at 17:40, Gustavo Padovan wrote: > Hi Emilio, > > 2016-09-19 Emilio López : > >> The ioctl name and description on the documentation block don't >> match the ioctl being defined. This was probably overlooked while >> renaming the ioctls during the sync file destag

[PATCH] dma-buf/sw_sync: mark sync_timeline_create() static

2016-09-20 Thread Sumit Semwal
Hi Baoyou, On 20 September 2016 at 16:43, Gustavo Padovan wrote: > 2016-09-18 Baoyou Xie : > >> We get 1 warning when building kernel with W=1: >> drivers/dma-buf/sw_sync.c:87:23: warning: no previous prototype for >> 'sync_timeline_create' [-Wmissing-prototypes] >> >> In fact, this function is

[PATCH xf86-video-ati] Use local implementation of RegionDuplicate for older xserver

2016-09-20 Thread Michel Dänzer
From: Michel Dänzer It was only added in xserver 1.15. Fixes build against older xserver. Reported-by: Pali Rohár Signed-off-by: Michel Dänzer --- configure.ac | 6 ++ src/radeon.h | 19 +++ 2 files changed, 25 insertions(+) diff --git a/configure.ac b/configure.ac in

[PATCH] Add drmModeAddFB2WithModifiers() which takes format modifiers

2016-09-20 Thread Alexandre Courbot
On 09/14/2016 07:07 AM, Kristian H. Kristensen wrote: > The only current user of this open codes the ioctl. Let's add an entry > point for this to libdrm. Tested-by: Alexandre Courbot Replaced one custom (staging) ioctl on Tegra with FB modifiers, noticed this was missing! > > Signed-off-by: K

[Bug 97840] [regression] [tonga] Freeze since new memory manager enabled

2016-09-20 Thread bugzilla-dae...@freedesktop.org
ng this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/16594dcc/attachment-0001.html>

[Bug 97840] [regression] [tonga] Freeze since new memory manager enabled

2016-09-20 Thread bugzilla-dae...@freedesktop.org
the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/dada5c3f/attachment-0001.html>

[Bug 97840] [regression] [tonga] Freeze since new memory manager enabled

2016-09-20 Thread bugzilla-dae...@freedesktop.org
e bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/c35eac93/attachment-0001.html>

[Bug 97840] [regression] [tonga] Freeze since new memory manager enabled

2016-09-20 Thread bugzilla-dae...@freedesktop.org
. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/70d06f45/attachment-0001.html>

[PATCH v3 11/20] drm: omapdrm: Check DSS manager state in the enable/disable helpers

2016-09-20 Thread Tomi Valkeinen
then I think the driver should make sure that omap_crtc_set_enabled() is not called if the output is already enabled/disabled. Maybe that can be done in omap_crtc_dss_enable/disable, using the new enabled flag. Tomi -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/2974e9b7/attachment-0001.sig>

[PATCH v3 10/20] drm: omapdrm: Only commit planes on active CRTCs

2016-09-20 Thread Tomi Valkeinen
right? Tomi -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/fb515fbd/attachment-0001.sig>

[PATCH][V2] drm/amdgpu: avoid out of bounds access on array interrupt_status_offsets

2016-09-20 Thread Colin King
From: Colin Ian King The check for an out of bound index into array interrupt_status_offsets is off-by-one. Fix this and also don't compared to a hard coded array size but use adev->mode_info.num_hpd instead. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 2 +- 1 fil

[PATCH v3 09/20] drm: omapdrm: Replace DSS manager state check with omapdrm CRTC state

2016-09-20 Thread Tomi Valkeinen
from the encoder enable/disable and suspend/resume handlers. */ which doesn't seem to hold true... Tomi -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/473f9457/attachment-0001.sig>

[PATCH] drm/amdgpu: avoid out of bounds access on array interrupt_status_offsets

2016-09-20 Thread Colin Ian King
On 20/09/16 16:39, Alex Deucher wrote: > On Tue, Sep 20, 2016 at 11:16 AM, Colin King > wrote: >> From: Colin Ian King >> >> The check for an out of bound index into array interrupt_status_offsets >> is off-by-one. Fix this and also don't compared to a hard coded array >> size but use ARRAY_SIZE

[PATCH v3 08/20] drm: omapdrm: Handle OCP error IRQ directly

2016-09-20 Thread Tomi Valkeinen
omap_irq_register(dev, error_handler); > - > dev->irq_enabled = true; > > return 0; > The removal of DISPC_IRQ_SYNC_LOST_DIGIT code above feels a bit out of place for this patch. Tomi -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/0156bcb1/attachment-0001.sig>

[RFC 2/2] drm/exynos: mixer: configure layers once in mixer_atomic_flush()

2016-09-20 Thread Andrzej Hajda
On 20.09.2016 16:09, Tobias Jakobi wrote: > Hi Andrzej, > > > Andrzej Hajda wrote: >> On 20.09.2016 14:34, Andrzej Hajda wrote: >>> On 20.09.2016 13:23, Tobias Jakobi wrote: Hello Andrzej, first of all, I've noticed an error myself. mixer_disable() calls mixer_disable_plane(), s

[PATCH v3 06/20] drm: omapdrm: Handle FIFO underflow IRQs internally

2016-09-20 Thread Tomi Valkeinen
pc, and have a similar func to dispc_mgr_get_sync_lost_irq(). Tomi -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/7411a729/attachment-0001.sig>

[PATCH v3 06/20] drm: omapdrm: Handle FIFO underflow IRQs internally

2016-09-20 Thread Tomi Valkeinen
e: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/0d488b54/attachment-0001.sig>

[PATCH] drm/amdgpu: avoid out of bounds access on array interrupt_status_offsets

2016-09-20 Thread Colin King
From: Colin Ian King The check for an out of bound index into array interrupt_status_offsets is off-by-one. Fix this and also don't compared to a hard coded array size but use ARRAY_SIZE instead. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 2 +- 1 file changed, 1

[RFC 2/2] drm/exynos: mixer: configure layers once in mixer_atomic_flush()

2016-09-20 Thread Tobias Jakobi
Hi Andrzej, Andrzej Hajda wrote: > On 20.09.2016 14:34, Andrzej Hajda wrote: >> On 20.09.2016 13:23, Tobias Jakobi wrote: >>> Hello Andrzej, >>> >>> first of all, I've noticed an error myself. mixer_disable() calls >>> mixer_disable_plane(), so it should also be modified. I'll send a v2 later. >>

[PATCH v3 04/20] drm: omapdrm: fb: Simplify mode command checks when creating framebuffer

2016-09-20 Thread Tomi Valkeinen
text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/cf4f7ff1/attachment-0001.sig>

[Bug 97806] GPU lockup with mesa-git and llvm-svn with rx 470 on Unigine Heaven and TombRaider 2013

2016-09-20 Thread bugzilla-dae...@freedesktop.org
-- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/4ccad3e7/attachment-0001.html>

[PATCH v3 02/20] drm: omapdrm: fb: Use format information provided by the DRM core

2016-09-20 Thread Tomi Valkeinen
eneric (and not specific to a UV plane). Tomi -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/3b233bda/attachment-0001.sig>

[PATCH 6/6] GPU-DRM-GMA500: Rename a jump label in mid_get_vbt_data()

2016-09-20 Thread Dan Carpenter
On Tue, Sep 20, 2016 at 01:08:12PM +0300, Jani Nikula wrote: > On Tue, 20 Sep 2016, SF Markus Elfring > wrote: > > From: Markus Elfring > > Date: Tue, 20 Sep 2016 10:40:22 +0200 > > > > * Adjust a jump target. > > Please don't. > Also there is nothing in CodingStyle that prohibits out: labels

[PULL] sti-drm-next-2016-09-20

2016-09-20 Thread Vincent ABRIOU
Hi Dave, Here are some patches for drm-next. It contains: - minor fixes for typo and warning. - sparse and coccicheck warning fixes - bunch of patches fixing issues found while testing drm/sti with an atomic version of weston - the removal of the support of stih415-416 sti platform Regards, Vinc

Kernel stability on baytrail machines

2016-09-20 Thread Michal Feix
Hi, I think there might be another clue on this one. One of the comments is also mentioning an unfixed erratum of certain Baytrail processors, named as "EOI Transaction May Not be Sent if Software Enters Core C6 During an Interrupt Service Routine". This erratum can be found on several differen

[PATCH 5/6] GPU-DRM-GMA500: One error message less for a GCT revision mismatch in mid_get_vbt_data()

2016-09-20 Thread Jani Nikula
On Tue, 20 Sep 2016, Dan Carpenter wrote: > Don't be a dummy... This is easy to review an it fixes a bug. > > I'm fine with you NAKing all these patches based on who they are from. > I mostly just delete these without responding because the guy has > history of introducing bugs and never listens

[RFC 2/2] drm/exynos: mixer: configure layers once in mixer_atomic_flush()

2016-09-20 Thread Andrzej Hajda
On 20.09.2016 14:34, Andrzej Hajda wrote: > On 20.09.2016 13:23, Tobias Jakobi wrote: >> Hello Andrzej, >> >> first of all, I've noticed an error myself. mixer_disable() calls >> mixer_disable_plane(), so it should also be modified. I'll send a v2 later. >> >> Now to your points... >> >> >> Andrzej

[PATCH v2] drm/sti: remove stih415-416 platform support

2016-09-20 Thread Benjamin Gaignard
Acked-by: Benjamin Gaignard 2016-09-20 11:25 GMT+02:00 Vincent Abriou : > stih415 and stih416 platform are obsolete and no more supported. > Only stih407 and stih410 platform are maintained. > > Signed-off-by: Vincent Abriou > --- > drivers/gpu/drm/sti/Kconfig| 6 +- > drivers

[RFC 2/2] drm/exynos: mixer: configure layers once in mixer_atomic_flush()

2016-09-20 Thread Andrzej Hajda
On 20.09.2016 13:23, Tobias Jakobi wrote: > Hello Andrzej, > > first of all, I've noticed an error myself. mixer_disable() calls > mixer_disable_plane(), so it should also be modified. I'll send a v2 later. > > Now to your points... > > > Andrzej Hajda wrote: >> On 19.09.2016 16:16, Tobias Jakobi w

GPU-DRM-GMA500: One error message less for a GCT revision mismatch in mid_get_vbt_data()

2016-09-20 Thread Dan Carpenter
On Tue, Sep 20, 2016 at 01:03:06PM +0200, SF Markus Elfring wrote: > Are you looking for any special information to improve > a corresponding discussion? If you restricted yourself to only sending bug fixes and not sending any more cleanups that would be good. Please stop sending clean up patches

[STLinux Kernel] [PATCH v2] drm/sti: remove stih415-416 platform support

2016-09-20 Thread Peter Griffin
Hi Vincent, On Tue, 20 Sep 2016, Vincent Abriou wrote: > stih415 and stih416 platform are obsolete and no more supported. > Only stih407 and stih410 platform are maintained. > > Signed-off-by: Vincent Abriou Subject should have a v2, but apart from that: - Acked-by: Peter Griffin Peter.

[PATCH 5/6] GPU-DRM-GMA500: One error message less for a GCT revision mismatch in mid_get_vbt_data()

2016-09-20 Thread Dan Carpenter
On Tue, Sep 20, 2016 at 01:07:35PM +0300, Jani Nikula wrote: > On Tue, 20 Sep 2016, SF Markus Elfring > wrote: > > From: Markus Elfring > > Date: Tue, 20 Sep 2016 10:36:19 +0200 > > > > A single error message should be sufficient to inform about > > the detection of an unknown GCT revision at th

[PATCH 04/14] drm/arm: Fix sparse warnings

2016-09-20 Thread Liviu Dudau
On Tue, Sep 20, 2016 at 09:06:24AM +0100, Liviu Dudau wrote: > On Mon, Sep 19, 2016 at 04:33:45PM +0300, ville.syrjala at linux.intel.com > wrote: > > From: Ville Syrjälä > > > > drm/arm/malidp_planes.c:49:24: warning: symbol > > 'malidp_duplicate_plane_state' was not declared. Should it be s

GPU-DRM-GMA500: One error message less for a GCT revision mismatch in mid_get_vbt_data()

2016-09-20 Thread SF Markus Elfring
> If you restricted yourself to only sending bug fixes and not sending > any more cleanups that would be good. Thanks for another bit of constructive feedback. > Please stop sending clean up patches. This will not happen for a while. I am in the process of informing various developers about so

[RFC 2/2] drm/exynos: mixer: configure layers once in mixer_atomic_flush()

2016-09-20 Thread Tobias Jakobi
Hello Andrzej, first of all, I've noticed an error myself. mixer_disable() calls mixer_disable_plane(), so it should also be modified. I'll send a v2 later. Now to your points... Andrzej Hajda wrote: > On 19.09.2016 16:16, Tobias Jakobi wrote: >> Only manipulate the MXR_CFG and MXR_LAYER_CFG re

[PATCH] dma-buf/sw_sync: mark sync_timeline_create() static

2016-09-20 Thread Gustavo Padovan
2016-09-18 Baoyou Xie : > We get 1 warning when building kernel with W=1: > drivers/dma-buf/sw_sync.c:87:23: warning: no previous prototype for > 'sync_timeline_create' [-Wmissing-prototypes] > > In fact, this function is only used in the file in which it is > declared and don't need a declarati

[PATCH 6/6] GPU-DRM-GMA500: Rename a jump label in mid_get_vbt_data()

2016-09-20 Thread Jani Nikula
On Tue, 20 Sep 2016, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 20 Sep 2016 10:40:22 +0200 > > * Adjust a jump target. Please don't. BR, Jani. > > * Delete the explicit initialisation for the local variable "ret" > which became unnecessary with this refactoring. > > Signed-

[PATCH 5/6] GPU-DRM-GMA500: One error message less for a GCT revision mismatch in mid_get_vbt_data()

2016-09-20 Thread Jani Nikula
On Tue, 20 Sep 2016, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 20 Sep 2016 10:36:19 +0200 > > A single error message should be sufficient to inform about > the detection of an unknown GCT revision at the end. > Thus return after the logging call in this case directly. Did you

[PATCH 1/6] GPU-DRM-GMA500: Use kmalloc_array() in mid_get_vbt_data_r10()

2016-09-20 Thread Jani Nikula
On Tue, 20 Sep 2016, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 20 Sep 2016 08:54:07 +0200 > > A multiplication for the size determination of a memory allocation > indicated that an array data structure should be processed. > Thus use the corresponding function "kmalloc_array".

[PATCH 2/6] GPU-DRM-GMA500: Rename a jump label in mid_get_vbt_data_r10()

2016-09-20 Thread Jani Nikula
On Tue, 20 Sep 2016, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 20 Sep 2016 09:09:10 +0200 > > Adjust a jump label according to the current Linux coding style convention. Generally, please don't send patches to fix checkpatch issues in existing code. Moreover, this particular "

GPU-DRM-GMA500: One error message less for a GCT revision mismatch in mid_get_vbt_data()

2016-09-20 Thread SF Markus Elfring
>>> A single error message should be sufficient to inform about >>> the detection of an unknown GCT revision at the end. >>> Thus return after the logging call in this case directly. >> >> Did you test this? >> > > Don't be a dummy... This is easy to review an it fixes a bug. Thanks for this kin

[PATCH RESEND] drm/ast: free correct pointer in astfb_create() error paths

2016-09-20 Thread Andrew Donnellan
On 20/09/16 11:56, Andrew Donnellan wrote: > In the err_free_vram and err_release_fbi error paths in astfb_create(), we > attempt to free afbdev->sysram. The only jumps to these error paths occur > before we assign afbdev->sysram = sysram. Free sysram instead. > > Signed-off-by: Andrew Donnellan >

[PATCH v9 17/19] drm/virtio: kconfig: Fix recursive dependency issue.

2016-09-20 Thread Jani Nikula
On Tue, 20 Sep 2016, Peter Griffin wrote: > Hi Emil, > > On Tue, 20 Sep 2016, Emil Velikov wrote: > >> On 5 September 2016 at 14:16, Peter Griffin >> wrote: >> > ST_SLIM_REMOTEPROC must select REMOTEPROC, which exposes the following >> > recursive dependency. > > >> > >> From a humble skim throu

GPU-DRM-GMA500: One error message less for a GCT revision mismatch in mid_get_vbt_data()

2016-09-20 Thread SF Markus Elfring
>> A single error message should be sufficient to inform about >> the detection of an unknown GCT revision at the end. >> Thus return after the logging call in this case directly. > > Did you test this? What is your software development opinion for this update suggestion? Regards, Markus

GPU-DRM-GMA500: Use kmalloc_array() in mid_get_vbt_data_r10()

2016-09-20 Thread SF Markus Elfring
>> A multiplication for the size determination of a memory allocation >> indicated that an array data structure should be processed. >> Thus use the corresponding function "kmalloc_array". >> >> This issue was detected by using the Coccinelle software. > > Did you test this running on the hardware

[Bug 93456] Intel D845GVAD2 ubuntu 15.10 GPU Hangs - stuck on render ring

2016-09-20 Thread bugzilla-dae...@freedesktop.org
44fc000:mono pattern dw1 0x0158: 0x: MI_NOOP -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/c820904e/

[PATCH] drm/omap: fix return value check in dsi_bind()

2016-09-20 Thread Peter Ujfalusi
On 09/17/16 18:53, Wei Yongjun wrote: > From: Wei Yongjun > > Fix the retrn value check which testing the wrong variable > in dsi_bind(). Reviewed-by: Peter Ujfalusi > > Signed-off-by: Wei Yongjun > --- > drivers/gpu/drm/omapdrm/dss/dsi.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 delet

Kernel error causing display issues.

2016-09-20 Thread Jani Nikula
On Tue, 20 Sep 2016, Edward Bartolo wrote: > I am experiencing display issues on HP Probook 4540s ( i5 based > laptop). Reading /var/log/kern.log I found the cause is probably > rooted in: > > [drm] Wrong MCH_SSKPD value: 0x16040307 Maybe, maybe not. Please file a bug at [1], attaching the dmesg

[RFC 2/2] drm/exynos: mixer: configure layers once in mixer_atomic_flush()

2016-09-20 Thread Andrzej Hajda
On 19.09.2016 16:16, Tobias Jakobi wrote: > Only manipulate the MXR_CFG and MXR_LAYER_CFG registers once > in mixer_cfg_layer(). > Trigger this via atomic flush. > > Signed-off-by: Tobias Jakobi > --- > drivers/gpu/drm/exynos/exynos_mixer.c | 104 > ++ > drivers/g

[PATCH 4/6] drm/radeon: Provide page_flip_target hook

2016-09-20 Thread Michel Dänzer
On 17/09/16 09:41 PM, Mario Kleiner wrote: > Hi Michel, > > all your patches, both the already merged kernel bits in radeon/amdgpu > and also all the userspace bits in libdrm/ati-ddx/amdgpu-ddx are all > > Reviewed-and-tested-by: Mario Kleiner > > I successfully tested with old/current userspac

[PATCH 1/2] drm/radeon: Slightly more robust flip completion handling for < DCE-4

2016-09-20 Thread Michel Dänzer
On 17/09/16 09:25 PM, Mario Kleiner wrote: > Pre DCE4 hardware doesn't have (reliable) pageflip completion > irqs, therefore we have to use the old polling method for flip > completion handling in vblank irq. > > As vblank irqs fire a bit before start of vblank (when the > linebuffer fifo read pos

[PATCH RESEND] drm/ast: free correct pointer in astfb_create() error paths

2016-09-20 Thread Andrew Donnellan
In the err_free_vram and err_release_fbi error paths in astfb_create(), we attempt to free afbdev->sysram. The only jumps to these error paths occur before we assign afbdev->sysram = sysram. Free sysram instead. Signed-off-by: Andrew Donnellan --- Found by Coverity Scan. Compile tested only. R

[PATCH 2/2] drm/radeon: Prevent races on pre DCE4 between flip submission and completion.

2016-09-20 Thread Michel Dänzer
On 17/09/16 09:25 PM, Mario Kleiner wrote: > Pre DCE4 hw doesn't have reliable pageflip completion > interrupts, so instead polling for flip completion is > used from within the vblank irq handler to complete > page flips. > > This causes a race if pageflip ioctl is called close to > vblank: > >

[PATCH][V2] drm/amdgpu: avoid out of bounds access on array interrupt_status_offsets

2016-09-20 Thread Alex Deucher
On Tue, Sep 20, 2016 at 11:49 AM, Colin King wrote: > From: Colin Ian King > > The check for an out of bound index into array interrupt_status_offsets > is off-by-one. Fix this and also don't compared to a hard coded array > size but use adev->mode_info.num_hpd instead. > > Signed-off-by: Colin

[PATCH] drm/amdgpu: avoid out of bounds access on array interrupt_status_offsets

2016-09-20 Thread Alex Deucher
On Tue, Sep 20, 2016 at 11:16 AM, Colin King wrote: > From: Colin Ian King > > The check for an out of bound index into array interrupt_status_offsets > is off-by-one. Fix this and also don't compared to a hard coded array > size but use ARRAY_SIZE instead. > > Signed-off-by: Colin Ian King > -

[PATCH v2] drm/sti: remove stih415-416 platform support

2016-09-20 Thread Vincent Abriou
stih415 and stih416 platform are obsolete and no more supported. Only stih407 and stih410 platform are maintained. Signed-off-by: Vincent Abriou --- drivers/gpu/drm/sti/Kconfig| 6 +- drivers/gpu/drm/sti/Makefile | 1 - drivers/gpu/drm/sti/sti_compositor.c

drm: NULL pointer dereference in drm_mode_object_find()

2016-09-20 Thread Alexander Potapenko
On Tue, Sep 20, 2016 at 11:21 AM, David Herrmann wrote: > Hi > > On Mon, Sep 5, 2016 at 10:30 AM, Dmitry Vyukov wrote: >> On Fri, Aug 19, 2016 at 7:10 PM, Alexander Potapenko >> wrote: >>> Hello, >>> >>> the program below triggers a NULL deref in DRM code when ran on QEMU: >>> >>>

drm: NULL pointer dereference in drm_mode_object_find()

2016-09-20 Thread David Herrmann
Hi On Mon, Sep 5, 2016 at 10:30 AM, Dmitry Vyukov wrote: > On Fri, Aug 19, 2016 at 7:10 PM, Alexander Potapenko > wrote: >> Hello, >> >> the program below triggers a NULL deref in DRM code when ran on QEMU: >> >> === >> BUG: unable to handle kerne

[Intel-gfx] Skylake graphics regression: projector failure with 4.8-rc3

2016-09-20 Thread Jani Nikula
On Mon, 19 Sep 2016, James Bottomley wrote: > On Mon, 2016-09-19 at 08:09 -0700, James Bottomley wrote: >> On Sun, 2016-09-18 at 13:35 +0200, Thorsten Leemhuis wrote: >> > Hi! James & Paulo: What's the current status of this? >> >> No, the only interaction has been the suggestion below for a rev

[PATCH 6/6] GPU-DRM-GMA500: Rename a jump label in mid_get_vbt_data()

2016-09-20 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 20 Sep 2016 10:40:22 +0200 * Adjust a jump target. * Delete the explicit initialisation for the local variable "ret" which became unnecessary with this refactoring. Signed-off-by: Markus Elfring --- drivers/gpu/drm/gma500/mid_bios.c | 11 +-- 1 file c

[PATCH 5/6] GPU-DRM-GMA500: One error message less for a GCT revision mismatch in mid_get_vbt_data()

2016-09-20 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 20 Sep 2016 10:36:19 +0200 A single error message should be sufficient to inform about the detection of an unknown GCT revision at the end. Thus return after the logging call in this case directly. Signed-off-by: Markus Elfring --- drivers/gpu/drm/gma500/mid_bio

[PATCH 4/6] GPU-DRM-GMA500: Fix indentation for a function call parameter in mid_get_vbt_data_r10()

2016-09-20 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 20 Sep 2016 10:34:28 +0200 Adjust the indentation for a single function call parameter here. Signed-off-by: Markus Elfring --- drivers/gpu/drm/gma500/mid_bios.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/gma500/mid_bios.c

[PATCH 3/6] GPU-DRM-GMA500: Move a variable assignment in mid_get_vbt_data_r10()

2016-09-20 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 20 Sep 2016 10:32:12 +0200 One local variable was set to an error code before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there. Signed-off-by: Markus Elfring --- drivers/gpu/dr

[PATCH 2/6] GPU-DRM-GMA500: Rename a jump label in mid_get_vbt_data_r10()

2016-09-20 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 20 Sep 2016 09:09:10 +0200 Adjust a jump label according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/gpu/drm/gma500/mid_bios.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/gma

[PATCH 1/6] GPU-DRM-GMA500: Use kmalloc_array() in mid_get_vbt_data_r10()

2016-09-20 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 20 Sep 2016 08:54:07 +0200 A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by using the Coccinelle software.

[PATCH 0/6] GPU-DRM-GMA500: Fine-tuning for two function implementations

2016-09-20 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 20 Sep 2016 10:48:04 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (6): Use kmalloc_array() in mid_get_vbt_data_r10() Rename a jump label in mid_get_vbt_data_r10() Move a variable assignment in mid_ge

[RFC 1/2] drm/exynos: mixer: convert booleans to flags in mixer context

2016-09-20 Thread Andrzej Hajda
On 19.09.2016 15:53, Tobias Jakobi wrote: > The mixer context struct already has a 'flags' field, so > we can use it to store the 'interlace', 'vp_enabled' and > 'has_sclk' booleans. > We use the non-atomic helper functions to access these bits. > > Signed-off-by: Tobias Jakobi Reviewed-by: Andrz

[PATCH] drm/exynos: mark exynos_dp_crtc_clock_enable() static

2016-09-20 Thread Andrzej Hajda
On 18.09.2016 15:58, Baoyou Xie wrote: > We get 1 warning when building kernel with W=1: > drivers/gpu/drm/exynos/exynos_dp.c:46:5: warning: no previous prototype for > 'exynos_dp_crtc_clock_enable' [-Wmissing-prototypes] > > In fact, this function is only used in the file in which it is > declare

Kernel error causing display issues.

2016-09-20 Thread Edward Bartolo
Hi, I am experiencing display issues on HP Probook 4540s ( i5 based laptop). Reading /var/log/kern.log I found the cause is probably rooted in: [drm] Wrong MCH_SSKPD value: 0x16040307 Just after the kernel takes control of the machine, I cannot read the boot text. Successive pixel lines are shif

[PATCH v3 3/3] drm/bridge: analogix_dp: detect Sink PSR state after configuring the PSR

2016-09-20 Thread Yakir Yang
Hi Sean, On 09/12/2016 09:52 PM, Sean Paul wrote: > On Fri, Sep 9, 2016 at 5:45 AM, Yakir Yang wrote: >> Make sure the request PSR state takes effect in analogix_dp_send_psr_spd() >> function, or print the sink PSR error state if we failed to apply the >> requested PSR setting. >> >> Signed-off-b

[PATCH v3 2/3] drm/bridge: analogix_dp: use jiffies to simulate timeout loop

2016-09-20 Thread Yakir Yang
Hi Sean, On 09/12/2016 09:51 PM, Sean Paul wrote: > On Fri, Sep 9, 2016 at 5:44 AM, Yakir Yang wrote: >> Signed-off-by: Yakir Yang >> --- >> Changes in v3: >> - Suggested by Sean >> >> Changes in v2: None >> >> drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 3 ++- >> drivers/gpu/drm/b

[PATCH] drm/sti: remove stih415-416 platform support

2016-09-20 Thread Vincent Abriou
stih415 and stih416 platform are obsolete and no more supported. Only stih407 and stih410 platform are maintained. Signed-off-by: Vincent Abriou --- drivers/gpu/drm/sti/Kconfig| 6 +- drivers/gpu/drm/sti/sti_compositor.c | 19 -- drivers/gpu/drm/sti/sti_hda.c

[PATCH 02/14] drm/blend: Fix sparse warnings

2016-09-20 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Monday 19 Sep 2016 16:33:43 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > drm/drm_blend.c:207:5: warning: symbol 'drm_atomic_normalize_zpos' was not > declared. Should it be static? > > Cc: Marek Szyprowski > Cc: Benjamin Gaignard >

[PATCH 2/8] drm/rockchip: Get rid of some unnecessary code

2016-09-20 Thread Mark yao
On 2016年09月18日 12:01, Tomasz Figa wrote: > Hi Mark, > > On Sun, Sep 18, 2016 at 10:50 AM, Mark yao wrote: >> On 2016年09月14日 20:54, Tomasz Figa wrote: >>> Current code implements prepare_fb and cleanup_fb callbacks only to >>> grab/release fb references, which is already done by atomic

[PATCH v9 17/19] drm/virtio: kconfig: Fix recursive dependency issue.

2016-09-20 Thread Peter Griffin
Hi Emil, On Tue, 20 Sep 2016, Emil Velikov wrote: > On 5 September 2016 at 14:16, Peter Griffin > wrote: > > ST_SLIM_REMOTEPROC must select REMOTEPROC, which exposes the following > > recursive dependency. > > > From a humble skim through remoteproc, drm and a few other subsystems > I think t

[PATCH 25/38] drm: Add reference counting to drm_atomic_state

2016-09-20 Thread Chris Wilson
drm_atomic_state has a complicated single owner model that tracks the single reference from allocation through to destruction on another thread - or perhaps on a local error path. We can simplify this tracking by using reference counting (at a cost of a few more atomics). This is even more benefici

[PATCH 4/4] drm/sun4i: dotclock: Round to closest clock rate

2016-09-20 Thread Maxime Ripard
her rate than the ideal rate. > > Including the divider makes it compare the actual dot clock frequency > if a given divider was used. > > Does this makes sense? Explaining this kind of makes my head spin... Yes, sorry, I didn't remember rounded was actually the rounded parent

[PATCH 4/4] drm/sun4i: dotclock: Round to closest clock rate

2016-09-20 Thread Maxime Ripard
s Embedded Linux and Kernel engineering http://free-electrons.com -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/201

[PATCH 04/14] drm/arm: Fix sparse warnings

2016-09-20 Thread Liviu Dudau
On Mon, Sep 19, 2016 at 04:33:45PM +0300, ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > drm/arm/malidp_planes.c:49:24: warning: symbol 'malidp_duplicate_plane_state' > was not declared. Should it be static? > drm/arm/malidp_planes.c:66:6: warning: symbol 'malidp_destroy_pl

[PATCH 5/5] drm/imx: add exclusive fence to plane state

2016-09-20 Thread Lucas Stach
Am Montag, den 19.09.2016, 22:48 +0200 schrieb Daniel Vetter: > On Thu, Aug 11, 2016 at 11:18 AM, Lucas Stach > wrote: > > This allows the atomic helper to wait on them, instead of open-coding > > the same in the imx-drm driver. > > > > Signed-off-by: Lucas Stach > > --- > > drivers/gpu/drm/imx

[PATCH v9 17/19] drm/virtio: kconfig: Fix recursive dependency issue.

2016-09-20 Thread Emil Velikov
sktop.org/archives/dri-devel/attachments/20160920/70146c43/attachment.html>

[PATCH v9 17/19] drm/virtio: kconfig: Fix recursive dependency issue.

2016-09-20 Thread Emil Velikov
-- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/65ca951e/attachment.html>

[Bug 68571] GPU lockup on AMD Radeon HD6850 with DPM=1

2016-09-20 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=68571 --- Comment #83 from Michel Dänzer --- (In reply to Weber K. from comment #81) > And since I put this in grub.cfg and fstab, never had a black screen again: > rootflags=relatime,lazytime,commit=60 iommu=noagp radeon.dpm=1 It could be iommu=noa

[Bug 97840] [regression] [tonga] Freeze since new memory manager enabled

2016-09-20 Thread bugzilla-dae...@freedesktop.org
. URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/7a070355/attachment.html>

[Bug 97840] [regression] [tonga] Freeze since new memory manager enabled

2016-09-20 Thread bugzilla-dae...@freedesktop.org
ause: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/6da7c42c/attachment.html>

[Bug 97840] [regression] [tonga] Freeze since new memory manager enabled

2016-09-20 Thread bugzilla-dae...@freedesktop.org
ceiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/94274066/attachment.html>

[Bug 68571] GPU lockup on AMD Radeon HD6850 with DPM=1

2016-09-20 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=68571 --- Comment #82 from Weber K. --- Ooops... Forgot to mention... I appreciate your good work supporting kernel! Thank all of you very much! -- You are receiving this mail because: You are watching the assignee of the bug.

[Bug 68571] GPU lockup on AMD Radeon HD6850 with DPM=1

2016-09-20 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=68571 --- Comment #81 from Weber K. --- I think so as well... Anyway before, I had to put this in kernel parameters, or else, black screen and hang up during boot: radeon.dpm=0 And since I put this in grub.cfg and fstab, never had a black screen aga

[Bug 97840] [regression] [tonga] Freeze since new memory manager enabled

2016-09-20 Thread bugzilla-dae...@freedesktop.org
. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/803654ff/attachment.html>

[RFC PATCH v3 1/2] ARM: dts: samsung: add rga-lvds panel in itop elite

2016-09-20 Thread ayaka
On 09/20/2016 03:12 AM, Fabio Estevam wrote: > On Mon, Sep 19, 2016 at 4:02 PM, Randy Li wrote: > >> + vcc_sys_lcd: sys-lcd { >> + compatible = "regulator-fixed"; >> + regulator-name = "vcc_5v"; >> + regulator-min-microvolt = <500>; >> +

[Bug 68571] GPU lockup on AMD Radeon HD6850 with DPM=1

2016-09-20 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=68571 --- Comment #80 from Alex Deucher --- dpm has nothing to do with the filesystem. -- You are receiving this mail because: You are watching the assignee of the bug.

[RFC PATCH v3 2/2] drm/panel: Add support for Chunghwa CLAA070WP03XG panel

2016-09-20 Thread Randy Li
The Chunghwa CLAA070WP03XG is a 7" 1280x800 panel, which can be supported by the simple panel driver. Signed-off-by: Randy Li --- .../display/panel/chunghwa,claa070wp03xg.txt | 7 ++ drivers/gpu/drm/panel/panel-simple.c | 27 ++ 2 files changed, 34 in

[RFC PATCH v3 1/2] ARM: dts: samsung: add rga-lvds panel in itop elite

2016-09-20 Thread Randy Li
It is actually a lvds panel connected through a rga-lvds bridge. The touchscreen is communicated with i2c bus but the driver is not support now. Signed-off-by: Randy Li --- arch/arm/boot/dts/exynos4412-itop-elite.dts | 54 +++-- 1 file changed, 52 insertions(+), 2 deletio

  1   2   >