Skip to content

Fix file_alloc() and file_open/close_ipc_handle() #865

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

Merged

Conversation

ldorau
Copy link
Contributor

@ldorau ldorau commented Nov 5, 2024

Description

Alignment passed to file_alloc() must be a power of two and a multiple or a divider of the page size.
Align up the alignment in file_alloc() to the page size.

file_open_ipc_handle() has to use the memory protection and visibility got from the IPC handle, so let's add the memory protection and visibility to the IPC handle.

Ref: #838

Fixes: #848

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 November 5, 2024 10:05
@ldorau ldorau requested review from vinser52 and bratpiorka November 5, 2024 10:05
@ldorau ldorau force-pushed the Fix_file_alloc_and_file_open_close_ipc_handle branch from 3a14747 to 7edcd28 Compare November 5, 2024 12:39
Alignment passed to `file_alloc()` must be a power of two
and a multiple or a divider of the page size.
Align up the alignment in `file_alloc()` to the page size.

file_open_ipc_handle() has to use the memory protection
and visibility got from the IPC handle, so let's add
the memory protection and visibility to the IPC handle.

Ref: oneapi-src#838

Fixes: oneapi-src#848

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the Fix_file_alloc_and_file_open_close_ipc_handle branch from 7edcd28 to aada873 Compare November 5, 2024 13:21
@ldorau ldorau merged commit 8b554a0 into oneapi-src:main Nov 5, 2024
74 checks passed
@ldorau ldorau deleted the Fix_file_alloc_and_file_open_close_ipc_handle branch November 6, 2024 06:49
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.

IPC API of the file provider does not work
3 participants