-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
ipc_service: Extend API with nocopy functions #44033
Conversation
3bbe8a1
to
ee614f3
Compare
@nordic-krch @pdunaj @hubertmis @arnopo I added some commits to this PR to also adapt the static vrings backend to the new API and changed the current sample so that we use the NOCOPY on one of the tested instances. |
89ef492
to
fa4371a
Compare
@hubertmis @nordic-krch some changes:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My apologize I'm quite late on the review...
First of all, congratulation for this very good work !
Few minor remarks + a proposal to save a copy
And add some more details in documentation. Signed-off-by: Carlo Caione <ccaione@baylibre.com>
Add several new functions to the IPC service API: - ipc_service_get_tx_buffer() - ipc_service_drop_tx_buffer() - ipc_service_send_nocopy() - ipc_service_release_rx_buffer() - ipc_service_hold_rx_buffer() This set of function is used to support backends with nocopy capability. Signed-off-by: Carlo Caione <ccaione@baylibre.com>
Add the newly introduced NOCOPY capability to the static vrings backend. Signed-off-by: Carlo Caione <ccaione@baylibre.com>
At least for one of the instances, use the NOCOPY feature of the backend. Signed-off-by: Carlo Caione <ccaione@baylibre.com>
This PR follows the discussion at #43924. The idea is that we want to extend the IPC service API to expose several new functions that can be used by backends with nocopy capabilities.
In this PR we also adapt the static vrings backend and the related sample.