An enqueue operation to a DSW port's input ring is guaranteed to
succeed, an thus a bulk type enqueue (instead of a burst enqueue) may
be used. There is also need not check the return code of such calls.
This change shaves off a handful of CPU cycles worth of latency per
enqueued event.
Signed-off-by: Mattias Rönnblom <mattias.ronnb...@ericsson.com>
---
drivers/event/dsw/dsw_event.c | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/event/dsw/dsw_event.c b/drivers/event/dsw/dsw_event.c
index 9932caf2ee..90d298a255 100644
--- a/drivers/event/dsw/dsw_event.c
+++ b/drivers/event/dsw/dsw_event.c
@@ -590,7 +590,6 @@ dsw_port_transmit_buffered(struct dsw_evdev *dsw, struct
dsw_port *source_port,
struct dsw_port *dest_port = &(dsw->ports[dest_port_id]);
uint16_t *buffer_len = &source_port->out_buffer_len[dest_port_id];
struct rte_event *buffer = source_port->out_buffer[dest_port_id];
- uint16_t enqueued = 0;
if (*buffer_len == 0)
return;
@@ -598,13 +597,8 @@ dsw_port_transmit_buffered(struct dsw_evdev *dsw, struct
dsw_port *source_port,
/* The rings are dimensioned to fit all in-flight events (even
* on a single ring), so looping will work.
*/
- do {
- enqueued +=
- rte_event_ring_enqueue_burst(dest_port->in_ring,
- buffer+enqueued,
- *buffer_len-enqueued,
- NULL);
- } while (unlikely(enqueued != *buffer_len));
+ rte_event_ring_enqueue_bulk(dest_port->in_ring, buffer, *buffer_len,
+ NULL);
(*buffer_len) = 0;
}
--
2.34.1
