[petsc-dev] nested timers transfers and reductions

[Klaij, Christiaan]

Thanks Barry, that looks good!

Chris


dr. ir. Christiaan Klaij  | Senior Researcher | Research & Development
MARIN | T +31 317 49 33 44 | mailto:C.Klaij at marin.nl | http://www.marin.nl

MARIN news: http://www.marin.nl/web/News/News-items/New-correction-procedure-for-shallowwater-effects-in-ship-speed-trials.htm

________________________________________
From: Smith, Barry F. <bsmith at mcs.anl.gov>
Sent: Monday, April 09, 2018 11:51 PM
To: Klaij, Christiaan
Cc: petsc-dev at mcs.anl.gov; Koos Huijssen
Subject: Re: nested timers transfers and reductions

   Hopefully all these problems are resolved with https://bitbucket.org/petsc/petsc/pull-requests/925/fix-some-missing-functionality-for-nested/diff

  Barry



> On Apr 9, 2018, at 4:37 AM, Klaij, Christiaan <C.Klaij at marin.nl> wrote:
>
> Hi Barry,
>
> I've put the functions and fortran bindings in the attached
> files. In our code, we use them as follows:
>
>  CALL mpi_bcast(value,1,MPI_LOGICAL,iproc,MPI_COMM_WORLD,ier)
>  IF(ier/=0) CALL tracing_abort('ERROR in bcast logical root !')
>
>  IF (iproc==thisprocess) THEN
>     ier = PetscAdminSend(1*(nprocesses-1),MPI_LOGICAL)
>  ELSE
>     ier = PetscAdminRecv(1,MPI_LOGICAL)
>  END IF
>
> I also noted that you changed the treshold from 0.1% to 0.01%, so
> I tried to change it as a user with PetscLogSetThreshold, but
> this function seems not accessible to users (and there's no
> fortran version or manual page).
>
> Chris
>
>
> dr. ir. Christiaan Klaij  | Senior Researcher | Research & Development
> MARIN | T +31 317 49 33 44 | mailto:C.Klaij at marin.nl | http://www.marin.nl
>
> MARIN news: http://www.marin.nl/web/News/News-items/Kom-zaterdag-10-november-naar-de-open-dag-in-Wageningen.htm
>
> ________________________________________
> From: Smith, Barry F. <bsmith at mcs.anl.gov>
> Sent: Saturday, April 07, 2018 5:36 PM
> To: Klaij, Christiaan
> Cc: petsc-dev at mcs.anl.gov; Koos Huijssen
> Subject: Re: nested timers transfers and reductions
>
>> On Apr 6, 2018, at 5:22 AM, Klaij, Christiaan <C.Klaij at marin.nl> wrote:
>>
>> Hi Barry,
>>
>> In our version of the nested timers, we had the functions
>> PetscAdminSend, PetscAdminRecv and PetscAdminReduce. So I was
>> expecting to find them at the end of the file
>> petsc-3.8.4/src/sys/logging/xmllogevent.c but apparently
>> you (re)moved them?
>
>   I do not remember why I would have removed these functions. If you can send a patch that would return them I would gladly try to get them working with PETSc again.
>
>    Barry
>
>>
>> We basically used these functions to fill the transfer(GiB/s) and
>> reductions/s columns of the log just like we use PetscLogFlops to
>> fill the compute(Mflops) column for our registered events.
>>
>> How should I do this with the petsc version of the nested timers?
>>
>> Chris
>>
>>
>> dr. ir. Christiaan Klaij  | Senior Researcher | Research & Development
>> MARIN | T +31 317 49 33 44 | mailto:C.Klaij at marin.nl | http://www.marin.nl
>>
>> MARIN news: http://www.marin.nl/web/News/News-items/Application-of-potential-flow-methods-to-fast-displacement-ships-at-transcritical-speeds-in-shallow-water-1.htm
>>
>
> <petsc_MPIadminF.F90><petsc_MPIadmin.c>