Revert moving free() to optional (ext) provider ops

[ldorau]

Description

Revert moving free() to optional (ext) provider ops.

This reverts commit b0bfbb7.

Remove umfDefaultFree() and umfIsFreeOpDefault().

Remove the disable_upstream_provider_free parameter
of the Coarse provider.

Remove the upstreamDoesNotFree argument of
the umfTrackingMemoryProviderCreate() function.

Requires:

• Use libcoarse in the file and the devdax providers #932

Checklist

• Code compiles without errors locally
• All tests pass locally
• CI workflows execute properly

[ldorau]

@bratpiorka @vinser52 please review

[vinser52]

@bratpiorka I think it should be part of the 0.10

[ldorau]

@bratpiorka I think it should be part of the 0.10

@vinser52 Are you sure? This change requires libcoarse which is not present in 0.10 ...

[vinser52]

@bratpiorka I think it should be part of the 0.10

@vinser52 Are you sure? This change requires libcoarse which is not present in 0.10 ...

We are changing interfaces, the earlier we do that -> less impact for the future users.

Regarding libcoarse, does it impact only file and devdax providers? Can we merge the libcoarse to the 0.10?

[ldorau]

Regarding libcoarse, does it impact only file and devdax providers?

Yes, only file and devdax providers.

[ldorau]

@bratpiorka I think it should be part of the 0.10

@bratpiorka ?

[bratpiorka]

@vinser52 @ldorau devdax and file providers would not be used in 0.10x and I prefer to minimize the number of changes there at least right now. If we know that there are no major problems with 0.10x, then we could think about merging changes related to the coarse lib (this is a lot of code and multiple PRs).