Skip to content

Commit 8d8f12b

Browse files
committed
zebra: Actually display I/O buffer sizes
An operator found a situation where zebra was backing up in a significant way towards BGP with EVPN changes taking up some serious amounts of memory. The key lines that would have clued us in on it were behind a dev build. Let's change this. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
1 parent 48805d6 commit 8d8f12b

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

lib/stream.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,9 +105,7 @@ struct stream_fifo {
105105

106106
/* number of streams in this fifo */
107107
atomic_size_t count;
108-
#if defined DEV_BUILD
109108
atomic_size_t max_count;
110-
#endif
111109

112110
struct stream *head;
113111
struct stream *tail;

zebra/zserv.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1125,12 +1125,10 @@ static void zebra_show_client_detail(struct vty *vty, struct zserv *client)
11251125
vty_out(vty, "ES-EVI %-12u%-12u%-12u\n",
11261126
client->local_es_evi_add_cnt, 0, client->local_es_evi_del_cnt);
11271127
vty_out(vty, "Errors: %u\n", client->error_cnt);
1128-
1129-
#if defined DEV_BUILD
11301128
vty_out(vty, "Input Fifo: %zu:%zu Output Fifo: %zu:%zu\n",
11311129
client->ibuf_fifo->count, client->ibuf_fifo->max_count,
11321130
client->obuf_fifo->count, client->obuf_fifo->max_count);
1133-
#endif
1131+
11341132
vty_out(vty, "\n");
11351133
}
11361134

0 commit comments

Comments
 (0)