Re: Drm: mgag200. Video adapter issue with 5.4.0-rc3 ; no graphics

2019-11-06 Thread Thomas Zimmermann
Hi John, apparently the vgaarb was not the problem. Am 07.11.19 um 03:29 schrieb John Donnelly: > Hi, > > I am investigating an issue where we lose video activity when the display is > switched from from “text mode” to “graphic mode” > on a number of servers using this driver.

[PATCH] drm/i915/gvt: fix dropping obj reference twice

2019-11-06 Thread Pan Bian
The reference count of obj will be decremented twice if error occurs in dma_buf_fd(). Additionally, attempting to read the reference count of obj after dropping reference may lead to a use after free bug. Here, we drop obj's reference until it is not used. Signed-off-by: Pan Bian ---

Re: [PATCH 1/7] dt-bindings: gpu: mali-midgard: Tidy up conversion to YAML

2019-11-06 Thread Andreas Färber
Am Mittwoch, den 06.11.2019, 08:24 -0600 schrieb Rob Herring: > On Sun, Nov 3, 2019 at 7:40 PM Andreas Färber > wrote: > > Instead of grouping alphabetically by third-party vendor, leading > > to > > one-element enums, sort by Mali model number, as done for Utgard. > > > > This already allows us

Re: Proposal to report GPU private memory allocations with sysfs nodes [plain text version]

2019-11-06 Thread Yiwei Zhang
For the sysfs approach, I'm assuming the upstream vendors still need to provide a pair of UMD and KMD, and this ioctl to label the BO is kept as driver private ioctl. Then will each driver just define their own set of "label"s and the KMD will only consume the corresponding ones so that the sysfs

Re: [PATCH V9 6/6] docs: sample driver to demonstrate how to implement virtio-mdev framework

2019-11-06 Thread Randy Dunlap
On 11/5/19 11:05 PM, Jason Wang wrote: > diff --git a/samples/Kconfig b/samples/Kconfig > index c8dacb4dda80..13a2443e18e0 100644 > --- a/samples/Kconfig > +++ b/samples/Kconfig > @@ -131,6 +131,16 @@ config SAMPLE_VFIO_MDEV_MDPY > mediated device. It is a simple framebuffer and supports

[PATCH libdrm v2] modetest: Add support for setting mode having floating vertical refresh rate

2019-11-06 Thread Devarsh Thakkar
For the scenario where user may require to modeset with a mode supporting a fractional value for vertical refresh-rate, appropriate mode can be selected by searching for mode having matching fractional vertical refresh rate using below equation. vrefresh = (1000 * pixel clock) / (htotal * vtotal)

Re: WARNING in drm_mode_createblob_ioctl

2019-11-06 Thread Dmitry Vyukov
On Wed, Nov 6, 2019 at 4:30 PM Daniel Vetter wrote: > > On Wed, Nov 6, 2019 at 4:20 PM syzbot > wrote: > > > > syzbot has bisected this bug to: > > > > commit 9e5a64c71b2f70ba530f8156046dd7dfb8a7a0ba > > Author: Kees Cook > > Date: Mon Nov 4 22:57:23 2019 + > > > > uaccess: disallow

[PATCH v2] drm/amdgpu: fix double reference dropping

2019-11-06 Thread Pan Bian
The reference to object fence is dropped at the end of the loop. However, it is dropped again outside the loop. The reference can be dropped immediately after calling dma_fence_wait() in the loop and thus the dropping operation outside the loop can be removed. Signed-off-by: Pan Bian --- v2: fix

Re: WARNING in drm_mode_createblob_ioctl

2019-11-06 Thread syzbot
syzbot has found a reproducer for the following crash on: HEAD commit:51309b9d Add linux-next specific files for 20191105 git tree: linux-next console output: https://syzkaller.appspot.com/x/log.txt?x=13f5c078e0 kernel config:

Drm: mgag200. Video adapter issue with 5.4.0-rc3

2019-11-06 Thread John Donnelly
Hi, I am investigating an issue where we lose video activity when the display is switched from from “text mode” to “graphic mode” on a number of servers using this driver. This adapter is Server Engines integrated Remote Video Acceleration Subsystem (RVAS) is used primarily as remote

[PATCH libdrm] modetest: Add support for setting mode having floating vertical refresh rate

2019-11-06 Thread Devarsh Thakkar
For the scenario where user may require to modeset with a mode supporting a fractional value for vertical refresh-rate, appropriate mode can be selected by searching for mode having matching fractional vertical refresh rate using below equation. vrefresh = 1000 * (pixel clock) / (htotal * vtotal)

[RFC PATCH 00/12] i2c: replace i2c_new_probed_device with an ERR_PTR variant

2019-11-06 Thread Wolfram Sang
From: Wolfram Sang In the on-going mission to let i2c_new_* calls return an ERR_PTR instead of NULL, here is a series converting i2c_new_probed_device(). A new function called i2c_new_scanned_device() is introduced with the new retval, but for now, a compatibility helper is provided until all

Re: [Freedreno] drm/msm: 'pp done time out' errors after async commit changes

2019-11-06 Thread Jeffrey Hugo
On Wed, Nov 6, 2019 at 9:30 AM Rob Clark wrote: > > On Wed, Nov 6, 2019 at 1:13 AM Brian Masney wrote: > > > > On Tue, Nov 05, 2019 at 08:23:27AM -0800, Rob Clark wrote: > > > On Tue, Nov 5, 2019 at 2:08 AM Brian Masney wrote: > > > > The 'pp done time out' errors go away if I revert the

[PATCH 2/3] drm/rockchip: add ability to handle external dphys in mipi-dsi

2019-11-06 Thread Heiko Stuebner
While the common case is that the dsi controller uses an internal dphy, accessed through the phy registers inside the dsi controller, there is also the possibility to use a separate dphy from a different vendor. One such case is the Rockchip px30 that uses a Innosilicon Mipi dphy, so add the

[RFC PATCH 05/12] video: fbdev: matrox: convert to i2c_new_scanned_device

2019-11-06 Thread Wolfram Sang
Move from the deprecated i2c_new_probed_device() to the new i2c_new_scanned_device(). Make use of the new ERRPTR if suitable. Signed-off-by: Wolfram Sang --- Build tested only. RFC, please comment and/or ack, but don't apply yet. drivers/video/fbdev/matrox/i2c-matroxfb.c | 4 ++-- 1 file

Re: [PATCH v2 01/15] mm/mmu_notifier: define the header pre-processor parts even if disabled

2019-11-06 Thread Jason Gunthorpe
On Tue, Nov 05, 2019 at 01:23:46PM -0800, John Hubbard wrote: > On 10/28/19 1:10 PM, Jason Gunthorpe wrote: > > From: Jason Gunthorpe > > > > Now that we have KERNEL_HEADER_TEST all headers are generally compile > > tested, so relying on makefile tricks to avoid compiling code that depends > >

[PATCH 1/3] drm/bridge/synopsys: dsi: move phy_ops callbacks around panel enablement

2019-11-06 Thread Heiko Stuebner
If implementation-specific phy_ops need to be defined they probably should be enabled before trying to talk to the panel and disabled only after the panel was disabled. Right now they are enabled last and disabled first, so might make it impossible to talk to some panels - example for this being

Re: WARNING in drm_mode_createblob_ioctl

2019-11-06 Thread syzbot
syzbot has bisected this bug to: commit 9e5a64c71b2f70ba530f8156046dd7dfb8a7a0ba Author: Kees Cook Date: Mon Nov 4 22:57:23 2019 + uaccess: disallow > INT_MAX copy sizes bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=125fe6dce0 start commit: 51309b9d Add

[PATCH] drm/amdgpu: fix potential double drop fence reference

2019-11-06 Thread Pan Bian
The object fence is not set to NULL after its reference is dropped. As a result, its reference may be dropped again if error occurs after that, which may lead to a use after free bug. To avoid the issue, fence is explicitly set to NULL after dropping its reference. Signed-off-by: Pan Bian ---

Re: [Freedreno] drm/msm: 'pp done time out' errors after async commit changes

2019-11-06 Thread Brian Masney
On Tue, Nov 05, 2019 at 08:23:27AM -0800, Rob Clark wrote: > On Tue, Nov 5, 2019 at 2:08 AM Brian Masney wrote: > > The 'pp done time out' errors go away if I revert the following three > > commits: > > > > cd6d923167b1 ("drm/msm/dpu: async commit support") > > d934a712c5e6 ("drm/msm: add atomic

[PATCH 3/3] drm/rockchip: dsi: add px30 support

2019-11-06 Thread Heiko Stuebner
Add the compatible and GRF definitions for the PX30 soc. Signed-off-by: Heiko Stuebner --- .../gpu/drm/rockchip/dw-mipi-dsi-rockchip.c | 27 +++ 1 file changed, 27 insertions(+) diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi-rockchip.c

[RESEND, v4, 2/7] dt-bindings: msm/mdp5: Document optional TBU and TBU_RT clocks

2019-11-06 Thread kholk11
From: AngeloGioacchino Del Regno These two clocks aren't present in all versions of the MDP5 HW: where present, they are needed to enable the Translation Buffer Unit(s). Signed-off-by: AngeloGioacchino Del Regno Acked-by: Rob Herring --- Documentation/devicetree/bindings/display/msm/mdp5.txt

Re: WARNING in drm_mode_createblob_ioctl

2019-11-06 Thread Dmitry Vyukov
On Wed, Nov 6, 2019 at 4:28 PM Daniel Vetter wrote: > > On Wed, Nov 6, 2019 at 4:23 PM Daniel Vetter wrote: > > > > On Wed, Nov 6, 2019 at 4:20 PM syzbot > > wrote: > > > > > > syzbot has bisected this bug to: > > > > > > commit 9e5a64c71b2f70ba530f8156046dd7dfb8a7a0ba > > > Author: Kees Cook

[PATCH] drm/amdgpu: fix double reference dropping

2019-11-06 Thread Pan Bian
After dropping the reference of object fence in the loop, it should be set to NULL to protecting dropping its reference again outside the loop. Signed-off-by: Pan Bian --- drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [RFC PATCH 00/12] i2c: replace i2c_new_probed_device with an ERR_PTR variant

2019-11-06 Thread Sean Young
On Wed, Nov 06, 2019 at 10:50:18AM +0100, Wolfram Sang wrote: > From: Wolfram Sang > > In the on-going mission to let i2c_new_* calls return an ERR_PTR instead > of NULL, here is a series converting i2c_new_probed_device(). A new > function called i2c_new_scanned_device() is introduced with the

Re: [PATCH 9/9] drm/ast: Enable atomic modesetting

2019-11-06 Thread Thomas Zimmermann
Hi Am 07.11.19 um 07:55 schrieb Gerd Hoffmann: >>> Hmm. Pretty much a dummy encoder implementation. Maybe ast is simple >>> enough that the simple pipe helpers can do the trick? >> >> As Daniel said, simple pipe helpers don't support cursors. I >> investigated his comment on a encoder helpers

Re: [PATCH 9/9] drm/ast: Enable atomic modesetting

2019-11-06 Thread Gerd Hoffmann
> > Hmm. Pretty much a dummy encoder implementation. Maybe ast is simple > > enough that the simple pipe helpers can do the trick? > > As Daniel said, simple pipe helpers don't support cursors. I > investigated his comment on a encoder helpers and found that many > drivers (including ast) only

[Bug 111481] AMD Navi GPU frequent freezes on both Manjaro/Ubuntu with kernel 5.3 and mesa 19.2 -git/llvm9

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=111481 --- Comment #219 from Shmerl --- (In reply to L.S.S. from comment #218) > > I'm not an expert of apitrace, but the reporter provided a trace that would > 100% reproduce the lockup, and he was able to bisect the call that caused > the lockup

[Bug 111481] AMD Navi GPU frequent freezes on both Manjaro/Ubuntu with kernel 5.3 and mesa 19.2 -git/llvm9

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=111481 --- Comment #218 from L.S.S. --- It seems the page fault issue has been already reported here. I also found similar page faults in the log sometimes when the lockup occurred (I think it'll definitely show up if I leave the system as is for a

[Bug 111481] AMD Navi GPU frequent freezes on both Manjaro/Ubuntu with kernel 5.3 and mesa 19.2 -git/llvm9

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=111481 --- Comment #217 from Shmerl --- (In reply to lptech1024 from comment #216) > >Hang occurred during a gaming cutscene. > >... > Nov 06 [SNIP] kernel: [drm:amdgpu_job_timedout [amdgpu]] *ERROR* ring > gfx_0.0.0 timeout, signaled seq=2827901,

[Bug 111481] AMD Navi GPU frequent freezes on both Manjaro/Ubuntu with kernel 5.3 and mesa 19.2 -git/llvm9

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=111481 --- Comment #216 from lptech1...@gmail.com --- GPU: PowerColor Red Devil Radeon RX 5700 XT using OC BIOS (default) Stock Fedora 31: Kernel 5.3.8, GNOME 3.34, Mesa 19.2.2, linux-firmware 20190923, LLVM 9.0.0 I experienced frequent hangs using

Re: [PATCH V8 0/6] mdev based hardware virtio offloading support

2019-11-06 Thread Jason Wang
On 2019/11/7 上午5:13, Alex Williamson wrote: On Wed, 6 Nov 2019 14:25:23 -0500 "Michael S. Tsirkin" wrote: On Wed, Nov 06, 2019 at 12:03:12PM -0700, Alex Williamson wrote: On Wed, 6 Nov 2019 11:56:46 +0800 Jason Wang wrote: On 2019/11/6 上午1:58, Alex Williamson wrote: On Tue, 5 Nov

[pull] amdgpu, radeon drm-fixes-5.4

2019-11-06 Thread Alex Deucher
Hi Dave, Daniel, Fixes for 5.4. The following changes since commit a99d8080aaf358d5d23581244e5da23b35e340b9: Linux 5.4-rc6 (2019-11-03 14:07:26 -0800) are available in the Git repository at: git://people.freedesktop.org/~agd5f/linux tags/drm-fixes-5.4-2019-11-06 for you to fetch changes

Re: [PATCH 2/3] drm: rcar-du: Recognize "renesas,vsps" in addition to "vsps"

2019-11-06 Thread Laurent Pinchart
Hi Geert, Thank you for the patch. On Tue, Nov 05, 2019 at 07:35:03PM +0100, Geert Uytterhoeven wrote: > The Renesas-specific "vsps" property lacks a vendor prefix. > Add a "renesas," prefix to comply with DT best practises. > Retain backward compatibility with old DTBs by falling back to "vsps"

Drm: mgag200. Video adapter issue with 5.4.0-rc3 ; no graphics

2019-11-06 Thread John Donnelly
Hi, I am investigating an issue where we lose video activity when the display is switched from from “text mode” to “graphic mode” on a number of servers using this driver.Specifically starting the GNOME desktop. This adapter is Server Engines Integrated Remote Video Acceleration

Re: [PATCH 1/3] dt-bindings: display: renesas: du: Add vendor prefix to vsps property

2019-11-06 Thread Laurent Pinchart
Hi Geert, Thank you for the patch. On Tue, Nov 05, 2019 at 07:35:02PM +0100, Geert Uytterhoeven wrote: > The Renesas-specific "vsps" property lacks a vendor prefix. > Add a "renesas," prefix to comply with DT best practises. > > Move "renesas,vsps" below "renesas,cmms" to preserve alphabetical

Re: [PATCH 3/3] arm64: dts: renesas: rcar-gen3: Replace "vsps" by "renesas,vsps"

2019-11-06 Thread Laurent Pinchart
Hi Geert, Thank you for the patch. On Tue, Nov 05, 2019 at 07:35:04PM +0100, Geert Uytterhoeven wrote: > The Renesas-specific "vsps" property lacks a vendor prefix. > Add a "renesas," prefix to comply with DT best practises. > > Signed-off-by: Geert Uytterhoeven > --- > This depends on '[PATCH

Re: [PATCH v2 07/18] infiniband: set FOLL_PIN, FOLL_LONGTERM via pin_longterm_pages*()

2019-11-06 Thread Ira Weiny
On Mon, Nov 04, 2019 at 04:57:38PM -0400, Jason Gunthorpe wrote: > On Mon, Nov 04, 2019 at 12:48:13PM -0800, John Hubbard wrote: > > On 11/4/19 12:33 PM, Jason Gunthorpe wrote: > > ... > > >> diff --git a/drivers/infiniband/core/umem.c > > >> b/drivers/infiniband/core/umem.c > > >> index

Re: [PATCH v2 05/18] mm/gup: introduce pin_user_pages*() and FOLL_PIN

2019-11-06 Thread Ira Weiny
> > > ... > >> +This document describes the following functions: :: > >> + > >> + pin_user_pages > >> + pin_user_pages_fast > >> + pin_user_pages_remote > >> + > >> + pin_longterm_pages > >> + pin_longterm_pages_fast > >> + pin_longterm_pages_remote > >> + > >> +Basic description of FOLL_PIN >

[GIT PULL] mediatek drm next for 5.5 - 2

2019-11-06 Thread CK Hu
Hi Dave, Daniel: This round add new color format support and reflect function. Regards, CK The following changes since commit 631005b255aab5f846f7ca03606613f898d70207: drm/mediatek: add no_clk into ddp private data (2019-10-09 16:58:13 +0800) are available in the Git repository at:

Re: [PATCH v2 02/15] mm/mmu_notifier: add an interval tree notifier

2019-11-06 Thread Jerome Glisse
On Wed, Nov 06, 2019 at 04:23:21PM -0800, John Hubbard wrote: > On 10/28/19 1:10 PM, Jason Gunthorpe wrote: [...] > > /** > > * enum mmu_notifier_event - reason for the mmu notifier callback > > @@ -32,6 +34,9 @@ struct mmu_notifier_range; > > * access flags). User should soft dirty the

Re: [PATCH v2 7/7] drm/mediatek: Support 180 degree rotation

2019-11-06 Thread CK Hu
Hi, Sean: On Wed, 2019-11-06 at 09:42 -0500, Sean Paul wrote: > On Wed, Nov 6, 2019 at 4:07 AM CK Hu wrote: > > > > Hi, Sean: > > > > On Tue, 2019-11-05 at 16:10 -0500, Sean Paul wrote: > > > From: Sean Paul > > > > > > Now that we support both reflections, we can expose 180 degree rotation > >

Re: [PATCH 1/3] dt-bindings: display: renesas: du: Add vendor prefix to vsps property

2019-11-06 Thread Rob Herring
On Tue, 5 Nov 2019 19:35:02 +0100, Geert Uytterhoeven wrote: > The Renesas-specific "vsps" property lacks a vendor prefix. > Add a "renesas," prefix to comply with DT best practises. > > Move "renesas,vsps" below "renesas,cmms" to preserve alphabetical sort > order. > > Fixes: 06711e6385a4ab4c

[Bug 111481] AMD Navi GPU frequent freezes on both Manjaro/Ubuntu with kernel 5.3 and mesa 19.2 -git/llvm9

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=111481 --- Comment #215 from Shmerl --- (In reply to Pierre-Eric Pelloux-Prayer from comment #141) > If anyone has a reliable way to trigger the issue, the most helpful thing to > do for now is an apitrace capture. Does the trace in comment #199 help

Re: [PATCH v2 02/15] mm/mmu_notifier: add an interval tree notifier

2019-11-06 Thread John Hubbard
On 10/28/19 1:10 PM, Jason Gunthorpe wrote: ... > include/linux/mmu_notifier.h | 98 +++ > mm/Kconfig | 1 + > mm/mmu_notifier.c| 533 +-- > 3 files changed, 607 insertions(+), 25 deletions(-) > > diff --git

Re: [Intel-gfx] [PATCH] drm/fbdev: Fallback to non tiled mode if all tiles not present

2019-11-06 Thread Manasi Navare
On Wed, Nov 06, 2019 at 12:50:21PM +1000, Dave Airlie wrote: > On Tue, 5 Nov 2019 at 07:00, Manasi Navare wrote: > > > > On Mon, Nov 04, 2019 at 07:48:26PM +1000, David Airlie wrote: > > > On Mon, Nov 4, 2019 at 7:18 PM Daniel Vetter wrote: > > > > > > > > On Thu, Oct 31, 2019 at 02:48:39PM

[PATCH] drm/msm/dpu: Add UBWC support for RGB8888 formats

2019-11-06 Thread Fritz Koenig
Hardware only natively supports BGR UBWC. UBWC support for RGB can be had by pretending that the buffer is BGR. Signed-off-by: Fritz Koenig --- drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c| 18 ++ .../drm/msm/disp/dpu1/dpu_hw_catalog_format.h | 2 ++ 2 files

Re: [PATCH V9 6/6] docs: sample driver to demonstrate how to implement virtio-mdev framework

2019-11-06 Thread Alex Williamson
On Wed, 6 Nov 2019 14:50:30 -0800 Randy Dunlap wrote: > On 11/5/19 11:05 PM, Jason Wang wrote: > > diff --git a/samples/Kconfig b/samples/Kconfig > > index c8dacb4dda80..13a2443e18e0 100644 > > --- a/samples/Kconfig > > +++ b/samples/Kconfig > > @@ -131,6 +131,16 @@ config SAMPLE_VFIO_MDEV_MDPY

[PULL] drm-intel-fixes

2019-11-06 Thread Rodrigo Vivi
Hi Dave and Daniel, Here goes drm-intel-fixes-2019-11-06: - Fix HPD poll to avoid kworker consuming a lot of cpu cycles. - Do not use TBT type for non Type-C ports. Thanks, Rodrigo. The following changes since commit a99d8080aaf358d5d23581244e5da23b35e340b9: Linux 5.4-rc6 (2019-11-03

Re: [PATCH V8 0/6] mdev based hardware virtio offloading support

2019-11-06 Thread Alex Williamson
On Wed, 6 Nov 2019 14:25:23 -0500 "Michael S. Tsirkin" wrote: > On Wed, Nov 06, 2019 at 12:03:12PM -0700, Alex Williamson wrote: > > On Wed, 6 Nov 2019 11:56:46 +0800 > > Jason Wang wrote: > > > > > On 2019/11/6 上午1:58, Alex Williamson wrote: > > > > On Tue, 5 Nov 2019 17:32:34 +0800 > >

Re: [Intel-gfx] [PATCH] drm/fbdev: Fallback to non tiled mode if all tiles not present

2019-11-06 Thread Manasi Navare
On Wed, Nov 06, 2019 at 12:47:16PM +1000, Dave Airlie wrote: > On Fri, 1 Nov 2019 at 07:46, Manasi Navare wrote: > > > > In case of tiled displays, if we hotplug just one connector, > > fbcon currently just selects the preferred mode and if it is > > tiled mode then that becomes a problem if rest

[PULL] drm-misc-next-fixes

2019-11-06 Thread Sean Paul
Hi Dave & Daniel, drm-misc-next-fixes is open and just a couple of patches this week. drm-misc-next-fixes-2019-11-06: -msm: Fix up a6xx debugbus register names (Sharat) -mst: Avoid u64 division (Sean) Cc: Sharat Masetty Cc: Sean Paul Cheers, Sean The following changes since commit

Re: [PATCH] drm/mst: Fix up u64 division

2019-11-06 Thread Sean Paul
On Wed, Nov 06, 2019 at 10:14:34PM +0200, Ville Syrjälä wrote: > On Wed, Nov 06, 2019 at 02:41:15PM -0500, Sean Paul wrote: > > From: Sean Paul > > > > Change rem_nsec to u32 since that's what do_div returns, this avoids the > > u64 divide in the drm_print args. > > > > Changes in v2: > > -

Re: [PATCH] drm/mst: Fix up u64 division

2019-11-06 Thread Ville Syrjälä
On Wed, Nov 06, 2019 at 02:41:15PM -0500, Sean Paul wrote: > From: Sean Paul > > Change rem_nsec to u32 since that's what do_div returns, this avoids the > u64 divide in the drm_print args. > > Changes in v2: > - Instead of doing do_div in drm_print, make rem_nsec u32 (Ville) > > Link to v1:

[Bug 111481] AMD Navi GPU frequent freezes on both Manjaro/Ubuntu with kernel 5.3 and mesa 19.2 -git/llvm9

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=111481 --- Comment #214 from Marco Liedtke --- Created attachment 145904 --> https://bugs.freedesktop.org/attachment.cgi?id=145904=edit dmesg with gpu recovery enabled -- You are receiving this mail because: You are the assignee for the

[PATCH] drm/mst: Fix up u64 division

2019-11-06 Thread Sean Paul
From: Sean Paul Change rem_nsec to u32 since that's what do_div returns, this avoids the u64 divide in the drm_print args. Changes in v2: - Instead of doing do_div in drm_print, make rem_nsec u32 (Ville) Link to v1:

[Bug 111481] AMD Navi GPU frequent freezes on both Manjaro/Ubuntu with kernel 5.3 and mesa 19.2 -git/llvm9

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=111481 --- Comment #213 from Marco Liedtke --- (In reply to wychuchol from comment #212) > (In reply to Marco Liedtke from comment #211) > > > > I have already set AMD_DEBUG=nodam in /etc/environment and in ~/.profile. > > Last time i played World of

[Bug 109955] amdgpu [RX Vega 64] system freeze while gaming

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=109955 --- Comment #131 from Wilko Bartels --- (In reply to haro41 from comment #130) > > > > > > echo "manual" > > > > /sys/class/drm/card0/device/power_dpm_force_performance_level > > > echo "1 2 3" > /sys/class/drm/card0/device/pp_dpm_mclk > > >

Re: [PATCH V8 0/6] mdev based hardware virtio offloading support

2019-11-06 Thread Michael S. Tsirkin
On Wed, Nov 06, 2019 at 12:03:12PM -0700, Alex Williamson wrote: > On Wed, 6 Nov 2019 11:56:46 +0800 > Jason Wang wrote: > > > On 2019/11/6 上午1:58, Alex Williamson wrote: > > > On Tue, 5 Nov 2019 17:32:34 +0800 > > > Jason Wang wrote: > > > > > >> Hi all: > > >> > > >> There are hardwares

Re: [PATCH V8 0/6] mdev based hardware virtio offloading support

2019-11-06 Thread Alex Williamson
On Wed, 6 Nov 2019 11:56:46 +0800 Jason Wang wrote: > On 2019/11/6 上午1:58, Alex Williamson wrote: > > On Tue, 5 Nov 2019 17:32:34 +0800 > > Jason Wang wrote: > > > >> Hi all: > >> > >> There are hardwares that can do virtio datapath offloading while > >> having its own control path. This

Re: [PATCH 1/2 v2] drm/atomic: fix self-refresh helpers crtc state dereference

2019-11-06 Thread Sean Paul
On Mon, Nov 04, 2019 at 09:37:36AM -0800, Rob Clark wrote: > From: Rob Clark > > drm_self_refresh_helper_update_avg_times() was incorrectly accessing the > new incoming state after drm_atomic_helper_commit_hw_done(). But this > state might have already been superceeded by an !nonblock atomic

[Bug 109955] amdgpu [RX Vega 64] system freeze while gaming

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=109955 --- Comment #130 from har...@gmx.de --- > > > > echo "manual" > > > /sys/class/drm/card0/device/power_dpm_force_performance_level > > echo "1 2 3" > /sys/class/drm/card0/device/pp_dpm_mclk > > > > do you have any suggestion to automate this?

Re: [PATCH] drm/mst: Fix up u64 division

2019-11-06 Thread Ville Syrjälä
On Wed, Nov 06, 2019 at 12:36:15PM -0500, Sean Paul wrote: > From: Sean Paul > > Use do_div to divide rem_nsec instead of /. > > Fixes: 12a280c72868 ("drm/dp_mst: Add topology ref history tracking for > debugging") > Cc: Juston Li > Cc: Imre Deak > Cc: Ville Syrjälä > Cc: Harry Wentland >

Re: [PATCH v15 1/5] dma-buf: Add dma-buf heaps framework

2019-11-06 Thread Andrew F. Davis
On 11/6/19 12:18 PM, Andrew F. Davis wrote: > On 11/6/19 12:03 PM, John Stultz wrote: >> On Wed, Nov 6, 2019 at 5:52 AM Andrew F. Davis wrote: >>> >>> On 11/5/19 11:22 PM, John Stultz wrote: +unsigned int dma_heap_ioctl_cmds[] = { + DMA_HEAP_IOC_ALLOC, +}; + +static

Re: [PATCH] drm: Limit to INT_MAX in create_blob ioctl

2019-11-06 Thread Andrew Morton
On Wed, 6 Nov 2019 09:24:18 -0800 Kees Cook wrote: > > Since this is -mm can I have a stable sha1 or something for > > referencing? Or do you want to include this in the -mm patch bomb for > > the merge window? > > Traditionally these things live in akpm's tree when they are fixes for > patches

Re: [PATCH] drm: Limit to INT_MAX in create_blob ioctl

2019-11-06 Thread Daniel Vetter
On Wed, Nov 6, 2019 at 6:24 PM Kees Cook wrote: > > On Wed, Nov 06, 2019 at 05:47:55PM +0100, Daniel Vetter wrote: > > The hardened usercpy code is too paranoid ever since: > > > > commit 6a30afa8c1fbde5f10f9c584c2992aa3c7f7a8fe > > Author: Kees Cook > > Date: Wed Nov 6 16:07:01 2019 +1100 > >

Re: drm/komeda: Use devm_platform_ioremap_resource() in komeda_dev_create()

2019-11-06 Thread Mihail Atanassov
On Wednesday, 9 October 2019 09:57:11 GMT james qian wang (Arm Technology China) wrote: > On Sat, Sep 21, 2019 at 07:50:46PM +0200, Markus Elfring wrote: > > From: Markus Elfring > > Date: Sat, 21 Sep 2019 19:43:51 +0200 > > > > Simplify this function implementation by using a known wrapper

[PATCH 4/4] drm/amdgpu: Avoid accidental thread thread reactivation.

2019-11-06 Thread Andrey Grodzovsky
Problem: During GPU reset we call the GPU scheduler to suspend it's thread, those two functions in amdgpu also suspend and resume the sceduler for their needs but this can collide with GPU reset in progress and accidently restart a suspended thread before time. Fix: Serialize with GPU reset.

[PATCH 2/4] drm/sched: Use completion to wait for entity idle

2019-11-06 Thread Andrey Grodzovsky
Removes thread park/unpark hack from drm_sched_entity_fini and by this fixes reactivation of scheduler thread while the thread is supposed to be stopped. Signed-off-by: Andrey Grodzovsky Suggested-by: Christian König --- drivers/gpu/drm/scheduler/sched_entity.c | 12

[PATCH 3/4] drm/sched: Avoid job cleanup if sched thread is parked.

2019-11-06 Thread Andrey Grodzovsky
When the sched thread is parked we assume ring_mirror_list is not accessed from here. Signed-off-by: Andrey Grodzovsky --- drivers/gpu/drm/scheduler/sched_main.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/scheduler/sched_main.c

[PATCH 1/4] Revert "drm/amdgpu: dont schedule jobs while in reset"

2019-11-06 Thread Andrey Grodzovsky
This reverts commit 3cdf9bd0089723c468d5f6240e54d1afa52e9a04. We will do a proper fix in next patch. Signed-off-by: Andrey Grodzovsky --- drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c

Re: [PATCH] drm/shmem: Add docbook comments for drm_gem_shmem_object madvise fields

2019-11-06 Thread Sean Paul
On Fri, Nov 01, 2019 at 10:37:54AM -0500, Rob Herring wrote: > Add missing docbook comments to madvise fields in struct > drm_gem_shmem_object which fixes these warnings: > > include/drm/drm_gem_shmem_helper.h:87: warning: Function parameter or member > 'madv' not described in

Re: [Freedreno] [PATCH] drm: msm: a6xx: fix debug bus register configuration

2019-11-06 Thread Sean Paul
On Wed, Nov 06, 2019 at 05:19:23PM +0530, Sharat Masetty wrote: > Fix the cx debugbus related register configuration, to collect accurate > bus data during gpu snapshot. This helps with complete snapshot dump > and also complete proper GPU recovery. > > Signed-off-by: Sharat Masetty Applied to

[PATCH] drm/mst: Fix up u64 division

2019-11-06 Thread Sean Paul
From: Sean Paul Use do_div to divide rem_nsec instead of /. Fixes: 12a280c72868 ("drm/dp_mst: Add topology ref history tracking for debugging") Cc: Juston Li Cc: Imre Deak Cc: Ville Syrjälä Cc: Harry Wentland Cc: Daniel Vetter Cc: Sean Paul Cc: Lyude Paul Cc: Maarten Lankhorst Cc:

[Bug 109955] amdgpu [RX Vega 64] system freeze while gaming

2019-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=109955 --- Comment #129 from Wilko Bartels --- (In reply to haro41 from comment #122) > In my experience, this issue is related to mclk switching and it affects the > lowest mclk level only. > > So you guy's can save a lot of power, if you, insteed

Re: [PATCH] drm: Limit to INT_MAX in create_blob ioctl

2019-11-06 Thread Kees Cook
On Wed, Nov 06, 2019 at 05:47:55PM +0100, Daniel Vetter wrote: > The hardened usercpy code is too paranoid ever since: > > commit 6a30afa8c1fbde5f10f9c584c2992aa3c7f7a8fe > Author: Kees Cook > Date: Wed Nov 6 16:07:01 2019 +1100 > > uaccess: disallow > INT_MAX copy sizes > > Code itself

Re: [PATCH v2] drm/amdgpu: fix double reference dropping

2019-11-06 Thread Alex Deucher
On Wed, Nov 6, 2019 at 7:48 AM Koenig, Christian wrote: > > Am 06.11.19 um 12:35 schrieb Pan Bian: > > The reference to object fence is dropped at the end of the loop. > > However, it is dropped again outside the loop. The reference can be > > dropped immediately after calling dma_fence_wait() in

Re: [Freedreno] [PATCH] drm: msm: a6xx: fix debug bus register configuration

2019-11-06 Thread Jordan Crouse
On Wed, Nov 06, 2019 at 08:18:59AM -0800, Rob Clark wrote: > On Wed, Nov 6, 2019 at 3:49 AM Sharat Masetty wrote: > > > > Fix the cx debugbus related register configuration, to collect accurate > > bus data during gpu snapshot. This helps with complete snapshot dump > > and also complete proper

Re: [PATCH] drm/amd/powerplay: fix struct init in renoir_print_clk_levels

2019-11-06 Thread Alex Deucher
On Tue, Nov 5, 2019 at 6:10 PM Raul E Rangel wrote: > > drivers/gpu/drm/amd/powerplay/renoir_ppt.c:186:2: error: missing braces > around initializer [-Werror=missing-braces] > SmuMetrics_t metrics = {0}; > ^ > > Fixes: 8b8031703bd7 ("drm/amd/powerplay: implement sysfs for getting dpm >

Re: [PATCH] drm/amdgpu: fix potential double drop fence reference

2019-11-06 Thread Alex Deucher
On Wed, Nov 6, 2019 at 4:39 AM Koenig, Christian wrote: > > Am 06.11.19 um 10:14 schrieb Pan Bian: > > The object fence is not set to NULL after its reference is dropped. As a > > result, its reference may be dropped again if error occurs after that, > > which may lead to a use after free bug. To

Re: [PATCH v15 1/5] dma-buf: Add dma-buf heaps framework

2019-11-06 Thread Andrew F. Davis
On 11/6/19 12:03 PM, John Stultz wrote: > On Wed, Nov 6, 2019 at 5:52 AM Andrew F. Davis wrote: >> >> On 11/5/19 11:22 PM, John Stultz wrote: >>> +unsigned int dma_heap_ioctl_cmds[] = { >>> + DMA_HEAP_IOC_ALLOC, >>> +}; >>> + >>> +static long dma_heap_ioctl(struct file *file, unsigned int

Re: [PATCH 7/7] drm/amdgpu: remove set but not used variable 'mc_shared_chmap'

2019-11-06 Thread Alex Deucher
On Mon, Nov 4, 2019 at 8:54 AM yu kuai wrote: > > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c: In function > ‘gfx_v8_0_gpu_early_init’: > drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1713:6: warning: variable > ‘mc_shared_chmap’ set but not used

Re: [PATCH] drm: Limit to INT_MAX in create_blob ioctl

2019-11-06 Thread Ville Syrjälä
On Wed, Nov 06, 2019 at 05:47:55PM +0100, Daniel Vetter wrote: > The hardened usercpy code is too paranoid ever since: > > commit 6a30afa8c1fbde5f10f9c584c2992aa3c7f7a8fe > Author: Kees Cook > Date: Wed Nov 6 16:07:01 2019 +1100 > > uaccess: disallow > INT_MAX copy sizes > > Code itself

Re: [PATCH v15 1/5] dma-buf: Add dma-buf heaps framework

2019-11-06 Thread John Stultz
On Wed, Nov 6, 2019 at 5:52 AM Andrew F. Davis wrote: > > On 11/5/19 11:22 PM, John Stultz wrote: > > +unsigned int dma_heap_ioctl_cmds[] = { > > + DMA_HEAP_IOC_ALLOC, > > +}; > > + > > +static long dma_heap_ioctl(struct file *file, unsigned int ucmd, > > +unsigned

[PATCH 7/7] drm/mcde: Handle pending vblank while disabling display

2019-11-06 Thread Stephan Gerhold
Disabling the display using MCDE currently results in a warning together with a delay caused by some timeouts: mcde a035.mcde: MCDE display is disabled [ cut here ] WARNING: CPU: 0 PID: 20 at drivers/gpu/drm/drm_atomic_helper.c:2258

[PATCH 3/7] drm/mcde: dsi: Make video mode errors more verbose

2019-11-06 Thread Stephan Gerhold
Triggering an error conditions in DSI video mode only results in a very generic "some video mode error status" error message at the moment. Make this more clear by adding separate error messages for each bit. Signed-off-by: Stephan Gerhold --- drivers/gpu/drm/mcde/mcde_dsi.c | 22

[PATCH 0/7] drm/mcde: DSI video mode fixes

2019-11-06 Thread Stephan Gerhold
This is a collection of fixes to make DSI video mode work better using the MCDE driver. With these changes, MCDE appears to work properly for the video mode panel I have been testing with. Note: The patch that fixes the DSI link register setup for video mode [1] is still necessary; but we still

[PATCH 5/7] drm/mcde: dsi: Fix duplicated DSI connector

2019-11-06 Thread Stephan Gerhold
Using a (single) DSI display with MCDE currently results in two "connected" connectors: Connector: DSI-1 id : 34 encoder id : 0 conn : connected size : 0x0 (mm) count_modes: 0 count_props: 5

[PATCH 6/7] drm/mcde: dsi: Enable clocks in pre_enable() instead of mode_set()

2019-11-06 Thread Stephan Gerhold
The DSI initialization sequence incorrectly assumes that the mode_set() function of the DRM bridge is always called when (re-)enabling the display. This is not necessarily the case. Keeping the device idle in the framebuffer console for a while results in the display being turned off using the

[PATCH 1/7] drm/mcde: Provide vblank handling unconditionally

2019-11-06 Thread Stephan Gerhold
At the moment, vblank handling is only enabled together with TE synchronization. However, the vblank IRQ is also working with on displays without TE synchronization (e.g. DSI video mode panels). It seems like the vblank IRQ is actually generated by the MCDE hardware for the channel. Therefore,

[PATCH 2/7] drm/mcde: Fix frame sync setup for video mode panels

2019-11-06 Thread Stephan Gerhold
The MCDE driver differentiates only between "te_sync" (for hardware TE0 sync) and software sync (i.e. manually triggered updates) at the moment. However, none of these options work correctly for video mode panels. Therefore, we need to make some changes to make them work correctly: - Select

[PATCH 4/7] drm/mcde: dsi: Delay start of video stream generator

2019-11-06 Thread Stephan Gerhold
The initialization order for DSI video mode is important - if we enable the video stream generator (VSG) before the MCDE DSI formatter starts sending pixel data, it will immediately run into an error and disable itself again. Avoid this problem by delaying the activation of the VSG until the MCDE

Re: [Freedreno] drm/msm: 'pp done time out' errors after async commit changes

2019-11-06 Thread Rob Clark
On Wed, Nov 6, 2019 at 8:47 AM Jeffrey Hugo wrote: > > On Wed, Nov 6, 2019 at 9:30 AM Rob Clark wrote: > > > > On Wed, Nov 6, 2019 at 1:13 AM Brian Masney wrote: > > > > > > On Tue, Nov 05, 2019 at 08:23:27AM -0800, Rob Clark wrote: > > > > On Tue, Nov 5, 2019 at 2:08 AM Brian Masney > > > >

Re: Proposal to report GPU private memory allocations with sysfs nodes [plain text version]

2019-11-06 Thread Rob Clark
On Tue, Nov 5, 2019 at 1:47 AM Daniel Vetter wrote: > > On Mon, Nov 04, 2019 at 11:34:33AM -0800, Yiwei Zhang wrote: > > Hi folks, > > > > (Daniel, I just moved you to this thread) > > > > Below are the latest thoughts based on all the feedback and comments. > > > > First, I need to clarify on

[PATCH] drm: Limit to INT_MAX in create_blob ioctl

2019-11-06 Thread Daniel Vetter
The hardened usercpy code is too paranoid ever since: commit 6a30afa8c1fbde5f10f9c584c2992aa3c7f7a8fe Author: Kees Cook Date: Wed Nov 6 16:07:01 2019 +1100 uaccess: disallow > INT_MAX copy sizes Code itself should have been fine as-is. Reported-by:

[PATCH v2 3/4] drm: imx: Add i.MX 6 MIPI DSI host driver

2019-11-06 Thread Adrian Ratiu
This adds support for the Synopsis DesignWare MIPI DSI v1.01 host controller which is embedded in i.MX 6 SoCs. Based on following patches, but updated/extended to work with existing support found in the kernel: - drm: imx: Support Synopsys DesignWare MIPI DSI host controller Signed-off-by: Liu

[PATCH v2 2/4] drm: bridge: dw_mipi_dsi: abstract register access using reg_fields

2019-11-06 Thread Adrian Ratiu
Register existence, address/offsets, field layouts, reserved bits and so on differ between MIPI-DSI versions and between SoC vendor boards. Despite these differences the hw IP and protocols are mostly the same so the generic driver can be made to compensate these differences. The current Rockchip

[PATCH v2 0/4] Genericize DW MIPI DSI bridge and add i.MX 6 driver

2019-11-06 Thread Adrian Ratiu
in advance for any breakage! Many thanks to Boris Brezillon for suggesting the regmap solution and to Liu Ying for doing the initial i.MX platform driver implementation. This series applies on top of latest linux-next tree, next-20191106. v1 -> v2: * Moved the register definitions &

[PATCH v2 1/4] drm: bridge: dw_mipi_dsi: access registers via a regmap

2019-11-06 Thread Adrian Ratiu
Convert the common bridge code and the two rockchip & stm drivers which currently use it to the regmap API in anticipation for further changes to make it more generic and add older DSI host controller support as found on i.mx6 based devices. The regmap becomes an internal state of the bridge. No

[PATCH v2 4/4] dt-bindings: display: add IMX MIPI DSI host controller doc

2019-11-06 Thread Adrian Ratiu
Reviewed-by: Neil Armstrong Reviewed-by: Emil Velikov Signed-off-by: Sjoerd Simons Signed-off-by: Martyn Welch Signed-off-by: Adrian Ratiu --- .../bindings/display/imx/mipi-dsi.txt | 56 +++ 1 file changed, 56 insertions(+) create mode 100644

  1   2   3   >