Skip to content

Use libcoarse in the file and the devdax providers#932

Merged
ldorau merged 5 commits into
oneapi-src:mainfrom
ldorau:Use_the_coarse_library
Dec 5, 2024
Merged

Use libcoarse in the file and the devdax providers#932
ldorau merged 5 commits into
oneapi-src:mainfrom
ldorau:Use_the_coarse_library

Conversation

@ldorau

@ldorau ldorau commented Nov 25, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Use libcoarse in the file and the devdax providers.

This PR is required by 3 PRs that fix the following issues:

Checklist

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

@ldorau ldorau force-pushed the Use_the_coarse_library branch from d2c0a9b to b17a69f Compare November 25, 2024 12:33
@ldorau ldorau mentioned this pull request Nov 25, 2024
Merged
3 tasks
@ldorau ldorau force-pushed the Use_the_coarse_library branch from b17a69f to 92f98f1 Compare November 26, 2024 07:09
This was referenced Nov 26, 2024
Merged
Merged
Merged
@ldorau ldorau force-pushed the Use_the_coarse_library branch 4 times, most recently from 88a1381 to 6fbe907 Compare December 2, 2024 08:48
@ldorau ldorau marked this pull request as ready for review December 2, 2024 08:48
@ldorau ldorau requested a review from a team as a code owner December 2, 2024 08:48
@ldorau ldorau changed the title [DRAFT] Use libcoarse in the file and the devdax providers Use libcoarse in the file and the devdax providers Dec 2, 2024
@ldorau ldorau requested review from bratpiorka and vinser52 December 2, 2024 08:48
@ldorau ldorau mentioned this pull request Dec 2, 2024
Merged
4 tasks
@ldorau

ldorau commented Dec 3, 2024

Copy link
Copy Markdown
Contributor Author

This PR is required by 3 PRs that fix the following issues:

@vinser52 @bratpiorka please review

bratpiorka
bratpiorka approved these changes Dec 3, 2024
View reviewed changes
Comment thread src/provider/provider_devdax_memory.c
Comment thread src/provider/provider_devdax_memory.c Outdated
ldorau added 5 commits December 3, 2024 11:40
@ldorau
Use libcoarse in the file provider
73d012e 
Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
@ldorau
Add free() op to the file provider
f81c647 
Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
@ldorau
Use libcoarse in the devdax provider
a27c45f 
Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
@ldorau
Add free() to the devdax provider
96605c4
@ldorau
Remove free_not_supp from the ipcTestParams tuple
d9f1fee 
Remove free_not_supp from the ipcTestParams tuple.
It is not needed any more.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
@ldorau ldorau force-pushed the Use_the_coarse_library branch from 6fbe907 to d9f1fee Compare December 3, 2024 10:47
@ldorau

ldorau commented Dec 4, 2024

Copy link
Copy Markdown
Contributor Author

@vinser52 @lplewa @lukaszstolarczuk @PatKamin @kswiecicki @KFilipek please review

@ldorau

ldorau commented Dec 5, 2024

Copy link
Copy Markdown
Contributor Author

Please review

@ldorau ldorau merged commit fb53918 into oneapi-src:main Dec 5, 2024
@ldorau ldorau deleted the Use_the_coarse_library branch December 5, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bratpiorka bratpiorka bratpiorka approved these changes
@kswiecicki kswiecicki kswiecicki approved these changes
@vinser52 vinser52 Awaiting requested review from vinser52
@lukaszstolarczuk lukaszstolarczuk Awaiting requested review from lukaszstolarczuk
@KFilipek KFilipek Awaiting requested review from KFilipek
@PatKamin PatKamin Awaiting requested review from PatKamin

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ldorau @bratpiorka @kswiecicki