[PATCH 04/35] drm: Forbid legacy MAP functions for DRIVER_MODESET

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 07:29:37PM +0200, Daniel Vetter wrote: > Like in > > commit 0e975980d435d58df2d430d688b8c18778b42218 > Author: Peter Antoine > Date: Tue Jun 23 08:18:49 2015 +0100 > > drm: Turn off Legacy Context Functions > > we need to again make an exception for nouveau, but

[Bug 117131] vga_switcheroo does not switch IGP -> DIS ( IGP == i915 , DIS == radeon )

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117131 --- Comment #26 from Alex Deucher --- (In reply to Jason Vas Dias from comment #22) > It looks like the Intel driver is actually internally using the Radeon > driver? No. The drivers are independent. vga_switcheroo provides the intermediary

[PATCH 03/35] drm: Move drm_getmap into drm_bufs.c and give it a legacy prefix

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 07:29:36PM +0200, Daniel Vetter wrote: > It belongs right next to the addmap and rmmap functions really. And > for OCD consistency name it drm_legacy_getmap_ioctl. > > Signed-off-by: Daniel Vetter Reviewed-by: Chris Wilson -Chris -- Chris Wilson, Intel Open Source

[PATCH 06/35] drm: Hide master MAP cleanup in drm_bufs.c

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 07:29:39PM +0200, Daniel Vetter wrote: > And again make sure it's a no-op for modern drivers, again with the > exception of nouveau. Another case of dev->struct_mutex gone for > modern drivers! > > v2: Also add a DRIVER_* check like for all other maps functions to > really

[Bug 117131] vga_switcheroo does not switch IGP -> DIS ( IGP == i915 , DIS == radeon )

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117131 --- Comment #25 from Alex Deucher --- Please attach you logs rather than pasting them in the comments, it's much harder to follow with them inline. Also, as per comment 11, please try removing your xorg config file completely. The drivers are

[Bug 117131] vga_switcheroo does not switch IGP -> DIS ( IGP == i915 , DIS == radeon )

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117131 --- Comment #24 from Alex Deucher --- (In reply to Jason Vas Dias from comment #20) > When I look at linux 4.5.0 boot dmesg log messages like : > > [3.115380] vga_switcheroo: enabled > [3.115463] ATPX version 1, functions 0x0033 >

[PATCH 09/35] drm/gem: support BO freeing without dev->struct_mutex

2016-04-26 Thread Daniel Vetter
On Tue, Apr 26, 2016 at 08:47:32PM +0100, Chris Wilson wrote: > On Tue, Apr 26, 2016 at 07:29:42PM +0200, Daniel Vetter wrote: > > Finally all the core gem and a lot of drivers are entirely free of > > dev->struct_mutex depencies, and we can start to have an entirely > > lockless unref path. > >

[PATCH 01/35] drm: Give drm_agp_clear drm_legacy_ prefix

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 07:29:34PM +0200, Daniel Vetter wrote: > It has a DRIVER_MODESET check to sure make it's not creating havoc > for drm drivers. Make that clear in the name too. The pattern is that any extern that starts if (drm_core_check_feature(MODESET)) return becomes

[RFC v2 5/8] drm/fence: add in-fences support

2016-04-26 Thread Daniel Vetter
On Tue, Apr 26, 2016 at 09:55:06PM +0300, Ville Syrjälä wrote: > On Tue, Apr 26, 2016 at 08:23:46PM +0200, Daniel Vetter wrote: > > On Tue, Apr 26, 2016 at 08:40:45PM +0300, Ville Syrjälä wrote: > > > On Tue, Apr 26, 2016 at 07:20:49PM +0200, Daniel Vetter wrote: > > > > On Tue, Apr 26, 2016

i951 Xv playback uneven

2016-04-26 Thread Florian Zumbiehl
Hi, with the drm-intel-nightly I am currently running, I am also seeing some quite noticable stuttering in Xv playback. It's particularly obvious in stuff like rolling credits. It's not directly a performance problem, as the effect can be seen even in "stop motion" (i.e., frame by frame)

[RFC v2 5/8] drm/fence: add in-fences support

2016-04-26 Thread Ville Syrjälä
On Tue, Apr 26, 2016 at 08:23:46PM +0200, Daniel Vetter wrote: > On Tue, Apr 26, 2016 at 08:40:45PM +0300, Ville Syrjälä wrote: > > On Tue, Apr 26, 2016 at 07:20:49PM +0200, Daniel Vetter wrote: > > > On Tue, Apr 26, 2016 at 07:26:21PM +0300, Ville Syrjälä wrote: > > > > On Tue, Apr 26, 2016

[Intel-gfx] [PATCH 05/35] drm: Push struct_mutex into ->master_destroy

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 07:29:38PM +0200, Daniel Vetter wrote: > Only two drivers implement this hook. vmwgfx (which doesn't need it > really) and legacy radeon (which since v1 has been nuked, yay). > > v1: Rebase over radeon ums removal. > > Cc: Thomas Hellstrom > Cc: Alex Deucher >

[Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 07:29:41PM +0200, Daniel Vetter wrote: > amdgpu gained dev->struct_mutex usage, and that's because it's walking > the dev->filelist list. Protect that list with it's own lock to take > one more step towards getting rid of struct_mutex usage in drivers > once and for all. >

[PATCH 09/35] drm/gem: support BO freeing without dev->struct_mutex

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 10:10:14PM +0200, Daniel Vetter wrote: > On Tue, Apr 26, 2016 at 08:47:32PM +0100, Chris Wilson wrote: > > On Tue, Apr 26, 2016 at 07:29:42PM +0200, Daniel Vetter wrote: > > > Finally all the core gem and a lot of drivers are entirely free of > > > dev->struct_mutex

[PATCH v6 03/25] io-mapping: Specify mapping size for io_mapping_map_wc()

2016-04-26 Thread Chris Wilson
The ioremap() hidden behind the io_mapping_map_wc() convenience helper can be used for remapping multiple pages. Extend the helper so that future callers can use it for larger ranges. Signed-off-by: Chris Wilson Cc: Tvrtko Ursulin Cc: Daniel Vetter Cc: Jani Nikula Cc: David Airlie Cc: Yishai

[PATCH 09/35] drm/gem: support BO freeing without dev->struct_mutex

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 07:29:42PM +0200, Daniel Vetter wrote: > Finally all the core gem and a lot of drivers are entirely free of > dev->struct_mutex depencies, and we can start to have an entirely > lockless unref path. > > To make sure that no one who touches the core code accidentally breaks

[RFC v2 5/8] drm/fence: add in-fences support

2016-04-26 Thread Ville Syrjälä
On Tue, Apr 26, 2016 at 07:20:49PM +0200, Daniel Vetter wrote: > On Tue, Apr 26, 2016 at 07:26:21PM +0300, Ville Syrjälä wrote: > > On Tue, Apr 26, 2016 at 04:36:36PM +0200, Daniel Vetter wrote: > > > On Tue, Apr 26, 2016 at 11:14:22AM -0300, Gustavo Padovan wrote: > > > > 2016-04-26 Ville

[RFC v2 5/8] drm/fence: add in-fences support

2016-04-26 Thread Daniel Vetter
On Tue, Apr 26, 2016 at 08:40:45PM +0300, Ville Syrjälä wrote: > On Tue, Apr 26, 2016 at 07:20:49PM +0200, Daniel Vetter wrote: > > On Tue, Apr 26, 2016 at 07:26:21PM +0300, Ville Syrjälä wrote: > > > On Tue, Apr 26, 2016 at 04:36:36PM +0200, Daniel Vetter wrote: > > > > On Tue, Apr 26, 2016

[Bug 71789] [r300g] Visuals not found in (default) depth = 24

2016-04-26 Thread bugzilla-dae...@freedesktop.org
art -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160426/b107fe48/attachment.html>

[RFC v2 7/8] drm/fence: add fence timeline to drm_crtc

2016-04-26 Thread Ville Syrjälä
On Tue, Apr 26, 2016 at 11:23:06AM -0300, Gustavo Padovan wrote: > 2016-04-26 Ville Syrjälä : > > > On Mon, Apr 25, 2016 at 07:33:27PM -0300, Gustavo Padovan wrote: > > > From: Gustavo Padovan > > > > > > Create one timeline context for each CRTC to be able to handle out-fences > > > and

[PATCH 35/35] drm/rockchip: Use cma gem vm ops

2016-04-26 Thread Daniel Vetter
No need to reinvent this little wheel. Cc: Mark Yao Signed-off-by: Daniel Vetter --- drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c

[PATCH 34/35] drm/virtio: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
With Benjanim's patch to remove the dev->struct_mutex cargo cult the sti driver is now also entirely legacy locking free. Let's convert it too. Cc: Benjamin Gaignard Signed-off-by: Daniel Vetter --- drivers/gpu/drm/sti/sti_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 33/35] drm: sti: remove useless call to dev->struct_mutex

2016-04-26 Thread Daniel Vetter
From: Benjamin Gaignard No need to protect debugfs functions with dev->struct_mutex Signed-off-by: Benjamin Gaignard Signed-off-by: Daniel Vetter --- drivers/gpu/drm/sti/sti_cursor.c | 7 --- drivers/gpu/drm/sti/sti_drv.c| 6 --

[PATCH 32/35] drm/virtio: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Gerd Hoffmann Cc: David Airlie Signed-off-by: Daniel Vetter --- drivers/gpu/drm/virtio/virtgpu_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.c b/drivers/gpu/drm/virtio/virtgpu_drv.c

[PATCH 31/35] drm/vgem: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: seanpaul at chromium.org Signed-off-by: Daniel Vetter --- drivers/gpu/drm/vgem/vgem_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/vgem/vgem_drv.c b/drivers/gpu/drm/vgem/vgem_drv.c index

[PATCH 30/35] drm/vc4: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Eric Anholt Signed-off-by: Daniel Vetter --- drivers/gpu/drm/vc4/vc4_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/vc4/vc4_drv.c b/drivers/gpu/drm/vc4/vc4_drv.c index b7d2ff0e6e1f..13f704de39e1 100644 ---

[PATCH 29/35] drm/vc4: Use drm_gem_object_unreference_unlocked

2016-04-26 Thread Daniel Vetter
Since my last struct_mutex crusade someone escaped! This already has the advantage that for the common case when someone else holds a ref the unref won't even acquire dev->struct_mutex. And I'm working on code to allow drivers to completely opt-out of any and all dev->struct_mutex usage, but that

[PATCH 28/35] drm/tilcdc: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Tomi Valkeinen Signed-off-by: Daniel Vetter --- drivers/gpu/drm/tilcdc/tilcdc_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tilcdc/tilcdc_drv.c b/drivers/gpu/drm/tilcdc/tilcdc_drv.c index

[PATCH 27/35] drm/tegra: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Thierry Reding Cc: Terje Bergström Cc: linux-tegra at vger.kernel.org Signed-off-by: Daniel Vetter --- drivers/gpu/drm/tegra/drm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tegra/drm.c

[PATCH 26/35] drm/shmob: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Laurent Pinchart Signed-off-by: Daniel Vetter --- drivers/gpu/drm/shmobile/shmob_drm_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/shmobile/shmob_drm_drv.c b/drivers/gpu/drm/shmobile/shmob_drm_drv.c

[PATCH 25/35] drm/rockchip: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Mark Yao Signed-off-by: Daniel Vetter --- drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c

[PATCH 24/35] drm/rcar-du: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Laurent Pinchart Signed-off-by: Daniel Vetter --- drivers/gpu/drm/rcar-du/rcar_du_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_du_drv.c b/drivers/gpu/drm/rcar-du/rcar_du_drv.c index

[PATCH 23/35] drm/radeon: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Alex Deucher Signed-off-by: Daniel Vetter --- drivers/gpu/drm/radeon/radeon_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c index

[PATCH 22/35] drm/qxl: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Dave Airlie Signed-off-by: Daniel Vetter --- drivers/gpu/drm/qxl/qxl_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/qxl/qxl_drv.c b/drivers/gpu/drm/qxl/qxl_drv.c index dc9df5fe50ba..460bbceae297 100644 ---

[PATCH 21/35] drm/nouveau: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Ben Skeggs Signed-off-by: Daniel Vetter --- drivers/gpu/drm/nouveau/nouveau_drm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c b/drivers/gpu/drm/nouveau/nouveau_drm.c index

[PATCH 20/35] drm/mga200g: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Dave Airlie Signed-off-by: Daniel Vetter --- drivers/gpu/drm/mgag200/mgag200_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c b/drivers/gpu/drm/mgag200/mgag200_drv.c index

[PATCH 19/35] drm/imx: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Sascha Hauer Cc: Philipp Zabel Signed-off-by: Daniel Vetter --- drivers/gpu/drm/imx/imx-drm-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/imx/imx-drm-core.c b/drivers/gpu/drm/imx/imx-drm-core.c index

[PATCH 18/35] drm/fls-dcu: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Jianwei Wang Cc: Stefan Agner Signed-off-by: Daniel Vetter --- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c

[PATCH 17/35] drm/exynos: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Inki Dae Signed-off-by: Daniel Vetter --- drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.c b/drivers/gpu/drm/exynos/exynos_drm_drv.c index

[PATCH 16/35] drm/etnaviv: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Christian Gmeiner Cc: Russell King Cc: Lucas Stach Signed-off-by: Daniel Vetter --- drivers/gpu/drm/etnaviv/etnaviv_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 15/35] drm/cirrus: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Dave Airlie Signed-off-by: Daniel Vetter --- drivers/gpu/drm/cirrus/cirrus_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/cirrus/cirrus_drv.c b/drivers/gpu/drm/cirrus/cirrus_drv.c index

[PATCH 14/35] drm/bochs: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Gerd Hoffmann Cc: Dave Airlie Signed-off-by: Daniel Vetter --- drivers/gpu/drm/bochs/bochs_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bochs/bochs_drv.c b/drivers/gpu/drm/bochs/bochs_drv.c index

[PATCH 13/35] drm/atmel: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Boris Brezillon Signed-off-by: Daniel Vetter --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c

[PATCH 12/35] drm/ast: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Dave Airlie Signed-off-by: Daniel Vetter --- drivers/gpu/drm/ast/ast_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/ast/ast_drv.c b/drivers/gpu/drm/ast/ast_drv.c index fcd9c0714836..f54afd2113a9 100644 ---

[PATCH 11/35] drm/armada: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Russell King Signed-off-by: Daniel Vetter --- drivers/gpu/drm/armada/armada_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/armada/armada_drv.c

[PATCH 10/35] drm/amdgpu: Use lockless gem BO free callback

2016-04-26 Thread Daniel Vetter
No dev->struct_mutex anywhere to be seen. Cc: Alex Deucher Signed-off-by: Daniel Vetter --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c index

[PATCH 09/35] drm/gem: support BO freeing without dev->struct_mutex

2016-04-26 Thread Daniel Vetter
Finally all the core gem and a lot of drivers are entirely free of dev->struct_mutex depencies, and we can start to have an entirely lockless unref path. To make sure that no one who touches the core code accidentally breaks existing drivers which still require dev->struct_mutex I've made the

[PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-26 Thread Daniel Vetter
amdgpu gained dev->struct_mutex usage, and that's because it's walking the dev->filelist list. Protect that list with it's own lock to take one more step towards getting rid of struct_mutex usage in drivers once and for all. While doing the conversion I noticed that 2 debugfs files in i915

[PATCH 07/35] drm: Make drm_vm_open/close_locked private to drm_vm.c

2016-04-26 Thread Daniel Vetter
It's only used for legacy mmaping support now. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_internal.h | 2 -- drivers/gpu/drm/drm_vm.c | 16 2 files changed, 4 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/drm/drm_internal.h

[PATCH 06/35] drm: Hide master MAP cleanup in drm_bufs.c

2016-04-26 Thread Daniel Vetter
And again make sure it's a no-op for modern drivers, again with the exception of nouveau. Another case of dev->struct_mutex gone for modern drivers! v2: Also add a DRIVER_* check like for all other maps functions to really short-circuit the code. And give drm_legacy_rmmap used by the dev

[PATCH 05/35] drm: Push struct_mutex into ->master_destroy

2016-04-26 Thread Daniel Vetter
Only two drivers implement this hook. vmwgfx (which doesn't need it really) and legacy radeon (which since v1 has been nuked, yay). v1: Rebase over radeon ums removal. Cc: Thomas Hellstrom Cc: Alex Deucher Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_drv.c | 2 +- 1 file changed, 1

[PATCH 04/35] drm: Forbid legacy MAP functions for DRIVER_MODESET

2016-04-26 Thread Daniel Vetter
Like in commit 0e975980d435d58df2d430d688b8c18778b42218 Author: Peter Antoine Date: Tue Jun 23 08:18:49 2015 +0100 drm: Turn off Legacy Context Functions we need to again make an exception for nouveau, but everyone else really doesn't need this. Cc: Peter Antoine Cc: Ben Skeggs

[PATCH 03/35] drm: Move drm_getmap into drm_bufs.c and give it a legacy prefix

2016-04-26 Thread Daniel Vetter
It belongs right next to the addmap and rmmap functions really. And for OCD consistency name it drm_legacy_getmap_ioctl. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_bufs.c | 52 ++ drivers/gpu/drm/drm_ioctl.c | 54

[PATCH 02/35] drm: Put legacy lastclose work into drm_legacy_dev_reinit

2016-04-26 Thread Daniel Vetter
Except for the ->lasclose driver callback evrything in drm_lastclose() is all legacy cruft and can be hidden. Which means another dev->struct_mutex site disappears entirely for modern drivers! Also while at it change the return value of drm_lastclose to void since it will always succeed. No one

[PATCH 01/35] drm: Give drm_agp_clear drm_legacy_ prefix

2016-04-26 Thread Daniel Vetter
It has a DRIVER_MODESET check to sure make it's not creating havoc for drm drivers. Make that clear in the name too. v2: Move misplaced hunk, spotted by 0day and Thierry. Cc: Thierry Reding Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_agpsupport.c | 4 ++-- drivers/gpu/drm/drm_fops.c

[PATCH 00/35] Moar struct_mutex nuking

2016-04-26 Thread Daniel Vetter
Hi all, First few patches are resends (reviews, pretty please), but most of this is all new. After all this all the legacy code is guarded by DRIVER_* feature tests, and struct_mutex is never even acquired for modern drivers (except nouveau, because). And there's just 2 things left where

[RFC v2 5/8] drm/fence: add in-fences support

2016-04-26 Thread Ville Syrjälä
On Tue, Apr 26, 2016 at 04:36:36PM +0200, Daniel Vetter wrote: > On Tue, Apr 26, 2016 at 11:14:22AM -0300, Gustavo Padovan wrote: > > 2016-04-26 Ville Syrjälä : > > > > > On Mon, Apr 25, 2016 at 07:33:25PM -0300, Gustavo Padovan wrote: > > > > From: Gustavo Padovan > > > > > > > > There is

[RFC v2 5/8] drm/fence: add in-fences support

2016-04-26 Thread Ville Syrjälä
On Tue, Apr 26, 2016 at 11:14:22AM -0300, Gustavo Padovan wrote: > 2016-04-26 Ville Syrjälä : > > > On Mon, Apr 25, 2016 at 07:33:25PM -0300, Gustavo Padovan wrote: > > > From: Gustavo Padovan > > > > > > There is now a new property called FENCE_FD attached to every plane > > > state that

[RFC v2 5/8] drm/fence: add in-fences support

2016-04-26 Thread Daniel Vetter
On Tue, Apr 26, 2016 at 07:26:21PM +0300, Ville Syrjälä wrote: > On Tue, Apr 26, 2016 at 04:36:36PM +0200, Daniel Vetter wrote: > > On Tue, Apr 26, 2016 at 11:14:22AM -0300, Gustavo Padovan wrote: > > > 2016-04-26 Ville Syrjälä : > > > > > > > On Mon, Apr 25, 2016 at 07:33:25PM -0300, Gustavo

[PATCH v2 4/8] drm/fb-helper: Add fb_deferred_io support

2016-04-26 Thread Daniel Vetter
On Tue, Apr 26, 2016 at 06:24:54PM +0200, Noralf Trønnes wrote: > > Den 25.04.2016 11:09, skrev Daniel Vetter: > >On Sun, Apr 24, 2016 at 10:48:58PM +0200, Noralf Trønnes wrote: > >>This adds deferred io support if CONFIG_FB_DEFERRED_IO is enabled. > >>The fbdev framebuffer changes are flushed

[Bug 117131] vga_switcheroo does not switch IGP -> DIS ( IGP == i915 , DIS == radeon )

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117131 --- Comment #23 from Jason Vas Dias --- Xorg server log of successful session [ 173.214] X.Org X Server 1.18.3 Release Date: 2016-04-04 [ 173.214] X Protocol Version 11, Revision 0 [ 173.214] Build Operating System: Linux 4.5.0 x86_64 [

[Bug 117131] vga_switcheroo does not switch IGP -> DIS ( IGP == i915 , DIS == radeon )

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117131 --- Comment #22 from Jason Vas Dias --- It looks like the Intel driver is actually internally using the Radeon driver? -- You are receiving this mail because: You are watching the assignee of the bug.

[Bug 117131] vga_switcheroo does not switch IGP -> DIS ( IGP == i915 , DIS == radeon )

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117131 --- Comment #21 from Jason Vas Dias --- Aha! It was : Section "Device" Identifier "Intel" Driver "intel" Option "Monitor-eDP-1" "eDP-1" Option "Monitor-DisplayPort-0" "eDP-1" EndSection and also : Section "ServerLayout"

[PATCH v2 1/8] drm/rect: Add some drm_clip_rect utility functions

2016-04-26 Thread Noralf Trønnes
Den 25.04.2016 23:13, skrev Laurent Pinchart: > Hi Noralf, > > On Monday 25 Apr 2016 20:35:18 Noralf Trønnes wrote: >> Den 25.04.2016 18:38, skrev Ville Syrjälä: >>> On Mon, Apr 25, 2016 at 06:05:20PM +0200, Daniel Vetter wrote: On Mon, Apr 25, 2016 at 06:09:44PM +0300, Ville Syrjälä

[PATCH v2 4/8] drm/fb-helper: Add fb_deferred_io support

2016-04-26 Thread Noralf Trønnes
Den 25.04.2016 11:09, skrev Daniel Vetter: > On Sun, Apr 24, 2016 at 10:48:58PM +0200, Noralf Trønnes wrote: >> This adds deferred io support if CONFIG_FB_DEFERRED_IO is enabled. >> The fbdev framebuffer changes are flushed using the callback >> (struct drm_framebuffer *)->funcs->dirty() by a

[PATCH 09/35] drm/gem: support BO freeing without dev->struct_mutex

2016-04-26 Thread Alex Deucher
On Tue, Apr 26, 2016 at 1:29 PM, Daniel Vetter wrote: > Finally all the core gem and a lot of drivers are entirely free of > dev->struct_mutex depencies, and we can start to have an entirely > lockless unref path. > > To make sure that no one who touches the core code accidentally breaks >

[Intel-gfx] [PATCH 08/35] drm: Protect dev->filelist with its own mutex

2016-04-26 Thread Alex Deucher
On Tue, Apr 26, 2016 at 4:52 PM, Chris Wilson wrote: > On Tue, Apr 26, 2016 at 07:29:41PM +0200, Daniel Vetter wrote: >> amdgpu gained dev->struct_mutex usage, and that's because it's walking >> the dev->filelist list. Protect that list with it's own lock to take >> one more step towards getting

[PATCH 07/35] drm: Make drm_vm_open/close_locked private to drm_vm.c

2016-04-26 Thread Alex Deucher
On Tue, Apr 26, 2016 at 1:29 PM, Daniel Vetter wrote: > It's only used for legacy mmaping support now. > > Signed-off-by: Daniel Vetter Reviewed-by: Alex Deucher > --- > drivers/gpu/drm/drm_internal.h | 2 -- > drivers/gpu/drm/drm_vm.c | 16 > 2 files changed, 4

[PATCH 06/35] drm: Hide master MAP cleanup in drm_bufs.c

2016-04-26 Thread Alex Deucher
On Tue, Apr 26, 2016 at 1:29 PM, Daniel Vetter wrote: > And again make sure it's a no-op for modern drivers, again with the > exception of nouveau. Another case of dev->struct_mutex gone for > modern drivers! > > v2: Also add a DRIVER_* check like for all other maps functions to > really

[PATCH 05/35] drm: Push struct_mutex into ->master_destroy

2016-04-26 Thread Alex Deucher
On Tue, Apr 26, 2016 at 1:29 PM, Daniel Vetter wrote: > Only two drivers implement this hook. vmwgfx (which doesn't need it > really) and legacy radeon (which since v1 has been nuked, yay). > > v1: Rebase over radeon ums removal. > > Cc: Thomas Hellstrom > Cc: Alex Deucher > Signed-off-by:

[PATCH 04/35] drm: Forbid legacy MAP functions for DRIVER_MODESET

2016-04-26 Thread Alex Deucher
On Tue, Apr 26, 2016 at 1:29 PM, Daniel Vetter wrote: > Like in > > commit 0e975980d435d58df2d430d688b8c18778b42218 > Author: Peter Antoine > Date: Tue Jun 23 08:18:49 2015 +0100 > > drm: Turn off Legacy Context Functions > > we need to again make an exception for nouveau, but everyone

[PATCH 03/35] drm: Move drm_getmap into drm_bufs.c and give it a legacy prefix

2016-04-26 Thread Alex Deucher
On Tue, Apr 26, 2016 at 1:29 PM, Daniel Vetter wrote: > It belongs right next to the addmap and rmmap functions really. And > for OCD consistency name it drm_legacy_getmap_ioctl. > > Signed-off-by: Daniel Vetter Reviewed-by: Alex Deucher > --- > drivers/gpu/drm/drm_bufs.c | 52

[PATCH 02/35] drm: Put legacy lastclose work into drm_legacy_dev_reinit

2016-04-26 Thread Alex Deucher
On Tue, Apr 26, 2016 at 1:29 PM, Daniel Vetter wrote: > Except for the ->lasclose driver callback evrything in drm_lastclose() > is all legacy cruft and can be hidden. Which means another > dev->struct_mutex site disappears entirely for modern drivers! > > Also while at it change the return

[Bug 117131] vga_switcheroo does not switch IGP -> DIS ( IGP == i915 , DIS == radeon )

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117131 --- Comment #20 from Jason Vas Dias --- When I look at linux 4.5.0 boot dmesg log messages like : [3.115380] vga_switcheroo: enabled [3.115463] ATPX version 1, functions 0x0033 [3.115541] [drm] DMAR active, disabling use of

[RFC v2 8/8] drm/fence: add out-fences support

2016-04-26 Thread Daniel Vetter
On Mon, Apr 25, 2016 at 07:33:28PM -0300, Gustavo Padovan wrote: > From: Gustavo Padovan > > Support DRM out-fences creating a sync_file with a fence for each crtc > update with the DRM_MODE_ATOMIC_OUT_FENCE flag. > > We then send an struct drm_out_fences array with the out-fences fds back in >

[RFC v2 1/8] dma-buf/fence: add fence_collection fences

2016-04-26 Thread Daniel Vetter
On Mon, Apr 25, 2016 at 07:33:21PM -0300, Gustavo Padovan wrote: > From: Gustavo Padovan > > struct fence_collection inherits from struct fence and carries a > collection of fences that needs to be waited together. > > It is useful to translate a sync_file to a fence to remove the complexity >

[RFC v2 5/8] drm/fence: add in-fences support

2016-04-26 Thread Daniel Vetter
On Tue, Apr 26, 2016 at 11:14:22AM -0300, Gustavo Padovan wrote: > 2016-04-26 Ville Syrjälä : > > > On Mon, Apr 25, 2016 at 07:33:25PM -0300, Gustavo Padovan wrote: > > > From: Gustavo Padovan > > > > > > There is now a new property called FENCE_FD attached to every plane > > > state that

[PATCH 00/12] drm/atomic: Rename async to nonblocking.

2016-04-26 Thread Daniel Vetter
On Tue, Apr 26, 2016 at 04:11:33PM +0200, Maarten Lankhorst wrote: > Userspace calls the atomic ioctl with DRM_MODE_ATOMIC_NONBLOCK > when it doesn't want the ioctl to block for completion. > > In the kernel it's called async commit, which is confusing. The legacy > pageflip and atomic ioctl also

[PATCH 12/12] drm/i915: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/i915/intel_display.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_display.c

[PATCH 11/12] drm/vc4: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Cc: Eric Anholt Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/vc4/vc4_kms.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_kms.c b/drivers/gpu/drm/vc4/vc4_kms.c index

[PATCH 10/12] drm/rockchip: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Cc: Mark Yao Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/rockchip/rockchip_drm_fb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_fb.c

[PATCH 09/12] drm/tegra: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Cc: Thierry Reding Cc: "Terje Bergström" Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/tegra/drm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/tegra/drm.c

[PATCH 08/12] drm/sti: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Cc: Benjamin Gaignard Cc: Vincent Abriou Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/sti/sti_drv.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/sti/sti_drv.c

[PATCH 07/12] drm/rcar-du: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Cc: Laurent Pinchart Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/rcar-du/rcar_du_kms.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_du_kms.c

[PATCH 06/12] drm/omapdrm: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Cc: Tomi Valkeinen Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/omapdrm/omap_drv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/omapdrm/omap_drv.c

[PATCH 05/12] drm/msm: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Also comments seem to be a bit outdated, as it looks like nonblocking commit is supported by msm. Cc: Rob Clark Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/msm/msm_atomic.c | 9 - drivers/gpu/drm/msm/msm_drv.h

[PATCH 04/12] drm/exynos: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Cc: Inki Dae Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/exynos/exynos_drm_drv.c | 4 ++-- drivers/gpu/drm/exynos/exynos_drm_drv.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH 03/12] drm/arm/hdlcd: Rename async to nonblock.

2016-04-26 Thread Maarten Lankhorst
The async name is deprecated and should be changed to nonblocking. Cc: Liviu Dudau Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/arm/hdlcd_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/arm/hdlcd_drv.c b/drivers/gpu/drm/arm/hdlcd_drv.c index

[PATCH 02/12] drm/atomic: Rename drm_atomic_async_commit to nonblocking.

2016-04-26 Thread Maarten Lankhorst
Another step in renaming async to nonblocking for atomic commit. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_atomic.c| 12 ++-- drivers/gpu/drm/drm_atomic_helper.c | 4 ++-- include/drm/drm_atomic.h| 2 +- 3 files changed, 9 insertions(+), 9

[PATCH 01/12] drm/atomic: Rename async parameter to nonblocking.

2016-04-26 Thread Maarten Lankhorst
This is the first step of renaming async commit to nonblocking commit. The flag passed by userspace is NONBLOCKING, and async has a different meaning for page flips, where it means as soon as possible. Fixing up comments in drm core is done manually, to make sure I didn't miss anything. For

[PATCH 00/12] drm/atomic: Rename async to nonblocking.

2016-04-26 Thread Maarten Lankhorst
Userspace calls the atomic ioctl with DRM_MODE_ATOMIC_NONBLOCK when it doesn't want the ioctl to block for completion. In the kernel it's called async commit, which is confusing. The legacy pageflip and atomic ioctl also have a flag DRM_MODE_PAGE_FLIP_ASYNC which means that the flip should happen

[RFC v2 1/8] dma-buf/fence: add fence_collection fences

2016-04-26 Thread Chris Wilson
On Mon, Apr 25, 2016 at 07:33:21PM -0300, Gustavo Padovan wrote: > +static const char *fence_collection_get_timeline_name(struct fence *fence) > +{ > + return "no context"; "unbound" to distinguish from fence contexts within a timeline? > +static bool fence_collection_enable_signaling(struct

[PATCH 03/12] drm/arm/hdlcd: Rename async to nonblock.

2016-04-26 Thread Liviu Dudau
On Tue, Apr 26, 2016 at 04:11:36PM +0200, Maarten Lankhorst wrote: > The async name is deprecated and should be changed to nonblocking. > > Cc: Liviu Dudau > Signed-off-by: Maarten Lankhorst Acked-by: Liviu Dudau > --- > drivers/gpu/drm/arm/hdlcd_drv.c | 2 +- > 1 file changed, 1

[Bug 117131] vga_switcheroo does not switch IGP -> DIS ( IGP == i915 , DIS == radeon )

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117131 --- Comment #19 from Jason Vas Dias --- Oops, the link should have been: https://bugs.freedesktop.org/show_bug.cgi?id=95140 but as it turns out, that other bug could be equally relevant. -- You are receiving this mail because: You are

[GIT PULL] drm: Add support of ARC PGU display controller

2016-04-26 Thread Alexey Brodkin
Hi Dave, This is DRM driver for ARC PGU - simple bitstreamer used on Synopsys ARC SDP boards (both AXS101 and AXS103). Those sources are from v6 series floated to mailing list here https://lists.freedesktop.org/archives/dri-devel/2016-April/105156.html and based on top of today's drm-next

[Bug 117131] vga_switcheroo does not switch IGP -> DIS ( IGP == i915 , DIS == radeon )

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117131 --- Comment #18 from Jason Vas Dias --- I think FGLRX is evidently able to run my laptop's display in dual-driver mode whereby the Intel IGP card runs the LCD in some sort of "slave" mode under the control of the fglrx Radeon 8970M controller

[Bug 116251] radeon 5a427809cd9143bef89ee3110f45e84f37484218 "drm/radeon: disable runtime pm on PX laptop" makes dGPU never stop

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=116251 --- Comment #7 from Alex Deucher --- The patch has been reverted: http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=bfaddd9fc8ac048b99475f000dbef6f08297417f and will be reverted in stable branches as well. -- You are

[Bug 116251] radeon 5a427809cd9143bef89ee3110f45e84f37484218 "drm/radeon: disable runtime pm on PX laptop" makes dGPU never stop

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=116251 Kertesz Laszlo changed: What|Removed |Added CC||laszlo.kertesz at gmail.com ---

[Bug 51381] [drm:atom_op_jump] *ERROR* atombios stuck in loop for more than 5secs aborting, when disabled via vgaswitcheroo

2016-04-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=51381 Peter Uchno changed: What|Removed |Added CC||peter.uchno at gmail.com --- Comment #41

[RFC v2 5/8] drm/fence: add in-fences support

2016-04-26 Thread Greg Hackmann
On 04/26/2016 01:05 PM, Daniel Vetter wrote: > On Tue, Apr 26, 2016 at 09:55:06PM +0300, Ville Syrjälä wrote: >> On Tue, Apr 26, 2016 at 08:23:46PM +0200, Daniel Vetter wrote: >>> On Tue, Apr 26, 2016 at 08:40:45PM +0300, Ville Syrjälä wrote: >>> But really the reason for per-plane is hw

  1   2   >