ring: guarantee load/load order in enqueue and dequeue
[dpdk.git] / test / test / test_ring.c
index 3891f5d..5eb40a0 100644 (file)
@@ -43,7 +43,6 @@
 #include <rte_common.h>
 #include <rte_log.h>
 #include <rte_memory.h>
-#include <rte_memzone.h>
 #include <rte_launch.h>
 #include <rte_cycles.h>
 #include <rte_eal.h>
@@ -54,7 +53,6 @@
 #include <rte_malloc.h>
 #include <rte_ring.h>
 #include <rte_random.h>
-#include <rte_common.h>
 #include <rte_errno.h>
 #include <rte_hexdump.h>
 
  *      - Dequeue one object, two objects, MAX_BULK objects
  *      - Check that dequeued pointers are correct
  *
- *    - Test watermark and default bulk enqueue/dequeue:
- *
- *      - Set watermark
- *      - Set default bulk value
- *      - Enqueue objects, check that -EDQUOT is returned when
- *        watermark is exceeded
- *      - Check that dequeued pointers are correct
- *
- * #. Check live watermark change
- *
- *    - Start a loop on another lcore that will enqueue and dequeue
- *      objects in a ring. It will monitor the value of watermark.
- *    - At the same time, change the watermark on the master lcore.
- *    - The slave lcore will check that watermark changes from 16 to 32.
- *
  * #. Performance tests.
  *
  * Tests done in test_ring_perf.c
@@ -115,123 +98,6 @@ static struct rte_ring *r;
 
 #define        TEST_RING_FULL_EMTPY_ITER       8
 
-static int
-check_live_watermark_change(__attribute__((unused)) void *dummy)
-{
-       uint64_t hz = rte_get_timer_hz();
-       void *obj_table[MAX_BULK];
-       unsigned watermark, watermark_old = 16;
-       uint64_t cur_time, end_time;
-       int64_t diff = 0;
-       int i, ret;
-       unsigned count = 4;
-
-       /* init the object table */
-       memset(obj_table, 0, sizeof(obj_table));
-       end_time = rte_get_timer_cycles() + (hz / 4);
-
-       /* check that bulk and watermark are 4 and 32 (respectively) */
-       while (diff >= 0) {
-
-               /* add in ring until we reach watermark */
-               ret = 0;
-               for (i = 0; i < 16; i ++) {
-                       if (ret != 0)
-                               break;
-                       ret = rte_ring_enqueue_bulk(r, obj_table, count);
-               }
-
-               if (ret != -EDQUOT) {
-                       printf("Cannot enqueue objects, or watermark not "
-                              "reached (ret=%d)\n", ret);
-                       return -1;
-               }
-
-               /* read watermark, the only change allowed is from 16 to 32 */
-               watermark = r->watermark;
-               if (watermark != watermark_old &&
-                   (watermark_old != 16 || watermark != 32)) {
-                       printf("Bad watermark change %u -> %u\n", watermark_old,
-                              watermark);
-                       return -1;
-               }
-               watermark_old = watermark;
-
-               /* dequeue objects from ring */
-               while (i--) {
-                       ret = rte_ring_dequeue_bulk(r, obj_table, count);
-                       if (ret != 0) {
-                               printf("Cannot dequeue (ret=%d)\n", ret);
-                               return -1;
-                       }
-               }
-
-               cur_time = rte_get_timer_cycles();
-               diff = end_time - cur_time;
-       }
-
-       if (watermark_old != 32 ) {
-               printf(" watermark was not updated (wm=%u)\n",
-                      watermark_old);
-               return -1;
-       }
-
-       return 0;
-}
-
-static int
-test_live_watermark_change(void)
-{
-       unsigned lcore_id = rte_lcore_id();
-       unsigned lcore_id2 = rte_get_next_lcore(lcore_id, 0, 1);
-
-       printf("Test watermark live modification\n");
-       rte_ring_set_water_mark(r, 16);
-
-       /* launch a thread that will enqueue and dequeue, checking
-        * watermark and quota */
-       rte_eal_remote_launch(check_live_watermark_change, NULL, lcore_id2);
-
-       rte_delay_ms(100);
-       rte_ring_set_water_mark(r, 32);
-       rte_delay_ms(100);
-
-       if (rte_eal_wait_lcore(lcore_id2) < 0)
-               return -1;
-
-       return 0;
-}
-
-/* Test for catch on invalid watermark values */
-static int
-test_set_watermark( void ){
-       unsigned count;
-       int setwm;
-
-       struct rte_ring *r = rte_ring_lookup("test_ring_basic_ex");
-       if(r == NULL){
-               printf( " ring lookup failed\n" );
-               goto error;
-       }
-       count = r->size * 2;
-       setwm = rte_ring_set_water_mark(r, count);
-       if (setwm != -EINVAL){
-               printf("Test failed to detect invalid watermark count value\n");
-               goto error;
-       }
-
-       count = 0;
-       rte_ring_set_water_mark(r, count);
-       if (r->watermark != r->size) {
-               printf("Test failed to detect invalid watermark count value\n");
-               goto error;
-       }
-       return 0;
-
-error:
-       return -1;
-}
-
 /*
  * helper routine for test_ring_basic
  */
@@ -249,20 +115,21 @@ test_ring_basic_full_empty(void * const src[], void *dst[])
                rand = RTE_MAX(rte_rand() % RING_SIZE, 1UL);
                printf("%s: iteration %u, random shift: %u;\n",
                    __func__, i, rand);
-               TEST_RING_VERIFY(-ENOBUFS != rte_ring_enqueue_bulk(r, src,
-                   rand));
-               TEST_RING_VERIFY(0 == rte_ring_dequeue_bulk(r, dst, rand));
+               TEST_RING_VERIFY(rte_ring_enqueue_bulk(r, src, rand,
+                               NULL) != 0);
+               TEST_RING_VERIFY(rte_ring_dequeue_bulk(r, dst, rand,
+                               NULL) == rand);
 
                /* fill the ring */
-               TEST_RING_VERIFY(-ENOBUFS != rte_ring_enqueue_bulk(r, src,
-                   rsz));
+               TEST_RING_VERIFY(rte_ring_enqueue_bulk(r, src, rsz, NULL) != 0);
                TEST_RING_VERIFY(0 == rte_ring_free_count(r));
                TEST_RING_VERIFY(rsz == rte_ring_count(r));
                TEST_RING_VERIFY(rte_ring_full(r));
                TEST_RING_VERIFY(0 == rte_ring_empty(r));
 
                /* empty the ring */
-               TEST_RING_VERIFY(0 == rte_ring_dequeue_bulk(r, dst, rsz));
+               TEST_RING_VERIFY(rte_ring_dequeue_bulk(r, dst, rsz,
+                               NULL) == rsz);
                TEST_RING_VERIFY(rsz == rte_ring_free_count(r));
                TEST_RING_VERIFY(0 == rte_ring_count(r));
                TEST_RING_VERIFY(0 == rte_ring_full(r));
@@ -301,39 +168,39 @@ test_ring_basic(void)
        cur_dst = dst;
 
        printf("enqueue 1 obj\n");
-       ret = rte_ring_sp_enqueue_bulk(r, cur_src, 1);
+       ret = rte_ring_sp_enqueue_bulk(r, cur_src, 1, NULL);
        cur_src += 1;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("enqueue 2 objs\n");
-       ret = rte_ring_sp_enqueue_bulk(r, cur_src, 2);
+       ret = rte_ring_sp_enqueue_bulk(r, cur_src, 2, NULL);
        cur_src += 2;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("enqueue MAX_BULK objs\n");
-       ret = rte_ring_sp_enqueue_bulk(r, cur_src, MAX_BULK);
+       ret = rte_ring_sp_enqueue_bulk(r, cur_src, MAX_BULK, NULL);
        cur_src += MAX_BULK;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("dequeue 1 obj\n");
-       ret = rte_ring_sc_dequeue_bulk(r, cur_dst, 1);
+       ret = rte_ring_sc_dequeue_bulk(r, cur_dst, 1, NULL);
        cur_dst += 1;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("dequeue 2 objs\n");
-       ret = rte_ring_sc_dequeue_bulk(r, cur_dst, 2);
+       ret = rte_ring_sc_dequeue_bulk(r, cur_dst, 2, NULL);
        cur_dst += 2;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("dequeue MAX_BULK objs\n");
-       ret = rte_ring_sc_dequeue_bulk(r, cur_dst, MAX_BULK);
+       ret = rte_ring_sc_dequeue_bulk(r, cur_dst, MAX_BULK, NULL);
        cur_dst += MAX_BULK;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        /* check data */
@@ -347,39 +214,39 @@ test_ring_basic(void)
        cur_dst = dst;
 
        printf("enqueue 1 obj\n");
-       ret = rte_ring_mp_enqueue_bulk(r, cur_src, 1);
+       ret = rte_ring_mp_enqueue_bulk(r, cur_src, 1, NULL);
        cur_src += 1;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("enqueue 2 objs\n");
-       ret = rte_ring_mp_enqueue_bulk(r, cur_src, 2);
+       ret = rte_ring_mp_enqueue_bulk(r, cur_src, 2, NULL);
        cur_src += 2;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("enqueue MAX_BULK objs\n");
-       ret = rte_ring_mp_enqueue_bulk(r, cur_src, MAX_BULK);
+       ret = rte_ring_mp_enqueue_bulk(r, cur_src, MAX_BULK, NULL);
        cur_src += MAX_BULK;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("dequeue 1 obj\n");
-       ret = rte_ring_mc_dequeue_bulk(r, cur_dst, 1);
+       ret = rte_ring_mc_dequeue_bulk(r, cur_dst, 1, NULL);
        cur_dst += 1;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("dequeue 2 objs\n");
-       ret = rte_ring_mc_dequeue_bulk(r, cur_dst, 2);
+       ret = rte_ring_mc_dequeue_bulk(r, cur_dst, 2, NULL);
        cur_dst += 2;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        printf("dequeue MAX_BULK objs\n");
-       ret = rte_ring_mc_dequeue_bulk(r, cur_dst, MAX_BULK);
+       ret = rte_ring_mc_dequeue_bulk(r, cur_dst, MAX_BULK, NULL);
        cur_dst += MAX_BULK;
-       if (ret != 0)
+       if (ret == 0)
                goto fail;
 
        /* check data */
@@ -394,13 +261,13 @@ test_ring_basic(void)
 
        printf("fill and empty the ring\n");
        for (i = 0; i<RING_SIZE/MAX_BULK; i++) {
-               ret = rte_ring_mp_enqueue_bulk(r, cur_src, MAX_BULK);
+               ret = rte_ring_mp_enqueue_bulk(r, cur_src, MAX_BULK, NULL);
                cur_src += MAX_BULK;
-               if (ret != 0)
+               if (ret == 0)
                        goto fail;
-               ret = rte_ring_mc_dequeue_bulk(r, cur_dst, MAX_BULK);
+               ret = rte_ring_mc_dequeue_bulk(r, cur_dst, MAX_BULK, NULL);
                cur_dst += MAX_BULK;
-               if (ret != 0)
+               if (ret == 0)
                        goto fail;
        }
 
@@ -418,34 +285,33 @@ test_ring_basic(void)
        cur_src = src;
        cur_dst = dst;
 
-       printf("test watermark and default bulk enqueue / dequeue\n");
-       rte_ring_set_water_mark(r, 20);
+       printf("test default bulk enqueue / dequeue\n");
        num_elems = 16;
 
        cur_src = src;
        cur_dst = dst;
 
-       ret = rte_ring_enqueue_bulk(r, cur_src, num_elems);
+       ret = rte_ring_enqueue_bulk(r, cur_src, num_elems, NULL);
        cur_src += num_elems;
-       if (ret != 0) {
+       if (ret == 0) {
                printf("Cannot enqueue\n");
                goto fail;
        }
-       ret = rte_ring_enqueue_bulk(r, cur_src, num_elems);
+       ret = rte_ring_enqueue_bulk(r, cur_src, num_elems, NULL);
        cur_src += num_elems;
-       if (ret != -EDQUOT) {
-               printf("Watermark not exceeded\n");
+       if (ret == 0) {
+               printf("Cannot enqueue\n");
                goto fail;
        }
-       ret = rte_ring_dequeue_bulk(r, cur_dst, num_elems);
+       ret = rte_ring_dequeue_bulk(r, cur_dst, num_elems, NULL);
        cur_dst += num_elems;
-       if (ret != 0) {
+       if (ret == 0) {
                printf("Cannot dequeue\n");
                goto fail;
        }
-       ret = rte_ring_dequeue_bulk(r, cur_dst, num_elems);
+       ret = rte_ring_dequeue_bulk(r, cur_dst, num_elems, NULL);
        cur_dst += num_elems;
-       if (ret != 0) {
+       if (ret == 0) {
                printf("Cannot dequeue2\n");
                goto fail;
        }
@@ -506,39 +372,39 @@ test_ring_burst_basic(void)
 
        printf("Test SP & SC basic functions \n");
        printf("enqueue 1 obj\n");
-       ret = rte_ring_sp_enqueue_burst(r, cur_src, 1);
+       ret = rte_ring_sp_enqueue_burst(r, cur_src, 1, NULL);
        cur_src += 1;
-       if ((ret & RTE_RING_SZ_MASK) != 1)
+       if (ret != 1)
                goto fail;
 
        printf("enqueue 2 objs\n");
-       ret = rte_ring_sp_enqueue_burst(r, cur_src, 2);
+       ret = rte_ring_sp_enqueue_burst(r, cur_src, 2, NULL);
        cur_src += 2;
-       if ((ret & RTE_RING_SZ_MASK) != 2)
+       if (ret != 2)
                goto fail;
 
        printf("enqueue MAX_BULK objs\n");
-       ret = rte_ring_sp_enqueue_burst(r, cur_src, MAX_BULK;
+       ret = rte_ring_sp_enqueue_burst(r, cur_src, MAX_BULK, NULL);
        cur_src += MAX_BULK;
-       if ((ret & RTE_RING_SZ_MASK) != MAX_BULK)
+       if (ret != MAX_BULK)
                goto fail;
 
        printf("dequeue 1 obj\n");
-       ret = rte_ring_sc_dequeue_burst(r, cur_dst, 1;
+       ret = rte_ring_sc_dequeue_burst(r, cur_dst, 1, NULL);
        cur_dst += 1;
-       if ((ret & RTE_RING_SZ_MASK) != 1)
+       if (ret != 1)
                goto fail;
 
        printf("dequeue 2 objs\n");
-       ret = rte_ring_sc_dequeue_burst(r, cur_dst, 2);
+       ret = rte_ring_sc_dequeue_burst(r, cur_dst, 2, NULL);
        cur_dst += 2;
-       if ((ret & RTE_RING_SZ_MASK) != 2)
+       if (ret != 2)
                goto fail;
 
        printf("dequeue MAX_BULK objs\n");
-       ret = rte_ring_sc_dequeue_burst(r, cur_dst, MAX_BULK);
+       ret = rte_ring_sc_dequeue_burst(r, cur_dst, MAX_BULK, NULL);
        cur_dst += MAX_BULK;
-       if ((ret & RTE_RING_SZ_MASK) != MAX_BULK)
+       if (ret != MAX_BULK)
                goto fail;
 
        /* check data */
@@ -554,24 +420,23 @@ test_ring_burst_basic(void)
 
        printf("Test enqueue without enough memory space \n");
        for (i = 0; i< (RING_SIZE/MAX_BULK - 1); i++) {
-               ret = rte_ring_sp_enqueue_burst(r, cur_src, MAX_BULK);
+               ret = rte_ring_sp_enqueue_burst(r, cur_src, MAX_BULK, NULL);
                cur_src += MAX_BULK;
-               if ((ret & RTE_RING_SZ_MASK) != MAX_BULK) {
+               if (ret != MAX_BULK)
                        goto fail;
-               }
        }
 
        printf("Enqueue 2 objects, free entries = MAX_BULK - 2  \n");
-       ret = rte_ring_sp_enqueue_burst(r, cur_src, 2);
+       ret = rte_ring_sp_enqueue_burst(r, cur_src, 2, NULL);
        cur_src += 2;
-       if ((ret & RTE_RING_SZ_MASK) != 2)
+       if (ret != 2)
                goto fail;
 
        printf("Enqueue the remaining entries = MAX_BULK - 2  \n");
        /* Always one free entry left */
-       ret = rte_ring_sp_enqueue_burst(r, cur_src, MAX_BULK);
+       ret = rte_ring_sp_enqueue_burst(r, cur_src, MAX_BULK, NULL);
        cur_src += MAX_BULK - 3;
-       if ((ret & RTE_RING_SZ_MASK) != MAX_BULK - 3)
+       if (ret != MAX_BULK - 3)
                goto fail;
 
        printf("Test if ring is full  \n");
@@ -579,27 +444,27 @@ test_ring_burst_basic(void)
                goto fail;
 
        printf("Test enqueue for a full entry  \n");
-       ret = rte_ring_sp_enqueue_burst(r, cur_src, MAX_BULK);
-       if ((ret & RTE_RING_SZ_MASK) != 0)
+       ret = rte_ring_sp_enqueue_burst(r, cur_src, MAX_BULK, NULL);
+       if (ret != 0)
                goto fail;
 
        printf("Test dequeue without enough objects \n");
        for (i = 0; i<RING_SIZE/MAX_BULK - 1; i++) {
-               ret = rte_ring_sc_dequeue_burst(r, cur_dst, MAX_BULK);
+               ret = rte_ring_sc_dequeue_burst(r, cur_dst, MAX_BULK, NULL);
                cur_dst += MAX_BULK;
-               if ((ret & RTE_RING_SZ_MASK) != MAX_BULK)
+               if (ret != MAX_BULK)
                        goto fail;
        }
 
        /* Available memory space for the exact MAX_BULK entries */
-       ret = rte_ring_sc_dequeue_burst(r, cur_dst, 2);
+       ret = rte_ring_sc_dequeue_burst(r, cur_dst, 2, NULL);
        cur_dst += 2;
-       if ((ret & RTE_RING_SZ_MASK) != 2)
+       if (ret != 2)
                goto fail;
 
-       ret = rte_ring_sc_dequeue_burst(r, cur_dst, MAX_BULK);
+       ret = rte_ring_sc_dequeue_burst(r, cur_dst, MAX_BULK, NULL);
        cur_dst += MAX_BULK - 3;
-       if ((ret & RTE_RING_SZ_MASK) != MAX_BULK - 3)
+       if (ret != MAX_BULK - 3)
                goto fail;
 
        printf("Test if ring is empty \n");
@@ -621,39 +486,39 @@ test_ring_burst_basic(void)
        printf("Test MP & MC basic functions \n");
 
        printf("enqueue 1 obj\n");
-       ret = rte_ring_mp_enqueue_burst(r, cur_src, 1);
+       ret = rte_ring_mp_enqueue_burst(r, cur_src, 1, NULL);
        cur_src += 1;
-       if ((ret & RTE_RING_SZ_MASK) != 1)
+       if (ret != 1)
                goto fail;
 
        printf("enqueue 2 objs\n");
-       ret = rte_ring_mp_enqueue_burst(r, cur_src, 2);
+       ret = rte_ring_mp_enqueue_burst(r, cur_src, 2, NULL);
        cur_src += 2;
-       if ((ret & RTE_RING_SZ_MASK) != 2)
+       if (ret != 2)
                goto fail;
 
        printf("enqueue MAX_BULK objs\n");
-       ret = rte_ring_mp_enqueue_burst(r, cur_src, MAX_BULK);
+       ret = rte_ring_mp_enqueue_burst(r, cur_src, MAX_BULK, NULL);
        cur_src += MAX_BULK;
-       if ((ret & RTE_RING_SZ_MASK) != MAX_BULK)
+       if (ret != MAX_BULK)
                goto fail;
 
        printf("dequeue 1 obj\n");
-       ret = rte_ring_mc_dequeue_burst(r, cur_dst, 1);
+       ret = rte_ring_mc_dequeue_burst(r, cur_dst, 1, NULL);
        cur_dst += 1;
-       if ((ret & RTE_RING_SZ_MASK) != 1)
+       if (ret != 1)
                goto fail;
 
        printf("dequeue 2 objs\n");
-       ret = rte_ring_mc_dequeue_burst(r, cur_dst, 2);
+       ret = rte_ring_mc_dequeue_burst(r, cur_dst, 2, NULL);
        cur_dst += 2;
-       if ((ret & RTE_RING_SZ_MASK) != 2)
+       if (ret != 2)
                goto fail;
 
        printf("dequeue MAX_BULK objs\n");
-       ret = rte_ring_mc_dequeue_burst(r, cur_dst, MAX_BULK);
+       ret = rte_ring_mc_dequeue_burst(r, cur_dst, MAX_BULK, NULL);
        cur_dst += MAX_BULK;
-       if ((ret & RTE_RING_SZ_MASK) != MAX_BULK)
+       if (ret != MAX_BULK)
                goto fail;
 
        /* check data */
@@ -669,13 +534,13 @@ test_ring_burst_basic(void)
 
        printf("fill and empty the ring\n");
        for (i = 0; i<RING_SIZE/MAX_BULK; i++) {
-               ret = rte_ring_mp_enqueue_burst(r, cur_src, MAX_BULK);
+               ret = rte_ring_mp_enqueue_burst(r, cur_src, MAX_BULK, NULL);
                cur_src += MAX_BULK;
-               if ((ret & RTE_RING_SZ_MASK) != MAX_BULK)
+               if (ret != MAX_BULK)
                        goto fail;
-               ret = rte_ring_mc_dequeue_burst(r, cur_dst, MAX_BULK);
+               ret = rte_ring_mc_dequeue_burst(r, cur_dst, MAX_BULK, NULL);
                cur_dst += MAX_BULK;
-               if ((ret & RTE_RING_SZ_MASK) != MAX_BULK)
+               if (ret != MAX_BULK)
                        goto fail;
        }
 
@@ -692,41 +557,41 @@ test_ring_burst_basic(void)
 
        printf("Test enqueue without enough memory space \n");
        for (i = 0; i<RING_SIZE/MAX_BULK - 1; i++) {
-               ret = rte_ring_mp_enqueue_burst(r, cur_src, MAX_BULK);
+               ret = rte_ring_mp_enqueue_burst(r, cur_src, MAX_BULK, NULL);
                cur_src += MAX_BULK;
-               if ((ret & RTE_RING_SZ_MASK) != MAX_BULK)
+               if (ret != MAX_BULK)
                        goto fail;
        }
 
        /* Available memory space for the exact MAX_BULK objects */
-       ret = rte_ring_mp_enqueue_burst(r, cur_src, 2);
+       ret = rte_ring_mp_enqueue_burst(r, cur_src, 2, NULL);
        cur_src += 2;
-       if ((ret & RTE_RING_SZ_MASK) != 2)
+       if (ret != 2)
                goto fail;
 
-       ret = rte_ring_mp_enqueue_burst(r, cur_src, MAX_BULK);
+       ret = rte_ring_mp_enqueue_burst(r, cur_src, MAX_BULK, NULL);
        cur_src += MAX_BULK - 3;
-       if ((ret & RTE_RING_SZ_MASK) != MAX_BULK - 3)
+       if (ret != MAX_BULK - 3)
                goto fail;
 
 
        printf("Test dequeue without enough objects \n");
        for (i = 0; i<RING_SIZE/MAX_BULK - 1; i++) {
-               ret = rte_ring_mc_dequeue_burst(r, cur_dst, MAX_BULK);
+               ret = rte_ring_mc_dequeue_burst(r, cur_dst, MAX_BULK, NULL);
                cur_dst += MAX_BULK;
-               if ((ret & RTE_RING_SZ_MASK) != MAX_BULK)
+               if (ret != MAX_BULK)
                        goto fail;
        }
 
        /* Available objects - the exact MAX_BULK */
-       ret = rte_ring_mc_dequeue_burst(r, cur_dst, 2);
+       ret = rte_ring_mc_dequeue_burst(r, cur_dst, 2, NULL);
        cur_dst += 2;
-       if ((ret & RTE_RING_SZ_MASK) != 2)
+       if (ret != 2)
                goto fail;
 
-       ret = rte_ring_mc_dequeue_burst(r, cur_dst, MAX_BULK);
+       ret = rte_ring_mc_dequeue_burst(r, cur_dst, MAX_BULK, NULL);
        cur_dst += MAX_BULK - 3;
-       if ((ret & RTE_RING_SZ_MASK) != MAX_BULK - 3)
+       if (ret != MAX_BULK - 3)
                goto fail;
 
        /* check data */
@@ -742,12 +607,12 @@ test_ring_burst_basic(void)
 
        printf("Covering rte_ring_enqueue_burst functions \n");
 
-       ret = rte_ring_enqueue_burst(r, cur_src, 2);
+       ret = rte_ring_enqueue_burst(r, cur_src, 2, NULL);
        cur_src += 2;
-       if ((ret & RTE_RING_SZ_MASK) != 2)
+       if (ret != 2)
                goto fail;
 
-       ret = rte_ring_dequeue_burst(r, cur_dst, 2);
+       ret = rte_ring_dequeue_burst(r, cur_dst, 2, NULL);
        cur_dst += 2;
        if (ret != 2)
                goto fail;
@@ -881,13 +746,13 @@ test_ring_basic_ex(void)
        }
 
        /* Covering the ring burst operation */
-       ret = rte_ring_enqueue_burst(rp, obj, 2);
-       if ((ret & RTE_RING_SZ_MASK) != 2) {
+       ret = rte_ring_enqueue_burst(rp, obj, 2, NULL);
+       if (ret != 2) {
                printf("test_ring_basic_ex: rte_ring_enqueue_burst fails \n");
                goto fail_test;
        }
 
-       ret = rte_ring_dequeue_burst(rp, obj, 2);
+       ret = rte_ring_dequeue_burst(rp, obj, 2, NULL);
        if (ret != 2) {
                printf("test_ring_basic_ex: rte_ring_dequeue_burst fails \n");
                goto fail_test;
@@ -901,6 +766,77 @@ fail_test:
        return ret;
 }
 
+static int
+test_ring_with_exact_size(void)
+{
+       struct rte_ring *std_ring = NULL, *exact_sz_ring = NULL;
+       void *ptr_array[16];
+       static const unsigned int ring_sz = RTE_DIM(ptr_array);
+       unsigned int i;
+       int ret = -1;
+
+       std_ring = rte_ring_create("std", ring_sz, rte_socket_id(),
+                       RING_F_SP_ENQ | RING_F_SC_DEQ);
+       if (std_ring == NULL) {
+               printf("%s: error, can't create std ring\n", __func__);
+               goto end;
+       }
+       exact_sz_ring = rte_ring_create("exact sz", ring_sz, rte_socket_id(),
+                       RING_F_SP_ENQ | RING_F_SC_DEQ | RING_F_EXACT_SZ);
+       if (exact_sz_ring == NULL) {
+               printf("%s: error, can't create exact size ring\n", __func__);
+               goto end;
+       }
+
+       /*
+        * Check that the exact size ring is bigger than the standard ring
+        */
+       if (rte_ring_get_size(std_ring) >= rte_ring_get_size(exact_sz_ring)) {
+               printf("%s: error, std ring (size: %u) is not smaller than exact size one (size %u)\n",
+                               __func__,
+                               rte_ring_get_size(std_ring),
+                               rte_ring_get_size(exact_sz_ring));
+               goto end;
+       }
+       /*
+        * check that the exact_sz_ring can hold one more element than the
+        * standard ring. (16 vs 15 elements)
+        */
+       for (i = 0; i < ring_sz - 1; i++) {
+               rte_ring_enqueue(std_ring, NULL);
+               rte_ring_enqueue(exact_sz_ring, NULL);
+       }
+       if (rte_ring_enqueue(std_ring, NULL) != -ENOBUFS) {
+               printf("%s: error, unexpected successful enqueue\n", __func__);
+               goto end;
+       }
+       if (rte_ring_enqueue(exact_sz_ring, NULL) == -ENOBUFS) {
+               printf("%s: error, enqueue failed\n", __func__);
+               goto end;
+       }
+
+       /* check that dequeue returns the expected number of elements */
+       if (rte_ring_dequeue_burst(exact_sz_ring, ptr_array,
+                       RTE_DIM(ptr_array), NULL) != ring_sz) {
+               printf("%s: error, failed to dequeue expected nb of elements\n",
+                               __func__);
+               goto end;
+       }
+
+       /* check that the capacity function returns expected value */
+       if (rte_ring_get_capacity(exact_sz_ring) != ring_sz) {
+               printf("%s: error, incorrect ring capacity reported\n",
+                               __func__);
+               goto end;
+       }
+
+       ret = 0; /* all ok if we get here */
+end:
+       rte_ring_free(std_ring);
+       rte_ring_free(exact_sz_ring);
+       return ret;
+}
+
 static int
 test_ring(void)
 {
@@ -930,16 +866,6 @@ test_ring(void)
                return -1;
 
        /* basic operations */
-       if (test_live_watermark_change() < 0)
-               return -1;
-
-       if ( test_set_watermark() < 0){
-               printf ("Test failed to detect invalid parameter\n");
-               return -1;
-       }
-       else
-               printf ( "Test detected forced bad watermark values\n");
-
        if ( test_create_count_odd() < 0){
                        printf ("Test failed to detect odd count\n");
                        return -1;
@@ -962,6 +888,9 @@ test_ring(void)
        if (test_ring_creation_with_an_used_name() < 0)
                return -1;
 
+       if (test_ring_with_exact_size() < 0)
+               return -1;
+
        /* dump the ring status */
        rte_ring_list_dump(stdout);