Fix the "Initialization" in the Sysman Programming Guide

Open
#152 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
45/100
Issue type
Documentation
Clarity
Mostly clear
Activity status
Stale
Tech stack
cpp
Domain
documentation

Research direction

Start with the Initialization section of the Sysman Programming Guide at spec.oneapi.io/level-zero/latest/sysman/PROG.html, then read compute-runtime issue 686 for the compatibility details. Update the quoted statement so it matches the documented ZesInit() behavior, and verify the rendered guide no longer claims that the cast is valid in that case.

Written by the indexing model from the issue text.

Description

From https://spec.oneapi.io/level-zero/latest/sysman/PROG.html
"For compatibility, an application may also use the Level0 Core API to enumerate through available accelerator devices in the system. For each device handle, an application can cast it to a sysman device handle to manage the system resources of the device."
This cast is invalid when using ZesInit() as discussed in https://github.com/intel/compute-runtime/issues/686, it's only valid when using zeInit() + the env var to enable Sysman. I asked for the fix there last year but the issue was closed without the spec fix.

Dominant language
C++
Stars
335
Forks
140
Avg merge
12h 32m
Merged PRs (30d)
5

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from oneapi-src/level-zero

All issues in oneapi-src/level-zero

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.