Skip to content

Conversation

@SeyedMir
Copy link
Contributor

@SeyedMir SeyedMir commented Feb 9, 2021

What

This PR adds an internal UCT service to query memory attributes.

Why ?

Currently, we cannot query memory attributes of a given pointer at UCT layer. The existing uct_md_mem_query()
requires a handle to an MD and queries the attributes only from that MD.

How ?

It is optional but for complex PRs please provide information about the design,
architecture, approach, etc.

@SeyedMir SeyedMir added the WIP-DNM Work in progress / Do not review label Feb 9, 2021
@SeyedMir SeyedMir requested a review from yosefe February 9, 2021 22:13
@swx-jenkins3
Copy link
Collaborator

Can one of the admins verify this patch?

@SeyedMir
Copy link
Contributor Author

SeyedMir commented Feb 9, 2021

@yosefe This is how the new API might look like. It will have a lot of overlap with the existing uct_md_mem_query().
Any early comments?

@SeyedMir SeyedMir force-pushed the topic/uct_mem_attr branch 4 times, most recently from 08b7fa1 to 6bb4a80 Compare February 12, 2021 23:20
@SeyedMir SeyedMir force-pushed the topic/uct_mem_attr branch 2 times, most recently from 5310e78 to 7262d42 Compare February 18, 2021 19:06
@SeyedMir SeyedMir force-pushed the topic/uct_mem_attr branch from 7262d42 to eec86cc Compare March 11, 2021 19:10
@SeyedMir SeyedMir changed the title UCT/MD: Add a new UCT API to query memory attributes Query memory attributes in UCT Mar 11, 2021
Unlike the existing uct_md_mem_query(), then new API is not specific
to a single MD. Therefore, it does not need a handle to an opened MD.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WIP-DNM Work in progress / Do not review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants