cp2k / dbcsr

DBCSR: Distributed Block Compressed Sparse Row matrix library
https://cp2k.github.io/dbcsr/
GNU General Public License v2.0
135 stars 46 forks source link

Extend database of autotuned and predicted A100 kernels #680

Closed mkrack closed 1 year ago

mkrack commented 1 year ago
dev-zero commented 1 year ago

@mkrack do you see any notable performance improvements?

dev-zero commented 1 year ago

ACK from my side: @mkrack unless you want to do it yourself, I will pull the pre-commit.ci fix into their respective commits with git absorb and force push. Essentially running this after checking out your feature branch:

git reset --soft HEAD~1
git absorb --base develop
mkrack commented 1 year ago

@mkrack do you see any notable performance improvements?

I do not observe a performance boost on JURECA. There are simply too many CPU cores available per GPU (32 core per A100 while Piz Daint has only 12 cores per P100). On the other hand, you need tuned parameters for an efficient usage of the A100 GPUs, otherwise, it might be better to ignore the GPUs completely.

hfp commented 1 year ago

Did anyone try the OpenCL backend?

mkrack commented 1 year ago

ACK from my side: @mkrack unless you want to do it yourself, I will pull the pre-commit.ci fix into their respective commits with git absorb and force push. Essentially running this after checking out your feature branch:

git reset --soft HEAD~1
git absorb --base develop

@dev-zero Applied git-adsorb, but not sure, if the result is what you wanted?

dev-zero commented 1 year ago

ACK from my side: @mkrack unless you want to do it yourself, I will pull the pre-commit.ci fix into their respective commits with git absorb and force push. Essentially running this after checking out your feature branch:

git reset --soft HEAD~1
git absorb --base develop

@dev-zero Applied git-adsorb, but not sure, if the result is what you wanted?

Sorry, I forgot the git rebase -i --autosquash develop up there (or the --and-rebase option for git absorb).

mkrack commented 1 year ago

Sorry, I forgot the git rebase -i --autosquash develop up there (or the --and-rebase option for git absorb).

@dev-zero Better you squash the commits.

dev-zero commented 1 year ago

Sorry, I forgot the git rebase -i --autosquash develop up there (or the --and-rebase option for git absorb).

@dev-zero Better you squash the commits.

Done. Sorry for the confusion. For completeness, the complete command would have been either

# to revert the commit done by the precommit.ci bot back into staged
git reset --soft HEAD~1
# to let git absorb figure out which hunks belong to which commits, commit them as fixups and then do a rebase
# ... if no reference given will automatically use the one from which this branch forked off
git absorb --and-rebase

or if manual control is wanted:

git reset --soft HEAD~1
# only generate commits
git absorb
# do the rebase with squash separately
git rebase -i --autosquash develop