Re: [PATCH] drm/scheduler: fix setting the priorty for entities - bisected

2018-08-07 Thread Dieter Nützel
Am 06.08.2018 02:13, schrieb Dieter Nützel: Am 04.08.2018 06:18, schrieb Dieter Nützel: Am 04.08.2018 06:12, schrieb Dieter Nützel: Am 04.08.2018 05:27, schrieb Dieter Nützel: Am 03.08.2018 13:09, schrieb Christian König: Am 03.08.2018 um 03:08 schrieb Dieter Nützel: Hello Christian, AMD

DRM inquiry to Backported drm kernel modules

2018-08-07 Thread John Sledge
Hi all, The main goal is to have the SLES12 SP3 kernel version 4.4.140-94.42.1 the fix it needed for Kabylake graphics issue and also enable the dp aux character device CONFIG_DRM_DP_AUX_CHARDEV=y. I like to inquiry if there's a way or how to proceed with this correctly. My question is I have a

[Bug 107518] polaris powerplay init fails: There must be 1 or more PCIE levels defined in PPTable

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107518 --- Comment #4 from Shawn Anastasio --- (In reply to Alex Deucher from comment #3) > (In reply to Shawn Anastasio from comment #2) > > Upon further testing, the issue seems to go away when the firmware is > > removed from petitboot, preventing

[PATCH v3 13/13] drm/msm/dpu: use private obj to track hw resources

2018-08-07 Thread Jeykumar Sankaran
Switch to state based resource management. This patch overhauls the resource manager and HW allocation methods by maintaining the global resource pool and allocated hw blocks in respective drm component states. Global resource manager(RM) is tracked in private object. Allocation strategy is

[PATCH v3 11/13] drm/msm/dpu: remove display H_TILE from encoder

2018-08-07 Thread Jeykumar Sankaran
Encoder H_TILE values are not used for allocating the hw blocks. no. of hw_intf blocks provides the info. changes in v2: - none changes in v3: - none Change-Id: I1c1c13e9b9f608fbaa8c5897f9f1892029107ac5 Signed-off-by: Jeykumar Sankaran ---

[PATCH v3 12/13] drm/msm/dpu: add atomic private object to dpu kms

2018-08-07 Thread Jeykumar Sankaran
Subclass drm private state for DPU for handling driver specific data. Adds atomic private object and private object lock to dpu kms. Provides helper function to retrieve DPU private data from current atomic state. changes in v2: - none changes in v3: - rebase on [1] [1]

[PATCH v3 10/13] drm/msm/dpu: remove topology name

2018-08-07 Thread Jeykumar Sankaran
Strip down the support for topology enums. It can be replaced with simple hw count checks. changes in v2: - none changes in v3: - none Change-Id: If9b2a4db5bbdf8545b99b6d90825e256d014382d Signed-off-by: Jeykumar Sankaran --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c

[PATCH v3 06/13] drm/msm/dpu: iterate for assigned hw ctl in virtual encoder

2018-08-07 Thread Jeykumar Sankaran
Instead of iterating for hw ctrl per physical encoder, this patch moves the iterations and assignment to the virtual encoder. changes in v2: - none changes in v3: - none Change-Id: I896a8c36d6353986582e9d0fe3da9b2293579d4b Signed-off-by: Jeykumar Sankaran ---

[PATCH v3 07/13] drm/msm/dpu: avoid querying for hw intf before assignment

2018-08-07 Thread Jeykumar Sankaran
hw intf blocks are needed only during encoder enable to program timing engines(for video panels). encoder->enable is triggered only after atomic_modeset at which point we assign the resources for the display pipeline. This patch defers the hw_intf look-up until encoder enable. changes in v2:

[PATCH v3 09/13] drm/msm/dpu: rename hw_ctl to lm_ctl

2018-08-07 Thread Jeykumar Sankaran
Prep change for state based resource management. Rename hw_ctl to lm_ctl to mean the ctl associated with the hw layer mixer block. changes in v2: - none changes in v3: - none Change-Id: If6e6249e089b89225cdfafe9158f7509e97b Signed-off-by: Jeykumar Sankaran ---

[PATCH v3 08/13] drm/msm/dpu: move hw resource tracking to crtc state

2018-08-07 Thread Jeykumar Sankaran
Prep changes for state based resource management. Moves all the hw block tracking for the crtc to the state object. changes in v2: - none changes in v3: - none Change-Id: I2816e9e28b27f1126b477d62eb3858a30a652747 Signed-off-by: Jeykumar Sankaran ---

[PATCH v3 04/13] drm/msm/dpu: program master-slave encoders explicitly

2018-08-07 Thread Jeykumar Sankaran
Identify slave-master encoders and program them explicitly. changes in v2: - none changes in v3: - none Change-Id: I0ebfada05bd7f8437f842ad860490a678aa8f4cd Signed-off-by: Jeykumar Sankaran --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 39 ++--- 1

[PATCH v3 05/13] drm/msm/dpu: use kms stored hw mdp block

2018-08-07 Thread Jeykumar Sankaran
Avoid querying RM for hw mdp block. Use the one stored in KMS during initialization. changes in v2: - none changes in v3: - none Change-Id: I52129b96bd561a5547507d7f567bcaa3dbe554aa Signed-off-by: Jeykumar Sankaran --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 12

[PATCH v3 02/13] drm/msm/dpu: remove resource pool manager

2018-08-07 Thread Jeykumar Sankaran
resource pool manager utility was introduced to manage rotator sessions. Removing the support as the rotator feature doesn't exist. changes in v2: - none changes in v3: - rebase on [1] [1] https://gitlab.freedesktop.org/seanpaul/dpu-staging/commits/for-next Change-Id:

[PATCH v3 01/13] drm/msm/dpu: remove scalar config definitions

2018-08-07 Thread Jeykumar Sankaran
cleans up left out scalar config definitions from headers changes in v2: - none changes in v3: - none Change-Id: Id824dd5075c666f97b964573c97215bb786eac75 Signed-off-by: Jeykumar Sankaran --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h| 2 --

[PATCH v3 00/13] Atomic resource management

2018-08-07 Thread Jeykumar Sankaran
This patchset series introduces drm private object in KMS to manage HW resource management. It modifies the resource manager by introducing API's to do per DRM object resource allocation/cleanups. Patches 00/13 to 11/13 are clean up patches to prepare DPU for the above migration. major changes

[PATCH v3 03/13] drm/msm/dpu: remove ping pong split topology variables

2018-08-07 Thread Jeykumar Sankaran
removes left out variables of previous ping pong split topology cleanup. changes in v2: - none changes in v3: - none Change-Id: I1bf9d242039ce7cfd271233fa27840e83184fb95 Signed-off-by: Jeykumar Sankaran --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 3 --- 1 file changed, 3

[PATCH] drm/msm/disp/dpu: fix early dereference of physical encoder

2018-08-07 Thread Jeykumar Sankaran
This change validates the physical encoder before it is dereferenced. Signed-off-by: Jeykumar Sankaran --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c

Re: [PATCH] drm/atomic: add ATOMIC_AMEND flag to the Atomic IOCTL.

2018-08-07 Thread Gustavo Padovan
On Tue, Aug 07, 2018 at 06:40:39PM +0200, Daniel Vetter wrote: > On Mon, Aug 06, 2018 at 06:01:02PM +0200, Enric Balletbo i Serra wrote: > > From: Gustavo Padovan > > > > This flag tells core to jump ahead the queued update if the conditions > > in drm_atomic_async_check() are met. That means we

[Bug 200645] 4.18-rc regression bisected to e03fd3f30: amdgpu polaris11/rx460 only activates on one output/monitor of two

2018-08-07 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=200645 Duncan (1i5t5.dun...@cox.net) changed: What|Removed |Added Attachment #277491|0 |1 is obsolete|

[PATCH libdrm v2] radeon, evergreen: ensure equal sizes for depth-stencil npot textures

2018-08-07 Thread Gert Wollny
On evergreen depth-stencil textures are allocated as two objects, and when using the eg_surface_init_1d_miptrees code path the size evaluation uses the generalized surf_minify function. Here when allocating the depth texture the alignment takes the depth bpe value into account, and uses bpe=1 for

[PATCH] drm/nouveau: Don't forget to cancel hpd_work on suspend/unload

2018-08-07 Thread Lyude Paul
Currently, there's nothing in nouveau that actually cancels this work struct. So, cancel it on suspend/unload. Otherwise, if we're unlucky enough hpd_work might try to keep running up until the system is suspended. Signed-off-by: Lyude Paul Cc: sta...@vger.kernel.org ---

[Bug 107518] polaris powerplay init fails: There must be 1 or more PCIE levels defined in PPTable

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107518 --- Comment #3 from Alex Deucher --- (In reply to Shawn Anastasio from comment #2) > Upon further testing, the issue seems to go away when the firmware is > removed from petitboot, preventing it from initializing the card before the > host OS.

[PATCH v5 09/13] drm/nouveau: Fix deadlock with fb_helper by inhibiting it's HPD

2018-08-07 Thread Lyude Paul
I'm sure I don't need to tell you that fb_helper's locking is a mess. That being said; fb_helper's locking mess can seriously complicate the runtime suspend/resume operations of drivers because it can invoke atomic commits and connector probing from anywhere that calls

[PATCH v5 13/13] drm/nouveau: Call pm_runtime_get_noresume() from hpd handlers

2018-08-07 Thread Lyude Paul
We can't and don't need to try resuming the device from our hotplug handlers, but hotplug events are generally something we'd like to keep the device awake for whenever possible. So, grab a PM ref safely in our hotplug handlers using pm_runtime_get_noresume() and mark the device as busy once we're

[PATCH v5 08/13] drm/nouveau: Don't ignore nouveau_do_suspend() ret value

2018-08-07 Thread Lyude Paul
Otherwise, how will we roll everything back? Signed-off-by: Lyude Paul Cc: sta...@vger.kernel.org Cc: Lukas Wunner Cc: Karol Herbst --- drivers/gpu/drm/nouveau/nouveau_drm.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c

[PATCH v5 12/13] drm/nouveau: Fix deadlocks in nouveau_connector_detect()

2018-08-07 Thread Lyude Paul
When we disable hotplugging on the GPU, we need to be able to synchronize with each connector's hotplug interrupt handler before the interrupt is finally disabled. This can be a problem however, since nouveau_connector_detect() currently grabs a runtime power reference when handling connector

[PATCH v5 11/13] drm/nouveau: Respond to HPDs by probing one conn at a time

2018-08-07 Thread Lyude Paul
There isn't actually any reason we need to call drm_hpd_irq_event() from our hotplug handler, as we already know which connector the hotplug event was fired for. We're also going to need to avoid probing all connectors needlessly from hotplug handlers anyway so that we can track when

[PATCH v5 05/13] drm/nouveau: Remove useless poll_enable() call in drm_load()

2018-08-07 Thread Lyude Paul
Again, this doesn't do anything. drm_kms_helper_poll_enable() will have already been called in nouveau_display_init() Signed-off-by: Lyude Paul Cc: Lukas Wunner Cc: Karol Herbst [omitting stable Cc, this probably doesn't fix any real bugs] --- drivers/gpu/drm/nouveau/nouveau_drm.c | 4 +--- 1

[PATCH v5 10/13] drm/nouveau: Use pm_runtime_get_noresume() in connector_detect()

2018-08-07 Thread Lyude Paul
It's true we can't resume the device from poll workers in nouveau_connector_detect(). We can however, prevent the autosuspend timer from elapsing immediately if it hasn't already without risking any sort of deadlock with the runtime suspend/resume operations. So do that instead of entirely

[PATCH v5 04/13] drm/nouveau: Remove useless poll_disable() call in switcheroo_set_state()

2018-08-07 Thread Lyude Paul
This won't do anything but potentially make us miss hotplugs. We already call drm_kms_helper_poll_disable() in nouveau_pmops_suspend()->nouveau_display_suspend()->nouveau_display_fini() Signed-off-by: Lyude Paul Cc: Lukas Wunner Cc: Karol Herbst [omitting stable Cc, this likely doesn't fix

[PATCH v5 06/13] drm/nouveau: Call optimus_dsm functions after nouveau_do_suspend()

2018-08-07 Thread Lyude Paul
It's not necessary to call these before we call nouveau_do_suspend(). Ideally; we shouldn't have to call this at all here, but getting that to work will take a bit more effort. So for the time being, just move this call after nouveau_do_suspend() to allow us to easily be able to abort the runtime

[PATCH v5 03/13] drm/nouveau: Remove useless poll_enable() call in switcheroo_set_state()

2018-08-07 Thread Lyude Paul
This doesn't do anything, drm_kms_helper_poll_enable() gets called in nouveau_pmops_resume()->nouveau_display_resume()->nouveau_display_init() already. Signed-off-by: Lyude Paul Cc: Lukas Wunner Cc: Karol Herbst [omitting stable Cc, this likely doesn't fix any real bugs] ---

[PATCH v5 02/13] drm/nouveau: Remove duplicate poll_enable() in pmops_runtime_suspend()

2018-08-07 Thread Lyude Paul
Since actual hotplug notifications don't get disabled until nouveau_display_fini() is called, all this will do is cause any hotplugs that happen between this drm_kms_helper_poll_disable() call and the actual hotplug disablement to potentially be dropped if ACPI isn't around to help us.

[PATCH v5 00/13] Fix connector probing deadlocks from RPM bugs

2018-08-07 Thread Lyude Paul
This is the latest version of https://patchwork.freedesktop.org/series/46815/ I moved everything out of fb_helper and back into nouveau, because it seems that other drivers actually do have this handled already as far as I can tell. Lyude Paul (13): drm/nouveau: Fix bogus

[PATCH v5 01/13] drm/nouveau: Fix bogus drm_kms_helper_poll_enable() placement

2018-08-07 Thread Lyude Paul
Turns out this part is my fault for not noticing when reviewing 9a2eba337cace ("drm/nouveau: Fix drm poll_helper handling"). Currently we call drm_kms_helper_poll_enable() from nouveau_display_hpd_work(). This makes basically no sense however, because that means we're calling

[PATCH v5 07/13] drm/nouveau: Add missing unroll functions in nouveau_do_suspend()

2018-08-07 Thread Lyude Paul
Currently, it appears that nouveau_do_suspend() forgets to roll back suspending fbcon and suspending the device LEDs. We also currently skip the entire rollback process if nouveau_display_suspend() fails. So, fix that. Signed-off-by: Lyude Paul Cc: sta...@vger.kernel.org Cc: Lukas Wunner Cc:

Re: [PATCH libdrm] radeon, evergreen: ensure equal sizes for depth-stencil npot textures

2018-08-07 Thread Gert Wollny
It seems I did a wrong indication in the subject, Am Montag, den 06.08.2018, 10:13 +0200 schrieb Gert Wollny: > On evergreen depth-stencil textures are allocated as two objects, and > when using the eg_surface_init_1d_miptrees code path the size > evaluation > uses the generalized surf_minify

[Bug 107518] polaris powerplay init fails: There must be 1 or more PCIE levels defined in PPTable

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107518 --- Comment #2 from Shawn Anastasio --- Upon further testing, the issue seems to go away when the firmware is removed from petitboot, preventing it from initializing the card before the host OS. This indicates that it may have something to do

Re: [PATCH] drm/sun4i: Avoid failing to init fbdev without any connector

2018-08-07 Thread Paul Kocialkowski
Hi, Le mardi 07 août 2018 à 22:18 +0200, Daniel Vetter a écrit : > On Tue, Aug 07, 2018 at 09:39:19PM +0200, Paul Kocialkowski wrote: > > Initializing and registering fbdev requires at least one DRM connector > > and will fail otherwise. In order to support headless setups (for > > instance for

Re: [PATCH] drm/sun4i: Avoid failing to init fbdev without any connector

2018-08-07 Thread Daniel Vetter
On Tue, Aug 07, 2018 at 09:39:19PM +0200, Paul Kocialkowski wrote: > Initializing and registering fbdev requires at least one DRM connector > and will fail otherwise. In order to support headless setups (for > instance for GPU rendering with the GBM backend, where a DRI card node > is required to

[PATCH] drm/doc: clarify how to acquire required vblank event reference

2018-08-07 Thread Stefan Agner
As a driver write it is not entirely obvious that a reference to the event e mentioned in the doc can be obtained via drm_crtc_vblank_get(). Clarify how to obtain the reference. Signed-off-by: Stefan Agner --- drivers/gpu/drm/drm_vblank.c | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [GIT PULL] tilcdc fixes for v4.19

2018-08-07 Thread Dave Airlie
On 3 August 2018 at 02:55, Jyri Sarha wrote: > Hi Dave, > Please pull the tilcdc fixes for Linux v4.19. This fix has been pending > for some while, because I was planning for a bigger clean up that would > remove the need for this fix. But as it is the time for it - at least > for v4.19 - is gone

[Bug 107518] polaris powerplay init fails: There must be 1 or more PCIE levels defined in PPTable

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107518 --- Comment #1 from Shawn Anastasio --- Created attachment 141003 --> https://bugs.freedesktop.org/attachment.cgi?id=141003=edit dmesg for 4.18.0-rc8+ -- You are receiving this mail because: You are the assignee for the

[PATCH v6] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Chris Wilson
amdgpu only uses shared-fences internally, but dmabuf importers rely on implicit write hazard tracking via the reservation_object.fence_excl. For example, the importer use the write hazard for timing a page flip to only occur after the exporter has finished flushing its write into the surface. As

[PATCH] drm/sun4i: Avoid failing to init fbdev without any connector

2018-08-07 Thread Paul Kocialkowski
Initializing and registering fbdev requires at least one DRM connector and will fail otherwise. In order to support headless setups (for instance for GPU rendering with the GBM backend, where a DRI card node is required to provide GEM memory reservation), add a check on the number of registered

[PATCH] drm/vc4: Make sure to emit a tile coordinates between two MSAA loads.

2018-08-07 Thread Eric Anholt
The HW only executes a load once the tile coordinates packet happens, and only tracks one at a time, so by emitting our two MSAA loads back to back we would end up with an undefined color or Z buffer. Fixes dEQP-EGL.functional.render.multi_context.gles2.rgb888_window Signed-off-by: Eric Anholt

[Bug 107518] polaris powerplay init fails: There must be 1 or more PCIE levels defined in PPTable

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107518 Bug ID: 107518 Summary: polaris powerplay init fails: There must be 1 or more PCIE levels defined in PPTable Product: DRI Version: unspecified Hardware: PowerPC

Re: [PATCH] dma-buf: Remove requirement for ops->map() from dma_buf_export

2018-08-07 Thread Daniel Vetter
On Tue, Aug 07, 2018 at 07:36:47PM +0100, Chris Wilson wrote: > Since commit 9ea0dfbf972 ("dma-buf: make map_atomic and map function > pointers optional"), the core provides the no-op functions when map and > map_atomic are not provided, so we no longer need assert that are > supplied by a dma-buf

Re: [PATCH v3 4/4] drm/mxsfb: Switch to drm_atomic_helper_commit_tail_rpm

2018-08-07 Thread Stefan Agner
On 06.08.2018 21:31, Leonard Crestez wrote: > The lcdif block is only powered on when display is active so plane > updates when not enabled are not valid. Writing to an unpowered IP block > is mostly ignored but can trigger bus errors on some chips. > > Prevent this situation by switching to

Re: [PATCH v5] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Christian König
Am 07.08.2018 um 20:32 schrieb Chris Wilson: amdgpu only uses shared-fences internally, but dmabuf importers rely on implicit write hazard tracking via the reservation_object.fence_excl. For example, the importer use the write hazard for timing a page flip to only occur after the exporter has

Re: [PATCH v3 3/4] drm/mxsfb: Add PM_SLEEP support

2018-08-07 Thread Stefan Agner
On 06.08.2018 21:31, Leonard Crestez wrote: > Since power to the lcdif block can be lost on suspend implement > PM_SLEEP_OPS using drm_mode_config_helper_suspend/resume to save/restore > the current mode. > > Signed-off-by: Leonard Crestez Reviewed-by: Stefan Agner > --- >

[PATCH] dma-buf: Remove requirement for ops->map() from dma_buf_export

2018-08-07 Thread Chris Wilson
Since commit 9ea0dfbf972 ("dma-buf: make map_atomic and map function pointers optional"), the core provides the no-op functions when map and map_atomic are not provided, so we no longer need assert that are supplied by a dma-buf exporter. Fixes: 09ea0dfbf972 ("dma-buf: make map_atomic and map

Re: [PATCH v3 2/4] drm/mxsfb: Add pm_runtime calls to pipe_enable/disable

2018-08-07 Thread Stefan Agner
On 06.08.2018 21:31, Leonard Crestez wrote: > Adding lcdif nodes to a power domain currently results in > black/corrupted screens or hangs because power is not correctly enabled > when required. > > Ensure power is on when display is active by adding > pm_runtime_get/put_sync to

[PATCH v5] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Chris Wilson
amdgpu only uses shared-fences internally, but dmabuf importers rely on implicit write hazard tracking via the reservation_object.fence_excl. For example, the importer use the write hazard for timing a page flip to only occur after the exporter has finished flushing its write into the surface. As

Re: [PATCH v4] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Chris Wilson
Quoting Christian König (2018-08-07 19:18:55) > Am 07.08.2018 um 20:14 schrieb Chris Wilson: > > Quoting Christian König (2018-08-07 18:57:16) > >> Am 07.08.2018 um 18:08 schrieb Chris Wilson: > >>> amdgpu only uses shared-fences internally, but dmabuf importers rely on > >>> implicit write hazard

Re: [PATCH v3 1/4] drm/mxsfb: Fix initial corrupt frame when activating display

2018-08-07 Thread Stefan Agner
On 06.08.2018 21:31, Leonard Crestez wrote: > LCDIF will repeatedly display data from CUR_BUF and set CUR_BUF to > NEXT_BUF when done. Since we are only ever writing to NEXT_BUF the > display will show an initial corrupt frame. > > Fix by writing the FB paddr to both CUR_BUF and NEXT_BUF when >

Re: [PATCH v4] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Christian König
Am 07.08.2018 um 20:14 schrieb Chris Wilson: Quoting Christian König (2018-08-07 18:57:16) Am 07.08.2018 um 18:08 schrieb Chris Wilson: amdgpu only uses shared-fences internally, but dmabuf importers rely on implicit write hazard tracking via the reservation_object.fence_excl. For example, the

Re: [PATCH v4] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Chris Wilson
Quoting Christian König (2018-08-07 18:57:16) > Am 07.08.2018 um 18:08 schrieb Chris Wilson: > > amdgpu only uses shared-fences internally, but dmabuf importers rely on > > implicit write hazard tracking via the reservation_object.fence_excl. > > For example, the importer use the write hazard for

Re: [PATCH v4] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Christian König
Am 07.08.2018 um 18:08 schrieb Chris Wilson: amdgpu only uses shared-fences internally, but dmabuf importers rely on implicit write hazard tracking via the reservation_object.fence_excl. For example, the importer use the write hazard for timing a page flip to only occur after the exporter has

Re: [PATCH] drm: Remove defunct dma_buf_kmap stubs

2018-08-07 Thread Christian König
Am 07.08.2018 um 19:47 schrieb Chris Wilson: Since commit 09ea0dfbf972 ("dma-buf: make map_atomic and map function pointers optional"), we no longer need to provide stub no-op functions as the core now provides them directly. References: 09ea0dfbf972 ("dma-buf: make map_atomic and map function

Re: [PATCH] drm: Remove defunct dma_buf_kmap stubs

2018-08-07 Thread Daniel Vetter
On Tue, Aug 07, 2018 at 06:47:48PM +0100, Chris Wilson wrote: > Since commit 09ea0dfbf972 ("dma-buf: make map_atomic and map function > pointers optional"), we no longer need to provide stub no-op functions > as the core now provides them directly. > > References: 09ea0dfbf972 ("dma-buf: make

[PATCH] drm: Remove defunct dma_buf_kmap stubs

2018-08-07 Thread Chris Wilson
Since commit 09ea0dfbf972 ("dma-buf: make map_atomic and map function pointers optional"), we no longer need to provide stub no-op functions as the core now provides them directly. References: 09ea0dfbf972 ("dma-buf: make map_atomic and map function pointers optional") Signed-off-by: Chris

Re: [PATCH v1 5/5] tinydrm: add winstar wg160160 driver

2018-08-07 Thread Noralf Trønnes
Den 02.08.2018 21.45, skrev Sam Ravnborg: Add driver for the winstar wg160160 display. The driver utilises pardata-dbi that again utilise the pardata subsystem. Signed-off-by: Sam Ravnborg --- MAINTAINERS| 5 + drivers/gpu/drm/tinydrm/Kconfig| 10 ++

Re: [PATCH v5 0/7] drm: add simpledrm driver

2018-08-07 Thread Daniel Vetter
On Tue, Jul 24, 2018 at 02:27:33PM +0200, Mads Lønsethagen wrote: > > Hey > > > > On request of Noralf, I picked up the patches and prepared v5. Works > > fine with > > Xorg, if configured according to: > > https://lists.freedesktop.org/archives/dri-devel/2014-January/052777.html > > If anyone

Re: [PATCH 2/2] drm/vkms: Compute CRC with Cursor Plane

2018-08-07 Thread Daniel Vetter
On Mon, Aug 06, 2018 at 06:58:29AM +0300, Haneen Mohammed wrote: > This patch compute CRC for output frame with cursor and primary plane. > Blend cursor with primary plane and compute CRC on the resulted frame. > > Signed-off-by: Haneen Mohammed Nice! I assume with this you're passing all the

Re: [PATCH] drm/atomic: add ATOMIC_AMEND flag to the Atomic IOCTL.

2018-08-07 Thread Daniel Vetter
On Mon, Aug 06, 2018 at 06:01:02PM +0200, Enric Balletbo i Serra wrote: > From: Gustavo Padovan > > This flag tells core to jump ahead the queued update if the conditions > in drm_atomic_async_check() are met. That means we are only able to do an > async update if no modeset is pending and

Re: [PATCH v1 2/5] pardata: new bus for parallel data access

2018-08-07 Thread Noralf Trønnes
Hi Sam, Den 02.08.2018 21.45, skrev Sam Ravnborg: The pardata supports implement a simple bus for devices that are connected using a parallel bus driven by GPIOs. The is often used in combination with simple displays that is often seen in older embedded designs. There is a demand for this

[PATCH v4] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Chris Wilson
amdgpu only uses shared-fences internally, but dmabuf importers rely on implicit write hazard tracking via the reservation_object.fence_excl. For example, the importer use the write hazard for timing a page flip to only occur after the exporter has finished flushing its write into the surface. As

Re: [PATCH] drm/tinydrm: Replace drm_dev_unref with drm_dev_put

2018-08-07 Thread Noralf Trønnes
Den 13.07.2018 15.46, skrev Thomas Zimmermann: This patch unifies the naming of DRM functions for reference counting of struct drm_device. The resulting code is more aligned with the rest of the Linux kernel interfaces. Signed-off-by: Thomas Zimmermann --- Thanks, applied to drm-misc.

Re: [PATCH libdrm] tests/util: Add support for stm module

2018-08-07 Thread Emil Velikov
On 7 August 2018 at 14:39, Benjamin Gaignard wrote: > 2018-07-20 13:32 GMT+02:00 Benjamin Gaignard : >> Signed-off-by: Benjamin Gaignard >> --- >> tests/util/kms.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/tests/util/kms.c b/tests/util/kms.c >> index 8b3e7878..a2d1d7ba 100644

[Bug 103199] [CI] igt@drv_missed_irq - fail - Failed assertion: missed_rings == expect_rings

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103199 --- Comment #3 from Martin Peres --- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_93/fi-icl-u/igt@drv_missed_irq.html (drv_missed_irq:1510) CRITICAL: Test assertion failure function __real_main96, file ../tests/drv_missed_irq.c:159:

[Bug 107516] Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107516 --- Comment #5 from Mike --- PS. security.sandbox.content.read_path_whitelist works. Need to add /sys/ (with trailing slash, as https://wiki.mozilla.org/Security/Sandbox says), and restart Firefox. -- You are receiving this mail because: You

Re: [PATCH libdrm v3] tests/modetest: Add atomic support

2018-08-07 Thread Emil Velikov
On 7 August 2018 at 14:53, Daniel Stone wrote: > Hi Emil, > This is off-topic for the list, but ... > > On Tue, 7 Aug 2018 at 14:46, Emil Velikov wrote: >> Aside: libdrm following X/Wayland in that it lacks contributor/push access >> docs. >> Might be worth, copying the Mesa ones and adding a

Re: [PATCH v3] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Daniel Vetter
On Tue, Aug 7, 2018 at 3:54 PM, Christian König wrote: > Am 07.08.2018 um 15:37 schrieb Daniel Vetter: >> >> On Tue, Aug 07, 2018 at 03:17:06PM +0200, Christian König wrote: >>> >>> Am 07.08.2018 um 14:59 schrieb Daniel Vetter: On Tue, Aug 07, 2018 at 02:51:50PM +0200, Christian König

[Bug 107516] Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107516 Mike changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 107516] Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107516 --- Comment #4 from Mike --- (In reply to Emil Velikov from comment #3) > https://bugzilla.mozilla.org/show_bug.cgi?id=1480755#c1 Indeed, this is Firefox's problem. Аlthough security.sandbox.content.read_path_whitelist trick didn't work for

Re: [PATCH v3] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Christian König
Am 07.08.2018 um 15:37 schrieb Daniel Vetter: On Tue, Aug 07, 2018 at 03:17:06PM +0200, Christian König wrote: Am 07.08.2018 um 14:59 schrieb Daniel Vetter: On Tue, Aug 07, 2018 at 02:51:50PM +0200, Christian König wrote: Am 07.08.2018 um 14:43 schrieb Daniel Vetter: On Tue, Aug 07, 2018 at

Re: [PATCH libdrm v3] tests/modetest: Add atomic support

2018-08-07 Thread Daniel Stone
Hi Emil, This is off-topic for the list, but ... On Tue, 7 Aug 2018 at 14:46, Emil Velikov wrote: > Aside: libdrm following X/Wayland in that it lacks contributor/push access > docs. > Might be worth, copying the Mesa ones and adding a doc in-tree.

[Bug 107516] Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107516 --- Comment #3 from Emil Velikov --- The FF sandboxing is fairly, ahem, strange. The following Mozilla bug addresses that. https://bugzilla.mozilla.org/show_bug.cgi?id=1480755#c1 -- You are receiving this mail because: You are the assignee

Re: [PATCH libdrm v3] tests/modetest: Add atomic support

2018-08-07 Thread Emil Velikov
On 25 July 2018 at 15:00, Benjamin Gaignard wrote: > If "-a" option is set this make modetest use atomic API instead > of legacy API. > > Test the frame rate ("-v") it does a loop and swap between two > framebuffer for each active planes. > > Signed-off-by: Benjamin Gaignard > --- > > version 3:

[Bug 107516] Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107516 --- Comment #2 from Mike --- One more thing. This bug might be firefox-specific, because other OpenGL apps, say, glxgears, use correct driver and get HW accel. -- You are receiving this mail because: You are the assignee for the

[Bug 107516] Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107516 Mike changed: What|Removed |Added See Also||https://bugs.freedesktop.or |

[Bug 98629] OpenGL applications warns "MESA-LOADER: failed to retrieve device information"

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98629 Mike changed: What|Removed |Added See Also||https://bugs.freedesktop.or |

[Bug 107384] random tab crashes in firefox nightly

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107384 Mike changed: What|Removed |Added See Also||https://bugs.freedesktop.or |

[Bug 107516] Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107516 --- Comment #1 from Mike --- Possibly related to bugs 107384 and/or 98629. If firefox start with LIBGL_DEBUG=verbose parameter, it shows: libGL error: MESA-LOADER: failed to retrieve device information libGL error: unable to load driver:

[Bug 107516] Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107516 Mike changed: What|Removed |Added See Also||https://bugs.freedesktop.or |

Re: [PATCH libdrm] tests/util: Add support for stm module

2018-08-07 Thread Benjamin Gaignard
2018-07-20 13:32 GMT+02:00 Benjamin Gaignard : > Signed-off-by: Benjamin Gaignard > --- > tests/util/kms.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tests/util/kms.c b/tests/util/kms.c > index 8b3e7878..a2d1d7ba 100644 > --- a/tests/util/kms.c > +++ b/tests/util/kms.c > @@ -144,6

[Bug 107516] Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so

2018-08-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107516 Bug ID: 107516 Summary: Firefox for WebGL fallbacks to swrast_dri.so, not using radeon_si.so Product: DRI Version: DRI git Hardware: Other OS: All

Re: [PATCH v3] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Daniel Vetter
On Tue, Aug 07, 2018 at 03:17:06PM +0200, Christian König wrote: > Am 07.08.2018 um 14:59 schrieb Daniel Vetter: > > On Tue, Aug 07, 2018 at 02:51:50PM +0200, Christian König wrote: > > > Am 07.08.2018 um 14:43 schrieb Daniel Vetter: > > > > On Tue, Aug 07, 2018 at 01:23:19PM +0200, Christian

Re: [PATCH v2 1/2] drm/sti: do not remove the drm_bridge that was never added

2018-08-07 Thread Benjamin Gaignard
2018-08-06 8:19 GMT+02:00 Peter Rosin : > Removing the drm_bridge_remove call should avoid a NULL dereference > during list processing in drm_bridge_remove if the error path is ever > taken. > > The more natural approach would perhaps be to add a drm_bridge_add, > but there are several other

Re: [PATCH v3 10/10] drm/vmwgfx: Use __drm_atomic_helper_plane_reset instead of copying the logic

2018-08-07 Thread Sinclair Yeh
On Tue, Aug 07, 2018 at 09:03:27AM +0100, Alexandru-Cosmin Gheorghe wrote: > Hi Sinclair, > > Is it ok if I merge this patch through drm-misc-next ? Sure. Thanks for handling this. Sinclair > Thank you, > Alex Gheorghe > > On Mon, Aug 06, 2018 at 09:57:53AM -0700, Sinclair Yeh wrote: > >

Re: [RFC] drm: Allow DRM_IOCTL_MODE_MAP_DUMB for render nodes

2018-08-07 Thread Emil Velikov
On 7 August 2018 at 13:28, Daniel Vetter wrote: > On Tue, Aug 07, 2018 at 12:01:50PM +0100, Emil Velikov wrote: >> On 3 August 2018 at 20:50, Sean Paul wrote: >> > On Fri, Aug 03, 2018 at 06:03:50PM +0100, Emil Velikov wrote: >> >> On 3 August 2018 at 16:06, Martin Fuzzey >> >> wrote: >> >> >

Re: [Mesa-dev] libdrm: Fix amdgpu build failure

2018-08-07 Thread Christian König
Hi Mike, it is not the right mailing list, but thanks for the info. I'm going to push that patch ASAP. Christian. Am 07.08.2018 um 14:38 schrieb Mike Lothian: Hi I'm not sure if this is the right mailing list or not but the following patch gets things building with meson again

Re: [PATCH v3] drm/amdgpu: Transfer fences to dmabuf importer

2018-08-07 Thread Christian König
Am 07.08.2018 um 14:59 schrieb Daniel Vetter: On Tue, Aug 07, 2018 at 02:51:50PM +0200, Christian König wrote: Am 07.08.2018 um 14:43 schrieb Daniel Vetter: On Tue, Aug 07, 2018 at 01:23:19PM +0200, Christian König wrote: Am 07.08.2018 um 13:05 schrieb Chris Wilson: amdgpu only uses

Re: [PATCH 1/8] devfreq: rockchip-dfi: Move GRF definitions to a common place.

2018-08-07 Thread Enric Balletbo i Serra
Hi, On 01/08/18 10:36, Chanwoo Choi wrote: > Hi Enric, > > On 2018년 07월 30일 17:11, Enric Balletbo i Serra wrote: >> Some rk3399 GRF (Generic Register Files) definitions can be used for >> different drivers. Move these definitions to a common include so we >> don't need to duplicate these

Re: [PATCH V4 0/8] backlight: qcom-wled: Support for QCOM wled driver

2018-08-07 Thread kgunda
On 2018-08-03 13:15, Daniel Thompson wrote: On Fri, Aug 03, 2018 at 12:49:34PM +0530, kgu...@codeaurora.org wrote: Hi Bjorn, Can you please help review this patch series ? Pavel, Rob, Daniel reviewed the series except the "auto string detection" patch. I did take a glance at the last

[PATCH] gpu/drm/hisilicon: Convert drm_atomic_helper_suspend/resume()

2018-08-07 Thread Souptick Joarder
convert drm_atomic_helper_suspend/resume() to use drm_mode_config_helper_suspend/resume(). Fixed one sparse warning by making hibmc_drm_interrupt static. Signed-off-by: Ajit Negi Signed-off-by: Souptick Joarder Reviewed-by: Xinliang Liu --- v2: remove ret variable from both suspend and

Re: [PATCH V4 5/8] backlight: qcom-wled: Restructure the driver for WLED3

2018-08-07 Thread Bjorn Andersson
On Mon 09 Jul 03:22 PDT 2018, Kiran Gunda wrote: > diff --git a/drivers/video/backlight/qcom-wled.c > b/drivers/video/backlight/qcom-wled.c [..] > @@ -365,6 +434,15 @@ static int wled_configure(struct wled *wled, struct > device *dev) > > cfg->num_strings = cfg->num_strings + 1; > > +

[PATCH v3 0/4] drm/mxsfb: Fix runtime PM for unpowering lcdif block

2018-08-07 Thread Leonard Crestez
Adding lcdif nodes to a power domain currently does work, it results in black/corrupted screens or hangs. While the driver does enable runtime pm it does not deal correctly with the block being unpowered. Changes since v2: * Split into multiple commits. * Use #ifdef CONFIG_PM_SLEEP. * Switch

  1   2   >