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.

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

Fix for linear image with preinitialized intial layout #1960

Merged
merged 2 commits into from Jun 6, 2018

Conversation

Qining
Copy link
Contributor

@Qining Qining commented Jun 5, 2018

  1. If an image has linear tiling and has initial layout set to
    PREINITIALIZED, all its subresources data pool will be linked back to
    the bound device memory, no new pool will be created for those
    subresources.

  2. If an image has linear tiling and has initial layout set to
    PREINITIALIZED, and it can not be used as TRANSFER_DST, render target or
    imageStore target, its data will be primed by maping memory and flush
    map memory ranges.

1) If an image has linear tiling and has initial layout set to
PREINITIALIZED, all its subresources data pool will be linked back to
the bound device memory, no new pool will be created for those
subresources.

2) If an image has linear tiling and has initial layout set to
PREINITIALIZED, and it can not be used as TRANSFER_DST, render target or
imageStore target, its data will be primed by maping memory and flush
map memory ranges.
@Qining Qining requested a review from AWoloszyn June 5, 2018 23:00
@Qining
Copy link
Contributor Author

Qining commented Jun 5, 2018

Most of the changes in image_primer.go is just to return 'better' errors. The only real change in that file is func primeByPreinitialization().

@Qining
Copy link
Contributor Author

Qining commented Jun 6, 2018

Windows and Mac builds are tested locally, all passed.

@Qining Qining merged commit b468b98 into google:master Jun 6, 2018
@Qining Qining deleted the preinitialized-image branch October 23, 2018 17:32
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.

None yet

2 participants