* Re: [PATCH 53/59] drm/arc: Move to drm/tiny @ 2020-05-25 16:36 Eugeniy Paltsev 2020-05-25 17:26 ` Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Eugeniy Paltsev @ 2020-05-25 16:36 UTC (permalink / raw) To: Daniel Vetter; +Cc: dri-devel Hi Daniel, looks like I'll finally have some time for review of "drm/arc: Move to drm/tiny". So, is it possible to find all these changes in any public git repo? Thanks. --- Eugeniy Paltsev _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-05-25 16:36 [PATCH 53/59] drm/arc: Move to drm/tiny Eugeniy Paltsev @ 2020-05-25 17:26 ` Daniel Vetter 2020-05-25 21:18 ` Eugeniy Paltsev 0 siblings, 1 reply; 20+ messages in thread From: Daniel Vetter @ 2020-05-25 17:26 UTC (permalink / raw) To: Eugeniy Paltsev; +Cc: dri-devel On Mon, May 25, 2020 at 6:36 PM Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> wrote: > > Hi Daniel, > > looks like I'll finally have some time for review of "drm/arc: Move to drm/tiny". > So, is it possible to find all these changes in any public git repo? I have a horrible pile with everything here: https://cgit.freedesktop.org/~danvet/drm/log/ I can also give you a cherry-picked branch with just the arc patches, on drm-tip (maybe also linux-next, not sure everything is there because of the merge window freeze we're in already). -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-05-25 17:26 ` Daniel Vetter @ 2020-05-25 21:18 ` Eugeniy Paltsev 0 siblings, 0 replies; 20+ messages in thread From: Eugeniy Paltsev @ 2020-05-25 21:18 UTC (permalink / raw) To: Daniel Vetter; +Cc: linux-snps-arc, Alexey Brodkin, dri-devel So, I've smoke tested it. Though it is working, it's really unstable and causes kernel crash in many cases. I.e: This kernel crash (due to invalid read in kernel mode by 0x0000007c address) happens on HDMI cable disconnection: ----------------------------------->8------------------------------------------- Oops Path: (null) CPU: 0 PID: 12 Comm: kworker/0:1 Not tainted 5.7.0-rc6-03007-g4ceda91a4176 #5 Workqueue: events adv7511_hpd_work Invalid Read @ 0x0000007c by insn @ drm_gem_fb_get_obj+0xa/0x10 ECR: 0x00050100 EFA: 0x0000007c ERET: 0x813b954a STAT32: 0x80080442 [IE K DE ] BTA: 0x813eef62 BLK: arc_pgu_update+0x1a/0x30 SP: 0x9f055dd0 FP: 0x00000000 LPS: 0x8135603c LPE: 0x81356044 LPC: 0x00000000 r00: 0x00000000 r01: 0x00000000 r02: 0x9f6bc000 r03: 0x9f0d8000 r04: 0x9f0d8388 r05: 0x00000000 r06: 0x00000000 r07: 0x00000000 r08: 0x00000000 r09: 0x00000000 r10: 0x00000000 r11: 0x00000000 r12: 0x813eef62 Stack Trace: drm_gem_fb_get_obj+0xa/0x10 arc_pgu_update+0x1a/0x30 drm_atomic_helper_commit_planes.constprop.35+0xc8/0x23c commit_tail+0x164/0x274 drm_atomic_helper_commit+0x36a/0x398 drm_client_modeset_commit_atomic+0x136/0x1a4 drm_client_modeset_commit_locked+0x3a/0x118 drm_client_modeset_commit+0x18/0x34 drm_fb_helper_set_par+0x54/0xb0 drm_fb_helper_hotplug_event.part.16+0x70/0x8c drm_client_dev_hotplug+0x4a/0x74 adv7511_hpd_work+0x6c/0x124 process_one_work+0x19a/0x358 worker_thread+0x108/0x494 kthread+0xec/0x100 ret_from_fork+0x18/0x1c ----------------------------------->8------------------------------------------- This kernel crash (due to invalid read in kernel mode by 0x0 address) happens on weston stop: ----------------------------------->8------------------------------------------- Oops Path: (null) CPU: 0 PID: 158 Comm: weston Not tainted 5.7.0-rc6-03007-g4ceda91a4176 #5 Invalid Read @ 0x00000000 by insn @ drm_gem_fb_destroy+0x32/0x130 ECR: 0x00050100 EFA: 0x00000000 ERET: 0x813b9a76 STAT32: 0x80080002 [IE K ] BTA: 0x81345c66 BLK: drm_fb_release+0x66/0xb0 SP: 0x9f5c7db8 FP: 0x5fa4fa7c LPS: 0x813560ec LPE: 0x813560f0 LPC: 0x00000000 r00: 0x00000000 r01: 0x9e864392 r02: 0x00000002 r03: 0x00000001 r04: 0x00000026 r05: 0x00000000 r06: 0x00000114 r07: 0x00000001 r08: 0x9f0d8140 r09: 0xfffffe20 r10: 0x30405c00 r11: 0x00000039 r12: 0x813d6c16 r13: 0x00022cc8 r14: 0xffffffff r15: 0x00023ef0 r16: 0x5fa4f874 r17: 0x00022c58 r18: 0x5fa4fde4 r19: 0x00000000 r20: 0x000203c0 r21: 0x00000000 r22: 0x00022d40 r23: 0xffffffff r24: 0x5fa4fa80 r25: 0x2000b808 Stack Trace: drm_gem_fb_destroy+0x32/0x130 drm_fb_release+0x66/0xb0 drm_file_free.part.11+0x112/0x1bc drm_release+0x80/0x120 __fput+0x98/0x1bc task_work_run+0x6e/0xa8 do_exit+0x2b4/0x7fc do_group_exit+0x2a/0x8c get_signal+0x9a/0x5f0 do_signal+0x86/0x23c resume_user_mode_begin+0x88/0xd0 ----------------------------------->8------------------------------------------- It was tested on AXS103 board with ARC PGU + adv7511 HDMI bridge. Probably you have some ideas about how to fix it? Thanks. > From: Daniel Vetter <daniel@ffwll.ch> > Sent: Monday, May 25, 2020 20:26 > To: Eugeniy Paltsev > Cc: dri-devel > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > On Mon, May 25, 2020 at 6:36 PM Eugeniy Paltsev > <Eugeniy.Paltsev@synopsys.com> wrote: > > > > Hi Daniel, > > > > looks like I'll finally have some time for review of "drm/arc: Move to drm/tiny". > > So, is it possible to find all these changes in any public git repo? > > I have a horrible pile with everything here: > > https://urldefense.com/v3/__https://cgit.freedesktop.org/*danvet/drm/log/__;fg!!A4F2R9G_pg!KUgdCRZoVj_85WhtTr0NUhsLhDgVTN_7s2aNdS4mMmRDfstvp4JbIYdip4J0P3Hg1NQLG2E$ > > I can also give you a cherry-picked branch with just the arc patches, > on drm-tip (maybe also linux-next, not sure everything is there > because of the merge window freeze we're in already). > -Daniel > -- > Daniel Vetter > Software Engineer, Intel Corporation > +41 (0) 79 365 57 48 - https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!KUgdCRZoVj_85WhtTr0NUhsLhDgVTN_7s2aNdS4mMmRDfstvp4JbIYdip4J0P3HgteJOwjA$ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 00/59] devm_drm_dev_alloc, v2 @ 2020-04-15 7:39 Daniel Vetter 2020-04-15 7:40 ` [PATCH 53/59] drm/arc: Move to drm/tiny Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Daniel Vetter @ 2020-04-15 7:39 UTC (permalink / raw) To: Intel Graphics Development; +Cc: Daniel Vetter, DRI Development Hi all, Here's the next round of devm_drm_dev_alloc. Big changes: - lots of acks, reviews and some testing tags. Very much appreciated! - I dropped the core driver patch, and hence also the patches for vgem, vkms and i915-selftests. That needs more discussions first. - I started to convert some of the other drivers that still used drm_dev_alloc to the shiny new world, with a pile of cleanups on top. So lots more patches to test and review. Looking at git grep output for drm_dev_alloc() I think there's enough room for another few similarly sized patch series. I'm kinda using that also to survey the state of drivers, so I have a good idea what drmm_ versions for various modeset objects should look like. Atm still not sure how to best approach this, with the mix of drivers we do have: Some use devm_kzalloc, some explicit kzalloc + kfree in cleanup code, some embedded it somewhere else. As usual, review, testing and comments very much appreciated. Cheers, Daniel Daniel Vetter (59): drm: Add devm_drm_dev_alloc macro drm/vboxvideo: drop DRM_MTRR_WC #define drm/vboxvideo: Use devm_drm_dev_alloc drm/vboxvideo: Stop using drm_device->dev_private drm/vboxvidoe: use managed pci functions drm/vboxvideo: Use devm_gen_pool_create drm/v3d: Don't set drm_device->dev_private drm/v3d: Use devm_drm_dev_alloc drm/v3d: Delete v3d_dev->dev drm/v3d: Delete v3d_dev->pdev drm/udl: Use devm_drm_dev_alloc drm/udl: don't set drm_device->dev_private drm/st7735r: Use devm_drm_dev_alloc drm/st7586: Use devm_drm_dev_alloc drm/repaper: Use devm_drm_dev_alloc drm/mi0283qt: Use devm_drm_dev_alloc drm/ili9486: Use devm_drm_dev_alloc drm/ili9341: Use devm_drm_dev_alloc drm/ili9225: Use devm_drm_dev_alloc drm/hx8357d: Use devm_drm_dev_alloc drm/gm12u320: Use devm_drm_dev_alloc drm/gm12u320: Don't use drm_device->dev_private drm/tidss: Use devm_drm_dev_alloc drm/tidss: Don't use drm_device->dev_private drm/tidss: Delete tidss->saved_state drm/qxl: Use devm_drm_dev_alloc drm/qxl: Don't use drm_device->dev_private drm/mcde: Use devm_drm_dev_alloc drm/mcde: Don't use drm_device->dev_private drm/ingenic: Use devm_drm_dev_alloc drm/ingenic: Don't set drm_device->dev_private drm/komeda: use devm_drm_dev_alloc drm/armada: Use devm_drm_dev_alloc drm/armada: Don't use drm_device->dev_private drm/cirrus: Use devm_drm_dev_alloc drm/cirrus: Don't use drm_device->dev_private drm/cirrus: Move to drm/tiny drm/i915: Use devm_drm_dev_alloc drm/arcpgu: Switch to devm_drm_dev_alloc drm/arcpgu: Stop using drm_device->dev_private drm/arcpgu: Delete arcpgu_priv->fb drm/arc: Embedded a drm_simple_display_pipe drm/arc: Embedd a drm_connector for sim case drm/arc: Drop surplus connector registration drm/arc: Use drmm_mode_config_cleanup drm/arc: Align with simple pipe helpers drm/arc: Convert to drm_simple_kms_pipe_helper drm/arc: Drop fb/crtc check in arc_pgu_update drm/arc: Inline arcpgu_crtc.c drm/arc: Inline arcpgu_drm_hdmi_init drm/arc: Inline remaining files drm/arc: Initialize sim connector before display pipe drm/arc: Move to drm/tiny drm/aspeed: Drop aspeed_gfx->fbdev drm/aspeed: Use devm_drm_dev_alloc drm/aspeed: Use managed drmm_mode_config_cleanup drm/ast: Use managed pci functions drm/ast: Drop explicit connector register/unregister drm/bochs: Remove explicit drm_connector_register MAINTAINERS | 4 +- drivers/gpu/drm/Kconfig | 4 - drivers/gpu/drm/Makefile | 2 - drivers/gpu/drm/arc/Kconfig | 10 - drivers/gpu/drm/arc/Makefile | 3 - drivers/gpu/drm/arc/arcpgu.h | 37 -- drivers/gpu/drm/arc/arcpgu_crtc.c | 222 --------- drivers/gpu/drm/arc/arcpgu_drv.c | 234 --------- drivers/gpu/drm/arc/arcpgu_hdmi.c | 48 -- drivers/gpu/drm/arc/arcpgu_regs.h | 31 -- drivers/gpu/drm/arc/arcpgu_sim.c | 108 ----- .../gpu/drm/arm/display/komeda/komeda_kms.c | 16 +- drivers/gpu/drm/armada/armada_crtc.c | 4 +- drivers/gpu/drm/armada/armada_debugfs.c | 2 +- drivers/gpu/drm/armada/armada_drm.h | 2 + drivers/gpu/drm/armada/armada_drv.c | 30 +- drivers/gpu/drm/armada/armada_fbdev.c | 4 +- drivers/gpu/drm/armada/armada_gem.c | 4 +- drivers/gpu/drm/armada/armada_overlay.c | 8 +- drivers/gpu/drm/aspeed/aspeed_gfx.h | 3 +- drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c | 2 +- drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 42 +- drivers/gpu/drm/aspeed/aspeed_gfx_out.c | 2 +- drivers/gpu/drm/ast/ast_drv.c | 10 +- drivers/gpu/drm/ast/ast_main.c | 3 - drivers/gpu/drm/ast/ast_mode.c | 3 - drivers/gpu/drm/bochs/bochs_kms.c | 1 - drivers/gpu/drm/cirrus/Kconfig | 19 - drivers/gpu/drm/cirrus/Makefile | 2 - drivers/gpu/drm/drm_drv.c | 23 + drivers/gpu/drm/i915/i915_drv.c | 17 +- drivers/gpu/drm/i915/i915_pci.c | 2 - drivers/gpu/drm/ingenic/ingenic-drm.c | 15 +- drivers/gpu/drm/mcde/mcde_display.c | 10 +- drivers/gpu/drm/mcde/mcde_drm.h | 2 + drivers/gpu/drm/mcde/mcde_drv.c | 21 +- drivers/gpu/drm/mcde/mcde_dsi.c | 2 +- drivers/gpu/drm/qxl/qxl_debugfs.c | 7 +- drivers/gpu/drm/qxl/qxl_display.c | 32 +- drivers/gpu/drm/qxl/qxl_drv.c | 23 +- drivers/gpu/drm/qxl/qxl_drv.h | 7 +- drivers/gpu/drm/qxl/qxl_dumb.c | 2 +- drivers/gpu/drm/qxl/qxl_gem.c | 2 +- drivers/gpu/drm/qxl/qxl_ioctl.c | 14 +- drivers/gpu/drm/qxl/qxl_irq.c | 2 +- drivers/gpu/drm/qxl/qxl_kms.c | 13 +- drivers/gpu/drm/qxl/qxl_object.c | 2 +- drivers/gpu/drm/qxl/qxl_release.c | 2 +- drivers/gpu/drm/qxl/qxl_ttm.c | 2 +- drivers/gpu/drm/tidss/tidss_crtc.c | 16 +- drivers/gpu/drm/tidss/tidss_drv.c | 17 +- drivers/gpu/drm/tidss/tidss_drv.h | 4 +- drivers/gpu/drm/tidss/tidss_irq.c | 12 +- drivers/gpu/drm/tidss/tidss_kms.c | 2 +- drivers/gpu/drm/tidss/tidss_plane.c | 6 +- drivers/gpu/drm/tiny/Kconfig | 29 ++ drivers/gpu/drm/tiny/Makefile | 2 + drivers/gpu/drm/tiny/arcpgu.c | 450 ++++++++++++++++++ drivers/gpu/drm/{cirrus => tiny}/cirrus.c | 22 +- drivers/gpu/drm/tiny/gm12u320.c | 24 +- drivers/gpu/drm/tiny/hx8357d.c | 13 +- drivers/gpu/drm/tiny/ili9225.c | 13 +- drivers/gpu/drm/tiny/ili9341.c | 13 +- drivers/gpu/drm/tiny/ili9486.c | 13 +- drivers/gpu/drm/tiny/mi0283qt.c | 13 +- drivers/gpu/drm/tiny/repaper.c | 14 +- drivers/gpu/drm/tiny/st7586.c | 13 +- drivers/gpu/drm/tiny/st7735r.c | 13 +- drivers/gpu/drm/udl/udl_connector.c | 4 +- drivers/gpu/drm/udl/udl_drv.c | 27 +- drivers/gpu/drm/udl/udl_modeset.c | 6 +- drivers/gpu/drm/v3d/v3d_debugfs.c | 12 +- drivers/gpu/drm/v3d/v3d_drv.c | 47 +- drivers/gpu/drm/v3d/v3d_drv.h | 7 +- drivers/gpu/drm/v3d/v3d_gem.c | 17 +- drivers/gpu/drm/v3d/v3d_irq.c | 16 +- drivers/gpu/drm/v3d/v3d_mmu.c | 10 +- drivers/gpu/drm/v3d/v3d_sched.c | 10 +- drivers/gpu/drm/vboxvideo/vbox_drv.c | 26 +- drivers/gpu/drm/vboxvideo/vbox_drv.h | 1 + drivers/gpu/drm/vboxvideo/vbox_irq.c | 2 +- drivers/gpu/drm/vboxvideo/vbox_main.c | 29 +- drivers/gpu/drm/vboxvideo/vbox_mode.c | 10 +- drivers/gpu/drm/vboxvideo/vbox_ttm.c | 12 - include/drm/drm_drv.h | 33 ++ 85 files changed, 822 insertions(+), 1196 deletions(-) delete mode 100644 drivers/gpu/drm/arc/Kconfig delete mode 100644 drivers/gpu/drm/arc/Makefile delete mode 100644 drivers/gpu/drm/arc/arcpgu.h delete mode 100644 drivers/gpu/drm/arc/arcpgu_crtc.c delete mode 100644 drivers/gpu/drm/arc/arcpgu_drv.c delete mode 100644 drivers/gpu/drm/arc/arcpgu_hdmi.c delete mode 100644 drivers/gpu/drm/arc/arcpgu_regs.h delete mode 100644 drivers/gpu/drm/arc/arcpgu_sim.c delete mode 100644 drivers/gpu/drm/cirrus/Kconfig delete mode 100644 drivers/gpu/drm/cirrus/Makefile create mode 100644 drivers/gpu/drm/tiny/arcpgu.c rename drivers/gpu/drm/{cirrus => tiny}/cirrus.c (97%) -- 2.25.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 53/59] drm/arc: Move to drm/tiny 2020-04-15 7:39 [PATCH 00/59] devm_drm_dev_alloc, v2 Daniel Vetter @ 2020-04-15 7:40 ` Daniel Vetter 2020-04-15 8:04 ` Thomas Zimmermann 2020-04-15 9:45 ` Sam Ravnborg 0 siblings, 2 replies; 20+ messages in thread From: Daniel Vetter @ 2020-04-15 7:40 UTC (permalink / raw) To: Intel Graphics Development Cc: Daniel Vetter, Alexey Brodkin, DRI Development, Daniel Vetter Because it is. Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> Cc: Alexey Brodkin <abrodkin@synopsys.com> --- MAINTAINERS | 2 +- drivers/gpu/drm/Kconfig | 2 -- drivers/gpu/drm/Makefile | 1 - drivers/gpu/drm/arc/Kconfig | 10 ---------- drivers/gpu/drm/arc/Makefile | 3 --- drivers/gpu/drm/tiny/Kconfig | 10 ++++++++++ drivers/gpu/drm/tiny/Makefile | 1 + drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} | 0 8 files changed, 12 insertions(+), 17 deletions(-) delete mode 100644 drivers/gpu/drm/arc/Kconfig delete mode 100644 drivers/gpu/drm/arc/Makefile rename drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} (100%) diff --git a/MAINTAINERS b/MAINTAINERS index 0a5cf105ee37..748244b1625b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1295,7 +1295,7 @@ ARC PGU DRM DRIVER M: Alexey Brodkin <abrodkin@synopsys.com> S: Supported F: Documentation/devicetree/bindings/display/snps,arcpgu.txt -F: drivers/gpu/drm/arc/ +F: drivers/gpu/drm/tiny/arcpgu.c ARCNET NETWORK LAYER M: Michael Grzeschik <m.grzeschik@pengutronix.de> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig index 4f4e7fa001c1..a0a89025d6fa 100644 --- a/drivers/gpu/drm/Kconfig +++ b/drivers/gpu/drm/Kconfig @@ -354,8 +354,6 @@ source "drivers/gpu/drm/vc4/Kconfig" source "drivers/gpu/drm/etnaviv/Kconfig" -source "drivers/gpu/drm/arc/Kconfig" - source "drivers/gpu/drm/hisilicon/Kconfig" source "drivers/gpu/drm/mediatek/Kconfig" diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile index 2c0e5a7e5953..e69eafbf9e39 100644 --- a/drivers/gpu/drm/Makefile +++ b/drivers/gpu/drm/Makefile @@ -109,7 +109,6 @@ obj-y += panel/ obj-y += bridge/ obj-$(CONFIG_DRM_FSL_DCU) += fsl-dcu/ obj-$(CONFIG_DRM_ETNAVIV) += etnaviv/ -obj-$(CONFIG_DRM_ARCPGU)+= arc/ obj-y += hisilicon/ obj-$(CONFIG_DRM_ZTE) += zte/ obj-$(CONFIG_DRM_MXSFB) += mxsfb/ diff --git a/drivers/gpu/drm/arc/Kconfig b/drivers/gpu/drm/arc/Kconfig deleted file mode 100644 index e8f3d63e0b91..000000000000 --- a/drivers/gpu/drm/arc/Kconfig +++ /dev/null @@ -1,10 +0,0 @@ -# SPDX-License-Identifier: GPL-2.0-only -config DRM_ARCPGU - tristate "ARC PGU" - depends on DRM && OF - select DRM_KMS_CMA_HELPER - select DRM_KMS_HELPER - help - Choose this option if you have an ARC PGU controller. - - If M is selected the module will be called arcpgu. diff --git a/drivers/gpu/drm/arc/Makefile b/drivers/gpu/drm/arc/Makefile deleted file mode 100644 index b26f2495c532..000000000000 --- a/drivers/gpu/drm/arc/Makefile +++ /dev/null @@ -1,3 +0,0 @@ -# SPDX-License-Identifier: GPL-2.0-only -arcpgu-y := arcpgu_drv.o -obj-$(CONFIG_DRM_ARCPGU) += arcpgu.o diff --git a/drivers/gpu/drm/tiny/Kconfig b/drivers/gpu/drm/tiny/Kconfig index 2b6414f0fa75..9bbaa1a69050 100644 --- a/drivers/gpu/drm/tiny/Kconfig +++ b/drivers/gpu/drm/tiny/Kconfig @@ -1,5 +1,15 @@ # SPDX-License-Identifier: GPL-2.0-only +config DRM_ARCPGU + tristate "ARC PGU" + depends on DRM && OF + select DRM_KMS_CMA_HELPER + select DRM_KMS_HELPER + help + Choose this option if you have an ARC PGU controller. + + If M is selected the module will be called arcpgu. + config DRM_CIRRUS_QEMU tristate "Cirrus driver for QEMU emulated device" depends on DRM && PCI && MMU diff --git a/drivers/gpu/drm/tiny/Makefile b/drivers/gpu/drm/tiny/Makefile index 6ae4e9e5a35f..bef6780bdd6f 100644 --- a/drivers/gpu/drm/tiny/Makefile +++ b/drivers/gpu/drm/tiny/Makefile @@ -1,5 +1,6 @@ # SPDX-License-Identifier: GPL-2.0-only +obj-$(CONFIG_DRM_ARCPGU) += arcpgu.o obj-$(CONFIG_DRM_CIRRUS_QEMU) += cirrus.o obj-$(CONFIG_DRM_GM12U320) += gm12u320.o obj-$(CONFIG_TINYDRM_HX8357D) += hx8357d.o diff --git a/drivers/gpu/drm/arc/arcpgu_drv.c b/drivers/gpu/drm/tiny/arcpgu.c similarity index 100% rename from drivers/gpu/drm/arc/arcpgu_drv.c rename to drivers/gpu/drm/tiny/arcpgu.c -- 2.25.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-04-15 7:40 ` [PATCH 53/59] drm/arc: Move to drm/tiny Daniel Vetter @ 2020-04-15 8:04 ` Thomas Zimmermann 2020-04-15 8:22 ` Daniel Vetter 2020-04-15 9:45 ` Sam Ravnborg 1 sibling, 1 reply; 20+ messages in thread From: Thomas Zimmermann @ 2020-04-15 8:04 UTC (permalink / raw) To: Daniel Vetter, Intel Graphics Development Cc: Daniel Vetter, Alexey Brodkin, DRI Development [-- Attachment #1.1.1: Type: text/plain, Size: 4796 bytes --] Hi Am 15.04.20 um 09:40 schrieb Daniel Vetter: > Because it is. Yes. OTOH, as much as I appreciate the simplification, I think it should be in a separate series. > > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> > Cc: Alexey Brodkin <abrodkin@synopsys.com> Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de> > --- > MAINTAINERS | 2 +- > drivers/gpu/drm/Kconfig | 2 -- > drivers/gpu/drm/Makefile | 1 - > drivers/gpu/drm/arc/Kconfig | 10 ---------- > drivers/gpu/drm/arc/Makefile | 3 --- > drivers/gpu/drm/tiny/Kconfig | 10 ++++++++++ > drivers/gpu/drm/tiny/Makefile | 1 + > drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} | 0 > 8 files changed, 12 insertions(+), 17 deletions(-) > delete mode 100644 drivers/gpu/drm/arc/Kconfig > delete mode 100644 drivers/gpu/drm/arc/Makefile > rename drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} (100%) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 0a5cf105ee37..748244b1625b 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1295,7 +1295,7 @@ ARC PGU DRM DRIVER > M: Alexey Brodkin <abrodkin@synopsys.com> > S: Supported > F: Documentation/devicetree/bindings/display/snps,arcpgu.txt > -F: drivers/gpu/drm/arc/ > +F: drivers/gpu/drm/tiny/arcpgu.c > > ARCNET NETWORK LAYER > M: Michael Grzeschik <m.grzeschik@pengutronix.de> > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig > index 4f4e7fa001c1..a0a89025d6fa 100644 > --- a/drivers/gpu/drm/Kconfig > +++ b/drivers/gpu/drm/Kconfig > @@ -354,8 +354,6 @@ source "drivers/gpu/drm/vc4/Kconfig" > > source "drivers/gpu/drm/etnaviv/Kconfig" > > -source "drivers/gpu/drm/arc/Kconfig" > - > source "drivers/gpu/drm/hisilicon/Kconfig" > > source "drivers/gpu/drm/mediatek/Kconfig" > diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile > index 2c0e5a7e5953..e69eafbf9e39 100644 > --- a/drivers/gpu/drm/Makefile > +++ b/drivers/gpu/drm/Makefile > @@ -109,7 +109,6 @@ obj-y += panel/ > obj-y += bridge/ > obj-$(CONFIG_DRM_FSL_DCU) += fsl-dcu/ > obj-$(CONFIG_DRM_ETNAVIV) += etnaviv/ > -obj-$(CONFIG_DRM_ARCPGU)+= arc/ > obj-y += hisilicon/ > obj-$(CONFIG_DRM_ZTE) += zte/ > obj-$(CONFIG_DRM_MXSFB) += mxsfb/ > diff --git a/drivers/gpu/drm/arc/Kconfig b/drivers/gpu/drm/arc/Kconfig > deleted file mode 100644 > index e8f3d63e0b91..000000000000 > --- a/drivers/gpu/drm/arc/Kconfig > +++ /dev/null > @@ -1,10 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0-only > -config DRM_ARCPGU > - tristate "ARC PGU" > - depends on DRM && OF > - select DRM_KMS_CMA_HELPER > - select DRM_KMS_HELPER > - help > - Choose this option if you have an ARC PGU controller. > - > - If M is selected the module will be called arcpgu. > diff --git a/drivers/gpu/drm/arc/Makefile b/drivers/gpu/drm/arc/Makefile > deleted file mode 100644 > index b26f2495c532..000000000000 > --- a/drivers/gpu/drm/arc/Makefile > +++ /dev/null > @@ -1,3 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0-only > -arcpgu-y := arcpgu_drv.o > -obj-$(CONFIG_DRM_ARCPGU) += arcpgu.o > diff --git a/drivers/gpu/drm/tiny/Kconfig b/drivers/gpu/drm/tiny/Kconfig > index 2b6414f0fa75..9bbaa1a69050 100644 > --- a/drivers/gpu/drm/tiny/Kconfig > +++ b/drivers/gpu/drm/tiny/Kconfig > @@ -1,5 +1,15 @@ > # SPDX-License-Identifier: GPL-2.0-only > > +config DRM_ARCPGU > + tristate "ARC PGU" > + depends on DRM && OF > + select DRM_KMS_CMA_HELPER > + select DRM_KMS_HELPER > + help > + Choose this option if you have an ARC PGU controller. > + > + If M is selected the module will be called arcpgu. > + > config DRM_CIRRUS_QEMU > tristate "Cirrus driver for QEMU emulated device" > depends on DRM && PCI && MMU > diff --git a/drivers/gpu/drm/tiny/Makefile b/drivers/gpu/drm/tiny/Makefile > index 6ae4e9e5a35f..bef6780bdd6f 100644 > --- a/drivers/gpu/drm/tiny/Makefile > +++ b/drivers/gpu/drm/tiny/Makefile > @@ -1,5 +1,6 @@ > # SPDX-License-Identifier: GPL-2.0-only > > +obj-$(CONFIG_DRM_ARCPGU) += arcpgu.o > obj-$(CONFIG_DRM_CIRRUS_QEMU) += cirrus.o > obj-$(CONFIG_DRM_GM12U320) += gm12u320.o > obj-$(CONFIG_TINYDRM_HX8357D) += hx8357d.o > diff --git a/drivers/gpu/drm/arc/arcpgu_drv.c b/drivers/gpu/drm/tiny/arcpgu.c > similarity index 100% > rename from drivers/gpu/drm/arc/arcpgu_drv.c > rename to drivers/gpu/drm/tiny/arcpgu.c > -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany (HRB 36809, AG Nürnberg) Geschäftsführer: Felix Imendörffer [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-04-15 8:04 ` Thomas Zimmermann @ 2020-04-15 8:22 ` Daniel Vetter 0 siblings, 0 replies; 20+ messages in thread From: Daniel Vetter @ 2020-04-15 8:22 UTC (permalink / raw) To: Thomas Zimmermann Cc: Daniel Vetter, Intel Graphics Development, Alexey Brodkin, DRI Development On Wed, Apr 15, 2020 at 10:04 AM Thomas Zimmermann <tzimmermann@suse.de> wrote: > > Hi > > Am 15.04.20 um 09:40 schrieb Daniel Vetter: > > Because it is. > > Yes. > > OTOH, as much as I appreciate the simplification, I think it should be > in a separate series. Right now they all still need to be here because of the devm_drm_dev_alloc patch 1 adds. After that I guess I can split up, but it's kinda more work. The series is really "clean up driver load/unload code around drm_device", and for some of the older drivers there's a _lot_ that can be done. -Daniel > > > > > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> > > Cc: Alexey Brodkin <abrodkin@synopsys.com> > > Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de> > > > --- > > MAINTAINERS | 2 +- > > drivers/gpu/drm/Kconfig | 2 -- > > drivers/gpu/drm/Makefile | 1 - > > drivers/gpu/drm/arc/Kconfig | 10 ---------- > > drivers/gpu/drm/arc/Makefile | 3 --- > > drivers/gpu/drm/tiny/Kconfig | 10 ++++++++++ > > drivers/gpu/drm/tiny/Makefile | 1 + > > drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} | 0 > > 8 files changed, 12 insertions(+), 17 deletions(-) > > delete mode 100644 drivers/gpu/drm/arc/Kconfig > > delete mode 100644 drivers/gpu/drm/arc/Makefile > > rename drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} (100%) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 0a5cf105ee37..748244b1625b 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -1295,7 +1295,7 @@ ARC PGU DRM DRIVER > > M: Alexey Brodkin <abrodkin@synopsys.com> > > S: Supported > > F: Documentation/devicetree/bindings/display/snps,arcpgu.txt > > -F: drivers/gpu/drm/arc/ > > +F: drivers/gpu/drm/tiny/arcpgu.c > > > > ARCNET NETWORK LAYER > > M: Michael Grzeschik <m.grzeschik@pengutronix.de> > > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig > > index 4f4e7fa001c1..a0a89025d6fa 100644 > > --- a/drivers/gpu/drm/Kconfig > > +++ b/drivers/gpu/drm/Kconfig > > @@ -354,8 +354,6 @@ source "drivers/gpu/drm/vc4/Kconfig" > > > > source "drivers/gpu/drm/etnaviv/Kconfig" > > > > -source "drivers/gpu/drm/arc/Kconfig" > > - > > source "drivers/gpu/drm/hisilicon/Kconfig" > > > > source "drivers/gpu/drm/mediatek/Kconfig" > > diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile > > index 2c0e5a7e5953..e69eafbf9e39 100644 > > --- a/drivers/gpu/drm/Makefile > > +++ b/drivers/gpu/drm/Makefile > > @@ -109,7 +109,6 @@ obj-y += panel/ > > obj-y += bridge/ > > obj-$(CONFIG_DRM_FSL_DCU) += fsl-dcu/ > > obj-$(CONFIG_DRM_ETNAVIV) += etnaviv/ > > -obj-$(CONFIG_DRM_ARCPGU)+= arc/ > > obj-y += hisilicon/ > > obj-$(CONFIG_DRM_ZTE) += zte/ > > obj-$(CONFIG_DRM_MXSFB) += mxsfb/ > > diff --git a/drivers/gpu/drm/arc/Kconfig b/drivers/gpu/drm/arc/Kconfig > > deleted file mode 100644 > > index e8f3d63e0b91..000000000000 > > --- a/drivers/gpu/drm/arc/Kconfig > > +++ /dev/null > > @@ -1,10 +0,0 @@ > > -# SPDX-License-Identifier: GPL-2.0-only > > -config DRM_ARCPGU > > - tristate "ARC PGU" > > - depends on DRM && OF > > - select DRM_KMS_CMA_HELPER > > - select DRM_KMS_HELPER > > - help > > - Choose this option if you have an ARC PGU controller. > > - > > - If M is selected the module will be called arcpgu. > > diff --git a/drivers/gpu/drm/arc/Makefile b/drivers/gpu/drm/arc/Makefile > > deleted file mode 100644 > > index b26f2495c532..000000000000 > > --- a/drivers/gpu/drm/arc/Makefile > > +++ /dev/null > > @@ -1,3 +0,0 @@ > > -# SPDX-License-Identifier: GPL-2.0-only > > -arcpgu-y := arcpgu_drv.o > > -obj-$(CONFIG_DRM_ARCPGU) += arcpgu.o > > diff --git a/drivers/gpu/drm/tiny/Kconfig b/drivers/gpu/drm/tiny/Kconfig > > index 2b6414f0fa75..9bbaa1a69050 100644 > > --- a/drivers/gpu/drm/tiny/Kconfig > > +++ b/drivers/gpu/drm/tiny/Kconfig > > @@ -1,5 +1,15 @@ > > # SPDX-License-Identifier: GPL-2.0-only > > > > +config DRM_ARCPGU > > + tristate "ARC PGU" > > + depends on DRM && OF > > + select DRM_KMS_CMA_HELPER > > + select DRM_KMS_HELPER > > + help > > + Choose this option if you have an ARC PGU controller. > > + > > + If M is selected the module will be called arcpgu. > > + > > config DRM_CIRRUS_QEMU > > tristate "Cirrus driver for QEMU emulated device" > > depends on DRM && PCI && MMU > > diff --git a/drivers/gpu/drm/tiny/Makefile b/drivers/gpu/drm/tiny/Makefile > > index 6ae4e9e5a35f..bef6780bdd6f 100644 > > --- a/drivers/gpu/drm/tiny/Makefile > > +++ b/drivers/gpu/drm/tiny/Makefile > > @@ -1,5 +1,6 @@ > > # SPDX-License-Identifier: GPL-2.0-only > > > > +obj-$(CONFIG_DRM_ARCPGU) += arcpgu.o > > obj-$(CONFIG_DRM_CIRRUS_QEMU) += cirrus.o > > obj-$(CONFIG_DRM_GM12U320) += gm12u320.o > > obj-$(CONFIG_TINYDRM_HX8357D) += hx8357d.o > > diff --git a/drivers/gpu/drm/arc/arcpgu_drv.c b/drivers/gpu/drm/tiny/arcpgu.c > > similarity index 100% > > rename from drivers/gpu/drm/arc/arcpgu_drv.c > > rename to drivers/gpu/drm/tiny/arcpgu.c > > > > -- > Thomas Zimmermann > Graphics Driver Developer > SUSE Software Solutions Germany GmbH > Maxfeldstr. 5, 90409 Nürnberg, Germany > (HRB 36809, AG Nürnberg) > Geschäftsführer: Felix Imendörffer > -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-04-15 7:40 ` [PATCH 53/59] drm/arc: Move to drm/tiny Daniel Vetter 2020-04-15 8:04 ` Thomas Zimmermann @ 2020-04-15 9:45 ` Sam Ravnborg 2020-04-15 12:02 ` Alexey Brodkin 1 sibling, 1 reply; 20+ messages in thread From: Sam Ravnborg @ 2020-04-15 9:45 UTC (permalink / raw) To: Daniel Vetter Cc: Daniel Vetter, Intel Graphics Development, Alexey Brodkin, DRI Development Hi Daniel. On Wed, Apr 15, 2020 at 09:40:28AM +0200, Daniel Vetter wrote: > Because it is. > > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> > Cc: Alexey Brodkin <abrodkin@synopsys.com> > --- > MAINTAINERS | 2 +- > drivers/gpu/drm/Kconfig | 2 -- > drivers/gpu/drm/Makefile | 1 - > drivers/gpu/drm/arc/Kconfig | 10 ---------- > drivers/gpu/drm/arc/Makefile | 3 --- > drivers/gpu/drm/tiny/Kconfig | 10 ++++++++++ > drivers/gpu/drm/tiny/Makefile | 1 + > drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} | 0 > 8 files changed, 12 insertions(+), 17 deletions(-) > delete mode 100644 drivers/gpu/drm/arc/Kconfig > delete mode 100644 drivers/gpu/drm/arc/Makefile > rename drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} (100%) We have "DRM: ARC: add HDMI 2.0 TX encoder support" which adds another platform driver to drm/arc/ This speaks against the move to tiny IMO Sam > > diff --git a/MAINTAINERS b/MAINTAINERS > index 0a5cf105ee37..748244b1625b 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1295,7 +1295,7 @@ ARC PGU DRM DRIVER > M: Alexey Brodkin <abrodkin@synopsys.com> > S: Supported > F: Documentation/devicetree/bindings/display/snps,arcpgu.txt > -F: drivers/gpu/drm/arc/ > +F: drivers/gpu/drm/tiny/arcpgu.c > > ARCNET NETWORK LAYER > M: Michael Grzeschik <m.grzeschik@pengutronix.de> > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig > index 4f4e7fa001c1..a0a89025d6fa 100644 > --- a/drivers/gpu/drm/Kconfig > +++ b/drivers/gpu/drm/Kconfig > @@ -354,8 +354,6 @@ source "drivers/gpu/drm/vc4/Kconfig" > > source "drivers/gpu/drm/etnaviv/Kconfig" > > -source "drivers/gpu/drm/arc/Kconfig" > - > source "drivers/gpu/drm/hisilicon/Kconfig" > > source "drivers/gpu/drm/mediatek/Kconfig" > diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile > index 2c0e5a7e5953..e69eafbf9e39 100644 > --- a/drivers/gpu/drm/Makefile > +++ b/drivers/gpu/drm/Makefile > @@ -109,7 +109,6 @@ obj-y += panel/ > obj-y += bridge/ > obj-$(CONFIG_DRM_FSL_DCU) += fsl-dcu/ > obj-$(CONFIG_DRM_ETNAVIV) += etnaviv/ > -obj-$(CONFIG_DRM_ARCPGU)+= arc/ > obj-y += hisilicon/ > obj-$(CONFIG_DRM_ZTE) += zte/ > obj-$(CONFIG_DRM_MXSFB) += mxsfb/ > diff --git a/drivers/gpu/drm/arc/Kconfig b/drivers/gpu/drm/arc/Kconfig > deleted file mode 100644 > index e8f3d63e0b91..000000000000 > --- a/drivers/gpu/drm/arc/Kconfig > +++ /dev/null > @@ -1,10 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0-only > -config DRM_ARCPGU > - tristate "ARC PGU" > - depends on DRM && OF > - select DRM_KMS_CMA_HELPER > - select DRM_KMS_HELPER > - help > - Choose this option if you have an ARC PGU controller. > - > - If M is selected the module will be called arcpgu. > diff --git a/drivers/gpu/drm/arc/Makefile b/drivers/gpu/drm/arc/Makefile > deleted file mode 100644 > index b26f2495c532..000000000000 > --- a/drivers/gpu/drm/arc/Makefile > +++ /dev/null > @@ -1,3 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0-only > -arcpgu-y := arcpgu_drv.o > -obj-$(CONFIG_DRM_ARCPGU) += arcpgu.o > diff --git a/drivers/gpu/drm/tiny/Kconfig b/drivers/gpu/drm/tiny/Kconfig > index 2b6414f0fa75..9bbaa1a69050 100644 > --- a/drivers/gpu/drm/tiny/Kconfig > +++ b/drivers/gpu/drm/tiny/Kconfig > @@ -1,5 +1,15 @@ > # SPDX-License-Identifier: GPL-2.0-only > > +config DRM_ARCPGU > + tristate "ARC PGU" > + depends on DRM && OF > + select DRM_KMS_CMA_HELPER > + select DRM_KMS_HELPER > + help > + Choose this option if you have an ARC PGU controller. > + > + If M is selected the module will be called arcpgu. > + > config DRM_CIRRUS_QEMU > tristate "Cirrus driver for QEMU emulated device" > depends on DRM && PCI && MMU > diff --git a/drivers/gpu/drm/tiny/Makefile b/drivers/gpu/drm/tiny/Makefile > index 6ae4e9e5a35f..bef6780bdd6f 100644 > --- a/drivers/gpu/drm/tiny/Makefile > +++ b/drivers/gpu/drm/tiny/Makefile > @@ -1,5 +1,6 @@ > # SPDX-License-Identifier: GPL-2.0-only > > +obj-$(CONFIG_DRM_ARCPGU) += arcpgu.o > obj-$(CONFIG_DRM_CIRRUS_QEMU) += cirrus.o > obj-$(CONFIG_DRM_GM12U320) += gm12u320.o > obj-$(CONFIG_TINYDRM_HX8357D) += hx8357d.o > diff --git a/drivers/gpu/drm/arc/arcpgu_drv.c b/drivers/gpu/drm/tiny/arcpgu.c > similarity index 100% > rename from drivers/gpu/drm/arc/arcpgu_drv.c > rename to drivers/gpu/drm/tiny/arcpgu.c > -- > 2.25.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* RE: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-04-15 9:45 ` Sam Ravnborg @ 2020-04-15 12:02 ` Alexey Brodkin 2020-04-15 12:20 ` Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Alexey Brodkin @ 2020-04-15 12:02 UTC (permalink / raw) To: Daniel Vetter Cc: Sam Ravnborg, Daniel Vetter, Intel Graphics Development, Eugeniy Paltsev, DRI Development Hi Daniel, > -----Original Message----- > From: Sam Ravnborg <sam@ravnborg.org> > Sent: Wednesday, April 15, 2020 12:45 PM > To: Daniel Vetter <daniel.vetter@ffwll.ch> > Cc: Intel Graphics Development <intel-gfx@lists.freedesktop.org>; Alexey Brodkin > <abrodkin@synopsys.com>; DRI Development <dri-devel@lists.freedesktop.org>; Daniel Vetter > <daniel.vetter@intel.com> > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > Hi Daniel. > On Wed, Apr 15, 2020 at 09:40:28AM +0200, Daniel Vetter wrote: > > Because it is. > > > > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> > > Cc: Alexey Brodkin <abrodkin@synopsys.com> > > --- > > MAINTAINERS | 2 +- > > drivers/gpu/drm/Kconfig | 2 -- > > drivers/gpu/drm/Makefile | 1 - > > drivers/gpu/drm/arc/Kconfig | 10 ---------- > > drivers/gpu/drm/arc/Makefile | 3 --- > > drivers/gpu/drm/tiny/Kconfig | 10 ++++++++++ > > drivers/gpu/drm/tiny/Makefile | 1 + > > drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} | 0 > > 8 files changed, 12 insertions(+), 17 deletions(-) > > delete mode 100644 drivers/gpu/drm/arc/Kconfig > > delete mode 100644 drivers/gpu/drm/arc/Makefile > > rename drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} (100%) > > We have "DRM: ARC: add HDMI 2.0 TX encoder support" which > adds another platform driver to drm/arc/ > This speaks against the move to tiny IMO Indeed that's an interesting question, see v3 series here: https://lists.freedesktop.org/archives/dri-devel/2020-April/262352.html BTW should I pull that series in my tree and send you a pull-request or that kind of change needs to go through another tree? Also I'd like to test the change we discuss here to make sure stuff still works. Once we do that I'll send an update. Any hint on when that change needs to be acked/nacked? -Alexey _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-04-15 12:02 ` Alexey Brodkin @ 2020-04-15 12:20 ` Daniel Vetter 2020-04-28 14:08 ` Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Daniel Vetter @ 2020-04-15 12:20 UTC (permalink / raw) To: Alexey Brodkin Cc: Sam Ravnborg, Daniel Vetter, Intel Graphics Development, Eugeniy Paltsev, DRI Development On Wed, Apr 15, 2020 at 2:03 PM Alexey Brodkin <Alexey.Brodkin@synopsys.com> wrote: > > Hi Daniel, > > > -----Original Message----- > > From: Sam Ravnborg <sam@ravnborg.org> > > Sent: Wednesday, April 15, 2020 12:45 PM > > To: Daniel Vetter <daniel.vetter@ffwll.ch> > > Cc: Intel Graphics Development <intel-gfx@lists.freedesktop.org>; Alexey Brodkin > > <abrodkin@synopsys.com>; DRI Development <dri-devel@lists.freedesktop.org>; Daniel Vetter > > <daniel.vetter@intel.com> > > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > > > Hi Daniel. > > On Wed, Apr 15, 2020 at 09:40:28AM +0200, Daniel Vetter wrote: > > > Because it is. > > > > > > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> > > > Cc: Alexey Brodkin <abrodkin@synopsys.com> > > > --- > > > MAINTAINERS | 2 +- > > > drivers/gpu/drm/Kconfig | 2 -- > > > drivers/gpu/drm/Makefile | 1 - > > > drivers/gpu/drm/arc/Kconfig | 10 ---------- > > > drivers/gpu/drm/arc/Makefile | 3 --- > > > drivers/gpu/drm/tiny/Kconfig | 10 ++++++++++ > > > drivers/gpu/drm/tiny/Makefile | 1 + > > > drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} | 0 > > > 8 files changed, 12 insertions(+), 17 deletions(-) > > > delete mode 100644 drivers/gpu/drm/arc/Kconfig > > > delete mode 100644 drivers/gpu/drm/arc/Makefile > > > rename drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} (100%) > > > > We have "DRM: ARC: add HDMI 2.0 TX encoder support" which > > adds another platform driver to drm/arc/ > > This speaks against the move to tiny IMO > > Indeed that's an interesting question, see v3 series here: > https://lists.freedesktop.org/archives/dri-devel/2020-April/262352.html Looking at this patch series, feels a bit like hand-rolling of bridge code, badly. We should get away from that. Once you have that I think the end result is tiny enough that it can stay, bridges intergrate quite well into simple display pipe drivers. > BTW should I pull that series in my tree and send you a pull-request > or that kind of change needs to go through another tree? > > Also I'd like to test the change we discuss here to make sure stuff > still works. Once we do that I'll send an update. Any hint on > when that change needs to be acked/nacked? Simplest is if this can all land through drm-misc, is arc not maintained in there? And there's plenty of time for testing, I'm just slowly crawling through the tree to get everything polished and cleaned up in this area. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-04-15 12:20 ` Daniel Vetter @ 2020-04-28 14:08 ` Daniel Vetter 2020-05-08 13:56 ` Alexey Brodkin 0 siblings, 1 reply; 20+ messages in thread From: Daniel Vetter @ 2020-04-28 14:08 UTC (permalink / raw) To: Alexey Brodkin Cc: Sam Ravnborg, Daniel Vetter, Intel Graphics Development, Eugeniy Paltsev, DRI Development On Wed, Apr 15, 2020 at 02:20:35PM +0200, Daniel Vetter wrote: > On Wed, Apr 15, 2020 at 2:03 PM Alexey Brodkin > <Alexey.Brodkin@synopsys.com> wrote: > > > > Hi Daniel, > > > > > -----Original Message----- > > > From: Sam Ravnborg <sam@ravnborg.org> > > > Sent: Wednesday, April 15, 2020 12:45 PM > > > To: Daniel Vetter <daniel.vetter@ffwll.ch> > > > Cc: Intel Graphics Development <intel-gfx@lists.freedesktop.org>; Alexey Brodkin > > > <abrodkin@synopsys.com>; DRI Development <dri-devel@lists.freedesktop.org>; Daniel Vetter > > > <daniel.vetter@intel.com> > > > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > > > > > Hi Daniel. > > > On Wed, Apr 15, 2020 at 09:40:28AM +0200, Daniel Vetter wrote: > > > > Because it is. > > > > > > > > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> > > > > Cc: Alexey Brodkin <abrodkin@synopsys.com> > > > > --- > > > > MAINTAINERS | 2 +- > > > > drivers/gpu/drm/Kconfig | 2 -- > > > > drivers/gpu/drm/Makefile | 1 - > > > > drivers/gpu/drm/arc/Kconfig | 10 ---------- > > > > drivers/gpu/drm/arc/Makefile | 3 --- > > > > drivers/gpu/drm/tiny/Kconfig | 10 ++++++++++ > > > > drivers/gpu/drm/tiny/Makefile | 1 + > > > > drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} | 0 > > > > 8 files changed, 12 insertions(+), 17 deletions(-) > > > > delete mode 100644 drivers/gpu/drm/arc/Kconfig > > > > delete mode 100644 drivers/gpu/drm/arc/Makefile > > > > rename drivers/gpu/drm/{arc/arcpgu_drv.c => tiny/arcpgu.c} (100%) > > > > > > We have "DRM: ARC: add HDMI 2.0 TX encoder support" which > > > adds another platform driver to drm/arc/ > > > This speaks against the move to tiny IMO > > > > Indeed that's an interesting question, see v3 series here: > > https://lists.freedesktop.org/archives/dri-devel/2020-April/262352.html > > Looking at this patch series, feels a bit like hand-rolling of bridge > code, badly. We should get away from that. > > Once you have that I think the end result is tiny enough that it can > stay, bridges intergrate quite well into simple display pipe drivers. > > > BTW should I pull that series in my tree and send you a pull-request > > or that kind of change needs to go through another tree? > > > > Also I'd like to test the change we discuss here to make sure stuff > > still works. Once we do that I'll send an update. Any hint on > > when that change needs to be acked/nacked? > > Simplest is if this can all land through drm-misc, is arc not > maintained in there? And there's plenty of time for testing, I'm just > slowly crawling through the tree to get everything polished and > cleaned up in this area. Any updates on testing this pile here? First patch landed now, and I've started to push driver patches. So would be good to get this sorted out too. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* RE: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-04-28 14:08 ` Daniel Vetter @ 2020-05-08 13:56 ` Alexey Brodkin 2020-05-08 18:07 ` Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Alexey Brodkin @ 2020-05-08 13:56 UTC (permalink / raw) To: Daniel Vetter Cc: Sam Ravnborg, Daniel Vetter, Intel Graphics Development, Eugeniy Paltsev, DRI Development Hi Daniel, > > Looking at this patch series, feels a bit like hand-rolling of bridge > > code, badly. We should get away from that. > > > > Once you have that I think the end result is tiny enough that it can > > stay, bridges intergrate quite well into simple display pipe drivers. > > > > > BTW should I pull that series in my tree and send you a pull-request > > > or that kind of change needs to go through another tree? > > > > > > Also I'd like to test the change we discuss here to make sure stuff > > > still works. Once we do that I'll send an update. Any hint on > > > when that change needs to be acked/nacked? > > > > Simplest is if this can all land through drm-misc, is arc not > > maintained in there? And there's plenty of time for testing, I'm just > > slowly crawling through the tree to get everything polished and > > cleaned up in this area. > > Any updates on testing this pile here? First patch landed now, and I've > started to push driver patches. So would be good to get this sorted out > too. Sorry we're in the middle of 2 long weekends so missed this one. I guess we'll be able to test it in a week or two from now. Is that OK? -Alexey _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-05-08 13:56 ` Alexey Brodkin @ 2020-05-08 18:07 ` Daniel Vetter 2020-06-04 8:05 ` Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Daniel Vetter @ 2020-05-08 18:07 UTC (permalink / raw) To: Alexey Brodkin Cc: Sam Ravnborg, Daniel Vetter, Intel Graphics Development, Eugeniy Paltsev, DRI Development On Fri, May 8, 2020 at 3:56 PM Alexey Brodkin <Alexey.Brodkin@synopsys.com> wrote: > > Hi Daniel, > > > > Looking at this patch series, feels a bit like hand-rolling of bridge > > > code, badly. We should get away from that. > > > > > > Once you have that I think the end result is tiny enough that it can > > > stay, bridges intergrate quite well into simple display pipe drivers. > > > > > > > BTW should I pull that series in my tree and send you a pull-request > > > > or that kind of change needs to go through another tree? > > > > > > > > Also I'd like to test the change we discuss here to make sure stuff > > > > still works. Once we do that I'll send an update. Any hint on > > > > when that change needs to be acked/nacked? > > > > > > Simplest is if this can all land through drm-misc, is arc not > > > maintained in there? And there's plenty of time for testing, I'm just > > > slowly crawling through the tree to get everything polished and > > > cleaned up in this area. > > > > Any updates on testing this pile here? First patch landed now, and I've > > started to push driver patches. So would be good to get this sorted out > > too. > > Sorry we're in the middle of 2 long weekends so missed this one. > I guess we'll be able to test it in a week or two from now. > > Is that OK? This aren't high-priority, so totally ok. As long as you don't land a driver rewrite and I have to rebase everything :-) -Daniel > > -Alexey > > -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-05-08 18:07 ` Daniel Vetter @ 2020-06-04 8:05 ` Daniel Vetter 2020-06-04 10:38 ` Eugeniy Paltsev 0 siblings, 1 reply; 20+ messages in thread From: Daniel Vetter @ 2020-06-04 8:05 UTC (permalink / raw) To: Alexey Brodkin Cc: Sam Ravnborg, Daniel Vetter, Intel Graphics Development, Eugeniy Paltsev, DRI Development On Fri, May 08, 2020 at 08:07:41PM +0200, Daniel Vetter wrote: > On Fri, May 8, 2020 at 3:56 PM Alexey Brodkin > <Alexey.Brodkin@synopsys.com> wrote: > > > > Hi Daniel, > > > > > > Looking at this patch series, feels a bit like hand-rolling of bridge > > > > code, badly. We should get away from that. > > > > > > > > Once you have that I think the end result is tiny enough that it can > > > > stay, bridges intergrate quite well into simple display pipe drivers. > > > > > > > > > BTW should I pull that series in my tree and send you a pull-request > > > > > or that kind of change needs to go through another tree? > > > > > > > > > > Also I'd like to test the change we discuss here to make sure stuff > > > > > still works. Once we do that I'll send an update. Any hint on > > > > > when that change needs to be acked/nacked? > > > > > > > > Simplest is if this can all land through drm-misc, is arc not > > > > maintained in there? And there's plenty of time for testing, I'm just > > > > slowly crawling through the tree to get everything polished and > > > > cleaned up in this area. > > > > > > Any updates on testing this pile here? First patch landed now, and I've > > > started to push driver patches. So would be good to get this sorted out > > > too. > > > > Sorry we're in the middle of 2 long weekends so missed this one. > > I guess we'll be able to test it in a week or two from now. > > > > Is that OK? > > This aren't high-priority, so totally ok. As long as you don't land a > driver rewrite and I have to rebase everything :-) Ping for a bit of testing on this stuff ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-06-04 8:05 ` Daniel Vetter @ 2020-06-04 10:38 ` Eugeniy Paltsev 2020-06-04 11:19 ` Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Eugeniy Paltsev @ 2020-06-04 10:38 UTC (permalink / raw) To: Daniel Vetter Cc: Daniel Vetter, Intel Graphics Development, Alexey Brodkin, Sam Ravnborg, DRI Development Hi Daniel, I've already tested it (and found several issues), so please check my reply here: https://www.mail-archive.com/linux-snps-arc@lists.infradead.org/msg07403.html Not sure why you didn't receive my reply (probably the reason is because it was sent to your @ffwll.ch mail instead of @intel.com one). ________________________________________ From: Daniel Vetter <daniel@ffwll.ch> Sent: Thursday, June 4, 2020 11:05 To: Alexey Brodkin Cc: Intel Graphics Development; DRI Development; Daniel Vetter; Eugeniy Paltsev; Sam Ravnborg Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny On Fri, May 08, 2020 at 08:07:41PM +0200, Daniel Vetter wrote: > On Fri, May 8, 2020 at 3:56 PM Alexey Brodkin > <Alexey.Brodkin@synopsys.com> wrote: > > > > Hi Daniel, > > > > > > Looking at this patch series, feels a bit like hand-rolling of bridge > > > > code, badly. We should get away from that. > > > > > > > > Once you have that I think the end result is tiny enough that it can > > > > stay, bridges intergrate quite well into simple display pipe drivers. > > > > > > > > > BTW should I pull that series in my tree and send you a pull-request > > > > > or that kind of change needs to go through another tree? > > > > > > > > > > Also I'd like to test the change we discuss here to make sure stuff > > > > > still works. Once we do that I'll send an update. Any hint on > > > > > when that change needs to be acked/nacked? > > > > > > > > Simplest is if this can all land through drm-misc, is arc not > > > > maintained in there? And there's plenty of time for testing, I'm just > > > > slowly crawling through the tree to get everything polished and > > > > cleaned up in this area. > > > > > > Any updates on testing this pile here? First patch landed now, and I've > > > started to push driver patches. So would be good to get this sorted out > > > too. > > > > Sorry we're in the middle of 2 long weekends so missed this one. > > I guess we'll be able to test it in a week or two from now. > > > > Is that OK? > > This aren't high-priority, so totally ok. As long as you don't land a > driver rewrite and I have to rebase everything :-) Ping for a bit of testing on this stuff ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!Ncpf9M5g5wUEicELHfzz8syA0c0KogYc2E0tdnXGHGmUwGbROv-vwMDISCh7u6w58Dgs-ws$ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-06-04 10:38 ` Eugeniy Paltsev @ 2020-06-04 11:19 ` Daniel Vetter 2020-06-04 19:00 ` Eugeniy Paltsev 0 siblings, 1 reply; 20+ messages in thread From: Daniel Vetter @ 2020-06-04 11:19 UTC (permalink / raw) To: Eugeniy Paltsev Cc: Daniel Vetter, Intel Graphics Development, Alexey Brodkin, Sam Ravnborg, DRI Development Hi Eugeniy, Apologies, somehow I missed your mail. I looked at the code again, and I think I fumbled something. Does the below diff help to prevent the issues? Thanks, Daniel diff --git a/drivers/gpu/drm/tiny/arcpgu.c b/drivers/gpu/drm/tiny/arcpgu.c index 857812f25bec..33d812a5ad7f 100644 --- a/drivers/gpu/drm/tiny/arcpgu.c +++ b/drivers/gpu/drm/tiny/arcpgu.c @@ -228,6 +228,9 @@ static void arc_pgu_update(struct drm_simple_display_pipe *pipe, struct arcpgu_drm_private *arcpgu; struct drm_gem_cma_object *gem; + if (!pipe->plane.state->fb) + return; + arcpgu = pipe_to_arcpgu_priv(pipe); gem = drm_fb_cma_get_gem_obj(pipe->plane.state->fb, 0); arc_pgu_write(arcpgu, ARCPGU_REG_BUF0_ADDR, gem->paddr); -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-06-04 11:19 ` Daniel Vetter @ 2020-06-04 19:00 ` Eugeniy Paltsev 2020-06-05 19:55 ` Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Eugeniy Paltsev @ 2020-06-04 19:00 UTC (permalink / raw) To: Daniel Vetter Cc: Intel Graphics Development, Alexey Brodkin, DRI Development, Daniel Vetter, Sam Ravnborg, snps-arc I've tested your change and one issue gone. However I still see kernel crash (due to invalid read in kernel mode by 0x0 address) on weston stop: ----------------------------------->8------------------------------------------- Oops Path: (null) CPU: 0 PID: 12 Comm: kworker/0:1 Not tainted 5.7.0-rc6-01594-g4ceda91a4176-dirty #6 Workqueue: events drm_mode_rmfb_work_fn Invalid Read @ 0x00000000 by insn @ drm_gem_fb_destroy+0x32/0x130 ECR: 0x00050100 EFA: 0x00000000 ERET: 0x813b9a76 STAT32: 0x80080602 [IE K ] BTA: 0x813b9a72 BLK: drm_gem_fb_destroy+0xc0/0x130 SP: 0x9f055ea4 FP: 0x00000000 LPS: 0x813560ec LPE: 0x813560f0 LPC: 0x00000000 r00: 0x00000000 r01: 0x9f6a6100 r02: 0x00000001 r03: 0x9fd5dde8 r04: 0x810f5de8 r05: 0x00000000 r06: 0x00000000 r07: 0x00000000 r08: 0x000000e1 r09: 0x00000000 r10: 0x00000000 r11: 0x000000e1 r12: 0x813b9b04 Stack Trace: drm_gem_fb_destroy+0x32/0x130 drm_framebuffer_remove+0x1d2/0x358 drm_mode_rmfb_work_fn+0x28/0x38 process_one_work+0x19a/0x358 worker_thread+0x2c4/0x494 kthread+0xec/0x100 ret_from_fork+0x18/0x1c ----------------------------------->8------------------------------------------- The stack traces may vary but always end in drm_gem_fb_destroy: ----------------------------------->8------------------------------------------- Stack Trace: drm_gem_fb_destroy+0x32/0x130 drm_mode_rmfb+0x10e/0x148 drm_ioctl_kernel+0x70/0xa0 drm_ioctl+0x284/0x410 ksys_ioctl+0xea/0xa3c EV_Trap+0xcc/0xd0 ----------------------------------->8------------------------------------------- Stack Trace: drm_gem_fb_destroy+0x32/0x130 drm_fb_release+0x66/0xb0 drm_file_free.part.11+0x112/0x1bc drm_release+0x80/0x120 __fput+0x98/0x1bc task_work_run+0x6e/0xa8 do_exit+0x2b4/0x7fc do_group_exit+0x2a/0x8c get_signal+0x9a/0x5f0 do_signal+0x86/0x23c resume_user_mode_begin+0x88/0xd0 ----------------------------------->8------------------------------------------- --- Eugeniy Paltsev ________________________________________ From: Daniel Vetter <daniel@ffwll.ch> Sent: Thursday, June 4, 2020 14:19 To: Eugeniy Paltsev Cc: Intel Graphics Development; DRI Development; Daniel Vetter; Sam Ravnborg; Alexey Brodkin Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny Hi Eugeniy, Apologies, somehow I missed your mail. I looked at the code again, and I think I fumbled something. Does the below diff help to prevent the issues? Thanks, Daniel diff --git a/drivers/gpu/drm/tiny/arcpgu.c b/drivers/gpu/drm/tiny/arcpgu.c index 857812f25bec..33d812a5ad7f 100644 --- a/drivers/gpu/drm/tiny/arcpgu.c +++ b/drivers/gpu/drm/tiny/arcpgu.c @@ -228,6 +228,9 @@ static void arc_pgu_update(struct drm_simple_display_pipe *pipe, struct arcpgu_drm_private *arcpgu; struct drm_gem_cma_object *gem; + if (!pipe->plane.state->fb) + return; + arcpgu = pipe_to_arcpgu_priv(pipe); gem = drm_fb_cma_get_gem_obj(pipe->plane.state->fb, 0); arc_pgu_write(arcpgu, ARCPGU_REG_BUF0_ADDR, gem->paddr); -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!P0EvyJfMuDwqbeZmHZM5S9po30QWr4KgGrggRirNfgo7wrRXfnUO-8iq0AA4fQCW2WGPlDc$ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-06-04 19:00 ` Eugeniy Paltsev @ 2020-06-05 19:55 ` Daniel Vetter 2020-06-09 12:08 ` Eugeniy Paltsev 0 siblings, 1 reply; 20+ messages in thread From: Daniel Vetter @ 2020-06-05 19:55 UTC (permalink / raw) To: Eugeniy Paltsev Cc: Intel Graphics Development, Alexey Brodkin, DRI Development, Daniel Vetter, Sam Ravnborg, snps-arc Hi Eugeniy, Thanks for testing. I looked at the second one (I hoped it would just magically disappear) and I still don't understand what's going on there. My patch series isn't touching that area at all, so really confused. I squashed in the bugfix from the previous round into the right patches, and pushed a branch with just the arcpgu changes here: https://cgit.freedesktop.org/~danvet/drm/log/?h=for-eugeniy Maybe it's something in my pile of not-so-tested stuff :-) Can you pls test this? And if it still fails, try to bisect where it breaks? Thanks, Daniel On Thu, Jun 4, 2020 at 9:00 PM Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> wrote: > > I've tested your change and one issue gone. > > However I still see kernel crash (due to invalid read in kernel mode by 0x0 address) on weston stop: > ----------------------------------->8------------------------------------------- > Oops > Path: (null) > CPU: 0 PID: 12 Comm: kworker/0:1 Not tainted 5.7.0-rc6-01594-g4ceda91a4176-dirty #6 > Workqueue: events drm_mode_rmfb_work_fn > Invalid Read @ 0x00000000 by insn @ drm_gem_fb_destroy+0x32/0x130 > ECR: 0x00050100 EFA: 0x00000000 ERET: 0x813b9a76 > STAT32: 0x80080602 [IE K ] BTA: 0x813b9a72 > BLK: drm_gem_fb_destroy+0xc0/0x130 > SP: 0x9f055ea4 FP: 0x00000000 > LPS: 0x813560ec LPE: 0x813560f0 LPC: 0x00000000 > r00: 0x00000000 r01: 0x9f6a6100 r02: 0x00000001 > r03: 0x9fd5dde8 r04: 0x810f5de8 r05: 0x00000000 > r06: 0x00000000 r07: 0x00000000 r08: 0x000000e1 > r09: 0x00000000 r10: 0x00000000 r11: 0x000000e1 > r12: 0x813b9b04 > > Stack Trace: > drm_gem_fb_destroy+0x32/0x130 > drm_framebuffer_remove+0x1d2/0x358 > drm_mode_rmfb_work_fn+0x28/0x38 > process_one_work+0x19a/0x358 > worker_thread+0x2c4/0x494 > kthread+0xec/0x100 > ret_from_fork+0x18/0x1c > ----------------------------------->8------------------------------------------- > > > The stack traces may vary but always end in drm_gem_fb_destroy: > ----------------------------------->8------------------------------------------- > Stack Trace: > drm_gem_fb_destroy+0x32/0x130 > drm_mode_rmfb+0x10e/0x148 > drm_ioctl_kernel+0x70/0xa0 > drm_ioctl+0x284/0x410 > ksys_ioctl+0xea/0xa3c > EV_Trap+0xcc/0xd0 > ----------------------------------->8------------------------------------------- > Stack Trace: > drm_gem_fb_destroy+0x32/0x130 > drm_fb_release+0x66/0xb0 > drm_file_free.part.11+0x112/0x1bc > drm_release+0x80/0x120 > __fput+0x98/0x1bc > task_work_run+0x6e/0xa8 > do_exit+0x2b4/0x7fc > do_group_exit+0x2a/0x8c > get_signal+0x9a/0x5f0 > do_signal+0x86/0x23c > resume_user_mode_begin+0x88/0xd0 > ----------------------------------->8------------------------------------------- > > > --- > Eugeniy Paltsev > > > ________________________________________ > From: Daniel Vetter <daniel@ffwll.ch> > Sent: Thursday, June 4, 2020 14:19 > To: Eugeniy Paltsev > Cc: Intel Graphics Development; DRI Development; Daniel Vetter; Sam Ravnborg; Alexey Brodkin > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > Hi Eugeniy, > > Apologies, somehow I missed your mail. I looked at the code again, and I > think I fumbled something. Does the below diff help to prevent the issues? > > Thanks, Daniel > > > diff --git a/drivers/gpu/drm/tiny/arcpgu.c b/drivers/gpu/drm/tiny/arcpgu.c > index 857812f25bec..33d812a5ad7f 100644 > --- a/drivers/gpu/drm/tiny/arcpgu.c > +++ b/drivers/gpu/drm/tiny/arcpgu.c > @@ -228,6 +228,9 @@ static void arc_pgu_update(struct drm_simple_display_pipe *pipe, > struct arcpgu_drm_private *arcpgu; > struct drm_gem_cma_object *gem; > > + if (!pipe->plane.state->fb) > + return; > + > arcpgu = pipe_to_arcpgu_priv(pipe); > gem = drm_fb_cma_get_gem_obj(pipe->plane.state->fb, 0); > arc_pgu_write(arcpgu, ARCPGU_REG_BUF0_ADDR, gem->paddr); > -- > Daniel Vetter > Software Engineer, Intel Corporation > +41 (0) 79 365 57 48 - https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!P0EvyJfMuDwqbeZmHZM5S9po30QWr4KgGrggRirNfgo7wrRXfnUO-8iq0AA4fQCW2WGPlDc$ -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-06-05 19:55 ` Daniel Vetter @ 2020-06-09 12:08 ` Eugeniy Paltsev 2020-06-09 13:02 ` Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Eugeniy Paltsev @ 2020-06-09 12:08 UTC (permalink / raw) To: Daniel Vetter Cc: Intel Graphics Development, Alexey Brodkin, DRI Development, Daniel Vetter, Sam Ravnborg, snps-arc Hi Daniel, I've got pretty strange results so I need some time to investigate it and probably retest. I'll send you update in a few days. --- Eugeniy Paltsev ________________________________________ From: Daniel Vetter <daniel@ffwll.ch> Sent: Friday, June 5, 2020 22:55 To: Eugeniy Paltsev Cc: Intel Graphics Development; DRI Development; Daniel Vetter; Sam Ravnborg; Alexey Brodkin; snps-arc@lists.infradead.org Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny Hi Eugeniy, Thanks for testing. I looked at the second one (I hoped it would just magically disappear) and I still don't understand what's going on there. My patch series isn't touching that area at all, so really confused. I squashed in the bugfix from the previous round into the right patches, and pushed a branch with just the arcpgu changes here: https://urldefense.com/v3/__https://cgit.freedesktop.org/*danvet/drm/log/?h=for-eugeniy__;fg!!A4F2R9G_pg!IJ1o4XiXVdStPu--Q-SCTUpRbsbqrjX255R34nuD7L7ptPywOy4SKr21dwSpfOkXIVqH5pM$ Maybe it's something in my pile of not-so-tested stuff :-) Can you pls test this? And if it still fails, try to bisect where it breaks? Thanks, Daniel On Thu, Jun 4, 2020 at 9:00 PM Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> wrote: > > I've tested your change and one issue gone. > > However I still see kernel crash (due to invalid read in kernel mode by 0x0 address) on weston stop: > ----------------------------------->8------------------------------------------- > Oops > Path: (null) > CPU: 0 PID: 12 Comm: kworker/0:1 Not tainted 5.7.0-rc6-01594-g4ceda91a4176-dirty #6 > Workqueue: events drm_mode_rmfb_work_fn > Invalid Read @ 0x00000000 by insn @ drm_gem_fb_destroy+0x32/0x130 > ECR: 0x00050100 EFA: 0x00000000 ERET: 0x813b9a76 > STAT32: 0x80080602 [IE K ] BTA: 0x813b9a72 > BLK: drm_gem_fb_destroy+0xc0/0x130 > SP: 0x9f055ea4 FP: 0x00000000 > LPS: 0x813560ec LPE: 0x813560f0 LPC: 0x00000000 > r00: 0x00000000 r01: 0x9f6a6100 r02: 0x00000001 > r03: 0x9fd5dde8 r04: 0x810f5de8 r05: 0x00000000 > r06: 0x00000000 r07: 0x00000000 r08: 0x000000e1 > r09: 0x00000000 r10: 0x00000000 r11: 0x000000e1 > r12: 0x813b9b04 > > Stack Trace: > drm_gem_fb_destroy+0x32/0x130 > drm_framebuffer_remove+0x1d2/0x358 > drm_mode_rmfb_work_fn+0x28/0x38 > process_one_work+0x19a/0x358 > worker_thread+0x2c4/0x494 > kthread+0xec/0x100 > ret_from_fork+0x18/0x1c > ----------------------------------->8------------------------------------------- > > > The stack traces may vary but always end in drm_gem_fb_destroy: > ----------------------------------->8------------------------------------------- > Stack Trace: > drm_gem_fb_destroy+0x32/0x130 > drm_mode_rmfb+0x10e/0x148 > drm_ioctl_kernel+0x70/0xa0 > drm_ioctl+0x284/0x410 > ksys_ioctl+0xea/0xa3c > EV_Trap+0xcc/0xd0 > ----------------------------------->8------------------------------------------- > Stack Trace: > drm_gem_fb_destroy+0x32/0x130 > drm_fb_release+0x66/0xb0 > drm_file_free.part.11+0x112/0x1bc > drm_release+0x80/0x120 > __fput+0x98/0x1bc > task_work_run+0x6e/0xa8 > do_exit+0x2b4/0x7fc > do_group_exit+0x2a/0x8c > get_signal+0x9a/0x5f0 > do_signal+0x86/0x23c > resume_user_mode_begin+0x88/0xd0 > ----------------------------------->8------------------------------------------- > > > --- > Eugeniy Paltsev > > > ________________________________________ > From: Daniel Vetter <daniel@ffwll.ch> > Sent: Thursday, June 4, 2020 14:19 > To: Eugeniy Paltsev > Cc: Intel Graphics Development; DRI Development; Daniel Vetter; Sam Ravnborg; Alexey Brodkin > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > Hi Eugeniy, > > Apologies, somehow I missed your mail. I looked at the code again, and I > think I fumbled something. Does the below diff help to prevent the issues? > > Thanks, Daniel > > > diff --git a/drivers/gpu/drm/tiny/arcpgu.c b/drivers/gpu/drm/tiny/arcpgu.c > index 857812f25bec..33d812a5ad7f 100644 > --- a/drivers/gpu/drm/tiny/arcpgu.c > +++ b/drivers/gpu/drm/tiny/arcpgu.c > @@ -228,6 +228,9 @@ static void arc_pgu_update(struct drm_simple_display_pipe *pipe, > struct arcpgu_drm_private *arcpgu; > struct drm_gem_cma_object *gem; > > + if (!pipe->plane.state->fb) > + return; > + > arcpgu = pipe_to_arcpgu_priv(pipe); > gem = drm_fb_cma_get_gem_obj(pipe->plane.state->fb, 0); > arc_pgu_write(arcpgu, ARCPGU_REG_BUF0_ADDR, gem->paddr); > -- > Daniel Vetter > Software Engineer, Intel Corporation > +41 (0) 79 365 57 48 - https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!P0EvyJfMuDwqbeZmHZM5S9po30QWr4KgGrggRirNfgo7wrRXfnUO-8iq0AA4fQCW2WGPlDc$ -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!IJ1o4XiXVdStPu--Q-SCTUpRbsbqrjX255R34nuD7L7ptPywOy4SKr21dwSpfOkXpn86Q20$ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-06-09 12:08 ` Eugeniy Paltsev @ 2020-06-09 13:02 ` Daniel Vetter 2020-07-17 9:04 ` Daniel Vetter 0 siblings, 1 reply; 20+ messages in thread From: Daniel Vetter @ 2020-06-09 13:02 UTC (permalink / raw) To: Eugeniy Paltsev Cc: Intel Graphics Development, Alexey Brodkin, DRI Development, Daniel Vetter, Sam Ravnborg, snps-arc Hi Eugeniy, Very much appreciated, and kinda expected. That 2nd backtrace really confuses me, so "something strange is going on" and the bisect looks funny is within expectations. Hopefully we can track down what's going on. Thanks, Daniel On Tue, Jun 9, 2020 at 2:08 PM Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> wrote: > > Hi Daniel, > > I've got pretty strange results so I need some time to investigate it and probably retest. > I'll send you update in a few days. > > --- > Eugeniy Paltsev > > > ________________________________________ > From: Daniel Vetter <daniel@ffwll.ch> > Sent: Friday, June 5, 2020 22:55 > To: Eugeniy Paltsev > Cc: Intel Graphics Development; DRI Development; Daniel Vetter; Sam Ravnborg; Alexey Brodkin; snps-arc@lists.infradead.org > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > Hi Eugeniy, > > Thanks for testing. I looked at the second one (I hoped it would just > magically disappear) and I still don't understand what's going on > there. My patch series isn't touching that area at all, so really > confused. > > I squashed in the bugfix from the previous round into the right > patches, and pushed a branch with just the arcpgu changes here: > https://urldefense.com/v3/__https://cgit.freedesktop.org/*danvet/drm/log/?h=for-eugeniy__;fg!!A4F2R9G_pg!IJ1o4XiXVdStPu--Q-SCTUpRbsbqrjX255R34nuD7L7ptPywOy4SKr21dwSpfOkXIVqH5pM$ > > Maybe it's something in my pile of not-so-tested stuff :-) > > Can you pls test this? And if it still fails, try to bisect where it breaks? > > Thanks, Daniel > > On Thu, Jun 4, 2020 at 9:00 PM Eugeniy Paltsev > <Eugeniy.Paltsev@synopsys.com> wrote: > > > > I've tested your change and one issue gone. > > > > However I still see kernel crash (due to invalid read in kernel mode by 0x0 address) on weston stop: > > ----------------------------------->8------------------------------------------- > > Oops > > Path: (null) > > CPU: 0 PID: 12 Comm: kworker/0:1 Not tainted 5.7.0-rc6-01594-g4ceda91a4176-dirty #6 > > Workqueue: events drm_mode_rmfb_work_fn > > Invalid Read @ 0x00000000 by insn @ drm_gem_fb_destroy+0x32/0x130 > > ECR: 0x00050100 EFA: 0x00000000 ERET: 0x813b9a76 > > STAT32: 0x80080602 [IE K ] BTA: 0x813b9a72 > > BLK: drm_gem_fb_destroy+0xc0/0x130 > > SP: 0x9f055ea4 FP: 0x00000000 > > LPS: 0x813560ec LPE: 0x813560f0 LPC: 0x00000000 > > r00: 0x00000000 r01: 0x9f6a6100 r02: 0x00000001 > > r03: 0x9fd5dde8 r04: 0x810f5de8 r05: 0x00000000 > > r06: 0x00000000 r07: 0x00000000 r08: 0x000000e1 > > r09: 0x00000000 r10: 0x00000000 r11: 0x000000e1 > > r12: 0x813b9b04 > > > > Stack Trace: > > drm_gem_fb_destroy+0x32/0x130 > > drm_framebuffer_remove+0x1d2/0x358 > > drm_mode_rmfb_work_fn+0x28/0x38 > > process_one_work+0x19a/0x358 > > worker_thread+0x2c4/0x494 > > kthread+0xec/0x100 > > ret_from_fork+0x18/0x1c > > ----------------------------------->8------------------------------------------- > > > > > > The stack traces may vary but always end in drm_gem_fb_destroy: > > ----------------------------------->8------------------------------------------- > > Stack Trace: > > drm_gem_fb_destroy+0x32/0x130 > > drm_mode_rmfb+0x10e/0x148 > > drm_ioctl_kernel+0x70/0xa0 > > drm_ioctl+0x284/0x410 > > ksys_ioctl+0xea/0xa3c > > EV_Trap+0xcc/0xd0 > > ----------------------------------->8------------------------------------------- > > Stack Trace: > > drm_gem_fb_destroy+0x32/0x130 > > drm_fb_release+0x66/0xb0 > > drm_file_free.part.11+0x112/0x1bc > > drm_release+0x80/0x120 > > __fput+0x98/0x1bc > > task_work_run+0x6e/0xa8 > > do_exit+0x2b4/0x7fc > > do_group_exit+0x2a/0x8c > > get_signal+0x9a/0x5f0 > > do_signal+0x86/0x23c > > resume_user_mode_begin+0x88/0xd0 > > ----------------------------------->8------------------------------------------- > > > > > > --- > > Eugeniy Paltsev > > > > > > ________________________________________ > > From: Daniel Vetter <daniel@ffwll.ch> > > Sent: Thursday, June 4, 2020 14:19 > > To: Eugeniy Paltsev > > Cc: Intel Graphics Development; DRI Development; Daniel Vetter; Sam Ravnborg; Alexey Brodkin > > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > > > Hi Eugeniy, > > > > Apologies, somehow I missed your mail. I looked at the code again, and I > > think I fumbled something. Does the below diff help to prevent the issues? > > > > Thanks, Daniel > > > > > > diff --git a/drivers/gpu/drm/tiny/arcpgu.c b/drivers/gpu/drm/tiny/arcpgu.c > > index 857812f25bec..33d812a5ad7f 100644 > > --- a/drivers/gpu/drm/tiny/arcpgu.c > > +++ b/drivers/gpu/drm/tiny/arcpgu.c > > @@ -228,6 +228,9 @@ static void arc_pgu_update(struct drm_simple_display_pipe *pipe, > > struct arcpgu_drm_private *arcpgu; > > struct drm_gem_cma_object *gem; > > > > + if (!pipe->plane.state->fb) > > + return; > > + > > arcpgu = pipe_to_arcpgu_priv(pipe); > > gem = drm_fb_cma_get_gem_obj(pipe->plane.state->fb, 0); > > arc_pgu_write(arcpgu, ARCPGU_REG_BUF0_ADDR, gem->paddr); > > -- > > Daniel Vetter > > Software Engineer, Intel Corporation > > +41 (0) 79 365 57 48 - https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!P0EvyJfMuDwqbeZmHZM5S9po30QWr4KgGrggRirNfgo7wrRXfnUO-8iq0AA4fQCW2WGPlDc$ > > > > -- > Daniel Vetter > Software Engineer, Intel Corporation > +41 (0) 79 365 57 48 - https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!IJ1o4XiXVdStPu--Q-SCTUpRbsbqrjX255R34nuD7L7ptPywOy4SKr21dwSpfOkXpn86Q20$ -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 53/59] drm/arc: Move to drm/tiny 2020-06-09 13:02 ` Daniel Vetter @ 2020-07-17 9:04 ` Daniel Vetter 0 siblings, 0 replies; 20+ messages in thread From: Daniel Vetter @ 2020-07-17 9:04 UTC (permalink / raw) To: Eugeniy Paltsev Cc: Intel Graphics Development, Alexey Brodkin, DRI Development, Daniel Vetter, Sam Ravnborg, snps-arc On Tue, Jun 09, 2020 at 03:02:14PM +0200, Daniel Vetter wrote: > Hi Eugeniy, > > Very much appreciated, and kinda expected. That 2nd backtrace really > confuses me, so "something strange is going on" and the bisect looks > funny is within expectations. Hopefully we can track down what's going > on. I'm going to resend the entire pile with the bugfix below and all rebased, I think retesting it all is probably a good idea now, since quite some time passed. Cheers, Daniel > > Thanks, Daniel > > On Tue, Jun 9, 2020 at 2:08 PM Eugeniy Paltsev > <Eugeniy.Paltsev@synopsys.com> wrote: > > > > Hi Daniel, > > > > I've got pretty strange results so I need some time to investigate it and probably retest. > > I'll send you update in a few days. > > > > --- > > Eugeniy Paltsev > > > > > > ________________________________________ > > From: Daniel Vetter <daniel@ffwll.ch> > > Sent: Friday, June 5, 2020 22:55 > > To: Eugeniy Paltsev > > Cc: Intel Graphics Development; DRI Development; Daniel Vetter; Sam Ravnborg; Alexey Brodkin; snps-arc@lists.infradead.org > > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > > > Hi Eugeniy, > > > > Thanks for testing. I looked at the second one (I hoped it would just > > magically disappear) and I still don't understand what's going on > > there. My patch series isn't touching that area at all, so really > > confused. > > > > I squashed in the bugfix from the previous round into the right > > patches, and pushed a branch with just the arcpgu changes here: > > https://urldefense.com/v3/__https://cgit.freedesktop.org/*danvet/drm/log/?h=for-eugeniy__;fg!!A4F2R9G_pg!IJ1o4XiXVdStPu--Q-SCTUpRbsbqrjX255R34nuD7L7ptPywOy4SKr21dwSpfOkXIVqH5pM$ > > > > Maybe it's something in my pile of not-so-tested stuff :-) > > > > Can you pls test this? And if it still fails, try to bisect where it breaks? > > > > Thanks, Daniel > > > > On Thu, Jun 4, 2020 at 9:00 PM Eugeniy Paltsev > > <Eugeniy.Paltsev@synopsys.com> wrote: > > > > > > I've tested your change and one issue gone. > > > > > > However I still see kernel crash (due to invalid read in kernel mode by 0x0 address) on weston stop: > > > ----------------------------------->8------------------------------------------- > > > Oops > > > Path: (null) > > > CPU: 0 PID: 12 Comm: kworker/0:1 Not tainted 5.7.0-rc6-01594-g4ceda91a4176-dirty #6 > > > Workqueue: events drm_mode_rmfb_work_fn > > > Invalid Read @ 0x00000000 by insn @ drm_gem_fb_destroy+0x32/0x130 > > > ECR: 0x00050100 EFA: 0x00000000 ERET: 0x813b9a76 > > > STAT32: 0x80080602 [IE K ] BTA: 0x813b9a72 > > > BLK: drm_gem_fb_destroy+0xc0/0x130 > > > SP: 0x9f055ea4 FP: 0x00000000 > > > LPS: 0x813560ec LPE: 0x813560f0 LPC: 0x00000000 > > > r00: 0x00000000 r01: 0x9f6a6100 r02: 0x00000001 > > > r03: 0x9fd5dde8 r04: 0x810f5de8 r05: 0x00000000 > > > r06: 0x00000000 r07: 0x00000000 r08: 0x000000e1 > > > r09: 0x00000000 r10: 0x00000000 r11: 0x000000e1 > > > r12: 0x813b9b04 > > > > > > Stack Trace: > > > drm_gem_fb_destroy+0x32/0x130 > > > drm_framebuffer_remove+0x1d2/0x358 > > > drm_mode_rmfb_work_fn+0x28/0x38 > > > process_one_work+0x19a/0x358 > > > worker_thread+0x2c4/0x494 > > > kthread+0xec/0x100 > > > ret_from_fork+0x18/0x1c > > > ----------------------------------->8------------------------------------------- > > > > > > > > > The stack traces may vary but always end in drm_gem_fb_destroy: > > > ----------------------------------->8------------------------------------------- > > > Stack Trace: > > > drm_gem_fb_destroy+0x32/0x130 > > > drm_mode_rmfb+0x10e/0x148 > > > drm_ioctl_kernel+0x70/0xa0 > > > drm_ioctl+0x284/0x410 > > > ksys_ioctl+0xea/0xa3c > > > EV_Trap+0xcc/0xd0 > > > ----------------------------------->8------------------------------------------- > > > Stack Trace: > > > drm_gem_fb_destroy+0x32/0x130 > > > drm_fb_release+0x66/0xb0 > > > drm_file_free.part.11+0x112/0x1bc > > > drm_release+0x80/0x120 > > > __fput+0x98/0x1bc > > > task_work_run+0x6e/0xa8 > > > do_exit+0x2b4/0x7fc > > > do_group_exit+0x2a/0x8c > > > get_signal+0x9a/0x5f0 > > > do_signal+0x86/0x23c > > > resume_user_mode_begin+0x88/0xd0 > > > ----------------------------------->8------------------------------------------- > > > > > > > > > --- > > > Eugeniy Paltsev > > > > > > > > > ________________________________________ > > > From: Daniel Vetter <daniel@ffwll.ch> > > > Sent: Thursday, June 4, 2020 14:19 > > > To: Eugeniy Paltsev > > > Cc: Intel Graphics Development; DRI Development; Daniel Vetter; Sam Ravnborg; Alexey Brodkin > > > Subject: Re: [PATCH 53/59] drm/arc: Move to drm/tiny > > > > > > Hi Eugeniy, > > > > > > Apologies, somehow I missed your mail. I looked at the code again, and I > > > think I fumbled something. Does the below diff help to prevent the issues? > > > > > > Thanks, Daniel > > > > > > > > > diff --git a/drivers/gpu/drm/tiny/arcpgu.c b/drivers/gpu/drm/tiny/arcpgu.c > > > index 857812f25bec..33d812a5ad7f 100644 > > > --- a/drivers/gpu/drm/tiny/arcpgu.c > > > +++ b/drivers/gpu/drm/tiny/arcpgu.c > > > @@ -228,6 +228,9 @@ static void arc_pgu_update(struct drm_simple_display_pipe *pipe, > > > struct arcpgu_drm_private *arcpgu; > > > struct drm_gem_cma_object *gem; > > > > > > + if (!pipe->plane.state->fb) > > > + return; > > > + > > > arcpgu = pipe_to_arcpgu_priv(pipe); > > > gem = drm_fb_cma_get_gem_obj(pipe->plane.state->fb, 0); > > > arc_pgu_write(arcpgu, ARCPGU_REG_BUF0_ADDR, gem->paddr); > > > -- > > > Daniel Vetter > > > Software Engineer, Intel Corporation > > > +41 (0) 79 365 57 48 - https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!P0EvyJfMuDwqbeZmHZM5S9po30QWr4KgGrggRirNfgo7wrRXfnUO-8iq0AA4fQCW2WGPlDc$ > > > > > > > > -- > > Daniel Vetter > > Software Engineer, Intel Corporation > > +41 (0) 79 365 57 48 - https://urldefense.com/v3/__http://blog.ffwll.ch__;!!A4F2R9G_pg!IJ1o4XiXVdStPu--Q-SCTUpRbsbqrjX255R34nuD7L7ptPywOy4SKr21dwSpfOkXpn86Q20$ > > > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2020-07-17 9:04 UTC | newest] Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-05-25 16:36 [PATCH 53/59] drm/arc: Move to drm/tiny Eugeniy Paltsev 2020-05-25 17:26 ` Daniel Vetter 2020-05-25 21:18 ` Eugeniy Paltsev -- strict thread matches above, loose matches on Subject: below -- 2020-04-15 7:39 [PATCH 00/59] devm_drm_dev_alloc, v2 Daniel Vetter 2020-04-15 7:40 ` [PATCH 53/59] drm/arc: Move to drm/tiny Daniel Vetter 2020-04-15 8:04 ` Thomas Zimmermann 2020-04-15 8:22 ` Daniel Vetter 2020-04-15 9:45 ` Sam Ravnborg 2020-04-15 12:02 ` Alexey Brodkin 2020-04-15 12:20 ` Daniel Vetter 2020-04-28 14:08 ` Daniel Vetter 2020-05-08 13:56 ` Alexey Brodkin 2020-05-08 18:07 ` Daniel Vetter 2020-06-04 8:05 ` Daniel Vetter 2020-06-04 10:38 ` Eugeniy Paltsev 2020-06-04 11:19 ` Daniel Vetter 2020-06-04 19:00 ` Eugeniy Paltsev 2020-06-05 19:55 ` Daniel Vetter 2020-06-09 12:08 ` Eugeniy Paltsev 2020-06-09 13:02 ` Daniel Vetter 2020-07-17 9:04 ` Daniel Vetter
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).