michaelh/zephyr
1
0
Fork 0
Code Releases Activity

ipc: static_vrings: Add -ENOMEM case

Browse source 
Sometimes it is important to know when the backend fails to send out
data because no memory / buffers are available. Return -ENOMEM in that
case.

Signed-off-by: Carlo Caione <ccaione@baylibre.com>
This commit is contained in:
Carlo Caione 2022-05-16 13:04:24 +02:00 • committed by Carles Cufí
commit 11f0bb9d1a
3 changed files with 11 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
include/zephyr/ipc/ipc_service.h
View file

@ -233,6 +233,7 @@ int ipc_service_register_endpoint(const struct device *instance,
* @retval -EBADMSG when the data is invalid (i.e. invalid data format, * @retval -EBADMSG when the data is invalid (i.e. invalid data format,
* invalid length, ...) * invalid length, ...)
* @retval -EBUSY when the instance is busy. * @retval -EBUSY when the instance is busy.
* @retval -ENOMEM when no memory / buffers are available.
* *
* @retval bytes number of bytes sent. * @retval bytes number of bytes sent.
* @retval other errno codes depending on the implementation of the backend. * @retval other errno codes depending on the implementation of the backend.

1
include/zephyr/ipc/ipc_service_backend.h
View file

@ -47,6 +47,7 @@ struct ipc_service_backend {
* @retval -EINVAL when instance is invalid. * @retval -EINVAL when instance is invalid.
* @retval -EBADMSG when the message is invalid. * @retval -EBADMSG when the message is invalid.
* @retval -EBUSY when the instance is busy or not ready. * @retval -EBUSY when the instance is busy or not ready.
* @retval -ENOMEM when no memory / buffers are available.
* *
* @retval bytes number of bytes sent. * @retval bytes number of bytes sent.
* @retval other errno codes depending on the implementation of the * @retval other errno codes depending on the implementation of the

10
subsys/ipc/ipc_service/backends/ipc_rpmsg_static_vrings.c
View file

@ -394,6 +394,7 @@ static int send(const struct device *instance, void *token,
{ {
struct backend_data_t *data = instance->data; struct backend_data_t *data = instance->data;
struct ipc_rpmsg_ept *rpmsg_ept; struct ipc_rpmsg_ept *rpmsg_ept;
int ret;
/* Instance is not ready */ /* Instance is not ready */
if (atomic_get(&data->state) != STATE_INITED) { if (atomic_get(&data->state) != STATE_INITED) {
@ -407,7 +408,14 @@ static int send(const struct device *instance, void *token,
rpmsg_ept = (struct ipc_rpmsg_ept *) token; rpmsg_ept = (struct ipc_rpmsg_ept *) token;
return rpmsg_send(&rpmsg_ept->ep, msg, len); ret = rpmsg_send(&rpmsg_ept->ep, msg, len);
/* No buffers available */
if (ret == RPMSG_ERR_NO_BUFF) {
return -ENOMEM;
}
return ret;
} }
static int send_nocopy(const struct device *instance, void *token, static int send_nocopy(const struct device *instance, void *token,