search

IO500 / io500

IO500 Storage Benchmark source code
MIT License
98 stars 31 forks source link

Discussion staging area for the IOR testing branch #20

Closed JulianKunkel closed 3 years ago

gflofst commented 3 years ago

io500.sh line 109 seems to have an error. io500_workdir when it was removed elsewhere. Why remove the prefix?

JulianKunkel commented 3 years ago

@gflofst you can make inline comments to the code, that is easier visible. I made the change cause I made a couple of times minor mistakes typing it. $io500_workdir is harder to spot.

The setup() function now takes an argument and assigns it to the local variable workdir. That makes it clear to users how the variable is called and modularizes the script code better.

gflofst commented 3 years ago

That is more clear when I look at the file itself. The diff is hard to read. I haven't run it, but reading it seems like it is fine.

adilger commented 3 years ago

I was able to run a full test on my local system in "normal" mode, but hit an error when I added --mode=extended to the command line:

[RESULT]                 find       51.540760 kIOPS : time 24.522 seconds
--------------------------------------------------------------------------
MPI_ABORT was invoked on rank 1 in communicator MPI_COMM_WORLD with errorcode -1.

NOTE: invoking MPI_ABORT causes Open MPI to kill all MPI processes.
You may or may not see output from other processes, depending on
exactly when Open MPI kills them.
--------------------------------------------------------------------------
ERROR: a StoneWallingStatusFile is only sensible when splitting write/read into
multiple executions of ior, please use -k, (ior.c:1446)