Skip to content

fix(static loader): do NOT call zeInit with sysmanOnly set - #517

Open
oskarweb wants to merge 1 commit into
oneapi-src:masterfrom
oskarweb:zesInitFix
Open

oskarweb wants to merge 1 commit into
oneapi-src:masterfrom
oskarweb:zesInitFix

Conversation

@oskarweb

Copy link
Copy Markdown

Previously, the static loader implicitly called zeInit() first. With ZES_ENABLE_SYSMAN=1, this activated legacy Sysman initialization, causing the driver to reject the subsequent explicit zesInit() call.

Signed-off-by: Oskar Hubert Weber oskar.hubert.weber@intel.com

Previously, the static loader implicitly called zeInit() first. With
ZES_ENABLE_SYSMAN=1, this activated legacy Sysman initialization,
causing the driver to reject the subsequent explicit zesInit() call.

Signed-off-by: Oskar Hubert Weber <oskar.hubert.weber@intel.com>
@rwmcguir

Copy link
Copy Markdown
Contributor

@shubskmr @joshuaranjan if your team could look into this and see if it works for both OLD and NEW systems, loader has to be backward compatible.

This branch has not been deployed

No deployments
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.

2 participants