Skip to content

Verify if the provider supports the split operation #1465

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 24, 2025

Description

Verify in the initialization of the jemalloc pool
if a memory provider supports the split() operation,
because the jemalloc pool requires that.

Replace UMF_RESULT_ERROR_UNKNOWN with UMF_RESULT_ERROR_NOT_SUPPORTED
in provider_base_t.

Checklist

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

@ldorau ldorau force-pushed the Verify_if_the_provider_supports_the_split_operation branch 2 times, most recently from c867ef6 to 2e2bf9c Compare July 24, 2025 13:14
@ldorau ldorau marked this pull request as ready for review July 24, 2025 13:15
@ldorau ldorau requested a review from a team as a code owner July 24, 2025 13:15
@ldorau ldorau force-pushed the Verify_if_the_provider_supports_the_split_operation branch from 2e2bf9c to ff27025 Compare July 25, 2025 10:19
@ldorau ldorau changed the base branch from main to v1.0.x July 25, 2025 10:29
ldorau added 2 commits July 25, 2025 12:31
Replace UMF_RESULT_ERROR_UNKNOWN with UMF_RESULT_ERROR_NOT_SUPPORTED
in provider_base_t.

Signed-off-by: Lukasz Dorau <[email protected]>
Verify in the initialization of the jemalloc pool
if a memory provider supports the split() operation,
because the jemalloc pool requires that.

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the Verify_if_the_provider_supports_the_split_operation branch from ff27025 to 73858f2 Compare July 25, 2025 10:37
@lukaszstolarczuk lukaszstolarczuk self-requested a review July 25, 2025 10:37
@ldorau
Copy link
Contributor Author

ldorau commented Jul 25, 2025

@lukaszstolarczuk Rebased to the v1.0.x branch

@ldorau
Copy link
Contributor Author

ldorau commented Jul 25, 2025

@ldorau
Copy link
Contributor Author

ldorau commented Jul 25, 2025

@ldorau
Copy link
Contributor Author

ldorau commented Jul 28, 2025

@lukaszstolarczuk please review

@lukaszstolarczuk lukaszstolarczuk merged commit e296e31 into oneapi-src:v1.0.x Jul 29, 2025
146 of 148 checks passed
@ldorau ldorau deleted the Verify_if_the_provider_supports_the_split_operation branch July 30, 2025 07:04
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.

3 participants