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

UTIL: Add riscv64 support #829

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

rbradford
Copy link
Contributor

What

Enables support for RISC-V (64-bit)

Why ?

UCX has recently gained support for this architecture so it would be nice to add UCC support

How ?

The changes needed to support just consist of implementing the fence functions using the fence instructions - the ones used here match added to UCX.

@swx-jenkins3
Copy link

Can one of the admins verify this patch?

src/utils/arch/cpu.h Show resolved Hide resolved
src/utils/arch/riscv64/cpu.h Outdated Show resolved Hide resolved
@rbradford rbradford force-pushed the 2023-08-25-riscv64-port branch from 7a7f41f to 36e04ca Compare August 28, 2023 09:53
@rbradford rbradford changed the title UTILS/ARCH: Add riscv64 support UTIL: Add riscv64 support Aug 28, 2023
@shimmybalsam
Copy link
Collaborator

@rbradford please add the new file you created to Makefile here:
https://github.com/openucx/ucc/blob/master/src/Makefile.am#L53

@rbradford rbradford force-pushed the 2023-08-25-riscv64-port branch 2 times, most recently from 5abcee7 to 6a85f17 Compare August 28, 2023 13:03
@rbradford
Copy link
Contributor Author

@rbradford please add the new file you created to Makefile here: https://github.com/openucx/ucc/blob/master/src/Makefile.am#L53

Fixed. Thank you!

@rbradford rbradford force-pushed the 2023-08-25-riscv64-port branch from 6a85f17 to 5e16b9a Compare August 29, 2023 14:31
@manjugv
Copy link
Contributor

manjugv commented Aug 30, 2023

@rbradford Have you signed CLA with UCF? Thanks.

@rbradford
Copy link
Contributor Author

@rbradford Have you signed CLA with UCF? Thanks.

Yes, my employer (Rivos Inc) has signed the corporate CLA.

@rbradford rbradford force-pushed the 2023-08-25-riscv64-port branch 2 times, most recently from 2059aee to 5831a3f Compare September 4, 2023 16:14
@rbradford
Copy link
Contributor Author

Ping? Any update on this?

@rbradford rbradford force-pushed the 2023-08-25-riscv64-port branch from 5831a3f to c6ff51d Compare September 4, 2023 16:33
@Sergei-Lebedev
Copy link
Contributor

@rbradford please rebase

Signed-off-by: Rob Bradford <[email protected]>
@rbradford rbradford force-pushed the 2023-08-25-riscv64-port branch from c6ff51d to 3bac9a7 Compare September 7, 2023 22:10
@Sergei-Lebedev Sergei-Lebedev merged commit b8e12e9 into openucx:master Sep 8, 2023
7 of 9 checks passed
nsarka pushed a commit to nsarka/ucc that referenced this pull request Oct 24, 2023
nsarka pushed a commit to nsarka/ucc that referenced this pull request Oct 24, 2023
janjust pushed a commit to janjust/ucc that referenced this pull request Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants