Re: [PATCH v4 3/3] drm/panel-simple: allow LVDS format override

2023-08-18 Thread Johannes Zink
Hi Dan, do you have any input on this for me? Best regards Johannes On 7/28/23 16:16, Johannes Zink wrote: Some panels support multiple LVDS data mapping formats, which can be used e.g. run displays on jeida-18 format when only 3 LVDS lanes are available. Add parsing of an optional data-mappi

Re: [PATCH 03/13] scatterlist: Add sg_set_folio()

2023-08-18 Thread Zhu Yanjun
在 2023/7/31 5:42, Matthew Wilcox 写道: On Sun, Jul 30, 2023 at 09:57:06PM +0800, Zhu Yanjun wrote: 在 2023/7/30 19:18, Matthew Wilcox 写道: On Sun, Jul 30, 2023 at 07:01:26PM +0800, Zhu Yanjun wrote: Does the following function have folio version? " int sg_alloc_append_table_from_pages(struct sg

Re: [PATCH drm-misc-next] drm/ttm/tests: Require MMU when testing

2023-08-18 Thread Karolina Stolarek
Hi Thomas, On 17.08.2023 15:11, Thomas Zimmermann wrote: Hi I've cherry-picked the patch into drm-misc-next-fixes, so it will show up in upstream soon. Many thanks for your help. I'm sorry this one slipped through. Am 11.08.23 um 13:36 schrieb Karolina Stolarek: Satisfy MMU dependency wh

Adds support for ConfigFS to VKMS!

2023-08-18 Thread Brandon Pollack
Since Jim is busy with other work and I'm working on some things that rely on this, I've taken up the task of doing the iterations. I've addressed the comments as best I can (those replies are to each individual change) and here is the patch set to go with those. I added my own signoff to each co

[PATCH v3 1/7] drm/vkms: Back VKMS with DRM memory management instead of static objects

2023-08-18 Thread Brandon Pollack
From: Jim Shargo This is a small refactor to make ConfigFS support easier. Once we support ConfigFS, there can be multiple devices instantiated by the driver, and so moving everything into managed memory makes things much easier. This should be a no-op refactor. Signed-off-by: Jim Shargo Signe

[PATCH v3 2/7] drm/vkms: Support multiple DRM objects (crtcs, etc.) per VKMS device

2023-08-18 Thread Brandon Pollack
From: Jim Shargo This change supports multiple CRTCs, encoders, connectors instead of one of each per device. Since ConfigFS-based devices will support multiple crtcs, it's useful to move all of the writeback/composition data from being per-"output" to being per-CRTC. Since there's still only e

[PATCH v3 3/7] drm/vkms: Provide platform data when creating VKMS devices

2023-08-18 Thread Brandon Pollack
From: Jim Shargo This is a small refactor to make ConfigFS support easier. This should be a no-op refactor. Signed-off-by: Jim Shargo Signed-off-by: Brandon Pollack --- drivers/gpu/drm/vkms/vkms_drv.c| 14 -- drivers/gpu/drm/vkms/vkms_drv.h| 9 ++--- drivers/gpu/drm/v

[PATCH v3 4/7] drm/vkms: Add ConfigFS scaffolding to VKMS

2023-08-18 Thread Brandon Pollack
From: Jim Shargo This change adds the basic scaffolding for ConfigFS, including setting up the default directories. It does not allow for the registration of configfs-backed devices, which is complex and provided in a follow-up commit. This CL includes docs about using ConfigFS with VKMS, but I'

[PATCH v3 5/7] drm/vkms: Support enabling ConfigFS devices

2023-08-18 Thread Brandon Pollack
From: Jim Shargo VKMS now supports creating and using virtual devices! In addition to the enabling logic, this commit also prevents users from adding new objects once a card is registered. Signed-off-by: Jim Shargo Signed-off-by: Brandon Pollack --- drivers/gpu/drm/vkms/vkms_configfs.c | 37

[PATCH v3 6/7] drm/vkms: Add a module param to enable/disable the default device

2023-08-18 Thread Brandon Pollack
From: Jim Shargo In many testing circumstances, we will want to just create a new device and test against that. If we create a default device, it can be annoying to have to manually select the new device instead of choosing the only one that exists. The param, enable_default, is defaulted to tru

[PATCH v3 7/7] drm/vkms Add hotplug support via configfs to VKMS.

2023-08-18 Thread Brandon Pollack
This change adds the ability to read or write a "1" or a "0" to the newly added "connected" attribute of a connector in the vkms entry in configfs. A write will trigger a call to drm_kms_helper_hotplug_event, causing a hotplug uevent. With this we can write virtualized multidisplay tests that inv

Re: Implement svm without BO concept in xe driver

2023-08-18 Thread Thomas Hellström
On 8/17/23 04:12, Zeng, Oak wrote: -Original Message- From: Dave Airlie Sent: August 16, 2023 6:52 PM To: Felix Kuehling Cc: Zeng, Oak ; Christian König ; Thomas Hellström ; Brost, Matthew ; maarten.lankho...@linux.intel.com; Vishwanathapura, Niranjana ; Welty, Brian ; Philip Yang ; i

[PATCH v3 0/7] Adds support for ConfigFS to VKMS!

2023-08-18 Thread Brandon Pollack
Since Jim is busy with other work and I'm working on some things that rely on this, I've taken up the task of doing the iterations. I've addressed the comments as best I can (those replies are to each individual change) and here is the patch set to go with those. I added my own signoff to each co

[PATCH v3 1/7] drm/vkms: Back VKMS with DRM memory management instead of static objects

2023-08-18 Thread Brandon Pollack
From: Jim Shargo This is a small refactor to make ConfigFS support easier. Once we support ConfigFS, there can be multiple devices instantiated by the driver, and so moving everything into managed memory makes things much easier. This should be a no-op refactor. Signed-off-by: Jim Shargo Signe

[PATCH v3 2/7] drm/vkms: Support multiple DRM objects (crtcs, etc.) per VKMS device

2023-08-18 Thread Brandon Pollack
From: Jim Shargo This change supports multiple CRTCs, encoders, connectors instead of one of each per device. Since ConfigFS-based devices will support multiple crtcs, it's useful to move all of the writeback/composition data from being per-"output" to being per-CRTC. Since there's still only e

[PATCH v3 7/7] drm/vkms Add hotplug support via configfs to VKMS.

2023-08-18 Thread Brandon Pollack
This change adds the ability to read or write a "1" or a "0" to the newly added "connected" attribute of a connector in the vkms entry in configfs. A write will trigger a call to drm_kms_helper_hotplug_event, causing a hotplug uevent. With this we can write virtualized multidisplay tests that inv

[PATCH v3 3/7] drm/vkms: Provide platform data when creating VKMS devices

2023-08-18 Thread Brandon Pollack
From: Jim Shargo This is a small refactor to make ConfigFS support easier. This should be a no-op refactor. Signed-off-by: Jim Shargo Signed-off-by: Brandon Pollack --- drivers/gpu/drm/vkms/vkms_drv.c| 14 -- drivers/gpu/drm/vkms/vkms_drv.h| 9 ++--- drivers/gpu/drm/v

[PATCH v3 5/7] drm/vkms: Support enabling ConfigFS devices

2023-08-18 Thread Brandon Pollack
From: Jim Shargo VKMS now supports creating and using virtual devices! In addition to the enabling logic, this commit also prevents users from adding new objects once a card is registered. Signed-off-by: Jim Shargo Signed-off-by: Brandon Pollack --- drivers/gpu/drm/vkms/vkms_configfs.c | 37

[PATCH v3 4/7] drm/vkms: Add ConfigFS scaffolding to VKMS

2023-08-18 Thread Brandon Pollack
From: Jim Shargo This change adds the basic scaffolding for ConfigFS, including setting up the default directories. It does not allow for the registration of configfs-backed devices, which is complex and provided in a follow-up commit. This CL includes docs about using ConfigFS with VKMS, but I'

[PATCH v3 6/7] drm/vkms: Add a module param to enable/disable the default device

2023-08-18 Thread Brandon Pollack
From: Jim Shargo In many testing circumstances, we will want to just create a new device and test against that. If we create a default device, it can be annoying to have to manually select the new device instead of choosing the only one that exists. The param, enable_default, is defaulted to tru

Re: [PATCH] drm/panel: Add prepare_prev_first flag to Visionox VTDR6130

2023-08-18 Thread neil . armstrong
Hi Dmitry, On 17/08/2023 20:35, Dmitry Baryshkov wrote: On 16/08/2023 10:51, neil.armstr...@linaro.org wrote: Hi Abhinav, On 14/08/2023 20:02, Abhinav Kumar wrote: Sending HS commands will always work on any controller, it's all about LP commands. The Samsung panels you listed only send

Re: [PATCH v2 1/2] drm/v3d: Implement show_fdinfo() callback for GPU usage stats

2023-08-18 Thread Melissa Wen
On 08/07, Maíra Canal wrote: > This patch exposes the accumulated amount of active time per client > through the fdinfo infrastructure. The amount of active time is exposed > for each V3D queue: BIN, RENDER, CSD, TFU and CACHE_CLEAN. > > In order to calculate the amount of active time per client,

Re: [PATCH v2] drm: bridge: samsung-dsim: Fix init during host transfer

2023-08-18 Thread Neil Armstrong
On 24/07/2023 17:16, Frieder Schrempf wrote: From: Frieder Schrempf In case the downstream bridge or panel uses DSI transfers before the DSI host was actually initialized through samsung_dsim_atomic_enable() which clears the stop state (LP11) mode, all transfers will fail. This happens with do

Re: [PATCH v2] drm: bridge: samsung-dsim: Fix init during host transfer

2023-08-18 Thread Neil Armstrong
Hi, On Mon, 24 Jul 2023 17:16:32 +0200, Frieder Schrempf wrote: > In case the downstream bridge or panel uses DSI transfers before the > DSI host was actually initialized through samsung_dsim_atomic_enable() > which clears the stop state (LP11) mode, all transfers will fail. > > This happens with

Re: [PATCH v2 2/2] drm/v3d: Expose the total GPU usage stats on sysfs

2023-08-18 Thread Melissa Wen
On 08/07, Maíra Canal wrote: > The previous patch exposed the accumulated amount of active time per > client for each V3D queue. But this doesn't provide a global notion of > the GPU usage. > > Therefore, provide the accumulated amount of active time for each V3D > queue (BIN, RENDER, CSD, TFU and

Re: [PATCH v2 0/2] drm/v3d: Expose GPU usage stats

2023-08-18 Thread Melissa Wen
On 08/07, Maíra Canal wrote: > This patchset exposes GPU usages stats both globally and per-file > descriptor. > > The first patch exposes the accumulated amount of active time per client > through the fdinfo infrastructure. The amount of active time is exposed > for each V3D queue. Moreover, it e

Re: [Linaro-mm-sig] [PATCH] dma-buf/sw_sync: Avoid recursive lock during fence signal

2023-08-18 Thread Christian König
Am 17.08.23 um 23:37 schrieb Rob Clark: From: Rob Clark If a signal callback releases the sw_sync fence, that will trigger a deadlock as the timeline_fence_release recurses onto the fence->lock (used both for signaling and the the timeline tree). To avoid that, temporarily hold an extra refere

Re: [PATCH 1/5] drm/bridge: tc358762: Split register programming from pre-enable to enable

2023-08-18 Thread Dmitry Baryshkov
On 16/06/2023 22:02, Sam Ravnborg wrote: Hi Marek, On Thu, Jun 15, 2023 at 10:18:58PM +0200, Marek Vasut wrote: Move the register programming part, which actually enables the bridge and makes it push data out of its DPI side, into the enable callback. The DSI host like DSIM may not be able to t

Re: [PATCH v4] PCI/VGA: Make the vga_is_firmware_default() less arch-dependent

2023-08-18 Thread suijingfeng
Hi, On 2023/8/18 06:08, Bjorn Helgaas wrote: This patch makes the vga_is_firmware_default() function works on whatever arch that has UEFI GOP support. But we make it available only on platforms where PCI resource relocation happens. if the provided method proves to be effective and reliable, it

Re: [PATCH v5 02/17] dt-bindings: gpu: Add Imagination Technologies PowerVR GPU

2023-08-18 Thread Linus Walleij
Hi Sarah, thanks for your patch! Patches adding device tree bindings need to be CC:ed to devicet...@vger.kernel.org and the DT binding maintainers, I have added it for now. On Wed, Aug 16, 2023 at 10:26 AM Sarah Walker wrote: > Add the device tree binding documentation for the Series AXE GPU u

Re: [RFC PATCH v2 00/11] Device Memory TCP

2023-08-18 Thread Pavel Begunkov
On 8/14/23 02:12, David Ahern wrote: On 8/9/23 7:57 PM, Mina Almasry wrote: Changes in RFC v2: -- ... ** Test Setup Kernel: net-next with this RFC and memory provider API cherry-picked locally. Hardware: Google Cloud A3 VMs. NIC: GVE with header split & RSS & flow steering s

Re: [PATCH v4] PCI/VGA: Make the vga_is_firmware_default() less arch-dependent

2023-08-18 Thread suijingfeng
Hi, On 2023/8/18 06:08, Bjorn Helgaas wrote: Please note that before apply this patch, vgaarb can not select the right boot vga due to weird logic introduced with the commit 57fc7323a8e7c ("LoongArch: Add PCI controller support") If we need this reference to 57fc7323a8e7c, we need more specifi

Re: [PATCH] drm/panel: Add prepare_prev_first flag to Visionox VTDR6130

2023-08-18 Thread Dmitry Baryshkov
On 18/08/2023 11:25, neil.armstr...@linaro.org wrote: Hi Dmitry, On 17/08/2023 20:35, Dmitry Baryshkov wrote: On 16/08/2023 10:51, neil.armstr...@linaro.org wrote: Hi Abhinav, On 14/08/2023 20:02, Abhinav Kumar wrote: Sending HS commands will always work on any controller, it's all abo

Re: [PATCH 3/4] drm/uapi: document the USB subconnector type

2023-08-18 Thread Dmitry Baryshkov
On 18/08/2023 09:24, Simon Ser wrote: On Thursday, August 17th, 2023 at 21:33, Dmitry Baryshkov wrote: We have been looking for a way to document that the corresponding DP port is represented by the USB connector on the device. Consequently, I believe the best way to document it, would be to

Re: [PATCH v5 02/17] dt-bindings: gpu: Add Imagination Technologies PowerVR GPU

2023-08-18 Thread Krzysztof Kozlowski
On 16/08/2023 10:25, Sarah Walker wrote: > Add the device tree binding documentation for the Series AXE GPU used in > TI AM62 SoCs. > > Co-developed-by: Frank Binns > Signed-off-by: Frank Binns > Signed-off-by: Sarah Walker > --- > Changes since v4: > - Add clocks constraint for ti,am62-gpu P

Re: [PATCH v5 02/17] dt-bindings: gpu: Add Imagination Technologies PowerVR GPU

2023-08-18 Thread Krzysztof Kozlowski
On 18/08/2023 11:36, Linus Walleij wrote: > Hi Sarah, > > thanks for your patch! > > Patches adding device tree bindings need to be CC:ed to > devicet...@vger.kernel.org > and the DT binding maintainers, I have added it for now. > This won't help, I think. Patch will not be tested. I was alrea

Re: [PATCH v5 17/17] arm64: dts: ti: k3-am62-main: Add GPU device node [DO NOT MERGE]

2023-08-18 Thread Krzysztof Kozlowski
On 16/08/2023 10:25, Sarah Walker wrote: > Add the Series AXE GPU node to the AM62 device tree. > > Changes since v4: > - Remove interrupt name > - Make property order consistent across dts and bindings doc > - Fixed formatting (replaced spaces with tabs) > Nope, DTS go via SoC tree. You skipped

Re: [PATCH RFC v5 02/10] drm: Introduce solid fill DRM plane property

2023-08-18 Thread Pekka Paalanen
On Fri, 4 Aug 2023 16:59:00 +0300 Dmitry Baryshkov wrote: > On Fri, 4 Aug 2023 at 16:44, Sebastian Wick wrote: > > > > On Fri, Aug 4, 2023 at 3:27 PM Dmitry Baryshkov > > wrote: > > > > > > On Fri, 28 Jul 2023 at 20:03, Jessica Zhang > > > wrote: > > > > > > > > Document and add support f

Re: [EXTERNAL] Re: [PATCH v5 13/17] drm/imagination: Implement context creation/destruction ioctls

2023-08-18 Thread Sarah Walker
On Fri, 2023-08-18 at 00:42 +0200, Jann Horn wrote: > *** CAUTION: This email originates from a source not known to Imagination > Technologies. Think before you click a link or open an attachment *** > > On Wed, Aug 16, 2023 at 10:25 AM Sarah Walker wrote: > > Implement ioctls for the creation a

Re: [PATCH RFC v5 02/10] drm: Introduce solid fill DRM plane property

2023-08-18 Thread Dmitry Baryshkov
On 18/08/2023 13:51, Pekka Paalanen wrote: On Fri, 4 Aug 2023 16:59:00 +0300 Dmitry Baryshkov wrote: On Fri, 4 Aug 2023 at 16:44, Sebastian Wick wrote: On Fri, Aug 4, 2023 at 3:27 PM Dmitry Baryshkov wrote: On Fri, 28 Jul 2023 at 20:03, Jessica Zhang wrote: Document and add support fo

Re: [Intel-gfx] [RESEND PATCH] drm/i915: constify pointers to hwmon_channel_info

2023-08-18 Thread Jani Nikula
On Thu, 25 May 2023, Jani Nikula wrote: > On Thu, 11 May 2023, Krzysztof Kozlowski > wrote: >> Statically allocated array of pointers to hwmon_channel_info can be made >> const for safety. > > Btw if you want to further make things const, the compound literals > defined by HWMON_CHANNEL_INFO() s

Re: [PATCH -next] drm/msm/adreno: adreno_gpu: Switch to memdup_user_nul() helper

2023-08-18 Thread Dmitry Baryshkov
On 10/08/2023 15:04, Ruan Jinjie wrote: Use memdup_user_nul() helper instead of open-coding to simplify the code. Signed-off-by: Ruan Jinjie --- drivers/gpu/drm/msm/adreno/adreno_gpu.c | 14 +++--- 1 file changed, 3 insertions(+), 11 deletions(-) Reviewed-by: Dmitry Baryshkov --

Re: [PATCH] drm/vmwgfx: Fix possible invalid drm gem put calls

2023-08-18 Thread Martin Krastev (VMware)
From: Martin Krastev LGTM! Reviewed-by: Martin Krastev Regards, Martin On 18 Aug 2023 04:13:14, Zack Rusin wrote: >From: Zack Rusin > >vmw_bo_unreference sets the input buffer to null on exit, resulting in >null ptr deref's on the subsequent drm gem put calls. > >This went unnoticed be

Re: [PATCH 7/7] drm: adv7511: Add hpd_override_enable feature bit to struct adv7511_chip_info

2023-08-18 Thread Adam Ford
On Sun, Aug 13, 2023 at 1:06 PM Biju Das wrote: > > As per spec, it is allowed to pulse the HPD signal to indicate that the > EDID information has changed. Some monitors do this when they wake up > from standby or are enabled. When the HPD goes low the adv7511 is > reset and the outputs are disabl

[PATCH -next] drm/scheduler: Remove unused declarations

2023-08-18 Thread Yue Haibing
Commit 06a2d7cc3f04 ("drm/amdgpu: revert "implement tdr advanced mode"") removed drm_sched_reset_karma()/drm_sched_increase_karma_ext() but leave the declarations. Commit 2cf9886e2816 ("drm/scheduler: remove drm_sched_dependency_optimized") removed drm_sched_dependency_optimized() but not its decla

Re: [PATCH v2 1/9] drm/sched: Convert drm scheduler to use a work queue rather than kthread

2023-08-18 Thread Matthew Brost
On Fri, Aug 18, 2023 at 07:40:41AM +0200, Christian König wrote: > Am 18.08.23 um 05:08 schrieb Matthew Brost: > > On Thu, Aug 17, 2023 at 01:13:31PM +0200, Danilo Krummrich wrote: > > > On 8/17/23 07:33, Christian König wrote: > > > > Am 16.08.23 um 18:33 schrieb Danilo Krummrich: > > > > > On 8/1

Re: [PATCH v2 2/4] drm/bridge: imx: add bridge wrapper driver for i.MX8MP DWC HDMI

2023-08-18 Thread Luca Ceresoli
Hi Lucas, On Fri, 16 Dec 2022 22:07:40 +0100 Lucas Stach wrote: > Add a simple wrapper driver for the DWC HDMI bridge driver that > implements the few bits that are necessary to abstract the i.MX8MP > SoC integration. > > Signed-off-by: Lucas Stach > Reviewed-by: Laurent Pinchart > Tested-by:

Re: [PATCH v2 4/4] drm/bridge: imx: add driver for HDMI TX Parallel Video Interface

2023-08-18 Thread Luca Ceresoli
Hi Lucas, On Fri, 16 Dec 2022 22:07:42 +0100 Lucas Stach wrote: > This IP block is found in the HDMI subsystem of the i.MX8MP SoC. It has a > full timing generator and can switch between different video sources. On > the i.MX8MP however the only supported source is the LCDIF. The block > just ne

Re: [PATCH v4] drm/sysfs: Link DRM connectors to corresponding Type-C connectors

2023-08-18 Thread Dmitry Baryshkov
On 11/11/2022 11:52, Daniel Vetter wrote: On Thu, Nov 10, 2022 at 11:33:11AM -0800, Won Chung wrote: Hi Daniel, Thank you very much for a review. On Wed, Nov 9, 2022 at 3:54 AM Daniel Vetter wrote: On Tue, Nov 08, 2022 at 06:50:04PM +, Won Chung wrote: Create a symlink pointing to USB

Re: [PATCH v2 4/9] drm/sched: Split free_job into own work item

2023-08-18 Thread Matthew Brost
On Fri, Aug 18, 2023 at 07:27:33AM +0200, Christian König wrote: > Am 17.08.23 um 19:54 schrieb Matthew Brost: > > On Thu, Aug 17, 2023 at 03:39:40PM +0200, Christian König wrote: > > > Am 11.08.23 um 04:31 schrieb Matthew Brost: > > > > Rather than call free_job and run_job in same work item have

[PATCH] drm/amd/display: fix mode scaling (RMX_.*)

2023-08-18 Thread Hamza Mahfooz
As made mention of in commit 4a2df0d1f28e ("drm/amd/display: Fixed non-native modes not lighting up"), we shouldn't call drm_mode_set_crtcinfo() once the crtc timings have been decided. Since, it can cause settings to be unintentionally overwritten. So, since dm_state is never NULL now, we can use

[PATCH 2/2] drivers/tidss: Add support for AM62A7 DSS

2023-08-18 Thread Aradhya Bhatia
Add support for the DSS controller on TI's AM62A7 SoC in the tidss driver. This contrller has 2 video pipelines that can render 2 video planes on over a screen, using the overlay managers. The output of the DSS comes from video port 2 (VP2) in the form of RGB88 DPI signals, while the VP1 is tied o

Re: [PATCH 1/1] drm/fourcc: Add documentation about software color conversion.

2023-08-18 Thread Pekka Paalanen
On Thu, 10 Aug 2023 09:45:27 +0200 Maxime Ripard wrote: > Hi > > On Mon, Aug 07, 2023 at 03:45:15PM +0200, Jocelyn Falempe wrote: > > After discussions on IRC, the consensus is that the DRM drivers should > > not do software color conversion, and only advertise the supported formats. > > Update

[Bug 217664] Laptop doesnt wake up from suspend mode.

2023-08-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=217664 --- Comment #20 from Alex Deucher (alexdeuc...@gmail.com) --- (In reply to popus_czy_to_ty from comment #18) > banned all in grub ( GRUB_CMDLINE_LINUX_DEFAULT="quiet splash > module_blacklist=nvidia,nvidia-current,nvidia_drm,nvidia_uvm,nvidia_mode

[PATCH 0/2] Add DSS support for TI AM62A7 SoC

2023-08-18 Thread Aradhya Bhatia
This patch series adds a new compatible for the Display SubSystem (DSS) controller on TI's AM62A7 SoC. It further adds the required support, for the same, in the tidss driver. The DSS controller is similar to the recently added AM625 DSS, with the key difference being the absence of VP1 output on

Re: [PATCH] drm/amd/display: fix mode scaling (RMX_.*)

2023-08-18 Thread Alex Deucher
On Fri, Aug 18, 2023 at 9:25 AM Hamza Mahfooz wrote: > > As made mention of in commit 4a2df0d1f28e ("drm/amd/display: Fixed > non-native modes not lighting up"), we shouldn't call > drm_mode_set_crtcinfo() once the crtc timings have been decided. Since, > it can cause settings to be unintentionall

Re: [PATCH v2 1/9] drm/sched: Convert drm scheduler to use a work queue rather than kthread

2023-08-18 Thread Danilo Krummrich
On 8/17/23 18:17, Christian König wrote: Am 17.08.23 um 14:48 schrieb Danilo Krummrich: On 8/17/23 15:35, Christian König wrote: Am 17.08.23 um 13:13 schrieb Danilo Krummrich: On 8/17/23 07:33, Christian König wrote: [SNIP] My proposal would be to just keep the hw_submission_limit (maybe ren

RE: [PATCH 7/7] drm: adv7511: Add hpd_override_enable feature bit to struct adv7511_chip_info

2023-08-18 Thread Biju Das
Hi Adam Ford, Thanks for the feedback. > Subject: Re: [PATCH 7/7] drm: adv7511: Add hpd_override_enable feature bit > to struct adv7511_chip_info > > On Sun, Aug 13, 2023 at 1:06 PM Biju Das > wrote: > > > > As per spec, it is allowed to pulse the HPD signal to indicate that > > the EDID inform

Re: [PATCH 7/7] drm: adv7511: Add hpd_override_enable feature bit to struct adv7511_chip_info

2023-08-18 Thread Adam Ford
On Fri, Aug 18, 2023 at 8:35 AM Biju Das wrote: > > Hi Adam Ford, > > Thanks for the feedback. > > > Subject: Re: [PATCH 7/7] drm: adv7511: Add hpd_override_enable feature bit > > to struct adv7511_chip_info > > > > On Sun, Aug 13, 2023 at 1:06 PM Biju Das > > wrote: > > > > > > As per spec, it i

Re: [PATCH v2 1/9] drm/sched: Convert drm scheduler to use a work queue rather than kthread

2023-08-18 Thread Danilo Krummrich
On 8/18/23 14:49, Matthew Brost wrote: On Fri, Aug 18, 2023 at 07:40:41AM +0200, Christian König wrote: Am 18.08.23 um 05:08 schrieb Matthew Brost: On Thu, Aug 17, 2023 at 01:13:31PM +0200, Danilo Krummrich wrote: On 8/17/23 07:33, Christian König wrote: Am 16.08.23 um 18:33 schrieb Danilo Kr

Re: [PATCH 2/2] drivers/tidss: Add support for AM62A7 DSS

2023-08-18 Thread Aradhya Bhatia
On 18-Aug-23 18:47, Aradhya Bhatia wrote: > Add support for the DSS controller on TI's AM62A7 SoC in the tidss > driver. > > This contrller has 2 video pipelines that can render 2 video planes on > over a screen, using the overlay managers. The output of the DSS comes > from video port 2 (VP2)

Re: [BUG] KCSAN: data-race in drm_sched_entity_is_ready [gpu_sched] / drm_sched_entity_push_job [gpu_sched]

2023-08-18 Thread Mirsad Todorovac
On 8/17/23 21:54, Mirsad Todorovac wrote: Hi, This is your friendly bug reporter. The environment is vanilla torvalds tree kernel on Ubuntu 22.04 LTS and a Ryzen 7950X box. Please find attached the complete dmesg output from the ring buffer and lshw output. NOTE: The kernel reports tainted

RE: [PATCH 7/7] drm: adv7511: Add hpd_override_enable feature bit to struct adv7511_chip_info

2023-08-18 Thread Biju Das
Hi Adam, > Subject: Re: [PATCH 7/7] drm: adv7511: Add hpd_override_enable feature bit > to struct adv7511_chip_info > > On Fri, Aug 18, 2023 at 8:35 AM Biju Das > wrote: > > > > Hi Adam Ford, > > > > Thanks for the feedback. > > > > > Subject: Re: [PATCH 7/7] drm: adv7511: Add hpd_override_enabl

[PATCH 1/2] dt-bindings: display: ti: Add support for am62a7 dss

2023-08-18 Thread Aradhya Bhatia
The DSS controller on TI's AM62A7 SoC is an update from that on TI's AM625 SoC. Like the DSS in AM625, the DSS in this SoC has 2 video pipelines, but unlike the former, the latter only has one output port on VP2 to service DPI display sinks. Add the new controller's compatible. Signed-off-by: Ara

Re: [PATCH] drm/amd/display: fix mode scaling (RMX_.*)

2023-08-18 Thread Hamza Mahfooz
On 8/18/23 09:28, Alex Deucher wrote: On Fri, Aug 18, 2023 at 9:25 AM Hamza Mahfooz wrote: As made mention of in commit 4a2df0d1f28e ("drm/amd/display: Fixed non-native modes not lighting up"), we shouldn't call drm_mode_set_crtcinfo() once the crtc timings have been decided. Since, it can c

Re: [EXTERNAL] Re: [PATCH v5 03/17] drm/imagination/uapi: Add PowerVR driver UAPI

2023-08-18 Thread Sarah Walker
On Thu, 2023-08-17 at 19:43 -0500, Faith Ekstrand wrote: > On Wed, Aug 16, 2023 at 3:26 AM Sarah Walker wrote: > > > > +/** > > + * struct drm_pvr_dev_query_runtime_info - Container used to fetch > > information > > + * about the graphics runtime. > > + * > > + * When fetching this type &struct

Re: [PATCH RFC v5 02/10] drm: Introduce solid fill DRM plane property

2023-08-18 Thread Pekka Paalanen
On Fri, 18 Aug 2023 14:03:14 +0300 Dmitry Baryshkov wrote: > On 18/08/2023 13:51, Pekka Paalanen wrote: > > On Fri, 4 Aug 2023 16:59:00 +0300 > > Dmitry Baryshkov wrote: > > > >> On Fri, 4 Aug 2023 at 16:44, Sebastian Wick > >> wrote: > >>> > >>> On Fri, Aug 4, 2023 at 3:27 PM Dmitry Bary

Re: [PATCH 0/7] ADV7511 driver enhancements

2023-08-18 Thread Fabio Estevam
Hi Biju, On Sun, Aug 13, 2023 at 3:05 PM Biju Das wrote: > > This patch series aims to improve ADV7511 driver by adding > feature bits and data instead of comparing enum adv7511_type for > various hardware differences between ADV7511, ADV7533 and ADV7535. > > This patch series tested with[1] on R

Re: [06/12] arch: Declare screen_info in

2023-08-18 Thread suijingfeng
Hi, Why this patch get dropped in the end? Since the global screen_info is an arch-specific thing, Whenever an arch-neutral module or subsystem references the global screen_info, There are some complaints from either compile testing robot. Well, a programmer may handle it by using the CONFIG_SY

Re: [EXTERNAL] Re: [PATCH v5 08/17] drm/imagination: Add GEM and VM related code

2023-08-18 Thread Sarah Walker
On Fri, 2023-08-18 at 00:42 +0200, Jann Horn wrote: > *** CAUTION: This email originates from a source not known to Imagination > Technologies. Think before you click a link or open an attachment *** > > Hi! > > Thanks, I think it's great that Imagination is writing an upstream > driver for Powe

Re: [06/12] arch: Declare screen_info in

2023-08-18 Thread suijingfeng
On 2023/8/18 22:04, suijingfeng wrote: Hi, Why this patch get dropped in the end? Since the global screen_info is an arch-specific thing, Whenever an arch-neutral module or subsystem references the global screen_info, There are some complaints from either compile testing robot. There ar

[PATCH v2 0/2] Add DSS support for TI AM62A7 SoC

2023-08-18 Thread Aradhya Bhatia
This patch series adds a new compatible for the Display SubSystem (DSS) controller on TI's AM62A7 SoC. It further adds the required support, for the same, in the tidss driver. The DSS controller is similar to the recently added AM625 DSS, with the key difference being the absence of VP1 output on

[PATCH v2 1/2] dt-bindings: display: ti: Add support for am62a7 dss

2023-08-18 Thread Aradhya Bhatia
The DSS controller on TI's AM62A7 SoC is an update from that on TI's AM625 SoC. Like the DSS in AM625, the DSS in this SoC has 2 video pipelines, but unlike the former, the latter only has one output port on VP2 to service DPI display sinks. Add the new controller's compatible. Signed-off-by: Ara

[PATCH v2 2/2] drivers/tidss: Add support for AM62A7 DSS

2023-08-18 Thread Aradhya Bhatia
Add support for the DSS controller on TI's AM62A7 SoC in the tidss driver. This contrller has 2 video pipelines that can render 2 video planes on over a screen, using the overlay managers. The output of the DSS comes from video port 2 (VP2) in the form of RGB88 DPI signals, while the VP1 is tied o

Re: [PATCH v2 10/15] drm/panthor: Add the heap logical block

2023-08-18 Thread Steven Price
On 09/08/2023 17:53, Boris Brezillon wrote: > Tiler heap growing requires some kernel driver involvement: when the > tiler runs out of heap memory, it will raise an exception which is > either directly handled by the firmware if some free heap chunks are > available in the heap context, or passed b

Re: [Linaro-mm-sig] [PATCH] dma-buf/sw_sync: Avoid recursive lock during fence signal

2023-08-18 Thread Rob Clark
On Fri, Aug 18, 2023 at 2:09 AM Christian König wrote: > > Am 17.08.23 um 23:37 schrieb Rob Clark: > > From: Rob Clark > > > > If a signal callback releases the sw_sync fence, that will trigger a > > deadlock as the timeline_fence_release recurses onto the fence->lock > > (used both for signaling

Re: [PATCH 1/7] drm: adv7511: Add struct adv7511_chip_info and use i2c_get_match_data()

2023-08-18 Thread Adam Ford
On Sun, Aug 13, 2023 at 1:05 PM Biju Das wrote: > > Add struct adv7511_chip_info to handle hw differences between various > chips rather checking against the 'type' variable in various places. > Replace 'adv->type'->'info->type' by moving variable 'type' from > struct adv7511 to struct adv7511_chi

[PATCH v2] dma-buf/sw_sync: Avoid recursive lock during fence signal

2023-08-18 Thread Rob Clark
From: Rob Clark If a signal callback releases the sw_sync fence, that will trigger a deadlock as the timeline_fence_release recurses onto the fence->lock (used both for signaling and the the timeline tree). To avoid that, temporarily hold an extra reference to the signalled fences until after we

Re: [PATCH] drm/amd/display: fix mode scaling (RMX_.*)

2023-08-18 Thread Harry Wentland
On 2023-08-18 09:17, Hamza Mahfooz wrote: > As made mention of in commit 4a2df0d1f28e ("drm/amd/display: Fixed > non-native modes not lighting up"), we shouldn't call > drm_mode_set_crtcinfo() once the crtc timings have been decided. Since, > it can cause settings to be unintentionally overwritt

[PATCH 0/4] drm/xe: Support optional pinning of userptr pages

2023-08-18 Thread Thomas Hellström
This series add a flag at VM_BIND time to pin the memory backing a VMA. Initially this is needed for long-running workloads on hardware that neither support mid-thread preemption nor pagefaults, since without it the userptr MMU notifier will wait for preemption until preemption times out. Moving f

[PATCH 1/4] drm/xe/vm: Use onion unwind for xe_vma_userptr_pin_pages()

2023-08-18 Thread Thomas Hellström
Use onion error unwind since that makes the function easier to read and extend. No functional change. Signed-off-by: Thomas Hellström --- drivers/gpu/drm/xe/xe_vm.c | 37 +++-- 1 file changed, 19 insertions(+), 18 deletions(-) diff --git a/drivers/gpu/drm/xe/xe_v

[PATCH 2/4] drm/xe/vm: Implement userptr page pinning

2023-08-18 Thread Thomas Hellström
Implement pinning of userptrs between VM_BIND and VM_UNBIND, which will facilitate avoiding long hangs on non-preemptible workloads. But don't hook it up to userspace just yet. Signed-off-by: Thomas Hellström --- drivers/gpu/drm/xe/xe_vm.c | 76 ++-- drivers/gpu

[PATCH 3/4] drm/xe/vm: Perform accounting of userptr pinned pages

2023-08-18 Thread Thomas Hellström
Account these pages against RLIMIT_MEMLOCK following how RDMA does this with CAP_IPC_LOCK bypassing the limit. Signed-off-by: Thomas Hellström --- drivers/gpu/drm/xe/xe_vm.c | 43 -- 1 file changed, 41 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/dr

[PATCH 4/4] drm/xe/uapi: Support pinning of userptr vmas

2023-08-18 Thread Thomas Hellström
Support pinning of vmas using XE_VM_BIND_FLAG_PIN, initially for userptr only. Pinned memory becomes accounted against RLIMIT_MEMLOCK and processes with CAP_IPC_LOCK will not apply the limit. This is pretty similar to mlock()'ing userptr memory with the added benefit that the driver is aware and ca

Re: [PATCH v5 08/17] drm/imagination: Add GEM and VM related code

2023-08-18 Thread Danilo Krummrich
Hi Sarah, On Wed, Aug 16, 2023 at 09:25:23AM +0100, Sarah Walker wrote: > Add a GEM implementation based on drm_gem_shmem, and support code for the > PowerVR GPU MMU. The GPU VA manager is used for address space management. > > Changes since v4: > - Correct sync function in vmap/vunmap function d

Re: [v2,2/2] doc: uapi: Add document describing dma-buf semantics

2023-08-18 Thread suijingfeng
Hi, On 2023/8/3 23:47, Daniel Stone wrote: Since there's a lot of confusion around this, document both the rules and the best practice around negotiating, allocating, importing, and Probably, best practices? using buffers when crossing context/process/device/subsystem boundaries. This ti

Re: (subset) [PATCH] backlight: lp855x: Drop ret variable in brightness change function

2023-08-18 Thread Lee Jones
On Wed, 09 Aug 2023 13:42:16 +0200, Artur Weber wrote: > Fixes the following warning: > > drivers/video/backlight/lp855x_bl.c:252:7: warning: variable 'ret' is used > uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized] > > Applied, thanks! [1/1] backlight: lp855x: Drop r

Re: [PATCH v2 11/15] drm/panthor: Add the scheduler logical block

2023-08-18 Thread Steven Price
On 09/08/2023 17:53, Boris Brezillon wrote: > This is the piece of software interacting with the FW scheduler, and > taking care of some scheduling aspects when the FW comes short of slots > scheduling slots. Indeed, the FW only expose a few slots, and the kernel > has to give all submission contex

RE: Implement svm without BO concept in xe driver

2023-08-18 Thread Zeng, Oak
Thanks Thomas. I will then look into more details of option 3: * create a lean drm layer vram manager, a central control place for vram eviction and cgroup accounting. Single LRU for eviction fairness. * pretty much move the current ttm_resource eviction/cgroups logic to drm layer * the

RE: [PATCH v5 06/11] drm/radeon: Use RMW accessors for changing LNKCTL

2023-08-18 Thread Deucher, Alexander
[Public] > -Original Message- > From: Ilpo Järvinen > Sent: Monday, July 17, 2023 8:05 AM > To: linux-...@vger.kernel.org; Bjorn Helgaas ; Lorenzo > Pieralisi ; Rob Herring ; > Krzysztof Wilczyński ; Emmanuel Grumbach > ; Rafael J . Wysocki ; > Heiner Kallweit ; Lukas Wunner ; > Andy Shev

Re: [PATCH] drivers: gpu: drm: radeon: possible buffer overflow

2023-08-18 Thread Alex Deucher
Applied. Thanks! On Thu, Aug 17, 2023 at 7:34 AM Konstantin Meskhidze wrote: > > Buffer 'afmt_status' of size 6 could overflow, since index 'afmt_idx' is > checked after access. > > Fixes: 5cc4e5fc293b ("drm/radeon: Cleanup HDMI audio interrupt handling for > evergreen") > Co-developed-by: Ivan

Re: [PATCH] drm/amd/pm: Fix unsigned expression compared with zero

2023-08-18 Thread Alex Deucher
On Thu, Aug 17, 2023 at 2:07 AM Jiapeng Chong wrote: > > The val is defined as unsigned int type, if(val<0) is invalid, modify > to int type. > > drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_pm.c:2813 > amdgpu_hwmon_show_power_input() warn: unsigned 'val' is never less than zero. > drivers/gpu/drm/amd

Re: Implement svm without BO concept in xe driver

2023-08-18 Thread Felix Kuehling
On 2023-08-18 12:10, Zeng, Oak wrote: Thanks Thomas. I will then look into more details of option 3: * create a lean drm layer vram manager, a central control place for vram eviction and cgroup accounting. Single LRU for eviction fairness. * pretty much move the current ttm_resource e

[PATCH v2 0/2] Match data improvements for it66121 driver

2023-08-18 Thread Biju Das
This patch series aims to add match data improvements for it66121 driver. v1->v2: * Split the patch into two. * patch#1 extend match support for OF tables compared to legacy ID lookup and fixes tag. * patch#2 simplifies the probe() by using i2c_get_match_data. * Dropped sentence for droppi

[PATCH v2 1/2] drm: bridge: it66121: Extend match support for OF tables

2023-08-18 Thread Biju Das
The driver has OF match table, still it uses ID lookup table for retrieving match data. Currently the driver is working on the assumption that a I2C device registered via OF will always match a legacy I2C device ID. The correct approach is to have an OF device ID table using of_device_match_data()

[PATCH v2 2/2] drm: bridge: it66121: Simplify probe()

2023-08-18 Thread Biju Das
Simplify probe() by replacing of_device_get_match_data() and ID lookup for retrieving match data by i2c_get_match_data(). Signed-off-by: Biju Das --- v1->v2: * Dropped sentence for dropping local variable as it is integral part of the patch. --- drivers/gpu/drm/bridge/ite-it66121.c | 6 +

[PATCH 0/3] Define a final failure state when link training fails

2023-08-18 Thread Gil Dekel
Currently, when link training fails after all fallback values have been exhausted, the i915 driver seizes to send uevents to userspace. This leave userspace thinking that the last passing atomic commit was successful, and that all connectors (displays) are connected and operational, when in fact, t

[PATCH 1/3] drm/i915/dp_link_training: Add a final failing state to link training fallback

2023-08-18 Thread Gil Dekel
Instead of silently giving up when all link-training fallback values are exhausted, this patch modifies the fallback's failure branch to reduces both max_link_lane_count and max_link_rate to zero (0) and continues to emit uevents until userspace stops attempting to modeset. By doing so, we ensure

[PATCH 2/3] drm/i915/dp_link_training: Add a final failing state to link training fallback for MST

2023-08-18 Thread Gil Dekel
Currently, MST link training has no fallback whatsoever. This means that if an MST base connector fails to link-train once, the training completely fails, which makes this case significantly more common than a complete SST link training failure. Until MST fallback is fully implemented, this patch

[PATCH 3/3] drm/i915/dp_link_training: Emit a link-status=Bad uevent with trigger property

2023-08-18 Thread Gil Dekel
When a link-training attempt fails, emit a uevent to user space that includes the trigger property, which in this case will be link-statue=Bad. This will allow userspace to parse the uevent property and better understand the reason for the previous display configuration failure. Cc: Jani Nikula

  1   2   >