Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page:

include/odp/api/spec/packet.h
line 32
@@ -235,6 +246,18 @@ int odp_packet_reset(odp_packet_t pkt, uint32_t len);
  */
 odp_packet_t odp_packet_from_event(odp_event_t ev);
 
+/**
+ * Convert multiple packet events to packet handles
+ *
+ * All events must be of type ODP_EVENT_PACKET.
+ *
+ * @param[out] pkt  Packet handle array for output
+ * @param      ev   Array of event handles to convert
+ * @param      num  Number of packets and events
+ */
+void odp_packet_from_event_multi(odp_packet_t pkt[], const odp_event_t ev[],
+                                int num);


Comment:
Even with the `odp_event_type_multi()` API semantics, it's still not clear why 
this is needed. Given that `odp_packet_from_event()` is likely just a cast 
there doesn't seem to be a great deal to be gained by having a `multi` version 
of this which can't be easily inlined away.

> Bill Fischofer(Bill-Fischofer-Linaro) wrote:
> Again since `odp_packet_to_event()` is likely to be just a cast that can be 
> inlined away it's not clear why a `multi` version is needed.


>> Bill Fischofer(Bill-Fischofer-Linaro) wrote:
>> Again, it's not clear why this is needed. 


>>> Bill Fischofer(Bill-Fischofer-Linaro) wrote:
>>> Not clear why this is needed. How does an application determine this more 
>>> efficiently than the implementation would?


https://github.com/Linaro/odp/pull/318#discussion_r154377171
updated_at 2017-12-01 15:58:07

Reply via email to