Merge remote-tracking branch 'origin/release'

Merge branch 'jczhang/2025-07-23/fix-log-nested-bug' into 'release'

Fix a bug with unsync'ed variables resulting in unmatched MPI_Allreduce

See merge request petsc/petsc!8583

Sys: fix datetype and op used in MPIU_Allreduce wrt time

Log: fix a bug with unsync'ed variables resulting in unmatched MPI_Allreduce

The variable is a timer such that the bug was non-deterministic.

The test code ex2f.F90 is adapted by Christiaan Klaij <

Log: fix a bug with unsync'ed variables resulting in unmatched MPI_Allreduce

The variable is a timer such that the bug was non-deterministic.

The test code ex2f.F90 is adapted by Christiaan Klaij <C.Klaij@marin.nl>

Reported-by: Christiaan Klaij <C.Klaij@marin.nl>
Reported-by: Zongze Yang <yangzongze@gmail.com>

show more ...

Merge remote-tracking branch 'origin/release'

Merge branch 'jolivet/fix-petsc-case' into 'release'

Fix wrong case for PETSc

See merge request petsc/petsc!8266

Fix wrong case for PETSc

Merge branch 'stefanozampini/useless-cast' into 'main'

Remove useless cast

See merge request petsc/petsc!7894

Brain dead fixes for useless casts

Merge branch 'barry/2024-09-02/rebased-fix-conversion-warnings' into 'main'

Compiler finds (and forbid) casts from higher precision integers to lower

See merge request petsc/petsc!7806

Fix MPIU_* routines to always return MPI error codes and thus be usable with PetscCallMPI()

Introduce MPIU_Count to be used when MPI_Count is not available

Merge remote-tracking branch 'origin/release'

Merge branch 'jolivet/rm-unneeded-include' into 'release'

Remove unneeded include <petsc/private/fortranimpl.h>

See merge request petsc/petsc!7561

Remove unneeded include <petsc/private/fortranimpl.h>

Merge branch 'jolivet/clang-format-18' into 'main'

clang-format version 18

See merge request petsc/petsc!6902

checkbadSource: rules for PetscFunctionBegin and derivatives

Merge branch 'tisaac/feature-log-handler' into 'main'

Deglobalize logging into PetscLogHandler and PetscLogState

See merge request petsc/petsc!6709

Profiling: Add composed functions for default log handler functions

This makes them extensible to other implementations.
Where appropriate, the global log handler will now dispatch
these functions t

Profiling: Add composed functions for default log handler functions

This makes them extensible to other implementations.
Where appropriate, the global log handler will now dispatch
these functions to all running log handlers.

It is not always appropriate to dispatch to all running log handlers:
when a single return value is expected or when a filename
is given, it will continue to dispatch only to the default
log handler.

show more ...

Profiling: Improve !defined(PETSC_USE_LOG) #defines

- If we cast macro arguments to void, we can avoid unused errors that have
forced lots of conditionally included code like this:

```c
#if def

Profiling: Improve !defined(PETSC_USE_LOG) #defines

- If we cast macro arguments to void, we can avoid unused errors that have
forced lots of conditionally included code like this:

```c
#if defined(PETSC_USE_LOG)
PetscLogEvent event;
#endif
```

- Having global counters like petsc_TotalFlops externed event
if !PetscDefined(USE_LOG) helps to remove a lot of needless
`#ifdef`s around code that can instead be placed in
`if (PetscDefined(USE_LOG)) {}` blocks.

show more ...

Profiling: Add nested PetscLogHandler implementation

This log handler has behavior like the behavior of
PetscLogNestedBegin().

Unlike the implementation of PetscLogHandler_Default, I did not try to

Profiling: Add nested PetscLogHandler implementation

This log handler has behavior like the behavior of
PetscLogNestedBegin().

Unlike the implementation of PetscLogHandler_Default, I did not try to
do a faithful translation of the existing nested logging behavior, for
two reasons.

- The existing implementation keeps sorted lists of nested paths ("Event
A;Event B;"), and for each of those paths keeps a sorted list of the
top-level events ("Event C", "Event D") that map that path to other
paths (path + event => other path). It's a complicated data structure,
and the benefit of the sorting (O(log n) lookup) is probably not worth
the more expensive insertion cost each time a new path + event combo
appears. Instead I implemented the path + event => other path
lookup as a hash map, which simplified the implementation
significantly.

- The existing implementation assigns an event to each path, modifying
the event registry in place. This messes things up if you want
to both -log_view and -log_view ::ascii_flamegraph output, the
original motivation for refactoring the profiling infrastructure.
In the new implementation, the nested handler has its own
PetscLogState that it modifies, and an inner copy of
PetscLogHandler_Default the acts on that nested state.

- The PetscLogGlobalNames data structure also simplifies the
construction of a linearized tree of events in the final
viewing stage, although the low-level xml construction
routines are copied almost verbatim from the previous
implementation.

show more ...