Junchao, I see this in cupminit.inc (twice)

#if defined(PETSC_HAVE_KOKKOS)
    ierr = PetscKokkosInitialize_Private();CHKERRQ(ierr);
    PetscBeganKokkos = PETSC_TRUE;
#endif

And I see

ierr = PetscKokkosInitializeCheck();CHKERRQ(ierr);

In the Kokkos operators.

Are these redundant?

On Thu, Oct 15, 2020 at 10:44 PM Mark Adams <[email protected]> wrote:

> Si it seems like these two calls in cupminit.inc are
> inconsistent with lazy:
>
> 22:41 adams/gamg-reduce-opt-cuda *= ~/petsc$ git grep PetscBeganKokkos
> src/sys/objects/cupminit.inc:    PetscBeganKokkos = PETSC_TRUE;
> src/sys/objects/cupminit.inc:    PetscBeganKokkos = PETSC_TRUE;
>
> I can do an MR to remove these if that is the case.
>
> Mark
>
> On Thu, Oct 15, 2020 at 8:34 PM Barry Smith <[email protected]> wrote:
>
>>
>>
>>   I thought the plan was that Kokkos also had a lazy initialization but
>> perhaps it does not and needs to be fixed.
>>
>>   Barry
>>
>> > On Oct 15, 2020, at 6:49 PM, Mark Adams <[email protected]> wrote:
>> >
>> > I am running a on SUMMIt with a Kokkos cuda configuration and while
>> debugging with ddt I noticed that it spent a long time in KokkosInit, but I
>> was not using Kokkos. KokkosInit was call in PETSc's GPU init, which seems
>> logical enough, but it would be better if it is not called if you are not
>> using Kokkos.
>> >
>> > I recall seeing places where Kokkos is checked when calling a Kokkos
>> method (ie, lazy initialization). Do we have policy on whether we are being
>> lazy with KokkosInit or not?
>> >
>> > Mark
>>
>>

Reply via email to