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

error: too few arguments to function 'cuGetProcAddress_v2' #8936

Closed
cponder opened this issue Mar 13, 2023 · 7 comments
Closed

error: too few arguments to function 'cuGetProcAddress_v2' #8936

cponder opened this issue Mar 13, 2023 · 7 comments

Comments

@cponder
Copy link

cponder commented Mar 13, 2023

I'm building v 1.14.0 using CUDA 12.0 with GCC 11.3.0, and getting this error

#54 34.71   CC       cuda_ipc/libuct_cuda_la-cuda_ipc_cache.lo
#54 34.89 In file included from base/cuda_iface.h:14,
#54 34.89                  from base/cuda_md.c:11:
#54 34.89 base/cuda_md.c: In function 'uct_cuda_copy_get_dmabuf_fd':
#54 34.89 /usr/local/cuda/include/cuda.h:151:45: error: too few arguments to function 'cuGetProcAddress_v2'
#54 34.89   151 | #define cuGetProcAddress                    cuGetProcAddress_v2
#54 34.89       |                                             ^~~~~~~~~~~~~~~~~~~
#54 34.89 base/cuda_md.c:234:14: note: in expansion of macro 'cuGetProcAddress'
#54 34.89   234 |     cu_err = cuGetProcAddress("cuMemGetHandleForAddressRange",
#54 34.89       |              ^~~~~~~~~~~~~~~~
#54 34.90 /usr/local/cuda/include/cuda.h:151:45: note: declared here
#54 34.90   151 | #define cuGetProcAddress                    cuGetProcAddress_v2
#54 34.90       |                                             ^~~~~~~~~~~~~~~~~~~
#54 34.90 /usr/local/cuda/include/cuda.h:21041:18: note: in expansion of macro 'cuGetProcAddress'
#54 34.90 21041 | CUresult CUDAAPI cuGetProcAddress(const char *symbol, void **pfn, int cudaVersion, cuuint64_t flags, CUdriverProcAddressQueryResult *symbolStatus);
#54 34.90       |                  ^~~~~~~~~~~~~~~~
#54 34.90 make[3]: *** [Makefile:786: base/libuct_cuda_la-cuda_md.lo] Error 1
#54 34.90 make[3]: *** Waiting for unfinished jobs....
#54 35.26 make[3]: Leaving directory '/usr/local/src/ucx-1.14.0/src/uct/cuda'
#54 35.26 make[2]: *** [Makefile:857: install-recursive] Error 1
#54 35.26 make[2]: Leaving directory '/usr/local/src/ucx-1.14.0/src/uct/cuda'
#54 35.26 make[1]: *** [Makefile:1184: install-recursive] Error 1
#54 35.26 make[1]: Leaving directory '/usr/local/src/ucx-1.14.0/src/uct'
#54 35.26 make: *** [Makefile:767: install-recursive] Error 1

Is there some flag I can set to get around this?

@cponder
Copy link
Author

cponder commented Mar 13, 2023

I'm building using these settings

ARG UCX_VERSION=1.14.0

    curl --retry 5 -fSsL "https://github.com/openucx/ucx/archive/v${UCX_VERSION}.tar.gz" | tar xz && \
    cd ucx-* && \
    ./autogen.sh && ./configure     \
        --prefix=/usr/local         \
        --enable-shared             \
        --disable-static            \
        --disable-doxygen-doc       \
        --enable-optimizations      \
        --enable-cma                \
        --enable-devel-headers      \
        --with-cuda=/usr/local/cuda \
        --with-knem=/usr/local      \
        --with-verbs                \
        --with-dm                   \
        --with-gdrcopy=/usr/local   \
        --enable-mt                 \
        --enable-debug              \
        --with-mlx5-dv &&           \
    make -j install

This is on Ubuntu 22.04.

@brminich
Copy link
Contributor

v1.14.0 does not support CUDA 12 yet. This issue will be fixed in the next release (and already fixed on master)

@cponder
Copy link
Author

cponder commented Mar 14, 2023

Can you push out a 1.14.1-rc1 with it?

@yosefe
Copy link
Contributor

yosefe commented Mar 14, 2023

Can you push out a 1.14.1-rc1 with it?

We plan to start publishing v1.14.1 release candidates in about a week after merging additional bug fixes

@cponder
Copy link
Author

cponder commented Mar 14, 2023

Ok that'd be good, thanks.

@samcmill
Copy link

We plan to start publishing v1.14.1 release candidates in about a week after merging additional bug fixes

When do you expect v1.14.1 release candidates to be available?

@jakirkham
Copy link
Contributor

Recently the 1.14.1 release went out

Is there anything else still needed for this issue?

@rakhmets rakhmets closed this as completed Jun 6, 2024
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

No branches or pull requests

6 participants