Skip to content
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

Wrong cl_image_desc setup collected from VkImageCreateInfo #2155

Open
shajder opened this issue Nov 20, 2024 · 1 comment · May be fixed by #2289
Open

Wrong cl_image_desc setup collected from VkImageCreateInfo #2155

shajder opened this issue Nov 20, 2024 · 1 comment · May be fixed by #2289
Labels
mobica-backlog Issue approved by WG for Mobica to work on

Comments

@shajder
Copy link
Contributor

shajder commented Nov 20, 2024

Procedure getCLImageInfoFromVkImageInfo returns image width/height/depth 256x256x10 despite the fact image has 256x16x10 dimensions. Source of the problem is invalid usage of get2DImageDimensions which computes height without taking into accout depth value:

height = totalImageSize / row_pitch;
@nikhiljnv nikhiljnv added the mobica-triage Issue proposed for addition to Mobica's backlog (needs WG approval) label Dec 3, 2024
@neiltrevett neiltrevett added mobica-backlog Issue approved by WG for Mobica to work on and removed mobica-triage Issue proposed for addition to Mobica's backlog (needs WG approval) labels Jan 21, 2025
@joshqti
Copy link
Contributor

joshqti commented Jan 28, 2025

Good catch, we should fix

@ewagalamon ewagalamon added the in_development Mobica is currently working on this label Feb 21, 2025
@shajder shajder linked a pull request Feb 21, 2025 that will close this issue
Open
@shajder shajder removed the in_development Mobica is currently working on this label Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
mobica-backlog Issue approved by WG for Mobica to work on
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Corrected procedure to collect proper size of image from VkImageCreateInfo shajder/OpenCL-CTS
5 participants
@shajder @neiltrevett @nikhiljnv @ewagalamon @joshqti