<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div class=""><br class=""></div><div class=""> Since we cannot post issues (reported here <a href="https://forum.gitlab.com/t/creating-new-issue-gives-cannot-create-issue-getting-whoops-something-went-wrong-on-our-end/41966?u=bsmith" class="">https://forum.gitlab.com/t/creating-new-issue-gives-cannot-create-issue-getting-whoops-something-went-wrong-on-our-end/41966?u=bsmith</a>) here is my issue so I don't forget it.</div><div class=""><br class=""></div><div class=""> I think </div><div class=""><br class=""></div><div class=""><div class=""> err = WaitForCUDA();CHKERRCUDA(err);</div><div class=""> ierr = PetscLogGpuTimeEnd();CHKERRQ(ierr);</div></div><div class=""><br class=""></div><div class="">should be changed to include WaitForCUDA() actually WaitForDevice() inside the PetscLogGpuTimeEnd().</div><div class=""><br class=""></div><div class="">Currently sometimes the WaitForCUDA() is missing in a few places resulting in bad timing. </div><div class=""><br class=""></div><div class="">Also some _SeqCUDA() don't have the PetscLogGpuTimeEnd() and need to be fixed.</div><div class=""><br class=""></div><div class="">The current model is a maintenance nightmare.</div><div class=""><br class=""></div><div class="">Does anyone see a problem with making this change?</div><div class=""><br class=""></div><div class="">Thanks</div><div class=""><br class=""></div><div class=""> Barry</div><div class=""><br class=""></div></body></html>