Skip to content
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

[UMA] add more tests for allocations #535

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

DamianDuy
Copy link

Basic tests for UMA allocations. More will be added in the following PRs.


#ifndef UMA_TEST_MEMORY_POOL_OPS_HPP
#define UMA_TEST_MEMORY_POOL_OPS_HPP

#define NTHREADS 5
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be a static constexpr defined in the umaPoolTest class.

Copy link
Author

Choose a reason for hiding this comment

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

Done.

test/unified_memory_allocation/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_memory_allocation/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_memory_allocation/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_memory_allocation/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_memory_allocation/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_memory_allocation/memoryPool.hpp Outdated Show resolved Hide resolved
@DamianDuy DamianDuy force-pushed the addAllocationUMAtests branch 8 times, most recently from b9cd735 to bb2f599 Compare June 26, 2023 06:05
@DamianDuy DamianDuy force-pushed the addAllocationUMAtests branch 4 times, most recently from c83bb38 to 669f438 Compare June 29, 2023 09:19
@DamianDuy DamianDuy force-pushed the addAllocationUMAtests branch 5 times, most recently from fa45b52 to ce9de22 Compare July 13, 2023 10:53
@DamianDuy DamianDuy force-pushed the addAllocationUMAtests branch 4 times, most recently from 049047b to 23bfaea Compare July 18, 2023 12:01
test/unified_malloc_framework/memoryPool.hpp Show resolved Hide resolved
test/unified_malloc_framework/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_malloc_framework/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_malloc_framework/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_malloc_framework/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_malloc_framework/memoryPool.hpp Outdated Show resolved Hide resolved
test/unified_malloc_framework/memoryPool.hpp Outdated Show resolved Hide resolved
@DamianDuy DamianDuy force-pushed the addAllocationUMAtests branch 2 times, most recently from 9181f2d to 7a94de0 Compare July 24, 2023 15:03
@DamianDuy DamianDuy marked this pull request as ready for review July 25, 2023 09:34
@igchor igchor merged commit 0169603 into oneapi-src:main Jul 25, 2023
20 checks passed
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