Repository navigation
Conversation
Signed-off-by: aknvda <anikkulkarni@nvidia.com>
Signed-off-by: aknvda <anikkulkarni@nvidia.com>
Signed-off-by: aknvda <anikkulkarni@nvidia.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remote delivery currently gives little visibility into framework pin waits or native write ownership. Add optional KVCR-domain NVTX annotations for the pin callback, asynchronous pin/waiter lifecycle, NIXL posting, first native DONE, cancellation, handle release, and unresolved shutdown. Static event names carry fresh structured payloads; logical failure remains distinct from native completion and safe release.
Tracing defaults to low when the optional profiling dependencies are installed.
KVCR_NVTX_LEVEL=off|low|mediumcontrols payload work without checking profiler attachment. The schema keeps signed operation handles, source-operation and target-incarnation identities, explicit unknown fields, and bounded UTF-8 context mapping. Target/hint/caller hooks are a follow-up.Validation: 183 focused Linux tests passed on the source branch; real pin capture example and profiling documentation included. The combined source/target follow-up has separate two-process DRAM/VRAM capture evidence. An independent review found a lost route-refusal cause; the added regression failed before the fix and passes afterward. Commit sign-offs included.
Known review item: a source that selects zero descriptors reports PARTIAL with an explicit zero count; the target/caller reports FAILED. This does not change transfer or memory ownership. Framework utilization, session and parent-session IDs are unavailable through current bindings and are not inferred. Full public-fetch fan-out is outside this contribution.