Skip to content

Commit

Permalink
zebra: Actually display I/O buffer sizes
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
donaldsharp committed Apr 9, 2024
1 parent 48805d6 commit 8d8f12b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
2 changes: 0 additions & 2 deletions lib/stream.h
Original file line number Diff line number Diff line change
Expand Up @@ -105,9 +105,7 @@ struct stream_fifo {

/* number of streams in this fifo */
atomic_size_t count;
#if defined DEV_BUILD
atomic_size_t max_count;
#endif

struct stream *head;
struct stream *tail;
Expand Down
4 changes: 1 addition & 3 deletions zebra/zserv.c
Original file line number Diff line number Diff line change
Expand Up @@ -1125,12 +1125,10 @@ static void zebra_show_client_detail(struct vty *vty, struct zserv *client)
vty_out(vty, "ES-EVI %-12u%-12u%-12u\n",
client->local_es_evi_add_cnt, 0, client->local_es_evi_del_cnt);
vty_out(vty, "Errors: %u\n", client->error_cnt);

#if defined DEV_BUILD
vty_out(vty, "Input Fifo: %zu:%zu Output Fifo: %zu:%zu\n",
client->ibuf_fifo->count, client->ibuf_fifo->max_count,
client->obuf_fifo->count, client->obuf_fifo->max_count);
#endif

vty_out(vty, "\n");
}

Expand Down

0 comments on commit 8d8f12b

Please sign in to comment.