Re: [PATCH 7/7] drm/ttm: enable swapout of per VM BOs during allocation and allows reaping of deleted BOs

2017-12-20 Thread Thomas Hellstrom
What about "Enable recursive locking at swapout time to make it possible to swap out BOs that share the same reservation object." Is "per VM BOs" an AMD specific name?  In that case, I'd avoid using it in the TTM code since most people have no idea what they are and why the need specific

Re: [Intel-gfx] [PATCH] drm/syncobj: Stop reusing the same struct file for all syncobj -> fd

2017-12-20 Thread Chris Wilson
Quoting Dave Airlie (2017-12-21 02:42:56) > > @@ -494,12 +473,11 @@ static int drm_syncobj_fd_to_handle(struct drm_file > > *file_private, > > spin_unlock(_private->syncobj_table_lock); > > idr_preload_end(); > > > > - if (ret < 0) { > > - fput(syncobj->file);

Re: [PATCH 6/7] drm/ttm: add ttm_bo_evict_swapout_allowable to check bo is allowable to evict or swapout

2017-12-20 Thread Thomas Hellstrom
On 12/20/2017 11:35 AM, Roger He wrote: extract this function since eviction and swapout share same logic But it's only used in eviction? Change-Id: I80a475a93fceed8d66d74a1832c815a0756341ac Signed-off-by: Roger He --- drivers/gpu/drm/ttm/ttm_bo.c | 29

Re: [PATCH 1/7] drm/ttm: call ttm_bo_swapout directly when ttm shrink

2017-12-20 Thread Thomas Hellstrom
Roger, 5 out of 7 patches in this series are completely lacking a commit log message, and thats not OK. Really. https://www.kernel.org/doc/html/v4.12/process/submitting-patches.html#describe-your-changes I'll review these, but IIRC the no_wait in the memory accounting code is different in

[Bug 198221] nouveau DRM driver scheduling invalid work

2017-12-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=198221 --- Comment #3 from Petr Vandrovec (p...@vandrovec.name) --- Created attachment 261285 --> https://bugzilla.kernel.org/attachment.cgi?id=261285=edit Bandaid to prevent processing of works not fully set up I've attached bandaid that seems to

[PATCH] drm: move lease init after validation in drm_lease_create

2017-12-20 Thread Keith Packard
Patch bd36d3bab2e3d08f80766c86487090dbceed4651 fixed a deadlock in the failure path of drm_lease_create. This made the partially initialized lease object visible for a short window of time. To avoid having the lessee state appear transiently, I've rearranged the code so that the lessor fields are

Re: [PATCH] drm/dp: Power cycle display if LINK_ADDRESS fails.

2017-12-20 Thread Jani Nikula
On Wed, 20 Dec 2017, Dhinakaran Pandiyan wrote: > Occasionally there are LINK_ADDRESS sideband messages timing out with the > Lenovo MST dock + Dell MST monitor(w/ in-built branch) setup I have. These > failures lead to the display not coming up on boot. Power

[PATCH] drm/dp: Power cycle display if LINK_ADDRESS fails.

2017-12-20 Thread Dhinakaran Pandiyan
Occasionally there are LINK_ADDRESS sideband messages timing out with the Lenovo MST dock + Dell MST monitor(w/ in-built branch) setup I have. These failures lead to the display not coming up on boot. Power cycling the port corresponding to the MST monitor's branch device and resending the message

[PATCH i-g-t 1/2] tests/kms_sequence: Add tests for new CRTC get/queue sequence ioctls [v2]

2017-12-20 Thread Keith Packard
These ioctls replace drmWaitVBlank and add ns time resolution and 64-bit sequence numbers to comply with the Vulkan API specifications. The tests were derived from the existing kms_vblank tests with the 'wait' variant elided as the new API doesn't provide a mechanism for blocking in the kernel.

[PATCH i-g-t 0/2] tests: Add tests for new sequence API and leases

2017-12-20 Thread Keith Packard
Both of these APIs are now in the kernel; it would be nice to have the tests integrated into the test suite. This includes an updated version of the kms_lease patch (v3) which adapts to the final kernel API changes. ___ dri-devel mailing list

[PATCH i-g-t 2/2] tests/kms_lease: add tests for lease ioctls [v3]

2017-12-20 Thread Keith Packard
Validate that the leasing API creates leases that allow access to a subset of the available resources and that lease revocation works. v2: from Dave Airlie * Update ioctl numbers to latest proposed values. * Fix commit message * Add tests for get_lease and list_lessees

[RFC 3/3] drm/msm: Don't subclass drm_atomic_state anymore

2017-12-20 Thread Archit Taneja
With the addition of "private_objs" in drm_atomic_state, we no longer need to subclass drm_atomic_state to store state of share resources that don't perfectly fit within planes/crtc/connector state information. We can now save this state within drm_atomic_state itself using the private objects.

[RFC 0/3] drm/msm: Avoid subclassing of drm_atomic_state

2017-12-20 Thread Archit Taneja
It's been recommended that we use drm_private_objs embedded in drm_atomic_state to hold shared resources instead of subclassing drm_atomic_state. This will also help us in getting one step closer to using the atomic commit helpers instead of the msm_atomic_commit() funcs in msm_atomic.c I've

[RFC 1/3] drm/msm/mdp5: Add global state as a private atomic object

2017-12-20 Thread Archit Taneja
Global shared resources (hwpipes, hwmixers and SMP) for MDP5 are implemented as a part of atomic state by subclassing drm_atomic_state. The preferred approach is to use the drm_private_obj infrastructure available in the atomic core. mdp5_global_state is introduced as a drm atomic private

[RFC 2/3] drm/msm/mdp5: Use the new private_obj state

2017-12-20 Thread Archit Taneja
This replaces the usage of the subclassed atomic state (mdp5_state) with a private_obj state embedded within drm_atomic_state. The latter method is the preferred approach, since it's simpler to implement and less prone to errors. The new API replaces the older and equivalent mdp5_state usage in

RE: [PATCH 3/7] drm/ttm: use an operation ctx for ttm_mem_global_alloc_page

2017-12-20 Thread He, Roger
-Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Wednesday, December 20, 2017 9:36 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH 3/7] drm/ttm: use an operation ctx for

linux-next: Signed-off-by missing for commits in the drm tree

2017-12-20 Thread Stephen Rothwell
Hi all, Commits bb5cdf8d1c76 ("drm: omapdrm: Remove filename from header and fix copyright tag") d66c36a3ee79 ("drm: omapdrm: Simplify platform registration") are missing a Signed-off-by from their committer. -- Cheers, Stephen Rothwell ___

[Bug 198221] nouveau DRM driver scheduling invalid work

2017-12-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=198221 Petr Vandrovec (p...@vandrovec.name) changed: What|Removed |Added Regression|No |Yes --- Comment

Re: [Intel-gfx] [PATCH] drm/syncobj: Stop reusing the same struct file for all syncobj -> fd

2017-12-20 Thread Dave Airlie
> @@ -494,12 +473,11 @@ static int drm_syncobj_fd_to_handle(struct drm_file > *file_private, > spin_unlock(_private->syncobj_table_lock); > idr_preload_end(); > > - if (ret < 0) { > - fput(syncobj->file); > - return ret; > - } > -

[Bug 198221] nouveau DRM driver scheduling invalid work

2017-12-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=198221 Ilia Mirkin (imir...@alum.mit.edu) changed: What|Removed |Added CC||imir...@alum.mit.edu

Re: [Mesa-dev] Allocator Nouveau driver, Mesa EXT_external_objects, and DRM metadata import interfaces

2017-12-20 Thread Ilia Mirkin
On Wed, Dec 20, 2017 at 6:22 PM, Kristian Kristensen wrote: > On Wed, Dec 20, 2017 at 12:41 PM, Miguel Angel Vico > wrote: >> On Wed, 20 Dec 2017 11:54:10 -0800 >> Kristian Høgsberg wrote: >> > I'd like to see concrete examples of

Re: [RFC PATCH v2 1/1] drm/tegra: sor: Fix hang on tegra124 due to NULL clk_out

2017-12-20 Thread Thierry Reding
On Wed, Dec 20, 2017 at 11:32:23AM +, Guillaume Tucker wrote: > When neither HDMI nor DP is supported such as on the tegra124, the > sor->clk_out is not initialised and remains NULL. In this case, the > parent clock can't be assigned to it so revert to the previous > behaviour of assigning it

[Bug 102130] pp_dpm_mclk always on level 1 (cant set to level 0) rx 480

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102130 --- Comment #5 from Alex Deucher --- Is this still an issue? -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list

[Bug 97471] kworker consumes 100% of a cpu core when screen sleeps with amdgpu kernel driver.

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97471 Alex Deucher changed: What|Removed |Added Status|NEW |RESOLVED

[Bug 94973] R9 380X (Tonga) SMC failure with pcie_dpm_key_disabled=0

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94973 --- Comment #2 from Alex Deucher --- Is this still an issue? -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list

[Bug 97993] amdgpu_query_gpu_info () segfaults when user does not have permission to open /dev/dri/card*

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97993 Alex Deucher changed: What|Removed |Added Status|NEW |RESOLVED

Re: [PATCH v3 2/5] drm/tegra: Restore opaque and drop alpha formats on Tegra20/30

2017-12-20 Thread Thierry Reding
On Wed, Dec 20, 2017 at 06:46:11PM +0300, Dmitry Osipenko wrote: > Commit 7772fdaef939 ("drm/tegra: Support ARGB and ABGR formats") broke > DRM's MODE_ADDFB IOCTL on Tegra20/30, because IOCTL uses XRGB format if > requested FB depth is 24bpp. As a result, Xorg doesn't work anymore with > both

[Bug 96866] REGRESSION System frozen after resume

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96866 Alex Deucher changed: What|Removed |Added Status|NEW |RESOLVED

[Bug 104347] AMD RX 580: Hide/Show window sometimes corrupts screen (see screenshot)

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104347 --- Comment #2 from Michel Dänzer --- Please attach the corresponding glxinfo output. Did this not happen with an older version of Mesa / LLVM? If so, can you bisect? -- You are receiving this mail because: You are the

RE: [PATCH 1/3] drm/prime: forward begin_cpu_access callback to drivers

2017-12-20 Thread Li, Samuel
Ping... can someone please review this patch? Samuel Li > -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Samuel Li > Sent: Friday, December 15, 2017 11:28 AM > To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org > Cc:

[Bug 104347] AMD RX 580: Hide/Show window sometimes corrupts screen (see screenshot)

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104347 --- Comment #1 from network...@rkmail.ru --- I had the same issue with my RX480, but it does not happen on hd7950 with amdgpu. -- You are receiving this mail because: You are the assignee for the

Re: [RFC PATCH v2 03/13] bootsplash: Flush framebuffer after drawing

2017-12-20 Thread Max Staudt
On 12/20/2017 04:35 PM, Ray Strode wrote: > Hi, > >> Actually, I don't want that :) >> >> This was a design decision that I've made to keep the code small and simple >> to audit. >> As it is, the simple bootsplash code will make 99% of people happy. > You think only 1% of linux users have more

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Max Staudt
On 12/20/2017 04:21 PM, Ray Strode wrote: > If we've reached the scenario you're discussing above, the real > failure is that the KMS > driver took too long to load. DRM is the platform graphics api. If > it's not loading > timely enough to show graphics then that's the problem! It sounds > like

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Max Staudt
On 12/20/2017 04:19 PM, Daniel Vetter wrote: > On Wed, Dec 20, 2017 at 4:11 PM, Daniel Vetter wrote: >> On Wed, Dec 20, 2017 at 3:55 PM, Max Staudt wrote: >>> On 12/20/2017 11:14 AM, Daniel Vetter wrote: btw since I'm probably sounding a bit too grumpy

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Max Staudt
On 12/20/2017 04:11 PM, Daniel Vetter wrote: > So fundamentally I don't think an in-kernel bootsplash is a bad idea. > But most likely you want this on a highly embedded system, which > probably is compiled for your exact hw, with pretty much everything > built in. Also, no fbcon, maybe even no vt

Re: [RFC PATCH v2 03/13] bootsplash: Flush framebuffer after drawing

2017-12-20 Thread Ray Strode
Hi, > Actually, I don't want that :) > > This was a design decision that I've made to keep the code small and simple > to audit. > As it is, the simple bootsplash code will make 99% of people happy. You think only 1% of linux users have more than one monitor or a 4k screen? > I've made this

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Daniel Vetter
On Wed, Dec 20, 2017 at 4:19 PM, Daniel Vetter wrote: > On Wed, Dec 20, 2017 at 4:11 PM, Daniel Vetter wrote: >> On Wed, Dec 20, 2017 at 3:55 PM, Max Staudt wrote: >>> On 12/20/2017 11:14 AM, Daniel Vetter wrote: btw since I'm probably

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Ray Strode
Hi, > The problem that I am stumbling upon is different: > - the system starts with an FB driver > - after the ShowDelay time, Plymouth opens /dev/fb0 > - the system finally loads the DRM driver, which tries to kick the previous > FB driver > - loading the DRM driver fails because Plymouth

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Daniel Vetter
On Wed, Dec 20, 2017 at 4:11 PM, Daniel Vetter wrote: > On Wed, Dec 20, 2017 at 3:55 PM, Max Staudt wrote: >> On 12/20/2017 11:14 AM, Daniel Vetter wrote: >>> btw since I'm probably sounding a bit too grumpy here: I'd very much >>> support this. I think

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Daniel Vetter
On Wed, Dec 20, 2017 at 3:55 PM, Max Staudt wrote: > On 12/20/2017 11:14 AM, Daniel Vetter wrote: >> btw since I'm probably sounding a bit too grumpy here: I'd very much >> support this. I think bootsplash in kernel has a bunch of uses, and it >> shouldn't be hard to get non-suse

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Max Staudt
On 12/20/2017 11:14 AM, Daniel Vetter wrote: > btw since I'm probably sounding a bit too grumpy here: I'd very much > support this. I think bootsplash in kernel has a bunch of uses, and it > shouldn't be hard to get non-suse people to cheer for it (makes merging > easier if it's not just a one-off

[Bug 104275] Stoney Ridge laptop display goes blank after HDMI gets plugged/unplugged in extended mode.

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104275 --- Comment #9 from Harry Wentland --- Created attachment 136321 --> https://bugs.freedesktop.org/attachment.cgi?id=136321=edit DC scatter gather patch You'll need this DC patch. -- You are receiving this mail

Re: [PATCH 4/7] drm/ttm: use an operation ctx for ttm_tt_populate in ttm_bo_driver

2017-12-20 Thread Christian König
Am 20.12.2017 um 11:34 schrieb Roger He: Change-Id: I803ea52d11e5c06add0dffab836c3aecc00b56dd Signed-off-by: Roger He Commit message! And please double check the coding style of ast_ttm_tt_populate. With that fixed that patch is Reviewed-by: Christian König

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Max Staudt
On 12/20/2017 11:06 AM, Neil Armstrong wrote: > My 2cents about this patchset: > You did a good job about all the animation and splash logic, but for me all > this fbcon > stuff is a huge hack, please use a standard and modern display subsystem en > leave fbcon > die alone Thanks for the

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Max Staudt
On 12/20/2017 10:43 AM, Daniel Vetter wrote: > On Tue, Dec 19, 2017 at 07:40:12PM +0100, Max Staudt wrote: >> On 12/19/2017 06:26 PM, Daniel Vetter wrote: >>> So essentially you're telling me that on a current general purpose >>> distro the gfx driver loading is a dumpster fire, and we're fixing

Re: [PATCH v2] drm/i915/vlv: Add cdclk workaround for DSI

2017-12-20 Thread Ville Syrjälä
On Wed, Dec 20, 2017 at 11:50:17AM +0100, Hans de Goede wrote: > At least on the Chuwi Vi8 (non pro/plus) the LCD panel will show an image > shifted aprox. 20% to the left (with wraparound) and sometimes also wrong > colors, showing that the panel controller is starting with sampling the >

Re: [PATCH v2 0/7] Add drm_fb_helper_fbdev_setup/teardown()

2017-12-20 Thread Noralf Trønnes
Den 15.12.2017 18.51, skrev Noralf Trønnes: Hi, This is a new attempt at simplifying fbdev setup/teardown in drivers. Only doc changes in this version based on feedback from Daniel. Noralf. Changes since version 1: - Document a case where drm_fb_helper_fbdev_setup() can't be used:

Re: [PATCH 3/7] drm/ttm: use an operation ctx for ttm_mem_global_alloc_page

2017-12-20 Thread Christian König
Commit message! Am 20.12.2017 um 11:34 schrieb Roger He: Change-Id: I4104a12e09a374b6477a0dd5a8fce26dce27a746 Signed-off-by: Roger He --- drivers/gpu/drm/ttm/ttm_memory.c | 15 --- drivers/gpu/drm/ttm/ttm_page_alloc.c | 6 +-

Re: [PATCH 2/7] drm/ttm: use an operation ctx for ttm_mem_global_alloc

2017-12-20 Thread Christian König
Commit message needed! Something like: Forward the operation context to ttm_mem_global_alloc as well. Am 20.12.2017 um 11:34 schrieb Roger He: Change-Id: I5279b5cd3560c4082b00f822219575a5f9c3808a Signed-off-by: Roger He With the commit message fixed, patch is Reviewed-by:

Re: [PATCH 1/7] drm/ttm: call ttm_bo_swapout directly when ttm shrink

2017-12-20 Thread Christian König
Am 20.12.2017 um 11:34 schrieb Roger He: then remove superfluous functions We need a better commit message. Something like: Remove the extra indirection, cause we have only one implementation anyway. Change-Id: Iea020f0e30a239e0265e7a1500168c7d7f819bd9 Signed-off-by: Roger He

Re: [-next PATCH 2/4] treewide: Use DEVICE_ATTR_RW

2017-12-20 Thread Julia Lawall
On Wed, 20 Dec 2017, Joe Perches wrote: > On Wed, 2017-12-20 at 10:59 +0100, Greg Kroah-Hartman wrote: > > > > Why you didn't send that patch to the sysfs maintainer is a bit odd... > > > > :) > > > > > > So here's an opportunity for you: > > > > > > The sysfs maintainer hasn't added

Re: [RFC PATCH v2 03/13] bootsplash: Flush framebuffer after drawing

2017-12-20 Thread Max Staudt
On 12/19/2017 10:01 PM, Ray Strode wrote: > Hi, > > On Tue, Dec 19, 2017 at 10:41 AM, Max Staudt wrote: >> I'm hooking into the in-kernel terminal emulator, because the bootsplash is a >> functional extension of that. It just happens that fbcon sits on top of FB, >> so I >>

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Max Staudt
On 12/19/2017 09:30 PM, Ray Strode wrote: > Hi, > >> For example, having a userspace splash that starts as early as it can >> (thus on vesafb/efifb on a PC) will cause the KMS driver to fail >> reserving the entirety of video RAM, and thus fail loading. This cannot be >> fixed. > well the fix

[PULL] drm-intel-fixes

2017-12-20 Thread Jani Nikula
Hi Dave, seasons greetings, just a few small fixes. BR, Jani. The following changes since commit 2cf654db8d7eafb973d28eb3cddf043d353e1345: drm/i915/fence: Use rcu to defer freeing of irq_work (2017-12-14 10:58:59 +0200) are available in the git repository at:

Re: [PATCHv4] drm/sun4i: validate modes for HDMI

2017-12-20 Thread Maxime Ripard
Hi Daniel, On Tue, Dec 19, 2017 at 11:05:04AM +0100, Daniel Vetter wrote: > On Mon, Dec 18, 2017 at 08:50:46AM +0100, Maxime Ripard wrote: > > Hi, > > > > On Fri, Dec 15, 2017 at 06:06:32PM +0100, Daniel Vetter wrote: > > > On Fri, Dec 15, 2017 at 05:46:19PM +0100, Hans Verkuil wrote: > > > >

[RFC PATCH v2 0/1] Fix drm driver hang on tegra124-nyan-big

2017-12-20 Thread Guillaume Tucker
This patch was initially sent along with another one to fix a first hang in the nouveau drm driver[1]. I'm now sending it again as a separate patch as it's to fix a second hang which is not strictly related. It is hidden by the first hang though, as this happens later on during the driver

[RFC PATCH v2 1/1] drm/tegra: sor: Fix hang on tegra124 due to NULL clk_out

2017-12-20 Thread Guillaume Tucker
When neither HDMI nor DP is supported such as on the tegra124, the sor->clk_out is not initialised and remains NULL. In this case, the parent clock can't be assigned to it so revert to the previous behaviour of assigning it to the main sor->clk instead. This fixes a kernel hang on tegra124 and

[Bug 104347] AMD RX 580: Hide/Show window sometimes corrupts screen (see screenshot)

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104347 Bug ID: 104347 Summary: AMD RX 580: Hide/Show window sometimes corrupts screen (see screenshot) Product: Mesa Version: 17.3 Hardware: x86-64 (AMD64) OS:

Re: [PATCH] drm/rockchip: analogix_dp: Ensure that the bridge is powered before poking it

2017-12-20 Thread Andrzej Hajda
On 19.12.2017 12:42, Marc Zyngier wrote: > On 19/12/17 07:55, Andrzej Hajda wrote: >> On 18.12.2017 12:28, Marc Zyngier wrote: >>> Stopping the X display manager on a kevin platform results in the >>> following crash: >>> >>> [ 674.833536] Synchronous External Abort: synchronous external abort

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Daniel Stone
Hi Johannes, On 20 December 2017 at 11:08, Johannes Thumshirn wrote: > On Tue, Dec 19, 2017 at 05:16:30PM +0100, Daniel Vetter wrote: >> Ok I've realized that my assumptions about why you need this aren't >> So from reading these patches it sounded like you want an in-kernel

[Bug 196197] [drm:r600_ring_test [radeon]] *ERROR* radeon: ring 0 test failed (scratch(0x8504)=0xCAFEDEAD)

2017-12-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=196197 --- Comment #17 from Andreas Brogle (an...@ok.de) --- What do you suggest ? Who is the maintainer for fixing this bug ? -- You are receiving this mail because: You are watching the assignee of the bug.

Re: [-next PATCH 2/4] treewide: Use DEVICE_ATTR_RW

2017-12-20 Thread Joe Perches
On Wed, 2017-12-20 at 10:59 +0100, Greg Kroah-Hartman wrote: > > > Why you didn't send that patch to the sysfs maintainer is a bit odd... :) > > > > So here's an opportunity for you: > > > > The sysfs maintainer hasn't added include/linux/sysfs.h to > > the list of maintained files... > > > >

[Bug 196197] [drm:r600_ring_test [radeon]] *ERROR* radeon: ring 0 test failed (scratch(0x8504)=0xCAFEDEAD)

2017-12-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=196197 --- Comment #16 from Michel Dänzer (mic...@daenzer.net) --- (In reply to Andreas Brogle from comment #14) > 60e8d3e11645a1b9c4197d9786df3894332c1685 (BAD) > 190c3ee06a0f0660839785b7ad8a830e832d9481 (GOOD) There are only PCI subsystem changes

[Bug 104345] Playing video hangs X-Server with showing scrambled picture, sound still playing.

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104345 --- Comment #3 from Michel Dänzer --- Please attach the output of glxinfo and vdpauinfo. -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel

[PATCH 6/7] drm/ttm: add ttm_bo_evict_swapout_allowable to check bo is allowable to evict or swapout

2017-12-20 Thread Roger He
extract this function since eviction and swapout share same logic Change-Id: I80a475a93fceed8d66d74a1832c815a0756341ac Signed-off-by: Roger He --- drivers/gpu/drm/ttm/ttm_bo.c | 29 +++-- 1 file changed, 19 insertions(+), 10 deletions(-) diff --git

[PATCH 7/7] drm/ttm: enable swapout of per VM BOs during allocation and allows reaping of deleted BOs

2017-12-20 Thread Roger He
Change-Id: I1e87954564f38ad298bf6e4ff88c9f26f291a62d Signed-off-by: Roger He --- drivers/gpu/drm/ttm/ttm_bo.c | 15 +++ drivers/gpu/drm/ttm/ttm_memory.c | 12 include/drm/ttm/ttm_bo_api.h | 3 ++- 3 files changed, 21 insertions(+), 9

[PATCH 5/7] drm/ttm: use an operation ctx for ttm_tt_bind

2017-12-20 Thread Roger He
Change-Id: I42a7df8c50e1ce3b527ee9cb78809f8e58136f07 Signed-off-by: Roger He --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 2 +- drivers/gpu/drm/nouveau/nouveau_bo.c| 2 +- drivers/gpu/drm/radeon/radeon_ttm.c | 2 +- drivers/gpu/drm/ttm/ttm_bo.c| 2 +-

[PATCH 4/7] drm/ttm: use an operation ctx for ttm_tt_populate in ttm_bo_driver

2017-12-20 Thread Roger He
Change-Id: I803ea52d11e5c06add0dffab836c3aecc00b56dd Signed-off-by: Roger He --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 7 --- drivers/gpu/drm/ast/ast_ttm.c| 5 +++-- drivers/gpu/drm/cirrus/cirrus_ttm.c | 5 +++--

[Bug 104345] Playing video hangs X-Server with showing scrambled picture, sound still playing.

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104345 --- Comment #2 from bernhardu --- (In reply to Michel Dänzer from comment #1) > It's more likely a Mesa or maybe LLVM issue. Did this start happening after > you upgraded Mesa and/or LLVM, or maybe another component such

[PATCH 2/7] drm/ttm: use an operation ctx for ttm_mem_global_alloc

2017-12-20 Thread Roger He
Change-Id: I5279b5cd3560c4082b00f822219575a5f9c3808a Signed-off-by: Roger He --- drivers/gpu/drm/ttm/ttm_bo.c| 2 +- drivers/gpu/drm/ttm/ttm_memory.c| 15 +-- drivers/gpu/drm/ttm/ttm_object.c| 13 ++---

[PATCH 3/7] drm/ttm: use an operation ctx for ttm_mem_global_alloc_page

2017-12-20 Thread Roger He
Change-Id: I4104a12e09a374b6477a0dd5a8fce26dce27a746 Signed-off-by: Roger He --- drivers/gpu/drm/ttm/ttm_memory.c | 15 --- drivers/gpu/drm/ttm/ttm_page_alloc.c | 6 +- drivers/gpu/drm/ttm/ttm_page_alloc_dma.c | 8 ++--

[PATCH 1/7] drm/ttm: call ttm_bo_swapout directly when ttm shrink

2017-12-20 Thread Roger He
then remove superfluous functions Change-Id: Iea020f0e30a239e0265e7a1500168c7d7f819bd9 Signed-off-by: Roger He --- drivers/gpu/drm/ttm/ttm_bo.c | 21 +++- drivers/gpu/drm/ttm/ttm_memory.c | 12 ++- include/drm/ttm/ttm_bo_api.h | 1 +

Re: [PATCH 3/3] drm/i915: Disable all planes for load detection, v2.

2017-12-20 Thread Daniel Vetter
On Wed, Dec 20, 2017 at 10:35:45AM +0100, Maarten Lankhorst wrote: > From: Ville Syrjälä > > We don't need any active planes during load detection, so just disable > them all. This saves us from having to come up with a suitable > framebuffer. And we also avoid

Re: [PATCH 2/3] drm/framebuffer: Print task that allocated the fb in debug info.

2017-12-20 Thread Daniel Vetter
On Wed, Dec 20, 2017 at 10:35:44AM +0100, Maarten Lankhorst wrote: > This is is very useful to finding sources of leaked framebufers. > The fbcon fb is annotated with [fbcon], to give it a better name > than kworker. > > Signed-off-by: Maarten Lankhorst I

Re: [PATCH 1/3] drm/plane: Make framebuffer refcounting the responsibility of setplane_internal callers

2017-12-20 Thread Daniel Vetter
On Wed, Dec 20, 2017 at 10:35:43AM +0100, Maarten Lankhorst wrote: > lock_all_ctx in setplane_internal may return -EINTR, and > __setplane_internal could return -EDEADLK. Making more > special cases for fb would make the code even harder to > read, so the easiest solution is not taking over the fb

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Daniel Vetter
On Wed, Dec 20, 2017 at 11:06:34AM +0100, Neil Armstrong wrote: > On 20/12/2017 10:43, Daniel Vetter wrote: > > On Tue, Dec 19, 2017 at 07:40:12PM +0100, Max Staudt wrote: > >> On 12/19/2017 06:26 PM, Daniel Vetter wrote: > >>> On Tue, Dec 19, 2017 at 6:04 PM, Max Staudt wrote: >

[Bug 104345] Playing video hangs X-Server with showing scrambled picture, sound still playing.

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104345 Michel Dänzer changed: What|Removed |Added QA Contact|

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Neil Armstrong
On 20/12/2017 10:43, Daniel Vetter wrote: > On Tue, Dec 19, 2017 at 07:40:12PM +0100, Max Staudt wrote: >> On 12/19/2017 06:26 PM, Daniel Vetter wrote: >>> On Tue, Dec 19, 2017 at 6:04 PM, Max Staudt wrote: Well, those could enable fbcon if they want the bootsplash.

[Bug 104345] Playing video hangs X-Server with showing scrambled picture, sound still playing.

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104345 Bug ID: 104345 Summary: Playing video hangs X-Server with showing scrambled picture, sound still playing. Product: DRI Version: unspecified Hardware: x86-64 (AMD64)

[Bug 104331] [r600g] Ogre demo "TutorialUAV01" crash at r600_decompress_color_images

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104331 --- Comment #5 from Michel Dänzer --- (In reply to MWATTT from comment #2) > However, the demo does not render properly(I don't know if I should make > another report for that). Please do. -- You are receiving this mail

Re: [RFC PATCH 01/60] hyper_dmabuf: initial working version of hyper_dmabuf drv

2017-12-20 Thread Daniel Vetter
On Tue, Dec 19, 2017 at 03:27:31PM -0800, Dongwon Kim wrote: > I forgot to include this brief information about this patch series. > > This patch series contains the implementation of a new device driver, > hyper_dmabuf, which provides a method for DMA-BUF sharing across > different OSes running

Re: [-next PATCH 2/4] treewide: Use DEVICE_ATTR_RW

2017-12-20 Thread Greg Kroah-Hartman
On Wed, Dec 20, 2017 at 01:54:41AM -0800, Joe Perches wrote: > On Wed, 2017-12-20 at 10:32 +0100, Greg Kroah-Hartman wrote: > > On Wed, Dec 20, 2017 at 01:24:44AM -0800, Joe Perches wrote: > > > On Wed, 2017-12-20 at 10:34 +0200, Jarkko Nikula wrote: > > > > On Tue, Dec 19, 2017 at 10:15:07AM

[Bug 104275] Stoney Ridge laptop display goes blank after HDMI gets plugged/unplugged in extended mode.

2017-12-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104275 --- Comment #8 from Michel Dänzer --- (In reply to Alex_Zhang1 from comment #7) Without the patch referenced in comment 3, a buffer must be pinned to VRAM before it can be scanned out (displayed). The problem described in

Re: [-next PATCH 2/4] treewide: Use DEVICE_ATTR_RW

2017-12-20 Thread Joe Perches
On Wed, 2017-12-20 at 10:32 +0100, Greg Kroah-Hartman wrote: > On Wed, Dec 20, 2017 at 01:24:44AM -0800, Joe Perches wrote: > > On Wed, 2017-12-20 at 10:34 +0200, Jarkko Nikula wrote: > > > On Tue, Dec 19, 2017 at 10:15:07AM -0800, Joe Perches wrote: > > > > Convert DEVICE_ATTR uses to

Re: [RFC PATCH v2 03/13] bootsplash: Flush framebuffer after drawing

2017-12-20 Thread Daniel Vetter
On Tue, Dec 19, 2017 at 05:23:52PM +0100, Max Staudt wrote: > On 12/19/2017 05:02 PM, Daniel Vetter wrote: > > On Tue, Dec 19, 2017 at 4:41 PM, Max Staudt wrote: > >> On 12/19/2017 02:57 PM, Daniel Vetter wrote: > >>> The problem is that defio is totally not how a real driver

Re: [RFC PATCH v2 00/13] Kernel based bootsplash

2017-12-20 Thread Daniel Vetter
On Tue, Dec 19, 2017 at 07:40:12PM +0100, Max Staudt wrote: > On 12/19/2017 06:26 PM, Daniel Vetter wrote: > > On Tue, Dec 19, 2017 at 6:04 PM, Max Staudt wrote: > >> Well, those could enable fbcon if they want the bootsplash. Shouldn't make > >> a difference anyway if they're

[PATCH 1/3] drm/plane: Make framebuffer refcounting the responsibility of setplane_internal callers

2017-12-20 Thread Maarten Lankhorst
lock_all_ctx in setplane_internal may return -EINTR, and __setplane_internal could return -EDEADLK. Making more special cases for fb would make the code even harder to read, so the easiest solution is not taking over the fb refcount, and making callers responsible for dropping the ref. Bugzilla:

[PATCH 3/3] drm/i915: Disable all planes for load detection, v2.

2017-12-20 Thread Maarten Lankhorst
From: Ville Syrjälä We don't need any active planes during load detection, so just disable them all. This saves us from having to come up with a suitable framebuffer. And we also avoid leaving sprite/cursor planes on and potentially presenting them at a peculiar

[PATCH 2/3] drm/framebuffer: Print task that allocated the fb in debug info.

2017-12-20 Thread Maarten Lankhorst
This is is very useful to finding sources of leaked framebufers. The fbcon fb is annotated with [fbcon], to give it a better name than kworker. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_fb_helper.c | 1 + drivers/gpu/drm/drm_framebuffer.c | 2

[PATCH 0/3] drm: Plug framebuffer leaks.

2017-12-20 Thread Maarten Lankhorst
There's a small leak excerbated by interruptible waiting in setplane_internal, which was triggered by kms_atomic. Another leak is in load detection, and easily plugged by just removing the fb handling. Maarten Lankhorst (2): drm/plane: Make framebuffer refcounting the responsibility of

[Bug 197925] [amdgpu_dc][carrizo] multiple monitor handling errors

2017-12-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=197925 --- Comment #11 from Charles (ylang-yl...@libertysurf.fr) --- Hi team, Just to say... same error : [drm:hwss_wait_for_blank_complete [amdgpu]] *ERROR* DC: failed to blank crtc! with 4.15-rc4 (on the same noteboook) -- You are receiving this

Re: [-next PATCH 2/4] treewide: Use DEVICE_ATTR_RW

2017-12-20 Thread Greg Kroah-Hartman
On Wed, Dec 20, 2017 at 01:24:44AM -0800, Joe Perches wrote: > On Wed, 2017-12-20 at 10:34 +0200, Jarkko Nikula wrote: > > On Tue, Dec 19, 2017 at 10:15:07AM -0800, Joe Perches wrote: > > > Convert DEVICE_ATTR uses to DEVICE_ATTR_RW where possible. > [] > > > diff --git a/sound/soc/omap/mcbsp.c

Re: [-next PATCH 2/4] treewide: Use DEVICE_ATTR_RW

2017-12-20 Thread Joe Perches
On Wed, 2017-12-20 at 10:34 +0200, Jarkko Nikula wrote: > On Tue, Dec 19, 2017 at 10:15:07AM -0800, Joe Perches wrote: > > Convert DEVICE_ATTR uses to DEVICE_ATTR_RW where possible. [] > > diff --git a/sound/soc/omap/mcbsp.c b/sound/soc/omap/mcbsp.c [] > > @@ -854,7 +854,7 @@ static ssize_t

Re: [-next PATCH 0/4] sysfs and DEVICE_ATTR_

2017-12-20 Thread Corey Minyard
On 12/19/2017 12:15 PM, Joe Perches wrote: drivers/char/ipmi/ipmi_msghandler.c| 17 +++--- For ipmi: Acked-by: Corey Minyard ___ dri-devel mailing list dri-devel@lists.freedesktop.org

Re: [Xen-devel] [RFC PATCH 01/60] hyper_dmabuf: initial working version of hyper_dmabuf drv

2017-12-20 Thread Juergen Gross
On 20/12/17 00:27, Dongwon Kim wrote: > I forgot to include this brief information about this patch series. > > This patch series contains the implementation of a new device driver, > hyper_dmabuf, which provides a method for DMA-BUF sharing across > different OSes running on the same virtual OS

Re: [trivial PATCH] treewide: Align function definition open/close braces

2017-12-20 Thread Mauro Carvalho Chehab
Em Sun, 17 Dec 2017 16:28:44 -0800 Joe Perches escreveu: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work >

Re: [Bug 198123] Console is the wrong color at boot with radeon 6670

2017-12-20 Thread Peter Rosin
On 2017-12-18 12:37, Michel Dänzer wrote: > > Following up by e-mail, since I can't find Peter Rosin in the kernel > bugzilla. > > > On 2017-12-16 02:41 AM, bugzilla-dae...@bugzilla.kernel.org wrote: >> https://bugzilla.kernel.org/show_bug.cgi?id=198123 >> >> --- Comment #8 from Deposite Pirate

Re: [PATCH v4 1/2] DTS: GTA04: improve panel compatibility string

2017-12-20 Thread Tony Lindgren
* Tomi Valkeinen [171219 10:51]: > On 11/12/17 17:22, Tony Lindgren wrote: > > * H. Nikolaus Schaller [171201 07:44]: > > > Official vendor string is now "tpo" and not "toppoly". > > > > > > Requires patch "omapdrm: panel: fix compatible vendor string

Re: [PATCH] kthread: finer-grained lockdep/cross-release completion

2017-12-20 Thread Byungchul Park
On 12/19/2017 6:59 PM, Daniel Vetter wrote: On Mon, Dec 18, 2017 at 09:42:13AM -0800, Linus Torvalds wrote: On Sun, Dec 17, 2017 at 11:11 PM, Daniel Vetter wrote: This didn't seem to have made it into -rc4. Anything needed to get it going? Do you actually see the problem

Re: [PATCH] drm/rockchip: analogix_dp: Ensure that the bridge is powered before poking it

2017-12-20 Thread Marc Zyngier
On 19/12/17 07:55, Andrzej Hajda wrote: > On 18.12.2017 12:28, Marc Zyngier wrote: >> Stopping the X display manager on a kevin platform results in the >> following crash: >> >> [ 674.833536] Synchronous External Abort: synchronous external abort >> (0x9610) at 0x0c970640 >> [

  1   2   >