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

fabtests/configure, efa: call provider specific configure #10420

Merged
merged 1 commit into from
Sep 29, 2024

Conversation

shijin-aws
Copy link
Contributor

Create efa provider's own configure.m4 and make configure.ac call it.
Build efa_rdma_checker conditionally when all symbols are available.

@shijin-aws
Copy link
Contributor Author

This should fix Intel CI issue that cannot build efa specific fabtests

fabtests/configure.ac Outdated Show resolved Hide resolved
@zachdworkin
Copy link
Contributor

The current Intel CI run looks okay with just this change applied. I will test it in combination with the other one now

@zachdworkin
Copy link
Contributor

The current Intel CI run looks okay with just this change applied. I will test it in combination with the other one now

the --enable-efa=no option and these changes both allow fabtests to be built on our node which is missing the package

@zachdworkin
Copy link
Contributor

The build that is failing now is SHMEM middleware. Its not finding stdlib.h... im going to re-run

Copy link
Contributor

@aingerson aingerson left a comment

Choose a reason for hiding this comment

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

Fixes the issue I'm seeing - thanks!

Create efa provider's own configure.m4 and make configure.ac
call it.
Build efa_rdma_checker conditionally when all symbols are
available.

Signed-off-by: Shi Jin <sjina@amazon.com>
@shijin-aws shijin-aws merged commit fc5e85c into ofiwg:main Sep 29, 2024
13 checks passed
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