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

[C++][CI][CUDA] CUDA Crossbow build failure #37523

Closed
zeroshade opened this issue Sep 1, 2023 · 0 comments · Fixed by #37497
Closed

[C++][CI][CUDA] CUDA Crossbow build failure #37523

zeroshade opened this issue Sep 1, 2023 · 0 comments · Fixed by #37497

Comments

@zeroshade
Copy link
Member

zeroshade commented Sep 1, 2023

Describe the bug, including details regarding any error messages, version, and platform.

#37365 was built locally using a newer version of the CUDA driver API than the crossbow builds run with causing the crossbow build to fail due to a change in macro names. This puts the macro name back to allow it to build with the older version of the cuda driver api.

test-cuda-cpp:
https://github.com/ursacomputing/crossbow/actions/runs/6043867298/job/16401600820#step:5:5758

FAILED: src/arrow/gpu/CMakeFiles/arrow_cuda_objlib.dir/cuda_context.cc.o 
/usr/local/bin/sccache /usr/lib/ccache/c++  -DARROW_EXPORTING -DARROW_HAVE_RUNTIME_AVX2 -DARROW_HAVE_RUNTIME_AVX512 -DARROW_HAVE_RUNTIME_BMI2 -DARROW_HAVE_RUNTIME_SSE4_2 -DARROW_HAVE_SSE4_2 -DARROW_NO_DEPRECATED_API -DARROW_WITH_BROTLI -DARROW_WITH_BZ2 -DARROW_WITH_LZ4 -DARROW_WITH_RE2 -DARROW_WITH_SNAPPY -DARROW_WITH_UTF8PROC -DARROW_WITH_ZLIB -DARROW_WITH_ZSTD -DAZ_RTTI -DBOOST_ALL_NO_LIB -DURI_STATIC_BUILD -Isrc -I/arrow/cpp/src -I/arrow/cpp/src/generated -I_deps/azure_sdk-src/sdk/storage/azure-storage-files-datalake/inc -I_deps/azure_sdk-src/sdk/storage/azure-storage-blobs/inc -I_deps/azure_sdk-src/sdk/storage/azure-storage-common/inc -I_deps/azure_sdk-src/sdk/core/azure-core/inc -I_deps/azure_sdk-src/sdk/identity/azure-identity/inc -isystem /usr/local/cuda/include -isystem /arrow/cpp/thirdparty/flatbuffers/include -isystem /arrow/cpp/thirdparty/hadoop/include -isystem xsimd_ep/src/xsimd_ep-install/include -Wno-noexcept-type  -fdiagnostics-color=always  -Wall -Wno-conversion -Wno-sign-conversion -Wunused-result -Wdate-time -fno-semantic-interposition -msse4.2  -g -Werror -O0 -ggdb -g1 -fPIC   -pthread -std=c++17 -MD -MT src/arrow/gpu/CMakeFiles/arrow_cuda_objlib.dir/cuda_context.cc.o -MF src/arrow/gpu/CMakeFiles/arrow_cuda_objlib.dir/cuda_context.cc.o.d -o src/arrow/gpu/CMakeFiles/arrow_cuda_objlib.dir/cuda_context.cc.o -c /arrow/cpp/src/arrow/gpu/cuda_context.cc
In file included from /arrow/cpp/src/arrow/gpu/cuda_context.cc:29:
/arrow/cpp/src/arrow/gpu/cuda_context.cc: In member function 'virtual arrow::Status arrow::cuda::CudaDevice::Stream::WaitEvent(const arrow::Device::SyncEvent&)':
/arrow/cpp/src/arrow/gpu/cuda_context.cc:326:57: error: 'CU_EVENT_WAIT_DEFAULT' was not declared in this scope; did you mean 'CU_EVENT_DEFAULT'?
  326 |                    cuStreamWaitEvent(value(), cu_event, CU_EVENT_WAIT_DEFAULT));
      |                                                         ^~~~~~~~~~~~~~~~~~~~~
/arrow/cpp/src/arrow/gpu/cuda_internal.h:41:23: note: in definition of macro 'CU_RETURN_NOT_OK'
   41 |     CUresult __res = (STMT);                                            \
      |                       ^~~~
/arrow/cpp/src/arrow/gpu/cuda_context.cc: In member function 'virtual arrow::Status arrow::cuda::CudaDevice::SyncEvent::Record(const arrow::Device::Stream&, unsigned int)':
/arrow/cpp/src/arrow/gpu/cuda_context.cc:350:20: error: 'cuEventRecordWithFlags' was not declared in this scope
  350 |                    cuEventRecordWithFlags(value(), cuda_stream->value(), flags));
      |                    ^~~~~~~~~~~~~~~~~~~~~~
/arrow/cpp/src/arrow/gpu/cuda_internal.h:41:23: note: in definition of macro 'CU_RETURN_NOT_OK'
   41 |     CUresult __res = (STMT);                                            \
      |                       ^~~~

Component(s)

C++, Continuous Integration, GPU

@zeroshade zeroshade changed the title Cuda Crossbow build failure [C++][CI][CUDA] Cuda Crossbow build failure Sep 1, 2023
@kou kou changed the title [C++][CI][CUDA] Cuda Crossbow build failure [C++][CI][CUDA] CUDA Crossbow build failure Sep 1, 2023
@zeroshade zeroshade added this to the 14.0.0 milestone Sep 5, 2023
zeroshade added a commit that referenced this issue Sep 5, 2023
…pendencies (#37497)

### Rationale for this change
#37365 was built locally using a newer version of the CUDA driver API than the crossbow builds run with causing the crossbow build to fail due to a change in macro names. This puts the macro name back to allow it to build with the older version of the cuda driver api.

* Closes: #37523

Authored-by: Matt Topol <zotthewizard@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
loicalleyne pushed a commit to loicalleyne/arrow that referenced this issue Nov 13, 2023
…UDA dependencies (apache#37497)

### Rationale for this change
apache#37365 was built locally using a newer version of the CUDA driver API than the crossbow builds run with causing the crossbow build to fail due to a change in macro names. This puts the macro name back to allow it to build with the older version of the cuda driver api.

* Closes: apache#37523

Authored-by: Matt Topol <zotthewizard@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
…UDA dependencies (apache#37497)

### Rationale for this change
apache#37365 was built locally using a newer version of the CUDA driver API than the crossbow builds run with causing the crossbow build to fail due to a change in macro names. This puts the macro name back to allow it to build with the older version of the cuda driver api.

* Closes: apache#37523

Authored-by: Matt Topol <zotthewizard@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment