update copyright date to 2013
[dpdk.git] / lib / librte_ring / rte_ring.c
index 08a3c80..18ad22a 100644 (file)
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  * 
- *   Copyright(c) 2010-2012 Intel Corporation. All rights reserved.
+ *   Copyright(c) 2010-2013 Intel Corporation. All rights reserved.
  *   All rights reserved.
  * 
  *   Redistribution and use in source and binary forms, with or without 
 #include <rte_launch.h>
 #include <rte_tailq.h>
 #include <rte_eal.h>
+#include <rte_eal_memconfig.h>
 #include <rte_atomic.h>
 #include <rte_per_lcore.h>
 #include <rte_lcore.h>
 #include <rte_branch_prediction.h>
 #include <rte_errno.h>
 #include <rte_string_fns.h>
+#include <rte_spinlock.h>
 
 #include "rte_ring.h"
 
 TAILQ_HEAD(rte_ring_list, rte_ring);
 
-/* global list of ring (used for debug/dump) */
-static struct rte_ring_list *ring_list = NULL;
-
 /* true if x is a power of 2 */
 #define POWEROF2(x) ((((x)-1) & (x)) == 0)
 
@@ -106,6 +105,7 @@ rte_ring_create(const char *name, unsigned count, int socket_id,
        const struct rte_memzone *mz;
        size_t ring_size;
        int mz_flags = 0;
+       struct rte_ring_list* ring_list = NULL;
 
        /* compilation-time checks */
        RTE_BUILD_BUG_ON((sizeof(struct rte_ring) &
@@ -122,47 +122,51 @@ rte_ring_create(const char *name, unsigned count, int socket_id,
 #endif
 
        /* check that we have an initialised tail queue */
-       if (ring_list == NULL)
-               if ((ring_list = RTE_TAILQ_RESERVE("RTE_RING", rte_ring_list)) == NULL){
-                       rte_errno = E_RTE_NO_TAILQ;
-                       return NULL;
-               }
+       if ((ring_list = 
+            RTE_TAILQ_LOOKUP_BY_IDX(RTE_TAILQ_RING, rte_ring_list)) == NULL) {
+               rte_errno = E_RTE_NO_TAILQ;
+               return NULL;    
+       }
 
        /* count must be a power of 2 */
-       if (!POWEROF2(count)) {
+       if ((!POWEROF2(count)) || (count > RTE_RING_SZ_MASK )) {
                rte_errno = EINVAL;
-               RTE_LOG(ERR, RING, "Requested size is not a power of 2\n");
+               RTE_LOG(ERR, RING, "Requested size is invalid, must be power of 2, and "
+                               "do not exceed the size limit %u\n", RTE_RING_SZ_MASK);
                return NULL;
        }
 
        rte_snprintf(mz_name, sizeof(mz_name), "RG_%s", name);
        ring_size = count * sizeof(void *) + sizeof(struct rte_ring);
 
+       rte_rwlock_write_lock(RTE_EAL_TAILQ_RWLOCK);
+
        /* reserve a memory zone for this ring. If we can't get rte_config or
         * we are secondary process, the memzone_reserve function will set
         * rte_errno for us appropriately - hence no check in this this function */
        mz = rte_memzone_reserve(mz_name, ring_size, socket_id, mz_flags);
-       if (mz == NULL) {
-               RTE_LOG(ERR, RING, "Cannot reserve memory\n");
-               return NULL;
-       }
+       if (mz != NULL) {
+               r = mz->addr;
 
-       r = mz->addr;
+               /* init the ring structure */
+               memset(r, 0, sizeof(*r));
+               rte_snprintf(r->name, sizeof(r->name), "%s", name);
+               r->flags = flags;
+               r->prod.watermark = count;
+               r->prod.sp_enqueue = !!(flags & RING_F_SP_ENQ);
+               r->cons.sc_dequeue = !!(flags & RING_F_SC_DEQ);
+               r->prod.size = r->cons.size = count;
+               r->prod.mask = r->cons.mask = count-1;
+               r->prod.head = r->cons.head = 0;
+               r->prod.tail = r->cons.tail = 0;
 
-       /* init the ring structure */
-       memset(r, 0, sizeof(*r));
-       rte_snprintf(r->name, sizeof(r->name), "%s", name);
-       r->flags = flags;
-       r->prod.bulk_default = r->cons.bulk_default = 1;
-       r->prod.watermark = count;
-       r->prod.sp_enqueue = !!(flags & RING_F_SP_ENQ);
-       r->cons.sc_dequeue = !!(flags & RING_F_SC_DEQ);
-       r->prod.size = r->cons.size = count;
-       r->prod.mask = r->cons.mask = count-1;
-       r->prod.head = r->cons.head = 0;
-       r->prod.tail = r->cons.tail = 0;
-
-       TAILQ_INSERT_TAIL(ring_list, r, next);
+               TAILQ_INSERT_TAIL(ring_list, r, next);
+       } else {
+               r = NULL;
+               RTE_LOG(ERR, RING, "Cannot reserve memory\n");
+       }
+       rte_rwlock_write_unlock(RTE_EAL_TAILQ_RWLOCK);
+       
        return r;
 }
 
@@ -206,7 +210,6 @@ rte_ring_dump(const struct rte_ring *r)
                printf("  watermark=0\n");
        else
                printf("  watermark=%"PRIu32"\n", r->prod.watermark);
-       printf("  bulk_default=%"PRIu32"\n", r->prod.bulk_default);
 
        /* sum and dump statistics */
 #ifdef RTE_LIBRTE_RING_DEBUG
@@ -244,17 +247,22 @@ void
 rte_ring_list_dump(void)
 {
        const struct rte_ring *mp;
+       struct rte_ring_list *ring_list;
 
        /* check that we have an initialised tail queue */
-       if (ring_list == NULL)
-               if ((ring_list = RTE_TAILQ_RESERVE("RTE_RING", rte_ring_list)) == NULL){
-                       rte_errno = E_RTE_NO_TAILQ;
-                       return;
-               }
+       if ((ring_list = 
+            RTE_TAILQ_LOOKUP_BY_IDX(RTE_TAILQ_RING, rte_ring_list)) == NULL) {
+               rte_errno = E_RTE_NO_TAILQ;
+               return; 
+       }
+
+       rte_rwlock_read_lock(RTE_EAL_TAILQ_RWLOCK);
 
        TAILQ_FOREACH(mp, ring_list, next) {
                rte_ring_dump(mp);
        }
+
+       rte_rwlock_read_unlock(RTE_EAL_TAILQ_RWLOCK);
 }
 
 /* search a ring from its name */
@@ -262,19 +270,24 @@ struct rte_ring *
 rte_ring_lookup(const char *name)
 {
        struct rte_ring *r;
+       struct rte_ring_list *ring_list;
 
-       /* check that we have an initialised tail queue */
-       if (ring_list == NULL)
-               if ((ring_list = RTE_TAILQ_RESERVE("RTE_RING", rte_ring_list)) == NULL){
-                       rte_errno = E_RTE_NO_TAILQ;
-                       return NULL;
-               }
+       /* check that we have an initialized tail queue */
+       if ((ring_list = 
+            RTE_TAILQ_LOOKUP_BY_IDX(RTE_TAILQ_RING, rte_ring_list)) == NULL) {
+               rte_errno = E_RTE_NO_TAILQ;
+               return NULL;    
+       }
 
+       rte_rwlock_read_lock(RTE_EAL_TAILQ_RWLOCK);
+       
        TAILQ_FOREACH(r, ring_list, next) {
                if (strncmp(name, r->name, RTE_RING_NAMESIZE) == 0)
                        break;
        }
 
+       rte_rwlock_read_unlock(RTE_EAL_TAILQ_RWLOCK);
+
        if (r == NULL)
                rte_errno = ENOENT;