Re: [PATCH 8/8] drm: todo: remove task about switch to drm_connector_list_iter

2017-05-26 Thread Gustavo Padovan
2017-05-11 Gustavo Padovan : > From: Gustavo Padovan > > This is now completed. > > Signed-off-by: Gustavo Padovan > --- > Documentation/gpu/todo.rst | 13 - > 1 file changed, 13 deletions(-) > >

[Bug 100990] Not running a lot of games on the graphics card Radeon R9 290

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100990 --- Comment #3 from Nemial --- Is there any progress in solving this bug? -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list

Re: iMac 10,1 with Ubuntu 16.04: black screen after suspend

2017-05-26 Thread Lukas Wunner
[cc += dri-devel, amd-gfx] On Fri, May 26, 2017 at 02:13:29PM +0200, Florian Echtler wrote: > I'm running Ubuntu 16.04.2 on a 27" unibody iMac 10,1 from 2009. However, even > with the most recent HWE stack (kernel 4.8), the display stays black after > suspend. I can ssh into the machine, so

[PULL] drm-misc-next

2017-05-26 Thread Sean Paul
Hi Dave, Here's another misc-next pull for you. We have some nice improvements in core adding mode_valid hooks and de-duping the allocation code. Daniel continues to improve documentation (\o/), and a bunch of little stuff was fixed up. Also noteworthy is that I backmerged -next so Daniel could

[Bug 99784] AMD 7470m When VGA plugged into laptop, only one monitor working at a time

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99784 --- Comment #16 from lazane...@gmail.com --- Created attachment 131532 --> https://bugs.freedesktop.org/attachment.cgi?id=131532=edit "dmesg | grep drm" output (4.12.rc2, libdrm 2.4.8) I updated libdrm and xorg from edgers ppa, and drm-tip

[Bug 99349] Failed to build shader (translation from TGSI)

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99349 --- Comment #10 from Gert Wollny --- Well, it turns out that the shader simply uses too many registers, and since this is only tested at the end, at one point the indices of the temporaries used to store constants are

[Bug 101189] Latest git fails to compile with radeon

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101189 --- Comment #7 from Pavel Vinogradov --- Sorry, I meant the amdgpu kms library in drm itself. (In reply to Alex Deucher from comment #6) > (In reply to Pavel Vinogradov from comment #5) > > hm, strange. do I need to

[Bug 93652] Random crashes/freezing with amdgpu Fury X mesa 11.1

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=93652 --- Comment #20 from Marek Olšák --- I've never seen any hangs with Fiji (Fury) and it's my main development card. There is a VBIOS switch on the side of the card. When the computer is powered off, flip the switch to the other

[Bug 101189] Latest git fails to compile with radeon

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101189 --- Comment #6 from Alex Deucher --- (In reply to Pavel Vinogradov from comment #5) > hm, strange. do I need to build amdgpu kms as well? for simple radeon hd > 4650? You don't need the amdgpu kernel driver. -- You are

[Bug 101189] Latest git fails to compile with radeon

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101189 --- Comment #5 from Pavel Vinogradov --- hm, strange. do I need to build amdgpu kms as well? for simple radeon hd 4650? (In reply to Alex Deucher from comment #4) > (In reply to Pavel Vinogradov from comment #3) > > drm

[Bug 101189] Latest git fails to compile with radeon

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101189 --- Comment #4 from Alex Deucher --- (In reply to Pavel Vinogradov from comment #3) > drm from git doesn't provide amdgpu.h, but amdgpu_drm.h. It provides both: https://cgit.freedesktop.org/mesa/drm/tree/amdgpu/amdgpu.h

[Bug 101189] Latest git fails to compile with radeon

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101189 Pavel Vinogradov changed: What|Removed |Added Resolution|FIXED |---

[Bug 100593] corruption in total war warhammer when using mesa 17.1 - git

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100593 tarp...@gmx.de changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

Re: [PATCH 5/5] drm: dw-hdmi-i2s: add .get_dai_id callback for ALSA SoC

2017-05-26 Thread Mark Brown
On Wed, May 24, 2017 at 11:34:46PM +, Kuninori Morimoto wrote: > > As far as I understand what's going on with the graph code this seems to > > make sense to me. How do we want to go about handling the patch? > This is comment to me ? or DRM maintainer ? > If to me, any case (pickup by

[PATCH libdrm 1/1] drmsltest: Check expected neighbours

2017-05-26 Thread Jan Vesely
This test cements behaviour set in 3b2ee2b5bfc0d68525fee936e51297a9b6c629f1 drmSL: Fix neighbor lookup (2015-02-27) Signed-off-by: Jan Vesely --- tests/drmsl.c | 28 1 file changed, 20 insertions(+), 8 deletions(-) diff --git

[Bug 101189] Latest git fails to compile with radeon

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101189 Eric Engestrom changed: What|Removed |Added Status|NEW |RESOLVED

[Bug 101190] building gallium/R600 fails due to missing amdgpu.h

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101190 Eric Engestrom changed: What|Removed |Added Resolution|--- |DUPLICATE

[Bug 101189] Latest git fails to compile with radeon

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101189 Eric Engestrom changed: What|Removed |Added CC|

[Bug 194761] amdgpu driver breaks on Oland (SI)

2017-05-26 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=194761 --- Comment #24 from Alex Deucher (alexdeuc...@gmail.com) --- Created attachment 256727 --> https://bugzilla.kernel.org/attachment.cgi?id=256727=edit possible fix Does this patch help? -- You are receiving this mail because: You are watching

[Bug 100593] corruption in total war warhammer when using mesa 17.1 - git

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100593 --- Comment #10 from Marek Olšák --- (In reply to tarpoon from comment #9) > Oibaf still has problems on my R9 290. > > But with Padoka the issue is fixed now. However, Padoka is now installing > mesa 17.2 git, so I have no

[Bug 100919] [DC] "Out of range" on a display connected to a DVI-D output of RX460 card via DVI-D<=>HDMI-A cable

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100919 Alex Deucher changed: What|Removed |Added Status|NEW |RESOLVED

[Bug 101198] Major performance regression for AMD card

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101198 --- Comment #6 from leoxs...@gmail.com --- I can reproduce the problem with kernel that has no vcn patch, and the fix sent to mesa-dev list was tested and have issue fixed. Thanks, Leo -- You are receiving this mail because: You are the

[Bug 194761] amdgpu driver breaks on Oland (SI)

2017-05-26 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=194761 --- Comment #23 from Marek Olšák (mar...@gmail.com) --- Hi, (BTW I'm from AMD) First of all, if a kernel change breaks Mesa, the kernel change should be reverted. Kernel changes should never ever break existing userspace. There are released

[Bug 100919] [DC] "Out of range" on a display connected to a DVI-D output of RX460 card via DVI-D<=>HDMI-A cable

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100919 --- Comment #11 from Thomas R. --- Forgot to add a note: I'm not actually using HDMI or any converters, I had this problem with going directly DVI card <-> display. -- You are receiving this mail because:

[Bug 100919] [DC] "Out of range" on a display connected to a DVI-D output of RX460 card via DVI-D<=>HDMI-A cable

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100919 --- Comment #10 from Thomas R. --- Can confirm it works for me, too (although I accidentally also updated to 4.11 while testing it, but the relevant DC code looks unchanged and the patch applied, so I presume

Re: [PULL] drm-misc-fixes

2017-05-26 Thread Sean Paul
On Fri, May 26, 2017 at 10:40:15AM +0300, Jani Nikula wrote: > On Fri, 26 May 2017, Daniel Vetter wrote: > > On Thu, May 25, 2017 at 7:44 PM, Sean Paul wrote: > >> The pull is noisy > >> because it includes -rc2. > > > > dim has you covered for

Re: [PATCH] dim: Enforce review requirements

2017-05-26 Thread Sean Paul
On Wed, May 24, 2017 at 11:28:12AM +0200, Daniel Vetter wrote: > We can't check this when applying (since r-b/a-b tags often get added > afterwards), but we can check this when pushing. This only looks at > patches authored by the pusher. > > Also update the docs to highlight that review

Re: [PATCH] drm/rockchip: Don't allow zero sized gem buffer

2017-05-26 Thread Sean Paul
On Fri, May 26, 2017 at 10:30:09AM +0800, jeffy wrote: > Hi sean, > > On 05/25/2017 11:30 PM, Sean Paul wrote: > > On Tue, May 23, 2017 at 02:39:43PM +0800, Jeffy Chen wrote: > > > The system would crash when trying to alloc zero sized gem buffer: > > > [6.712435] Unable to handle kernel NULL

[Bug 99349] Failed to build shader (translation from TGSI)

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99349 --- Comment #9 from Gert Wollny --- It turns out that in r600_shader.c:tgsi_split_constant the constants should be moved to the GPR range, but for large shaders this is not sufficient, since the temporary registers used

Re: [PATCH v9 5/5] drm/i915: Set PWM divider to match desired frequency in vbt

2017-05-26 Thread Jani Nikula
On Tue, 23 May 2017, Puthikorn Voravootivat wrote: > Read desired PWM frequency from panel vbt and calculate the > value for divider in DPCD address 0x724 and 0x728 to have > as many bits as possible for PWM duty cyle for granularity of > brightness adjustment while the

[Bug 101198] Major performance regression for AMD card

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101198 Fabian Maurer changed: What|Removed |Added Status|NEW |RESOLVED

Re: [PATCH v9 4/5] drm: Add definition for eDP backlight frequency

2017-05-26 Thread Jani Nikula
On Wed, 24 May 2017, Jani Nikula wrote: > On Tue, 23 May 2017, Puthikorn Voravootivat wrote: >> This patch adds the following definition >> - Bit mask for EDP_PWMGEN_BIT_COUNT and min/max cap >> register which only use bit 0:4 >> - Base

Re: [PATCH v9 1/5] drm/i915: Drop AUX backlight enable check for backlight control

2017-05-26 Thread Jani Nikula
On Wed, 24 May 2017, Jani Nikula wrote: > On Tue, 23 May 2017, Puthikorn Voravootivat wrote: >> There are some panel that >> (1) does not support display backlight enable via AUX >> (2) support display backlight adjustment via AUX >> (3) support

[Bug 101198] Major performance regression for AMD card

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101198 --- Comment #4 from Dieter Nützel --- (In reply to Dieter Nützel from comment #2) > Hi Fabian, > > David reverted it and Leo Liu sent an update just in time: > Please try with this one: >

[Bug 101198] Major performance regression for AMD card

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101198 --- Comment #3 from Kai --- (In reply to Fabian Maurer from comment #0) > With the latest git mesa, [...] Can't be the latest, since the commit you've identified is already reverted:

[Bug 101198] Major performance regression for AMD card

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101198 --- Comment #2 from Dieter Nützel --- Hi Fabian, David reverted it and Leo Liu sent an update just in time: Please try with this one: https://lists.freedesktop.org/archives/mesa-dev/2017-May/156930.html Dicussion was

[Bug 101198] Major performance regression for AMD card

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101198 --- Comment #1 from Fabian Maurer --- Just noticed, this seems to be reverted already during the time I need to setup and bisect mesa. -- You are receiving this mail because: You are the assignee for the

Re: [PATCH] drm/tilcdc: Turn raster off in crtc reset, if it was on in the HW

2017-05-26 Thread Daniel Vetter
On Fri, May 26, 2017 at 12:56 PM, Jyri Sarha wrote: > The best possible crtc reset callback implementation would be reading > the video mode information from the registers and generating the new > crtc state based on that. But I have no idea what I should do with > framebuffer that

[Bug 101198] Major performance regression for AMD card

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101198 Fabian Maurer changed: What|Removed |Added Keywords||bisected,

[Bug 101198] Major performance regression for AMD card

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101198 Bug ID: 101198 Summary: Major performance regression for AMD card Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: All Status: NEW Severity:

Re: Maintaining small drm drivers

2017-05-26 Thread Daniel Vetter
On Fri, May 26, 2017 at 1:42 PM, Jani Nikula wrote: > On Fri, 26 May 2017, Daniel Vetter wrote: >> On Thu, May 25, 2017 at 1:09 AM, Patrik Jakobsson >> wrote: >>> Either way, I don't want to turn this into a long

Re: Maintaining small drm drivers

2017-05-26 Thread Jani Nikula
On Fri, 26 May 2017, Daniel Vetter wrote: > On Thu, May 25, 2017 at 1:09 AM, Patrik Jakobsson > wrote: >> Either way, I don't want to turn this into a long discussion. If this >> is the way it needs to work then that is fine by me. Most of the time

Re: [RFC PATCH 6/7] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2017-05-26 Thread Daniel Vetter
On Fri, May 26, 2017 at 12:34 PM, Hans Verkuil wrote: >>> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig >>> index 78d7fc0ebb57..dd771ce8a3d0 100644 >>> --- a/drivers/gpu/drm/Kconfig >>> +++ b/drivers/gpu/drm/Kconfig >>> @@ -120,6 +120,9 @@ config

Re: [Intel-gfx] i915 4.9 regression: DP AUX CH sanitization no longer working on Asus desktops

2017-05-26 Thread Jani Nikula
On Wed, 24 May 2017, Daniel Drake wrote: > On Fri, May 5, 2017 at 4:29 AM, Ville Syrjälä > wrote: >> On Thu, May 04, 2017 at 02:52:09PM -0600, Daniel Drake wrote: >>> On Thu, May 4, 2017 at 2:37 PM, Ville Syrjälä >>>

Re: [PATCH 36/37] drm/zte: Drop drm_vblank_cleanup

2017-05-26 Thread Shawn Guo
On Fri, May 26, 2017 at 08:57:32AM +0200, Daniel Vetter wrote: > On Thu, May 25, 2017 at 5:01 AM, Shawn Guo wrote: > > On Wed, May 24, 2017 at 04:52:11PM +0200, Daniel Vetter wrote: > >> It again looks all cargo-culted for no good reasons. > > > > drm_vblank_cleanup() is

[PATCH] drm/tilcdc: Turn raster off in crtc reset, if it was on in the HW

2017-05-26 Thread Jyri Sarha
This forces the HW to be in sync with the empty state. This should help with the problem when LCDC is already enabled (e.g. by the bootloader) at the initialization phase and the enable fails when a new mode is applied. Signed-off-by: Jyri Sarha --- There is two earlier patches

Re: [RFC PATCH 6/7] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2017-05-26 Thread Hans Verkuil
On 05/26/2017 09:18 AM, Daniel Vetter wrote: > On Thu, May 25, 2017 at 05:06:25PM +0200, Hans Verkuil wrote: >> From: Hans Verkuil >> >> This adds support for the DisplayPort CEC-Tunneling-over-AUX >> feature that is part of the DisplayPort 1.3 standard. >> >>

Re: [RFC PATCH 7/7] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-05-26 Thread Hans Verkuil
On 05/26/2017 09:15 AM, Daniel Vetter wrote: > On Thu, May 25, 2017 at 05:06:26PM +0200, Hans Verkuil wrote: >> From: Hans Verkuil >> >> Implement support for this DisplayPort feature. >> >> The cec device is created whenever it detects an adapter that >> has this feature.

Re: [RFC PATCH 7/7] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-05-26 Thread Hans Verkuil
On 05/26/2017 12:13 PM, Jani Nikula wrote: > On Thu, 25 May 2017, Hans Verkuil wrote: >> @@ -4179,6 +4181,33 @@ intel_dp_check_mst_status(struct intel_dp *intel_dp) >> return -EINVAL; >> } >> >> +static bool >> +intel_dp_check_cec_status(struct intel_dp *intel_dp) >>

Re: [RFC PATCH 7/7] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-05-26 Thread Jani Nikula
On Thu, 25 May 2017, Hans Verkuil wrote: > @@ -4179,6 +4181,33 @@ intel_dp_check_mst_status(struct intel_dp *intel_dp) > return -EINVAL; > } > > +static bool > +intel_dp_check_cec_status(struct intel_dp *intel_dp) > +{ > + bool handled = false; > + > + for

Re: [PATCH] drm/i915: prevent generating unusable gvt build which no mpt module is selected

2017-05-26 Thread Jani Nikula
On Thu, 25 May 2017, "Du, Changbin" wrote: > Hi, Jani, just relized you are in i915 team. :) > >> > +menu "Intel GVT-g graphics virtualization host support" >> > + depends on DRM_I915 >> > + depends on 64BIT >> > + >> > config DRM_I915_GVT >> > -bool "Enable

[Bug 99349] Failed to build shader (translation from TGSI)

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99349 --- Comment #8 from Gert Wollny --- The mesa-code with the added debugging output can be found at: https://github.com/gerddie/mesa -- You are receiving this mail because: You are the assignee for the

[Bug 99349] Failed to build shader (translation from TGSI)

2017-05-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99349 --- Comment #7 from Gert Wollny --- Now, just multiplying two constants/uniforms not necessarily trigger the bug. With a simple shader program like uniform vec4 base_color; uniform vec4 test; uniform vec4 test2; uniform

Re: [PATCH RESEND 1/6] drm/atomic: initial support for asynchronous plane update

2017-05-26 Thread Archit Taneja
On 05/26/2017 01:21 AM, Gustavo Padovan wrote: From: Gustavo Padovan In some cases, like cursor updates, it is interesting to update the plane in an asynchronous fashion to avoid big delays. The current queued update could be still waiting for a fence to signal

Re: [PATCH RESEND 5/6] drm/msm: remove mdp5_cursor_plane_funcs

2017-05-26 Thread Archit Taneja
On 05/26/2017 01:21 AM, Gustavo Padovan wrote: From: Gustavo Padovan After converting legacy cursor updates to atomic async commits mdp5_cursor_plane_funcs just duplicates mdp5_plane_funcs now. Tested-by: Archit Taneja Cc: Rob

Re: [PATCH RESEND 4/6] drm/msm: update cursors asynchronously through atomic

2017-05-26 Thread Archit Taneja
On 05/26/2017 01:21 AM, Gustavo Padovan wrote: From: Gustavo Padovan Add support to async updates of cursors by using the new atomic interface for that. Basically what this commit does is do what mdp5_update_cursor_plane_legacy() did but through atomic.

Re: [RFC PATCH 5/7] drm/cec: Add CEC over Aux register definitions

2017-05-26 Thread Jani Nikula
On Thu, 25 May 2017, Hans Verkuil wrote: > From: Clint Taylor > > Adding DPCD register definitions from the DP 1.3 specification for CEC > over AUX support. > > V2: Add DP_ prefix to all defines. > V3: missed prefixes from the ESI1 defines > > Cc:

[Bug 194761] amdgpu driver breaks on Oland (SI)

2017-05-26 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=194761 Michel Dänzer (mic...@daenzer.net) changed: What|Removed |Added CC|mic...@daenzer.net | --- Comment #22

[Bug 194761] amdgpu driver breaks on Oland (SI)

2017-05-26 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=194761 --- Comment #21 from flora@amd.com --- Hi Michel, This issue might has something with mesa. I add you to the cc list for you're expert on open source umd. It can be reproduced with oland + ubuntu + xinit + glxgears + mesa. reverting (commit

Re: [PULL] drm-misc-fixes

2017-05-26 Thread Jani Nikula
On Fri, 26 May 2017, Daniel Vetter wrote: > On Thu, May 25, 2017 at 7:44 PM, Sean Paul wrote: >> The pull is noisy >> because it includes -rc2. > > dim has you covered for this, in case you've rolled forward but Dave > hasn't yet, you can regenerate

[PATCH] drm/nouveau: Remove no longer needed local ACPI_VIDEO_NOTIFY_PROBE define

2017-05-26 Thread Hans de Goede
The local #define of ACPI_VIDEO_NOTIFY_PROBE was only added temporarily to avoid a dependency between the acpi and nouveau trees while merging. This is now properly defined in acpi/video.h and the local #define can be dropped. Signed-off-by: Hans de Goede ---

Re: [RFC PATCH 6/7] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 05:06:25PM +0200, Hans Verkuil wrote: > From: Hans Verkuil > > This adds support for the DisplayPort CEC-Tunneling-over-AUX > feature that is part of the DisplayPort 1.3 standard. > > Unfortunately, not all DisplayPort/USB-C to HDMI adapters with

Re: [RFC PATCH 7/7] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 05:06:26PM +0200, Hans Verkuil wrote: > From: Hans Verkuil > > Implement support for this DisplayPort feature. > > The cec device is created whenever it detects an adapter that > has this feature. It is only removed when a new adapter is connected

Re: [PATCH 19/37] drm/fsl: Drop drm_vblank_cleanup

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 10:18 AM, Stefan Agner wrote: > On 2017-05-24 07:51, Daniel Vetter wrote: >> Again cleanup before irq disabling doesn't really stop the races, >> so just drop it. Proper fix would be to put drm_atomic_helper_shutdown >> before everything gets cleaned up. >

Re: [PATCH 36/37] drm/zte: Drop drm_vblank_cleanup

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 5:01 AM, Shawn Guo wrote: > On Wed, May 24, 2017 at 04:52:11PM +0200, Daniel Vetter wrote: >> It again looks all cargo-culted for no good reasons. > > drm_vblank_cleanup() is called to release the resources allocated by > drm_vblank_init(). I think

Re: [PATCH] drm/rockchip: Don't allow zero sized gem buffer

2017-05-26 Thread Daniel Vetter
On Fri, May 26, 2017 at 7:52 AM, Christoph Hellwig wrote: > On Fri, May 26, 2017 at 10:30:09AM +0800, jeffy wrote: >> Hi sean, >> >> On 05/25/2017 11:30 PM, Sean Paul wrote: >> > On Tue, May 23, 2017 at 02:39:43PM +0800, Jeffy Chen wrote: >> > > The system would crash when

Re: Maintaining small drm drivers

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 1:09 AM, Patrik Jakobsson wrote: > On Wed, May 24, 2017 at 9:52 PM, Daniel Vetter wrote: >> On Wed, May 24, 2017 at 6:57 PM, Patrik Jakobsson >> wrote: >>> Hi Dave and Daniel, >>> >>> We had a

Re: [PATCH] dim: Enforce review requirements

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 7:37 AM, Lukas Wunner wrote: > On Wed, May 24, 2017 at 11:28:12AM +0200, Daniel Vetter wrote: >> We can't check this when applying (since r-b/a-b tags often get added >> afterwards), but we can check this when pushing. This only looks at >> patches

Re: [PULL] drm-misc-fixes

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 7:44 PM, Sean Paul wrote: > The pull is noisy > because it includes -rc2. dim has you covered for this, in case you've rolled forward but Dave hasn't yet, you can regenerate against linus upstream branch for a cleaner pull (but still warn Dave ofc):