Re: [ovs-dev] [PATCH 1/2] bfd: Change the update of forwarding_if_rx_detect_time.

2013-12-09 Thread Ethan Jackson
k sounds good. Ethan On Mon, Dec 9, 2013 at 3:21 PM, Alex Wang wrote: > okay, this makes sense. we could drop this patch. > > > On Mon, Dec 9, 2013 at 3:18 PM, Ethan Jackson wrote: >> >> Doesn't the next patch essentially resolve this issue by making the >> forwarding_if_rx calculation more re

Re: [ovs-dev] [PATCH 1/2] bfd: Change the update of forwarding_if_rx_detect_time.

2013-12-09 Thread Alex Wang
okay, this makes sense. we could drop this patch. On Mon, Dec 9, 2013 at 3:18 PM, Ethan Jackson wrote: > Doesn't the next patch essentially resolve this issue by making the > forwarding_if_rx calculation more responsive? With the new threaded > put stuff that's coming, we can more or less gua

Re: [ovs-dev] [PATCH 1/2] bfd: Change the update of forwarding_if_rx_detect_time.

2013-12-09 Thread Ethan Jackson
Doesn't the next patch essentially resolve this issue by making the forwarding_if_rx calculation more responsive? With the new threaded put stuff that's coming, we can more or less guarantee that stats will be updated at worst once every 1.5 seconds which makes me think that 3000 seconds is overki

[ovs-dev] [PATCH 1/2] bfd: Change the update of forwarding_if_rx_detect_time.

2013-12-09 Thread Alex Wang
Currently, the ovs updates the rx_packet counter approximately every 2000 ms. If there is rx_packet and forwarding_if_rx is enabled, the forwarding_if_rx_detect_time will be incremented by 2000 ms. And as long as forwarding_if_rx_detect_time is greater than current time, the forwarding flag value