Skip to content

Commit

Permalink
Add TODO items to code and README
Browse files Browse the repository at this point in the history
  • Loading branch information
Ricks-Lab committed Dec 2, 2018
1 parent cc3dd80 commit c8092dc
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
10 changes: 10 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,3 +31,13 @@
name will be included in the name of the testData subdirectory for the current run.


## Development plans
* GPU multi-threaded implementation. Currently total_gpu_threads = total_gpu_count, a future development opportunity is to implement a max number of threads per GPU
* Consider using opencl instead of lshw to get valid GPU compute platforms, but maybe won't work for cuda apps
* Read benchMT command line options from mode lines in BenchCFG file
* Remove -device arg if specified, since -device is automatically added based on slot assignment
* Need to make a lock_file in the working directory to prevent a second occurrence of benchMT from using the same directory
* Should consider executing job with time command. This should give total and CPU time metrics
* Need to figure out how to run a job without a shell
* Deal with an immediate fail to spawn a process when executing a job

2 changes: 2 additions & 0 deletions benchMT
Original file line number Diff line number Diff line change
Expand Up @@ -457,6 +457,7 @@ class JOB_LIST:
job = BENCH_JOB(uuid4().hex)
job.app_name = lineitems[0]
if len(lineitems) > 1:
# TODO remove -device arg if specified
job.app_args = lineitems[1]
else:
job.app_args = ""
Expand Down Expand Up @@ -1086,6 +1087,7 @@ def main():
env.psv_file_ptr = open(env.psv_file, 'w')
print(__program_name__ + " " + __version__ + " \u2015 SETI MB Benchmarking Utility \u2015 Linux edition\n",
file=env.sum_file_ptr)
# TODO check if there is already and instance running in this directory
if args.noBS == False:
print("Suspending BOINC\n", file=env.sum_file_ptr)
env.suspend_boinc()
Expand Down

0 comments on commit c8092dc

Please sign in to comment.