Skip to content

Conversation

@lumag
Copy link

@lumag lumag commented Sep 26, 2023

This switches kms support from using Intel Vulkan extensions (which are not even a part of Mesa anymore) to VK_EXT_image_drm_format_modifier and VK_EXT_external_memory_dma_buf. I'm opening this as a draft for now to prevent it from being merged for now.

@lumag lumag marked this pull request as ready for review November 16, 2023 03:38
@lumag
Copy link
Author

lumag commented Jan 15, 2024

@krh gracious ping

@lumag
Copy link
Author

lumag commented Aug 21, 2024

@gfxstrand also gracious ping.

@lumag lumag force-pushed the kms branch 5 times, most recently from 2704bfa to a79dc4b Compare September 9, 2024 08:08
Copy link

@MarijnS95 MarijnS95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this, and it works!

I just got caught out by init_vt() being called - and disabling all terminal/error output - before the implementation (unchanged here, so we should postpone it to a future PR) hardcodedly opens /dev/dri/card0 (does not exist on my system) and exits without printing any useful error message 😬

(On Intel Arc, card0 is skipped and card1 (but renderD128) is created instead.)

@lumag
Copy link
Author

lumag commented Sep 9, 2024

@MarijnS95 thanks for the review. Now I hope we can get @krh or @gfxstrand attention.

Copy link

@MarijnS95 MarijnS95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Latest force-push makes sense, just a rebase on latest master while removing an unintended change in init_headless() and inlining some const struct allocations.

Just a commit-body nit on:

These extensions have been deprecated, Mesa dropped support for it in Mesa, version 22.0.0.

Maybe drop the second Mesa, i.e. ..., Mesa dropped support for it in/since version 22.0.0?

@lumag
Copy link
Author

lumag commented Nov 15, 2024

@MarijnS95 fixed commit message.

@krh @gfxstrand can this be reviewed / merged?

VkCube supports rendering to bare KMS only if Vulkan supports special
set of Intel extensions. These extensions have been deprecated, Mesa
dropped support for it in version 22.0.0.

Replace vkCreateDmaBufImageINTEL usage with
VK_EXT_image_drm_format_modifier and VK_EXT_external_memory_dma_buf.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants