This touches the api "b/include/odp/api/packet_io.h" so  the subject should
add api: like this

"api: packet_io: clarify what happens when not all packets are sent"

Generally for ODP the approximate path to the code is in the subject
"include/odp/api/packet_io.h" becomes

include/odp/api   --> api:
then the module area or file name as appropriate - packet_io: in this case
as you have it.



On 5 June 2015 at 07:53, Zoltan Kiss <[email protected]> wrote:

> Currently our examples are not handling this situation as well.
>
> Signed-off-by: Zoltan Kiss <[email protected]>
> ---
>  include/odp/api/packet_io.h | 4 +++-
>  include/odp/api/queue.h     | 7 +++++--
>  2 files changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/include/odp/api/packet_io.h b/include/odp/api/packet_io.h
> index 89356a6..5a71e52 100644
> --- a/include/odp/api/packet_io.h
> +++ b/include/odp/api/packet_io.h
> @@ -111,7 +111,9 @@ int odp_pktio_recv(odp_pktio_t pktio, odp_packet_t
> pkt_table[], int len);
>   * @param pkt_table[]  Array of packets to send
>   * @param len          length of pkt_table[]
>   *
> - * @return Number of packets sent
> + * @return Number of packets sent. If it is less than 'len', the remaining
> + * packets at the end of pkt_table[] are not consumed, and caller has to
> take
> + * care of them.
>   * @retval <0 on failure
>   */
>  int odp_pktio_send(odp_pktio_t pktio, odp_packet_t pkt_table[], int len);
> diff --git a/include/odp/api/queue.h b/include/odp/api/queue.h
> index 6a8b15f..ac4147d 100644
> --- a/include/odp/api/queue.h
> +++ b/include/odp/api/queue.h
> @@ -242,7 +242,8 @@ void *odp_queue_get_context(odp_queue_t queue);
>   * @param ev      Event handle
>   *
>   * @retval 0 on success
> - * @retval <0 on failure (e.g. queue full)
> + * @retval <0 on failure (e.g. queue full), the event is not consumed,
> caller
> + * has to take care of it.
>   */
>  int odp_queue_enq(odp_queue_t queue, odp_event_t ev);
>
> @@ -253,7 +254,9 @@ int odp_queue_enq(odp_queue_t queue, odp_event_t ev);
>   * @param[in] events Array of event handles
>   * @param num     Number of event handles to enqueue
>   *
> - * @return Number of events actually enqueued (0 ... num)
> + * @return Number of events actually enqueued (0 ... num). If it is less
> than
> + * 'num', the remaining events at the end of events[] are not consumed,
> and
> + * caller has to take care of them.
>   * @retval <0 on failure
>   */
>  int odp_queue_enq_multi(odp_queue_t queue, const odp_event_t events[],
> int num);
> --
> 1.9.1
>
> _______________________________________________
> lng-odp mailing list
> [email protected]
> https://lists.linaro.org/mailman/listinfo/lng-odp
>



-- 
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to