[Bug 205497] Attempt to read amd gpu id causes a freeze

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205497 --- Comment #14 from Trek (tre...@inbox.ru) --- (In reply to Alex Deucher from comment #11) > If you access the BAR directly you will likely have problems in certain > power saving modes. thank you, I'll add a warning message when accessing BAR

[Bug 205497] Attempt to read amd gpu id causes a freeze

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205497 --- Comment #13 from Luya Tshimbalanga (l...@fedoraproject.org) --- (In reply to Alex Deucher from comment #11) > (In reply to Trek from comment #7) > > by default, radeontop calls amdgpu_read_mm_registers, amdgpu_query_info and > >

[Bug 201539] AMDGPU R9 390 automatic fan speed control in Linux 4.19/4.20/5.0

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=201539 --- Comment #28 from MasterCATZ (masterc...@hotmail.com) --- hmm maybe not it lets me briefly access manual Found the following PWM controls: hwmon1/pwm1 current value: 68 hwmon1/pwm1 is currently setup for automatic speed

[Bug 201539] AMDGPU R9 390 automatic fan speed control in Linux 4.19/4.20/5.0

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=201539 --- Comment #27 from MasterCATZ (masterc...@hotmail.com) --- pwmconfig seems to be the only thing that allows me to get manual mode back on I wounder if this is the actual program giving grieve -- You are receiving this mail because: You are

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

2019-11-15 Thread Stephen Rothwell
Hi all, Commit b4011644b03c ("drm/vmwgfx: remove set but not used variable 'srf'") is missing a Signed-off-by from its committer. -- Cheers, Stephen Rothwell pgpCwR7wqyTYm.pgp Description: OpenPGP digital signature ___ dri-devel mailing list

Re: [PATCH 3/4] drm/ttm: rework BO delayed delete.

2019-11-15 Thread Felix Kuehling
On 2019-11-11 9:58 a.m., Christian König wrote: This patch reworks the whole delayed deletion of BOs which aren't idle. Instead of having two counters for the BO structure we resurrect the BO when we find that a deleted BO is not idle yet. This has many advantages, especially that we don't

Re: [PATCH 2/4] drm/ttm: cleanup ttm_buffer_object_transfer

2019-11-15 Thread Felix Kuehling
The subject doesn't match the change. This changes ttm_bo_cleanup_refs, not ttm_buffer_object_transfer. On 2019-11-11 9:58 a.m., Christian König wrote: The function is always called with deleted BOs. While at it cleanup the indentation as well. Signed-off-by: Christian König ---

Re: [PATCH] drm/amdkfd: remove set but not used variable 'top_dev'

2019-11-15 Thread Felix Kuehling
On 2019-11-13 10:20 p.m., zhengbin wrote: Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/amd/amdkfd/kfd_iommu.c: In function kfd_iommu_device_init: drivers/gpu/drm/amd/amdkfd/kfd_iommu.c:65:30: warning: variable top_dev set but not used [-Wunused-but-set-variable] Reported-by:

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

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=111481 --- Comment #232 from viste.sylv...@gmail.com --- (In reply to Sander Lienaerts from comment #231) > Been following this thread for a while now. Can't believe this has been > known for 3 months, without a fix released. > > Just a moment ago a

[Bug 112266] [Navi] Pathfinder: Kingmaker is causing a GPU hang: flip_done timed out error

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112266 --- Comment #8 from rLy --- (In reply to Alex Deucher from comment #6) > Created attachment 145971 [details] [review] > possible fix > > Does this patch help? I tried it as well on 5.4-rc7 and fixed every game I mentioned(CS:GO, SOTTR,

[PATCH 3/3] drm/nouveau/kms/nv50-: Limit MST BPC to 8

2019-11-15 Thread Lyude Paul
Noticed this while working on some unrelated CRC stuff. Currently, userspace has very little support for BPCs higher than 8. While this doesn't matter for most things, on MST topologies we need to be careful about ensuring that we do our best to make any given display configuration fit within the

[PATCH 2/3] drm/nouveau/kms/nv50-: Store the bpc we're using in nv50_head_atom

2019-11-15 Thread Lyude Paul
In order to be able to use bpc values that are different from what the connector reports, we want to be able to store the bpc value we decide on using for an atomic state in nv50_head_atom and refer to that instead of simply using the value that the connector reports throughout the whole atomic

[PATCH 1/3] drm/nouveau/kms/nv50-: Call outp_atomic_check_view() before handling PBN

2019-11-15 Thread Lyude Paul
Since nv50_outp_atomic_check_view() can set crtc_state->mode_changed, we probably should be calling it before handling any PBN changes. Just a precaution. Signed-off-by: Lyude Paul Fixes: 232c9eec417a ("drm/nouveau: Use atomic VCPI helpers for MST") Cc: Ben Skeggs Cc: Daniel Vetter Cc: David

[PATCH 0/3] MST BPC fixes for nouveau

2019-11-15 Thread Lyude Paul
Realized when I moved nouveau over to using the atomic DP MST VCPI helpers that I forgot to ensure that we clamp the BPC to 8 to make us less likely to run out of bandwidth on a topology when enabling multiple displays that support >8 BPC - something we want to do until we have support for

[PATCH] MAINTAINERS: Remove myself from drm-misc entry

2019-11-15 Thread Sean Paul
Cc: Dave Airlie Cc: Daniel Vetter Cc: Maarten Lankhorst Cc: Maxime Ripard Signed-off-by: Sean Paul --- I need to step away from -misc to prioritize other work that I have on my plate. It's been great fun (most of the time) and very rewarding doing this job. I will really miss it! I plan on

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

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=111481 --- Comment #231 from Sander Lienaerts --- Been following this thread for a while now. Can't believe this has been known for 3 months, without a fix released. Just a moment ago a random freeze occurred running Firefox and other applications,

[Bug 205169] AMDGPU driver with Navi card hangs Xorg in fullscreen only.

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205169 --- Comment #23 from ArneJ (kernelbug5...@arnej.de) --- I just let Borderlands 2 run for about one hour in the menu which causes a hang without this patch in at most 3 minutes. Consider Borderlands 2 also fixed with this :) -- You are

Re: [PATCH 0/3] drm/gma500: remove some set but not used variables

2019-11-15 Thread Daniel Vetter
On Fri, Nov 15, 2019 at 10:27:04PM +0800, zhengbin wrote: > zhengbin (3): > drm/gma500: remove set but not used variable 'htotal' > drm/gma500: remove set but not used variable 'error' > drm/gma500: remove set but not used variable 'is_hdmi','is_crt' All three applied, thanks for the

[PATCH 6/7] drm/atomic: Fix the early return in drm_atomic_set_mode_for_crtc()

2019-11-15 Thread Ville Syrjala
From: Ville Syrjälä The early return in drm_atomic_set_mode_for_crtc() isn't quite right. It would mistakenly return and fail to update crtc_state->enable if someone actually tried to set a zeroed mode on a currently disabled crtc. That should never actually happen in response to any userspace

[PATCH 7/7] drm/atomic: Reduce setplane locking

2019-11-15 Thread Ville Syrjala
From: Ville Syrjälä Currently setplane grabs all modeset locks, which seems a bit excessive. Let's reduce that to just the locks we really need on atomic drivers. For non-atomic drivers let's stick to the current scheme for now. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_plane.c |

[PATCH 5/7] drm/selftests: Add some selftests for drm_atomic_set_mode_for_crtc()

2019-11-15 Thread Ville Syrjala
From: Ville Syrjälä Test the basics of drm_atomic_set_mode_for_crtc(), and in particular verify that the function doesn't take the shortcut incorrectly. Cc: Daniel Vetter Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/selftests/Makefile| 3 +-

[PATCH 2/7] drm: Allocate the page flip event earlier

2019-11-15 Thread Ville Syrjala
From: Ville Syrjälä Can't see why we need to delay the page flip event allocation until the last moment. Move it earlier to simplify error handling. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_plane.c | 45 +++-- 1 file changed, 23 insertions(+), 22

[PATCH 3/7] drm: Extract page_flip_{internal,atomic}()

2019-11-15 Thread Ville Syrjala
From: Ville Syrjälä Yank out the code for the plane->fb/old_fb/crtc handling from the page flip path into page_flip_internal(), and provide a simpler variant for atomic drivers. We'll also move the fb vs. src viewport checks into the new functions as they are slightly different between the two

[PATCH 4/7] drm: Simplify the setplane old_fb handling further

2019-11-15 Thread Ville Syrjala
From: Ville Syrjälä Instead of doing the things in a convoluted way with the failure and success paths mixed up let's just clear old_fb when we encounter an error and bail out immediately. We already did this for the pageflip path. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_plane.c

[PATCH 0/7] drm: Random pile of core stuff

2019-11-15 Thread Ville Syrjala
From: Ville Syrjälä I found this random pile of stuff lying around. Dusted it off and tossed in the new selftests. Ville Syrjälä (7): drm: Move page_flip fb lookup earlier drm: Allocate the page flip event earlier drm: Extract page_flip_{internal,atomic}() drm: Simplify the setplane

[PATCH 1/7] drm: Move page_flip fb lookup earlier

2019-11-15 Thread Ville Syrjala
From: Ville Syrjälä No reason that I can see to delay the fb lookup this late. Moving it earlier allows us to keep it outside of the lock retry loop. This makes error handling and whatnot simpler. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_plane.c | 29 +++--

Re: [Intel-gfx] [PATCH] drm/amdgpu/dm: Do not throw an error for a display with no audio

2019-11-15 Thread Ville Syrjälä
On Fri, Nov 15, 2019 at 10:18:26AM +0100, Daniel Vetter wrote: > On Fri, Nov 15, 2019 at 10:04 AM Jean Delvare wrote: > > > > Hi Chris, > > > > On Thu, 14 Nov 2019 20:44:13 +, Chris Wilson wrote: > > > An old display with no audio may not have an EDID with a CEA block, or > > > it may simply

Re: [PATCH 2/2] drm/edid: Add alternate clock for SMPTE 4K

2019-11-15 Thread Ville Syrjälä
On Tue, Nov 12, 2019 at 10:36:54AM +0100, Neil Armstrong wrote: > On 12/11/2019 08:53, Wayne Lin wrote: > > [Why] > > In hdmi_mode_alternate_clock(), it adds an exception for VIC 4 > > mode (4096x2160@24) due to there is no alternate clock defined for > > that mode in HDMI1.4b. But HDMI2.0 adds

Re: [PATCH 2/2] drm/edid: Add alternate clock for SMPTE 4K

2019-11-15 Thread Ville Syrjälä
On Tue, Nov 12, 2019 at 03:53:07PM +0800, Wayne Lin wrote: > [Why] > In hdmi_mode_alternate_clock(), it adds an exception for VIC 4 > mode (4096x2160@24) due to there is no alternate clock defined for > that mode in HDMI1.4b. But HDMI2.0 adds 23.98Hz for that mode. > > [How] > Remove the

Re: [PATCH 1/2] drm/edid: Add aspect ratios to HDMI 4K modes

2019-11-15 Thread Ville Syrjälä
On Tue, Nov 12, 2019 at 03:53:06PM +0800, Wayne Lin wrote: > [Why] > HDMI 2.0 adds aspect ratio attribute to distinguish different > 4k modes. According to Appendix E of HDMI 2.0 spec, source should > use VSIF to indicate video mode only when the mode is one defined > in HDMI 1.4b 4K modes.

[Bug 205169] AMDGPU driver with Navi card hangs Xorg in fullscreen only.

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205169 --- Comment #22 from Shmerl (shtetl...@gmail.com) --- It fixes Pathfinder: Kingamer too. But first let the patch be upstreamed, then it's OK to close the bug :) -- You are receiving this mail because: You are watching the assignee of the bug.

[Bug 205169] AMDGPU driver with Navi card hangs Xorg in fullscreen only.

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205169 --- Comment #21 from Dmitri Seletski (drj...@gmail.com) --- (In reply to Alex Deucher from comment #20) > Created attachment 285935 [details] > possible fix > > Does this patch help? It did not just solve one problem, but two! First of all it

[Bug 112266] [Navi] Pathfinder: Kingmaker is causing a GPU hang: flip_done timed out error

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112266 --- Comment #7 from Shmerl --- (In reply to Alex Deucher from comment #6) > Created attachment 145971 [details] [review] > possible fix > > Does this patch help? Just applied that patch on on top of latest 5.4-rc7+ and tested it. It prevents

Re: [PATCH v5 09/24] vfio, mm: fix get_user_pages_remote() and FOLL_LONGTERM

2019-11-15 Thread Ira Weiny
On Thu, Nov 14, 2019 at 09:53:25PM -0800, John Hubbard wrote: > As it says in the updated comment in gup.c: current FOLL_LONGTERM > behavior is incompatible with FAULT_FLAG_ALLOW_RETRY because of the > FS DAX check requirement on vmas. > > However, the corresponding restriction in

Re: [PATCH 1/2] drm/nouveau: remove set but not used variable 'pclks','width'

2019-11-15 Thread Lyude Paul
Reviewed-by: Lyude Paul On Fri, 2019-11-15 at 21:42 +0800, zhengbin wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/gpu/drm/nouveau/dispnv04/arb.c: In function nv04_calc_arb: > drivers/gpu/drm/nouveau/dispnv04/arb.c:59:21: warning: variable pclks set > but not used

Re: [PATCH 2/2] drm/nouveau: remove set but not used variable 'mem'

2019-11-15 Thread Lyude Paul
Reviewed-by: Lyude Paul On Fri, 2019-11-15 at 21:42 +0800, zhengbin wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/gpu/drm/nouveau/nouveau_ttm.c: In function nouveau_vram_manager_new: > drivers/gpu/drm/nouveau/nouveau_ttm.c:66:22: warning: variable mem set but > not used

Re: [git pull] drm fixes for 5.4-rc8

2019-11-15 Thread pr-tracker-bot
The pull request you sent on Fri, 15 Nov 2019 11:18:16 +1000: > git://anongit.freedesktop.org/drm/drm tags/drm-fixes-2019-11-15 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/37b49f31e800b563ed7a601816ea4b6fc3c5d165 Thank you! -- Deet-doot-dot, I am a bot.

[Bug 205497] Attempt to read amd gpu id causes a freeze

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205497 --- Comment #12 from V.I.S. (itemc...@mail.ru) --- I need approx 3-5 days for testing, because this bug is not persistent. -- You are receiving this mail because: You are watching the assignee of the bug.

[pull] radeon and amdgpu drm-next-5.5

2019-11-15 Thread Alex Deucher
Hi Dave, Daniel, Misc fixes for 5.5. The following changes since commit 53dbc27ad5a93932ff1892a8e4ef266827d74a0f: drm/amdgpu/powerplay: fix AVFS handling with custom powerplay table (2019-11-08 12:30:24 -0500) are available in the Git repository at:

[Bug 205521] 5.3.11 update broke AMDGPU Raven Ridge

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205521 Luya Tshimbalanga (l...@fedoraproject.org) changed: What|Removed |Added Status|NEW |RESOLVED

[Bug 112266] [Navi] Pathfinder: Kingmaker is causing a GPU hang: flip_done timed out error

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112266 --- Comment #6 from Alex Deucher --- Created attachment 145971 --> https://bugs.freedesktop.org/attachment.cgi?id=145971=edit possible fix Does this patch help? -- You are receiving this mail because: You are the assignee for the

[Bug 205169] AMDGPU driver with Navi card hangs Xorg in fullscreen only.

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205169 Alex Deucher (alexdeuc...@gmail.com) changed: What|Removed |Added CC|

[Bug 112290] sanoop

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112290 Andre Klapper changed: What|Removed |Added Group||spam Version|XOrg git

[Bug 205497] Attempt to read amd gpu id causes a freeze

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205497 --- Comment #11 from Alex Deucher (alexdeuc...@gmail.com) --- (In reply to Trek from comment #7) > by default, radeontop calls amdgpu_read_mm_registers, amdgpu_query_info and > amdgpu_query_sensor_info, but it can be forced by the command line to

Re: drm core/helpers and MIT license

2019-11-15 Thread Alex Deucher
On Tue, Nov 12, 2019 at 10:03 AM Daniel Vetter wrote: > > Hi all, > > Dave and me chatted about this last week on irc. Essentially we have: > > $ git grep SPDX.*GPL -- ':(glob)drivers/gpu/drm/*c' > drivers/gpu/drm/drm_client.c:// SPDX-License-Identifier: GPL-2.0 >

Re: [PATCH v2] drm/gem: Fix mmap fake offset handling for drm_gem_object_funcs.mmap

2019-11-15 Thread Daniel Vetter
On Fri, Nov 15, 2019 at 11:56 AM Gerd Hoffmann wrote: > > On Fri, Nov 15, 2019 at 11:18:28AM +0100, Daniel Vetter wrote: > > On Fri, Nov 15, 2019 at 10:37 AM Gerd Hoffmann wrote: > > > > > > > You need memory pressure, to force ttm to unmap the bo, not userspace. > > > > So > > > > roughly > >

Re: [Intel-gfx] [PATCH 1/8] drm/fb: More paranoia in addfb checks

2019-11-15 Thread Daniel Vetter
On Fri, Nov 15, 2019 at 1:44 PM Ville Syrjälä wrote: > > On Fri, Nov 15, 2019 at 10:21:13AM +0100, Daniel Vetter wrote: > > - Our limit is uint32_t, make that explicit. > > > > - Untangle the one overflow check, I think (but not sure) that with > > all three together you could overflow the

[Bug 112288] Blank display on Ubuntu 19.10 on Ryzen 3 2200G and Ryzen 5 2400G APUs

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112288 --- Comment #12 from Alex Deucher --- Does disabling the iommu help? Append iommu=off or iommu=pt to the kernel command line. -- You are receiving this mail because: You are the assignee for the

[Bug 112297] AMDGPU.gpu_recovery does recover desktop to an unusable state

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112297 --- Comment #1 from Daniel Suarez --- Don't expect this to work anytime soon unfortunately. AMD really slacked off with the drivers for this and similar bug reports have been open for months. As for your work, save early, save often while

[Bug 112297] AMDGPU.gpu_recovery does recover desktop to an unusable state

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112297 Daniel Suarez changed: What|Removed |Added Priority|not set |highest -- You are receiving this

Re: [PATCH 1/1] drm/radeon drm/amdgpu: fix bad DMA from INTERRUPT_CNTL2

2019-11-15 Thread Alex Deucher
On Thu, Nov 14, 2019 at 9:05 PM Sam Bobroff wrote: > > The INTERRUPT_CNTL2 register expects a valid DMA address, but is > currently set with a GPU MC address. This can cause problems on > systems that detect the resulting DMA read from an invalid address > (found on a Power8 guest). > > Instead,

Re: drm core/helpers and MIT license

2019-11-15 Thread Noralf Trønnes
Den 15.11.2019 13.34, skrev Ville Syrjälä: > On Thu, Nov 14, 2019 at 08:01:32PM +, co...@sdf.org wrote: >> Hi Daniel, >> >> I don't think we can make any complaints about GPL being more widely >> used in the DRM code. It's nice to have the code at all, the MIT license >> is a bonus. Thanks

Re: [PATCH 5/8] drm/tilcdc: Drop drm_gem_fb_create wrapper

2019-11-15 Thread Jyri Sarha
On 15/11/2019 11:21, Daniel Vetter wrote: > Doesn't do anything. > > Signed-off-by: Daniel Vetter > Cc: Jyri Sarha > Cc: Tomi Valkeinen Acked-by: Jyri Sarha > --- > drivers/gpu/drm/tilcdc/tilcdc_drv.c | 8 +--- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git

Re: [Intel-gfx] [PATCH 1/8] drm/fb: More paranoia in addfb checks

2019-11-15 Thread Ville Syrjälä
On Fri, Nov 15, 2019 at 10:21:13AM +0100, Daniel Vetter wrote: > - Our limit is uint32_t, make that explicit. > > - Untangle the one overflow check, I think (but not sure) that with > all three together you could overflow the uint64_t and it'd look > cool again. It can't overflow. All theree

Re: drm core/helpers and MIT license

2019-11-15 Thread Ville Syrjälä
On Thu, Nov 14, 2019 at 08:01:32PM +, co...@sdf.org wrote: > Hi Daniel, > > I don't think we can make any complaints about GPL being more widely > used in the DRM code. It's nice to have the code at all, the MIT license > is a bonus. Thanks for writing it and bearing with us. > > Would

[Bug 112297] AMDGPU.gpu_recovery does recover desktop to an unusable state

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112297 David Biró changed: What|Removed |Added Summary|AMDGPU recovery does|AMDGPU.gpu_recovery does

[Bug 112297] AMDGPU recovery does recover desktop to an unstable state

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112297 Bug ID: 112297 Summary: AMDGPU recovery does recover desktop to an unstable state Product: DRI Version: unspecified Hardware: x86-64 (AMD64) OS: Linux

Re: [PATCH 0/2] add new DRM_DEV_WARN macro

2019-11-15 Thread Jani Nikula
On Thu, 14 Nov 2019, Wambui Karuga wrote: > This adds a new DRM_DEV_WARN helper macro for warnings log output that include > device pointers. It also includes the use of the DRM_DEV_WARN macro in > drm/rockchip to replace dev_warn. I'm trying to solicit new struct drm_device based logging

Re: drm-next + i915 CVE yolo merge

2019-11-15 Thread Jani Nikula
On Fri, 15 Nov 2019, Joonas Lahtinen wrote: > Quoting Dave Airlie (2019-11-14 03:33:24) >> The landing of the i915 CVE fixes into Linus tree has created a bit of >> a mess in linux-next and downstream in drm-next trees. >> >> I talked to Daniel and he had talked to Joonas a bit, and I decided to

Re: drm-next + i915 CVE yolo merge

2019-11-15 Thread Joonas Lahtinen
Quoting Dave Airlie (2019-11-14 03:33:24) > The landing of the i915 CVE fixes into Linus tree has created a bit of > a mess in linux-next and downstream in drm-next trees. > > I talked to Daniel and he had talked to Joonas a bit, and I decided to > go with what Daniel describes as the YOLO merge,

Re: [PATCH v2] drm/gem: Fix mmap fake offset handling for drm_gem_object_funcs.mmap

2019-11-15 Thread Gerd Hoffmann
On Fri, Nov 15, 2019 at 11:18:28AM +0100, Daniel Vetter wrote: > On Fri, Nov 15, 2019 at 10:37 AM Gerd Hoffmann wrote: > > > > > You need memory pressure, to force ttm to unmap the bo, not userspace. So > > > roughly > > > 1. create bo > > > 2. mmap it through drm fd, write some stuff > > > 3.

Re: [PATCH 1/8] drm/fb: More paranoia in addfb checks

2019-11-15 Thread Pekka Paalanen
On Fri, 15 Nov 2019 10:21:13 +0100 Daniel Vetter wrote: > - Our limit is uint32_t, make that explicit. > > - Untangle the one overflow check, I think (but not sure) that with > all three together you could overflow the uint64_t and it'd look > cool again. Hence two steps. Also go with the

[PATCH v4 4/4] drm/bridge: add it6505 driver

2019-11-15 Thread allen
From: Allen Chen This adds support for the iTE IT6505. This device can convert DPI signal to DP output. Signed-off-by: Jitao Shi Signed-off-by: Yilun Lin Signed-off-by: Allen Chen Signed-off-by: Pi-Hsun Shih --- drivers/gpu/drm/bridge/Kconfig |7 + drivers/gpu/drm/bridge/Makefile

Re: [PATCH v10 4/4] arm64: dts: zynqmp: zcu106-revA: Wire up the DisplayPort subsystem

2019-11-15 Thread Michal Simek
On 08. 11. 19 18:59, Laurent Pinchart wrote: > Enable the dpsub device and wire it up to the PS-GTR PHY lanes routed to > the DisplayPort connector. > > Signed-off-by: Laurent Pinchart > --- > .../arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts | 16 > 1 file changed, 16

[PATCH v4 2/4] Revert "drm/tegra: Move drm_dp_link helpers to Tegra DRM"

2019-11-15 Thread allen
This reverts commit 9a42c7c647a9ad0f7ebb147a52eda3dcb7c84292. Signed-off-by: Allen Chen --- drivers/gpu/drm/drm_dp_helper.c | 128 ++ drivers/gpu/drm/tegra/Makefile | 1 - drivers/gpu/drm/tegra/dp.c | 876 drivers/gpu/drm/tegra/dp.h |

[PATCH v4 0/4] IT6505 cover letter

2019-11-15 Thread allen
The IT6505 is a high-performance DisplayPort 1.1a transmitter, fully compliant with DisplayPort 1.1a, HDCP 1.3 specifications. The IT6505 supports color depth of up to 36 bits (12 bits/color) and ensures robust transmission of high-quality uncompressed video content, along with uncompressed and

[PATCH v4 3/4] dt-bindings: Add binding for IT6505.

2019-11-15 Thread allen
From: Allen Chen Add a DT binding documentation for IT6505. Signed-off-by: Allen Chen Signed-off-by: Pi-Hsun Shih --- .../bindings/display/bridge/ite,it6505.txt | 28 ++ 1 file changed, 28 insertions(+) create mode 100644

Re: [PATCH] drm/amdgpu/dm: Do not throw an error for a display with no audio

2019-11-15 Thread Jean Delvare
Hi Chris, On Thu, 14 Nov 2019 20:44:13 +, Chris Wilson wrote: > An old display with no audio may not have an EDID with a CEA block, or > it may simply be too old to support audio. This is not a driver error, > so don't flag it as such. > > Bugzilla:

Re: [PATCH v2] drm/gem: Fix mmap fake offset handling for drm_gem_object_funcs.mmap

2019-11-15 Thread Daniel Vetter
On Fri, Nov 15, 2019 at 10:37 AM Gerd Hoffmann wrote: > > > You need memory pressure, to force ttm to unmap the bo, not userspace. So > > roughly > > 1. create bo > > 2. mmap it through drm fd, write some stuff > > 3. export to dma-buf, mmap it, verify stuff is there > > 4. create a pile more bo,

[PATCH][next] drm/amdgpu/powerplay: fix dereference before null check of pointer hwmgr

2019-11-15 Thread Colin King
From: Colin Ian King The assignment of adev dereferences pointer hwmgr before hwmgr is null checked, hence there is a potential null pointer deference issue. Fix this by assigning adev after the null check. Addresses-Coverity: ("Dereference before null check") Fixes: 0896d2f7ba4d

Re: [PATCH v2] drm/gem: Fix mmap fake offset handling for drm_gem_object_funcs.mmap

2019-11-15 Thread Gerd Hoffmann
> You need memory pressure, to force ttm to unmap the bo, not userspace. So > roughly > 1. create bo > 2. mmap it through drm fd, write some stuff > 3. export to dma-buf, mmap it, verify stuff is there > 4. create a pile more bo, mmap them write to them > 5. once you've thrashed all of vram

Re: [PATCH 2/8] drm/atmel: ditch fb_create wrapper

2019-11-15 Thread Boris Brezillon
On Fri, 15 Nov 2019 10:21:14 +0100 Daniel Vetter wrote: > Spotted while looking through them all. > > Signed-off-by: Daniel Vetter > Cc: Sam Ravnborg > Cc: Boris Brezillon Acked-by: Boris Brezillon > Cc: Nicolas Ferre > Cc: Alexandre Belloni > Cc: Ludovic Desroches > Cc:

[PATCH 3/8] drm/mediatek: don't open-code drm_gem_fb_create

2019-11-15 Thread Daniel Vetter
Aside: There's a few other fb_create implementations which simply check for valid buffer format (or an approximation thereof), and then call drm_gem_fb_create. For atomic drivers at least we could walk all planes and make sure the format/modifier combo is valid, and remove even more code. For

[PATCH 7/8] drm/hibmc: Use drm_gem_fb_create

2019-11-15 Thread Daniel Vetter
Again we could delete a lot more if we'd switch over to the generic fbdev stuff. Signed-off-by: Daniel Vetter --- .../gpu/drm/hisilicon/hibmc/hibmc_drm_de.c| 4 +- .../gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h | 11 +--- .../gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c | 5 +-

[PATCH 2/8] drm/atmel: ditch fb_create wrapper

2019-11-15 Thread Daniel Vetter
Spotted while looking through them all. Signed-off-by: Daniel Vetter Cc: Sam Ravnborg Cc: Boris Brezillon Cc: Nicolas Ferre Cc: Alexandre Belloni Cc: Ludovic Desroches Cc: linux-arm-ker...@lists.infradead.org --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 8 +--- 1 file changed, 1

[PATCH 6/8] drm/xen: Simplify fb_create

2019-11-15 Thread Daniel Vetter
The current code is a pretty good wtf moment, since we drop the reference before we use it. It's not a big deal, because a) we only use the pointer, so doesn't blow up and the real reason b) fb->obj[0] already holds a full reference for us. Might as well take the real pointer ins't of complicated

[PATCH 0/8] fb_create drive-through cleanups

2019-11-15 Thread Daniel Vetter
Hi all, Inspired by some chatting with Pekka on irc I looked a lot at our ->fb_create implementations. Some cleanups (the simpler ones) and some todos (the more involved stuff). It's not a lot of code that we can collect even with all the todos, but we have so many drivers nowadays it's worth it

[PATCH 8/8] drm/todo: Add entry for fb funcs related cleanups

2019-11-15 Thread Daniel Vetter
We're doing a great job for really simple drivers right now, but still a lot of boilerplate for the bigger ones. Signed-off-by: Daniel Vetter --- Documentation/gpu/todo.rst | 26 ++ 1 file changed, 26 insertions(+) diff --git a/Documentation/gpu/todo.rst

[PATCH 1/8] drm/fb: More paranoia in addfb checks

2019-11-15 Thread Daniel Vetter
- Our limit is uint32_t, make that explicit. - Untangle the one overflow check, I think (but not sure) that with all three together you could overflow the uint64_t and it'd look cool again. Hence two steps. Also go with the more common (and imo safer approach) of reducing the range we

[PATCH 4/8] drm/rockchip: Use drm_gem_fb_create_with_dirty

2019-11-15 Thread Daniel Vetter
If rockchip would switch over to the generic fbdev setup we could grabage collect even more of all this code (all of the remaining fb handling code really). Signed-off-by: Daniel Vetter Cc: Sandy Huang Cc: "Heiko Stübner" Cc: linux-arm-ker...@lists.infradead.org Cc:

[PATCH 5/8] drm/tilcdc: Drop drm_gem_fb_create wrapper

2019-11-15 Thread Daniel Vetter
Doesn't do anything. Signed-off-by: Daniel Vetter Cc: Jyri Sarha Cc: Tomi Valkeinen --- drivers/gpu/drm/tilcdc/tilcdc_drv.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/gpu/drm/tilcdc/tilcdc_drv.c b/drivers/gpu/drm/tilcdc/tilcdc_drv.c index

Re: [Intel-gfx] [PATCH] drm/amdgpu/dm: Do not throw an error for a display with no audio

2019-11-15 Thread Daniel Vetter
On Fri, Nov 15, 2019 at 10:04 AM Jean Delvare wrote: > > Hi Chris, > > On Thu, 14 Nov 2019 20:44:13 +, Chris Wilson wrote: > > An old display with no audio may not have an EDID with a CEA block, or > > it may simply be too old to support audio. This is not a driver error, > > so don't flag it

[Bug 205497] Attempt to read amd gpu id causes a freeze

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205497 --- Comment #10 from V.I.S. (itemc...@mail.ru) --- AMD Ryzen 5 2600G + AMD RX560 (multiseat system), system freezed after few days on kernel 4.19.83 in my case. -- You are receiving this mail because: You are watching the assignee of the bug.

[Bug 112288] Blank display on Ubuntu 19.10 on Ryzen 3 2200G and Ryzen 5 2400G APUs

2019-11-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=112288 --- Comment #11 from Michel Dänzer --- (In reply to Alex Deucher from comment #3) > [1.094054] [drm:amdgpu_init [amdgpu]] *ERROR* VGACON disables amdgpu > kernel modesetting. > Did you enable vga console? FWIW, that message is due to

[Bug 205497] Attempt to read amd gpu id causes a freeze

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205497 --- Comment #9 from Trek (tre...@inbox.ru) --- thanks, I was not aware of it, may be different hardware from the ones on which kernel 4.19/5.1 works? -- You are receiving this mail because: You are watching the assignee of the bug.

Re: drm core/helpers and MIT license

2019-11-15 Thread Daniel Vetter
On Thu, Nov 14, 2019 at 08:01:32PM +, co...@sdf.org wrote: > Hi Daniel, > > I don't think we can make any complaints about GPL being more widely > used in the DRM code. It's nice to have the code at all, the MIT license > is a bonus. Thanks for writing it and bearing with us. > > Would

[Bug 205497] Attempt to read amd gpu id causes a freeze

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205497 --- Comment #8 from V.I.S. (itemc...@mail.ru) --- Please read here... https://github.com/lestofante/ksysguard-gpu/issues/4 Same issue on 4.19.x LTS kernel. -- You are receiving this mail because: You are watching the assignee of the bug.

[Bug 205497] Attempt to read amd gpu id causes a freeze

2019-11-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205497 Trek (tre...@inbox.ru) changed: What|Removed |Added CC||tre...@inbox.ru --- Comment #7

[PATCH 2/4] drm/amd/display: remove set but not used variable 'bp' in bios_parser2.c

2019-11-15 Thread zhengbin
Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c: In function bios_get_board_layout_info: drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c:1826:22: warning: variable bp set but not used [-Wunused-but-set-variable] It is introduced by commit

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

2019-11-15 Thread Yiwei Zhang
Thanks for all the comments and feedback, and they are all so valuable to me. Let me summarize the main concerns so far here: (1) Open source driver never specifies what API is creating a gem object (opengl, vulkan, ...) nor what purpose (transient, shader, ...). (2) The ioctl to label anything

[PATCH v1 1/2] drm/panel: add support for rm69299 visionox panel driver

2019-11-15 Thread Harigovindan P
Add support for Visionox panel driver. Signed-off-by: Harigovindan P --- drivers/gpu/drm/panel/Kconfig | 9 + drivers/gpu/drm/panel/Makefile | 1 + drivers/gpu/drm/panel/panel-visionox-rm69299.c | 478 + 3 files changed, 488

[PATCH] drm/imx: parallel-display: Adjust bus_flags and bus_format handling

2019-11-15 Thread Marek Vasut
The bus_flags and bus_format handling logic does not seem to cover all potential usecases. Specifically, this seems to fail with an "edt,etm0700g0edh6" display attached to an 24bit display interface, with interface-pix-fmt = "rgb24" set in DT. In this specific setup, the panel-simple.c driver

[PATCH 4/4] drm/amd/display: remove set but not used variable 'min_content'

2019-11-15 Thread zhengbin
Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/amd/display/modules/color/color_gamma.c: In function build_freesync_hdr: drivers/gpu/drm/amd/display/modules/color/color_gamma.c:830:20: warning: variable min_content set but not used [-Wunused-but-set-variable] It is not used

[PATCH 0/2] add new DRM_DEV_WARN macro

2019-11-15 Thread Wambui Karuga
This adds a new DRM_DEV_WARN helper macro for warnings log output that include device pointers. It also includes the use of the DRM_DEV_WARN macro in drm/rockchip to replace dev_warn. Wambui Karuga (2): drm/print: add DRM_DEV_WARN macro drm/rockchip: use DRM_DEV_WARN macro in debug output

[PATCH 2/2] drm/rockchip: use DRM_DEV_WARN macro in debug output

2019-11-15 Thread Wambui Karuga
Replace the use of dev_warn in debug output with the new DRM specific DRM_DEV_WARN debug output macro to maintain consistency across the driver. Signed-off-by: Wambui Karuga --- drivers/gpu/drm/rockchip/inno_hdmi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 1/4] drm/amd/display: remove set but not used variable 'old_plane_crtc'

2019-11-15 Thread zhengbin
Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c: In function dm_determine_update_type_for_commit: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:6516:36: warning: variable old_plane_crtc set but not used [-Wunused-but-set-variable] It is

[PATCH 3/4] drm/amd/display: remove set but not used variable 'bp' in bios_parser.c

2019-11-15 Thread zhengbin
Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/amd/display/dc/bios/bios_parser.c: In function bios_get_board_layout_info: drivers/gpu/drm/amd/display/dc/bios/bios_parser.c:2743:22: warning: variable bp set but not used [-Wunused-but-set-variable] It is introduced by commit

[PATCH 0/4] drm/amd/display: remove some set but not used variables

2019-11-15 Thread zhengbin
zhengbin (4): drm/amd/display: remove set but not used variable 'old_plane_crtc' drm/amd/display: remove set but not used variable 'bp' in bios_parser2.c drm/amd/display: remove set but not used variable 'bp' in bios_parser.c drm/amd/display: remove set but not used variable

Re: [Freedreno] [PATCH v1 2/2] drm/msm: add DSI config changes to support DSI version

2019-11-15 Thread Jeffrey Hugo
On Thu, Nov 14, 2019 at 10:47 AM Rob Clark wrote: > > On Thu, Nov 14, 2019 at 2:16 AM Harigovindan P > wrote: > > > > Add DSI config changes to support DSI version. > > > > Signed-off-by: Harigovindan P > > Reviewed-by: Rob Clark Can we fix the subject/commit text for this to indicate what

[PATCH v1 0/2] Add suppport for rm69299 Visionox panel driver and add DSI config to support DSI version

2019-11-15 Thread Harigovindan P
Current patchset adds support for rm69299 visionox panel driver used in MSM reference platforms and also adds DSI config that supports the respective DSI version. The visionox panel driver supports a resolution of 1080x2248 with 4 lanes and supports only single DSI mode. Current patchset is

  1   2   >