http://www.perlmonks.org?node_id=11135522


in reply to Re^4: Inline::CUDA : looking for feedback
in thread Inline::CUDA : looking for feedback

There's 15 failed tests ("Failed 15/20 test programs."): I'll need to go through each to find the actual problems.

If all are simply quoting uses, then the solution is straightforward (if a little tedious to implement across the board). Let's see how that pans out before worrying about config modifications and the like.

I recall you specified a minimum Perl version of v5.6. I've checked https://perldoc.perl.org/5.6.0/functions/quotemeta and https://perldoc.perl.org/5.6.0/perlop; quotemeta, \Q and \E are all available in that version, so no issues there.

"If all else fails I will have to resort to prompting the user during installation. Any opinions on that style?"

As I said, let's wait and see if secondary options are necessary; however, my general "opinions on that style" are:

"Thanks for your time spent and valuable feedback. I am not in a hurry, please work on it in your own pace."

No worries. I am rather eager to see how this works.

I have, in fact, spent most of this week (at $work) modifying modules and repeatedly going through the make cycle, tweaking tests to reflect the changes. While I'm happy to do this, "All work and no play makes Ken a dull boy.", so I need a break. As I said this morning, I can probably look into this further over the weekend. [Current lockdown restrictions means that I'm not even allowed guests in my home, so I'm certainly free to do this work.]

— Ken

Replies are listed 'Best First'.
Re^6: Inline::CUDA : looking for feedback
by bliako (Monsignor) on Jul 30, 2021 at 12:33 UTC

    I have updated the github repository with a new version which uses ShellQuote::Any to do (some of) the quoting. I have also added a test which tries to run some perl-scripts with space in their name. That's in t/002-spaces-spaces-spaces.t.

    I have also added another repository: https://github.com/hadjiprocopis/perl-nvidia2-ml which takes over nvidia::ml, fixes its compilation errors and adds 1 more binding. If you install this first, it will be easier to enquire the GPU in current module. It requires nvidia-ml binary library which must be in the toolkit you downloaded. If you can't install that then it's not a problem but 1 test will fail (t/001-enquire-nvidia-gpu.t).

    I suggest you run perl Makefile.PL and if successful, then inspect config/Inline-CUDA.conf to see if cc, cxx, nvcc, ld are all set to valid names (unquoted).

    Following that, run just a few tests: t/000-require-modules.t, t/002-spaces-spaces-spaces.t, t/008-config.t, t/011-basic.t . The last two are compiling CUDA code. If all goes well, then it's likely that all tests will succeed. 031-matrix-multiplication-pass-parameters.t does a matrix multiplication with data passed from the Perl script and results returned back. That's a good base for any CUDA code you may want to write.

    The prompting during installation is only for windows, the user must somehow indicate where a c-compiler is if not in path already.

    BTW, you will be using a newer version of nvcc than mine. So a lot can go wrong there too.

    bw, bliako

[reply]
[d/l]
[select]
      "I suggest you run perl Makefile.PL ..." 
      $ perl Makefile.PL
PACK001=HASH(0x800172c08) Changes
PACK001=HASH(0x800172c08) ignore.txt
PACK001=HASH(0x800172c08) lib
PACK001=HASH(0x800172c08) nvidia-ml-pl.pdf
PACK001=HASH(0x800172c08) nvml.h
PACK001=HASH(0x800172c08) nvml_wrap.c
PACK001=HASH(0x800172c08) README
PACK001=HASH(0x800172c08) README.md
PACK001=HASH(0x800172c08) xt
PACK001=HASH(0x800172c08) .
PACK001=HASH(0x800172c08) nvidia2
PACK001=HASH(0x800172c08) blib/lib/nvidia2/ml.pm
PACK001=HASH(0x800172c08) blib/lib/nvidia2/smi.pm
PACK001=HASH(0x800172c08) ml
PACK001=HASH(0x800172c08) blib/lib/nvidia2/ml/bindings.pm
PACK001=HASH(0x800172c08) blib/lib/nvidia2/ml/Utils.pm
Warning (mostly harmless): No library found for -lnvidia-ml
Generating a Unix-style Makefile
Writing Makefile for nvidia2::ml::bindings
Writing MYMETA.yml and MYMETA.json

      [download]
      "... and if successful, then inspect config/Inline-CUDA.conf ..."

      Couldn't find Inline-CUDA.conf even with a case-insensitive search.

      I tried a make anyway: 

      $ make
cp lib/nvidia2/ml.pm blib/lib/nvidia2/ml.pm
cp lib/nvidia2/ml/Utils.pm blib/lib/nvidia2/ml/Utils.pm
cp lib/nvidia2/ml/bindings.pm blib/lib/nvidia2/ml/bindings.pm
cp lib/nvidia2/smi.pm blib/lib/nvidia2/smi.pm
Running Mkbootstrap for bindings ()
chmod 644 "bindings.bs"
"/home/ken/perl5/perlbrew/perls/perl-5.34.0/bin/perl.exe" -MExtUtils::
+Command::MM -e 'cp_nonempty' -- bindings.bs blib/arch/auto/nvidia2/ml
+/bindings/bindings.bs 644
gcc -c  -I. -DPERL_USE_SAFE_PUTENV -U__STRICT_ANSI__ -D_GNU_SOURCE -fw
+rapv -fno-strict-aliasing -pipe -fstack-protector-strong -D_FORTIFY_S
+OURCE=2 -DUSEIMPORTLIB -O3   -DVERSION=\"5.00\" -DXS_VERSION=\"5.00\"
+  "-I/home/ken/perl5/perlbrew/perls/perl-5.34.0/lib/5.34.0/cygwin-thr
+ead-multi/CORE"   nvml_wrap.c
rm -f blib/arch/auto/nvidia2/ml/bindings/bindings.dll
g++  --shared  -Wl,--enable-auto-import -Wl,--export-all-symbols -Wl,-
+-enable-auto-image-base -L/usr/local/lib -fstack-protector-strong  nv
+ml_wrap.o  -o blib/arch/auto/nvidia2/ml/bindings/bindings.dll  \
  /home/ken/perl5/perlbrew/perls/perl-5.34.0/lib/5.34.0/cygwin-thread-
+multi/CORE/cygperl5_34_0.dll -lpthread -ldl   \

/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: 
+nvml_wrap.o:nvml_wrap.c:(.text+0x6af5): undefined reference to `nvmlI
+nit'
/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: 
+nvml_wrap.o:nvml_wrap.c:(.text+0x6c05): undefined reference to `nvmlS
+hutdown'

... several screenfuls of lines like that ...

/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: 
+nvml_wrap.o:nvml_wrap.c:(.text+0x1e977): undefined reference to `nvml
+DeviceGetSupportedMemoryClocks'
/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: 
+nvml_wrap.o:nvml_wrap.c:(.text+0x1e9ca): undefined reference to `nvml
+DeviceGetSupportedGraphicsClocks'
collect2: error: ld returned 1 exit status
make: *** [Makefile:482: blib/arch/auto/nvidia2/ml/bindings/bindings.d
+ll] Error 1

      [download]

      — Ken

[reply]
[d/l]
[select]

        nvidia2-ml's Makefile.PL contains the directive to link to nvida-ml dll . Spexify there the dir this library resides prepending a -L adir use quotes if spaces in adir.

        re: the config file etc., i was referring to Inline::CUDA. let me know

        bliako

[reply]
[d/l]

In Section Cool Uses for Perl