On Thu, Nov 21, 2024 at 5:27 AM Christian König
<ckoenig.leichtzumerken@xxxxxxxxx> wrote:
>
> Hey Zack,
>
> vmwgfx is the only driver left using this component.
>
> So I though I go ahead and complete switching over to drm_exec.
> The patches are only compile tested, but relatively staright forward.
Hi, Christian.
Thank you so much for tackling this! It's on our todo.
I just applied the patches and they break rather badly, i.e. I don't
see any command submission with the changes. I won't have the time to
look at it today, I'll try to carve out some time over the weekend to
debug this. Two other observations:
1) The patches break the build, I think they're missing:
diff --git a/drivers/gpu/drm/vmwgfx/Kconfig b/drivers/gpu/drm/vmwgfx/Kconfig
index 6c3c2922ae8b..f7aff3fdfb1a 100644
--- a/drivers/gpu/drm/vmwgfx/Kconfig
+++ b/drivers/gpu/drm/vmwgfx/Kconfig
@@ -4,6 +4,7 @@ config DRM_VMWGFX
depends on DRM && PCI && MMU
depends on (X86 && HYPERVISOR_GUEST) || ARM64
select DRM_CLIENT_SELECTION
+ select DRM_EXEC
select DRM_TTM
select DRM_TTM_HELPER
select MAPPING_DIRTY_HELPERS
2) qxl would need to be patched as well. At least it uses
ttm_eu_reserve_buffers which you're removing.
z
