reorder: allow random number as starting point
authorSimon Kagstrom <simon.kagstrom@netinsight.net>
Wed, 20 May 2015 11:02:05 +0000 (13:02 +0200)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Mon, 22 Jun 2015 20:20:12 +0000 (22:20 +0200)
We use sequence numbers from a generator which has potentially started
long before the receiver. Therefore, the first number will typically
be > 0. The rte_reorder code will not work in this case, since the
packet is seen as outside of the buffer.

The patch instead records the first sequence number inserted as the
starting point.

Signed-off-by: Simon Kagstrom <simon.kagstrom@netinsight.net>
Signed-off-by: Johan Faltstrom <johan.faltstrom@netinsight.net>
Acked-by: Sergio Gonzalez Monroy <sergio.gonzalez.monroy@intel.com>
lib/librte_reorder/rte_reorder.c

index dc0e806..010dff6 100644 (file)
@@ -73,6 +73,7 @@ struct rte_reorder_buffer {
        unsigned int memsize; /**< memory area size of reorder buffer */
        struct cir_buffer ready_buf; /**< temp buffer for dequeued entries */
        struct cir_buffer order_buf; /**< buffer used to reorder entries */
+       int is_initialized;
 } __rte_cache_aligned;
 
 static void
@@ -325,6 +326,11 @@ rte_reorder_insert(struct rte_reorder_buffer *b, struct rte_mbuf *mbuf)
        uint32_t offset, position;
        struct cir_buffer *order_buf = &b->order_buf;
 
+       if (!b->is_initialized) {
+               b->min_seqn = mbuf->seqn;
+               b->is_initialized = 1;
+       }
+
        /*
         * calculate the offset from the head pointer we need to go.
         * The subtraction takes care of the sequence number wrapping.