Re: [GIT PULL] HDMI codec improvements, v2

2021-06-16 Thread Maxime Ripard
Hi Thomas, On Thu, Jun 10, 2021 at 02:25:50PM +0200, Maxime Ripard wrote: > Hi, > > Here's a PR for the changes to hdmi-codec that need to be shared between > drm-misc-next and ASoC. > > This is the second iteration, fixing a bisection issue with compilation > > Thanks! > Maxime > > > The fol

Re: New uAPI for color management proposal and feedback request

2021-06-16 Thread Maxime Ripard
Hi Pekka, On Mon, Jun 07, 2021 at 11:06:32AM +0300, Pekka Paalanen wrote: > On Mon, 7 Jun 2021 09:48:05 +0200 > Maxime Ripard wrote: > > > I've started to implement this for the raspberrypi some time ago. > > > > https://github.com/raspberrypi/linux/pull/4201 > > > > It's basically two propert

[PATCH v8 0/4] Add MIPI rx DPI support

2021-06-16 Thread Xin Ji
Hi all, this patch series implement MIPI rx DPI feature. Please help to review. This is the v8 version, rebase DT on the latest code, removed HDCP patch(I'll upload HDCP feature by a new patch). Any mistakes, please let me know, I'll fix it in the next series. Change history: v8: Fix Laurent Pinc

[PATCH v8 1/4] dt-bindings:drm/bridge:anx7625:add vendor define flags

2021-06-16 Thread Xin Ji
Add 'bus-type' and 'data-lanes' define for port0. Define DP tx lane0, lane1 swing register array define, and audio enable flag. Signed-off-by: Xin Ji --- .../display/bridge/analogix,anx7625.yaml | 57 ++- 1 file changed, 56 insertions(+), 1 deletion(-) diff --git a/Documen

[PATCH v8 2/4] drm/bridge: anx7625: fix not correct return value

2021-06-16 Thread Xin Ji
At some time, the original code may return non zero value, force return 0 if operation finished. Reviewed-by: Robert Foss Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/a

[PATCH v8 3/4] drm/bridge: anx7625: add MIPI DPI input feature

2021-06-16 Thread Xin Ji
The basic anx7625 driver only support MIPI DSI rx signal input. This patch add MIPI DPI rx input configuration support, after apply this patch, the driver can support DSI rx or DPI rx by adding 'bus-type' in DT. Reviewed-by: Robert Foss Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/

[PATCH v8 4/4] drm/bridge: anx7625: add HDMI audio function

2021-06-16 Thread Xin Ji
Add audio HDMI codec function support, enable it through device true flag "analogix,audio-enable". Reviewed-by: Robert Foss Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 227 ++ drivers/gpu/drm/bridge/analogix/anx7625.h | 5 + 2 files changed, 232 i

Re: [PATCH v2 2/2] drm/bridge: ti-sn65dsi86: Implement the pwm_chip

2021-06-16 Thread Uwe Kleine-König
Hello Bjorn, On Tue, Jun 15, 2021 at 06:18:28PM -0500, Bjorn Andersson wrote: > The SN65DSI86 provides the ability to supply a PWM signal on GPIO 4, > with the primary purpose of controlling the backlight of the attached > panel. Add an implementation that exposes this using the standard PWM > fra

[Bug 205517] nouveau MMIO read of 00000000 FAULT at 619444

2021-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205517 Lee Jones (lee.jo...@linaro.org) changed: What|Removed |Added CC||lee.jo...@linaro.org -

Fixing problems around shared fences and RCU in DMA-buf

2021-06-16 Thread Christian König
Hi everyone and especially Daniel, this is the revised patch set to fix and rework dma_buf_poll(). The new code should avoid problems with RCU and also now correctly waits for all fences in the resv object. The rest of the series is then the well known change to dma_resv_test_signaled(), nouve

[PATCH 1/7] dma-buf: add some more kerneldoc to dma_resv_add_shared_fence

2021-06-16 Thread Christian König
Explicitly document that code can't assume that shared fences signal after the exclusive fence. Signed-off-by: Christian König --- drivers/dma-buf/dma-resv.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/dma-buf/dma-resv.c b/drivers/dma-buf/dma-resv.c index f26c

[PATCH 3/7] dma-buf: fix dma_resv_test_signaled test_all handling v2

2021-06-16 Thread Christian König
As the name implies if testing all fences is requested we should indeed test all fences and not skip the exclusive one because we see shared ones. v2: fix logic once more Signed-off-by: Christian König --- drivers/dma-buf/dma-resv.c | 33 - 1 file changed, 12 ins

[PATCH 5/7] drm/msm: always wait for the exclusive fence

2021-06-16 Thread Christian König
Drivers also need to to sync to the exclusive fence when a shared one is present. Completely untested since the driver won't even compile on !ARM. Signed-off-by: Christian König --- drivers/gpu/drm/msm/msm_gem.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git

[PATCH 7/7] drm/amdgpu: drop CS workaround adding the shared manually

2021-06-16 Thread Christian König
Drop the workaround adding the shared fence manually in the CS. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c inde

[PATCH 2/7] dma-buf: fix and rework dma_buf_poll v2

2021-06-16 Thread Christian König
Daniel pointed me towards this function and there are multiple obvious problems in the implementation. First of all the retry loop is not working as intended. In general the retry makes only sense if you grab the reference first and then check the sequence values. It's also good practice to keep

[PATCH 4/7] drm/nouveau: always wait for the exclusive fence

2021-06-16 Thread Christian König
Drivers also need to to sync to the exclusive fence when a shared one is present. Signed-off-by: Christian König --- drivers/gpu/drm/nouveau/nouveau_fence.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c b/drivers/gpu/drm/nouveau/nouv

[PATCH 6/7] drm/amdgpu: drop workaround for adding page table clears as shared fence

2021-06-16 Thread Christian König
We no longer need to add the exclusive fence as shared fence as welldrm/amdgpu: drop workaround for adding page table clears as shared fence We no longer need to add the exclusive fence as shared fence as well.. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | 8 +---

Re: [PATCH] drm/ttm: fix error handling in ttm_bo_handle_move_mem()

2021-06-16 Thread Dan Carpenter
On Wed, Jun 16, 2021 at 08:46:33AM +0200, Christian König wrote: > Sending the first message didn't worked, so let's try again. > > Am 16.06.21 um 08:30 schrieb Dan Carpenter: > > There are three bugs here: > > 1) We need to call unpopulate() if ttm_tt_populate() succeeds. > > 2) The "new_man = tt

Re: [PATCH] drm/ttm: fix error handling in ttm_bo_handle_move_mem()

2021-06-16 Thread Christian König
Am 16.06.21 um 10:37 schrieb Dan Carpenter: On Wed, Jun 16, 2021 at 08:46:33AM +0200, Christian König wrote: Sending the first message didn't worked, so let's try again. Am 16.06.21 um 08:30 schrieb Dan Carpenter: There are three bugs here: 1) We need to call unpopulate() if ttm_tt_populate

[Bug 213391] AMDGPU retries page fault with some specific processes amdgpu and sometimes followed [gfxhub0] retry page fault until *ERROR* ring gfx timeout, but soft recovered

2021-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=213391 --- Comment #15 from Michel Dänzer (mic...@daenzer.net) --- (In reply to Dominic Letz from comment #14) > Having the same issue on an E495 with Kernel 5.12.9. Will try to downgrade > the /lib/firmware/amdgpu any hint to which git tag you would con

[PATCH] drm/i915/gem: Remove duplicated call to ops->pread

2021-06-16 Thread Daniel Vetter
Between commit ae30af84edb5b7cc95485922e43afd909a892e1b Author: Maarten Lankhorst Date: Tue Mar 23 16:50:00 2021 +0100 drm/i915: Disable userptr pread/pwrite support. and commit 0049b688459b846f819b6e51c24cd0781fcfde41 Author: Matthew Auld Date: Thu Nov 5 15:49:33 2020 + drm/

Re: [PATCH v8 1/4] dt-bindings:drm/bridge:anx7625:add vendor define flags

2021-06-16 Thread Neil Armstrong
Hi, On 16/06/2021 09:48, Xin Ji wrote: > Add 'bus-type' and 'data-lanes' define for port0. Define DP tx lane0, > lane1 swing register array define, and audio enable flag. > > Signed-off-by: Xin Ji > --- > .../display/bridge/analogix,anx7625.yaml | 57 ++- > 1 file changed, 5

Re: [PATCH v8 3/4] drm/bridge: anx7625: add MIPI DPI input feature

2021-06-16 Thread Neil Armstrong
On 16/06/2021 09:50, Xin Ji wrote: > The basic anx7625 driver only support MIPI DSI rx signal input. > This patch add MIPI DPI rx input configuration support, after apply > this patch, the driver can support DSI rx or DPI rx by adding > 'bus-type' in DT. > > Reviewed-by: Robert Foss > Signed-off-

[PATCH v5] drm/i915: Be more gentle with exiting non-persistent context

2021-06-16 Thread Tvrtko Ursulin
From: Tvrtko Ursulin When a non-persistent context exits we currently mark it as banned in order to trigger fast termination of any outstanding GPU jobs it may have left running. In doing so we apply a very strict 1ms limit in which the left over job has to preempt before we issues an engine res

Re: [PATCH 1/3] drm/i915/guc: Add fetch of hwconfig table

2021-06-16 Thread kernel test robot
use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/John-C-Harrison-Intel-com/Add-support-for-querying-hw-info-that-UMDs-need/20210616-141805 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: i386

Re: [PATCH v7 13/15] drm/tegra: Implement job submission part of new UAPI

2021-06-16 Thread Jon Hunter
Hi Mikko, On 10/06/2021 12:04, Mikko Perttunen wrote: > Implement the job submission IOCTL with a minimum feature set. > > Signed-off-by: Mikko Perttunen > --- > v7: > * Allocate gather BO with DMA API to get page-aligned > memory > * Add error prints to a few places where they were missing >

Re: [PATCH] drm/ttm: fix error handling in ttm_bo_handle_move_mem()

2021-06-16 Thread Dan Carpenter
On Wed, Jun 16, 2021 at 10:47:14AM +0200, Christian König wrote: > > > Am 16.06.21 um 10:37 schrieb Dan Carpenter: > > On Wed, Jun 16, 2021 at 08:46:33AM +0200, Christian König wrote: > > > Sending the first message didn't worked, so let's try again. > > > > > > Am 16.06.21 um 08:30 schrieb Dan

Re: [PATCH 0/2] clk: Implement a clock request API

2021-06-16 Thread Maxime Ripard
Hi Stephen, Mike, On Mon, May 24, 2021 at 02:48:11PM +0200, Maxime Ripard wrote: > Hi Stephen, Mike, > > On Mon, May 03, 2021 at 10:32:21AM +0200, Maxime Ripard wrote: > > Hi Stephen, > > > > On Fri, Apr 30, 2021 at 01:59:39PM -0700, Stephen Boyd wrote: > > > Quoting Maxime Ripard (2021-04-13 03

Re: [Intel-gfx] [PATCH 3/3] drm/i915/uapi: Add query for L3 bank count

2021-06-16 Thread Tvrtko Ursulin
On 10/06/2021 21:46, john.c.harri...@intel.com wrote: From: John Harrison Various UMDs need to know the L3 bank count. So add a query API for it. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/intel_gt.c | 15 +++ drivers/gpu/drm/i915/gt/intel_gt.h | 1 + drivers/

Re: [PATCH] drm/vc4: hdmi: Rely on interrupts to handle hotplug

2021-06-16 Thread Dave Stevenson
On Mon, 24 May 2021 at 14:20, Maxime Ripard wrote: > > DRM currently polls for the HDMI connector status every 10s, which can > be an issue when we connect/disconnect a display quickly or the device > on the other end only issues a hotplug pulse (for example on EDID > change). > > Switch the drive

[PATCH v5] drm/i915: Be more gentle with exiting non-persistent context

2021-06-16 Thread Tvrtko Ursulin
From: Tvrtko Ursulin When a non-persistent context exits we currently mark it as banned in order to trigger fast termination of any outstanding GPU jobs it may have left running. In doing so we apply a very strict 1ms limit in which the left over job has to preempt before we issues an engine res

Re: [Intel-gfx] [PATCH 3/3] drm/i915/uapi: Add query for L3 bank count

2021-06-16 Thread Daniel Vetter
On Thu, Jun 10, 2021 at 10:46 PM wrote: > > From: John Harrison > > Various UMDs need to know the L3 bank count. So add a query API for it. Please link to both the igt test submission for this (there's not even a Test-with: on the cover letter) and the merge requests for the various UMD which us

Re: [PATCH] dt-bindings: Drop redundant minItems/maxItems

2021-06-16 Thread Vinod Koul
On 15-06-21, 13:15, Rob Herring wrote: > If a property has an 'items' list, then a 'minItems' or 'maxItems' with the > same size as the list is redundant and can be dropped. Note that is DT > schema specific behavior and not standard json-schema behavior. The tooling > will fixup the final schema a

Re: [Intel-gfx] [PATCH] drm/i915/gem: Remove duplicated call to ops->pread

2021-06-16 Thread Matthew Auld
On Wed, 16 Jun 2021 at 10:04, Daniel Vetter wrote: > > Between > > commit ae30af84edb5b7cc95485922e43afd909a892e1b > Author: Maarten Lankhorst > Date: Tue Mar 23 16:50:00 2021 +0100 > > drm/i915: Disable userptr pread/pwrite support. > > and > > commit 0049b688459b846f819b6e51c24cd0781fcfde

[Bug 213391] AMDGPU retries page fault with some specific processes amdgpu and sometimes followed [gfxhub0] retry page fault until *ERROR* ring gfx timeout, but soft recovered

2021-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=213391 --- Comment #16 from Dominic Letz (dominic.l...@berlin.de) --- (In reply to Michel Dänzer from comment #15) > (In reply to Dominic Letz from comment #14) > > Having the same issue on an E495 with Kernel 5.12.9. Will try to downgrade > > the /lib/f

Re: [PATCH 2/3] drm/vc4: hdmi: Move the HSM clock enable to runtime_pm

2021-06-16 Thread Dave Stevenson
Hi Maxime On Tue, 25 May 2021 at 10:11, Maxime Ripard wrote: > > In order to access the HDMI controller, we need to make sure the HSM > clock is enabled. If we were to access it with the clock disabled, the > CPU would completely hang, resulting in an hard crash. > > Since we have different code

Re: [PATCH 3/3] drm/vc4: hdmi: Make sure the controller is powered in detect

2021-06-16 Thread Dave Stevenson
On Tue, 25 May 2021 at 10:11, Maxime Ripard wrote: > > If the HPD GPIO is not available and drm_probe_ddc fails, we end up > reading the HDMI_HOTPLUG register, but the controller might be powered > off resulting in a CPU hang. Make sure we have the power domain and the > HSM clock powered during t

Re: [PATCH] drm/ttm: fix error handling in ttm_bo_handle_move_mem()

2021-06-16 Thread Christian König
Am 16.06.21 um 11:36 schrieb Dan Carpenter: On Wed, Jun 16, 2021 at 10:47:14AM +0200, Christian König wrote: Am 16.06.21 um 10:37 schrieb Dan Carpenter: On Wed, Jun 16, 2021 at 08:46:33AM +0200, Christian König wrote: Sending the first message didn't worked, so let's try again. Am 16.06.2

[PATCH v11 01/10] mm: Remove special swap entry functions

2021-06-16 Thread Alistair Popple
Remove multiple similar inline functions for dealing with different types of special swap entries. Both migration and device private swap entries use the swap offset to store a pfn. Instead of multiple inline functions to obtain a struct page for each swap entry type use a common function pfn_swap

[PATCH v11 00/10] Add support for SVM atomics in Nouveau

2021-06-16 Thread Alistair Popple
Hi Andrew, This is my series to add support for SVM atomics in Nouveau rebased onto v5.13-rc6-mmots-2021-06-15-20-28. Functionally there are no changes from the previous v10, however some changes were made during the rebase. If anyone would like to see a diff-of-diffs I can post it but my exhausti

[PATCH v11 02/10] mm/swapops: Rework swap entry manipulation code

2021-06-16 Thread Alistair Popple
Both migration and device private pages use special swap entries that are manipluated by a range of inline functions. The arguments to these are somewhat inconsitent so rework them to remove flag type arguments and to make the arguments similar for both read and write entry creation. Signed-off-by

[PATCH v11 03/10] mm/rmap: Split try_to_munlock from try_to_unmap

2021-06-16 Thread Alistair Popple
The behaviour of try_to_unmap_one() is difficult to follow because it performs different operations based on a fairly large set of flags used in different combinations. TTU_MUNLOCK is one such flag. However it is exclusively used by try_to_munlock() which specifies no other flags. Therefore rather

[PATCH v11 04/10] mm/rmap: Split migration into its own function

2021-06-16 Thread Alistair Popple
Migration is currently implemented as a mode of operation for try_to_unmap_one() generally specified by passing the TTU_MIGRATION flag or in the case of splitting a huge anonymous page TTU_SPLIT_FREEZE. However it does not have much in common with the rest of the unmap functionality of try_to_unma

[PATCH v11 05/10] mm: Rename migrate_pgmap_owner

2021-06-16 Thread Alistair Popple
MMU notifier ranges have a migrate_pgmap_owner field which is used by drivers to store a pointer. This is subsequently used by the driver callback to filter MMU_NOTIFY_MIGRATE events. Other notifier event types can also benefit from this filtering, so rename the 'migrate_pgmap_owner' field to 'owne

[PATCH v11 06/10] mm/memory.c: Allow different return codes for copy_nonpresent_pte()

2021-06-16 Thread Alistair Popple
Currently if copy_nonpresent_pte() returns a non-zero value it is assumed to be a swap entry which requires further processing outside the loop in copy_pte_range() after dropping locks. This prevents other values being returned to signal conditions such as failure which a subsequent change requires

[PATCH v11 07/10] mm: Device exclusive memory access

2021-06-16 Thread Alistair Popple
Some devices require exclusive write access to shared virtual memory (SVM) ranges to perform atomic operations on that memory. This requires CPU page tables to be updated to deny access whilst atomic operations are occurring. In order to do this introduce a new swap entry type (SWP_DEVICE_EXCLUSIV

[PATCH v11 08/10] mm: Selftests for exclusive device memory

2021-06-16 Thread Alistair Popple
Adds some selftests for exclusive device memory. Signed-off-by: Alistair Popple Acked-by: Jason Gunthorpe Tested-by: Ralph Campbell Reviewed-by: Ralph Campbell --- v10: * Squashed a fix from Colin King. Not sure what the right attribution tag for that would be though? --- lib/test_hmm.c

[PATCH v11 09/10] nouveau/svm: Refactor nouveau_range_fault

2021-06-16 Thread Alistair Popple
Call mmu_interval_notifier_insert() as part of nouveau_range_fault(). This doesn't introduce any functional change but makes it easier for a subsequent patch to alter the behaviour of nouveau_range_fault() to support GPU atomic operations. Signed-off-by: Alistair Popple Reviewed-by: Ben Skeggs

[PATCH v11 10/10] nouveau/svm: Implement atomic SVM access

2021-06-16 Thread Alistair Popple
Some NVIDIA GPUs do not support direct atomic access to system memory via PCIe. Instead this must be emulated by granting the GPU exclusive access to the memory. This is achieved by replacing CPU page table entries with special swap entries that fault on userspace access. The driver then grants th

Re: [PATCH] dt-bindings: Drop redundant minItems/maxItems

2021-06-16 Thread Philipp Zabel
On Tue, 2021-06-15 at 13:15 -0600, Rob Herring wrote: > If a property has an 'items' list, then a 'minItems' or 'maxItems' with the > same size as the list is redundant and can be dropped. Note that is DT > schema specific behavior and not standard json-schema behavior. The tooling > will fixup the

Re: [PATCH] modified: gpu/drm/panfrost/panfrost_gpu.c

2021-06-16 Thread Steven Price
On 15/06/2021 08:04, Chunyou Tang wrote: > Hi steve, > After I send the V2,I found I setting a wrong email > configuration,I hope it doesn't affect the patch submission :) > Did you received my another patch about panfrost_job.c? There's still something odd going on with your emails -

Re: [PATCH] dt-bindings: Drop redundant minItems/maxItems

2021-06-16 Thread Mark Brown
On Tue, Jun 15, 2021 at 01:15:43PM -0600, Rob Herring wrote: > If a property has an 'items' list, then a 'minItems' or 'maxItems' with the > same size as the list is redundant and can be dropped. Note that is DT > schema specific behavior and not standard json-schema behavior. The tooling > will fi

Re: [PATCH -next] drivers: gpu: add missing MODULE_DEVICE_TABLE in anx7625.c

2021-06-16 Thread Robert Foss
Hey Yu, Thank you for submitting this, it looks good to me and I pulled into drm-misc-next. https://cgit.freedesktop.org/drm/drm-misc/commit/?id=ad5fd900a69b1ae24e6b22506dea637b6bbbdb55 On Wed, 16 Jun 2021 at 05:44, Yu Jiahua wrote: > > This patch adds missing MODULE_DEVICE_TABLE definition whi

Re: [PATCH v12 00/12] Restricted DMA

2021-06-16 Thread Will Deacon
Hi Claire, On Wed, Jun 16, 2021 at 02:21:45PM +0800, Claire Chang wrote: > This series implements mitigations for lack of DMA access control on > systems without an IOMMU, which could result in the DMA accessing the > system memory at unexpected times and/or unexpected addresses, possibly > leadin

Re: [PATCH] drm/vc4: hdmi: Rely on interrupts to handle hotplug

2021-06-16 Thread Maxime Ripard
On Wed, Jun 16, 2021 at 11:09:29AM +0100, Dave Stevenson wrote: > On Mon, 24 May 2021 at 14:20, Maxime Ripard wrote: > > > > DRM currently polls for the HDMI connector status every 10s, which can > > be an issue when we connect/disconnect a display quickly or the device > > on the other end only i

Re: [PATCH] dt-bindings: Drop redundant minItems/maxItems

2021-06-16 Thread Jonathan Cameron
On Tue, 15 Jun 2021 13:15:43 -0600 Rob Herring wrote: > If a property has an 'items' list, then a 'minItems' or 'maxItems' with the > same size as the list is redundant and can be dropped. Note that is DT > schema specific behavior and not standard json-schema behavior. The tooling > will fixup t

[PATCH] drm/i915: return DRIVER_NAME for the fence driver name

2021-06-16 Thread Matthew Auld
The first tracepoint for a request is trace_dma_fence_init which is called in the ctor before we have properly setup the request->engine. So if it's a non-recycled request the rq->engine might be NULL, or some garbage value, which leads to a crash. Since we are not permitted to use kmem_cache_zall

Re: [PATCH 3/3] drm/vc4: hdmi: Make sure the controller is powered in detect

2021-06-16 Thread Maxime Ripard
On Wed, Jun 16, 2021 at 11:51:26AM +0100, Dave Stevenson wrote: > On Tue, 25 May 2021 at 10:11, Maxime Ripard wrote: > > > > If the HPD GPIO is not available and drm_probe_ddc fails, we end up > > reading the HDMI_HOTPLUG register, but the controller might be powered > > off resulting in a CPU han

Re: [Intel-gfx] [PATCH 4/5] dma-buf: Stop using SLAB_TYPESAFE_BY_RCU in selftests

2021-06-16 Thread kernel test robot
BY_RCU-for-dma_fence/20210616-154432 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: sparc-randconfig-s032-20210615 (attached as .config) compiler: sparc-linux-gcc (GCC) 9.3.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross

Re: [PATCH] drm/i915: return DRIVER_NAME for the fence driver name

2021-06-16 Thread Daniel Vetter
On Wed, Jun 16, 2021 at 01:28:33PM +0100, Matthew Auld wrote: > The first tracepoint for a request is trace_dma_fence_init which is > called in the ctor before we have properly setup the request->engine. So > if it's a non-recycled request the rq->engine might be NULL, or some > garbage value, whic

Re: Allow mdev drivers to directly create the vfio_device (v3)

2021-06-16 Thread Jason Gunthorpe
On Wed, Jun 16, 2021 at 05:13:13AM +0200, Christoph Hellwig wrote: > On Tue, Jun 15, 2021 at 05:35:15PM -0300, Jason Gunthorpe wrote: > > Yes, the rest of the drivers will get converted eventually too. There > > is no reason to hold things back. Depending on timelines we might be > > able to get AP

[PATCH v2 1/3] drm/dp_helper: Rework the drm_dp_aux documentation

2021-06-16 Thread Maxime Ripard
Split the existing documentation to move the comments on particular fields next to them. Suggested-by: Daniel Vetter Signed-off-by: Maxime Ripard --- Changes from v1: - New patch --- include/drm/drm_dp_helper.h | 84 + 1 file changed, 57 insertions(+), 27

[PATCH v2 2/3] drm/dp_helper: Mention the concurrency requirement hw_mutex

2021-06-16 Thread Maxime Ripard
Drivers that allow concurrent access over multiple DP channels need to provide additional locking, even though the hw_mutex field might indicate otherwise. Clarify it in the documentation. Suggested-by: Daniel Vetter Signed-off-by: Maxime Ripard --- Changes from v1: - New patch --- include/

[PATCH v2 3/3] drm: Mention the power state requirement on side-channel operations

2021-06-16 Thread Maxime Ripard
The drm_connector detect, drm_dp_aux transfer and mipi_dsi_host operations typically require to access their underlying device to perform what is expected of them. However, there's no guarantee on the fact that the device has been enabled through atomic_enable or similar that will usually power th

Re: [PATCH] drm/i915: allow DG1 autoprobe for CONFIG_BROKEN

2021-06-16 Thread Matthew Auld
On Mon, 14 Jun 2021 at 10:22, Matthew Auld wrote: > > Purely for CI so we can get some pre-merge results for DG1. This is > especially useful for cross driver TTM changes where CI can hopefully > catch regressions. This is similar to how we already handle the DG1 > specific uAPI, which are also hi

Re: [PATCH 1/3] drm/i915/guc: Add fetch of hwconfig table

2021-06-16 Thread kernel test robot
use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/John-C-Harrison-Intel-com/Add-support-for-querying-hw-info-that-UMDs-need/20210616-141805 base: git://anongit.freedesktop.org/drm-intel for-linux-next conf

[PATCH v4] Documentation: gpu: Mention the requirements for new properties

2021-06-16 Thread Maxime Ripard
New KMS properties come with a bunch of requirements to avoid each driver from running their own, inconsistent, set of properties, eventually leading to issues like property conflicts, inconsistencies between drivers and semantics, etc. Let's document what we expect. Cc: Alexandre Belloni Cc: Al

[PATCH] drm/display: Fix duplicated argument

2021-06-16 Thread Wan Jiabing
Fix coccicheck warning: ./drivers/gpu/drm/amd/display/dc/dml/dcn31/display_rq_dlg_calc_31.c: 55:12-42: duplicated argument to && or || Signed-off-by: Wan Jiabing --- .../gpu/drm/amd/display/dc/dml/dcn31/display_rq_dlg_calc_31.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v4 2/7] drm/i915/ttm: add i915_sg_from_buddy_resource

2021-06-16 Thread Matthew Auld
We need to be able to build an sg table from our list of buddy blocks, so that we can later plug this into our ttm backend, and replace our use of the range manager. Signed-off-by: Matthew Auld Cc: Thomas Hellström Reviewed-by: Thomas Hellström --- drivers/gpu/drm/i915/i915_scatterlist.c | 80

[PATCH v4 1/7] drm/i915/ttm: add ttm_buddy_man

2021-06-16 Thread Matthew Auld
Add back our standalone i915_buddy allocator and integrate it into a ttm_resource_manager. This will plug into our ttm backend for managing device local-memory in the next couple of patches. v2(Thomas): - Return -ENOSPC from the buddy; ttm expects this in order to trigger eviction -

[PATCH v4 3/7] drm/i915/ttm: Calculate the object placement at get_pages time

2021-06-16 Thread Matthew Auld
From: Thomas Hellström Instead of relying on a static placement, calculate at get_pages() time. This should work for LMEM regions and system for now. For stolen we need to take preallocated range into account. That well be added later. Instead of relying on a static placement, calculate at get_p

[PATCH v4 4/7] drm/i915/ttm: pass along the I915_BO_ALLOC_CONTIGUOUS

2021-06-16 Thread Matthew Auld
Currently we just ignore the I915_BO_ALLOC_CONTIGUOUS flag, which is fine since everything is already contiguous with the ttm range manager. However in the next patch we want to switch over to the ttm buddy manager, where allocations are by default not contiguous. v2(Thomas): - Forward ALLOC_C

[PATCH v4 5/7] drm/i915/ttm: remove node usage in our naming

2021-06-16 Thread Matthew Auld
Now that ttm_resource_manager just returns a generic ttm_resource we don't need to reference the mm_node stuff anymore which mostly only makes sense for drm_mm_node. In the next few patches we want switch over to the ttm_buddy_man which is just another type of ttm_resource so reflect that in the na

[PATCH v4 6/7] drm/i915/ttm: switch over to ttm_buddy_man

2021-06-16 Thread Matthew Auld
Move back to the buddy allocator for managing device local memory, and restore the lost mock selftests. Keep around the range manager related bits, since we likely need this for managing stolen at some point. For stolen we also don't need to reserve anything so no need to support a generic reserve

[PATCH v4 7/7] drm/i915/ttm: restore min_page_size behaviour

2021-06-16 Thread Matthew Auld
We now have bo->page_alignment which perfectly describes what we need if we have min page size restrictions for lmem. We can also drop the flag here, since this is the default behaviour for all objects. v2(Thomas): - bo->page_alignment is in page units Signed-off-by: Matthew Auld Cc: Thomas

Re: [v6 5/5] drm/panel-simple: Add Samsung ATNA33XC20

2021-06-16 Thread Doug Anderson
Hi, On Sat, Jun 12, 2021 at 5:09 AM Rajeev Nandan wrote: > > +static const struct panel_desc samsung_atna33xc20 = { > + .modes = &samsung_atna33xc20_mode, > + .num_modes = 1, > + .bpc = 10, > + .size = { > + .width = 294, > + .height = 165, > +

[GIT PULL] mediatek drm next for 5.14

2021-06-16 Thread Chun-Kuang Hu
Hi, Dave & Daniel: This includes: 1. Add MT8167 HDMI support 2. Fix PM reference leak 3. Add MT8183 DPI dual edge support Regards, Chun-Kuang. The following changes since commit 6efb943b8616ec53a5e444193dccf1af9ad627b5: Linux 5.13-rc1 (2021-05-09 14:17:44 -0700) are available in the Git rep

Re: [Intel-gfx] [PATCH 0/3] Update firmware to v62.0.0

2021-06-16 Thread Mike Lothian
Hi Is there a place where we can download these new firmware images? Cheers Mike On Wed, 16 Jun 2021 at 00:55, Matthew Brost wrote: > As part of enabling GuC submission [1] we need to update to the latest > and greatest firmware. This series does that. All backwards > compatibility breaking c

[PATCH v5 1/7] drm/i915/ttm: add ttm_buddy_man

2021-06-16 Thread Matthew Auld
Add back our standalone i915_buddy allocator and integrate it into a ttm_resource_manager. This will plug into our ttm backend for managing device local-memory in the next couple of patches. v2(Thomas): - Return -ENOSPC from the buddy; ttm expects this in order to trigger eviction -

[PATCH v5 6/7] drm/i915/ttm: switch over to ttm_buddy_man

2021-06-16 Thread Matthew Auld
Move back to the buddy allocator for managing device local memory, and restore the lost mock selftests. Keep around the range manager related bits, since we likely need this for managing stolen at some point. For stolen we also don't need to reserve anything so no need to support a generic reserve

[PATCH v5 5/7] drm/i915/ttm: remove node usage in our naming

2021-06-16 Thread Matthew Auld
Now that ttm_resource_manager just returns a generic ttm_resource we don't need to reference the mm_node stuff anymore which mostly only makes sense for drm_mm_node. In the next few patches we want switch over to the ttm_buddy_man which is just another type of ttm_resource so reflect that in the na

[PATCH v5 7/7] drm/i915/ttm: restore min_page_size behaviour

2021-06-16 Thread Matthew Auld
We now have bo->page_alignment which perfectly describes what we need if we have min page size restrictions for lmem. We can also drop the flag here, since this is the default behaviour for all objects. v2(Thomas): - bo->page_alignment is in page units Signed-off-by: Matthew Auld Cc: Thomas

[PATCH v5 2/7] drm/i915/ttm: add i915_sg_from_buddy_resource

2021-06-16 Thread Matthew Auld
We need to be able to build an sg table from our list of buddy blocks, so that we can later plug this into our ttm backend, and replace our use of the range manager. Signed-off-by: Matthew Auld Cc: Thomas Hellström Reviewed-by: Thomas Hellström --- drivers/gpu/drm/i915/i915_scatterlist.c | 80

[PATCH v5 3/7] drm/i915/ttm: Calculate the object placement at get_pages time

2021-06-16 Thread Matthew Auld
From: Thomas Hellström Instead of relying on a static placement, calculate at get_pages() time. This should work for LMEM regions and system for now. For stolen we need to take preallocated range into account. That will if needed be added later. Signed-off-by: Thomas Hellström Reviewed-by: Matt

[PATCH v5 4/7] drm/i915/ttm: pass along the I915_BO_ALLOC_CONTIGUOUS

2021-06-16 Thread Matthew Auld
Currently we just ignore the I915_BO_ALLOC_CONTIGUOUS flag, which is fine since everything is already contiguous with the ttm range manager. However in the next patch we want to switch over to the ttm buddy manager, where allocations are by default not contiguous. v2(Thomas): - Forward ALLOC_C

Re: [Intel-gfx] [PATCH 06/31] drm/i915: Drop the CONTEXT_CLONE API (v2)

2021-06-16 Thread kernel test robot
the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Jason-Ekstrand/drm-i915-gem-ioctl-clean-ups-v6/20210616-151016 b

Re: [Intel-gfx] [PATCH 5/5] DONOTMERGE: dma-buf: Get rid of dma_fence_get_rcu_safe

2021-06-16 Thread kernel test robot
Hi Jason, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-intel/for-linux-next] [also build test WARNING on drm-tip/drm-tip drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next linus/master v5.13-rc6 next-20210616] [cannot apply to drm/drm-next] [If

Re: [PATCH 4/7] dma-buf: add dma_fence_chain_garbage_collect

2021-06-16 Thread kernel test robot
Hi "Christian, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-intel/for-linux-next] [also build test WARNING on drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next linus/master v5.13-rc6 next-20210616] [cannot apply to drm-tip/drm-tip drm/drm-next] [If

Re: [Mesa-dev] [PATCH 0/6] dma-buf: Add an API for exporting sync files (v12)

2021-06-16 Thread Jason Ekstrand
On Tue, Jun 15, 2021 at 3:41 AM Christian König wrote: > > Hi Jason & Daniel, > > maybe I should explain once more where the problem with this approach is > and why I think we need to get that fixed before we can do something > like this here. > > To summarize what this patch here does is that it

[PULL] drm-misc-next-fixes

2021-06-16 Thread Thomas Zimmermann
Hi Dave and Daniel, here's this week's PR for drm-misc-next-fixes. Best regards Thomas drm-misc-next-fixes-2021-06-16: Short summary of fixes pull: * hyperv: advertise the correct formatmodifiers for its primary plane * dp_mst: VCPI fixes to make it work with StarTech hub The following chang

Re: [PATCH] dt-bindings: Drop redundant minItems/maxItems

2021-06-16 Thread Wolfram Sang
On Tue, Jun 15, 2021 at 01:15:43PM -0600, Rob Herring wrote: > If a property has an 'items' list, then a 'minItems' or 'maxItems' with the > same size as the list is redundant and can be dropped. Note that is DT > schema specific behavior and not standard json-schema behavior. The tooling > will fi

Re: [PATCH 4/7] dma-buf: add dma_fence_chain_garbage_collect

2021-06-16 Thread kernel test robot
Hi "Christian, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-intel/for-linux-next] [also build test WARNING on drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next linus/master v5.13-rc6 next-20210616] [cannot apply to drm-tip/drm-tip drm/drm-next] [If

Re: [PATCH] drm/ttm: fix error handling in ttm_bo_handle_move_mem()

2021-06-16 Thread Dan Carpenter
On Wed, Jun 16, 2021 at 01:00:38PM +0200, Christian König wrote: > > > Am 16.06.21 um 11:36 schrieb Dan Carpenter: > > On Wed, Jun 16, 2021 at 10:47:14AM +0200, Christian König wrote: > > > > > > Am 16.06.21 um 10:37 schrieb Dan Carpenter: > > > > On Wed, Jun 16, 2021 at 08:46:33AM +0200, Christ

Re: [v6 1/5] drm/panel: add basic DP AUX backlight support

2021-06-16 Thread Lyude Paul
Reviewed-by: Lyude Paul On Sat, 2021-06-12 at 17:37 +0530, Rajeev Nandan wrote: > Some panels support backlight control over DP AUX channel using > VESA's standard backlight control interface. > Using new DRM eDP backlight helpers, add support to create and > register a backlight for those panels

Re: [PATCH v2 2/2] drm/dp_mst: Avoid to mess up payload table by ports in stale topology

2021-06-16 Thread Souza, Jose
On Wed, 2021-06-16 at 11:55 +0800, Wayne Lin wrote: > [Why] > After unplug/hotplug hub from the system, userspace might start to > clear stale payloads gradually. If we call drm_dp_mst_deallocate_vcpi() > to release stale VCPI of those ports which are not relating to current > topology, we have cha

[PATCH] drm/dp_mst: Add missing drm parameters to recently added call to drm_dbg_kms()

2021-06-16 Thread José Roberto de Souza
Commit 3769e4c0af5b ("drm/dp_mst: Avoid to mess up payload table by ports in stale topology") added to calls to drm_dbg_kms() but it missed the first parameter, the drm device breaking the build. Fixes: 3769e4c0af5b ("drm/dp_mst: Avoid to mess up payload table by ports in stale topology") Cc: Way

Re: [PATCH] drm/dp_mst: Add missing drm parameters to recently added call to drm_dbg_kms()

2021-06-16 Thread Lyude Paul
Reviewed-by: Lyude Paul Will go ahead and push this to drm-misc-next-fixes, thanks On Wed, 2021-06-16 at 12:44 -0700, José Roberto de Souza wrote: > Commit 3769e4c0af5b ("drm/dp_mst: Avoid to mess up payload table by > ports in stale topology") added to calls to drm_dbg_kms() but it > missed the

Re: [PATCH v2 1/2] drm/dp_mst: Do not set proposed vcpi directly

2021-06-16 Thread Harry Wentland
On 2021-06-15 11:55 p.m., Wayne Lin wrote: > [Why] > When we receive CSN message to notify one port is disconnected, we will > implicitly set its corresponding num_slots to 0. Later on, we will > eventually call drm_dp_update_payload_part1() to arrange down streams. > > In drm_dp_update_payload

Re: [PATCH 29/31] drm/i915/gem: Roll all of context creation together

2021-06-16 Thread kernel test robot
Hi Jason, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-intel/for-linux-next] [also build test WARNING on drm-tip/drm-tip drm-exynos/exynos-drm-next next-20210616] [cannot apply to tegra-drm/drm/tegra/for-next drm/drm-next v5.13-rc6] [If your patch is

Re: [PATCH 02/10] driver core: Better distinguish probe errors in really_probe

2021-06-16 Thread Kirti Wankhede
On 6/15/2021 7:05 PM, Christoph Hellwig wrote: really_probe tries to special case errors from ->probe, but due to all other initialization added to the function over time now a lot of internal errors hit that code path as well. Untangle that by adding a new probe_err local variable and apply

  1   2   >