diff options
| author | Niels van Adrichem <N.L.M.vanAdrichem@tudelft.nl> | 2014-10-07 15:04:13 +0000 |
|---|---|---|
| committer | Alex Wang <alexw@nicira.com> | 2014-12-03 13:40:50 -0800 |
| commit | 75f0cdd9b7d6c217c25542bb6c31e3d7a37f5607 (patch) | |
| tree | 1919db9221f99e139eccb00ed9eeb1228016a8c2 | |
| parent | 859df5b1bf4e3a8135a2199187d3eb30174a3608 (diff) | |
| download | openvswitch-75f0cdd9b7d6c217c25542bb6c31e3d7a37f5607.tar.gz | |
BFD: Decreasing minimal transmit and receive interval
I found the BFD transmit interval was lowerbounded by the default value
without warning, although documentation does not consider a lowerbound.
Testing has been performed with transmit and receive intervals as low as 1
ms, and although CPU overhead was effected (especially with multiple BFD
sessions such as 6 and higher), it worked well.
Signed-off-by: Niels van Adrichem <n.l.m.vanadrichem@tudelft.nl>
Signed-off-by: Ben Pfaff <blp@nicira.com>
| -rw-r--r-- | AUTHORS | 1 | ||||
| -rw-r--r-- | lib/bfd.c | 4 |
2 files changed, 3 insertions, 2 deletions
@@ -236,6 +236,7 @@ Mike Kruze mkruze@nicira.com Min Chen ustcer.tonychan@gmail.com Mikael Doverhag mdoverhag@nicira.com Nagi Reddy Jonnala njonnala@Brocade.com +Niels van Adrichem N.L.M.vanAdrichem@tudelft.nl Niklas Andersson nandersson@nicira.com Pankaj Thakkar thakkar@nicira.com Pasi Kärkkäinen pasik@iki.fi @@ -396,7 +396,7 @@ bfd_configure(struct bfd *bfd, const char *name, const struct smap *cfg, atomic_store(&bfd->check_tnl_key, smap_get_bool(cfg, "check_tnl_key", false)); min_tx = smap_get_int(cfg, "min_tx", 100); - min_tx = MAX(min_tx, 100); + min_tx = MAX(min_tx, 1); if (bfd->cfg_min_tx != min_tx) { bfd->cfg_min_tx = min_tx; if (bfd->state != STATE_UP @@ -407,7 +407,7 @@ bfd_configure(struct bfd *bfd, const char *name, const struct smap *cfg, } min_rx = smap_get_int(cfg, "min_rx", 1000); - min_rx = MAX(min_rx, 100); + min_rx = MAX(min_rx, 1); if (bfd->cfg_min_rx != min_rx) { bfd->cfg_min_rx = min_rx; if (bfd->state != STATE_UP |