compute-runtime icon indicating copy to clipboard operation
compute-runtime copied to clipboard

Zello-sysman tool does not support testing all L0 OneAPI spec metric types

Open eero-t opened this issue 3 years ago • 5 comments

https://github.com/intel/compute-runtime/blob/master/level_zero/tools/test/black_box_tests/zello_sysman.cpp#L1150

Is lacking options for testing metrics support for:

  • PSUs: https://spec.oneapi.io/level-zero/latest/sysman/api.html#psu
  • Fans: https://spec.oneapi.io/level-zero/latest/sysman/api.html#fan

Which have been supported by the public level-zero headers + loader already for months.

eero-t avatar Jun 14 '22 13:06 eero-t

thanks @eero-t . very likely we haven't added those because we dont have support in the driver yet, see here https://github.com/intel/compute-runtime/blob/d129cca4b0d073e5f0234a18d7cc8a36dea1cfb7/level_zero/api/sysman/zes_sysman_api_entrypoints.h#L451

Once we have support (no ETA at this moment), we could add something to zello_sysman.

jandres742 avatar Jun 14 '22 20:06 jandres742

@jandres742 I would think how backend handles unimplemented frontend APIs to also be obviously of interest. E.g. to verify that those are also done according to spec / provide correct return value (see: https://github.com/intel/compute-runtime/issues/510).

While more extensive support for APIs could wait until they are properly supported by backend, I think test tool could have basic support for all APIs that just calls enumeration functions and lists how many items are available (if any).

eero-t avatar Jun 15 '22 08:06 eero-t

PSU and Fan tests are still missing (no options listed for them by zello_sysman).

eero-t avatar Nov 25 '22 10:11 eero-t