Skip to content

Fix arguments checking in IPC API of OS and file memory providers #760

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 Sep 26, 2024

Description

IPC API of OS memory provider requires the UMF_MEM_MAP_SHARED memory visibility mode (UMF_RESULT_ERROR_INVALID_ARGUMENT is returned otherwise).

IPC API of file memory provider requires the UMF_MEM_MAP_SHARED
or UMF_MEM_MAP_SYNC memory visibility mode
(UMF_RESULT_ERROR_INVALID_ARGUMENT is returned otherwise).

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 September 26, 2024 10:09
@ldorau ldorau force-pushed the IPC_API_of_OS_memory_provider_requires_UMF_MEM_MAP_SHARED_visibility branch 4 times, most recently from 0a94fb1 to 1aa3671 Compare September 26, 2024 13:06
@ldorau ldorau changed the title IPC API of OS memory provider requires UMF_MEM_MAP_SHARED visibility Fix IPC API of OS and file memory providers Sep 26, 2024
IPC API of OS memory provider requires the `UMF_MEM_MAP_SHARED`
memory `visibility` mode (`UMF_RESULT_ERROR_INVALID_ARGUMENT`
is returned otherwise).

Signed-off-by: Lukasz Dorau <[email protected]>
IPC API of file memory provider requires the `UMF_MEM_MAP_SHARED`
or `UMF_MEM_MAP_SYNC` memory `visibility` mode
(`UMF_RESULT_ERROR_INVALID_ARGUMENT` is returned otherwise).

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the IPC_API_of_OS_memory_provider_requires_UMF_MEM_MAP_SHARED_visibility branch from 1aa3671 to b726d3e Compare September 26, 2024 16:14
@ldorau ldorau requested review from lukaszstolarczuk, a team and vinser52 September 26, 2024 16:22
@ldorau ldorau changed the title Fix IPC API of OS and file memory providers Fix arguments checking in IPC API of OS and file memory providers Sep 27, 2024
@lukaszstolarczuk lukaszstolarczuk merged commit 52e7da9 into oneapi-src:main Sep 30, 2024
72 checks passed
@ldorau ldorau deleted the IPC_API_of_OS_memory_provider_requires_UMF_MEM_MAP_SHARED_visibility branch October 1, 2024 06:33
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