[radeon-alex:amd-staging-drm-next 593/598] drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:5374:5: warning: no previous prototype for function 'amdgpu_dm_crtc_atomic_set_property'

2020-11-16 Thread kernel test robot
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next head: 11e500d2892a2b18e438c12e1e992b241aaf1c8b commit: 110d586ba77ed573eb7464ca69b6490ec0b70c5f [593/598] drm/amd/display: Expose new CRC window property config: arm64-randconfig-r034-20201115 (attached as .config) compil

[PATCH] drm/kmb: Fix possible oops in probe error handling

2020-11-16 Thread Dan Carpenter
If kmb_dsi_init() fails the error handling will dereference an error pointer which will cause an Oops. Fixes: 7f7b96a8a0a1 ("drm/kmb: Add support for KeemBay Display") Signed-off-by: Dan Carpenter --- drivers/gpu/drm/kmb/kmb_drv.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH] drm/kmb: Remove an unnecessary NULL check

2020-11-16 Thread Dan Carpenter
The NULL checking isn't done consistently in this function and it leads to a static checker warning: drivers/gpu/drm/kmb/kmb_drv.c:561 kmb_pm_suspend() error: we previously assumed 'drm' could be null (see line 559) Fortunately "drm" cannot be NULL at this point so the check can just be r

[radeon-alex:amd-staging-drm-next 576/598] drivers/gpu/drm/amd/amdgpu/../pm/swsmu/smu11/vangogh_ppt.c:764:5: warning: no previous prototype for function 'vangogh_set_default_dpm_tables'

2020-11-16 Thread kernel test robot
Hi Xiaojian, FYI, the error/warning still remains. tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next head: 11e500d2892a2b18e438c12e1e992b241aaf1c8b commit: 53501c771afda638485683d189a2b2e2ad776d38 [576/598] drm/amd/pm: add the fine grain tuning function for vangogh con

Re: [PATCH v3 0/5] console: Miscellaneous clean-ups, do not use FNTCHARCNT() in fbcon.c

2020-11-16 Thread Peilin Ye
On Mon, Nov 16, 2020 at 11:09:49AM +0100, Daniel Vetter wrote: > On Sat, Nov 14, 2020 at 07:47:16AM -0500, Peilin Ye wrote: > > On Sat, Nov 14, 2020 at 01:22:22PM +0100, Greg Kroah-Hartman wrote: > > > On Sat, Nov 14, 2020 at 01:18:06PM +0100, Greg Kroah-Hartman wrote: > > > > On Sat, Nov 14, 2020

[radeon-alex:amd-staging-drm-next 576/598] drivers/gpu/drm/amd/amdgpu/../pm/swsmu/smu11/vangogh_ppt.c:764:5: warning: no previous prototype for function 'vangogh_set_default_dpm_tables'

2020-11-16 Thread kernel test robot
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next head: 11e500d2892a2b18e438c12e1e992b241aaf1c8b commit: 53501c771afda638485683d189a2b2e2ad776d38 [576/598] drm/amd/pm: add the fine grain tuning function for vangogh config: arm64-randconfig-r034-20201115 (attached as .con

[PATCH] dt-bindings: display: Use OF graph schema

2020-11-16 Thread Rob Herring
Now that we have a graph schema, rework the display related schemas to use it. Mostly this is adding a reference to graph.yaml and dropping duplicate parts from schemas. Cc: Thierry Reding Cc: Sam Ravnborg Cc: Laurent Pinchart Cc: Maxime Ripard Cc: Maarten Lankhorst Cc: Thomas Zimmermann Sig

linux-next: manual merge of the amdgpu tree with the drm tree

2020-11-16 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the amdgpu tree got a conflict in: drivers/gpu/drm/radeon/radeon_drv.c between commit: 384bc5e059d4 ("drm/radeon: Stop changing the drm_driver struct") from the drm tree and commits: 4d3efadd374a ("drm/radeon/radeon: Move prototype into shared header"

[RESEND PATCH 1/3] drm/mediatek: Separate mtk_mipi_tx to an independent module

2020-11-16 Thread Chun-Kuang Hu
mtk_mipi_tx is a part of mtk_drm module, but phy driver should be an independent module rather than be part of drm module, so separate the phy driver to an independent module. Signed-off-by: Chun-Kuang Hu --- drivers/gpu/drm/mediatek/Kconfig | 8 drivers/gpu/drm/mediatek/Makefile

[RESEND PATCH 2/3] phy: mediatek: Move mtk_mipi_dsi_phy driver into drivers/phy/mediatek folder

2020-11-16 Thread Chun-Kuang Hu
mtk_mipi_dsi_phy is currently placed inside mediatek drm driver, but it's more suitable to place a phy driver into phy driver folder, so move mtk_mipi_dsi_phy driver into phy driver folder. Signed-off-by: Chun-Kuang Hu --- drivers/gpu/drm/mediatek/Kconfig | 7 --- d

[RESEND PATCH 3/3] MAINTAINERS: add files for Mediatek DRM drivers

2020-11-16 Thread Chun-Kuang Hu
Mediatek MIPI DSI phy driver is moved from drivers/gpu/drm/mediatek to drivers/phy/mediatek, so add the new folder to the Mediatek DRM drivers' information. Signed-off-by: Chun-Kuang Hu --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index e73636b75f

[RESEND PATCH 0/3] Move Mediatek MIPI DSI PHY driver from DRM folder to PHY folder

2020-11-16 Thread Chun-Kuang Hu
mtk_mipi_dsi_phy is currently placed inside mediatek drm driver, but it's more suitable to place a phy driver into phy driver folder, so move mtk_mipi_dsi_phy driver into phy driver folder. Chun-Kuang Hu (3): drm/mediatek: Separate mtk_mipi_tx to an independent module phy: mediatek: Move mtk_m

Re: [PATCH 29/42] drm/selftests/test-drm_dp_mst_helper: Place 'struct drm_dp_sideband_msg_req_body' onto the heap

2020-11-16 Thread Lyude Paul
Huh-could have sworn I had reviewed this one already. Reviewed-by: Lyude Paul On Mon, 2020-11-16 at 17:40 +, Lee Jones wrote: > The stack is too full. > > Fixes the following W=1 kernel build warning(s): > >  drivers/gpu/drm/selftests/test-drm_dp_mst_helper.c: In function > ‘sideband_msg_r

Re: [PATCH 08/10] drm/fb-helper: Restore damage area upon errors

2020-11-16 Thread Daniel Vetter
On Mon, Nov 16, 2020 at 09:04:35PM +0100, Thomas Zimmermann wrote: > If the damage handling fails, restore the damage area. The next invocation > of the damage worker will then perform the update. > > Signed-off-by: Thomas Zimmermann > --- > drivers/gpu/drm/drm_fb_helper.c | 23 +

Re: [PATCH 01/42] drm/amd/amdgpu/atombios_encoders: Remove set but unused variable 'backlight_level'

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:41 PM Lee Jones wrote: > > Also removing the call to > amdgpu_atombios_encoder_get_backlight_level_from_reg() > since, according to Alex Deucher, "We call it again below indirectly". > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/amd/amdgpu/at

Re: [PATCH 43/43] drm/radeon/radeon_drv: Move 'radeon_gem_prime_import_sg_table()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_prime.c:43:24: warning: no previous prototype > for ‘radeon_gem_prime_import_sg_table’ [-Wmissing-prototypes] > 43 | struct drm_gem_object *radeon_gem_prime_i

Re: [PATCH 10/10] drm/fb-helper: Acquire modeset lock around shadow-buffer flushing

2020-11-16 Thread Daniel Vetter
On Mon, Nov 16, 2020 at 09:04:37PM +0100, Thomas Zimmermann wrote: > Flushing the fbdev's shadow buffer requires vmap'ing the BO memory, which > in turn requires pinning the BO. While being pinned, the BO cannot be moved > into VRAM for scanout. Consequently, a concurrent modeset operation that > i

Re: [PATCH 42/43] drm/radeon/radeon_audio: Move 'r600_*' prototypes into shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/r600_hdmi.c:177:6: warning: no previous prototype for > ‘r600_hdmi_update_acr’ [-Wmissing-prototypes] > 177 | void r600_hdmi_update_acr(struct drm_encoder *encoder,

Re: [PATCH 41/43] drm/radeon/evergreen_cs: Move 'r600_dma_cs_next_reloc()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/r600_cs.c:2343:5: warning: no previous prototype for > ‘r600_dma_cs_next_reloc’ [-Wmissing-prototypes] > 2343 | int r600_dma_cs_next_reloc(struct radeon_cs_parser *p

Re: [PATCH 40/43] drm/radeon/cik: Move 'vce_v2_0_enable_mgcg()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/vce_v2_0.c:111:6: warning: no previous prototype for > ‘vce_v2_0_enable_mgcg’ [-Wmissing-prototypes] > 111 | void vce_v2_0_enable_mgcg(struct radeon_device *rdev, bo

Re: [PATCH v2 1/8] drm: Add dummy page per device or GEM object

2020-11-16 Thread Andrey Grodzovsky
On 11/16/20 3:36 PM, Christian König wrote: Am 16.11.20 um 20:00 schrieb Andrey Grodzovsky: On 11/16/20 4:48 AM, Christian König wrote: Am 15.11.20 um 07:34 schrieb Andrey Grodzovsky: On 11/14/20 4:51 AM, Daniel Vetter wrote: On Sat, Nov 14, 2020 at 9:41 AM Christian König wrote: Am 13.1

Re: [PATCH 39/43] drm/radeon/si_dpm: Move 'vce_v1_0_enable_mgcg()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/vce_v1_0.c:102:6: warning: no previous prototype for > ‘vce_v1_0_enable_mgcg’ [-Wmissing-prototypes] > 102 | void vce_v1_0_enable_mgcg(struct radeon_device *rdev, bo

Re: [PATCH 38/43] drm/radeon/cik: Move 'Move 'cik_sdma_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/cik_sdma.c:331:6: warning: no previous prototype for > ‘cik_sdma_enable’ [-Wmissing-prototypes] > 331 | void cik_sdma_enable(struct radeon_device *rdev, bool enable)

Re: [PATCH v2 1/8] drm: Add dummy page per device or GEM object

2020-11-16 Thread Christian König
Am 16.11.20 um 20:00 schrieb Andrey Grodzovsky: On 11/16/20 4:48 AM, Christian König wrote: Am 15.11.20 um 07:34 schrieb Andrey Grodzovsky: On 11/14/20 4:51 AM, Daniel Vetter wrote: On Sat, Nov 14, 2020 at 9:41 AM Christian König wrote: Am 13.11.20 um 21:52 schrieb Andrey Grodzovsky: On 6

Re: [PATCH 33/42] drm/ttm/ttm_range_manager: Demote non-conformant kernel-doc header

2020-11-16 Thread Christian König
Am 16.11.20 um 18:41 schrieb Lee Jones: Fixes the following W=1 kernel build warning(s): drivers/gpu/drm/ttm/ttm_range_manager.c:46: warning: cannot understand function prototype: 'struct ttm_range_manager ' Cc: Christian Koenig Cc: Huang Rui Cc: David Airlie Cc: Daniel Vetter Cc: dri-de

Re: [PATCH 31/42] drm/ttm/ttm_bo: Fix one function header - demote lots of kernel-doc abuses

2020-11-16 Thread Christian König
Am 16.11.20 um 18:41 schrieb Lee Jones: Fixes the following W=1 kernel build warning(s): drivers/gpu/drm/ttm/ttm_bo.c:51: warning: Function parameter or member 'ttm_global_mutex' not described in 'DEFINE_MUTEX' drivers/gpu/drm/ttm/ttm_bo.c:286: warning: Function parameter or member 'bo' no

Re: [PATCH 37/43] drm/radeon/ci_dpm: Move 'si_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/si_dpm.c:3802:4: warning: no previous prototype for > ‘si_get_ddr3_mclk_frequency_ratio’ [-Wmissing-prototypes] > 3802 | u8 si_get_ddr3_mclk_frequency_ratio(u32 memo

Re: [PATCH 36/43] drm/radeon/evergreen: Move 'cik_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/cik.c:6413:6: warning: no previous prototype for > ‘cik_init_cp_pg_table’ [-Wmissing-prototypes] > 6413 | void cik_init_cp_pg_table(struct radeon_device *rdev) > |

Re: [PATCH 35/43] drm/radeon/cik_sdma: Move 'amdgpu_cik_gpu_check_soft_reset()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/cik.c:4845:5: warning: no previous prototype for > ‘cik_gpu_check_soft_reset’ [-Wmissing-prototypes] > 4845 | u32 cik_gpu_check_soft_reset(struct radeon_device *rdev

Re: [PATCH 32/42] drm/ttm/ttm_tt: Demote kernel-doc header format abuses

2020-11-16 Thread Christian König
Am 16.11.20 um 18:41 schrieb Lee Jones: Fixes the following W=1 kernel build warning(s): drivers/gpu/drm/ttm/ttm_tt.c:45: warning: Function parameter or member 'bo' not described in 'ttm_tt_create' drivers/gpu/drm/ttm/ttm_tt.c:45: warning: Function parameter or member 'zero_alloc' not desc

Re: [PATCH 34/43] drm/radeon/evergreen: Move 'si_get_csb_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:37 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/si.c:5678:5: warning: no previous prototype for > ‘si_get_csb_size’ [-Wmissing-prototypes] > 5678 | u32 si_get_csb_size(struct radeon_device *rdev) > | ^~~~

Re: [PATCH 33/43] drm/radeon/radeon_audio: Move 'dce6_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/dce6_afmt.c:32:5: warning: no previous prototype for > ‘dce6_endpoint_rreg’ [-Wmissing-prototypes] > 32 | u32 dce6_endpoint_rreg(struct radeon_device *rdev, > | ^~~

Re: [PATCH 32/43] drm/radeon/btc_dpm: Move 'evergreen_get_pi's prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/rv770_dpm.c:62:30: warning: no previous prototype for > ‘evergreen_get_pi’ [-Wmissing-prototypes] > 62 | struct evergreen_power_info *evergreen_get_pi(struct radeon_

Re: [PATCH 31/43] drm/radeon/cik: Move 'si_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/si.c:4186:6: warning: no previous prototype for > ‘si_vram_gtt_location’ [-Wmissing-prototypes] > 4186 | void si_vram_gtt_location(struct radeon_device *rdev, > | ^

Re: [PATCH 30/43] drm/radeon/si_dma: Move 'si_gpu_check_soft_reset()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/si.c:3768:5: warning: no previous prototype for > ‘si_gpu_check_soft_reset’ [-Wmissing-prototypes] > 3768 | u32 si_gpu_check_soft_reset(struct radeon_device *rdev) >

Re: [PATCH 29/43] drm/radeon/si_dpm: Move 'si_mc_load_microcode()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/si.c:1570:5: warning: no previous prototype for > ‘si_mc_load_microcode’ [-Wmissing-prototypes] > > Cc: Alex Deucher > Cc: "Christian König" > Cc: David Airlie > C

Re: [PATCH 28/43] drm/radeon/ci_dpm: Move 'ci_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/cik.c:1868:5: warning: no previous prototype for > ‘ci_mc_load_microcode’ [-Wmissing-prototypes] > 1868 | int ci_mc_load_microcode(struct radeon_device *rdev) > | ^

Re: [PATCH 27/43] drm/radeon/radeon_encoders: Move 'radeon_atom_backlight_init's prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:37 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/atombios_encoders.c:186:6: warning: no previous > prototype for ‘radeon_atom_backlight_init’ [-Wmissing-prototypes] > 186 | void radeon_atom_backlight_init(struct ra

Re: [PATCH 26/43] drm/radeon/radeon_atombios: Move 'radeon_add_atom_encoder()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/atombios_encoders.c:2721:1: warning: no previous > prototype for ‘radeon_add_atom_encoder’ [-Wmissing-prototypes] > 2721 | radeon_add_atom_encoder(struct drm_device

Re: [PATCH 25/43] drm/radeon/ni_dma: Move 'cayman_gpu_check_soft_reset()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:37 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/ni.c:1733:5: warning: no previous prototype for > ‘cayman_gpu_check_soft_reset’ [-Wmissing-prototypes] > 1733 | u32 cayman_gpu_check_soft_reset(struct radeon_device

Re: [PATCH 24/43] drm/radeon/r600: Move 'evergreen_rlc_resume()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/evergreen.c:4380:5: warning: no previous prototype > for ‘evergreen_rlc_resume’ [-Wmissing-prototypes] > 4380 | int evergreen_rlc_resume(struct radeon_device *rdev)

Re: [PATCH 23/43] drm/radeon/evergreen_dma: Move 'evergreen_gpu_check_soft_reset()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/evergreen.c:3825:5: warning: no previous prototype > for ‘evergreen_gpu_check_soft_reset’ [-Wmissing-prototypes] > 3825 | u32 evergreen_gpu_check_soft_reset(struct r

Re: [PATCH 22/43] drm/radeon/cik: Move 'r600_ih_ring_{alloc, fini}()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/r600.c:3480:5: warning: no previous prototype for > ‘r600_ih_ring_alloc’ [-Wmissing-prototypes] > 3480 | int r600_ih_ring_alloc(struct radeon_device *rdev) > | ^~~~

Re: [PATCH 21/43] drm/radeon/r600_dma: Move 'r600_gpu_check_soft_reset()'s prototype to shared location

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/r600.c:1615:5: warning: no previous prototype for > ‘r600_gpu_check_soft_reset’ [-Wmissing-prototypes] > 1615 | u32 r600_gpu_check_soft_reset(struct radeon_device *r

Re: [PATCH 20/43] drm/radeon/evergreen: Move 'cayman_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/ni.c:1378:6: warning: no previous prototype for > ‘cayman_cp_int_cntl_setup’ [-Wmissing-prototypes] > 1378 | void cayman_cp_int_cntl_setup(struct radeon_device *rdev

Re: [PATCH 19/43] drm/radeon/ni_dpm: Move 'ni_get_{pi, ps}()'s into shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:38 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/ni_dpm.c:727:23: warning: no previous prototype for > ‘ni_get_pi’ [-Wmissing-prototypes] > 727 | struct ni_power_info *ni_get_pi(struct radeon_device *rdev) > | ^~~

[PATCH 2/4] drm/meson: Unbind all connectors on module removal

2020-11-16 Thread Marc Zyngier
Removing the meson DRM module results in the following splats: [ 42.689228] WARNING: CPU: 0 PID: 572 at drivers/gpu/drm/drm_irq.c:192 drm_irq_uninstall+0x130/0x160 [drm] [...] [ 42.812820] Hardware name: , BIOS 2021.01-rc2-00012-gde865f7ee1 11/16/2020 [ 42.819723] pstate: 80400089 (Nzcv da

[PATCH 3/4] drm/meson: dw-hdmi: Register a callback to disable the regulator

2020-11-16 Thread Marc Zyngier
Removing the meson-dw-hdmi module results in the following splat: i[ 43.340509] WARNING: CPU: 0 PID: 572 at drivers/regulator/core.c:2125 _regulator_put.part.0+0x16c/0x174 [...] [ 43.454870] CPU: 0 PID: 572 Comm: modprobe Tainted: GW E 5.10.0-rc4-00049-gd274813a4de3-dirty #2147

[PATCH 0/4] drm/meson: Module removal fixes

2020-11-16 Thread Marc Zyngier
Hi all, Having recently moved over to a top-of-the-tree u-boot on one of my VIM3L systems in order to benefit from unrelated improvements (automatic PCIe detection, EFI...), I faced the issue that my kernel would hang like this: [ OK ] Finished Helper to synchronize boot up for ifupdown. [ OK

[PATCH 4/4] drm/meson: dw-hdmi: Ensure that clocks are enabled before touching the TOP registers

2020-11-16 Thread Marc Zyngier
Removing the meson-dw-hdmi module and re-inserting it results in a hang as the driver writes to HDMITX_TOP_SW_RESET. Similar effects can be seen when booting with mainline u-boot and using the u-boot provided DT (which is highly desirable). The reason for the hang seem to be that the clocks are no

[PATCH 1/4] drm/meson: Free RDMA resources after tearing down DRM

2020-11-16 Thread Marc Zyngier
Removing the meson DRM module results in the following splat: [ 2179.451346] Hardware name: , BIOS 2021.01-rc2-00012-gde865f7ee1 11/16/2020 [ 2179.458316] Workqueue: events drm_mode_rmfb_work_fn [drm] [ 2179.463597] pstate: 80c9 (Nzcv daif +PAN +UAO -TCO BTYPE=--) [ 2179.469558] pc : meson_rd

Re: [PATCH 4/7] drm/radeon: Pin buffers while they are vmap'ed

2020-11-16 Thread Thomas Zimmermann
Hi Am 16.11.20 um 12:28 schrieb Christian König: > Am 13.11.20 um 08:59 schrieb Thomas Zimmermann: >> Hi Christian >> >> Am 12.11.20 um 18:16 schrieb Christian König: >>> Am 12.11.20 um 14:21 schrieb Thomas Zimmermann: In order to avoid eviction of vmap'ed buffers, pin them in their GEM

Re: [PATCH 18/43] drm/radeon/rv770: Move 'rv770_get_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:37 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/rv770_dpm.c:47:18: warning: no previous prototype for > ‘rv770_get_ps’ [-Wmissing-prototypes] > 47 | struct rv7xx_ps *rv770_get_ps(struct radeon_ps *rps) > | ^~

[PATCH 08/10] drm/fb-helper: Restore damage area upon errors

2020-11-16 Thread Thomas Zimmermann
If the damage handling fails, restore the damage area. The next invocation of the damage worker will then perform the update. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/drm_fb_helper.c | 23 --- 1 file changed, 20 insertions(+), 3 deletions(-) diff --git a/drivers/

[PATCH 06/10] drm/fb-helper: Separate shadow-buffer flushing and calling dirty callback

2020-11-16 Thread Thomas Zimmermann
Flushing the shadow framebuffer and invoking the dirty callback are two separate operations, so do them seprately. The flush operation is paired with calls to vmap and vunmap. They are not needed for the dirty callback, which performs its own invocations if necessary. Signed-off-by: Thomas Zimmerm

[PATCH 03/10] drm/client: Depend on GEM object kmap ref-counting

2020-11-16 Thread Thomas Zimmermann
DRM client's vmap/vunmap functions don't allow for multiple vmap operations. Calling drm_client_buffer_vmap() twice returns the same mapping, then calling drm_client_buffer_vunmap() twice already unmaps on the first call. This leads to unbalanced vmap refcounts. Fix this by calling drm_gem_vmap() u

[PATCH 01/10] drm/fb-helper: Call dirty helper after writing to fbdev

2020-11-16 Thread Thomas Zimmermann
If fbdev uses a shadow framebuffer, call the damage handler. Otherwise the update might not make it to the screen. Signed-off-by: Thomas Zimmermann Fixes: 222ec45f4c69 ("drm/fb_helper: Support framebuffers in I/O memory") Cc: Thomas Zimmermann Cc: Daniel Vetter Cc: Sam Ravnborg Cc: Maarten Lan

[PATCH 05/10] drm/fb-helper: Return early in dirty worker

2020-11-16 Thread Thomas Zimmermann
Returning early in the dirty worker if no update is required makes the code more readable. No functional changes are made. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/drm_fb_helper.c | 31 +++ 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/dr

[PATCH 02/10] drm/fb-helper: Unmap client buffer during shutdown

2020-11-16 Thread Thomas Zimmermann
The fbdev helper's generic probe function establishes a mapping for framebuffers without shadow buffer. The clean-up function did not unmap the buffer object. Add the unmap operation. As fbdev devices are usally released during system shutdown, this has not been a problem in practice. Signed-off-

[PATCH 09/10] drm/fb-helper: Copy dma-buf map before flushing shadow fb

2020-11-16 Thread Thomas Zimmermann
Copy the vmap()'ed instance of struct dma_buf_map before modifying it, in case the implementation of vunmap() depends on the exact address. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/drm_fb_helper.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/dr

[PATCH 07/10] drm/fb-helper: Move damage blit code and its setup into separate routine

2020-11-16 Thread Thomas Zimmermann
Introduce a separate function for the blit code and its vmap setup. Done in preparation of additional changes. No functional changes are made. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/drm_fb_helper.c | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) dif

[PATCH 00/10] drm/fb-helper: Various fixes and cleanups

2020-11-16 Thread Thomas Zimmermann
Here's a number of fb-helper patches that have been piling up recently. Patches 1 to 3 fix bugs that I spotted while going through the code. Because of the way the fbdev code works, they have been avoided so far. Patches 4 to 7 cleanup damage handling for fbdev's shadow buffer and fix a few issue

[PATCH 10/10] drm/fb-helper: Acquire modeset lock around shadow-buffer flushing

2020-11-16 Thread Thomas Zimmermann
Flushing the fbdev's shadow buffer requires vmap'ing the BO memory, which in turn requires pinning the BO. While being pinned, the BO cannot be moved into VRAM for scanout. Consequently, a concurrent modeset operation that involves the fbdev framebuffer would likely fail. Resolve this problem be a

[PATCH 04/10] drm/fb-helper: Rename dirty worker to damage worker

2020-11-16 Thread Thomas Zimmermann
The dirty worker handles all damage updates, instead of just calling the framebuffer's dirty callback. Rename it to damage worker. Also rename related variables accordingly. No functional changes are made. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/drm_fb_helper.c | 65 +++-

Re: [PATCH 17/43] drm/radeon/evergreen_hdmi: Move 'evergreen_*()' and 'dce4_*()' HDMI prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:37 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/evergreen_hdmi.c:37:6: warning: no previous prototype > for ‘dce4_audio_enable’ [-Wmissing-prototypes] > drivers/gpu/drm/radeon/evergreen_hdmi.c:67:6: warning: no pr

Re: [PATCH 16/43] drm/radeon/radeon_gem: Move 'radeon_gem_prime_*()'s prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:37 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_prime.c:34:18: warning: no previous prototype > for ‘radeon_gem_prime_get_sg_table’ [-Wmissing-prototypes] > 34 | struct sg_table *radeon_gem_prime_get_sg_tab

Re: [PATCH 15/43] drm/radeon/atom: Move 'radeon_atom_hw_i2c_*()'s prototypes into shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:37 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/atombios_i2c.c:100:5: warning: no previous prototype > for ‘radeon_atom_hw_i2c_xfer’ [-Wmissing-prototypes] > 100 | int radeon_atom_hw_i2c_xfer(struct i2c_adapter *i

Re: [PATCH 14/43] drm/radeon/radeon_drv: Move 'radeon_driver_irq_handler_kms's prototype into shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:37 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_irq_kms.c:53:13: warning: no previous > prototype for ‘radeon_driver_irq_handler_kms’ [-Wmissing-prototypes] > 53 | irqreturn_t radeon_driver_irq_handler_kms(

Re: [PATCH 13/43] drm/radeon/radeon_drv: Move 'radeon_mmap()'s prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:37 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_ttm.c:931:5: warning: no previous prototype > for ‘radeon_mmap’ [-Wmissing-prototypes] > 931 | int radeon_mmap(struct file *filp, struct vm_area_struct *vma)

Re: [PATCH v4 05/27] drm: fix some kernel-doc markups

2020-11-16 Thread Daniel Vetter
On Mon, Nov 16, 2020 at 01:37:11PM +0200, Jani Nikula wrote: > On Mon, 16 Nov 2020, Mauro Carvalho Chehab wrote: > > Some identifiers have different names between their prototypes > > and the kernel-doc markup. > > > > Others need to be fixed, as kernel-doc markups should use this format: > >

Re: [PATCH 12/43] drm/radeon/evergreen: Move 'evergreen_*' and 'sumo_*' prototypes out to shared location

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/evergreen.c:1281:6: warning: no previous prototype > for ‘evergreen_fix_pci_max_read_req_size’ [-Wmissing-prototypes] > 1281 | void evergreen_fix_pci_max_read_req_si

Re: [PATCH 11/43] drm/radeon/radeon_audio: Move 'dce3_2_*' prototypes to shared location

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/dce3_1_afmt.c:30:6: warning: no previous prototype > for ‘dce3_2_afmt_hdmi_write_speaker_allocation’ [-Wmissing-prototypes] > 30 | void dce3_2_afmt_hdmi_write_speake

Re: [PATCH 10/43] drm/radeon/radeon_pm: Move 'radeon_pm_acpi_event_handler' prototype into shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_pm.c:74:6: warning: no previous prototype for > ‘radeon_pm_acpi_event_handler’ [-Wmissing-prototypes] > 74 | void radeon_pm_acpi_event_handler(struct radeon_d

Re: [PATCH v4 10/27] video: fix some kernel-doc markups

2020-11-16 Thread Daniel Vetter
On Mon, Nov 16, 2020 at 07:11:12PM +0100, Sam Ravnborg wrote: > Hi Daniel > > > Feel free to just merge it via your tree. Patches here are pretty > > > much independent ;-) > > > > Ok I put it into drm-misc-next. I kinda assumed since there's also a huge > > effort going on to shut up warnings, pl

Re: [PATCH 09/43] drm/radeon/rv770: Move 'rv770_set_clk_bypass_mode' prototype to shared location

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/rv770.c:1138:6: warning: no previous prototype for > ‘rv770_set_clk_bypass_mode’ [-Wmissing-prototypes] > 1138 | void rv770_set_clk_bypass_mode(struct radeon_device

Re: [PATCH 08/43] drm/radeon/radeon_irq_kms: Demote non-conformant kernel-doc fix another

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_irq_kms.c:56: warning: Function parameter or > member 'irq' not described in 'radeon_driver_irq_handler_kms' > drivers/gpu/drm/radeon/radeon_irq_kms.c:56: war

Re: [PATCH 07/43] drm/radeon/radeon_legacy_encoders: Move 'radeon_add_legacy_encoder's prototype to shared location

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_legacy_encoders.c:1746:1: warning: no previous > prototype for ‘radeon_add_legacy_encoder’ [-Wmissing-prototypes] > 1746 | radeon_add_legacy_encoder(struct dr

Re: [PATCH 06/43] drm/radeon/radeon_legacy_encoders: Move 'radeon_add_legacy_encoder' prototype to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_legacy_encoders.c:1745:1: warning: no > previous prototype for ‘radeon_add_legacy_encoder’ [-Wmissing-prototypes] > 1745 | radeon_add_legacy_encoder(struct d

Re: [PATCH 05/43] drm/radeon: Move radeon_ttm{init, fini} to shared location

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_ttm.c: At top level: > drivers/gpu/drm/radeon/radeon_ttm.c:817:5: warning: no previous prototype > for ‘radeon_ttm_init’ [-Wmissing-prototypes] > 817 | int r

Re: [PATCH 04/43] drm/radeon/radeon_kms: Fix misnaming of 'radeon_info_ioctl's dev param

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_kms.c:226: warning: Function parameter or > member 'dev' not described in 'radeon_info_ioctl' > drivers/gpu/drm/radeon/radeon_kms.c:226: warning: Excess funct

Re: [PATCH 03/43] drm/radeon/radeon_kms: Move 'radeon_*_kms' prototypes to shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_kms.c:756:5: warning: no previous prototype > for ‘radeon_get_vblank_counter_kms’ [-Wmissing-prototypes] > 756 | u32 radeon_get_vblank_counter_kms(struct drm_

Re: [PATCH 02/43] drm/radeon/radeon: Move prototype into shared header

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Unfortunately, a suitable one didn't already exist. > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_device.c:637:6: warning: no previous prototype > for ‘radeon_device_is_virtual’ [-Wmissing-prototypes]

Patch "drm/gma500: Fix out-of-bounds access to struct drm_device.vblank[]" has been added to the 4.9-stable tree

2020-11-16 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/gma500: Fix out-of-bounds access to struct drm_device.vblank[] to the 4.9-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the pat

Re: [git pull] drm next pull for 5.10-rc1

2020-11-16 Thread Dave Airlie
> > Christ. It's been two weeks. I'm doing -rc4 today, and I still don't > have the fix. > > The problem seems entirely obvious, as reported by Kirill: the nv50 > code unconditionally calls the "atomic_{dis,en}able()" functions, even > when not everybody was converted. > > The fix seems to be to ei

Re: [PATCH 01/43] drm/radeon/atombios_encoders: Move 'radeon_atom_get_tv_timings()'s prototype into shared location

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 12:30 PM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_atombios.c:1791:6: warning: no previous > prototype for ‘radeon_atom_get_tv_timings’ [-Wmissing-prototypes] > 1791 | bool radeon_atom_get_tv_timings(struct ra

Patch "drm/gma500: Fix out-of-bounds access to struct drm_device.vblank[]" has been added to the 4.4-stable tree

2020-11-16 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/gma500: Fix out-of-bounds access to struct drm_device.vblank[] to the 4.4-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the pat

Re: [PATCH v3] dt-bindings: display: panel: one file of all simple LVDS panels with dual ports

2020-11-16 Thread Rob Herring
On Thu, Nov 12, 2020 at 02:17:11PM +0800, Liu Ying wrote: > To complement panel-simple.yaml, create panel-simple-lvds-dual-ports.yaml. > panel-simple-lvds-dual-ports.yaml is for all simple LVDS panels that > have dual LVDS ports and require only a single power-supply. > The first port receives odd

Re: [PATCH] drm/amdgpu: fix check oder in amdgpu_bo_move

2020-11-16 Thread Alex Deucher
On Mon, Nov 16, 2020 at 2:14 PM Christian König wrote: > > Reorder the code to fix checking if blitting is available. > > Signed-off-by: Christian König > Fixes: f5a89a5cae81 drm/amdgpu/ttm: use multihop Typo in the subject (oder -> order). With that fixes the patch is: Acked-by: Alex Deucher

[PATCH] drm/amdgpu: fix check oder in amdgpu_bo_move

2020-11-16 Thread Christian König
Reorder the code to fix checking if blitting is available. Signed-off-by: Christian König Fixes: f5a89a5cae81 drm/amdgpu/ttm: use multihop --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 53 +++-- 1 file changed, 23 insertions(+), 30 deletions(-) diff --git a/drivers/gpu/drm/a

Re: [PATCH v2 1/8] drm: Add dummy page per device or GEM object

2020-11-16 Thread Andrey Grodzovsky
On 11/16/20 4:48 AM, Christian König wrote: Am 15.11.20 um 07:34 schrieb Andrey Grodzovsky: On 11/14/20 4:51 AM, Daniel Vetter wrote: On Sat, Nov 14, 2020 at 9:41 AM Christian König wrote: Am 13.11.20 um 21:52 schrieb Andrey Grodzovsky: On 6/22/20 1:50 PM, Daniel Vetter wrote: On Mon, Jun

Re: [PATCH v4 10/27] video: fix some kernel-doc markups

2020-11-16 Thread Mauro Carvalho Chehab
Em Mon, 16 Nov 2020 18:24:04 +0100 Daniel Vetter escreveu: > On Mon, Nov 16, 2020 at 05:38:04PM +0100, Mauro Carvalho Chehab wrote: > > Em Mon, 16 Nov 2020 16:36:06 +0100 > > Daniel Vetter escreveu: > > > > > On Mon, Nov 16, 2020 at 11:18:06AM +0100, Mauro Carvalho Chehab wrote: > > > > Som

Re: [PATCH v1 0/4] Add bus format negotiation support for Cadence MHDP8546 driver

2020-11-16 Thread Tomi Valkeinen
Hi, On 13/11/2020 11:46, Yuti Amonkar wrote: > This patch series add bus format negotiation support for Cadence MHDP8546 > bridge > driver. > > The patch series has four patches in the below sequence: > 1. drm: bridge: cdns-mhdp8546: Add output bus format negotiation > Add minimal output bus for

[PATCH] drm/imx: depend on COMMON_CLK to fix compile tests

2020-11-16 Thread Krzysztof Kozlowski
The iMX DRM drivers use Common Clock Framework thus they cannot be built on platforms without it (e.g. compile test on MIPS with RALINK and SOC_RT305X): /usr/bin/mips-linux-gnu-ld: drivers/gpu/drm/imx/imx-ldb.o: in function `imx_ldb_encoder_disable': imx-ldb.c:(.text+0x400): undefined ref

Re: [PATCH v4 10/27] video: fix some kernel-doc markups

2020-11-16 Thread Sam Ravnborg
Hi Daniel > > Feel free to just merge it via your tree. Patches here are pretty > > much independent ;-) > > Ok I put it into drm-misc-next. I kinda assumed since there's also a huge > effort going on to shut up warnings, plus I think kerneldoc issues are > reported by a bunch of build bots nowada

[PATCH 1/2] drm/exynos: depend on COMMON_CLK to fix compile tests

2020-11-16 Thread Krzysztof Kozlowski
The Exynos DRM uses Common Clock Framework thus it cannot be built on platforms without it (e.g. compile test on MIPS with RALINK and SOC_RT305X): /usr/bin/mips-linux-gnu-ld: drivers/gpu/drm/exynos/exynos_mixer.o: in function `mixer_bind': exynos_mixer.c:(.text+0x958): undefined reference

[PATCH 2/2] drm/ingenic: depend on COMMON_CLK to fix compile tests

2020-11-16 Thread Krzysztof Kozlowski
The Ingenic DRM uses Common Clock Framework thus it cannot be built on platforms without it (e.g. compile test on MIPS with RALINK and SOC_RT305X): /usr/bin/mips-linux-gnu-ld: drivers/gpu/drm/ingenic/ingenic-drm-drv.o: in function `ingenic_drm_bind.isra.0': ingenic-drm-drv.c:(.text+0x1600

Re: [RESEND PATCH v2 4/5] drm/msm: add DRM_MSM_GEM_SYNC_CACHE for non-coherent cache maintenance

2020-11-16 Thread Rob Clark
On Mon, Nov 16, 2020 at 9:33 AM Christoph Hellwig wrote: > > On Sat, Nov 14, 2020 at 03:07:20PM -0500, Jonathan Marek wrote: > > qcom's vulkan driver has nonCoherentAtomSize=1, and it looks like > > dma_sync_single_for_cpu() does deal in some way with the partial cache line > > case, although I'm

[PATCH v2 0/3] drm/msm: Shrinker fixes and opts

2020-11-16 Thread Rob Clark
From: Rob Clark The last patch is the main thing, motivated by some cases where we would spend a lot of time in msm_gem_shrinker_count(). First two are fixes I noticed along the way. Rob Clark (3): drm/msm: Protect obj->active_count under obj lock drm/msm/shrinker: We can vmap shrink active

[PATCH v2 3/3] drm/msm/shrinker: Only iterate dontneed objs

2020-11-16 Thread Rob Clark
From: Rob Clark In situations where the GPU is mostly idle, all or nearly all buffer objects will be in the inactive list. But if the system is under memory pressure (from something other than GPU), we could still get a lot of shrinker calls. Which results in traversing a list of thousands of o

[PATCH v2 2/3] drm/msm/shrinker: We can vmap shrink active_list too

2020-11-16 Thread Rob Clark
From: Rob Clark Just because a obj is active, if the vmap_count is zero, we can still tear down the vmap. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/msm_gem_shrinker.c | 47 +++--- 1 file changed, 35 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/msm/msm

  1   2   3   >