Re: [PATCH] drm: add kms driver for loongson display controller

2023-02-02 Thread kernel test robot
'--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/suijingfeng/drm-add-kms-driver-for-loongson-display-controller/20230202-011138 base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next patch link

Re: [PATCH 1/3] drm/mediatek: Refactor pixel format logic

2023-02-02 Thread Matthias Brugger
On 02/02/2023 21:41, Justin Green wrote: Yes, I had a comment on the naming in that patch. Never the less, I think if we don't need to "overwrite" the value, we should use just one struct for the values instead of copying them to the different .c files and give them SoC specific names. I

Re: [PATCH 02/22] usb: remove the dead USB_OHCI_SH option

2023-02-02 Thread Greg Kroah-Hartman
On Fri, Feb 03, 2023 at 08:15:42AM +0100, Christoph Hellwig wrote: > On Fri, Jan 13, 2023 at 08:12:44AM +0100, Greg Kroah-Hartman wrote: > > Do you want all of these to go through a single tree, or can they go > > through the different driver subsystem trees? > > Looks like the big removal isn't

Re: [PATCH v2 3/4] dt-bindings: panel: Introduce dual-link LVDS panel

2023-02-02 Thread Aradhya Bhatia
Hi Rob, On 30-Jan-23 22:34, Rob Herring wrote: > On Tue, Jan 24, 2023 at 03:42:37PM +0530, Aradhya Bhatia wrote: >> Dual-link LVDS interfaces have 2 links, with even pixels traveling on >> one link, and odd pixels on the other. These panels are also generic in >> nature, with no documented

[git pull] drm fixes for 6.2-rc7

2023-02-02 Thread Dave Airlie
Hi Linus, A few more fixes this week, a bit more spread out though. We have a bunch of nouveau regression and stabilisation fixes, along with usual amdgpu, and i915. Otherwise just some minor misc ones. Dave. drm-fixes-2023-02-03: drm fixes for 6.2-rc7 dma-fence: - fix signaling bit for

linux-next: build warning after merge of the drm tree

2023-02-02 Thread Stephen Rothwell
Hi all, After merging the drm tree, today's linux-next build (htmldocs) produced this warning: Documentation/gpu/i915:64: drivers/gpu/drm/i915/gt/intel_workarounds.c:32: WARNING: Inline emphasis start-string without end-string. Documentation/gpu/i915:64:

[PATCH 3/3] drm/connector: Deprecate split for BT.2020 in drm_colorspace enum

2023-02-02 Thread Joshua Ashton
Userspace has no way of controlling or knowing the pixel encoding currently, so there is no way for it to ever get the right values here. When we do add pixel_encoding control from userspace,we can pick the right value for the colorimetry packet based on the pixel_encoding + the colorspace.

[PATCH 2/3] drm/connector: Add enum documentation to drm_colorspace

2023-02-02 Thread Joshua Ashton
To match the other enums, and add more information about these values. Signed-off-by: Joshua Ashton Cc: Pekka Paalanen Cc: Sebastian Wick Cc: vitaly.pros...@amd.com Cc: Uma Shankar Cc: Ville Syrjälä Cc: Joshua Ashton Cc: dri-devel@lists.freedesktop.org Cc: amd-...@lists.freedesktop.org ---

[PATCH 1/3] drm/connector: Convert DRM_MODE_COLORIMETRY to enum

2023-02-02 Thread Joshua Ashton
From: Harry Wentland This allows us to use strongly typed arguments. Signed-off-by: Harry Wentland Reviewed-by: Simon Ser Cc: Pekka Paalanen Cc: Sebastian Wick Cc: vitaly.pros...@amd.com Cc: Uma Shankar Cc: Ville Syrjälä Cc: Joshua Ashton Cc: dri-devel@lists.freedesktop.org Cc:

[PATCH][next] drm/vmwgfx: Replace one-element array with flexible-array member

2023-02-02 Thread Gustavo A. R. Silva
One-element arrays are deprecated, and we are replacing them with flexible array members instead. So, replace one-element array with flexible-array member in struct vmw_view. This helps with the ongoing efforts to tighten the FORTIFY_SOURCE routines on memcpy() and help us make progress towards

Re: [PATCH 1/2] drm/i915: Fix GEN8_MISCCPCTL

2023-02-02 Thread Matt Roper
On Thu, Feb 02, 2023 at 04:57:08PM -0800, Lucas De Marchi wrote: > Register 0x9424 is not replicated on any platform, so it shouldn't be > declared with REG_MCR(). Declaring it with _MMIO() is basically > duplicate of the GEN7 version, so just remove the GEN8 and change all > the callers to use

[PATCH 2/3] drm/i915/guc: Clean up of register capture search

2023-02-02 Thread John . C . Harrison
From: John Harrison The comparison in the search for a matching register capture node was not the most readable. So remove two redundant terms and re-format to keep each term on a single line, and only one term per line. Signed-off-by: John Harrison ---

[PATCH 0/3] More error capture improvements

2023-02-02 Thread John . C . Harrison
From: John Harrison Ecodes got lost with the switch to GuC based register lists. Put them back. Seqno values got lost with the switch to per context timelines. Put hose back too. Signed-off-by: John Harrison John Harrison (3): drm/i915/guc: Fix missing ecodes drm/i915/guc: Clean up of

[PATCH 1/3] drm/i915/guc: Fix missing ecodes

2023-02-02 Thread John . C . Harrison
From: John Harrison Error captures are tagged with an 'ecode'. This is a pseduo-unique magic number that is meant to distinguish similar seeming bugs with different underlying signatures. It is a combination of two ring state registers. Unfortunately, the register state being used is only valid

[PATCH 3/3] drm/i915: Include timelines in error capture

2023-02-02 Thread John . C . Harrison
From: John Harrison The seqno value actually written out to memory is no longer in the regular HWSP and therefore no longer visible in an error capture. Instead, it is now in its own private timeline buffer. So include that buffer in the capture too. Signed-off-by: John Harrison ---

[PATCH 2/2] drm/i915: Remove unused/wrong INF_UNIT_LEVEL_CLKGATE

2023-02-02 Thread Lucas De Marchi
INF_UNIT_LEVEL_CLKGATE is not replicated, but since it's not actually used it can just be removed. Signed-off-by: Lucas De Marchi --- drivers/gpu/drm/i915/gt/intel_gt_regs.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/intel_gt_regs.h

[PATCH 1/2] drm/i915: Fix GEN8_MISCCPCTL

2023-02-02 Thread Lucas De Marchi
Register 0x9424 is not replicated on any platform, so it shouldn't be declared with REG_MCR(). Declaring it with _MMIO() is basically duplicate of the GEN7 version, so just remove the GEN8 and change all the callers to use the right functions. Also use intel_uncore_rmw() rather than a read +

Re: [PATCH] drm/bridge: panel: Set orientation on panel_bridge connector

2023-02-02 Thread Doug Anderson
Hi, On Mon, Jan 23, 2023 at 8:05 AM Laurent Pinchart wrote: > > Hi John, > > On Mon, Jan 23, 2023 at 12:16:45PM +, John Keeping wrote: > > On Sun, Jan 22, 2023 at 05:01:27PM +0200, Laurent Pinchart wrote: > > > On Sat, Jan 21, 2023 at 05:58:11PM +, John Keeping wrote: > > > > On Sat, Jan

[PATCH 2/6] drm/i915/guc: More debug print updates - GSC firmware

2023-02-02 Thread John . C . Harrison
From: John Harrison Update a bunch more debug prints to use the new GT based scheme. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_gsc_fw.c | 8 +++- drivers/gpu/drm/i915/gt/uc/intel_gsc_uc.c | 7 +++ 2 files changed, 6 insertions(+), 9 deletions(-) diff --git

[PATCH 1/6] drm/i915/guc: More debug print updates - UC firmware

2023-02-02 Thread John . C . Harrison
From: John Harrison Update a bunch more debug prints to use the new GT based scheme. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_uc.c| 42 drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 116 +++ 2 files changed, 73 insertions(+), 85

[PATCH 4/6] drm/i915/guc: More debug print updates - GuC selftests

2023-02-02 Thread John . C . Harrison
From: John Harrison Update a bunch more debug prints to use the new GT based scheme. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/selftest_guc.c | 35 ++- .../drm/i915/gt/uc/selftest_guc_hangcheck.c | 23 ++--

[PATCH 3/6] drm/i915/guc: More debug print updates - GuC reg capture

2023-02-02 Thread John . C . Harrison
From: John Harrison Update a bunch more debug prints to use the new GT based scheme. Signed-off-by: John Harrison --- .../gpu/drm/i915/gt/uc/intel_guc_capture.c| 51 --- 1 file changed, 21 insertions(+), 30 deletions(-) diff --git

[PATCH 6/6] drm/i915/guc: More debug print updates - GuC logging

2023-02-02 Thread John . C . Harrison
From: John Harrison Update a bunch more debug prints to use the new GT based scheme. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/intel_gt_print.h | 3 +++ drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 3 +-- drivers/gpu/drm/i915/gt/uc/intel_guc_print.h | 3 +++ 3 files

[PATCH 5/6] drm/i915/guc: More debug print updates - GuC SLPC

2023-02-02 Thread John . C . Harrison
From: John Harrison Update a bunch more debug prints to use the new GT based scheme. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_guc_rc.c | 8 +-- drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 60 - 2 files changed, 26 insertions(+), 42

[PATCH 0/6] More drm_dbg to guc_dbg changes

2023-02-02 Thread John . C . Harrison
From: John Harrison Update more print messages to the new scheme. Signed-off-by: John Harrison John Harrison (6): drm/i915/guc: More debug print updates - UC firmware drm/i915/guc: More debug print updates - GSC firmware drm/i915/guc: More debug print updates - GuC reg capture

Re: [PATCH] drm/i915/huc: Add and use HuC oriented print macros

2023-02-02 Thread Ceraolo Spurio, Daniele
On 1/31/2023 2:28 PM, Michal Wajdeczko wrote: Like we did it for GuC, introduce some helper print macros for HuC to have unified format of messages that also include GT#. While around improve some messages and use %pe if possible. Signed-off-by: Michal Wajdeczko Cc: John Harrison ---

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-02-02 Thread T.J. Mercier
On Wed, Feb 1, 2023 at 6:52 AM Tvrtko Ursulin wrote: > > > On 01/02/2023 14:23, Tvrtko Ursulin wrote: > > > > On 01/02/2023 01:49, T.J. Mercier wrote: > >> On Tue, Jan 31, 2023 at 6:01 AM Tvrtko Ursulin > >> wrote: > >>> > >>> > >>> On 25/01/2023 20:04, T.J. Mercier wrote: > On Wed, Jan 25,

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-02-02 Thread T.J. Mercier
On Wed, Feb 1, 2023 at 6:23 AM Tvrtko Ursulin wrote: > > > On 01/02/2023 01:49, T.J. Mercier wrote: > > On Tue, Jan 31, 2023 at 6:01 AM Tvrtko Ursulin > > wrote: > >> > >> > >> On 25/01/2023 20:04, T.J. Mercier wrote: > >>> On Wed, Jan 25, 2023 at 9:31 AM Tvrtko Ursulin > >>> wrote: > >

Re: [PATCH v2] dt-bindings: display: bridge: sil, sii8620: convert to dtschema

2023-02-02 Thread Rob Herring
On Sun, 29 Jan 2023 17:05:37 +0100, Krzysztof Kozlowski wrote: > Convert the Silicon Image SiI8620 HDMI/MHL bridge bindings to DT schema. > > Signed-off-by: Krzysztof Kozlowski > > --- > > Changes since v1: > 1. Require also port@1 (Laurent) > --- >

Re: [Freedreno] [RFT PATCH v2 3/3] drm/msm/dsi: More properly handle errors in regards to dsi_mgr_bridge_power_on()

2023-02-02 Thread Doug Anderson
Hi, On Thu, Feb 2, 2023 at 2:37 PM Abhinav Kumar wrote: > > Hi Doug > > On 1/31/2023 2:18 PM, Douglas Anderson wrote: > > In commit 7d8e9a90509f ("drm/msm/dsi: move DSI host powerup to modeset > > time") the error handling with regards to dsi_mgr_bridge_power_on() > > got a bit worse.

Re: [Freedreno] [RFT PATCH v2 3/3] drm/msm/dsi: More properly handle errors in regards to dsi_mgr_bridge_power_on()

2023-02-02 Thread Abhinav Kumar
Hi Doug On 1/31/2023 2:18 PM, Douglas Anderson wrote: In commit 7d8e9a90509f ("drm/msm/dsi: move DSI host powerup to modeset time") the error handling with regards to dsi_mgr_bridge_power_on() got a bit worse. Specifically if we failed to power the bridge on then nothing would really notice.

Re: [PATCH 1/3] drm/mediatek: Refactor pixel format logic

2023-02-02 Thread Justin Green
> Yes, I had a comment on the naming in that patch. Never the less, I think if > we > don't need to "overwrite" the value, we should use just one struct for the > values instead of copying them to the different .c files and give them SoC > specific names. I don't have a very strong opinion about

Re: [PATCH 1/3] drm/mediatek: Refactor pixel format logic

2023-02-02 Thread Matthias Brugger
On 02/02/2023 19:59, Justin Green wrote: Hi Matthias, mt8173_formats are the same as the old struct formats. Maybe we should use that and only overwrite where we actually use a different array. I think this was sort of how the original patch worked, but we wanted to add some flexibility to

Re: [PATCH v2 12/27] drm/msm/dpu: remove dpu_hw_fmt_layout from struct dpu_hw_pipe_cfg

2023-02-02 Thread Abhinav Kumar
On 2/2/2023 12:10 PM, Dmitry Baryshkov wrote: On 02/02/2023 21:54, Abhinav Kumar wrote: On 2/2/2023 11:45 AM, Dmitry Baryshkov wrote: On Thu, 2 Feb 2023 at 21:38, Abhinav Kumar wrote: On 12/29/2022 11:18 AM, Dmitry Baryshkov wrote: Remove dpu_hw_fmt_layout instance from struct

Re: [PATCH v2 12/27] drm/msm/dpu: remove dpu_hw_fmt_layout from struct dpu_hw_pipe_cfg

2023-02-02 Thread Dmitry Baryshkov
On 02/02/2023 21:54, Abhinav Kumar wrote: On 2/2/2023 11:45 AM, Dmitry Baryshkov wrote: On Thu, 2 Feb 2023 at 21:38, Abhinav Kumar wrote: On 12/29/2022 11:18 AM, Dmitry Baryshkov wrote: Remove dpu_hw_fmt_layout instance from struct dpu_hw_pipe_cfg, leaving only src_rect and dst_rect.

Re: [RFT PATCH v2 2/3] drm/msm/dsi: Stop unconditionally powering up DSI hosts at modeset

2023-02-02 Thread Abhinav Kumar
On 2/1/2023 6:33 AM, Doug Anderson wrote: Hi, On Tue, Jan 31, 2023 at 3:32 PM Abhinav Kumar wrote: On 1/31/2023 2:18 PM, Douglas Anderson wrote: In commit 7d8e9a90509f ("drm/msm/dsi: move DSI host powerup to modeset time"), we moved powering up DSI hosts to modeset time. This wasn't

Re: [RFC 10/12] cgroup/drm: Introduce weight based drm cgroup control

2023-02-02 Thread Tejun Heo
Hello, On Thu, Feb 02, 2023 at 02:26:06PM +, Tvrtko Ursulin wrote: > When you say active/inactive - to what you are referring in the cgroup > world? Offline/online? For those my understanding was offline was a > temporary state while css is getting destroyed. Oh, it's just based on activity.

Re: [PATCH v2 12/27] drm/msm/dpu: remove dpu_hw_fmt_layout from struct dpu_hw_pipe_cfg

2023-02-02 Thread Abhinav Kumar
On 2/2/2023 11:45 AM, Dmitry Baryshkov wrote: On Thu, 2 Feb 2023 at 21:38, Abhinav Kumar wrote: On 12/29/2022 11:18 AM, Dmitry Baryshkov wrote: Remove dpu_hw_fmt_layout instance from struct dpu_hw_pipe_cfg, leaving only src_rect and dst_rect. This way right and left pipes will have

Re: [PATCH v2 12/27] drm/msm/dpu: remove dpu_hw_fmt_layout from struct dpu_hw_pipe_cfg

2023-02-02 Thread Dmitry Baryshkov
On Thu, 2 Feb 2023 at 21:38, Abhinav Kumar wrote: > > > > On 12/29/2022 11:18 AM, Dmitry Baryshkov wrote: > > Remove dpu_hw_fmt_layout instance from struct dpu_hw_pipe_cfg, leaving > > only src_rect and dst_rect. This way right and left pipes will have > > separate dpu_hw_pipe_cfg isntances,

Re: [PATCH v2 12/27] drm/msm/dpu: remove dpu_hw_fmt_layout from struct dpu_hw_pipe_cfg

2023-02-02 Thread Abhinav Kumar
On 12/29/2022 11:18 AM, Dmitry Baryshkov wrote: Remove dpu_hw_fmt_layout instance from struct dpu_hw_pipe_cfg, leaving only src_rect and dst_rect. This way right and left pipes will have separate dpu_hw_pipe_cfg isntances, while the layout is common to both of them. Sorry for not

Re: [PATCH v2 11/27] drm/msm/dpu: move stride programming to dpu_hw_sspp_setup_sourceaddress

2023-02-02 Thread Abhinav Kumar
On 2/2/2023 10:55 AM, Dmitry Baryshkov wrote: Hi Abhinav, On Thu, 2 Feb 2023 at 20:41, Abhinav Kumar wrote: On 12/29/2022 11:18 AM, Dmitry Baryshkov wrote: Move stride programming to dpu_hw_sspp_setup_sourceaddress(), so that dpu_hw_sspp_setup_rects() programs only source and

Re: [PATCH 1/3] drm/mediatek: Refactor pixel format logic

2023-02-02 Thread Justin Green
Hi Matthias, > mt8173_formats are the same as the old struct formats. Maybe we should use > that > and only overwrite where we actually use a different array. I think this was sort of how the original patch worked, but we wanted to add some flexibility to allow different components to support

Re: [PATCH v2 11/27] drm/msm/dpu: move stride programming to dpu_hw_sspp_setup_sourceaddress

2023-02-02 Thread Dmitry Baryshkov
Hi Abhinav, On Thu, 2 Feb 2023 at 20:41, Abhinav Kumar wrote: > > > > On 12/29/2022 11:18 AM, Dmitry Baryshkov wrote: > > Move stride programming to dpu_hw_sspp_setup_sourceaddress(), so that > > dpu_hw_sspp_setup_rects() programs only source and destination > > rectangles. > > > >

[PATCH] drm/msm/a6xx: Make GPU destroy a bit safer

2023-02-02 Thread Douglas Anderson
If, for whatever reason, we're trying process adreno_runtime_resume() at the same time that a6xx_destroy() is running then things can go boom. Specifically adreno_runtime_resume() will eventually call a6xx_pm_resume() and that may try to resume the gmu. Let's grab the GMU lock as we're destroying

Re: [PATCH v2 11/27] drm/msm/dpu: move stride programming to dpu_hw_sspp_setup_sourceaddress

2023-02-02 Thread Abhinav Kumar
On 12/29/2022 11:18 AM, Dmitry Baryshkov wrote: Move stride programming to dpu_hw_sspp_setup_sourceaddress(), so that dpu_hw_sspp_setup_rects() programs only source and destination rectangles. Signed-off-by: Dmitry Baryshkov Sorry but once again, I dont see a response to my comment

Re: [Nouveau] [PATCH drm-next 05/14] drm/nouveau: new VM_BIND uapi interfaces

2023-02-02 Thread Danilo Krummrich
On 2/2/23 12:53, Christian König wrote: Am 01.02.23 um 09:10 schrieb Dave Airlie: [SNIP] For drivers that don't intend to merge at all and (somehow) are capable of dealing with sparse regions without knowing the sparse region's boundaries, it'd be easy to make those gpuva_regions optional.

[PATCH] drm/i915: Make sure dsm_size has correct granularity

2023-02-02 Thread Nirmoy Das
DSM granularity is 1MB so make sure we stick to that. v2: replace "1 * SZ_1M" with SZ_1M (Andrzej). Cc: Matthew Auld Suggested-by: Lucas De Marchi Signed-off-by: Nirmoy Das Reviewed-by: Andrzej Hajda --- drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [Intel-gfx] [PATCH] drm/i915/pxp: limit drm-errors or warnings on firmware API failures

2023-02-02 Thread Teres Alexis, Alan Previn
On Thu, 2023-02-02 at 08:43 +, Tvrtko Ursulin wrote: > > On 02/02/2023 08:13, Alan Previn wrote: > > MESA driver is creating protected context on every driver handle > > initialization to query caps bit for app. So when running CI tests, > > they are observing hundreds of drm_errors when

Re: [PATCH v10 23/23] drm/i915/vm_bind: Support capture of persistent mappings

2023-02-02 Thread Andi Shyti
Hi Niranjana, On Tue, Jan 17, 2023 at 11:16:09PM -0800, Niranjana Vishwanathapura wrote: > Support dump capture of persistent mappings upon user request. > > Capture of a mapping is requested with the VM_BIND ioctl and > processed during the GPU error handling. They are synchronously > unbound

Re: [PATCH v3] drm/amdgpu/fence: Fix oops due to non-matching drm_sched init/fini

2023-02-02 Thread Guilherme G. Piccoli
On 02/02/2023 13:12, Luben Tuikov wrote: > Hi Guilherme, > > Thanks for redoing to a v3. This patch is: > > Reviewed-by: Luben Tuikov > > Regards, > Luben > Thank you for the reviews Luben, much appreciated!

Re: [RESEND PATCH v11 02/18] drm: bridge: panel: Add devm_drm_of_dsi_get_bridge helper

2023-02-02 Thread Jagan Teki
Hi Maxime, On Mon, Jan 30, 2023 at 6:28 PM Maxime Ripard wrote: > > On Thu, Jan 26, 2023 at 08:48:48PM +0530, Jagan Teki wrote: > > On Thu, Jan 26, 2023 at 5:42 PM Maxime Ripard wrote: > > > > > > Hi, > > > > > > On Mon, Jan 23, 2023 at 08:41:56PM +0530, Jagan Teki wrote: > > > > Add devm OF

Re: [PATCH v10 22/23] drm/i915/vm_bind: Properly build persistent map sg table

2023-02-02 Thread Andi Shyti
Hi Niranjana, On Tue, Jan 17, 2023 at 11:16:08PM -0800, Niranjana Vishwanathapura wrote: > Properly build the sg table for persistent mapping which can > be partial map of the underlying object. Ensure the sg pages > are properly set for page backed regions. The dump capture > support requires

Re: [PATCH v10 20/23] drm/i915/vm_bind: Render VM_BIND documentation

2023-02-02 Thread Andi Shyti
Hi Niranjana, On Tue, Jan 17, 2023 at 11:16:06PM -0800, Niranjana Vishwanathapura wrote: > Update i915 documentation to include VM_BIND changes > and render all VM_BIND related documentation. > > Reviewed-by: Matthew Auld > Signed-off-by: Niranjana Vishwanathapura looks good! Reviewed-by:

[PATCH] drm/i915/hwmon: Enable PL1 power limit

2023-02-02 Thread Ashutosh Dixit
Previous documentation suggested that PL1 power limit is always enabled. However we now find this not to be the case on some platforms (such as ATSM). Therefore enable PL1 power limit during hwmon initialization. Bspec: 51864 v2: Add Bspec reference (Gwan-gyeong) Signed-off-by: Ashutosh Dixit

Re: [PATCH v10 18/23] drm/i915/vm_bind: Limit vm_bind mode to non-recoverable contexts

2023-02-02 Thread Andi Shyti
Hi Niranjana, On Tue, Jan 17, 2023 at 11:16:04PM -0800, Niranjana Vishwanathapura wrote: > Only support vm_bind mode with non-recoverable contexts. > With new vm_bind mode with eb3 submission path, we need not > support older recoverable contexts. > > Reviewed-by: Matthew Auld > Signed-off-by:

Re: [PATCH v3] drm/amdgpu/fence: Fix oops due to non-matching drm_sched init/fini

2023-02-02 Thread Luben Tuikov
Hi Guilherme, Thanks for redoing to a v3. This patch is: Reviewed-by: Luben Tuikov Regards, Luben On 2023-02-02 08:48, Guilherme G. Piccoli wrote: > Currently amdgpu calls drm_sched_fini() from the fence driver sw fini > routine - such function is expected to be called only after the >

Re: [PATCH v10 12/23] drm/i915/vm_bind: Use common execbuf functions in execbuf path

2023-02-02 Thread Andi Shyti
Hi Niranjana, On Tue, Jan 17, 2023 at 11:15:58PM -0800, Niranjana Vishwanathapura wrote: > Update the execbuf path to use common execbuf functions to > reduce code duplication with the newer execbuf3 path. > > Reviewed-by: Matthew Auld > Signed-off-by: Niranjana Vishwanathapura Reviewed-by:

[linux-next:master] BUILD REGRESSION ea4dabbb4ad7eb52632a2ca0b8f89f0ea7c55dcf

2023-02-02 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: ea4dabbb4ad7eb52632a2ca0b8f89f0ea7c55dcf Add linux-next specific files for 20230202 Error/Warning reports: https://lore.kernel.org/oe-kbuild-all/202301301801.y5o08tqx-...@intel.com https

Re: [PATCH 4/4] accel/ivpu: Fix old dma_buf api usage

2023-02-02 Thread Jeffrey Hugo
On 2/2/2023 2:21 AM, Stanislaw Gruszka wrote: Update according to new dma-buf locking scheme. Remove redundant WARN_ON()'s, dma_buf functions internally have the same warnings already. Signed-off-by: Stanislaw Gruszka Reviewed-by: Jeffrey Hugo

Re: [PATCH 3/4] accel/ivpu: Set dma max_segment_size

2023-02-02 Thread Jeffrey Hugo
On 2/2/2023 2:21 AM, Stanislaw Gruszka wrote: Avoid below spurious warning: [ 264.844029] DMA-API: intel_vpu :00:0b.0: mapping sg segment longer than device claims to support [len=143360] [max=65536] [ 264.844038] WARNING: CPU: 0 PID: 1254 at kernel/dma/debug.c:1160

Re: [PATCH 2/4] accel/ivpu: Send VPU_JSM_MSG_CONTEXT_DELETE when deleting context

2023-02-02 Thread Jeffrey Hugo
On 2/2/2023 2:21 AM, Stanislaw Gruszka wrote: From: Andrzej Kacprowski The VPU_JSM_MSG_CONTEXT_DELETE will remove any resources associated with the SSID, that included any blobs create by the user space application. The command can also remove doorbell registrations, but since this does not

Re: [PATCH 1/4] accel/ivpu: Fix FW API data alignment issues

2023-02-02 Thread Jeffrey Hugo
On 2/2/2023 2:21 AM, Stanislaw Gruszka wrote: From: Andrzej Kacprowski FW API structures have been updated to fix misaligned structure members. Also changed JSM message header format to account for future improvements. Added explicit check for minimum supported JSM API version.

RE: [PULL] drm-misc-next

2023-02-02 Thread Deucher, Alexander
[Public] > -Original Message- > From: dim-tools On Behalf Of > John Paul Adrian Glaubitz > Sent: Monday, January 23, 2023 10:01 AM > To: tzimmerm...@suse.de > Cc: tvrtko.ursu...@linux.intel.com; dim-to...@lists.freedesktop.org; > daniel.vet...@ffwll.ch; intel-...@lists.freedesktop.org;

Re: [PATCH] drm/vgem: add missing mutex_destroy

2023-02-02 Thread Stanislaw Gruszka
On Thu, Feb 02, 2023 at 09:55:17AM -0300, Maíra Canal wrote: > vgem_fence_open() instantiates a mutex for a particular fence > instance, but never destroys it by calling mutex_destroy() in > vgem_fence_close(). > > So, add the missing mutex_destroy() to guarantee proper resource > destruction. >

Re: [RFC 10/12] cgroup/drm: Introduce weight based drm cgroup control

2023-02-02 Thread Tvrtko Ursulin
On 28/01/2023 01:11, Tejun Heo wrote: On Thu, Jan 12, 2023 at 04:56:07PM +, Tvrtko Ursulin wrote: ... + /* +* 1st pass - reset working values and update hierarchical weights and +* GPU utilisation. +*/ + if (!__start_scanning(root, period_us)) +

[PATCH] i915: fix memory leak with using debugfs_lookup()

2023-02-02 Thread Greg Kroah-Hartman
When calling debugfs_lookup() the result must have dput() called on it, otherwise the memory will leak over time. To make things simpler, just call debugfs_lookup_and_remove() instead which handles all of the logic at once. Cc: Zhenyu Wang Cc: Zhi Wang Cc: Jani Nikula Cc: Joonas Lahtinen Cc:

Re: [v1 3/3] drm/msm/disp/dpu1: reserve the resources on topology change

2023-02-02 Thread kernel test robot
-randconfig-r034-20230129 (https://download.01.org/0day-ci/archive/20230202/202302022254.37xyfgnr-...@intel.com/config) compiler: aarch64-linux-gcc (GCC) 12.1.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross

Re: [PATCH v2] drm/amdgpu/fence: Fix oops due to non-matching drm_sched init/fini

2023-02-02 Thread Guilherme G. Piccoli
On 02/02/2023 08:58, Christian König wrote: > [...] >> +if (!ring->no_scheduler && ring->sched.ops) >> drm_sched_fini(>sched); > > I think we should drop the check for no_scheduler here and just call > drm_sched_fini() when the scheduler instance was initialized

[PATCH v3] drm/amdgpu/fence: Fix oops due to non-matching drm_sched init/fini

2023-02-02 Thread Guilherme G. Piccoli
Currently amdgpu calls drm_sched_fini() from the fence driver sw fini routine - such function is expected to be called only after the respective init function - drm_sched_init() - was executed successfully. Happens that we faced a driver probe failure in the Steam Deck recently, and the function

[PULL] drm-intel-fixes

2023-02-02 Thread Rodrigo Vivi
Hi Dave and Daniel, Here goes this week's fixes with couple targeting stable. drm-intel-fixes-2023-02-02: - Fixes for potential use-after-free and double-free (Rob) - GuC locking and refcount fixes (John) - Display's reference clock value fix (Chaitanya) Thanks, Rodrigo. The following changes

Re: [PATCH] drm/client: Convert to VISIBLE_IF_KUNIT

2023-02-02 Thread kernel test robot
Hi Maxime, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-misc/drm-misc-next] [also build test WARNING on drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-tip/drm-tip linus/master v6.2-rc6 next-20230202] [If your patch is applied to the wrong git

Re: [PATCH] drm/client: Convert to VISIBLE_IF_KUNIT

2023-02-02 Thread Thomas Zimmermann
Hi Am 02.02.23 um 13:35 schrieb Maxime Ripard: Hi, On Thu, Feb 02, 2023 at 01:22:01PM +0100, Thomas Zimmermann wrote: Am 02.02.23 um 12:03 schrieb Maxime Ripard: Commit 8fc0380f6ba7 ("drm/client: Add some tests for drm_connector_pick_cmdline_mode()") was meant to introduce unit tests for the

[PATCH] drm/vgem: add missing mutex_destroy

2023-02-02 Thread Maíra Canal
vgem_fence_open() instantiates a mutex for a particular fence instance, but never destroys it by calling mutex_destroy() in vgem_fence_close(). So, add the missing mutex_destroy() to guarantee proper resource destruction. Signed-off-by: Maíra Canal --- drivers/gpu/drm/vgem/vgem_fence.c | 1 +

[PATCH][next] i915/gvt: Fix spelling mistake "vender" -> "vendor"

2023-02-02 Thread Colin Ian King
There is a spelling mistake in a literal string. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/gvt/firmware.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/gvt/firmware.c b/drivers/gpu/drm/i915/gvt/firmware.c index

Re: [PATCH] drm/vc4: allow DRM_VC4_KUNIT_TEST to be a loadable module

2023-02-02 Thread Maxime Ripard
On Thu, 02 Feb 2023 11:23:32 +0100, Arnd Bergmann wrote: > In configurations with CONFIG_KUNIT=m, builting the unit test > into the kernel causes a link failure: > > arm-linux-gnueabi-ld: drivers/gpu/drm/vc4/tests/vc4_mock.o: in function > `__build_mock': > vc4_mock.c:(.text+0x6e): undefined

Re: [PATCH] drm/client: Convert to VISIBLE_IF_KUNIT

2023-02-02 Thread Maxime Ripard
On Thu, Feb 02, 2023 at 08:31:27AM -0300, Maíra Canal wrote: > Hi Maxime, > > On 2/2/23 08:03, Maxime Ripard wrote: > > Commit 8fc0380f6ba7 ("drm/client: Add some tests for > > drm_connector_pick_cmdline_mode()") was meant to introduce unit tests > > for the static

Re: [PATCH] drm/client: Convert to VISIBLE_IF_KUNIT

2023-02-02 Thread Maxime Ripard
Hi, On Thu, Feb 02, 2023 at 01:22:01PM +0100, Thomas Zimmermann wrote: > Am 02.02.23 um 12:03 schrieb Maxime Ripard: > > Commit 8fc0380f6ba7 ("drm/client: Add some tests for > > drm_connector_pick_cmdline_mode()") was meant to introduce unit tests > > for the static

Re: [PATCH v2 11/17] drm/display/dp_mst: Add helpers to query for payload allocation errors

2023-02-02 Thread Imre Deak
drm-tip drm-tip > patch link: > https://lore.kernel.org/r/20230131150548.1614458-12-imre.deak%40intel.com > patch subject: [PATCH v2 11/17] drm/display/dp_mst: Add helpers to query for > payload allocation errors > config: x86_64-randconfig-m001 > (https://download.01.org/0day

Re: [PATCH] drm/client: Convert to VISIBLE_IF_KUNIT

2023-02-02 Thread Thomas Zimmermann
Hi Am 02.02.23 um 12:03 schrieb Maxime Ripard: Commit 8fc0380f6ba7 ("drm/client: Add some tests for drm_connector_pick_cmdline_mode()") was meant to introduce unit tests for the static drm_connector_pick_cmdline_mode() function. In such a case, the kunit documentation recommended to import the

Re: [PATCH v2 11/17] drm/display/dp_mst: Add helpers to query for payload allocation errors

2023-02-02 Thread Dan Carpenter
%40intel.com patch subject: [PATCH v2 11/17] drm/display/dp_mst: Add helpers to query for payload allocation errors config: x86_64-randconfig-m001 (https://download.01.org/0day-ci/archive/20230202/202302021855.yyqieq2o-...@intel.com/config) compiler: gcc-11 (Debian 11.3.0-8) 11.3.0 If you fix the issue

Re: [PATCH v2] drm/amdgpu/fence: Fix oops due to non-matching drm_sched init/fini

2023-02-02 Thread Christian König
Am 01.02.23 um 17:48 schrieb Guilherme G. Piccoli: Currently amdgpu calls drm_sched_fini() from the fence driver sw fini routine - such function is expected to be called only after the respective init function - drm_sched_init() - was executed successfully. Happens that we faced a driver probe

Re: [Nouveau] [PATCH drm-next 05/14] drm/nouveau: new VM_BIND uapi interfaces

2023-02-02 Thread Christian König
Am 01.02.23 um 09:10 schrieb Dave Airlie: [SNIP] For drivers that don't intend to merge at all and (somehow) are capable of dealing with sparse regions without knowing the sparse region's boundaries, it'd be easy to make those gpuva_regions optional. Yeah, but this then defeats the approach of

[PATCH v2 2/2] drm/vc4: replace obj lookup steps with drm_gem_objects_lookup

2023-02-02 Thread Maíra Canal
As vc4_cl_lookup_bos() performs the same steps as drm_gem_objects_lookup(), replace the open-coded implementation in vc4 to simply use the DRM function. Signed-off-by: Maíra Canal Reviewed-by: André Almeida --- drivers/gpu/drm/vc4/vc4_gem.c | 43 ++- 1 file

[PATCH v2 1/2] drm/vc4: replace drm_gem_dma_object for drm_gem_object in vc4_exec_info

2023-02-02 Thread Maíra Canal
The array of BOs that are lookup at the start of exec doesn't need to be instantiated as drm_gem_dma_object, as it doesn't benefit from its attributes. So, simplify the code by replacing the array of drm_gem_dma_object for an array of drm_gem_object in the struct vc4_exec_info. Suggested-by:

[PATCH v2 0/2] drm/vc4: Improve drm_gem_object handling

2023-02-02 Thread Maíra Canal
Currently, the array of BOs that are lookup up at the start of exec is being instantiated as drm_gem_dma_object, which is not needed and makes it difficult to use the drm_gem_objects_lookup() helper. Therefore, replace drm_gem_dma_object for drm_gem_object and then replace obj lookup steps with

Re: [PATCH] drm/client: Convert to VISIBLE_IF_KUNIT

2023-02-02 Thread Maíra Canal
Hi Maxime, On 2/2/23 08:03, Maxime Ripard wrote: Commit 8fc0380f6ba7 ("drm/client: Add some tests for drm_connector_pick_cmdline_mode()") was meant to introduce unit tests for the static drm_connector_pick_cmdline_mode() function. In such a case, the kunit documentation recommended to import

Re: [PATCH] drm/format-helper: Use KUNIT_EXPECT_MEMEQ macro

2023-02-02 Thread Maíra Canal
On 2/1/23 04:10, Thomas Zimmermann wrote: Am 30.01.23 um 13:55 schrieb Maíra Canal: Commit b8a926bea8b1 ("kunit: Introduce KUNIT_EXPECT_MEMEQ and KUNIT_EXPECT_MEMNEQ macros") introduced a new macro to compare blocks of memory and, if the test fails, print the result in a human-friendly

[PATCH] drm/client: Convert to VISIBLE_IF_KUNIT

2023-02-02 Thread Maxime Ripard
Commit 8fc0380f6ba7 ("drm/client: Add some tests for drm_connector_pick_cmdline_mode()") was meant to introduce unit tests for the static drm_connector_pick_cmdline_mode() function. In such a case, the kunit documentation recommended to import the tests source file directly from the source file

Re: [Intel-gfx] [PATCH] drm/i915/pcode: Wait 10 seconds for pcode to settle

2023-02-02 Thread Gwan-gyeong Mun
On 2/2/23 11:28 AM, Andi Shyti wrote: Hi GG, On Thu, Feb 02, 2023 at 10:22:30AM +0200, Gwan-gyeong Mun wrote: Hi Andi, You gave a lot of explanations, and confirmed that this patch solves the problem, but the root cause of this problem still seems to be unclear. In the logs where this

[PATCH] drm/vc4: allow DRM_VC4_KUNIT_TEST to be a loadable module

2023-02-02 Thread Arnd Bergmann
From: Arnd Bergmann In configurations with CONFIG_KUNIT=m, builting the unit test into the kernel causes a link failure: arm-linux-gnueabi-ld: drivers/gpu/drm/vc4/tests/vc4_mock.o: in function `__build_mock': vc4_mock.c:(.text+0x6e): undefined reference to `kunit_do_failed_assertion'

Re: [PATCH 3/3] drm/mediatek: Enable AR30 and BA30 overlays on MT8195

2023-02-02 Thread Matthias Brugger
On 01/02/2023 18:02, Justin Green wrote: Tested using "modetest -P" on an MT8195 device. Signed-off-by: Justin Green --- drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 1/3] drm/mediatek: Refactor pixel format logic

2023-02-02 Thread Matthias Brugger
On 01/02/2023 18:02, Justin Green wrote: Add an DDP component interface for querying pixel format support and move list of supported pixel formats into DDP components instead of mtk_drm_plane.c Tested by running Chrome on an MT8195. Signed-off-by: Justin Green ---

Re: [EXT] Re: [PATCH v2 1/1] tee: new ioctl to a register tee_shm from a dmabuf file descriptor

2023-02-02 Thread Etienne Carriere
On Thu, 2 Feb 2023 at 09:35, Sumit Garg wrote: > > Hi Cyrille, > > Please don't top post as it makes it harder to follow-up. > > On Thu, 2 Feb 2023 at 13:26, Cyrille Fleury wrote: > > > > Hi Sumit, all > > > > Upstream OP-TEE should support registering a dmabuf since a while, given > > how

Re: [PATCH v2] drm/mediatek: dp: Only trigger DRM HPD events if bridge is attached

2023-02-02 Thread Matthias Brugger
On 02/02/2023 05:57, Chen-Yu Tsai wrote: The MediaTek DisplayPort interface bridge driver starts its interrupts as soon as its probed. However when the interrupts trigger the bridge might not have been attached to a DRM device. As drm_helper_hpd_irq_event() does not check whether the passed

Re: [Intel-gfx] [PATCH v3] drm/i915: Consolidate TLB invalidation flow

2023-02-02 Thread Andrzej Hajda
On 02.02.2023 09:33, Tvrtko Ursulin wrote: On 02/02/2023 07:43, Andrzej Hajda wrote: On 01.02.2023 17:51, Tvrtko Ursulin wrote: [snip] Btw - do you have any idea why the test is suppressed already?! CI told me BAT was a success... Except this patch, igt@i915_selftest@live@gt_tlb

Re: [PATCH v29 0/7] Add MediaTek SoC DRM (vdosys1) support for mt8195

2023-02-02 Thread AngeloGioacchino Del Regno
Il 27/12/22 09:10, Nancy.Lin ha scritto: The hardware path of vdosys1 with DPTx output need to go through by several modules, such as, OVL_ADAPTOR and MERGE. Add DRM and these modules support by the patches below: Hello Chun-Kuang, This series reached version 29 and was tested for a long

Re: [Intel-gfx] [PATCH] drm/i915/pcode: Wait 10 seconds for pcode to settle

2023-02-02 Thread Andi Shyti
Hi GG, On Thu, Feb 02, 2023 at 10:22:30AM +0200, Gwan-gyeong Mun wrote: > Hi Andi, > > You gave a lot of explanations, and confirmed that this patch solves the > problem, but the root cause of this problem still seems to be unclear. > > In the logs where this problem was reported, the logs were

[PATCH 4/4] accel/ivpu: Fix old dma_buf api usage

2023-02-02 Thread Stanislaw Gruszka
Update according to new dma-buf locking scheme. Remove redundant WARN_ON()'s, dma_buf functions internally have the same warnings already. Signed-off-by: Stanislaw Gruszka --- drivers/accel/ivpu/ivpu_gem.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git

[PATCH 2/4] accel/ivpu: Send VPU_JSM_MSG_CONTEXT_DELETE when deleting context

2023-02-02 Thread Stanislaw Gruszka
From: Andrzej Kacprowski The VPU_JSM_MSG_CONTEXT_DELETE will remove any resources associated with the SSID, that included any blobs create by the user space application. The command can also remove doorbell registrations, but since this does not work in HW scheduling case, we do not depend on

[PATCH 3/4] accel/ivpu: Set dma max_segment_size

2023-02-02 Thread Stanislaw Gruszka
Avoid below spurious warning: [ 264.844029] DMA-API: intel_vpu :00:0b.0: mapping sg segment longer than device claims to support [len=143360] [max=65536] [ 264.844038] WARNING: CPU: 0 PID: 1254 at kernel/dma/debug.c:1160 debug_dma_map_sg+0x6ca/0xb70 Signed-off-by: Stanislaw Gruszka ---

  1   2   >