Skip to content

fix memory leaks in loader #417

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
Merged
nrspruit merged 1 commit into from
Jan 20, 2026
Merged

fix memory leaks in loader #417

nrspruit merged 1 commit into from
Jan 20, 2026
+0 −13

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
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