Skip to content

Clarify Unified Addressing#1005

Merged
gmlueck merged 5 commits into
KhronosGroup:mainfrom
VerenaBeckham:patch-1
Jun 4, 2026
Merged

Clarify Unified Addressing#1005
gmlueck merged 5 commits into
KhronosGroup:mainfrom
VerenaBeckham:patch-1

Conversation

@VerenaBeckham
Copy link
Copy Markdown
Contributor

Unified Addressing should only be within a single context (and the host).

Unified Addressing should only be within a single context (and the host).
Comment thread adoc/chapters/programming_interface.adoc Outdated
@hdelan
Copy link
Copy Markdown

hdelan commented Apr 29, 2026

Perhaps this section of the C++ spec could be extended to USM buffers within SYCL contexts, but not across different contexts.

@gmlueck
Copy link
Copy Markdown
Contributor

gmlueck commented Apr 29, 2026

Perhaps this section of the C++ spec could be extended to USM buffers within SYCL contexts, but not across different contexts.

I think that section of the C++ spec is mostly talking about nested objects. How do you think it could be extended to clarify our situation?

FWIW, I considered writing my proposed text in terms of C++ object addresses, but I changed it to refer to USM buffer addresses instead. I did this because the memory returned from the USM allocation routines is raw memory that does not contain any C++ objects.

@hdelan
Copy link
Copy Markdown

hdelan commented Apr 29, 2026

It says that except in the mentioned cases

[two objects] have distinct addresses and occupy disjoint bytes of storage

Maybe a better place to extend is here but again this refers to objects rather than memory as you say.

@TApplencourt
Copy link
Copy Markdown
Contributor

  • What is a USM Pointer?
    "An USM pointer, is a pointer who is returned by sycl::malloc_* like"

  • Can do that in another PR for "what is allocate USM Pointer"?

    • Thomas will open an Issue
  • Verena will update this PR with greg test

Copy link
Copy Markdown
Contributor

@gmlueck gmlueck left a comment

Choose a reason for hiding this comment

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

Looks good. Just a small formatting nit below.

Comment thread adoc/chapters/programming_interface.adoc Outdated
Comment thread adoc/chapters/programming_interface.adoc Outdated
@TApplencourt
Copy link
Copy Markdown
Contributor

WG: The WG agree to merge this PR at a clarification of 2020.

@gmlueck gmlueck merged commit e8d41c5 into KhronosGroup:main Jun 4, 2026
2 checks passed
gmlueck added a commit that referenced this pull request Jun 4, 2026
Cherry pick #1005 from main
(cherry picked from commit e8d41c5)
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.

5 participants