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

Set pDataSize equal to privateDataSize + bytesWritten #119

Merged
merged 1 commit into from
Jun 30, 2021

Conversation

samikhawaja
Copy link
Contributor

On success in vkGetPipelineCacheData, set pDataSize even if the privateDataSize is zero. It will be equal to byteswritten when
privateDataSize is zero.

On success in vkGetPipelineCacheData, set pDataSize even if the
privateDataSize is zero. It will be equal to byteswritten when
privateDataSize is zero.
@amdvlk-admin
Copy link
Collaborator

Can one of the admins verify this patch?

@JaxLinAMD
Copy link
Contributor

ok to test

@amdvlk-admin
Copy link
Collaborator

Test summary for commit 9380f18

Driver commits used in build
  • CWPACK: amd-master 7387247eb9889ddcabbc1053b9c2052e253b088e
  • METROHASH: amd-master 3c566dd9cda44ca7fd97659e0b53ac953f9037d2
  • LLPC: dev e23176874d33ad107eacb871980b0bfebe8331da
  • PAL: dev 66f4ef75eed4d098f7652739b9f7dd3cd8bda6e6
  • SPVGEN: dev 79f8e32dd7242d7d34fa0074f761c9d794f11811
  • LLVM-PROJECT: amd-gfx-gpuopen-dev 0cb96bc82ca13e0fef3684a4ac8f6eff50bce257
CTS tests (Failed: 0/228128)
  • Built with version 1.2.5.2
  • Rhel 8.2, Gfx10
    • Passed: 31146/57032 (54.6%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25886/57032 (45.4%)
    Ubuntu 18.04, Gfx9
    • Passed: 31145/57032 (54.6%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25887/57032 (45.4%)
    Ubuntu 20.04, Gfx8
    • Passed: 31285/57032 (54.9%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 25747/57032 (45.1%)
    Ubuntu 20.04, Gfx103
    • Passed: 34853/57032 (61.1%)
    • Failed: 0/57032 (0.0%)
    • Not Supported: 22179/57032 (38.9%)

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

4 participants