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

L0 Memory Leak on Windows in test-e2e\Graph\Explicit\reduction.cpp #267

Closed
EwanC opened this issue Jul 18, 2023 · 1 comment · Fixed by #274
Closed

L0 Memory Leak on Windows in test-e2e\Graph\Explicit\reduction.cpp #267

EwanC opened this issue Jul 18, 2023 · 1 comment · Fixed by #274
Labels
bug Something isn't working Graph Implementation Related to DPC++ implementation and testing

Comments

@EwanC
Copy link
Collaborator

EwanC commented Jul 18, 2023

When run on Windows with the ZE_DEBUG=4 environment variable set, the test-e2e\Graph\Explicit\reduction.cpp test reports a memory leak of L0 resources, causing it to fail

               zeContextCreate = 1     \--->              zeContextDestroy = 0     ---> LEAK = 1
          zeCommandQueueCreate = 1     \--->         zeCommandQueueDestroy = 0     ---> LEAK = 1
                zeModuleCreate = 1     \--->               zeModuleDestroy = 0     ---> LEAK = 1
                zeKernelCreate = 2     \--->               zeKernelDestroy = 0     ---> LEAK = 2
             zeEventPoolCreate = 3     \--->            zeEventPoolDestroy = 0     ---> LEAK = 3
  zeCommandListCreateImmediate = 1     |
           zeCommandListCreate = 4     \--->          zeCommandListDestroy = 1     ---> LEAK = 4
                 zeEventCreate = 6     \--->                zeEventDestroy = 0     ---> LEAK = 6
                 zeFenceCreate = 4     \--->                zeFenceDestroy = 1     ---> LEAK = 3
                 zeImageCreate = 0     \--->                zeImageDestroy = 0
               zeSamplerCreate = 0     \--->              zeSamplerDestroy = 0
              zeMemAllocDevice = 1     |
                zeMemAllocHost = 1     |
              zeMemAllocShared = 0     \--->                     zeMemFree = 0     ---> LEAK = 2

This leak should be investigated and fixed

@EwanC EwanC added bug Something isn't working Graph Implementation Related to DPC++ implementation and testing labels Jul 18, 2023
EwanC added a commit that referenced this issue Jul 18, 2023
Cherry-pick commit intel@b78844b
from `sycl-graph-patch-4` back to `sycl-graph-develop` until
these Windows fails are investigated and resolved

* #267
* #268
* #269
EwanC added a commit that referenced this issue Jul 19, 2023
Cherry-pick commit intel@b78844b
from `sycl-graph-patch-4` back to `sycl-graph-develop` until
these Windows fails are investigated and resolved

* #267
* #268
* #269
@EwanC
Copy link
Collaborator Author

EwanC commented Jul 19, 2023

If I modify the reduction test to remove the graphs code I still see the memory-leak. Looking at existing e2e reduction tests I think this is a known issue unrelated to our extension intel#9764

EwanC added a commit that referenced this issue Jul 19, 2023
Level Zero memory leaks with reductions are a known
issue intel#9764

Only run the leak component of the reduction test on Linux
to avoid this issue.

Closes #267
Bensuo pushed a commit that referenced this issue Jul 24, 2023
Level Zero memory leaks with reductions are a known
issue intel#9764

Only run the leak component of the reduction test on Linux
to avoid this issue.

Closes #267
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Graph Implementation Related to DPC++ implementation and testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant