Skip to content
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.

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

Clarify the meaning of different device types #727

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Pennycook
Copy link
Contributor

The description of what constitutes a CPU or GPU has always been unclear.

This commit adjusts these descriptions to clarify that it is ultimately up to the implementation to decide whether a device is identified as a specific device type. It also attempts to separate normative behaviors (like the connection between device type aspects and info::device_type) from non-normative descriptions of what a device type may mean.

Closes #7, closes #545.

The description of what constitutes a CPU or GPU has always been unclear.

This commit adjusts these descriptions to clarify that it is ultimately up to
the implementation to decide whether a device is identified as a specific
device type. It also attempts to separate normative behaviors (like the
connection between device type aspects and info::device_type) from
non-normative descriptions of what a device type may mean.
@Pennycook Pennycook added the editorial Some purely editorial problem label Feb 14, 2025
@TApplencourt
Copy link
Contributor

Amazing, thanks a lot! I wanted to make those kinds of changes but didn't know how to tackle them. They look nice!

Indicates that this device identfies itself as a CPU, and has device type

Typos, Should be identifies.

PS: I will not make any joke about using self-affirming terminology on the SYCL Spec...

@Pennycook
Copy link
Contributor Author

Typos, Should be identifies.

Good catch, thanks!

PS: I will not make any joke about using self-affirming terminology on the SYCL Spec...

Since you brought it up... I did wonder if this wording could be problematic, because somebody might think we're trying to make a joke. Maybe:

Indicates that the implementation identifies this device as a CPU, and has device type [api]#info::device_type::cpu#.

...would be better? It's arguably more accurate, too, since different implementations may choose to represent the same device in different ways. I'll make that change.

The previous wording could be read in a way that implied the (hardware) device
could somehow declare what its device type should be, which is misleading. It
is more accurate to say that the implementation identifies the device as a
specific type, since different implementations may choose to identify the same
piece of hardware differently.
@TApplencourt
Copy link
Contributor

Indicates that the implementation identifies this device as a CPU, and has device type [api]#info::device_type::cpu#.

Look good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial Some purely editorial problem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A definition of a GPU Device definitions
2 participants