Skip to content

Support vulkan backend #110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Support vulkan backend #110

wants to merge 6 commits into from

Conversation

xiaowei-guan
Copy link
Contributor

No description provided.

@xiaowei-guan xiaowei-guan marked this pull request as draft August 14, 2025 05:53
@JSUYA JSUYA self-requested a review August 14, 2025 07:47
@xiaowei-guan
Copy link
Contributor Author

@JSUYA Do you konw how to fix unittests build error?

@JSUYA
Copy link
Member

JSUYA commented Aug 26, 2025

@JSUYA Do you konw how to fix unittests build error?

I will look into it.

@JSUYA
Copy link
Member

JSUYA commented Aug 27, 2025

+) #113

@JSUYA
Copy link
Member

JSUYA commented Aug 27, 2025

+) #113

@xiaowei-guan
I found that the common profile image doesn't include Vulkan.
Has the Vulkan renderer been tested on the RPI? Do we need to install a separate rpm?

https://download.tizen.org/releases/milestone/TIZEN/Tizen/Tizen-Unified/tizen-unified_20221017.061100/images/standard/tizen-headed-armv7l/tizen-unified_20221017.061100_tizen-headed-armv7l.packages

@xiaowei-guan
Minor refactor
2519881 
1.Fix spelling issues.
2.Do not need to destroy swapchain image when destory the renderer.
3.Replace memset with zero-init.
4.Free command buffer when begin command buffer or end command buffer
failed.
5.Fix return wrong api version issue.
@JSUYA
Copy link
Member

JSUYA commented Aug 27, 2025

+) #113

@xiaowei-guan I found that the common profile image doesn't include Vulkan. Has the Vulkan renderer been tested on the RPI? Do we need to install a separate rpm?

https://download.tizen.org/releases/milestone/TIZEN/Tizen/Tizen-Unified/tizen-unified_20221017.061100/images/standard/tizen-headed-armv7l/tizen-unified_20221017.061100_tizen-headed-armv7l.packages

The problem was solved with the help of @swift-kim.

@xiaowei-guan
Copy link
Contributor Author

+) #113

@xiaowei-guan I found that the common profile image doesn't include Vulkan. Has the Vulkan renderer been tested on the RPI? Do we need to install a separate rpm?
https://download.tizen.org/releases/milestone/TIZEN/Tizen/Tizen-Unified/tizen-unified_20221017.061100/images/standard/tizen-headed-armv7l/tizen-unified_20221017.061100_tizen-headed-armv7l.packages

The problem was solved with the help of @swift-kim.

@JSUYA Yes, if we want to run vulkan app on RPI4, we need to install vulkan related rpm files.

@swift-kim @JSUYA Thank you, the issue has been fixed.

@xiaowei-guan xiaowei-guan marked this pull request as ready for review August 28, 2025 05:27
@xiaowei-guan xiaowei-guan mentioned this pull request Aug 28, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JSUYA JSUYA Awaiting requested review from JSUYA

@xuelian-bai xuelian-bai Awaiting requested review from xuelian-bai

@gin7773 gin7773 Awaiting requested review from gin7773

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xiaowei-guan @JSUYA