[PATCH 1/1] MAINTAINERS: Remove myself

2025-02-14 Thread Karol Herbst
here I simply couldn't continue the work I was so motivated doing as I've did in the early days. Please respect my wishes and put this statement as is into the tree. Leaving anything out destroys its entire meaning. Respectfully Karol Signed-off-by: Karol Herbst --- MAINT

[PATCH 0/1] Me stepping down as a nouveau kernel maintainer

2025-02-14 Thread Karol Herbst
y the best of luck and all the strength needed. Karol Herbst (1): MAINTAINERS: Remove myself MAINTAINERS | 2 -- 1 file changed, 2 deletions(-) -- 2.48.1

Re: [PATCH 1/2] gpu: nova-core: add initial driver stub

2025-02-01 Thread Karol Herbst
On Fri, Jan 31, 2025 at 11:04 PM Danilo Krummrich wrote: > > Add the initial nova-core driver stub. > > nova-core is intended to serve as a common base for nova-drm (the > corresponding DRM driver) and the vGPU manager VFIO driver, serving as a > hard- and firmware abstraction layer for GSP-based

Re: [PATCH 1/2] gpu: nova-core: add initial driver stub

2025-02-01 Thread Karol Herbst
On Sat, Feb 1, 2025 at 9:14 AM Karol Herbst wrote: > > On Fri, Jan 31, 2025 at 11:04 PM Danilo Krummrich wrote: > > > > Add the initial nova-core driver stub. > > > > nova-core is intended to serve as a common base for nova-drm (the > > corresponding DRM driv

Re: [PATCH v3 2/2] drm/nouveau: Add drm_panic support for nv50+

2024-10-21 Thread Karol Herbst
On Fri, Oct 18, 2024 at 11:50 PM Lyude Paul wrote: > > On Fri, 2024-09-13 at 09:03 +0200, Jocelyn Falempe wrote: > > Add drm_panic support, for nv50+ cards. > > It's enough to get the panic screen while running Gnome/Wayland on a > > GTX 1650. > > It doesn't support multi-plane or compressed forma

Re: [PATCH] drm/edid: add a quirk for two 240Hz Samsung monitors

2024-07-17 Thread Karol Herbst
On Thu, Nov 2, 2023 at 8:06 PM Alex Deucher wrote: > > On Thu, Nov 2, 2023 at 3:00 PM Hamza Mahfooz wrote: > > > > On 11/1/23 17:36, Alex Deucher wrote: > > > On Wed, Nov 1, 2023 at 5:01 PM Hamza Mahfooz > > > wrote: > > >> > > >> Without this fix the 5120x1440@240 timing of these monitors > >

Re: [PATCH] drm/nouveau/i2c: rename aux.c and aux.h to nvkm_i2c_aux.c and nvkm_i2c_aux.h

2024-06-03 Thread Karol Herbst
roE132r01M9mxZU5bk4M4uMO38l > HwZjS8YTSOmTA49PcLYbh+hr+e+GVFHP+X1wgmd/93wqlySRgqM7N1fh84jucg03IMS > D/tvbePTzg== > Content-Transfer-Encoding: quoted-printable > > 2024. 06. 02. 0:12 keltez=C3=A9ssel, Karol Herbst =C3=ADrta: > > On Sat, Jun 1, 2024 at 11:33=E2=80

Re: [PATCH] drm/nouveau/i2c: rename aux.c and aux.h to nvkm_i2c_aux.c and nvkm_i2c_aux.h

2024-06-01 Thread Karol Herbst
On Sat, Jun 1, 2024 at 11:33 PM wrote: > > s=20181004; d=freemail.hu; > > h=From:To:Cc:Subject:Date:Message-ID:MIME-Version:Content-Type:Content-Transfer-Encoding; > l=6462; bh=CnHSqkRX34kqCWN2/oPtvm9wm8HJCe8QhPwQypdpi3w=; > b=JDlDB9yOQDBY7EDnRMrt8qYnWfH8J/LKYKb904

[PATCH v2 2/2] drm/nouveau: move more missing UAPI bits

2024-03-05 Thread Karol Herbst
Those are already de-facto UAPI, so let's just move it into the uapi header. Signed-off-by: Karol Herbst Reviewed-by: Lyude Paul Reviewed-by: Danilo Krummrich Link: https://patchwork.freedesktop.org/patch/msgid/20240304183157.1587152-2-kher...@redhat.com --- drivers/gpu/drm/no

[PATCH v2 1/2] drm/nouveau: fix stale locked mutex in nouveau_gem_ioctl_pushbuf

2024-03-05 Thread Karol Herbst
Fixes: b88baab82871 ("drm/nouveau: implement new VM_BIND uAPI") Cc: Danilo Krummrich Cc: # v6.6+ Signed-off-by: Karol Herbst Reviewed-by: Lyude Paul Reviewed-by: Danilo Krummrich Link: https://patchwork.freedesktop.org/patch/msgid/20240304183157.1587152-1-kher...@redhat.com --- drivers/

[PATCH 2/2] drm/nouveau: move more missing UAPI bits

2024-03-04 Thread Karol Herbst
Those are already de-facto UAPI, so let's just move it into the uapi header. Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nouveau_abi16.c | 20 +++- drivers/gpu/drm/nouveau/nouveau_abi16.h | 12 include/uapi/drm/nouveau_drm.h

[PATCH 1/2] drm/nouveau: fix stale locked mutex in nouveau_gem_ioctl_pushbuf

2024-03-04 Thread Karol Herbst
Fixes: b88baab82871 ("drm/nouveau: implement new VM_BIND uAPI") Cc: Danilo Krummrich Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nouveau_gem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_gem.c b/drivers/gpu/drm/nouveau/nouv

[PATCH] drm/nouveau/disp: fix DP capable DSM connectors

2023-10-11 Thread Karol Herbst
/255 Fixes: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts") Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nvkm/engine/disp/uconn.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/n

Re: [PATCH 06/20] drm/nouveau/dispnv04/crtc: Demote kerneldoc abuses

2023-08-24 Thread Karol Herbst
kernel-doc.rst > drivers/gpu/drm/nouveau/dispnv04/crtc.c:629: warning: This comment starts > with '/**', but isn't a kernel-doc comment. Refer > Documentation/doc-guide/kernel-doc.rst > > Signed-off-by: Lee Jones > --- > Cc: Ben Skeggs > Cc: Karol Herbst &g

Re: [PATCH 05/20] drm/nouveau/nvkm/engine/gr/gf100: Demote kerneldoc abuse

2023-08-24 Thread Karol Herbst
c-guide/kernel-doc.rst > > Signed-off-by: Lee Jones > --- > Cc: Ben Skeggs > Cc: Karol Herbst > Cc: Lyude Paul > Cc: David Airlie > Cc: Daniel Vetter > Cc: dri-devel@lists.freedesktop.org > Cc: nouv...@lists.freedesktop.org > --- > drivers/gpu/drm/nouve

Re: [PATCH 04/20] drm/nouveau/nvkm/subdev/volt/gk20a: Demote kerneldoc abuses

2023-08-24 Thread Karol Herbst
c-guide/kernel-doc.rst > drivers/gpu/drm/nouveau/nvkm/subdev/volt/gk20a.c:62: warning: This comment > starts with '/**', but isn't a kernel-doc comment. Refer > Documentation/doc-guide/kernel-doc.rst > > Signed-off-by: Lee Jones > --- > Cc: Ben Skeggs > Cc:

Re: [PATCH 03/20] drm/nouveau/nvkm/subdev/bios/init: Demote a bunch of kernel-doc abuses

2023-08-24 Thread Karol Herbst
rm/nouveau/nvkm/subdev/bios/init.c:611: warning: expecting > prototype for INIT_DONE(). Prototype was for init_done() instead > [Snipped ~140 lines for brevity] > > Signed-off-by: Lee Jones > --- > Cc: Ben Skeggs > Cc: Karol Herbst > Cc: Lyude Paul > Cc: David Airlie

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

2023-08-17 Thread Karol Herbst
On Thu, Aug 17, 2023 at 12:11 PM Borislav Petkov wrote: > > On Thu, Aug 17, 2023 at 12:00:47PM +0200, Karol Herbst wrote: > > btw, what would help is to know where `nvkm_uconn_uevent` actually > > fails, or rather, are you running into this "/* TODO: support DP IRQ > &g

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

2023-08-17 Thread Karol Herbst
On Thu, Aug 17, 2023 at 11:58 AM Karol Herbst wrote: > > On Thu, Aug 17, 2023 at 10:10 AM Borislav Petkov wrote: > > > > On Thu, Aug 17, 2023 at 01:18:12AM +0200, Karol Herbst wrote: > > > do you have one of these? https://en.wikipedia.org/wiki/DMS-59 > > >

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

2023-08-17 Thread Karol Herbst
On Thu, Aug 17, 2023 at 10:10 AM Borislav Petkov wrote: > > On Thu, Aug 17, 2023 at 01:18:12AM +0200, Karol Herbst wrote: > > do you have one of these? https://en.wikipedia.org/wiki/DMS-59 > > Ah, DMS == Dual Monitor Solution :-) > > Yap, that's exactly what the GPU

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.wiki

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 -> ../../devices/pci

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 > >

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 11:30 AM Borislav Petkov wrote: > > On Mon, Aug 14, 2023 at 04:49:32PM +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

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Mon, Aug 14, 2023 at 4:19 PM Karol Herbst wrote: > > On Mon, Aug 14, 2023 at 3:35 PM Takashi Iwai wrote: > > > > On Mon, 14 Aug 2023 15:19:11 +0200, > > Karol Herbst wrote: > > > > > > On Mon, Aug 14, 2023 at 2:56 PM Karol Herbst wrote: > > >

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

2023-08-14 Thread Karol Herbst
uption in error handling related to that commit. Link: https://lore.kernel.org/lkml/20230806213107.GFZNARG6moWpFuSJ9W@fat_crate.local/ Fixes: 95983aea8003 ("drm/nouveau/disp: add connector class") Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nouveau_connector.c | 11 +

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Mon, Aug 14, 2023 at 3:35 PM Takashi Iwai wrote: > > On Mon, 14 Aug 2023 15:19:11 +0200, > Karol Herbst wrote: > > > > On Mon, Aug 14, 2023 at 2:56 PM Karol Herbst wrote: > > > > > > On Mon, Aug 14, 2023 at 2:48 PM Takashi Iwai wrote: > > &

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Tue, Aug 8, 2023 at 3:47 PM Borislav Petkov wrote: > > On Tue, Aug 08, 2023 at 12:39:32PM +0200, Karol Herbst wrote: > > ahh, that would have been good to know :) > > Yeah, I didn't see it before - it would only freeze. Only after I added > the printk you requeste

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Mon, Aug 14, 2023 at 2:56 PM Karol Herbst wrote: > > On Mon, Aug 14, 2023 at 2:48 PM Takashi Iwai wrote: > > > > On Mon, 14 Aug 2023 14:38:18 +0200, > > Karol Herbst wrote: > > > > > > On Wed, Aug 9, 2023 at 6:16 PM Takashi Iwai wrote: > >

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Mon, Aug 14, 2023 at 2:48 PM Takashi Iwai wrote: > > On Mon, 14 Aug 2023 14:38:18 +0200, > Karol Herbst wrote: > > > > On Wed, Aug 9, 2023 at 6:16 PM Takashi Iwai wrote: > > > > > > On Wed, 09 Aug 2023 16:46:38 +0200, > > > Takashi Iwai wro

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Wed, Aug 9, 2023 at 6:16 PM Takashi Iwai wrote: > > On Wed, 09 Aug 2023 16:46:38 +0200, > Takashi Iwai wrote: > > > > On Wed, 09 Aug 2023 15:13:23 +0200, > > Takashi Iwai wrote: > > > > > > On Wed, 09 Aug 2023 14:19:23 +0200, > > > Karol

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Karol Herbst
On Wed, Aug 9, 2023 at 8:28 PM Karol Herbst wrote: > > On Wed, Aug 9, 2023 at 4:04 PM Thorsten Leemhuis > wrote: > > > > On 09.08.23 15:13, Takashi Iwai wrote: > > > > > > If this can't be fixed quickly, I suppose it's safer to revert it from >

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Karol Herbst
On Wed, Aug 9, 2023 at 4:04 PM Thorsten Leemhuis wrote: > > On 09.08.23 15:13, Takashi Iwai wrote: > > > > If this can't be fixed quickly, I suppose it's safer to revert it from > > 6.4.y for now. 6.5 is still being cooked, but 6.4.x is already in > > wide deployment, hence the regression has to

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Karol Herbst
On Wed, Aug 9, 2023 at 1:46 PM Takashi Iwai wrote: > > On Wed, 09 Aug 2023 13:42:09 +0200, > Karol Herbst wrote: > > > > On Wed, Aug 9, 2023 at 11:22 AM Takashi Iwai wrote: > > > > > > On Tue, 08 Aug 2023 12:39:32 +0200, > > > Karol Herbst wrote

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Karol Herbst
On Wed, Aug 9, 2023 at 11:22 AM Takashi Iwai wrote: > > On Tue, 08 Aug 2023 12:39:32 +0200, > Karol Herbst wrote: > > > > On Mon, Aug 7, 2023 at 5:05 PM Borislav Petkov wrote: > > > > > > On Mon, Aug 07, 2023 at 01:49:42PM +0200, Karol Herbst wrote: >

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-08 Thread Karol Herbst
On Mon, Aug 7, 2023 at 5:05 PM Borislav Petkov wrote: > > On Mon, Aug 07, 2023 at 01:49:42PM +0200, Karol Herbst wrote: > > in what way does it stop? Just not progressing? That would be kinda > > concerning. Mind tracing with what arguments `nvkm_uevent_add` is > > call

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-07 Thread Karol Herbst
On Sun, Aug 6, 2023 at 11:40 PM Borislav Petkov wrote: > > Hi folks, > > the patch in $Subject breaks booting here on one of my test boxes, see > below. > > Reverting it ontop of -rc4 fixes the issue. > > Thx. > > [3.580535] ACPI: \_PR_.CP04: Found 4 idle states > [3.585694] ACPI: \_PR_.CP

Re: [PATCH drm-misc-next] nouveau/dmem: fix copy-paste error in nouveau_dmem_migrate_chunk()

2023-08-05 Thread Karol Herbst
nouveau_fence_emit(fence, drm->dmem->migrate.chan); > migrate_vma_pages(args); > nouveau_dmem_fence_done(&fence); > nouveau_pfns_map(svmm, args->vma->vm_mm, args->start, pfns, i); > > base-commit: 82d750e9d2f5d0594c8f7057ce59127e701af781 > -- > 2.41.0 > Reviewed-by: Karol Herbst

[PATCH] drm/nouveau/disp: Revert a NULL check inside nouveau_connector_get_modes

2023-08-05 Thread Karol Herbst
Paul Closes: https://gitlab.freedesktop.org/drm/nouveau/-/issues/238 Closes: https://gitlab.freedesktop.org/drm/nouveau/-/issues/245 Fixes: 20a2ce87fbaf8 ("drm/nouveau/dp: check for NULL nv_connector->native_mode") Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nouveau_co

Re: nouveau bug in linux/6.1.38-2

2023-08-04 Thread Karol Herbst
On Fri, Aug 4, 2023 at 8:10 PM Olaf Skibbe wrote: > > Dear all, > > On Fri, 4 Aug 2023 at 14:15, Karol Herbst wrote: > > >>> 62aecf23f3d1 drm/nouveau: add nv_encoder pointer check for NULL > >>> fb725beca62d drm/nouveau/dp: check for NULL nv_connector->

Re: nouveau bug in linux/6.1.38-2

2023-08-04 Thread Karol Herbst
On Fri, Aug 4, 2023 at 2:48 PM Olaf Skibbe wrote: > > On Fri, 4 Aug 2023 at 14:15, Karol Herbst wrote: > > > mind retrying with only fb725beca62d and 62aecf23f3d1 reverted? > > I will do this later this day (takes some time, it is a slow machine). > > > Would be weir

Re: nouveau bug in linux/6.1.38-2

2023-08-04 Thread Karol Herbst
On Fri, Aug 4, 2023 at 2:02 PM Thorsten Leemhuis wrote: > > Hi! > > On 02.08.23 23:28, Olaf Skibbe wrote: > > Dear Maintainers, > > > > Hereby I would like to report an apparent bug in the nouveau driver in > > linux/6.1.38-2. > > Thx for your report. Maybe your problem is caused by a incomplete >

Re: [PATCH] [RESEND] drm/nouveau: remove unused tu102_gr_load() function

2023-08-03 Thread Karol Herbst
t; - return gk20a_gr_load_net(gr, "gr/", "sw_veid_bundle_init", ver, > tu102_gr_av_to_init_veid, > -&gr->bundle_veid); > -} > - > static const struct gf100_gr_fwif > tu102_gr_fwif[] = { > { 0, gm200_gr_load, &tu102_gr, &gp108_gr_fecs_acr, > &gp108_gr_gpccs_acr }, > -- > 2.39.2 > Reviewed-by: Karol Herbst Will push to drm-misc in a sec

Re: [PATCH] drm/nouveau: fixup the uapi header file.

2023-08-01 Thread Karol Herbst
On Tue, Aug 1, 2023 at 5:15 PM Faith Ekstrand wrote: > On Tue, Aug 1, 2023 at 4:37 AM Karol Herbst wrote: > >> On Mon, Jul 31, 2023 at 9:16 PM Dave Airlie wrote: >> > >> > From: Dave Airlie >> > >> > nouveau > 10 years ago had a plan for new

Re: [PATCH] drm/nouveau: fixup the uapi header file.

2023-08-01 Thread Karol Herbst
) > +#define DRM_IOCTL_NOUVEAU_CHANNEL_ALLOC DRM_IOWR(DRM_COMMAND_BASE + > DRM_NOUVEAU_CHANNEL_ALLOC, struct drm_nouveau_channel_alloc) > +#define DRM_IOCTL_NOUVEAU_CHANNEL_FREE DRM_IOW (DRM_COMMAND_BASE + > DRM_NOUVEAU_CHANNEL_FREE, struct drm_nouveau_channel_free) > + > #define DRM_IOCTL_NOUVEAU_SVM_INIT DRM_IOWR(DRM_COMMAND_BASE + > DRM_NOUVEAU_SVM_INIT, struct drm_nouveau_svm_init) > #define DRM_IOCTL_NOUVEAU_SVM_BIND DRM_IOWR(DRM_COMMAND_BASE + > DRM_NOUVEAU_SVM_BIND, struct drm_nouveau_svm_bind) > > -- > 2.41.0 > Reviewed-by: Karol Herbst

Re: [PATCH] drm/nouveau/nvkm/dp: Add hack to fix DP 1.3+ DPCD issues

2023-07-28 Thread Karol Herbst
On Fri, Jul 28, 2023 at 1:11 AM Karol Herbst wrote: > > On Thu, Jul 27, 2023 at 11:57 PM Lyude Paul wrote: > > > > On Sun, 2023-07-09 at 01:42 +0200, Karol Herbst wrote: > > > On Fri, Jul 7, 2023 at 11:58 PM Lyude Paul wrote: > > > > > > > >

Re: [PATCH] drm/nouveau/nvkm/dp: Add hack to fix DP 1.3+ DPCD issues

2023-07-27 Thread Karol Herbst
On Thu, Jul 27, 2023 at 11:57 PM Lyude Paul wrote: > > On Sun, 2023-07-09 at 01:42 +0200, Karol Herbst wrote: > > On Fri, Jul 7, 2023 at 11:58 PM Lyude Paul wrote: > > > > > > Currently we use the drm_dp_dpcd_read_caps() helper in the DRM side of > > > no

Re: [PATCH] drm/nouveau/fifo:Fix Nineteen occurrences of the gk104.c error: ERROR: : trailing statements should be on next line

2023-07-15 Thread Karol Herbst
On Sat, Jul 15, 2023 at 1:07 AM Lyude Paul wrote: > > NAK - checkpatch.pl is a (strongish) guideline, but not a rule. In the cases > corrected in the patch series here, we format the switch cases on single lines > as it dramatically improves the readability of what is otherwise just a /long/ > lis

Re: [PATCH] drm/nouveau/iccsense:

2023-07-13 Thread Karol Herbst
On Thu, Jul 13, 2023 at 11:13 AM Karol Herbst wrote: > > On Mon, Jul 10, 2023 at 11:10 AM Bagas Sanjaya wrote: > > > > On Mon, Jul 10, 2023 at 03:06:47PM +0800, sunran...@208suo.com wrote: > > > Fixed error: 'do not use assignment in if condition' > >

Re: [PATCH] drm/nouveau/iccsense:

2023-07-13 Thread Karol Herbst
On Mon, Jul 10, 2023 at 11:10 AM Bagas Sanjaya wrote: > > On Mon, Jul 10, 2023 at 03:06:47PM +0800, sunran...@208suo.com wrote: > > Fixed error: 'do not use assignment in if condition' > > > > This patch fixes error: 'do not use assignment in if condition' > > in drm/nouveau/iccsense > > I guess t

Re: [PATCH] drm/nouveau/i2c: do not use assignment in if condition

2023-07-13 Thread Karol Herbst
L))) > +*paux = kzalloc(sizeof(**paux), GFP_KERNEL); > +if (!*paux) > return -ENOMEM; > return nvkm_i2c_aux_ctor(func, pad, id, *paux); > } > Reviewed-by: Karol Herbst

Re: [PATCH] drm/nouveau/nvkm: do not use assignment in if condition

2023-07-13 Thread Karol Herbst
if (!(pad = nvkm_i2c_pad_find(i2c, id))) > +pad = nvkm_i2c_pad_find(i2c, id); > +if (!pad) > ret = func->pad_s_new(i2c, id, &pad); > else > ret = 0; > Reviewed-by: Karol Herbst

Re: [PATCH] drm/nouveau/pci: ERROR: "foo * bar" should be "foo *bar"

2023-07-13 Thread Karol Herbst
t; } > > static int > -gk104_pcie_init(struct nvkm_pci * pci) > +gk104_pcie_init(struct nvkm_pci *pci) > { > enum nvkm_pcie_speed lnkctl_speed, max_speed, cap_speed; > struct nvkm_subdev *subdev = &pci->subdev; > Reviewed-by: Karol Herbst

Re: [PATCH] drm/nouveau/i2c: do not use assignment in if condition

2023-07-13 Thread Karol Herbst
Reviewed-by: Karol Herbst On Mon, Jul 10, 2023 at 9:23 AM wrote: > > Assignments in if condition are less readable and error-prone. Fixes > also checkpatch warning: > > ERROR: do not use assignment in if condition > > Signed-off-by: Ran Sun > --- > drivers/gpu/

Re: [PATCH] drm/nouveau/nvkm/dp: Add hack to fix DP 1.3+ DPCD issues

2023-07-08 Thread Karol Herbst
On Fri, Jul 7, 2023 at 11:58 PM Lyude Paul wrote: > > Currently we use the drm_dp_dpcd_read_caps() helper in the DRM side of > nouveau in order to read the DPCD of a DP connector, which makes sure we do > the right thing and also check for extended DPCD caps. However, it turns > out we're not curr

Re: [PATCH] drm/nouveau/disp/g94: enable HDMI

2023-07-07 Thread Karol Herbst
ured this one out a bit later. > On Fri, 2023-06-30 at 18:06 +0200, Karol Herbst wrote: > > Cc: Ben Skeggs > > Cc: Lyude Paul > > Fixes: f530bc60a30b ("drm/nouveau/disp: move HDMI config into acquire + > > infoframe methods") > > Signed-off-by: Karol H

[PATCH] drm/nouveau/disp/g94: enable HDMI

2023-06-30 Thread Karol Herbst
Cc: Ben Skeggs Cc: Lyude Paul Fixes: f530bc60a30b ("drm/nouveau/disp: move HDMI config into acquire + infoframe methods") Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nvkm/engine/disp/g94.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/nouveau/nvkm/e

[PATCH 3/3] drm/nouveau/disp: verify mode on atomic_check

2023-06-28 Thread Karol Herbst
one of my Tesla GPUs with a connected 4K display. Link: https://gitlab.freedesktop.org/drm/nouveau/-/issues/199 Cc: Ben Skeggs Cc: Lyude Paul Cc: sta...@vger.kernel.org # v6.1+ Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nouveau_connector.c | 19 +++ 1 file changed

[PATCH 2/3] drm/nouveau/disp: drop unused argument in nv50_dp_mode_valid

2023-06-28 Thread Karol Herbst
Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/dispnv50/disp.c | 2 +- drivers/gpu/drm/nouveau/nouveau_connector.c | 2 +- drivers/gpu/drm/nouveau/nouveau_dp.c| 3 +-- drivers/gpu/drm/nouveau/nouveau_encoder.h | 3 +-- 4 files changed, 4 insertions(+), 6 deletions(-) diff

[PATCH 1/3] drm/nouveau/disp: fix HDMI on gt215+

2023-06-28 Thread Karol Herbst
Cc: Ben Skeggs Cc: Lyude Paul Fixes: f530bc60a30b ("drm/nouveau/disp: move HDMI config into acquire + infoframe methods") Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nvkm/engine/disp/gt215.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drive

Re: [PATCH 3/9] drm/ttm: use per BO cleanup workers

2023-06-15 Thread Karol Herbst
On Thu, Jun 15, 2023 at 1:19 PM Christian König wrote: > > Am 13.06.23 um 16:18 schrieb Karol Herbst: > > On Tue, Jun 13, 2023 at 3:59 PM Christian König > > wrote: > >> Am 13.06.23 um 15:05 schrieb Karol Herbst: > >>> On Mon, Dec 5, 2022 at 2:40 PM C

Re: [PATCH] nouveau: fix client work fence deletion race

2023-06-15 Thread Karol Herbst
agree with the code being always broken as `dma_fence_is_signaled_locked` specifies: "This function requires &dma_fence.lock to be held." > + > + if (ret == true) > + dma_fence_put(fence); > + return ret; > } > > static void > -- > 2.40.1 > regardless of the patch fixing the issue users have seen: Reviewed-by: Karol Herbst

Re: [PATCH] drm/nouveau/kms/nv50-: Fix drm_dp_remove_payload() invocation

2023-06-13 Thread Karol Herbst
et's do that. > > Signed-off-by: Lyude Paul Reviewed-by: Karol Herbst > --- > drivers/gpu/drm/nouveau/dispnv50/disp.c | 8 ++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c > b/drivers/gpu/drm/nouveau

Re: [PATCH 3/9] drm/ttm: use per BO cleanup workers

2023-06-13 Thread Karol Herbst
On Tue, Jun 13, 2023 at 3:59 PM Christian König wrote: > > Am 13.06.23 um 15:05 schrieb Karol Herbst: > > On Mon, Dec 5, 2022 at 2:40 PM Christian König > > wrote: > >> Am 29.11.22 um 22:14 schrieb Felix Kuehling: > >>> On 2022-11-25 05:21, Christian

Re: [PATCH 3/9] drm/ttm: use per BO cleanup workers

2023-06-13 Thread Karol Herbst
On Mon, Dec 5, 2022 at 2:40 PM Christian König wrote: > > Am 29.11.22 um 22:14 schrieb Felix Kuehling: > > On 2022-11-25 05:21, Christian König wrote: > >> Instead of a single worker going over the list of delete BOs in regular > >> intervals use a per BO worker which blocks for the resv object an

Re: [RESEND 12/15] drm/nouveau/dispnv04/crtc: Demote kerneldoc abuses

2023-06-09 Thread Karol Herbst
kernel-doc.rst > drivers/gpu/drm/nouveau/dispnv04/crtc.c:629: warning: This comment starts > with '/**', but isn't a kernel-doc comment. Refer > Documentation/doc-guide/kernel-doc.rst > > Cc: Ben Skeggs > Cc: Karol Herbst > Cc: Lyude Paul > Cc: David Airli

Re: [RESEND 09/15] drm/nouveau/nvkm/engine/gr/gf100: Demote kerneldoc abuse

2023-06-09 Thread Karol Herbst
-guide/kernel-doc.rst > > Cc: Ben Skeggs > Cc: Karol Herbst > Cc: Lyude Paul > Cc: David Airlie > Cc: Daniel Vetter > Cc: dri-devel@lists.freedesktop.org > Cc: nouv...@lists.freedesktop.org > Signed-off-by: Lee Jones Reviewed-by: Karol Herbst > --- > drivers/

Re: [RESEND 10/15] drm/nouveau/nvkm/engine/gr/tu102: Staticify local function gf100_fifo_nonstall_block()

2023-06-09 Thread Karol Herbst
On Fri, Jun 9, 2023 at 10:18 AM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/nouveau/nvkm/engine/gr/tu102.c:210:1: warning: no previous > prototype for ‘tu102_gr_load’ [-Wmissing-prototypes] > > Cc: Ben Skeggs > Cc: Karol

Re: [RESEND 08/15] drm/nouveau/nvkm/subdev/volt/gk20a: Demote kerneldoc abuses

2023-06-09 Thread Karol Herbst
c-guide/kernel-doc.rst > drivers/gpu/drm/nouveau/nvkm/subdev/volt/gk20a.c:62: warning: This comment > starts with '/**', but isn't a kernel-doc comment. Refer > Documentation/doc-guide/kernel-doc.rst > > Cc: Ben Skeggs > Cc: Karol Herbst > Cc:

Re: [RESEND 06/15] drm/nouveau/nvkm/subdev/acr/lsfw: Remove unused variable 'loc'

2023-06-09 Thread Karol Herbst
: variable > ‘loc’ set but not used [-Wunused-but-set-variable] > > Cc: Ben Skeggs > Cc: Karol Herbst > Cc: Lyude Paul > Cc: David Airlie > Cc: Daniel Vetter > Cc: Gourav Samaiya > Cc: dri-devel@lists.freedesktop.org > Cc: nouv...@lists.freedesktop.org > Signed-

Re: [RESEND 07/15] drm/nouveau/nvkm/subdev/bios/init: Demote a bunch of kernel-doc abuses

2023-06-09 Thread Karol Herbst
rm/nouveau/nvkm/subdev/bios/init.c:611: warning: expecting > prototype for INIT_DONE(). Prototype was for init_done() instead > [Snipped ~140 lines for brevity] > > Cc: Ben Skeggs > Cc: Karol Herbst > Cc: Lyude Paul > Cc: David Airlie > Cc: Daniel Vetter > Cc: dr

Re: [Nouveau] [PATCH v2] drm/nouveau: bring back blit subchannel for pre nv50 GPUs

2023-06-06 Thread Karol Herbst
On Fri, May 26, 2023 at 5:21 PM Ilia Mirkin wrote: > > On Fri, May 26, 2023 at 5:11 AM Karol Herbst wrote: > > > > 1ba6113a90a0 removed a lot of the kernel GPU channel, but method 0x128 > > was important as otherwise the GPU spams us with `CACHE_ERROR` messages.

Re: [PATCH v2] drm/nouveau: don't detect DSM for non-NVIDIA device

2023-06-05 Thread Karol Herbst
c void nouveau_dsm_pci_probe(struct pci_dev *pdev, > acpi_handle *dhandle_out > int optimus_funcs; > struct pci_dev *parent_pdev; > > + if (pdev->vendor != PCI_VENDOR_ID_NVIDIA) > + return; > + > *has_pr3 = false; > parent_pdev = pci_upstr

[PATCH v2] drm/nouveau: bring back blit subchannel for pre nv50 GPUs

2023-05-26 Thread Karol Herbst
://gitlab.freedesktop.org/drm/nouveau/-/issues/201 Fixes: 4a16dd9d18a0 ("drm/nouveau/kms: switch to drm fbdev helpers") Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nouveau_chan.c | 1 + drivers/gpu/drm/nouveau/nouveau_chan.h | 1 + drivers/gpu/drm/nouveau/nouveau_drm.c | 20 +

Re: Fwd: absent both plymouth, and video= on linu lines, vtty[1-6] framebuffers produce vast raster right and bottom borders on the larger resolution of two displays

2023-05-25 Thread Karol Herbst
I wouldn't be surprised if 4a16dd9d18a0 is the culprit and that it was always broken like this with drm fbdev. Anyway, the logs don't indicate we are actually doing anything wrong, It's just that the tty handling is a little bonkers as always. On Thu, May 25, 2023 at 1:37 PM Bagas Sanjaya wrote:

Re: [PATCH] drm/nouveau/acr: Abort loading ACR if no firmware was found

2023-05-23 Thread Karol Herbst
On Mon, May 22, 2023 at 10:18 PM Karol Herbst wrote: > > This fixes a NULL pointer access inside nvkm_acr_oneinit in case necessary > firmware files couldn't be loaded. > > Closes: https://gitlab.freedesktop.org/drm/nouveau/-/issues/212 > Fixes: 4b569ded09fd ("dr

[PATCH] drm/nouveau/acr: Abort loading ACR if no firmware was found

2023-05-22 Thread Karol Herbst
This fixes a NULL pointer access inside nvkm_acr_oneinit in case necessary firmware files couldn't be loaded. Closes: https://gitlab.freedesktop.org/drm/nouveau/-/issues/212 Fixes: 4b569ded09fd ("drm/nouveau/acr/ga102: initial support") Signed-off-by: Karol Herbst --- drivers/

Re: [PATCH] drm/nouveau/acr/ga102: set variable ga102_gsps storage-class-specifier to static

2023-05-22 Thread Karol Herbst
Reviewed-by: Karol Herbst On Wed, May 17, 2023 at 3:31 PM Tom Rix wrote: > > smatch reports > drivers/gpu/drm/nouveau/nvkm/subdev/gsp/ga102.c:49:1: warning: symbol > 'ga102_gsps' was not declared. Should it be static? > > This variable is only used in its defining

Re: [PATCH] drm/nouveau: dispnv50: fix missing-prototypes warning

2023-05-22 Thread Karol Herbst
Reviewed-by: Karol Herbst will merge soon On Mon, Apr 17, 2023 at 11:03 PM Arnd Bergmann wrote: > > From: Arnd Bergmann > > nv50_display_create() is declared in another header, along with > a couple of declarations that are now outdated: > > drivers/gpu/drm/nouveau/d

[PATCH] drm/nouveau: bring back blit subchannel for pre nv50 GPUs

2023-05-15 Thread Karol Herbst
/201 Fixes: 4a16dd9d18a0 ("drm/nouveau/kms: switch to drm fbdev helpers") Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nouveau_chan.c | 1 + drivers/gpu/drm/nouveau/nouveau_chan.h | 1 + drivers/gpu/drm/nouveau/nouveau_drm.c | 19 --- 3 files changed, 18

Re: [RFC] Plane color pipeline KMS uAPI

2023-05-11 Thread Karol Herbst
On Wed, May 10, 2023 at 9:59 AM Jonas Ådahl wrote: > > On Tue, May 09, 2023 at 08:22:30PM +, Simon Ser wrote: > > On Tuesday, May 9th, 2023 at 21:53, Dave Airlie wrote: > > > > > There are also other vendor side effects to having this in userspace. > > > > > > Will the library have a loader?

Re: Disabling -Warray-bounds for gcc-13 too

2023-04-27 Thread Karol Herbst
On Fri, Apr 28, 2023 at 1:27 AM Lyude Paul wrote: > > On Fri, 2023-04-28 at 00:50 +0200, Karol Herbst wrote: > > On Fri, Apr 28, 2023 at 12:46 AM Lyude Paul wrote: > > > > > > Hey Linus, Kees. Responses below > > > > > > On Sun, 2023-04-23 at 1

Re: Disabling -Warray-bounds for gcc-13 too

2023-04-27 Thread Karol Herbst
On Fri, Apr 28, 2023 at 12:46 AM Lyude Paul wrote: > > Hey Linus, Kees. Responses below > > On Sun, 2023-04-23 at 13:23 -0700, Kees Cook wrote: > > On April 23, 2023 10:36:24 AM PDT, Linus Torvalds > > wrote: > > > Kees, > > > I made the mistake of upgrading my M2 Macbook Air to Fedora-38, and

Re: Disabling -Warray-bounds for gcc-13 too

2023-04-24 Thread Karol Herbst
On Sun, Apr 23, 2023 at 10:24 PM Kees Cook wrote: > > On April 23, 2023 10:36:24 AM PDT, Linus Torvalds > wrote: > >Kees, > > I made the mistake of upgrading my M2 Macbook Air to Fedora-38, and > >in the process I got gcc-13 which is not WERROR-clean because we only > >limited the 'array-bounds

Re: [PATCH] drm/nouveau/disp: More DP_RECEIVER_CAP_SIZE array fixes

2023-04-24 Thread Karol Herbst
> dpcd[DP_RECEIVER_CAP_SIZE], > >| > > ~~~^~ > > > > Fixes: 813443721331 ("drm/nouveau/disp: move DP link config into acquire") > > Cc: Ben Skeggs > > Cc: Lyude Paul > >

Re: [PATCH 0/9] GPU-DRM-nouveau: Adjustments for seven function implementations

2023-04-17 Thread Karol Herbst
On Sun, Apr 16, 2023 at 11:30 AM Markus Elfring wrote: > > Date: Sun, 16 Apr 2023 11:22:23 +0200 > > Several update suggestions were taken into account > from static source code analysis. > Reviewed-by: Karol Herbst > Markus Elfring (9): > debugfs: Move an express

Re: [PATCH v3] drm/nouveau: fix incorrect conversion to dma_resv_wait_timeout()

2023-04-17 Thread Karol Herbst
ready using > > dma_resv_wait_timeout() correctly. > > > > Fixes: 41d351f29528 ("drm/nouveau: stop using ttm_bo_wait") > > Reported-by: Tanmay Bhushan <0070472...@gmail.com> > > Link: > > https://lore.kernel.org/lkml/20230119225351.71657-1-0070472...@gmail.co

Re: [PATCH -next] drm/nouveau/disp: make gv100_disp_core_mthd_base static

2023-04-11 Thread Karol Herbst
Reviewed-by: Karol Herbst On Sat, Sep 24, 2022 at 10:00 AM ruanjinjie wrote: > > The symbol is not used outside of the file, so mark it static. > > Fixes the following warning: > > ./drivers/gpu/drm/nouveau/nvkm/engine/disp/gv100.c:591:1: warning: > symbol 'gv100_di

Re: [PATCH] drm/nouveau/mc/ga100: make ga100_mc_device static

2023-04-11 Thread Karol Herbst
Reviewed-by: Karol Herbst On Thu, Dec 29, 2022 at 4:52 PM Ben Dooks wrote: > > Make ga100_mc_device static as it isn't exported, to > fix the following sparse warning: > > drivers/gpu/drm/nouveau/nvkm/subdev/mc/ga100.c:51:1: warning: symbol > 'ga100_mc_device

[PATCH] drm/nouveau/fb: add missing sysmen flush callbacks

2023-04-05 Thread Karol Herbst
Closes: https://gitlab.freedesktop.org/drm/nouveau/-/issues/203 Fixes: 5728d064190e1 ("drm/nouveau/fb: handle sysmem flush page from common code") Signed-off-by: Karol Herbst --- drivers/gpu/drm/nouveau/nvkm/subdev/fb/gf108.c | 1 + drivers/gpu/drm/nouveau/nvkm/subdev/fb/gk104.c | 1

Re: [PATCH] drm/nouveau/disp: Support more modes by checking with lower bpc

2023-03-31 Thread Karol Herbst
On Fri, Mar 31, 2023 at 12:39 AM Karol Herbst wrote: > > This allows us to advertise more modes especially on HDR displays. > > Fixes using 4K@60 modes on my TV and main display both using a HDMI to DP > adapter. Also fixes similiar issues for users running into th

[PATCH] drm/nouveau/disp: Support more modes by checking with lower bpc

2023-03-30 Thread Karol Herbst
This allows us to advertise more modes especially on HDR displays. Fixes using 4K@60 modes on my TV and main display both using a HDMI to DP adapter. Also fixes similiar issues for users running into this. Cc: sta...@vger.kernel.org # 5.10+ Signed-off-by: Karol Herbst --- drivers/gpu/drm

Re: [PATCH 34/37] drm/nouveau/nvkm/engine/gr/tu102: Completely remove unused function ‘tu102_gr_load’

2023-03-17 Thread Karol Herbst
mistake not using this function. Ben, is it safe to remove it? The code seems to be slightly different things, but also looks like it needs to be rewritten a little anyway. > Cc: Ben Skeggs > Cc: Karol Herbst > Cc: Lyude Paul > Cc: David Airlie > Cc: Daniel Vetter > Cc:

Re: [PATCH 31/37] drm/nouveau/dispnv04/crtc: Demote kerneldoc abuses

2023-03-17 Thread Karol Herbst
nouveau changes up to here are: Reviewed-by: Karol Herbst On Fri, Mar 17, 2023 at 9:23 AM Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/nouveau/dispnv04/crtc.c:453: warning: This comment starts > with '/**', but isn'

Re: linux-6.2-rc4+ hangs on poweroff/reboot: Bisected

2023-03-10 Thread Karol Herbst
on't know when it's the next time it will be pushed upwards, but it should get there eventually. And because it also contains a Fixes tag it will be backported to older branches as well. > Chris > > On 20/02/2023 22:16, Ben Skeggs wrote: > > On Mon, 20 Feb 2023 at 21:27, Kar

Re: [PATCH RFC 01/18] rust: drm: ioctl: Add DRM ioctl abstraction

2023-03-09 Thread Karol Herbst
On Thu, Mar 9, 2023 at 9:24 PM Faith Ekstrand wrote: > > On Thu, 2023-03-09 at 15:04 +0900, Asahi Lina wrote: > > On 08/03/2023 02.34, Björn Roy Baron wrote: > > > > +// SAFETY: This is just the ioctl > > > > argument, which hopefully has the right type > > > > +

Re: [PATCH RFC 10/18] drm/scheduler: Add can_run_job callback

2023-03-08 Thread Karol Herbst
On Wed, Mar 8, 2023 at 4:09 PM Christian König wrote: > > Am 08.03.23 um 15:43 schrieb Karol Herbst: > > [SNIP] > > "further"? There was no discussion at all, > > Yeah, well that is exactly what I wanted to archive. > > > you just started off like

Re: [PATCH RFC 10/18] drm/scheduler: Add can_run_job callback

2023-03-08 Thread Karol Herbst
On Wed, Mar 8, 2023 at 2:47 PM Christian König wrote: > > Am 08.03.23 um 13:39 schrieb Karol Herbst: > > On Wed, Mar 8, 2023 at 9:46 AM Christian König > > wrote: > >> Am 07.03.23 um 15:25 schrieb Asahi Lina: > >>> Some hardware may require more

Re: [PATCH RFC 10/18] drm/scheduler: Add can_run_job callback

2023-03-08 Thread Karol Herbst
On Wed, Mar 8, 2023 at 9:46 AM Christian König wrote: > > Am 07.03.23 um 15:25 schrieb Asahi Lina: > > Some hardware may require more complex resource utilization accounting > > than the simple job count supported by drm_sched internally. Add a > > can_run_job callback to allow drivers to implemen

Re: [PATCH RFC 15/18] drm/asahi: Add the Asahi driver UAPI [DO NOT MERGE]

2023-03-07 Thread Karol Herbst
On Tue, Mar 7, 2023 at 3:28 PM Asahi Lina wrote: > > Adds the Asahi GPU driver UAPI. Note: this API is not yet stable and > therefore not ready for merging! > > Signed-off-by: Asahi Lina > --- > include/uapi/drm/asahi_drm.h | 556 > +++ > 1 file changed,

  1   2   3   4   5   >