Skip to content

Conversation

@ldorau
Copy link
Contributor

@ldorau ldorau commented Oct 21, 2024

Description

Add the jemalloc_coarse_file and scalable_coarse_file tests
that test the coarse provider with upstream file provider
and two pool managers: jemalloc and scalable pool.

Requires:

Checklist

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

@ldorau ldorau force-pushed the Add_the_jemalloc_coarse_file_and_scalable_coarse_file_tests branch from 5cf11e3 to 88ffac9 Compare October 21, 2024 12:40
Add the jemalloc_coarse_file and scalable_coarse_file tests
that test the coarse provider with upstream file provider
and two pool managers: jemalloc and scalable pool.

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the Add_the_jemalloc_coarse_file_and_scalable_coarse_file_tests branch from 88ffac9 to 9c91404 Compare October 21, 2024 19:02
@ldorau ldorau marked this pull request as ready for review October 21, 2024 19:02
@ldorau ldorau requested a review from a team as a code owner October 21, 2024 19:02
@ldorau ldorau requested a review from bratpiorka October 21, 2024 19:05
@ldorau
Copy link
Contributor Author

ldorau commented Oct 21, 2024

PR is ready for review!

@ldorau
Copy link
Contributor Author

ldorau commented Oct 22, 2024

Overall coverage rate:
lines......: 80.6% (4587 of 5692 lines)

https://github.com/oneapi-src/unified-memory-framework/actions/runs/11446838185

using umf_test::test;
using namespace umf_test;

#define FILE_PATH ((char *)"/tmp/file_provider")
Copy link
Contributor

Choose a reason for hiding this comment

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

why hard-coding this path? can't we use something like for dax (UMF_TESTS_FSDAX_PATH)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ops, race condition ;-)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will remember

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done: #821

@ldorau ldorau merged commit 7cbe2e9 into oneapi-src:main Oct 22, 2024
75 checks passed
@ldorau ldorau deleted the Add_the_jemalloc_coarse_file_and_scalable_coarse_file_tests branch October 22, 2024 11:47
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