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.

Sign up for GitHub

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

Jump to bottom

PCL-git build fail with CUDA? #1674

Closed
jiapei100 opened this issue Aug 9, 2016 · 5 comments
Closed

PCL-git build fail with CUDA? #1674

jiapei100 opened this issue Aug 9, 2016 · 5 comments

Comments

@jiapei100
Copy link

⚠️ This is a issue tracker, please use our mailing list for questions: www.pcl-users.org. ⚠️

Your Environment

  • Operating System and version: Ubuntu 16.04
  • Compiler: GCC 5.3.0 + Cuda 8.0.27 + CUDA_ARCH: 5.2 (GTX 980M)
  • PCL Version: 1.8.0 (Current Git)

.../pcl/gpu/people/src/cuda/nvidia/NPP_staging.cu(2082): error: identifier "__iAtomicCAS" is undefined

1 error detected in the compilation of "/tmp/tmpxft_00007691_00000000-5_NPP_staging.cpp4.ii".
-- Removing .../pcl/build/gpu/people/CMakeFiles/cuda_compile.dir/src/cuda/nvidia/./cuda_compile_generated_NPP_staging.cu.o
/usr/bin/cmake -E remove .../pcl/build/gpu/people/CMakeFiles/cuda_compile.dir/src/cuda/nvidia/./cuda_compile_generated_NPP_staging.cu.o
CMake Error at cuda_compile_generated_NPP_staging.cu.o.cmake:266 (message):
Error generating file
.../pcl/build/gpu/people/CMakeFiles/cuda_compile.dir/src/cuda/nvidia/./cuda_compile_generated_NPP_staging.cu.o

gpu/people/CMakeFiles/pcl_gpu_people.dir/build.make:1602: recipe for target 'gpu/people/CMakeFiles/cuda_compile.dir/src/cuda/nvidia/cuda_compile_generated_NPP_staging.cu.o' failed
make[2]: *** [gpu/people/CMakeFiles/cuda_compile.dir/src/cuda/nvidia/cuda_compile_generated_NPP_staging.cu.o] Error 1
make[2]: Leaving directory '.../pcl/build'
CMakeFiles/Makefile2:15380: recipe for target 'gpu/people/CMakeFiles/pcl_gpu_people.dir/all' failed
make[1]: *** [gpu/people/CMakeFiles/pcl_gpu_people.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

Exactly the same issue as asked here:
https://devtalk.nvidia.com/default/topic/950328/cant-install-pcl-point-cloud-with-cuda-help/

Can anybody help please?

Cheers
Pei
@hanshammel1337
Copy link

Did you solve this problem?

@hanshammel1337
Copy link

I solved this by changing line 2073 in cuda_compile_generated_NPP_staging.cu
https://github.com/PointCloudLibrary/pcl/blob/master/gpu/people/src/cuda/nvidia/NPP_staging.cu#L2073
to #if ((defined __CUDA_ARCH__) && (__CUDA_ARCH__ < 200))

I found the solution here https://devtalk.nvidia.com/default/topic/950328/cant-install-pcl-point-cloud-with-cuda-help/

CUDA 8.0.44, GCC 5.4 (Ubuntu 16.04 LTS), Arch 6.1

I didn't check if the people module works correctly, but the build is fine now. Maybe a pull request can be created for this.

@jiapei100
Copy link
Author

@hanshammel1337

Yes, That's great... Problem solved... Your solution is correct !!!

@jspricke
Copy link
Member

jspricke commented Jan 5, 2017

@hanshammel1337 great find. Could you please open a pull request?

@jspricke jspricke reopened this Jan 5, 2017
@caomw
Copy link

caomw commented May 9, 2017

Good

anuppari added a commit to anuppari/pcl that referenced this issue Jul 5, 2017
@anuppari
Fix issue PointCloudLibrary#1674
d81aa3c
SergioRAgostinho added a commit that referenced this issue Jul 6, 2017
@SergioRAgostinho
Merge pull request #1926 from anuppari/master
de90d4b 
Fix issue #1674
taketwo pushed a commit that referenced this issue Jul 18, 2017
@anuppari @taketwo
Fix issue #1674
1c35f48
UnaNancyOwen pushed a commit to UnaNancyOwen/pcl that referenced this issue Nov 24, 2017
@anuppari @UnaNancyOwen
Fix issue PointCloudLibrary#1674
3c85ce1
@SergioRAgostinho SergioRAgostinho mentioned this issue Aug 6, 2018
Closed
@SergioRAgostinho SergioRAgostinho mentioned this issue Aug 25, 2018
Merged
dantwinkler added a commit to 6RiverSystems/pcl that referenced this issue Feb 21, 2019
@dantwinkler
fix: Dependencies (#5)
0c750a4 
* Revert "PLYReader: thread safe colors"

This reverts commit 4c57676 from 1.8.1
release because it breaks ABI of pcl_io.

* Bump version to 1.8.1

* Add new gtest path

* Fix issue with finding pcl deployed out of path (PointCloudLibrary#1923)

* fix issue with finding pcl deployed out of path

* The path is set relative to PCLConfig.cmake

* Fix issue PointCloudLibrary#1674

* Added CUDA compute capability 5.3

* Update Ensenso tutorial for Ensenso X devices

* need to include instantiate.hpp to use PCL_INSTANTIATE

* Fixed error in documentation.

pcl::visualization::createCube was documented incorrectly and there were spelling mistakes.

* Fixed spelling and grammar errors

* Add test for macro _USE_MATH_DEFINES.

* Final changelist update for 1.8.1

* Fix: Install cuda based filters (#1)

* feat: Add install step for io buffers.

* fix: Build on circleci.

* fix: Add build job.

* fix: Fix circleci project.

* fix: Fix docker image name.

* fix: Build using correct docker image.

* fix: Correct env variable name.

* fix: Fixed build script.

* fix: Fix build image name.

* fix: Add directories to .gitignore.

* fix: remove circle

* fix: add x permissions

* fix: build

* fix: upload artifact

* fix: FPM

* fix: FPM

* fix: permissions

* fix: semver

* fix: get version

* fix: github repo for semrel

* fix: make install

* fix: Slim down pcl (no cuda).

* fix: Fix build scripts.

* fix: Add missing build step.

* fix: Fix package name and repo folder.

* fix: Fix package name and repo folder.

* fix: Add features.

* fix: Add surface.

* fix: Use default build settings.

* fix: Turn off VTK.

* fix: Force rebuild.

* fix: Fix build.

* fix: Fix build.

* fix: Fix build.

* fix: Fix build.

* fix: Smaller pcl library.

* fix: Fix build.

* fix: Remove sudo.

* fix: Fix build scripts.

* fix: Fix build scripts.

* fix: Fix build scripts.

* fix: Fix build scripts.

* fix: Fix build scripts.

* fix: Add qhull and png libraries.

* fix: Add zlib library.

* fix: Remove workspace.

* fix: Clean up build script and fix depends.

* fix: Use ccache.

* fix: Fix build permissions.

* fix: Remove jessie build.
dantwinkler added a commit to 6RiverSystems/pcl that referenced this issue Feb 21, 2019
@dantwinkler
fix: Fix dependencies. (#4)
1ed71f3 
* Revert "PLYReader: thread safe colors"

This reverts commit 4c57676 from 1.8.1
release because it breaks ABI of pcl_io.

* Bump version to 1.8.1

* Add new gtest path

* Fix issue with finding pcl deployed out of path (PointCloudLibrary#1923)

* fix issue with finding pcl deployed out of path

* The path is set relative to PCLConfig.cmake

* Fix issue PointCloudLibrary#1674

* Added CUDA compute capability 5.3

* Update Ensenso tutorial for Ensenso X devices

* need to include instantiate.hpp to use PCL_INSTANTIATE

* Fixed error in documentation.

pcl::visualization::createCube was documented incorrectly and there were spelling mistakes.

* Fixed spelling and grammar errors

* Add test for macro _USE_MATH_DEFINES.

* Final changelist update for 1.8.1

* Fix: Install cuda based filters (#1)

* feat: Add install step for io buffers.

* fix: Build on circleci.

* fix: Add build job.

* fix: Fix circleci project.

* fix: Fix docker image name.

* fix: Build using correct docker image.

* fix: Correct env variable name.

* fix: Fixed build script.

* fix: Fix build image name.

* fix: Add directories to .gitignore.

* fix: remove circle

* fix: add x permissions

* fix: build

* fix: upload artifact

* fix: FPM

* fix: FPM

* fix: permissions

* fix: semver

* fix: get version

* fix: github repo for semrel

* fix: make install

* fix: Slim down pcl (no cuda).

* fix: Fix build scripts.

* fix: Add missing build step.

* fix: Fix package name and repo folder.

* fix: Fix package name and repo folder.

* fix: Add features.

* fix: Add surface.

* fix: Use default build settings.

* fix: Turn off VTK.

* fix: Force rebuild.

* fix: Fix build.

* fix: Fix build.

* fix: Fix build.

* fix: Fix build.

* fix: Smaller pcl library.

* fix: Fix build.

* fix: Remove sudo.

* fix: Fix build scripts.

* fix: Fix build scripts.

* fix: Fix build scripts.

* fix: Fix build scripts.

* fix: Fix build scripts.

* fix: Add qhull and png libraries.

* fix: Add zlib library.

* fix: Remove workspace.

* fix: Clean up build script and fix depends.

* fix: Use ccache.

* fix: Fix build permissions.

* fix: Remove jessie build.

* fix: Use default  step

* fix: Clean up build directory.

* fix: Enable upload to artifactory.
dantwinkler added a commit to 6RiverSystems/pcl that referenced this issue Feb 26, 2019
@dantwinkler
feat: Release to master. (#3)
cbd8490 
* Revert "PLYReader: thread safe colors"

This reverts commit 4c57676 from 1.8.1
release because it breaks ABI of pcl_io.

* Bump version to 1.8.1

* Add new gtest path

* Fix issue with finding pcl deployed out of path (PointCloudLibrary#1923)

* fix issue with finding pcl deployed out of path

* The path is set relative to PCLConfig.cmake

* Fix issue PointCloudLibrary#1674

* Added CUDA compute capability 5.3

* Update Ensenso tutorial for Ensenso X devices

* need to include instantiate.hpp to use PCL_INSTANTIATE

* Fixed error in documentation.

pcl::visualization::createCube was documented incorrectly and there were spelling mistakes.

* Fixed spelling and grammar errors

* Add test for macro _USE_MATH_DEFINES.

* Final changelist update for 1.8.1

* Fix: Install cuda based filters (#1)

* feat: Add install step for io buffers.

* fix: Build on circleci.

* fix: Add build job.

* fix: Fix circleci project.

* fix: Fix docker image name.

* fix: Build using correct docker image.

* fix: Correct env variable name.

* fix: Fixed build script.

* fix: Fix build image name.

* fix: Add directories to .gitignore.

* fix: remove circle

* fix: add x permissions

* fix: build

* fix: upload artifact

* fix: FPM

* fix: FPM

* fix: permissions

* fix: semver

* fix: get version

* fix: github repo for semrel

* fix: make install

* fix: Slim down pcl (no cuda).

* fix: Fix build scripts.

* fix: Add missing build step.

* fix: Fix package name and repo folder.

* fix: Fix package name and repo folder.

* fix: Add features.

* fix: Add surface.

* fix: Use default build settings.

* fix: Turn off VTK.

* fix: Force rebuild.

* fix: Fix build.

* fix: Fix build.

* fix: Fix build.

* fix: Fix build.

* fix: Smaller pcl library.

* fix: Fix build.

* fix: Remove workspace file.

* fix: Merge master

* fix: Fix clang install

* fix: Run on bionic.

* fix: Remove sudo

* fix: Fix build script.

* fix: Fix debian package name.

* fix: Fix depends install.

* fix: Fix depends install.

* fix: Use xenial.

* fix: Do not use clang-6.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jiapei100 @jspricke @SergioRAgostinho @caomw @hanshammel1337