Skip to content

fix memory leaks in loader#417

Merged
nrspruit merged 1 commit intooneapi-src:masterfrom
vishnu-khanth:mem_leak_fix
Jan 20, 2026
Merged

fix memory leaks in loader#417
nrspruit merged 1 commit intooneapi-src:masterfrom
vishnu-khanth:mem_leak_fix

Conversation

@vishnu-khanth
Copy link
Contributor

@vishnu-khanth vishnu-khanth commented Jan 20, 2026

Related-To: NEO-17220

@vishnu-khanth
fix memory leaks in loader
e5494bd 
Related-To: NEO-17220

Signed-off-by: Vishnu Khanth <[email protected]>
@vishnu-khanth
Copy link
Contributor Author

vishnu-khanth commented Jan 20, 2026

More details with logs attached in NEO-17220

@vishnu-khanth
Copy link
Contributor Author

vishnu-khanth commented Jan 20, 2026 •
edited
Loading

  1. loaderZeDdiTable, loaderZetDdiTable, loaderZesDdiTable are removed since they are not being used anywhere. These were reported as still reachable by valgrind. Resolved after removing.
  2. For defaultZerDdiTable memory allocation was done and later gets reassigned and points to zeDrivers.front().dditable.zer. Hence the original allocated memory becomes orphaned. And reported as definitely lost by valgrind. The allocation is not necessary and hence removed.

@nrspruit nrspruit merged commit 787d016 into oneapi-src:master Jan 20, 2026
50 checks passed
@vishnu-khanth vishnu-khanth deleted the mem_leak_fix branch January 21, 2026 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nrspruit nrspruit nrspruit approved these changes

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

@vishnu-khanth @nrspruit