Skip to content

Conversation

@koparasy
Copy link
Member

This patch adds compiling configurations for XSBench, RSBench, and miniFE using the OpenMP offload programming model.

Copy link
Member

@ggeorgakoudis ggeorgakoudis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please reply to comments and address issues

config.yaml Outdated
run: 'env OMP_NUM_THREADS=1 OMP_PROC_BIND=true ./XSBench'
input: '-t 1 -k 1 -s small'
measure:
input: '-t 1 -m event -s small'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this input compatible with the other configurations (omp, seq)?

config.yaml Outdated
run: 'env OMP_NUM_THREADS=1 OMP_PROC_BIND=true ./rsbench'
input: '-t 1 -s small'
measure:
input: '-s small -m event -t 1'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this input compatible with the other configurations (seq, omp)?

config.yaml Outdated
],
omp-offload: [ 'cd openmp45/src;
make -j CC=clang CXX=clang++
CFLAGS="-v -O3 -fopenmp -fopenmp-targets=nvptx64-nvidia-cuda
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove -v

print_CLI_error();
}
- else
+ else if( strcmp(arg, "-t") == 0 )
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this to enable multi-threaded target creation? Is an option on the number of threads correct? Shouldn't be 1 for correct program execution?

+ }
// n_gridpoints (-g)
- if( strcmp(arg, "-g") == 0 )
+ else if( strcmp(arg, "-g") == 0 )
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to if, otherwise -t precludes -g

char * arg = argv[i];

+ // This will be ignored
+ if( strcmp(arg, "-t") == 0 )
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is an option on the number of threads correct? Shouldn't be 1 for correct program execution?

config.yaml Outdated

XSBench:
fetch: 'git clone https://github.com/ANL-CESAR/XSBench.git'
fetch: 'git clone https://github.com/ANL-CESAR/XSBench.git;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the command to clone/checkout the specific commit that this patch works with. Also for RSBench fetch. We should do it for every repo that needs a patch.

@ggeorgakoudis
Copy link
Member

ggeorgakoudis commented Apr 13, 2021

Possibly add:

miniQMC
QMCpack
gridmini (JD)
SPEC accel(?)
SPEC HPG (?)
Kripke (change header for OpenMP offloading policy)
Laghos (MFEM, what needs to be changed?)
HPG-MG (?) https://bitbucket.org/hpgmg/hpgmg/src (JD)

- Move offload applications to another configuration file.
- Add Clang and IBM compilation toolchain for applications.
- Extends Harness to parse information out of the nvlink info
- The parsed data are stored under the results/results-'application name'.yaml file
- Add miniQMC application
@koparasy koparasy force-pushed the features/openmp-offload branch from d4d3ca2 to 17bcb73 Compare April 16, 2021 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants