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

2023-08-16 Thread Christian König
Am 16.08.23 um 18:33 schrieb Danilo Krummrich: On 8/16/23 16:59, Christian König wrote: Am 16.08.23 um 14:30 schrieb Danilo Krummrich: On 8/16/23 16:05, Christian König wrote: Am 16.08.23 um 13:30 schrieb Danilo Krummrich: Hi Matt, On 8/11/23 04:31, Matthew Brost wrote: In XE, the new

Re: [RFC PATCH 1/3] drm/virtio: .release ops for virtgpu fence release

2023-08-16 Thread Kim, Dongwon
Hi, On 8/16/2023 10:05 PM, Dmitry Osipenko wrote: On 8/16/23 21:10, Kim, Dongwon wrote: Hi, On 8/14/2023 9:18 PM, Dmitry Osipenko wrote: On 7/13/23 01:44, Dongwon Kim wrote: virtio_gpu_fence_release is added to free virtio-gpu-fence upon release of dma_fence. Cc: Gerd Hoffmann Cc: Vivek

RE: [PATCH] drm/scheduler: Partially revert "drm/scheduler: track GPU active time per entity"

2023-08-16 Thread Chen, Guchun
[Public] Hi Xinhui, That patch has been reverted on Linux mainline. https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/gpu/drm/scheduler/sched_main.c?h=v6.5-rc6=baad10973fdb442912af676de3348e80bd8fe602 Regards, Guchun > -Original Message- > From: amd-gfx

RE: [PATCH] drm/scheduler: Partially revert "drm/scheduler: track GPU active time per entity"

2023-08-16 Thread Pan, Xinhui
[AMD Official Use Only - General] Can we just add kref for entity? Or just collect such job time usage somewhere else? -Original Message- From: Pan, Xinhui Sent: Thursday, August 17, 2023 1:05 PM To: amd-...@lists.freedesktop.org Cc: Tuikov, Luben ; airl...@gmail.com;

Re: [RFC PATCH 1/3] drm/virtio: .release ops for virtgpu fence release

2023-08-16 Thread Dmitry Osipenko
On 8/16/23 21:10, Kim, Dongwon wrote: > Hi, > > On 8/14/2023 9:18 PM, Dmitry Osipenko wrote: >> On 7/13/23 01:44, Dongwon Kim wrote: >>> virtio_gpu_fence_release is added to free virtio-gpu-fence >>> upon release of dma_fence. >>> >>> Cc: Gerd Hoffmann >>> Cc: Vivek Kasireddy >>> Signed-off-by:

[PATCH] drm/scheduler: Partially revert "drm/scheduler: track GPU active time per entity"

2023-08-16 Thread xinhui pan
This patch partially revert commit df622729ddbf ("drm/scheduler: track GPU active time per entity") which touchs entity without any reference. I notice there is one memory overwritten from gpu scheduler side. The case is like below. A(drm_sched_main) B(vm fini)

Re: [PATCH] Documentation/gpu: Update amdgpu documentation

2023-08-16 Thread Lazar, Lijo
On 8/17/2023 2:16 AM, Alex Deucher wrote: On Wed, Aug 16, 2023 at 12:15 AM Lijo Lazar wrote: 7957ec80ef97 ("drm/amdgpu: Add FRU sysfs nodes only if needed") moved the documentation for some of the sysfs nodes to amdgpu_fru_eeprom.c. Update the documentation accordingly. Signed-off-by:

Re: [PATCH 1/5] mm: move some shrinker-related function declarations to mm/internal.h

2023-08-16 Thread Qi Zheng
On 2023/8/16 23:01, kernel test robot wrote: Hi Qi, kernel test robot noticed the following build warnings: [auto build test WARNING on brauner-vfs/vfs.all] [also build test WARNING on linus/master v6.5-rc6 next-20230816] [cannot apply to akpm-mm/mm-everything drm-misc/drm-misc-next vfs

RE: Implement svm without BO concept in xe driver

2023-08-16 Thread Zeng, Oak
> -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 ; intel- >

Re: [PATCH v2] Documentation/gpu: VM_BIND locking document

2023-08-16 Thread kernel test robot
Hi Thomas, kernel test robot noticed the following build warnings: [auto build test WARNING on drm-misc/drm-misc-next] [also build test WARNING on drm/drm-next drm-tip/drm-tip linus/master v6.5-rc6 next-20230816] [If your patch is applied to the wrong git tree, kindly drop us a note. And when

Re: [PATCH] drm/nouveau/disp: fix use-after-free in error handling of nouveau_connector_create

2023-08-16 Thread Karol Herbst
On Thu, Aug 17, 2023 at 12:14 AM Borislav Petkov wrote: > > On Wed, Aug 16, 2023 at 11:27:05PM +0200, Karol Herbst wrote: > > that GPU has only a `DMS-59` connector, is that right? > > No clue. How do I figure that out? > do you have one of these? https://en.wikipedia.org/wiki/DMS-59 > -- >

[PATCH v2] drm/i915/huc: silence injected failure in the load via GSC path

2023-08-16 Thread Daniele Ceraolo Spurio
If we can't load the HuC due to an injected failure, we don't want to throw and error and trip CI. Using the gt_probe_error macro for logging ensure that the error is only printed if it wasn't explicitly injected. v2: keep the line to less than 100 characters (checkpatch). Link:

Re: Implement svm without BO concept in xe driver

2023-08-16 Thread Dave Airlie
On Thu, 17 Aug 2023 at 08:15, Felix Kuehling wrote: > > On 2023-08-16 13:30, Zeng, Oak wrote: > > I spoke with Thomas. We discussed two approaches: > > > > 1) make ttm_resource a central place for vram management functions such as > > eviction, cgroup memory accounting. Both the BO-based driver

Re: [PATCH] drm/nouveau/disp: fix use-after-free in error handling of nouveau_connector_create

2023-08-16 Thread Borislav Petkov
On Wed, Aug 16, 2023 at 11:27:05PM +0200, Karol Herbst wrote: > that GPU has only a `DMS-59` connector, is that right? No clue. How do I figure that out? -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette

Re: Implement svm without BO concept in xe driver

2023-08-16 Thread Felix Kuehling
On 2023-08-16 13:30, Zeng, Oak wrote: I spoke with Thomas. We discussed two approaches: 1) make ttm_resource a central place for vram management functions such as eviction, cgroup memory accounting. Both the BO-based driver and BO-less SVM codes call into ttm_resource_alloc/free functions for

Re: [PATCH] drm/nouveau/disp: fix use-after-free in error handling of nouveau_connector_create

2023-08-16 Thread Karol Herbst
On Wed, Aug 16, 2023 at 5:13 PM Borislav Petkov wrote: > > On Wed, Aug 16, 2023 at 04:57:28PM +0200, Karol Herbst wrote: > > Do you have any connectors listed in "/sys/class/drm"? > > tree /sys/class/drm/ > /sys/class/drm/ > ├── card0 ->

Re: [PATCH -next v3] drm/amd/amdgpu: Use kmemdup to simplify kmalloc and memcpy logic

2023-08-16 Thread Alex Deucher
Applied. Thanks! Alex On Tue, Aug 15, 2023 at 10:20 PM Chen Jiahao wrote: > > Using kmemdup() helper function rather than implementing it again > with kmalloc() + memcpy(), which improves the code readability. > > Signed-off-by: Chen Jiahao > --- >

Re: [PATCH] drm/nouveau/disp: fix use-after-free in error handling of nouveau_connector_create

2023-08-16 Thread Lyude Paul
Reviewed-by: Lyude Paul On Mon, 2023-08-14 at 16:49 +0200, Karol Herbst wrote: > We can't simply free the connector after calling drm_connector_init on it. > We need to clean up the drm side first. > > It might not fix all regressions from 2b5d1c29f6c4 ("drm/nouveau/disp: > PIOR DP uses GPIO

Re: [PATCH] Documentation/gpu: Update amdgpu documentation

2023-08-16 Thread Alex Deucher
On Wed, Aug 16, 2023 at 12:15 AM Lijo Lazar wrote: > > 7957ec80ef97 ("drm/amdgpu: Add FRU sysfs nodes only if needed") moved > the documentation for some of the sysfs nodes to amdgpu_fru_eeprom.c. > Update the documentation accordingly. > > Signed-off-by: Lijo Lazar > --- >

Re: [PATCH 2/2][next] nouveau/svm: Split assignment from if conditional

2023-08-16 Thread Kees Cook
On Wed, Aug 16, 2023 at 12:05:06PM -0600, Gustavo A. R. Silva wrote: > Fix checkpatch.pl ERROR: do not use assignment in if condition. > > Signed-off-by: Gustavo A. R. Silva Reviewed-by: Kees Cook -- Kees Cook

Re: [PATCH 1/2][next] nouveau/svm: Replace one-element array with flexible-array member in struct nouveau_svm

2023-08-16 Thread Kees Cook
On Wed, Aug 16, 2023 at 12:04:06PM -0600, Gustavo A. R. Silva wrote: > One-element and zero-length arrays are deprecated. So, replace > one-element array in struct nouveau_svm with flexible-array member. > > This results in no differences in binary output. > > Link:

[linux-next:master] BUILD REGRESSION ef66bf8aeb91fd331cf8f5dca8f9d7bca9ab2849

2023-08-16 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: ef66bf8aeb91fd331cf8f5dca8f9d7bca9ab2849 Add linux-next specific files for 20230816 Error/Warning reports: https://lore.kernel.org/oe-kbuild-all/202307281049.40t8s0uv-...@intel.com https

[PATCH] drm: pl111: fix clang -Wvoid-pointer-to-enum-cast warning

2023-08-16 Thread Justin Stitt
ntptr_t)clcd_id->data; } map = syscon_node_to_regmap(np); --- base-commit: 2ccdd1b13c591d306f0401d98dedc4bdcd02b421 change-id: 20230816-void-drivers-gpu-drm-pl111-pl111_versatile-43b109cfa7ad Best regards, -- Justin Stitt

[pull] amdgpu drm-fixes-6.5

2023-08-16 Thread Alex Deucher
Hi Dave, Daniel, Fixes for 6.5. The following changes since commit 2ccdd1b13c591d306f0401d98dedc4bdcd02b421: Linux 6.5-rc6 (2023-08-13 11:29:55 -0700) are available in the Git repository at: https://gitlab.freedesktop.org/agd5f/linux.git tags/amd-drm-fixes-6.5-2023-08-16 for you to

[PATCH] drm/repaper: fix -Wvoid-pointer-to-enum-cast warning

2023-08-16 Thread Justin Stitt
tch; + model = (enum repaper_model)(uintptr_t)match; } else { spi_id = spi_get_device_id(spi); model = (enum repaper_model)spi_id->driver_data; --- base-commit: 2ccdd1b13c591d306f0401d98dedc4bdcd02b421 change-id: 20230816-void-drivers-gpu-dr

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

2023-08-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=217664 --- Comment #13 from Alex Deucher (alexdeuc...@gmail.com) --- Are any of the external display connectors driven by the APU? If so, do any of those work correctly on resume? -- You may reply to this email to add a comment. You are receiving

Re: [RFC PATCH 1/3] drm/virtio: .release ops for virtgpu fence release

2023-08-16 Thread Kim, Dongwon
Hi, On 8/14/2023 9:18 PM, Dmitry Osipenko wrote: On 7/13/23 01:44, Dongwon Kim wrote: virtio_gpu_fence_release is added to free virtio-gpu-fence upon release of dma_fence. Cc: Gerd Hoffmann Cc: Vivek Kasireddy Signed-off-by: Dongwon Kim --- drivers/gpu/drm/virtio/virtgpu_fence.c | 8

[PATCH 2/2][next] nouveau/svm: Split assignment from if conditional

2023-08-16 Thread Gustavo A. R. Silva
Fix checkpatch.pl ERROR: do not use assignment in if condition. Signed-off-by: Gustavo A. R. Silva --- drivers/gpu/drm/nouveau/nouveau_svm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_svm.c b/drivers/gpu/drm/nouveau/nouveau_svm.c index

[PATCH 1/2][next] nouveau/svm: Replace one-element array with flexible-array member in struct nouveau_svm

2023-08-16 Thread Gustavo A. R. Silva
One-element and zero-length arrays are deprecated. So, replace one-element array in struct nouveau_svm with flexible-array member. This results in no differences in binary output. Link: https://github.com/KSPP/linux/issues/338 Signed-off-by: Gustavo A. R. Silva ---

[PATCH 0/2][next] nouveau/svm: Replace one-element array with flexible-array member

2023-08-16 Thread Gustavo A. R. Silva
This small series aims to replace a one-element array with a flexible-array member in struct nouveau_svm. And, while at it, fix a checkpatch.pl error. Gustavo A. R. Silva (2): nouveau/svm: Replace one-element array with flexible-array member in struct nouveau_svm nouveau/svm: Split

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

2023-08-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=217664 --- Comment #12 from Alex Deucher (alexdeuc...@gmail.com) --- The display mux is a switch that controls what GPU the built in display is routed to (dGPU or APU). If it's not set correctly it can route the display to the wrong GPU. AFAIK, it

RE: Implement svm without BO concept in xe driver

2023-08-16 Thread Zeng, Oak
I spoke with Thomas. We discussed two approaches: 1) make ttm_resource a central place for vram management functions such as eviction, cgroup memory accounting. Both the BO-based driver and BO-less SVM codes call into ttm_resource_alloc/free functions for vram allocation/free. *This way BO

Re: [PATCH v2 00/12] drm/bridge: tc358768: Fixes and timings improvements

2023-08-16 Thread Tomi Valkeinen
On 16/08/2023 14:25, Tomi Valkeinen wrote: This series contains various fixes and cleanups for TC358768. The target of this work is to get TC358768 working on Toradex's AM62 based board, which has the following display pipeline: AM62 DPI -> TC358768 -> LT8912B -> HDMI connector The main thing

Re: [PATCH v3 0/5] Add JDI LPM102A188A display panel support

2023-08-16 Thread neil . armstrong
On 16/08/2023 18:30, Thierry Reding wrote: On Mon, Aug 07, 2023 at 02:33:00PM +0100, Diogo Ivo wrote: Hello, These patches add support for the JDI LPM102A188A display panel, found in the Google Pixel C. Patch 1 adds the DT bindings for the panel. Patch 2 adds the panel driver, which is based

Re: (subset) [PATCH v3 0/5] Add JDI LPM102A188A display panel support

2023-08-16 Thread Neil Armstrong
Hi, On Mon, 07 Aug 2023 14:33:00 +0100, Diogo Ivo wrote: > These patches add support for the JDI LPM102A188A display panel, > found in the Google Pixel C. > > Patch 1 adds the DT bindings for the panel. > > Patch 2 adds the panel driver, which is based on the downstream > kernel driver

Re: [PATCH v2] drm/panel: simple: Fix Innolux G156HCE-L01 LVDS clock

2023-08-16 Thread Neil Armstrong
Hi, On Mon, 14 Aug 2023 15:40:24 +0200, Luca Ceresoli wrote: > This panel has been implemented in commit 225213f24c79 ("drm/panel-simple: > Add Innolux G156HCE-L01 panel entry") with a higher clock than the typical > one mentioned on the documentation to avoid flickering on the unit > tested.

Re: [PATCH v2] drm/syncobj: fix DRM_SYNCOBJ_WAIT_FLAGS_WAIT_AVAILABLE

2023-08-16 Thread Erik Kurzinger
That's actually a bit of a tricky question. It seems that the existing IGT syncobj_timeline test asserts the incorrect behavior that waiting for an unsubmitted fence with WAIT_AVAILABLE set should fail with EINVAL. I've sent a patch to igt-dev correcting this

Re: [PATCH v3 2/5] drm/panel: Add driver for JDI LPM102A188A

2023-08-16 Thread neil . armstrong
On 07/08/2023 15:33, Diogo Ivo wrote: The JDI LPM102A188A is a 2560x1800 IPS panel found in the Google Pixel C. This driver is based on the downstream GPLv2 driver released by Google written by Sean Paul [1], which was then adapted to the newer kernel APIs. [1]:

Re: [PATCH v2] drm/syncobj: fix DRM_SYNCOBJ_WAIT_FLAGS_WAIT_AVAILABLE

2023-08-16 Thread Simon Ser
BTW, question: do you know if we could add an IGT test to make sure we don't regress this?

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

2023-08-16 Thread Danilo Krummrich
On 8/16/23 16:59, Christian König wrote: Am 16.08.23 um 14:30 schrieb Danilo Krummrich: On 8/16/23 16:05, Christian König wrote: Am 16.08.23 um 13:30 schrieb Danilo Krummrich: Hi Matt, On 8/11/23 04:31, Matthew Brost wrote: In XE, the new Intel GPU driver, a choice has made to have a 1 to 1

Re: [PATCH v3 0/5] Add JDI LPM102A188A display panel support

2023-08-16 Thread Thierry Reding
On Mon, Aug 07, 2023 at 02:33:00PM +0100, Diogo Ivo wrote: > Hello, > > These patches add support for the JDI LPM102A188A display panel, > found in the Google Pixel C. > > Patch 1 adds the DT bindings for the panel. > > Patch 2 adds the panel driver, which is based on the downstream > kernel

Re: (subset) [PATCH v3 0/5] Add JDI LPM102A188A display panel support

2023-08-16 Thread Thierry Reding
From: Thierry Reding On Mon, 07 Aug 2023 14:33:00 +0100, Diogo Ivo wrote: > These patches add support for the JDI LPM102A188A display panel, > found in the Google Pixel C. > > Patch 1 adds the DT bindings for the panel. > > Patch 2 adds the panel driver, which is based on the downstream >

[PATCH v2] drm/syncobj: fix DRM_SYNCOBJ_WAIT_FLAGS_WAIT_AVAILABLE

2023-08-16 Thread Erik Kurzinger
If DRM_IOCTL_SYNCOBJ_TIMELINE_WAIT is invoked with the DRM_SYNCOBJ_WAIT_FLAGS_WAIT_AVAILABLE flag set but no fence has yet been submitted for the given timeline point the call will fail immediately with EINVAL. This does not match the intended behavior where the call should wait until the fence

[Bug 201847] nouveau 0000:01:00.0: fifo: fault 01 [WRITE] at 000000000a721000 engine 00 [GR] client 0f [GPC0/PROP_0] reason 82 [] on channel 4 [00ff85c000 X[3819]]

2023-08-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=201847 Simon Fogliato (simonfogli...@gmail.com) changed: What|Removed |Added CC|

Re: [PATCH v2 09/15] drm/panthor: Add the FW logical block

2023-08-16 Thread Steven Price
On 09/08/2023 17:53, Boris Brezillon wrote: > Contains everything that's FW related, that includes the code dealing > with the microcontroller unit (MCU) that's running the FW, and anything > related to allocating memory shared between the FW and the CPU. > > A few global FW events are processed

RE: [PATCH -next] drm/amd/display: Simplify bool conversion

2023-08-16 Thread SHANMUGAM, SRINIVASAN
[AMD Official Use Only - General] Thanks Li, for the fix, the fix is already in process of merging into amd-staging-drm-next. https://patchwork.freedesktop.org/patch/552568/ -Original Message- From: amd-gfx On Behalf Of Yang Li Sent: Wednesday, August 16, 2023 6:16 AM To:

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

2023-08-16 Thread Danilo Krummrich
On 8/16/23 16:38, Matthew Brost wrote: On Wed, Aug 16, 2023 at 02:30:38PM +0200, Danilo Krummrich wrote: On 8/16/23 16:05, Christian König wrote: Am 16.08.23 um 13:30 schrieb Danilo Krummrich: Hi Matt, On 8/11/23 04:31, Matthew Brost wrote: In XE, the new Intel GPU driver, a choice has made

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

2023-08-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=217664 --- Comment #11 from Jose Roberto (colombo...@gmail.com) --- If I may: have you tried to disable (or even better, uninstall) tlp? Most of modern GNU/Linuxes come with it previously installed. -- You may reply to this email to add a comment.

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

2023-08-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=217664 --- Comment #10 from popus_czy_to_ty (pentelja...@o2.pl) --- aafter reinstalling and many times trying response for your questions is 1 (mux) whatever it is --> no 2 no 3 no, via command its dead also 4.no 5. insyde corp (bios manufacturer

[GIT PULL] etnaviv-next for 6.6

2023-08-16 Thread Lucas Stach
Hi Dave, Daniel, please pull the following etnaviv changes for the next merge window.  This time mostly cleanups around the runtime power management handling and slightly improved GPU hang handling. Also some additions to the HWDB to get the driver working properly on more NXP i.MX8MP IP cores.

Re: [PATCH] drm/nouveau/disp: fix use-after-free in error handling of nouveau_connector_create

2023-08-16 Thread Borislav Petkov
On Wed, Aug 16, 2023 at 04:57:28PM +0200, Karol Herbst wrote: > Do you have any connectors listed in "/sys/class/drm"? tree /sys/class/drm/ /sys/class/drm/ ├── card0 -> ../../devices/pci:00/:00:02.0/:03:00.0/drm/card0 ├── renderD128 ->

Re: [PATCH 1/5] mm: move some shrinker-related function declarations to mm/internal.h

2023-08-16 Thread kernel test robot
Hi Qi, kernel test robot noticed the following build warnings: [auto build test WARNING on brauner-vfs/vfs.all] [also build test WARNING on linus/master v6.5-rc6 next-20230816] [cannot apply to akpm-mm/mm-everything drm-misc/drm-misc-next vfs-idmapping/for-next] [If your patch is applied

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

2023-08-16 Thread Christian König
Am 16.08.23 um 14:30 schrieb Danilo Krummrich: On 8/16/23 16:05, Christian König wrote: Am 16.08.23 um 13:30 schrieb Danilo Krummrich: Hi Matt, On 8/11/23 04:31, Matthew Brost wrote: In XE, the new Intel GPU driver, a choice has made to have a 1 to 1 mapping between a drm_gpu_scheduler and

Re: [PATCH] drm/nouveau/disp: fix use-after-free in error handling of nouveau_connector_create

2023-08-16 Thread Karol Herbst
On Wed, Aug 16, 2023 at 4:54 PM Borislav Petkov wrote: > > On Wed, Aug 16, 2023 at 11:51:50AM +0200, Karol Herbst wrote: > > Mind sharing your kernel logs with that patch applied? I suspect your > > system boots up but you might just not have the connector available or > > something? It could be

Re: [PATCH] drm/nouveau/disp: fix use-after-free in error handling of nouveau_connector_create

2023-08-16 Thread Borislav Petkov
On Wed, Aug 16, 2023 at 11:51:50AM +0200, Karol Herbst wrote: > Mind sharing your kernel logs with that patch applied? I suspect your > system boots up but you might just not have the connector available or > something? It could be that you have one of those GPUs affected by the > original change

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

2023-08-16 Thread Matthew Brost
On Wed, Aug 16, 2023 at 02:30:38PM +0200, Danilo Krummrich wrote: > On 8/16/23 16:05, Christian König wrote: > > Am 16.08.23 um 13:30 schrieb Danilo Krummrich: > > > Hi Matt, > > > > > > On 8/11/23 04:31, Matthew Brost wrote: > > > > In XE, the new Intel GPU driver, a choice has made to have a 1

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

2023-08-16 Thread Danilo Krummrich
On 8/16/23 16:05, Christian König wrote: Am 16.08.23 um 13:30 schrieb Danilo Krummrich: Hi Matt, On 8/11/23 04:31, Matthew Brost wrote: In XE, the new Intel GPU driver, a choice has made to have a 1 to 1 mapping between a drm_gpu_scheduler and drm_sched_entity. At first this seems a bit odd

[Bug 217797] [amdgpu/mm?] HSA_AMD_SVM=y causes/triggers PAT issues

2023-08-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=217797 Artem S. Tashkinov (a...@gmx.com) changed: What|Removed |Added Status|NEW |RESOLVED

Re: [PATCH v2] drm/amdgpu: register a dirty framebuffer callback for fbcon

2023-08-16 Thread Christian König
Am 16.08.23 um 15:41 schrieb Hamza Mahfooz: On 8/16/23 01:55, Christian König wrote: Am 15.08.23 um 19:26 schrieb Hamza Mahfooz: fbcon requires that we implement _framebuffer_funcs.dirty. Otherwise, the framebuffer might take a while to flush (which would manifest as noticeable lag).

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

2023-08-16 Thread Christian König
Am 16.08.23 um 13:30 schrieb Danilo Krummrich: Hi Matt, On 8/11/23 04:31, Matthew Brost wrote: In XE, the new Intel GPU driver, a choice has made to have a 1 to 1 mapping between a drm_gpu_scheduler and drm_sched_entity. At first this seems a bit odd but let us explain the reasoning below. 1.

Re: [PATCH 1/5] mm: move some shrinker-related function declarations to mm/internal.h

2023-08-16 Thread kernel test robot
Hi Qi, kernel test robot noticed the following build warnings: [auto build test WARNING on brauner-vfs/vfs.all] [also build test WARNING on linus/master v6.5-rc6 next-20230816] [cannot apply to akpm-mm/mm-everything drm-misc/drm-misc-next vfs-idmapping/for-next] [If your patch is applied

Re: [PATCH v2] drm/amdgpu: register a dirty framebuffer callback for fbcon

2023-08-16 Thread Hamza Mahfooz
On 8/16/23 01:55, Christian König wrote: Am 15.08.23 um 19:26 schrieb Hamza Mahfooz: fbcon requires that we implement _framebuffer_funcs.dirty. Otherwise, the framebuffer might take a while to flush (which would manifest as noticeable lag). However, we can't enable this callback for

Re: [PATCH] Documentation/gpu: Update amdgpu documentation

2023-08-16 Thread Deucher, Alexander
[AMD Official Use Only - General] Reviewed-by: Alex Deucher From: Lazar, Lijo Sent: Tuesday, August 15, 2023 11:56 PM To: amd-...@lists.freedesktop.org Cc: Zhang, Hawking ; Deucher, Alexander ; s...@canb.auug.org.au ; airl...@redhat.com ;

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

2023-08-16 Thread Danilo Krummrich
Hi Matt, On 8/11/23 04:31, Matthew Brost wrote: In XE, the new Intel GPU driver, a choice has made to have a 1 to 1 mapping between a drm_gpu_scheduler and drm_sched_entity. At first this seems a bit odd but let us explain the reasoning below. 1. In XE the submission order from multiple

Re: [PATCH 1/5] mm: move some shrinker-related function declarations to mm/internal.h

2023-08-16 Thread kernel test robot
Hi Qi, kernel test robot noticed the following build warnings: [auto build test WARNING on brauner-vfs/vfs.all] [also build test WARNING on linus/master v6.5-rc6 next-20230816] [cannot apply to akpm-mm/mm-everything drm-misc/drm-misc-next vfs-idmapping/for-next] [If your patch is applied

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

2023-08-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=217664 Alex Deucher (alexdeuc...@gmail.com) changed: What|Removed |Added CC|

[PATCH v2 10/12] drm/bridge: tc358768: Fix tc358768_ns_to_cnt()

2023-08-16 Thread Tomi Valkeinen
The tc358768_ns_to_cnt() is, most likely, supposed to do a div-round-up operation, but it misses subtracting one from the dividend. Fix this by just using DIV_ROUND_UP(). Fixes: ff1ca6397b1d ("drm/bridge: Add tc358768 driver") Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/bridge/tc358768.c

[PATCH v2 09/12] drm/bridge: tc358768: Clean up clock period code

2023-08-16 Thread Tomi Valkeinen
The driver defines TC358768_PRECISION as 1000, and uses "nsk" to refer to clock periods. The original author does not remember where all this came from. Effectively the driver is using picoseconds as the unit for clock periods, yet referring to them by "nsk". Clean this up by just saying the

[PATCH v2 12/12] drm/bridge: tc358768: Default to positive h/v syncs

2023-08-16 Thread Tomi Valkeinen
As the TC358768 is a DPI to DSI bridge, the DSI side does not need to define h/v sync polarities. This means that sometimes we have a mode without defined sync polarities, which does not work on the DPI side. Add a mode_fixup hook to default to positive sync polarities. Signed-off-by: Tomi

[PATCH v2 11/12] drm/bridge: tc358768: Attempt to fix DSI horizontal timings

2023-08-16 Thread Tomi Valkeinen
The DSI horizontal timing calculations done by the driver seem to often lead to underflows or overflows, depending on the videomode. There are two main things the current driver doesn't seem to get right: DSI HSW and HFP, and VSDly. However, even following Toshiba's documentation it seems we

[PATCH v2 05/12] drm/bridge: tc358768: Use struct videomode

2023-08-16 Thread Tomi Valkeinen
The TC358768 documentation uses HFP, HBP, etc. values to deal with the video mode, while the driver currently uses the DRM display mode (htotal, hsync_start, etc). Change the driver to convert the DRM display mode to struct videomode, which then allows us to use the same units the documentation

[PATCH v2 08/12] drm/bridge: tc358768: Rename dsibclk to hsbyteclk

2023-08-16 Thread Tomi Valkeinen
The Toshiba documentation talks about HSByteClk when referring to the DSI HS byte clock, whereas the driver uses 'dsibclk' name. Also, in a few places the driver calculates the byte clock from the DSI clock, even if the byte clock is already available in a variable. To align the driver with the

[PATCH v2 04/12] drm/bridge: tc358768: Cleanup PLL calculations

2023-08-16 Thread Tomi Valkeinen
As is quite common, some of TC358768's PLL register fields are to be programmed with (value - 1). Specifically, the FBD and PRD, multiplier and divider, are such fields. However, what the driver currently does is that it considers that the formula used for PLL rate calculation is: RefClk * [(FBD

[PATCH v2 06/12] drm/bridge: tc358768: Print logical values, not raw register values

2023-08-16 Thread Tomi Valkeinen
The driver debug prints DSI related timings as raw register values in hex. It is much more useful to see the "logical" value of the timing, not the register value. Change the prints to print the values separately, in case a single register contains multiple values, and use %u to have it in a more

[PATCH v2 07/12] drm/bridge: tc358768: Use dev for dbg prints, not priv->dev

2023-08-16 Thread Tomi Valkeinen
Simplify the code by capturing the priv->dev value to dev variable, and use it. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/bridge/tc358768.c | 41 --- 1 file changed, 21 insertions(+), 20 deletions(-) diff --git a/drivers/gpu/drm/bridge/tc358768.c

[PATCH v2 03/12] drm/bridge: tc358768: Fix bit updates

2023-08-16 Thread Tomi Valkeinen
The driver has a few places where it does: if (thing_is_enabled_in_config) update_thing_bit_in_hw() This means that if the thing is _not_ enabled, the bit never gets cleared. This affects the h/vsyncs and continuous DSI clock bits. Fix the driver to always update the bit. Fixes:

[PATCH v2 02/12] drm/bridge: tc358768: Fix use of uninitialized variable

2023-08-16 Thread Tomi Valkeinen
smatch reports: drivers/gpu/drm/bridge/tc358768.c:223 tc358768_update_bits() error: uninitialized symbol 'orig'. Fix this by bailing out from tc358768_update_bits() if the tc358768_read() produces an error. Fixes: ff1ca6397b1d ("drm/bridge: Add tc358768 driver") Signed-off-by: Tomi Valkeinen

[PATCH v2 01/12] drm/tegra: rgb: Parameterize V- and H-sync polarities

2023-08-16 Thread Tomi Valkeinen
From: Thierry Reding The polarities of the V- and H-sync signals are encoded as flags in the display mode, so use the existing information to setup the signals for the RGB interface. Signed-off-by: Thierry Reding Cc: Thierry Reding [tomi.valkei...@ideasonboard.com: default to positive sync]

[PATCH v2 00/12] drm/bridge: tc358768: Fixes and timings improvements

2023-08-16 Thread Tomi Valkeinen
This series contains various fixes and cleanups for TC358768. The target of this work is to get TC358768 working on Toradex's AM62 based board, which has the following display pipeline: AM62 DPI -> TC358768 -> LT8912B -> HDMI connector The main thing the series does is to improve the DSI HSW,

[PATCH] drm: renesas: rcar-du: use proper naming for R-Car

2023-08-16 Thread Wolfram Sang
Not RCAR, but R-Car. Signed-off-by: Wolfram Sang --- drivers/gpu/drm/renesas/rcar-du/rcar_du_plane.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/renesas/rcar-du/rcar_du_plane.h b/drivers/gpu/drm/renesas/rcar-du/rcar_du_plane.h index

Re: [PATCH v5 09/17] drm/imagination: Implement power management

2023-08-16 Thread Paul Cercueil
Hi Sarah, Le mercredi 16 août 2023 à 09:25 +0100, Sarah Walker a écrit : > Add power management to the driver, using runtime pm. The power off > sequence depends on firmware commands which are not implemented in > this > patch. > > Changes since v4: > - Suspend runtime PM before unplugging

[PATCH] drm/ttm/tests: Fix type conversion in ttm_pool_test

2023-08-16 Thread Karolina Stolarek
Fix a warning about casting an integer of different size in ttm_pool_alloc_basic_dma_addr() subtest. Cast the DMA address to uintptr_t before casting it to a generic pointer. Signed-off-by: Karolina Stolarek Cc: Christian König Reported-by: kernel test robot Closes:

[PATCH v3 28/41] drm: renesas: shmobile: Rename shmob_drm_connector.connector

2023-08-16 Thread Geert Uytterhoeven
Rename the "connector" member of the shmob_drm_connector subclass structure to "base", to improve readability. Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart Reviewed-by: Sui Jingfeng --- v3: - No changes, v2: - Add Reviewed-by. ---

[PATCH v3 17/41] drm: renesas: shmobile: Convert to use devm_request_irq()

2023-08-16 Thread Geert Uytterhoeven
Convert to managed IRQ handling, to simplify cleanup. Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart --- v3: - No changes, v2: - Add Reviewed-by. --- drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff

[PATCH v3 00/41] drm: renesas: shmobile: Atomic conversion + DT support

2023-08-16 Thread Geert Uytterhoeven
Hi all, It has been 3 years since the last conversion of a DRM driver to atomic modesetting, so I guess it's time for another one? ;-) Currently, there are two drivers for the LCD controller on Renesas SuperH-based and ARM-based SH-Mobile and R-Mobile SoCs: 1. sh_mobile_lcdcfb, using

[PATCH v3 12/41] drm: renesas: shmobile: Remove backlight support

2023-08-16 Thread Geert Uytterhoeven
From: Laurent Pinchart Backlight support should be implemented by panels, not by the LCDC driver. As the feature is currently unused anyway, remove it. Signed-off-by: Laurent Pinchart [geert: Cleanups] Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart --- v3: - No changes,

[PATCH v3 19/41] drm: renesas: shmobile: Use drmm_universal_plane_alloc()

2023-08-16 Thread Geert Uytterhoeven
According to the comments for drm_universal_plane_init(), the plane structure should not be allocated with devm_kzalloc(). Fix lifetime issues by using drmm_universal_plane_alloc() instead. Signed-off-by: Geert Uytterhoeven --- v3: - No changes, v2: - Split off removal of call to

[PATCH v3 05/41] drm: renesas: shmobile: Fix ARGB32 overlay format typo

2023-08-16 Thread Geert Uytterhoeven
When configuring a CHn Source Image Format Register (LDBBSIFR), one should use the corresponding LDBBSIFR_RPKF_* definition for overlay planes, not the DDFR_PKF_* definition for the primary plane. Fortunately both definitions resolve to the same value, so this bug did not cause any harm.

[PATCH v3 03/41] media: uapi: Add MEDIA_BUS_FMT_RGB666_2X9_BE format

2023-08-16 Thread Geert Uytterhoeven
Add the RGB666 9:9 format MEDIA_BUS_FMT_RGB666_2X9_BE, which is supported by the SH-Mobile LCD Controller. Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart --- Cc: Mauro Carvalho Chehab Cc: Hans Verkuil Cc: linux-me...@vger.kernel.org v3: - No changes, v2: - Add

[PATCH v3 27/41] drm: renesas: shmobile: Rename shmob_drm_crtc.crtc

2023-08-16 Thread Geert Uytterhoeven
Rename the "crtc" member of the shmob_drm_crtc subclass structure to "base", to improve readability. Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart Reviewed-by: Sui Jingfeng --- v3: - No changes, v2: - Add Reviewed-by. --- .../gpu/drm/renesas/shmobile/shmob_drm_crtc.c |

[PATCH v3 26/41] drm: renesas: shmobile: Unify plane allocation

2023-08-16 Thread Geert Uytterhoeven
Unify primary and overlay plane allocation: - Enhance shmob_drm_plane_create() so it can be used to create the primary plane, too, - Move overlay plane creation next to primary plane creation. Signed-off-by: Geert Uytterhoeven --- v3: - No changes, v2: - Pass plane type to

[PATCH v3 34/41] drm: renesas: shmobile: Shutdown the display on remove

2023-08-16 Thread Geert Uytterhoeven
When the device is unbound from the driver, the display may be active. Make sure it gets shut down. Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart --- v3: - No changes, v2: - Add Reviewed-by. --- drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c | 2 ++ 1 file changed, 2

[PATCH v3 02/41] dt-bindings: display: Add Renesas SH-Mobile LCDC bindings

2023-08-16 Thread Geert Uytterhoeven
Add device tree bindings for the LCD Controller (LCDC) found in Renesas SuperH SH-Mobile and ARM SH/R-Mobile SOCs. Based on a plain text prototype by Laurent Pinchart. Signed-off-by: Geert Uytterhoeven Reviewed-by: Rob Herring --- Cc: Rob Herring Cc: Krzysztof Kozlowski Cc: Conor Dooley Cc:

[PATCH v3 18/41] drm: renesas: shmobile: Remove custom plane destroy callback

2023-08-16 Thread Geert Uytterhoeven
There is no need to call drm_plane_force_disable() from the plane's .destroy() callback, as the plane should have been disabled already before. See also commit 3c858a33858baa8c ("drm/plane_helper: don't disable plane in destroy function") for the generic plane helper case. After removing this

[PATCH v3 23/41] drm: renesas: shmobile: Use struct videomode in platform data

2023-08-16 Thread Geert Uytterhoeven
From: Laurent Pinchart Replace the drm_mode_modeinfo field with videomode that includes more signal polarity flags. This simplifies driver handling of panel modes and prepares for DT support. Signed-off-by: Laurent Pinchart [geert: Simplify] Signed-off-by: Geert Uytterhoeven Reviewed-by:

[PATCH v3 07/41] drm: renesas: shmobile: Add support for Runtime PM

2023-08-16 Thread Geert Uytterhoeven
The SH-Mobile LCD Controller is part of a PM Domain on all relevant SoCs (clock domain on all, power domain on some). Hence it may not be sufficient to manage the LCDC module clock explicitly (e.g. if the selected clock source differs from SHMOB_DRM_CLK_BUS). Fix this by using Runtime PM for all

[PATCH v3 14/41] drm: renesas: shmobile: Rename input clocks

2023-08-16 Thread Geert Uytterhoeven
From: Laurent Pinchart Prepare for DT bindings by using more appropriate names for the input clocks. Note that all LDDCKR_ICKSEL_* definitions but the one for the bus clock are valid only for SH7724, so the clock selection code needs to be updated when extending clock support to other SoCs.

[PATCH v3 29/41] drm: renesas: shmobile: Rename shmob_drm_plane.plane

2023-08-16 Thread Geert Uytterhoeven
Rename the "plane" member of the shmob_drm_plane subclass structure to "base", to improve readability. Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart Reviewed-by: Sui Jingfeng --- v3: - No changes, v2: - Add Reviewed-by. ---

[PATCH v3 16/41] drm: renesas: shmobile: Improve error handling

2023-08-16 Thread Geert Uytterhoeven
Prepare for DT conversion, where panel probe can be deferred, by streamlining error propagation and handling: - Use dev_err_probe() to avoid printing error messages in case of probe deferral, - Propagate errors where needed. Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart

[PATCH v3 21/41] drm: renesas: shmobile: Convert container helpers to static inline functions

2023-08-16 Thread Geert Uytterhoeven
Replace to conversion helper macros using container_of() by static inline functions, to improve type-safety. Signed-off-by: Geert Uytterhoeven Reviewed-by: Laurent Pinchart --- v3: - No changes, v2: - Add Reviewed-by. --- drivers/gpu/drm/renesas/shmobile/shmob_drm_crtc.c | 11 ---

  1   2   >