[PATCH] drm/gud: Fix missing include

2022-12-07 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes Add missing vmalloc.h include. Fixes: c17d048609bf ("drm/gud: Use the shadow plane helper") Reported-by: kernel test robot Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/gud/gud_pipe.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/gud/gud_pipe.c b

[PATCH v2 6/6] drm/gud: Enable synchronous flushing by default

2022-11-30 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes gud has a module parameter that controls whether framebuffer flushing happens synchronously during the commit or asynchronously in a worker. GNOME before version 3.38 handled all displays in the same rendering loop. This lead to gud slowing down the refresh rate for a faster

[PATCH v2 3/6] drm/gud: Split up gud_flush_work()

2022-11-30 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes In preparation for inlining synchronous flushing split out the part of gud_flush_work() that can be shared by the sync and async code paths. Reviewed-by: Javier Martinez Canillas Reviewed-by: Thomas Zimmermann Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/gud/gud_pip

[PATCH v2 4/6] drm/gud: Prepare buffer for CPU access in gud_flush_work()

2022-11-30 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes In preparation for moving to the shadow plane helper prepare the framebuffer for CPU access as early as possible. v2: - Use src as variable name for iosys_map (Thomas) Reviewed-by: Javier Martinez Canillas Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/gud/gud_pipe.c

[PATCH v2 1/6] drm/gud: Fix UBSAN warning

2022-11-30 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes UBSAN complains about invalid value for bool: [ 101.165172] [drm] Initialized gud 1.0.0 20200422 for 2-3.2:1.0 on minor 1 [ 101.213360] gud 2-3.2:1.0: [drm] fb1: guddrmfb frame buffer device [ 101.213426] usbcore: registered new interface driver gud [ 101.989431] ==

[PATCH v2 5/6] drm/gud: Use the shadow plane helper

2022-11-30 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes Use the shadow plane helper to take care of mapping the framebuffer for CPU access. The synchronous flushing is now done inline without the use of a worker. The async path now uses a shadow buffer to hold framebuffer changes and it doesn't read the framebuffer behind userspac

[PATCH v2 2/6] drm/gud: Don't retry a failed framebuffer flush

2022-11-30 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes If a framebuffer flush fails the driver will do one retry by requeing the worker. Currently the worker is used even for synchronous flushing, but a later patch will inline it, so this needs to change. Thinking about how to solve this I came to the conclusion that this retry m

[PATCH v2 0/6] drm/gud: Use the shadow plane helper

2022-11-30 Thread Noralf Trønnes via B4 Submission Endpoint
Hi, I have started to look at igt for testing and want to use CRC tests. To implement support for this I need to move away from the simple kms helper. When looking around for examples I came across Thomas' nice shadow helper and thought, yes this is perfect for drm/gud. So I'll switch to that bef

[PATCH 6/6] drm/gud: Use the shadow plane helper

2022-11-22 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes Use the shadow plane helper to take care of preparing the framebuffer for CPU access. The synchronous flushing is now done inline without the use of a worker. The async path now uses a shadow buffer to hold framebuffer changes and it doesn't read the framebuffer behind usersp

[PATCH 4/6] drm/gud: Split up gud_flush_work()

2022-11-22 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes In preparation for inlining synchronous flushing split out the part of gud_flush_work() that can be shared by the sync and async code paths. Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/gud/gud_pipe.c | 72 +++--- 1 file changed, 39

[PATCH 5/6] drm/gud: Prepare buffer for CPU access in gud_flush_work()

2022-11-22 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes In preparation for moving to the shadow plane helper prepare the framebuffer for CPU access as early as possible. Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/gud/gud_pipe.c | 67 +- 1 file changed, 33 insertions(+), 34 deletion

[PATCH 0/6] drm/gud: Use the shadow plane helper

2022-11-22 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes Hi, I have started to look at igt for testing and want to use CRC tests. To implement support for this I need to move away from the simple kms helper. When looking around for examples I came across Thomas' nice shadow helper and thought, yes this is perfect for drm/gud. So

[PATCH 2/6] drm/gud: Fix UBSAN warning

2022-11-22 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes UBSAN complains about invalid value for bool: [ 101.165172] [drm] Initialized gud 1.0.0 20200422 for 2-3.2:1.0 on minor 1 [ 101.213360] gud 2-3.2:1.0: [drm] fb1: guddrmfb frame buffer device [ 101.213426] usbcore: registered new interface driver gud [ 101.989431] ==

[PATCH 3/6] drm/gud: Don't retry a failed framebuffer flush

2022-11-22 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes If a framebuffer flush fails the driver will do one retry by requeing the worker. Currently the worker is used even for synchronous flushing, but a later patch will inline it, so this needs to change. Thinking about how to solve this I came to the conclusion that this retry m

[PATCH 1/6] drm/gem: shadow_fb_access: Prepare imported buffers for CPU access

2022-11-22 Thread Noralf Trønnes via B4 Submission Endpoint
From: Noralf Trønnes Complete the shadow fb access functions by also preparing imported buffers for CPU access. Update the affected drivers that currently use drm_gem_fb_begin_cpu_access(). Through this change the following SHMEM drivers will now also make sure their imported buffers are prepare