Re: [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-12-03 Thread Andrzej Hajda
On 03.12.2018 22:38, Ville Syrjälä wrote: > On Thu, Nov 29, 2018 at 10:08:07AM +0100, Andrzej Hajda wrote: >> On 21.11.2018 19:19, Laurent Pinchart wrote: >>> Hi Ville, >>> >>> Thank you for the patch. >>> >>> On Tuesday, 20 November 2018 18:13:42 EET Ville Syrjala wrote: From: Ville Syrjälä

Re: [PATCH -next] drm/amdgpu: Fix return value check in amdgpu_allocate_static_csa()

2018-12-03 Thread Zhu, Rex
Hi Yongjun, This issue has been fixed. Thanks. Best Regards Rex commit 51f1f6f51712aade68cabb145ed8bab4a6c3997e Author: Rex Zhu Date: Fri Nov 23 18:52:21 2018 +0800 drm/amdgpu: Fix static checker warning drivers/gpu/drm/amd/amdgpu/amdgpu_csa.c:49

Re: [PATCH 0/3] mmu notifier contextual informations

2018-12-03 Thread Koenig, Christian
Am 03.12.18 um 21:18 schrieb jgli...@redhat.com: > From: Jérôme Glisse > > This patchset add contextual information, why an invalidation is > happening, to mmu notifier callback. This is necessary for user > of mmu notifier that wish to maintains their own data structure > without having to add

Re: [PATCH] drm/sched: Fix a use-after-free when tracing the scheduler.

2018-12-03 Thread Koenig, Christian
Am 03.12.18 um 21:14 schrieb Eric Anholt: > With DEBUG_SLAB (poisoning on free) enabled, I could quickly produce > an oops when tracing V3D. Good catch, but the solution is a clear NAK. drm_sched_entity_add_dependency_cb() can result in setting entity->dependency to NULL. That in turn can lead

Re: [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-12-03 Thread Andrzej Hajda
On 03.12.2018 22:48, Ville Syrjälä wrote: > On Thu, Nov 29, 2018 at 09:46:16AM +0100, Andrzej Hajda wrote: >> Quite late, hopefully not too late. >> >> >> On 21.11.2018 12:51, Ville Syrjälä wrote: >>> On Wed, Nov 21, 2018 at 01:40:43PM +0200, Jani Nikula wrote: > return; > diff

[Bug 108487] Wayland compositors are unable to use hardware acceleration on i915

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108487 --- Comment #8 from Duane Voth --- ugh. typo. $ weston --use-pixman (can't edit comments?) -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list

[Bug 108487] Wayland compositors are unable to use hardware acceleration on i915

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108487 --- Comment #7 from Duane Voth --- lspci on my laptop lists: Display controller: Intel Corporation Mobile 945GM/GMS/GME, 943/940GML Express Integrated Graphics Controller (rev 03) I get the same errors when trying to start weston: ... []

Re: [PATCH -next] drm/amdgpu: Fix return value check in amdgpu_allocate_static_csa()

2018-12-03 Thread zhoucm1
On 2018年12月04日 14:39, Wei Yongjun wrote: Fix the return value check which testing the wrong variable in amdgpu_allocate_static_csa(). Fixes: 7946340fa389 ("drm/amdgpu: Move csa related code to separate file") Signed-off-by: Wei Yongjun --- drivers/gpu/drm/amd/amdgpu/amdgpu_csa.c | 2 +- 1

RE: DP1.2 MST HUB

2018-12-03 Thread Zhang, Yanmin
+intel-gfx. > -Original Message- > From: Zhang, Yanmin > Sent: Tuesday, December 04, 2018 10:31 AM > To: Vivi, Rodrigo ; Deak, Imre > Cc: Syrjala, Ville ; dri-devel@lists.freedesktop.org > Subject: RE: DP1.2 MST HUB > > +People in community. > > Rodrigo, > > Thanks for the kind info.

Re: [Freedreno] [PATCH v2] drm/msm/dpu: add display port support in DPU

2018-12-03 Thread Jeykumar Sankaran
On 2018-12-03 06:47, Sean Paul wrote: On Tue, Nov 27, 2018 at 02:28:30PM -0800, Jeykumar Sankaran wrote: Add display port support in DPU by creating hooks for DP encoder enumeration and encoder mode initialization. This change is based on the SDM845 Display port driver changes[1]. changes in

RE: [PATCH 02/11] dma-buf: add new dma_fence_chain container v2

2018-12-03 Thread Zhou, David(ChunMing)
> -Original Message- > From: Christian König > Sent: Monday, December 03, 2018 9:56 PM > To: Zhou, David(ChunMing) ; Koenig, Christian > ; dri-devel@lists.freedesktop.org; amd- > g...@lists.freedesktop.org > Subject: Re: [PATCH 02/11] dma-buf: add new dma_fence_chain container > v2 > >

Re: [PATCH v5 1/3] arm64: dts: qcom: sdm845: Add dpu to sdm845 dts file

2018-12-03 Thread Jeykumar Sankaran
On 2018-12-03 16:57, Doug Anderson wrote: Hi, On Mon, Dec 3, 2018 at 2:27 PM Jeykumar Sankaran wrote: + dsi0: dsi@ae94000 { + compatible = "qcom,mdss-dsi-ctrl"; + reg = <0xae94000 0x400>; +

RE: DP1.2 MST HUB

2018-12-03 Thread Zhang, Yanmin
+People in community. Rodrigo, Thanks for the kind info. What's the answer to below question? I find i915 binds one pipeline per display. It means even with DP1.2 MST, i915 driver still restricts the display number to the display pipeline number. Am I right? In addition, I might mislead you

Re: [PATCH v5 1/3] arm64: dts: qcom: sdm845: Add dpu to sdm845 dts file

2018-12-03 Thread Doug Anderson
Hi, On Mon, Dec 3, 2018 at 2:27 PM Jeykumar Sankaran wrote: > + dsi0: dsi@ae94000 { > + compatible = "qcom,mdss-dsi-ctrl"; > + reg = <0xae94000 0x400>; > + reg-names = "dsi_ctrl"; > +

[Bug 201867] New: Nouveau + discrete GPU (GP104M = GTX 1070 M) Driver Crashes, System freezes, dual screen not working

2018-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=201867 Bug ID: 201867 Summary: Nouveau + discrete GPU (GP104M = GTX 1070 M) Driver Crashes, System freezes, dual screen not working Product: Drivers Version: 2.5 Kernel Version:

Re: [PATCH 2/3] mm/mmu_notifier: use structure for invalidate_range_start/end calls

2018-12-03 Thread Jerome Glisse
On Mon, Dec 03, 2018 at 03:18:16PM -0500, jgli...@redhat.com wrote: > diff --git a/mm/migrate.c b/mm/migrate.c > index f7e4bfdc13b7..4896dd9d8b28 100644 > --- a/mm/migrate.c > +++ b/mm/migrate.c > @@ -2303,8 +2303,13 @@ static int migrate_vma_collect_pmd(pmd_t *pmdp, > */ > static void

Re: [RFC v3 08/19] arch: um: add shim to trap to allow installing a fault catcher for tests

2018-12-03 Thread Luis Chamberlain
On Mon, Dec 03, 2018 at 03:34:57PM -0800, Brendan Higgins wrote: > On Thu, Nov 29, 2018 at 7:34 PM Luis Chamberlain wrote: > > > > On Wed, Nov 28, 2018 at 11:36:25AM -0800, Brendan Higgins wrote: > > > diff --git a/arch/um/kernel/trap.c b/arch/um/kernel/trap.c > > > index

Re: [RFC v3 07/19] kunit: test: add initial tests

2018-12-03 Thread Luis Chamberlain
On Mon, Dec 03, 2018 at 03:26:26PM -0800, Brendan Higgins wrote: > On Thu, Nov 29, 2018 at 7:40 PM Luis Chamberlain wrote: > > > > On Wed, Nov 28, 2018 at 11:36:24AM -0800, Brendan Higgins wrote: > > > Add a test for string stream along with a simpler example. > > > > > > Signed-off-by: Brendan

Re: [PATCH v8 0/7] mm: Merge hmm into devm_memremap_pages, mark GPL-only

2018-12-03 Thread Jerome Glisse
On Wed, Nov 21, 2018 at 05:20:55PM -0800, Andrew Morton wrote: > On Tue, 20 Nov 2018 15:12:49 -0800 Dan Williams > wrote: [...] > > I am also concerned that HMM was designed in a way to minimize further > > engagement with the core-MM. That, with these hooks in place, > > device-drivers are

[Bug 107823] [amdgpu/displayport] Blackscreen on native resolution

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107823 --- Comment #23 from Drexler --- Hi Jan, thanks for the video. just like to ask, were you also able to try on using another DP cable? short and or thicker DP cable? Also, would just like to ask if you observe the issue on Ubuntu 18.10? I

Re: [PATCH v2 08/11] drm/i915/psr: Check if source supports sink specific SU granularity

2018-12-03 Thread Souza, Jose
On Mon, 2018-12-03 at 15:12 -0800, Pandiyan, Dhinakaran wrote: > On Mon, 2018-12-03 at 14:45 -0800, Souza, Jose wrote: > > On Mon, 2018-12-03 at 12:59 -0800, Dhinakaran Pandiyan wrote: > > > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > > > According to eDP spec, sink can

Re: [PATCH v2 08/11] drm/i915/psr: Check if source supports sink specific SU granularity

2018-12-03 Thread Pandiyan, Dhinakaran
On Mon, 2018-12-03 at 14:45 -0800, Souza, Jose wrote: > On Mon, 2018-12-03 at 12:59 -0800, Dhinakaran Pandiyan wrote: > > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > > According to eDP spec, sink can required specific selective > > > update > > > granularity that source

Re: [Intel-gfx] [PATCH v2 10/11] drm/i915: Improve PSR2 CTL macros

2018-12-03 Thread Rodrigo Vivi
On Mon, Dec 03, 2018 at 03:03:52PM -0800, Dhinakaran Pandiyan wrote: > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > - Reusing the EDP_PSR2_FRAME_BEFORE_SU_SHIFT in > > EDP_PSR2_FRAME_BEFORE_SU > > - Removing unused EDP_PSR2_FRAME_BEFORE_SU_MASK > > - Adding

Re: [PATCH v2 10/11] drm/i915: Improve PSR2 CTL macros

2018-12-03 Thread Dhinakaran Pandiyan
On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > - Reusing the EDP_PSR2_FRAME_BEFORE_SU_SHIFT in > EDP_PSR2_FRAME_BEFORE_SU > - Removing unused EDP_PSR2_FRAME_BEFORE_SU_MASK > - Adding EDP_PSR2_FRAME_BEFORE_SU_MAX > - Adding EDP_PSR2_IDLE_FRAME() > - Adding EDP_PSR2_IDLE_FRAME_MAX

[Bug 108919] Parkitect (Unity Game) dispalys artifacts and black screens with Vega hardware

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108919 --- Comment #3 from Markus --- Similar issue with black flickering reported here on an RX480 with Mesa 18.2.2 and ambient occlusion turned on:

Re: [PATCH v2 01/11] drm/i915: Disable PSR in Apple panels

2018-12-03 Thread Souza, Jose
On Mon, 2018-12-03 at 14:45 -0800, Dhinakaran Pandiyan wrote: > On Mon, 2018-12-03 at 12:14 -0800, Souza, Jose wrote: > > On Fri, 2018-11-30 at 15:35 -0800, Dhinakaran Pandiyan wrote: > > > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > > > i915 yet don't support PSR in Apple

[Bug 108912] Null PTR deref on amd-staging-drm-next since rebase for rc3

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108912 Samantha McVey changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[PATCH v3 03/10] drm/msm/dpu: Remove dpu_crtc_is_enabled()

2018-12-03 Thread Jordan Crouse
The static inline function dpu_crtc_enabled() is only called once and the function that calls it in turn is only called once and the return value can be easily checked in the calling functions so collapse everything down. v3: No changes Reviewed-by: Sean Paul Signed-off-by: Jordan Crouse ---

[PATCH v3 06/10] drm/msm: Make irq_postinstall optional

2018-12-03 Thread Jordan Crouse
Allow the KMS operation 'irq_postinstall' to be optional so that the target display drivers don't need to define a dummy function if they don't need one. v3: No changes Reviewed-by: Sean Paul Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/msm_drv.c | 6 +- 1 file changed, 5

[PATCH v3 09/10] drm/msm/dpu: Further cleanups for static inline functions

2018-12-03 Thread Jordan Crouse
Remove more static inline functions that are lightly used and/or very simple and easy to build into the calling functions. v3: Fix a nit from Sean Paul v2: Removed another unused function from dpu_hw_lm.c and add back dpu_crtc_get_client_type() since there was a question regarding its usefulness.

[PATCH v3 08/10] drm/msm/dpu: Cleanup the debugfs functions

2018-12-03 Thread Jordan Crouse
Do some debugfs cleanups from across the DPU driver. The DRM destroy functions will do a recursive delete on the entire debugfs node so there is no need to store dentry pointers for the debugfs files that are persistent for the life of the driver. This also means that the destroy functions can go

[PATCH v3 00/10] drm/msm/dpu: cleanups

2018-12-03 Thread Jordan Crouse
This is a rebase of https://patchwork.freedesktop.org/series/51214/ On top of https://gitlab.freedesktop.org/seanpaul/dpu-staging/tags/for_jcrouse Which should be up to date for the latest and greatest of the DPU. This should be mostly unchanged since the last revision with the exception that

[PATCH v3 01/10] drm/msm/dpu: Remove dpu_dbg

2018-12-03 Thread Jordan Crouse
The functions in dpu_dbg.c aren't used. The two main dump functions fail after a lookup from dpu_dbg_base.reg_base_list which turns out to never be populated and once those are removed the rest of the file doesn't make any sense. v3: No changes v2: Moved some unrelated changes to another patch

[PATCH v3 04/10] drm/msm/dpu: Remove unused functions

2018-12-03 Thread Jordan Crouse
Remove some unused container_of() helper functions. v3: No changes v2: Retained still used helper functions in the name of readability Reviewed-by: Sean Paul Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h | 10 --

[PATCH v3 10/10] drm/msm/dpu: Clean up dpu_media_info.h static inline functions

2018-12-03 Thread Jordan Crouse
Do some cleanup in the static inline functions defined in dpu_media_info.h by cleaning up gotos and unneeded local variables. v3: Added spaces between operators per Seal Paul and Sam Ravnborg Reviewed-by: Sean Paul Signed-off-by: Jordan Crouse --- .../gpu/drm/msm/disp/dpu1/msm_media_info.h

[PATCH v3 02/10] drm/msm/dpu: Remove dpu_crtc_get_mixer_height

2018-12-03 Thread Jordan Crouse
dpu_crtc_get_mixer_height() is only used once and the value it returns can be easily derived from the calling function. v3: No changes Reviewed-by: Sean Paul Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 3 +-- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 13

[PATCH v3 07/10] drm/msm/dpu: Remove dpu_irq and unused functions

2018-12-03 Thread Jordan Crouse
dpu_irq.c does some unneeded checks and passes control to dpu_core_irq.c The simple functions can be defined in the same file where we use them and the files and their associated hangers on can be deleted. Additionally the postinstall hook isn't used even in dpu_core_irq.c so zap that entire

[PATCH v3 05/10] drm/msm/dpu: Cleanup callers of dpu_hw_blk_init

2018-12-03 Thread Jordan Crouse
Outside of superfluous parameter checks the dpu_hw_blk_init() doesn't have any failure paths. Switch it over to be a void function and we can remove error handling paths in all the functions that call it. While we're in those functions remove unneeded initialization for a static variable. v3: No

Re: [PATCH v2 01/11] drm/i915: Disable PSR in Apple panels

2018-12-03 Thread Dhinakaran Pandiyan
On Mon, 2018-12-03 at 12:14 -0800, Souza, Jose wrote: > On Fri, 2018-11-30 at 15:35 -0800, Dhinakaran Pandiyan wrote: > > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > > i915 yet don't support PSR in Apple panels, so lets keep it > > > disabled > > > while we work on that. >

Re: [PATCH v2 08/11] drm/i915/psr: Check if source supports sink specific SU granularity

2018-12-03 Thread Souza, Jose
On Mon, 2018-12-03 at 12:59 -0800, Dhinakaran Pandiyan wrote: > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > According to eDP spec, sink can required specific selective update > > granularity that source must comply. > > Here caching the value if required and checking if

[Bug 108577] Black X laptop screen with cursor if HDMI not plugged in, bisected

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108577 --- Comment #35 from Duncan Roe --- Tried to add a comment just now but got server failure --- Comment #36 from Duncan Roe --- Tried to add a comment just now but got server failure --- Comment #37 from Duncan Roe --- Control key straight

[PATCH v5 1/3] arm64: dts: qcom: sdm845: Add dpu to sdm845 dts file

2018-12-03 Thread Jeykumar Sankaran
DPU is short for the Display Processing Unit. It is the display controller on Qualcomm SDM845 chips. This change adds MDSS and DSI nodes to enable display on the target device. Changes in v2: - Beefed up commit message - Use SoC specific compatibles for mdss and dpu (Rob H)

[PATCH v2 6/6] drm/v3d: Invalidate the caches from the outside in.

2018-12-03 Thread Eric Anholt
This would be a fairly obscure race, but let's make sure we don't ever lose it. Signed-off-by: Eric Anholt --- drivers/gpu/drm/v3d/v3d_gem.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/v3d/v3d_gem.c b/drivers/gpu/drm/v3d/v3d_gem.c index

[PATCH v2 1/6] drm/v3d: Document cache flushing ABI.

2018-12-03 Thread Eric Anholt
Right now, userspace doesn't do any L2T writes, but we should lay out our expectations for how it works. v2: Explicitly mention the VCD cache flushing requirements and that we'll flush the other caches before each of the CLs. Signed-off-by: Eric Anholt --- include/uapi/drm/v3d_drm.h | 8

[PATCH v2 4/6] drm/v3d: Drop the wait for L2T flush to complete.

2018-12-03 Thread Eric Anholt
According to Dave, once you've started an L2T flush, all L2T accesses will be blocked until the flush completes. This fixes a consistent 3-4ms stall between the ioctl and running the job, and 3DMMES Taiji goes from 27fps to 110fps. v2: Leave a note about why we don't need to wait for completion.

[PATCH v2 5/6] drm/v3d: Stop trying to flush L2C on V3D 3.3+

2018-12-03 Thread Eric Anholt
This cache was replaced with the slice accessing the L2T in the newer generations. Noted by Dave during review. Signed-off-by: Eric Anholt --- drivers/gpu/drm/v3d/v3d_gem.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/v3d/v3d_gem.c

[PATCH v2 3/6] drm/v3d: Don't bother flushing L1TD at job start.

2018-12-03 Thread Eric Anholt
This is the write combiner for TMU writes. You're supposed to flush that at job end if you had dirtied any cachelines. Flushing it at job start then doesn't make any sense. Signed-off-by: Eric Anholt Fixes: 57692c94dcbe ("drm/v3d: Introduce a new DRM driver for Broadcom V3D V3.x+")

[PATCH v2 2/6] drm/v3d: Drop unused v3d_flush_caches().

2018-12-03 Thread Eric Anholt
Now that I've specified how the end-of-pipeline flushing should work, we're never going to use this function. Signed-off-by: Eric Anholt Reviewed-by: Dave Emett --- drivers/gpu/drm/v3d/v3d_drv.h | 1 - drivers/gpu/drm/v3d/v3d_gem.c | 21 - 2 files changed, 22 deletions(-)

Re: [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-12-03 Thread Ville Syrjälä
On Thu, Nov 29, 2018 at 09:46:16AM +0100, Andrzej Hajda wrote: > Quite late, hopefully not too late. > > > On 21.11.2018 12:51, Ville Syrjälä wrote: > > On Wed, Nov 21, 2018 at 01:40:43PM +0200, Jani Nikula wrote: > >> > >>> return; > >>> diff --git

Re: [PATCH v3 14/24] drm/msm: dpu: Grab the modeset locks in frame_event

2018-12-03 Thread Daniel Vetter
On Fri, Nov 30, 2018 at 05:00:02PM -0500, Sean Paul wrote: > From: Sean Paul > > This patch wraps dpu_core_perf_crtc_release_bw() with modeset locks > since it digs into the state objects. > > Changes in v2: > - None > Changes in v3: > - Use those nifty new DRM_MODESET_LOCK_ALL_* helpers

Re: [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-12-03 Thread Ville Syrjälä
On Thu, Nov 29, 2018 at 10:08:07AM +0100, Andrzej Hajda wrote: > On 21.11.2018 19:19, Laurent Pinchart wrote: > > Hi Ville, > > > > Thank you for the patch. > > > > On Tuesday, 20 November 2018 18:13:42 EET Ville Syrjala wrote: > >> From: Ville Syrjälä > >> > >> Make life easier for drivers by

Re: [PATCH v2 08/11] drm/i915/psr: Check if source supports sink specific SU granularity

2018-12-03 Thread Dhinakaran Pandiyan
On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > According to eDP spec, sink can required specific selective update > granularity that source must comply. > Here caching the value if required and checking if source supports > it. > > Cc: Rodrigo Vivi > Cc: Dhinakaran Pandiyan >

Re: [Intel-gfx] [PATCH RFC 2/5] cgroup: Add mechanism to register vendor specific DRM devices

2018-12-03 Thread Kuehling, Felix
On 2018-11-28 4:14 a.m., Joonas Lahtinen wrote: > Quoting Ho, Kenny (2018-11-27 17:41:17) >> On Tue, Nov 27, 2018 at 4:46 AM Joonas Lahtinen >> wrote: >>> I think a more abstract property "% of GPU (processing power)" might >>> be a more universal approach. One can then implement that through

[Bug 108917] gamma adjustments cause stuttering with amdgpu.dc=1, especially problematic with RedShift etc.

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108917 --- Comment #5 from tempel.jul...@gmail.com --- Thanks for the explanation, I misunderstood you. So I've tested night colors in a KDE Plasma Wayland session: And indeed, it shows the same behavior as RedShift on Xorg (stutter with amdgpu.dc=1,

Re: [PATCH v2 07/11] drm/i915/psr: Check if resolution is supported by default SU granularity

2018-12-03 Thread Souza, Jose
On Fri, 2018-11-30 at 16:37 -0800, Dhinakaran Pandiyan wrote: > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > Selective updates have a default granularity requirements as stated > > by eDP spec > Needs reference to the location in the spec. Done > > > , so check if HW can

Re: [PATCH v2 01/11] drm/i915: Disable PSR in Apple panels

2018-12-03 Thread Dhinakaran Pandiyan
On Fri, 2018-11-30 at 15:35 -0800, Dhinakaran Pandiyan wrote: > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > i915 yet don't support PSR in Apple panels, so lets keep it > > disabled > > while we work on that. > > > > v2: Renamed DP_DPCD_QUIRK_PSR_NOT_CURRENTLY_SUPPORTED to

Re: [PATCH 1/6] drm/v3d: Document cache flushing ABI.

2018-12-03 Thread Eric Anholt
Dave Emett writes: > On Mon, 3 Dec 2018 at 17:22, Dave Emett wrote: >> >> > + * The caches (L1T, slice, and L2T) will be flushed before the job >> > + * executes. The TLB writes are guaranteed to have been flushed by >> >> I would say before *each* job executes, as the caches are flushed >>

Re: [Freedreno] [PATCH v2 5/5] drm/msm: subclass work object for vblank events

2018-12-03 Thread Jeykumar Sankaran
On 2018-12-03 06:21, Sean Paul wrote: On Fri, Nov 30, 2018 at 04:21:15PM -0800, Jeykumar Sankaran wrote: On 2018-11-30 12:07, Sean Paul wrote: > On Fri, Nov 30, 2018 at 11:45:55AM -0800, Jeykumar Sankaran wrote: > > On 2018-11-29 14:15, Sean Paul wrote: > > > On Tue, Nov 20, 2018 at 02:04:14PM

Re: [PATCH v2 03/11] drm/i915/psr: Set PSR CRC verification bit in sink inside PSR1 block

2018-12-03 Thread Souza, Jose
On Fri, 2018-11-30 at 15:54 -0800, Dhinakaran Pandiyan wrote: > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > As we have a else block for the 'if (dev_priv->psr.psr2_enabled) {' > > and this bit is only set for PSR1 move it to that block to make it > > more easy to read. > >

[PATCH 0/3] mmu notifier contextual informations

2018-12-03 Thread jglisse
From: Jérôme Glisse This patchset add contextual information, why an invalidation is happening, to mmu notifier callback. This is necessary for user of mmu notifier that wish to maintains their own data structure without having to add new fields to struct vm_area_struct (vma). For instance

[PATCH 3/3] mm/mmu_notifier: contextual information for event triggering invalidation

2018-12-03 Thread jglisse
From: Jérôme Glisse CPU page table update can happens for many reasons, not only as a result of a syscall (munmap(), mprotect(), mremap(), madvise(), ...) but also as a result of kernel activities (memory compression, reclaim, migration, ...). Users of mmu notifier API track changes to the CPU

[PATCH 2/3] mm/mmu_notifier: use structure for invalidate_range_start/end calls

2018-12-03 Thread jglisse
From: Jérôme Glisse To avoid having to change many call sites everytime we want to add a parameter use a structure to group all parameters for the mmu_notifier invalidate_range_start/end cakks. No functional changes with this patch. Signed-off-by: Jérôme Glisse Cc: Andrew Morton Cc: Matthew

[PATCH 1/3] mm/mmu_notifier: use structure for invalidate_range_start/end callback

2018-12-03 Thread jglisse
From: Jérôme Glisse To avoid having to change many callback definition everytime we want to add a parameter use a structure to group all parameters for the mmu_notifier invalidate_range_start/end callback. No functional changes with this patch. Signed-off-by: Jérôme Glisse Cc: Andrew Morton

[PATCH] drm/sched: Fix a use-after-free when tracing the scheduler.

2018-12-03 Thread Eric Anholt
With DEBUG_SLAB (poisoning on free) enabled, I could quickly produce an oops when tracing V3D. Signed-off-by: Eric Anholt --- I think this patch is correct (though maybe a bigger refactor could avoid the extra get/put?), but I've still got this with "vblank_mode=0 perf record -a -e v3d:.\* -e

Re: [PATCH v2 01/11] drm/i915: Disable PSR in Apple panels

2018-12-03 Thread Souza, Jose
On Fri, 2018-11-30 at 15:35 -0800, Dhinakaran Pandiyan wrote: > On Thu, 2018-11-29 at 18:25 -0800, José Roberto de Souza wrote: > > i915 yet don't support PSR in Apple panels, so lets keep it > > disabled > > while we work on that. > > > > v2: Renamed DP_DPCD_QUIRK_PSR_NOT_CURRENTLY_SUPPORTED to

[Bug 108934] [Dualscreen] "No signal" issues when booting with DVI and HDMI screens both plugged in

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108934 Bug ID: 108934 Summary: [Dualscreen] "No signal" issues when booting with DVI and HDMI screens both plugged in Product: DRI Version: unspecified Hardware: x86-64 (AMD64)

[PATCH] drm/msm: dpu: Allocate proper amount for dpu_crtc_state

2018-12-03 Thread Sean Paul
From: Sean Paul Since dpu_crtc subclasses crtc_state, we need a custom .reset hook in order to allocate the right amount of memory to accommodate the additional struct members in dpu_crtc_state. So bring it [partially] back. Relevant KASAN splat: [ 10.82]

Re: [PATCH v3 14/24] drm/msm: dpu: Grab the modeset locks in frame_event

2018-12-03 Thread Jeykumar Sankaran
On 2018-11-30 14:00, Sean Paul wrote: From: Sean Paul This patch wraps dpu_core_perf_crtc_release_bw() with modeset locks since it digs into the state objects. Changes in v2: - None Changes in v3: - Use those nifty new DRM_MODESET_LOCK_ALL_* helpers (Daniel) Cc: Daniel Vetter Cc: Jeykumar

Re: [PATCH v2 12/24] drm/msm: dpu: Move crtc runtime resume to encoder

2018-12-03 Thread Jeykumar Sankaran
On 2018-11-16 10:42, Sean Paul wrote: From: Sean Paul The crtc runtime resume doesn't actually operate on the crtc, but rather its encoders. The problem with this is that we need to inspect the crtc state to get the currently connected encoders. Since runtime resume isn't guaranteed to be

Re: [PATCH v2 11/24] drm/msm: dpu: Add ->enabled to dpu_encoder_virt

2018-12-03 Thread Jeykumar Sankaran
On 2018-11-16 10:42, Sean Paul wrote: From: Sean Paul Add a bool to dpu_encoder_virt to track whether the encoder is enabled or not. Repurpose the enc_lock mutex to ensure that it is consistent with the hw state. Changes in v2: - None Cc: Jeykumar Sankaran Signed-off-by: Sean Paul ---

Re: [PATCH v2 04/24] drm/msm: dpu: Don't use power_event for vbif_init_memtypes

2018-12-03 Thread Jeykumar Sankaran
On 2018-11-16 10:42, Sean Paul wrote: From: Sean Paul power_events are only used for pm_runtime, and that's all handled in dpu_kms. So just call vbif_init_memtypes at the correct times. Changes in v2: - Removed obsolete comment (Jeykumar) Cc: Jeykumar Sankaran Signed-off-by: Sean Paul ---

Re: [PATCH v2 15/24] drm/msm: dpu: Stop using encoder->crtc pointer

2018-12-03 Thread Jeykumar Sankaran
On 2018-11-26 13:53, Sean Paul wrote: On Mon, Nov 19, 2018 at 12:03:53PM -0800, Jeykumar Sankaran wrote: On 2018-11-16 13:14, Sean Paul wrote: > On Fri, Nov 16, 2018 at 12:05:09PM -0800, Jeykumar Sankaran wrote: > > On 2018-11-16 10:42, Sean Paul wrote: > > > From: Sean Paul > > > > > > It's

[PATCH] drm/msm/a6xx: Remove unwanted regulator code

2018-12-03 Thread Jordan Crouse
The GMU code currently has some misguided code to try to work around a hardware quirk that requires the power domains on the GPU be collapsed in a certain order. Future changes will do this the right way so get rid of the unused and unwanted regulator code. Signed-off-by: Jordan Crouse ---

[PATCH] drm/msm/a6xx: Add a name for the crashdumper buffer

2018-12-03 Thread Jordan Crouse
Add a buffer object name for the a6xx crashdumper so it can be seen with the changes introduced by 7799a98edd ("drm/msm: Add a name field for gem objects"). Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-)

[PATCH] drm/msm/a6xx: Use new kernel API free function for gpu state

2018-12-03 Thread Jordan Crouse
dadb36b7ec42 ("drm/msm: Add a common function to free kernel buffer objects") missed freeing the crashdumper state for a6xx. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git

[Bug 107978] [amdgpu] Switching to tty fails with DisplayPort 1.2 monitor going to sleep (REG_WAIT timeout / dce110_stream_encoder_dp_blank)

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107978 --- Comment #19 from Shmerl --- This seem to commonly affect Dell monitors. Do they not follow DisplayPort 1.2 spec, or amdgpu is doing something incorrectly after all? -- You are receiving this mail because: You are the assignee for the

[Bug 107978] [amdgpu] Switching to tty fails with DisplayPort 1.2 monitor going to sleep (REG_WAIT timeout / dce110_stream_encoder_dp_blank)

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107978 --- Comment #18 from Robin --- Same problem for me. Using 2 identical DELL U2415 and a Radeon RX 580. On Arch with 4.18.x Daisychainig both monitors worked like a charm. After upgrading to 4.19 the monitors only work when I`m disableing DP1.2.

Re: [PATCH V2] mm: Replace all open encodings for NUMA_NO_NODE

2018-12-03 Thread Doug Ledford
On Mon, 2018-11-26 at 17:56 +0530, Anshuman Khandual wrote: > At present there are multiple places where invalid node number is encoded > as -1. Even though implicitly understood it is always better to have macros > in there. Replace these open encodings for an invalid node number with the >

Re: [Intel-gfx] [PATCH RFC 2/5] cgroup: Add mechanism to register vendor specific DRM devices

2018-12-03 Thread Matt Roper
On Mon, Dec 03, 2018 at 06:46:01AM +, Ho, Kenny wrote: > Hey Matt, > > On Fri, Nov 30, 2018 at 5:22 PM Matt Roper wrote: > > I think Joonas is describing something closer in > > design to the cgroup-v2 "cpu" controller, which partitions the general > > time/usage allocated to via cgroup;

Re: [PATCH 1/3] dma-buf: add dma_fence_get_stub

2018-12-03 Thread Christian König
Am 03.12.18 um 18:06 schrieb Chris Wilson: Quoting Christian König (2018-12-03 16:12:14) Am 03.12.18 um 17:08 schrieb Eric Anholt: Christian König writes: Extract of useful code from the timeline work. This provides a function to return a stub or dummy fence which is always signaled.

[Bug 108919] Parkitect (Unity Game) dispalys artifacts and black screens with Vega hardware

2018-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108919 --- Comment #2 from Andreas Backx --- I am the creator of the reddit post mentioned in the opening comment of the issue. The 3 images (1 on the reddit post, and the 2 attachments posted) show exactly what I get when I start the game, depending

Re: [PATCH 1/3] dma-buf: add dma_fence_get_stub

2018-12-03 Thread Chris Wilson
Quoting Christian König (2018-12-03 16:12:14) > Am 03.12.18 um 17:08 schrieb Eric Anholt: > > Christian König writes: > > > >> Extract of useful code from the timeline work. This provides a function > >> to return a stub or dummy fence which is always signaled. > >> > >> Signed-off-by: Christian

Re: [PATCH v2] drm: add non-desktop quirks to Sensics and OSVR headsets.

2018-12-03 Thread Philipp Zabel
On Mon, 2018-12-03 at 10:46 -0600, Ryan Pavlik wrote: > Add two EDID vendor/product pairs used across a variety of > Sensics products, as well as the OSVR HDK and HDK 2. > > Signed-off-by: Ryan Pavlik Reviewed-by: Philipp Zabel > --- > > Replaces the earlier patch: > drm/edid: Quirk Sensics,

Re: [PATCH 1/3] dma-buf: add dma_fence_get_stub

2018-12-03 Thread Eric Anholt
Christian König writes: > Am 03.12.18 um 17:08 schrieb Eric Anholt: >> Christian König writes: >> >>> Extract of useful code from the timeline work. This provides a function >>> to return a stub or dummy fence which is always signaled. >>> >>> Signed-off-by: Christian König >>> --- >>>

[PATCH v2] drm: add non-desktop quirks to Sensics and OSVR headsets.

2018-12-03 Thread Ryan Pavlik
Add two EDID vendor/product pairs used across a variety of Sensics products, as well as the OSVR HDK and HDK 2. Signed-off-by: Ryan Pavlik --- Replaces the earlier patch: drm/edid: Quirk Sensics, 3 Glasses HMD as non-desktop. Changed: Drops the 3 Glasses HMD, because it is using a "default" ID

Re: [PATCH 1/3] dma-buf: add dma_fence_get_stub

2018-12-03 Thread Chris Wilson
Quoting Eric Anholt (2018-12-03 16:08:40) > Christian König writes: > > > Extract of useful code from the timeline work. This provides a function > > to return a stub or dummy fence which is always signaled. > > > > Signed-off-by: Christian König > > --- > > drivers/dma-buf/dma-fence.c | 36

[PATCH] dt-bindings: panel: Add missing .txt suffix

2018-12-03 Thread Thierry Reding
From: Thierry Reding All other files in that directory have a .txt suffix, so add one for consistency. Signed-off-by: Thierry Reding --- I meant to make this change while applying but was too quick on the trigger. Sorry for the extra noise. Thierry .../bindings/display/panel/{auo,g101evn010

Re: [PATCH 1/3] dma-buf: add dma_fence_get_stub

2018-12-03 Thread Christian König
Am 03.12.18 um 17:08 schrieb Eric Anholt: Christian König writes: Extract of useful code from the timeline work. This provides a function to return a stub or dummy fence which is always signaled. Signed-off-by: Christian König --- drivers/dma-buf/dma-fence.c | 36

Re: [PATCH 1/3] dma-buf: add dma_fence_get_stub

2018-12-03 Thread Eric Anholt
Christian König writes: > Extract of useful code from the timeline work. This provides a function > to return a stub or dummy fence which is always signaled. > > Signed-off-by: Christian König > --- > drivers/dma-buf/dma-fence.c | 36 +++- >

Re: [PATCH v1 1/4] drm/panel: simple: Add AUO G101EVN010 panel support

2018-12-03 Thread Thierry Reding
On Thu, Oct 25, 2018 at 05:09:30PM +0200, Alex Gonzalez wrote: > The change adds support for the AU Optronics G101EVN010 10.1" TFT LCD > panel. > > Signed-off-by: Alex Gonzalez > --- > .../bindings/display/panel/auo,g101evn010 | 12 ++ > drivers/gpu/drm/panel/panel-simple.c

Re: [PATCH 1/3] dma-buf: add dma_fence_get_stub

2018-12-03 Thread Chris Wilson
Quoting Alex Deucher (2018-12-03 15:53:21) > On Mon, Dec 3, 2018 at 8:40 AM Christian König > wrote: > > > > Am 03.12.18 um 14:33 schrieb Chunming Zhou: > > > The series is Reviewed-by: Chunming Zhou > > > > > > for patch#2, please remove my Signed-off-by, it's new when using stub > > > from

Re: [PATCH 2/3] drm/syncobj: use dma_fence_get_stub

2018-12-03 Thread Chris Wilson
Quoting Christian König (2018-12-03 13:07:58) > Extract of useful code from the timeline work. Let's use just a single > stub fence instance instead of allocating a new one all the time. > > Signed-off-by: Chunming Zhou > Signed-off-by: Christian König Reviewed-by: Chris Wilson -Chris

Re: [PATCH 1/3] dma-buf: add dma_fence_get_stub

2018-12-03 Thread Chris Wilson
Quoting Christian König (2018-12-03 13:07:57) > Extract of useful code from the timeline work. This provides a function > to return a stub or dummy fence which is always signaled. > > Signed-off-by: Christian König Reviewed-by: Chris Wilson -Chris ___

Re: [PATCH] drm/tegra: sor: Support for audio over HDMI

2018-12-03 Thread Jani Nikula
On Mon, 03 Dec 2018, Thierry Reding wrote: > From: Thierry Reding > > This code is very similar to the audio over HDMI support on older chips. > Interoperation with the audio codec is done via a pair of codec scratch > registers and an interrupt that is raised at the SOR when the codec has >

Re: [PATCH] drm/vc4: Add support for X/Y reflection

2018-12-03 Thread Eric Anholt
Boris Brezillon writes: > Add support for X/Y reflection when the plane is using linear or T-tiled > formats. X/Y reflection hasn't been tested on SAND formats, so we reject > them until proper testing/debugging has been done. Reviewed-by: Eric Anholt signature.asc Description: PGP signature

Re: [PATCH v2] drm/vc4: Add a debugfs entry to disable/enable the load tracker

2018-12-03 Thread Eric Anholt
Boris Brezillon writes: > On Fri, 30 Nov 2018 12:30:52 -0800 > Eric Anholt wrote: > >> Paul Kocialkowski writes: >> >> > In order to test whether the load tracker is working as expected, we >> > need the ability to compare the commit result with the underrun >> > indication. With the load

Re: [PATCH 1/3] dma-buf: add dma_fence_get_stub

2018-12-03 Thread Alex Deucher
On Mon, Dec 3, 2018 at 8:40 AM Christian König wrote: > > Am 03.12.18 um 14:33 schrieb Chunming Zhou: > > The series is Reviewed-by: Chunming Zhou > > > > for patch#2, please remove my Signed-off-by, it's new when using stub > > from dma-fence. > > Yeah, ok. There is indeed nothing left from

Re: [PATCH v3 1/3] drm/connector: Add generic underscan properties

2018-12-03 Thread Eric Anholt
Boris Brezillon writes: > On Mon, 3 Dec 2018 16:40:11 +0200 > Ville Syrjälä wrote: > >> On Thu, Nov 22, 2018 at 12:23:29PM +0100, Boris Brezillon wrote: >> > @@ -924,6 +978,29 @@ struct drm_connector { >> > */ >> >struct drm_property_blob *path_blob_ptr; >> > >> > + /** >> > + *

Re: [PATCH] drm/edid: Quirk Sensics, 3 Glasses HMD as non-desktop.

2018-12-03 Thread Philipp Zabel
On Sat, 2018-10-20 at 20:42 +0100, Ryan Pavlik wrote: > Two of the EDID vendor/product pairs are used across a variety of > Sensics products, as well as the OSVR HDK and HDK 2. > > The third is for the "3 Glasses" brand "D3" HMD. > > Signed-off-by: Ryan Pavlik > --- >

Re: [PATCH v3 1/3] drm/connector: Add generic underscan properties

2018-12-03 Thread Boris Brezillon
On Mon, 3 Dec 2018 16:40:11 +0200 Ville Syrjälä wrote: > On Thu, Nov 22, 2018 at 12:23:29PM +0100, Boris Brezillon wrote: > > @@ -924,6 +978,29 @@ struct drm_connector { > > */ > > struct drm_property_blob *path_blob_ptr; > > > > + /** > > +* @underscan_mode_property: Optional

  1   2   >