Giter Club home page Giter Club logo

Comments (17)

drossetti avatar drossetti commented on May 24, 2024

@tonycurtis which OS/distro were you using?

from gdrcopy.

tonycurtis avatar tonycurtis commented on May 24, 2024

from gdrcopy.

drossetti avatar drossetti commented on May 24, 2024

Eventually I'll need to switch to autoconf/automake to take care of that.

from gdrcopy.

pakmarkthub avatar pakmarkthub commented on May 24, 2024

@tonycurtis We are switching to autoconf. Can you try this branch (https://github.com/pakmarkthub/gdrcopy/tree/dev-autotools) out and see whether it addresses your issue or not?

from gdrcopy.

tonycurtis avatar tonycurtis commented on May 24, 2024

Sure, will give it a go...

from gdrcopy.

tonycurtis avatar tonycurtis commented on May 24, 2024

Looks like it works, was able to compile gdrdrv

from gdrcopy.

pakmarkthub avatar pakmarkthub commented on May 24, 2024

Thank you for the confirmation.

Let me double-check just in case. Did you pass "--enable-test" to ./configure? Right now, copybw is not compiled by default since it is a test program.

from gdrcopy.

tonycurtis avatar tonycurtis commented on May 24, 2024

Oh, no. This is what configure gives me with --enable-test

configure: Test is enabled
checking for cuMemAlloc in -lcuda... yes
checking for cudaMalloc in -lcudart... no
configure: error: in `/usr/local/src/gdrcopy/autoconf':
configure: error: libcudart not found
See `config.log' for more details

There's nothing in the configure --help output about where CUDA is

from gdrcopy.

pakmarkthub avatar pakmarkthub commented on May 24, 2024

It cannot find cuda. Could you make sure that LD_LIBRARY_PATH, LIBRARY_PATH, C_PATH points to your cuda folder and run the script again?

from gdrcopy.

tonycurtis avatar tonycurtis commented on May 24, 2024

LD_LIBRARY_PATH and CPATH were set, but not LIBRARY_PATH. The latter is required. I know this is a matter of autoconf debate but being able to set this via --with-cuda would be nice.

from gdrcopy.

tonycurtis avatar tonycurtis commented on May 24, 2024

How do I run the tests?

eureka[1](/usr/local/src/gdrcopy/autoconf) make test
make: *** No rule to make target `test'.  Stop.
eureka[1](/usr/local/src/gdrcopy/autoconf) make tests
make: Nothing to be done for `tests'.
eureka[1](/usr/local/src/gdrcopy/autoconf) make check
Making check in src
make[1]: Entering directory `/usr/local/src/gdrcopy/autoconf/src'
Making check in gdrdrv
make[2]: Entering directory `/usr/local/src/gdrcopy/autoconf/src/gdrdrv'
make[2]: *** No rule to make target `check'.  Stop.
make[2]: Leaving directory `/usr/local/src/gdrcopy/autoconf/src/gdrdrv'
make[1]: *** [check-recursive] Error 1
make[1]: Leaving directory `/usr/local/src/gdrcopy/autoconf/src'
make: *** [check-recursive] Error 1
eureka[1](/usr/local/src/gdrcopy/autoconf) cd tests
eureka[1](/usr/local/src/gdrcopy/autoconf/tests) make
  CXX      basic.o
  CXXLD    basic
  CXX      copybw.o
  CXXLD    copybw
/usr/bin/ld: copybw.o: undefined reference to symbol 'clock_gettime@@GLIBC_2.2.5'
/usr/bin/ld: note: 'clock_gettime@@GLIBC_2.2.5' is defined in DSO /lib64/librt.so.1 so try adding it to the linker command line
/lib64/librt.so.1: could not read symbols: Invalid operation
collect2: error: ld returned 1 exit status
make: *** [copybw] Error 1

from gdrcopy.

pakmarkthub avatar pakmarkthub commented on May 24, 2024

Thank you for the feedback. Let me improve the build script and contact you back again. Surprisingly, the machines we have tested so far do not show this problem.

from gdrcopy.

tonycurtis avatar tonycurtis commented on May 24, 2024

It might be the much older Fedora. I can give you an account for testing if you like.

from gdrcopy.

pakmarkthub avatar pakmarkthub commented on May 24, 2024

@tonycurtis Thank you for waiting. I eventually have time to come back to this issue.

Can you give it another try? If the issue is still there, can I ask for an account so that I can debug on your machine?

P.S. I also add "--with-cuda" as you requested.

from gdrcopy.

tonycurtis avatar tonycurtis commented on May 24, 2024

That looks good, thanks. You could presumably simplify autogen.sh to something like e.g.

UCX

from gdrcopy.

pakmarkthub avatar pakmarkthub commented on May 24, 2024

Thank you for the feedback as well.

I will close this issue after the PR gets merged upstream.

from gdrcopy.

pakmarkthub avatar pakmarkthub commented on May 24, 2024

The fix has been integrated to the master branch. Closing this issue.

from gdrcopy.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.