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

02 of 10 LNX series - Minor fix to ofi_check_fabric_attr() #10027

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

amirshehataornl
Copy link
Contributor

util: Consider provider exclusion

When checking fabric attributes with ofi_check_fabric_attr() make sure to
consider provider exclusion.

When checking to see if a provider name is given, only consider ones which
are not excluded using the '^' character.

Signed-off-by: Amir Shehata shehataa@ornl.gov

When checking fabric attributes with ofi_check_fabric_attr() make sure to
consider provider exclusion.

When checking to see if a provider name is given, only consider ones which
are not excluded using the '^' character.

Signed-off-by: Amir Shehata <shehataa@ornl.gov>
@amirshehataornl amirshehataornl changed the title 02 lnx series util 02 of 09 LNX series - Minor fix to ofi_check_fabric_attr() May 16, 2024
@amirshehataornl amirshehataornl changed the title 02 of 09 LNX series - Minor fix to ofi_check_fabric_attr() 02 of 10 LNX series - Minor fix to ofi_check_fabric_attr() May 16, 2024
@j-xiong
Copy link
Contributor

j-xiong commented May 17, 2024

^ is only used in FI_PROVIDER to create provider filters. It should not appear in fabric_attr->prov_name.

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.

None yet

2 participants