Skip to content

Add disable_provider_free parameter to pool jemalloc #618

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

ldorau
Copy link
Contributor

@ldorau ldorau commented Jul 17, 2024

Description

Add the disable_provider_free parameter to pool jemalloc. It should be set to true if umfMemoryProviderFree() should never be called.

This option will be needed by the future memory providers that will not provide the free() op.

Checklist

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

@ldorau ldorau requested a review from a team as a code owner July 17, 2024 12:41
@ldorau ldorau force-pushed the Add_provider_free_always_fails_parameter_to_pool_jemalloc branch from 76af328 to c197f44 Compare July 17, 2024 13:16
@ldorau ldorau changed the title Add provider_free_always_fails parameter to pool jemalloc Add disable_provider_free parameter to pool jemalloc Jul 17, 2024
@ldorau ldorau force-pushed the Add_provider_free_always_fails_parameter_to_pool_jemalloc branch from c197f44 to 8e8db44 Compare July 17, 2024 13:18
@ldorau ldorau requested a review from bratpiorka July 17, 2024 13:19
@ldorau ldorau requested a review from KFilipek July 17, 2024 13:43
Add the disable_provider_free parameter to pool jemalloc.
It should be set to true if umfMemoryProviderFree()
should never be called.

This option will be needed by the future memory providers
that will not provide the free() op.

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the Add_provider_free_always_fails_parameter_to_pool_jemalloc branch from 8e8db44 to 996273c Compare July 17, 2024 14:33
Copy link
Contributor

@lukaszstolarczuk lukaszstolarczuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@KFilipek KFilipek merged commit cb6b8bb into oneapi-src:main Jul 18, 2024
63 checks passed
@ldorau ldorau deleted the Add_provider_free_always_fails_parameter_to_pool_jemalloc branch July 18, 2024 08:16
@ldorau ldorau mentioned this pull request Aug 14, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants