add FILE argument to debug functions
[dpdk.git] / lib / librte_ring / rte_ring.c
index 0d43a55..3a919b0 100644 (file)
@@ -94,18 +94,29 @@ TAILQ_HEAD(rte_ring_list, rte_ring);
 /* true if x is a power of 2 */
 #define POWEROF2(x) ((((x)-1) & (x)) == 0)
 
-/* create the ring */
-struct rte_ring *
-rte_ring_create(const char *name, unsigned count, int socket_id,
-               unsigned flags)
+/* return the size of memory occupied by a ring */
+ssize_t
+rte_ring_get_memsize(unsigned count)
 {
-       char mz_name[RTE_MEMZONE_NAMESIZE];
-       struct rte_ring *r;
-       const struct rte_memzone *mz;
-       size_t ring_size;
-       int mz_flags = 0;
-       struct rte_ring_list* ring_list = NULL;
+       ssize_t sz;
+
+       /* count must be a power of 2 */
+       if ((!POWEROF2(count)) || (count > RTE_RING_SZ_MASK )) {
+               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 -EINVAL;
+       }
 
+       sz = sizeof(struct rte_ring) + count * sizeof(void *);
+       sz = RTE_ALIGN(sz, CACHE_LINE_SIZE);
+       return sz;
+}
+
+int
+rte_ring_init(struct rte_ring *r, const char *name, unsigned count,
+       unsigned flags)
+{
        /* compilation-time checks */
        RTE_BUILD_BUG_ON((sizeof(struct rte_ring) &
                          CACHE_LINE_MASK) != 0);
@@ -122,23 +133,47 @@ rte_ring_create(const char *name, unsigned count, int socket_id,
                          CACHE_LINE_MASK) != 0);
 #endif
 
+       /* 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;
+
+       return 0;
+}
+
+/* create the ring */
+struct rte_ring *
+rte_ring_create(const char *name, unsigned count, int socket_id,
+               unsigned flags)
+{
+       char mz_name[RTE_MEMZONE_NAMESIZE];
+       struct rte_ring *r;
+       const struct rte_memzone *mz;
+       ssize_t ring_size;
+       int mz_flags = 0;
+       struct rte_ring_list* ring_list = NULL;
+
        /* check that we have an initialised tail queue */
-       if ((ring_list = 
+       if ((ring_list =
             RTE_TAILQ_LOOKUP_BY_IDX(RTE_TAILQ_RING, rte_ring_list)) == NULL) {
                rte_errno = E_RTE_NO_TAILQ;
-               return NULL;    
+               return NULL;
        }
 
-       /* count must be a power of 2 */
-       if ((!POWEROF2(count)) || (count > RTE_RING_SZ_MASK )) {
-               rte_errno = EINVAL;
-               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);
+       ring_size = rte_ring_get_memsize(count);
+       if (ring_size < 0) {
+               rte_errno = ring_size;
                return NULL;
        }
 
        rte_snprintf(mz_name, sizeof(mz_name), "%s%s", RTE_RING_MZ_PREFIX, name);
-       ring_size = count * sizeof(void *) + sizeof(struct rte_ring);
 
        rte_rwlock_write_lock(RTE_EAL_TAILQ_RWLOCK);
 
@@ -148,26 +183,16 @@ rte_ring_create(const char *name, unsigned count, int socket_id,
        mz = rte_memzone_reserve(mz_name, ring_size, socket_id, mz_flags);
        if (mz != NULL) {
                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;
-
+               /* no need to check return value here, we already checked the
+                * arguments above */
+               rte_ring_init(r, name, count, flags);
                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;
 }
 
@@ -191,26 +216,26 @@ rte_ring_set_water_mark(struct rte_ring *r, unsigned count)
 
 /* dump the status of the ring on the console */
 void
-rte_ring_dump(const struct rte_ring *r)
+rte_ring_dump(FILE *f, const struct rte_ring *r)
 {
 #ifdef RTE_LIBRTE_RING_DEBUG
        struct rte_ring_debug_stats sum;
        unsigned lcore_id;
 #endif
 
-       printf("ring <%s>@%p\n", r->name, r);
-       printf("  flags=%x\n", r->flags);
-       printf("  size=%"PRIu32"\n", r->prod.size);
-       printf("  ct=%"PRIu32"\n", r->cons.tail);
-       printf("  ch=%"PRIu32"\n", r->cons.head);
-       printf("  pt=%"PRIu32"\n", r->prod.tail);
-       printf("  ph=%"PRIu32"\n", r->prod.head);
-       printf("  used=%u\n", rte_ring_count(r));
-       printf("  avail=%u\n", rte_ring_free_count(r));
+       fprintf(f, "ring <%s>@%p\n", r->name, r);
+       fprintf(f, "  flags=%x\n", r->flags);
+       fprintf(f, "  size=%"PRIu32"\n", r->prod.size);
+       fprintf(f, "  ct=%"PRIu32"\n", r->cons.tail);
+       fprintf(f, "  ch=%"PRIu32"\n", r->cons.head);
+       fprintf(f, "  pt=%"PRIu32"\n", r->prod.tail);
+       fprintf(f, "  ph=%"PRIu32"\n", r->prod.head);
+       fprintf(f, "  used=%u\n", rte_ring_count(r));
+       fprintf(f, "  avail=%u\n", rte_ring_free_count(r));
        if (r->prod.watermark == r->prod.size)
-               printf("  watermark=0\n");
+               fprintf(f, "  watermark=0\n");
        else
-               printf("  watermark=%"PRIu32"\n", r->prod.watermark);
+               fprintf(f, "  watermark=%"PRIu32"\n", r->prod.watermark);
 
        /* sum and dump statistics */
 #ifdef RTE_LIBRTE_RING_DEBUG
@@ -227,25 +252,25 @@ rte_ring_dump(const struct rte_ring *r)
                sum.deq_fail_bulk += r->stats[lcore_id].deq_fail_bulk;
                sum.deq_fail_objs += r->stats[lcore_id].deq_fail_objs;
        }
-       printf("  size=%"PRIu32"\n", r->prod.size);
-       printf("  enq_success_bulk=%"PRIu64"\n", sum.enq_success_bulk);
-       printf("  enq_success_objs=%"PRIu64"\n", sum.enq_success_objs);
-       printf("  enq_quota_bulk=%"PRIu64"\n", sum.enq_quota_bulk);
-       printf("  enq_quota_objs=%"PRIu64"\n", sum.enq_quota_objs);
-       printf("  enq_fail_bulk=%"PRIu64"\n", sum.enq_fail_bulk);
-       printf("  enq_fail_objs=%"PRIu64"\n", sum.enq_fail_objs);
-       printf("  deq_success_bulk=%"PRIu64"\n", sum.deq_success_bulk);
-       printf("  deq_success_objs=%"PRIu64"\n", sum.deq_success_objs);
-       printf("  deq_fail_bulk=%"PRIu64"\n", sum.deq_fail_bulk);
-       printf("  deq_fail_objs=%"PRIu64"\n", sum.deq_fail_objs);
+       fprintf(f, "  size=%"PRIu32"\n", r->prod.size);
+       fprintf(f, "  enq_success_bulk=%"PRIu64"\n", sum.enq_success_bulk);
+       fprintf(f, "  enq_success_objs=%"PRIu64"\n", sum.enq_success_objs);
+       fprintf(f, "  enq_quota_bulk=%"PRIu64"\n", sum.enq_quota_bulk);
+       fprintf(f, "  enq_quota_objs=%"PRIu64"\n", sum.enq_quota_objs);
+       fprintf(f, "  enq_fail_bulk=%"PRIu64"\n", sum.enq_fail_bulk);
+       fprintf(f, "  enq_fail_objs=%"PRIu64"\n", sum.enq_fail_objs);
+       fprintf(f, "  deq_success_bulk=%"PRIu64"\n", sum.deq_success_bulk);
+       fprintf(f, "  deq_success_objs=%"PRIu64"\n", sum.deq_success_objs);
+       fprintf(f, "  deq_fail_bulk=%"PRIu64"\n", sum.deq_fail_bulk);
+       fprintf(f, "  deq_fail_objs=%"PRIu64"\n", sum.deq_fail_objs);
 #else
-       printf("  no statistics available\n");
+       fprintf(f, "  no statistics available\n");
 #endif
 }
 
 /* dump the status of all rings on the console */
 void
-rte_ring_list_dump(void)
+rte_ring_list_dump(FILE *f)
 {
        const struct rte_ring *mp;
        struct rte_ring_list *ring_list;
@@ -260,7 +285,7 @@ rte_ring_list_dump(void)
        rte_rwlock_read_lock(RTE_EAL_TAILQ_RWLOCK);
 
        TAILQ_FOREACH(mp, ring_list, next) {
-               rte_ring_dump(mp);
+               rte_ring_dump(f, mp);
        }
 
        rte_rwlock_read_unlock(RTE_EAL_TAILQ_RWLOCK);