diff options
author | Daniel Borkmann <daniel@iogearbox.net> | 2015-10-29 18:56:11 +0100 |
---|---|---|
committer | Daniel Borkmann <daniel@iogearbox.net> | 2015-10-29 18:56:11 +0100 |
commit | b1a4a6fa7b233abbca763178095b076d409cce80 (patch) | |
tree | 756254987b4e78c5fe4713dc598de1f420df3070 | |
parent | 55274adedcabdc3f1e4d66b2c3b77b1612acf20d (diff) |
ring: alloc_ring_frames_generic make types size_t
Lets make i and num as size_t, there's no particular reason for them
to be int. At least i is used to setup iov_base offsets.
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
-rw-r--r-- | ring.c | 5 | ||||
-rw-r--r-- | ring.h | 2 | ||||
-rw-r--r-- | ring_rx.c | 2 |
3 files changed, 4 insertions, 5 deletions
@@ -45,10 +45,9 @@ void mmap_ring_generic(int sock, struct ring *ring) panic("Cannot mmap {TX,RX}_RING!\n"); } -void alloc_ring_frames_generic(struct ring *ring, int num, size_t size) +void alloc_ring_frames_generic(struct ring *ring, size_t num, size_t size) { - int i; - size_t len = num * sizeof(*ring->frames); + size_t i, len = num * sizeof(*ring->frames); ring->frames = xmalloc_aligned(len, CO_CACHE_LINE_SIZE); fmemset(ring->frames, 0, len); @@ -200,7 +200,7 @@ static inline int get_sockopt_tpacket(int sock) extern void setup_ring_layout_generic(int sock, struct ring *ring, size_t size, bool jumbo_support); extern void mmap_ring_generic(int sock, struct ring *ring); -extern void alloc_ring_frames_generic(struct ring *ring, int num, size_t size); +extern void alloc_ring_frames_generic(struct ring *ring, size_t num, size_t size); extern void bind_ring_generic(int sock, struct ring *ring, int ifindex, bool tx_only); #endif /* RING_H */ @@ -91,7 +91,7 @@ static inline void setup_rx_ring_layout_v3(struct ring *ring __maybe_unused) { } -static inline int rx_ring_get_num(struct ring *ring, bool v3 __maybe_unused) +static inline size_t rx_ring_get_num(struct ring *ring, bool v3 __maybe_unused) { return ring->layout.tp_frame_nr; } |