[Bug 208333] Black screen with Kernel 5.8 rc1/2/3/4 with Nvidia Gtx 760 driver nouveau

2020-07-14 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=208333 --- Comment #7 from Roberto Guerrini (robyguerr...@yahoo.it) --- in wayland the system works while in gnome it remains on the black screen -- You are receiving this mail because: You are watching the assignee of the bug.

Re: [PATCH 0/4] drm: core: Convert logging to drm_* functions.

2020-07-14 Thread Suraj Upadhyay
On Sat, Jul 11, 2020 at 11:16:33AM -0700, Joe Perches wrote: > On Sat, 2020-07-11 at 20:41 +0530, Suraj Upadhyay wrote: > > On Fri, Jul 10, 2020 at 07:56:43PM +0200, Sam Ravnborg wrote: > > > Hi Suraj. > > > > > > On Tue, Jul 07, 2020 at 10:04:14PM +0530, Suraj Upadhyay wrote: > > > > This

Re: fbconsole needs more parameter validations.

2020-07-14 Thread Tetsuo Handa
On 2020/07/10 19:56, Greg Kroah-Hartman wrote: > Where is the over/underflow happening here when we set a size to be so > small? We should bound the size somewhere, and as you show, that's not > really working properly, right? It is bit_clear_margins() where integer underflow is happening

[PATCH v4 10/14] drm/panfrost: add regulators to devfreq

2020-07-14 Thread Clément Péron
Some OPP tables specify voltage for each frequency. Devfreq can handle these regulators but they should be get only 1 time to avoid issue and know who is in charge. If OPP table is probe don't init regulator. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron

Re: [PATCH] Replace HTTP links with HTTPS ones: BACKLIGHT CLASS/SUBSYSTEM

2020-07-14 Thread Alexander A. Klimov
Am 10.07.20 um 19:35 schrieb Sam Ravnborg: Hi Alexander. Hi, On Wed, Jul 08, 2020 at 06:38:47PM +0200, Alexander A. Klimov wrote: Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each

[PATCH v2 3/8] drm/msm/dpu: move some sspp caps to dpu_caps

2020-07-14 Thread Jonathan Marek
This isn't something that ever changes between planes, so move it to dpu_caps struct. Making this change will allow more re-use in the "SSPP sub blocks config" part of the catalog, in particular when adding support for SM8150 and SM8250 which have different max_linewidth. This also sets

[PATCH] vgacon: Fix an out-of-bounds in vgacon_scrollback_update()

2020-07-14 Thread Yang Yingliang
I got a slab-out-of-bounds report when I doing fuzz test. [ 334.989515] == [ 334.989577] BUG: KASAN: slab-out-of-bounds in vgacon_scroll+0x57a/0x8ed [ 334.989588] Write of size 1766 at addr 8883de69ff3e by task test/2658 [

[PATCH v4 0/3] drm/msm: handle for EPROBE_DEFER for of_icc_get

2020-07-14 Thread Jonathan Marek
Check for errors instead of silently not using icc if the msm driver probes before the interconnect driver. Allow ENODATA for ocmem path, as it is optional and this error is returned when "gfx-mem" path is provided but not "ocmem". Because msm_gpu_cleanup assumes msm_gpu_init has been called,

[PATCH v4 08/14] drm/panfrost: move devfreq_init()/fini() in device

2020-07-14 Thread Clément Péron
Later we will introduce devfreq probing regulator if they are present. As regulator should be probe only one time we need to get this logic in the device_init(). panfrost_device is already taking care of devfreq_resume() and devfreq_suspend(), so it's not totally illogic to move the

Re: [PATCH 4/6] staging: qlge: qlge_main: Simplify while statements.

2020-07-14 Thread Suraj Upadhyay
On Mon, Jul 13, 2020 at 05:12:35PM +0300, Dan Carpenter wrote: > On Mon, Jul 13, 2020 at 05:50:14PM +0530, Suraj Upadhyay wrote: > > Simplify while loops into more readable and simple for loops. > > > > I don't think either is more clear that the other. Walter Harms hates > count down loops and

[PATCH v5 14/14] [DO NOT MERGE] arm64: dts: allwinner: force GPU regulator to be always

2020-07-14 Thread Clément Péron
Signed-off-by: Clément Péron --- arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts b/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts index 3f7ceeb1a767..14257f7476b8 100644

Re: [PATCH v4 04/14] drm/panfrost: introduce panfrost_devfreq struct

2020-07-14 Thread Clément Péron
Hi, On Fri, 10 Jul 2020 at 11:25, Clément Péron wrote: > > Introduce a proper panfrost_devfreq to deal with devfreq variables. > > Reviewed-by: Steven Price > Reviewed-by: Alyssa Rosenzweig > Signed-off-by: Clément Péron > --- > drivers/gpu/drm/panfrost/panfrost_devfreq.c | 76

[PATCH v4 02/14] drm/panfrost: clean headers in devfreq

2020-07-14 Thread Clément Péron
Don't include not required headers and sort them. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_devfreq.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git

[PATCH v4 11/14] arm64: defconfig: Enable devfreq cooling device

2020-07-14 Thread Clément Péron
Devfreq cooling device framework is used in Panfrost to throttle GPU in order to regulate its temperature. Enable this driver for ARM64 SoC. Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- arch/arm64/configs/defconfig | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v4 12/14] arm64: dts: allwinner: h6: Add cooling map for GPU

2020-07-14 Thread Clément Péron
Add a simple cooling map for the GPU. Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 22 1 file changed, 22 insertions(+) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi

[PATCH v5 07/14] drm/panfrost: rename error labels in device_init

2020-07-14 Thread Clément Péron
Rename goto labels in device_init it will be easier to maintain. Reviewed-by: Alyssa Rosenzweig Reviewed-by: Steven Price Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_device.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git

[PATCH v8 2/2] display/drm/bridge: TC358775 DSI/LVDS driver

2020-07-14 Thread Vinay Simha BN
This driver is tested with two panels individually with Apq8016-IFC6309 board https://www.inforcecomputing.com/products/single-board-computers-sbc/qualcomm-snapdragon-410-inforce-6309-micro-sbc 1. 1366x768@60 auo,b101xtn01 data-mapping = "jeida-24" 2. 800x480@60 innolux,at070tn92 data-mapping =

[PATCH v2 1/8] drm/msm/dpu: use right setup_blend_config for sm8150 and sm8250

2020-07-14 Thread Jonathan Marek
All DPU versions starting from 4.0 use the sdm845 version, so check for that instead of checking each version individually. This chooses the right function for sm8150 and sm8250. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c | 5 ++--- 1 file changed, 2

[PATCH v4 14/14] [DO NOT MERGE] arm64: dts: allwinner: force GPU regulator to be always

2020-07-14 Thread Clément Péron
Signed-off-by: Clément Péron --- arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts b/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts index 3f7ceeb1a767..14257f7476b8 100644

[PATCH v5 11/14] arm64: defconfig: Enable devfreq cooling device

2020-07-14 Thread Clément Péron
Devfreq cooling device framework is used in Panfrost to throttle GPU in order to regulate its temperature. Enable this driver for ARM64 SoC. Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- arch/arm64/configs/defconfig | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v5 12/14] arm64: dts: allwinner: h6: Add cooling map for GPU

2020-07-14 Thread Clément Péron
Add a simple cooling map for the GPU. Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 22 1 file changed, 22 insertions(+) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi

Re: [PATCH RESEND v2 2/2] drm/bridge: dw-hdmi: Always add the bridge in the global bridge list

2020-07-14 Thread Liu Ying
On Fri, 2020-07-10 at 19:32 +0200, Sam Ravnborg wrote: > On Thu, Jul 09, 2020 at 10:02:36AM +0800, Liu Ying wrote: > > It doesn't hurt to add the bridge in the global bridge list also > > for > > platform specific dw-hdmi drivers which are based on the component > > framework. This can be

[PATCH v2 1/2] drm/msm/a6xx: hwcg tables in gpulist

2020-07-14 Thread Jonathan Marek
This will allow supporting different hwcg tables for a6xx. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 25 ++ drivers/gpu/drm/msm/adreno/adreno_device.c | 1 + drivers/gpu/drm/msm/adreno/adreno_gpu.h| 8 +++ 3 files changed, 20

[PATCH v2 6/8] drm/msm/dpu: intf timing path for displayport

2020-07-14 Thread Jonathan Marek
Calculate the correct timings for displayport, from downstream driver. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c | 20 +++- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c

[PATCH v2 5/8] drm/msm/dpu: set missing flush bits for INTF_2 and INTF_3

2020-07-14 Thread Jonathan Marek
This fixes flushing of INTF_2 and INTF_3 on SM8150 and SM8250 hardware. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c | 20 ++-- 1 file changed, 2 insertions(+), 18 deletions(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c

[PATCH v2 4/8] drm/msm/dpu: don't use INTF_INPUT_CTRL feature on sdm845

2020-07-14 Thread Jonathan Marek
The INTF_INPUT_CTRL feature is not available on sdm845, so don't set it. This also adds separate feature bits for INTF (based on downstream) instead of using CTL feature bit for it, and removes the unnecessary NULL check in the added bind_pingpong_blk function. Fixes:

[PATCH v4 06/14] drm/panfrost: properly handle error in probe

2020-07-14 Thread Clément Péron
Introduce a boolean to know if opp table has been added. With this, we can call panfrost_devfreq_fini() in case of error and release what has been initialised. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron ---

[PATCH v5 01/14] drm/panfrost: avoid static declaration

2020-07-14 Thread Clément Péron
This declaration can be avoided so change it. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_devfreq.c | 38 ++--- 1 file changed, 18 insertions(+), 20 deletions(-) diff --git

[PATCH v5 03/14] drm/panfrost: don't use pfdevfreq.busy_count to know if hw is idle

2020-07-14 Thread Clément Péron
This use devfreq variable that will be lock with spinlock in future patches. We should either introduce a function to access this one but as devfreq is optional let's just remove it. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron ---

[PATCH v5 13/14] [DO NOT MERGE] arm64: dts: allwinner: h6: Add GPU OPP table

2020-07-14 Thread Clément Péron
Add an Operating Performance Points table for the GPU to enable Dynamic Voltage & Frequency Scaling on the H6. The voltage range is set with minival voltage set to the target and the maximal voltage set to 1.2V. This allow DVFS framework to work properly on board with fixed regulator.

Re: fbconsole needs more parameter validations.

2020-07-14 Thread Tetsuo Handa
On 2020/07/11 15:16, Tetsuo Handa wrote: > On 2020/07/10 19:56, Greg Kroah-Hartman wrote: >> Where is the over/underflow happening here when we set a size to be so >> small? We should bound the size somewhere, and as you show, that's not >> really working properly, right? > > It is

[PATCH v5 02/14] drm/panfrost: clean headers in devfreq

2020-07-14 Thread Clément Péron
Don't include not required headers and sort them. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_devfreq.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git

[PATCH v4 13/14] [DO NOT MERGE] arm64: dts: allwinner: h6: Add GPU OPP table

2020-07-14 Thread Clément Péron
Add an Operating Performance Points table for the GPU to enable Dynamic Voltage & Frequency Scaling on the H6. The voltage range is set with minival voltage set to the target and the maximal voltage set to 1.2V. This allow DVFS framework to work properly on board with fixed regulator.

[PATCH v2 2/2] drm/msm/a6xx: add A640/A650 hwcg

2020-07-14 Thread Jonathan Marek
Initialize hardware clock-gating registers on A640 and A650 GPUs. At least for A650, this solves some performance issues. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/a6xx.xml.h | 8 ++ drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 117 -

[PATCH] f2fs: gc: fix the variable used in PTR_ERR()

2020-07-14 Thread Xu Wang
PTR_ERR should access the value just tested by IS_ERR, so fix the variable used in PTR_ERR(). Signed-off-by: Xu Wang --- drivers/gpu/drm/i915/gt/selftest_lrc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/gt/selftest_lrc.c

[PATCH v5 08/14] drm/panfrost: move devfreq_init()/fini() in device

2020-07-14 Thread Clément Péron
Later we will introduce devfreq probing regulator if they are present. As regulator should be probe only one time we need to get this logic in the device_init(). panfrost_device is already taking care of devfreq_resume() and devfreq_suspend(), so it's not totally illogic to move the

[PATCH v2 0/8] Initial SM8150 and SM8250 DPU bringup

2020-07-14 Thread Jonathan Marek
These patches bring up SM8150 and SM8250 with basic functionality. Tested with displayport output (single mixer, video mode case). v2: rebased Jonathan Marek (8): drm/msm/dpu: use right setup_blend_config for sm8150 and sm8250 drm/msm/dpu: update UBWC config for sm8150 and sm8250

Re: [PATCH 13/20] Documentation: mips/ingenic-tcu: eliminate duplicated word

2020-07-14 Thread Thomas Bogendoerfer
On Tue, Jul 07, 2020 at 11:04:07AM -0700, Randy Dunlap wrote: > Drop the doubled word "to". > > Signed-off-by: Randy Dunlap > Cc: Jonathan Corbet > Cc: linux-...@vger.kernel.org > Cc: Paul Cercueil > Cc: Thomas Bogendoerfer > Cc: linux-m...@vger.kernel.org > --- >

[PATCH v3] drm/msm: handle for EPROBE_DEFER for of_icc_get

2020-07-14 Thread Jonathan Marek
Check for errors instead of silently not using icc if the msm driver probes before the interconnect driver. Allow ENODATA for ocmem path, as it is optional and this error is returned when "gfx-mem" path is provided but not "ocmem". Because msm_gpu_cleanup assumes msm_gpu_init has been called,

Re: [Freedreno] [v1] drm/msm/dpu: enumerate second cursor pipe for external interface

2020-07-14 Thread kalyan_t
On 2020-07-10 22:19, Rob Clark wrote: On Thu, Jun 25, 2020 at 5:46 AM Kalyan Thota wrote: Setup an RGB HW pipe as cursor which can be used on secondary interface. For SC7180 2 HW pipes are enumerated as cursors 1 - primary interface 2 - secondary interface Signed-off-by: Kalyan Thota ---

[PATCH v2 7/8] drm/msm/dpu: add SM8150 to hw catalog

2020-07-14 Thread Jonathan Marek
This brings up basic video mode functionality for SM8150 DPU. Command mode and dual mixer/intf configurations are not working, future patches will address this. Scaler functionality and multiple planes is also untested. Signed-off-by: Jonathan Marek ---

Re: [Freedreno] [PATCH 0/9] drm/msm: Avoid possible infinite probe deferral and speed booting

2020-07-14 Thread Jeffrey Hugo
On Mon, Jul 13, 2020 at 8:11 AM Rob Herring wrote: > > On Fri, Jul 10, 2020 at 5:02 PM Douglas Anderson > wrote: > > > > I found that if I ever had a little mistake in my kernel config, > > or device tree, or graphics driver that my system would sit in a loop > > at bootup trying again and

[PATCH v2 2/8] drm/msm/dpu: update UBWC config for sm8150 and sm8250

2020-07-14 Thread Jonathan Marek
Update the UBWC registers to the right values for sm8150 and sm8250. This removes broken dpu_hw_reset_ubwc, which doesn't work because the "force blk offset to zero to access beginning of register region" hack is copied from downstream, where mapped region starts 0x1000 below what is used in the

[PATCH v5 00/14] Add regulator devfreq support to Panfrost

2020-07-14 Thread Clément Péron
Hi, This serie cleans and adds regulator support to Panfrost devfreq. This is mostly based on comment for the freshly introduced lima devfreq. We need to add regulator support because on Allwinner the GPU OPP table defines both frequencies and voltages. First patches [01-07] should not change

Re: [PATCH 0/4] drm: core: Convert logging to drm_* functions.

2020-07-14 Thread Suraj Upadhyay
On Fri, Jul 10, 2020 at 07:56:43PM +0200, Sam Ravnborg wrote: > Hi Suraj. > > On Tue, Jul 07, 2020 at 10:04:14PM +0530, Suraj Upadhyay wrote: > > > > This patchset converts logging to drm_* functions in drm core. > > > > The following functions have been converted to their respective > > DRM

[PATCH v2 8/8] drm/msm/dpu: add SM8250 to hw catalog

2020-07-14 Thread Jonathan Marek
This brings up basic video mode functionality for SM8250 DPU. Command mode and dual mixer/intf configurations are not working, future patches will address this. Scaler functionality and multiple planes is also untested. Signed-off-by: Jonathan Marek ---

Re: [Freedreno] [v1] drm/msm/dpu: add support for clk and bw scaling for display

2020-07-14 Thread kalyan_t
On 2020-07-10 22:38, Rob Clark wrote: On Thu, Jun 18, 2020 at 7:09 AM Kalyan Thota wrote: This change adds support to scale src clk and bandwidth as per composition requirements. Interconnect registration for bw has been moved to mdp device node from mdss to facilitate the scaling. Changes

[PATCH v5 04/14] drm/panfrost: introduce panfrost_devfreq struct

2020-07-14 Thread Clément Péron
Introduce a proper panfrost_devfreq to deal with devfreq variables. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_devfreq.c | 76 - drivers/gpu/drm/panfrost/panfrost_devfreq.h | 20 +-

[PATCH v8 1/2] dt-binding: Add DSI/LVDS TC358775 bridge bindings

2020-07-14 Thread Vinay Simha BN
- license modified to (GPL-2.0-only OR BSD-2-Clause) - single-link and dual-link lvds description and examples are added - proper indentation - VESA/JEIDA formats picked from panel-lvds dts - dsi data-lanes property removed, it will be picked from dsi0 ports - dual-link lvds port added and

Re: [PATCH 18/20] Documentation: security/keys: eliminate duplicated word

2020-07-14 Thread Mimi Zohar
On Tue, 2020-07-07 at 11:04 -0700, Randy Dunlap wrote: > Drop the doubled word "in". > > Signed-off-by: Randy Dunlap Reviewed-by: Mimi Zohar ___ dri-devel mailing list dri-devel@lists.freedesktop.org

Re: [PATCH v3 13/14] [DO NOT MERGE] arm64: dts: allwinner: h6: Add GPU OPP table

2020-07-14 Thread Clément Péron
Hi, On Fri, 10 Jul 2020 at 15:45, Piotr Oniszczuk wrote: > > > > > Wiadomość napisana przez Clément Péron w dniu > > 09.07.2020, o godz. 16:03: > > > > Add an Operating Performance Points table for the GPU to > > enable Dynamic Voltage & Frequency Scaling on the H6. > > > > The voltage range

[PATCH v4 2/3] drm/msm: reset devfreq freq_table/max_state before devfreq_add_device

2020-07-14 Thread Jonathan Marek
These never get set back to 0 when probing fails, so an attempt to probe again results in broken behavior. Fix the problem by setting thse to zero before they are used. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/msm_gpu.c | 4 1 file changed, 4 insertions(+) diff --git

Re: [PATCH 02/25] dma-fence: prime lockdep annotations

2020-07-14 Thread Jason Gunthorpe
On Fri, Jul 10, 2020 at 03:01:10PM +0200, Christian König wrote: > Am 10.07.20 um 14:54 schrieb Jason Gunthorpe: > > On Fri, Jul 10, 2020 at 02:48:16PM +0200, Christian König wrote: > > > Am 10.07.20 um 14:43 schrieb Jason Gunthorpe: > > > > On Thu, Jul 09, 2020 at 10:09:11AM +0200, Daniel Vetter

Re: [PATCH v2] drm/msm: handle for EPROBE_DEFER for of_icc_get

2020-07-14 Thread Jonathan Marek
On 7/9/20 11:15 AM, Rob Clark wrote: On Thu, Jul 9, 2020 at 7:35 AM Jonathan Marek wrote: Check for errors instead of silently not using icc if the msm driver probes before the interconnect driver. Allow ENODATA for ocmem path, as it is optional and this error is returned when "gfx-mem" path

[PATCH v4 09/14] drm/panfrost: dynamically alloc regulators

2020-07-14 Thread Clément Péron
We will later introduce regulators managed by OPP. Only alloc regulators when it's needed. This also help use to release the regulators only when they are allocated. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron ---

[PATCH] vgacon: fix a UAF in do_update_region()

2020-07-14 Thread Yang Yingliang
I got a UAF report in do_update_region() when I doing fuzz test. [ 51.161905] BUG: KASAN: use-after-free in do_update_region+0x579/0x600 [ 51.161918] Read of size 2 at addr 88800010 by task test/295 [ 51.161957] CPU: 2 PID: 295 Comm: test Not tainted 5.7.0+ #975 [ 51.161969]

[PATCH v4 1/3] drm/msm: fix unbalanced pm_runtime_enable in adreno_gpu_{init, cleanup}

2020-07-14 Thread Jonathan Marek
adreno_gpu_init calls pm_runtime_enable, so adreno_gpu_cleanup needs to call pm_runtime_disable. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/adreno_gpu.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/msm/adreno/adreno_gpu.c

Re: [PATCH 02/25] dma-fence: prime lockdep annotations

2020-07-14 Thread Jason Gunthorpe
On Thu, Jul 09, 2020 at 10:09:11AM +0200, Daniel Vetter wrote: > Hi Jason, > > Below the paragraph I've added after our discussions around dma-fences > outside of drivers/gpu. Good enough for an ack on this, or want something > changed? > > Thanks, Daniel > > > + * Note that only GPU drivers

Re: [PATCH 02/25] dma-fence: prime lockdep annotations

2020-07-14 Thread Jason Gunthorpe
On Fri, Jul 10, 2020 at 04:02:35PM +0200, Daniel Vetter wrote: > > dma_fence only possibly makes some sense if you intend to expose the > > completion outside a single driver. > > > > The prefered kernel design pattern for this is to connect things with > > a function callback. > > > > So the

[PATCH v5 06/14] drm/panfrost: properly handle error in probe

2020-07-14 Thread Clément Péron
Introduce a boolean to know if opp table has been added. With this, we can call panfrost_devfreq_fini() in case of error and release what has been initialised. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron ---

[PATCH v4 07/14] drm/panfrost: rename error labels in device_init

2020-07-14 Thread Clément Péron
Rename goto labels in device_init it will be easier to maintain. Reviewed-by: Alyssa Rosenzweig Reviewed-by: Steven Price Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_device.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git

Re: [RFC PATCH] interconnect: qcom: add functions to query addr/cmds for a path

2020-07-14 Thread Jonathan Marek
On 7/13/20 11:24 AM, Georgi Djakov wrote: On 7/1/20 07:25, Jonathan Marek wrote: The a6xx GMU can vote for ddr and cnoc bandwidth, but it needs to be able to query the interconnect driver for bcm addresses and commands. It's not very clear to me how the GMU firmware would be dealing with

[PATCH v4 3/3] drm/msm: handle for EPROBE_DEFER for of_icc_get

2020-07-14 Thread Jonathan Marek
Check for errors instead of silently not using icc if the msm driver probes before the interconnect driver. Allow ENODATA for ocmem path, as it is optional and this error is returned when "gfx-mem" path is provided but not "ocmem". Because msm_gpu_cleanup assumes msm_gpu_init has been called,

Re: [PATCH 4/6] staging: qlge: qlge_main: Simplify while statements.

2020-07-14 Thread Suraj Upadhyay
On Tue, Jul 14, 2020 at 02:41:37PM +0900, Benjamin Poirier wrote: > On 2020-07-13 17:50 +0530, Suraj Upadhyay wrote: > > Simplify while loops into more readable and simple for loops. > > > > Signed-off-by: Suraj Upadhyay > > --- > [...] > > @@ -1824,7 +1821,7 @@ static struct sk_buff

Re: [RFC PATCH] interconnect: qcom: add functions to query addr/cmds for a path

2020-07-14 Thread Georgi Djakov
On 7/1/20 07:25, Jonathan Marek wrote: > The a6xx GMU can vote for ddr and cnoc bandwidth, but it needs to be able > to query the interconnect driver for bcm addresses and commands. It's not very clear to me how the GMU firmware would be dealing with this? Does anyone have an idea whether the GMU

[PATCH] drm/vboxvideo: Replace HTTP links with HTTPS ones

2020-07-14 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

[PATCH 0/5] ARM: sunxi: Add support for MSI Primo73 tablet

2020-07-14 Thread Chen-Yu Tsai
From: Chen-Yu Tsai Hi everyone, This series add support for MSI's Primo73 tablet. This is a 7" tablet based on Allwinner's A20 SoC. The tablet uses some unknown LCD panel, with only serial number markings, and what appears to be a part number. Searching the Internet for said part number

[PATCH 2/5] drm/panel: simple: Set bpc from bits-per-color DT property for panel-dpi

2020-07-14 Thread Chen-Yu Tsai
From: Chen-Yu Tsai Now that the device tree binding for panel-dpi has the "bits-per-color" property, parse its value and set bpc in the panel description to the given value. This would allow encoders to detect less-than-8-bits color depth and employ color dithering if possible. Signed-off-by:

[PATCH 1/5] dt-bindings: display: panel-dpi: Add bits-per-color property

2020-07-14 Thread Chen-Yu Tsai
From: Chen-Yu Tsai Some LCD panels do not support 24-bit true color, or 8bits per channel RGB. Many low end ones only support up to 6 bits per channel natively. Add a device tree property to describe the native bit depth of the panel. This is separate from the bus width or format of the

[PATCH 3/5] dt-bindings: arm: sunxi: Add compatible for MSI Primo73 tablet

2020-07-14 Thread Chen-Yu Tsai
From: Chen-Yu Tsai Document board compatible name for MSI Primo73 tablet. Signed-off-by: Chen-Yu Tsai --- Documentation/devicetree/bindings/arm/sunxi.yaml | 5 + 1 file changed, 5 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml

[PATCH 5/5] ARM: dts: sun7i: Add MSI Primo73 tablet

2020-07-14 Thread Chen-Yu Tsai
From: Chen-Yu Tsai The Primo73 is an MSI branded Allwinner A20-based 7-inch tablet. It has a metal back case with a plastic insert around where the WiFi antenna is. The tablet is (as of July of 2020) no longer available from retailers. Kernel sources (as required by GPL) are no longer available

[PATCH 4/5] ARM: dts: sun7i: Add LCD0 RGB888 pins

2020-07-14 Thread Chen-Yu Tsai
From: Chen-Yu Tsai In some designs, the full 24 bits of RGB plus the control / sync signals for the LCD panel are used. Add a pinmux option for this. Signed-off-by: Chen-Yu Tsai --- arch/arm/boot/dts/sun7i-a20.dtsi | 12 1 file changed, 12 insertions(+) diff --git

Re: [PATCH] fbdev: Detect integer underflow at "struct fbcon_ops"->clear_margins.

2020-07-14 Thread Bartlomiej Zolnierkiewicz
[ Please Cc: fbdev Maintainer (happens to be me :) on fbdev patches, thanks. ] Hi, On 7/12/20 1:10 PM, Tetsuo Handa wrote: > I found that > > const int fd = open("/dev/fb0", O_ACCMODE); > struct fb_var_screeninfo var = { }; > ioctl(fd, FBIOGET_VSCREENINFO, ); > var.xres = var.yres =

[Bug 208539] New: Warning at drm_mod_object_add on when display is re-enabled (after display off)

2020-07-14 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=208539 Bug ID: 208539 Summary: Warning at drm_mod_object_add on when display is re-enabled (after display off) Product: Drivers Version: 2.5 Kernel Version: 5.8.0-rc5

Re: KASAN: use-after-free Read in drm_gem_object_release

2020-07-14 Thread Thomas Zimmermann
Hi Am 10.07.20 um 10:24 schrieb butt3rflyh4ck: > I report a bug (in linux-5.8.0-rc4) found by syzkaller. > > kernel config: > https://github.com/butterflyhack/syzkaller-fuzz/blob/master/v5.8.0-rc4.config > > I test the reproducer and crash too. > > In the drm_em_vram_t() function,

Re: [PATCH] drm/drm_fb_helper: fix fbdev with sparc64

2020-07-14 Thread Thomas Zimmermann
Hi Am 13.07.20 um 18:21 schrieb Daniel Vetter: > On Fri, Jul 10, 2020 at 08:28:16AM +0200, Thomas Zimmermann wrote: >> Hi >> >> Am 09.07.20 um 21:30 schrieb Sam Ravnborg: >>> Mark reported that sparc64 would panic while booting using qemu. >>> Mark bisected this to a patch that introduced generic

Re: [PATCH 1/1] drm/amdkfd: Add IPC API

2020-07-14 Thread Felix Kuehling
Am 2020-07-14 um 1:05 a.m. schrieb Li, Dennis: > [AMD Official Use Only - Internal Distribution Only] > > Hi, Felix, > amdgpu_gem_prime_export has different define in the old driver. I added > some comment in the below codes. > > Best Regards > Dennis Li > -Original Message- >

[PATCH v4 04/14] drm/panfrost: introduce panfrost_devfreq struct

2020-07-14 Thread Clément Péron
Introduce a proper panfrost_devfreq to deal with devfreq variables. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_devfreq.c | 76 - drivers/gpu/drm/panfrost/panfrost_devfreq.h | 20 +-

Re: [PATCH 02/25] dma-fence: prime lockdep annotations

2020-07-14 Thread Jason Gunthorpe
On Fri, Jul 10, 2020 at 02:48:16PM +0200, Christian König wrote: > Am 10.07.20 um 14:43 schrieb Jason Gunthorpe: > > On Thu, Jul 09, 2020 at 10:09:11AM +0200, Daniel Vetter wrote: > > > Hi Jason, > > > > > > Below the paragraph I've added after our discussions around dma-fences > > > outside of

[PATCH v4 00/14] Add regulator devfreq support to Panfrost

2020-07-14 Thread Clément Péron
Hi, This serie cleans and adds regulator support to Panfrost devfreq. This is mostly based on comment for the freshly introduced lima devfreq. We need to add regulator support because on Allwinner the GPU OPP table defines both frequencies and voltages. First patches [01-07] should not change

KASAN: use-after-free Read in drm_gem_object_release

2020-07-14 Thread butt3rflyh4ck
I report a bug (in linux-5.8.0-rc4) found by syzkaller. kernel config: https://github.com/butterflyhack/syzkaller-fuzz/blob/master/v5.8.0-rc4.config I test the reproducer and crash too. In the drm_em_vram_t() function, ttm_bo_init() function call ttm_bo_init_reserved(), the

[v5] dt-bindings: msm: disp: add yaml schemas for DPU and DSI bindings

2020-07-14 Thread Krishna Manikandan
MSM Mobile Display Subsytem (MDSS) encapsulates sub-blocks like DPU display controller, DSI etc. Add YAML schema for the device tree bindings for the same. Signed-off-by: Krishna Manikandan Changes in v2: - Changed dpu to DPU (Sam Ravnborg) - Fixed indentation issues (Sam

Re: [PATCH v6] drm/i915: Init lspcon after HPD in intel_dp_detect()

2020-07-14 Thread Kai-Heng Feng
> On Jun 30, 2020, at 16:37, Kai-Heng Feng wrote: > > >> On Jun 10, 2020, at 15:55, Kai-Heng Feng wrote: >> >> On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port >> becomes useless and never responds to cable hotplugging: >> [3.031904] [drm:lspcon_init [i915]]

[PATCH v3] vt: Reject zero-sized screen buffer size.

2020-07-14 Thread Tetsuo Handa
syzbot is reporting general protection fault in do_con_write() [1] caused by vc->vc_screenbuf == ZERO_SIZE_PTR caused by vc->vc_screenbuf_size == 0 caused by vc->vc_cols == vc->vc_rows == vc->vc_size_row == 0 caused by fb_set_var() from ioctl(FBIOPUT_VSCREENINFO) on /dev/fb0 , for gotoxy(vc, 0, 0)

[PATCH v4 01/14] drm/panfrost: avoid static declaration

2020-07-14 Thread Clément Péron
This declaration can be avoided so change it. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_devfreq.c | 38 ++--- 1 file changed, 18 insertions(+), 20 deletions(-) diff --git

Re: [PATCH v3 13/14] [DO NOT MERGE] arm64: dts: allwinner: h6: Add GPU OPP table

2020-07-14 Thread Piotr Oniszczuk
> Wiadomość napisana przez Clément Péron w dniu > 09.07.2020, o godz. 16:03: > > Add an Operating Performance Points table for the GPU to > enable Dynamic Voltage & Frequency Scaling on the H6. > > The voltage range is set with minival voltage set to the target > and the maximal voltage set

[PATCH] drm: omapdrm: Replace HTTP links with HTTPS ones

2020-07-14 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

[PATCH] fbdev: Detect integer underflow at "struct fbcon_ops"->clear_margins.

2020-07-14 Thread Tetsuo Handa
I found that const int fd = open("/dev/fb0", O_ACCMODE); struct fb_var_screeninfo var = { }; ioctl(fd, FBIOGET_VSCREENINFO, ); var.xres = var.yres = 1; ioctl(fd, FBIOPUT_VSCREENINFO, ); causes general protection fault in bitfill_aligned(), for vc_do_resize() updates vc->vc_{cols,rows}

Re: [PATCH v3 6/9] drm/bridge: ti-sn65dsi86: Use 18-bit DP if we can

2020-07-14 Thread Steev Klimaszewski
On 7/10/20 9:47 AM, Doug Anderson wrote: > Hi, > > > But should I continue on this path, > It's probably worth getting dithering working on your sdm845 anyway in > case anyone actually does put a 6bpp panel on this SoC. > > >> or should we be finding others who >> have an N61 and see what their

[PATCH v4 03/14] drm/panfrost: don't use pfdevfreq.busy_count to know if hw is idle

2020-07-14 Thread Clément Péron
This use devfreq variable that will be lock with spinlock in future patches. We should either introduce a function to access this one but as devfreq is optional let's just remove it. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron ---

Re: [Mesa-dev] [XDC 2020] Virtual conference + Call for Proposals extended 2 weeks more

2020-07-14 Thread Samuel Iglesias Gonsálvez
On 7/3/20 4:41 PM, Samuel Iglesias Gonsálvez wrote: > Hi, > > In the last meeting, X.Org Foundation board has decided that XDC 2020 > will be a virtual conference, given the uncertain COVID-19 situation in > Europe by September, including the possibility of a second wave, > outbreaks and travel

[PATCH v4 05/14] drm/panfrost: use spinlock instead of atomic

2020-07-14 Thread Clément Péron
Convert busy_count to a simple int protected by spinlock. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_devfreq.c | 43 +++-- drivers/gpu/drm/panfrost/panfrost_devfreq.h | 9 - 2 files changed,

[PATCH] fbdev: sm712fb: handle ioremap() errors in probe

2020-07-14 Thread Evgeny Novikov
smtcfb_pci_probe() does not handle ioremap() errors for case 0x720. The patch fixes that exactly like for case 0x710/2. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Evgeny Novikov --- drivers/video/fbdev/sm712fb.c | 8 1 file changed, 8 insertions(+)

[PATCH] drm: hold gem reference until object is no longer accessed

2020-07-14 Thread Steve Cohen
BUG: KASAN: use-after-free in drm_gem_open_ioctl There is potential for use-after-free here if the GEM object handle is closed between the idr lookup and retrieving the size from the object since a local reference is not being held at that point. Hold the local reference while the object can

Re: [PATCH 0/4] drm: core: Convert logging to drm_* functions.

2020-07-14 Thread Suraj Upadhyay
On Sun, Jul 12, 2020 at 12:07:45PM -0700, Joe Perches wrote: > On Mon, 2020-07-13 at 00:24 +0530, Suraj Upadhyay wrote: > > On Sat, Jul 11, 2020 at 11:16:33AM -0700, Joe Perches wrote: > [] > > > Perhaps change the __drm_printk macro to not > > > dereference the drm argument when NULL. > > > > >

[PATCH v5 10/14] drm/panfrost: add regulators to devfreq

2020-07-14 Thread Clément Péron
Some OPP tables specify voltage for each frequency. Devfreq can handle these regulators but they should be get only 1 time to avoid issue and know who is in charge. If OPP table is probe don't init regulator. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron

[PATCH v5 05/14] drm/panfrost: use spinlock instead of atomic

2020-07-14 Thread Clément Péron
Convert busy_count to a simple int protected by spinlock. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_devfreq.c | 43 +++-- drivers/gpu/drm/panfrost/panfrost_devfreq.h | 9 - 2 files changed,

[PATCH v5 09/14] drm/panfrost: dynamically alloc regulators

2020-07-14 Thread Clément Péron
We will later introduce regulators managed by OPP. Only alloc regulators when it's needed. This also help use to release the regulators only when they are allocated. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron ---

[PATCH] drm/tidss: Replace HTTP links with HTTPS ones

2020-07-14 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

Re: [PATCH v2] efi: avoid error message when booting under Xen

2020-07-14 Thread Bartlomiej Zolnierkiewicz
On 7/10/20 4:22 PM, Juergen Gross wrote: > efifb_probe() will issue an error message in case the kernel is booted > as Xen dom0 from UEFI as EFI_MEMMAP won't be set in this case. Avoid > that message by calling efi_mem_desc_lookup() only if EFI_MEMMAP is set. > > Fixes: 38ac0287b7f4

  1   2   3   >