Skip to content
This repository has been archived by the owner on Feb 5, 2024. It is now read-only.

Add minutes for 06-22-2023 Hardware SIG #122

Merged
merged 6 commits into from
Aug 16, 2023

Conversation

jandres742
Copy link
Contributor

No description provided.

Copy link

@alycm alycm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, some minor fixes and added a few audience questions from my own notes.

- Xinmin Tian, Intel
- Rod Burns, Codeplay
* - Brandon Yates, Intel
- DorotheeX Marie Clotilde Balas, Intel
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- DorotheeX Marie Clotilde Balas, Intel
- Dorothee Marie Clotilde Balas, Intel

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @alycm . Fixed.


* Work on extending TornadoVM to run on oneAPI stack,
focusing on Level Zero.
* Current acces to heterogeneous systems is fragmented
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Current acces to heterogeneous systems is fragmented
* Current access to heterogeneous systems is fragmented

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @alycm . Fixed.

* UR Adapters have been merged to the SYCL runtime.
* License was previously MIT and now being changed to Apache
to make it friendlier with LLVM.
* Current UR version is v0.6 and closing to be tagging next
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Current UR version is v0.6 and closing to be tagging next
* Current UR version is v0.6 and close to tagging next

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed.

* Migration counters.
* Async exception support.
* Device aggregation.
* Improvements to suggested group size returned by L0.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Improvements to suggested group size returned by L0.
* Improvements to suggested group size returned by L0.
* Question: Unified runtime would be more useful to this project as a standard? Answer: To convince Java community to use this software stack a standard is more appealing. Easier to justify a standard because not controlled by one party.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @alycm . Added.

select best configuration for a given workload, especially
when using architectures where access between tiles is
more costly than local access.
* Changes and new APIs to be added as part of L0 v1.7.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Changes and new APIs to be added as part of L0 v1.7.
* Changes and new APIs to be added as part of L0 v1.7.
* Questions:
* How is memory allocated between the different modes? Answer: In mode 0 get device handle and allocations split between two sub-devices (tiles), or could get sub-device and memory will be allocated on that tile. In mode 1 get handle to a tile as device (no sub-devices) and memory is allocated on that tile. No magic.
* Is ZE_FLAT_DEVICE_HIERARCHY an environment variable? Answer: Yes.
* Specification spec will say that mode 1 in the default? Answer: Up for debate, specification may say that default is implementation defined.
* In Mode 2 can't do an allocation on root device? Answer: Yes, you can. zeDeviceGet will return all tiles.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @alycm . Added.

Jaime Arteaga added 2 commits July 28, 2023 11:21
Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Jaime Arteaga <[email protected]>
@jandres742 jandres742 force-pushed the sigminutes branch 2 times, most recently from b6eca1b to bc53a75 Compare July 28, 2023 18:35
Jaime Arteaga added 3 commits July 28, 2023 11:41
Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Jaime Arteaga <[email protected]>
@rscohn2 rscohn2 merged commit 3e946a7 into oneapi-src:main Aug 16, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants