Development issues regarding the cerowrt test router project
 help / color / mirror / Atom feed
* [Cerowrt-devel] instruction traps on tcp_rcv - bug 360 regression - reopened
@ 2013-12-22 21:35 Dave Taht
  2013-12-22 22:26 ` Robert Bradley
  0 siblings, 1 reply; 2+ messages in thread
From: Dave Taht @ 2013-12-22 21:35 UTC (permalink / raw)
  To: cerowrt-devel

http://www.bufferbloat.net/issues/360

I would be very reluctant to take seriously any benchmarks going to
the router until this one is quashed. Really bad behavior (I put a
plot up on the bug)

Stuff going through the router appears to be sane, but if you are monitoring
the router at the same time (like via ssh or a web page) there are
side effects visible at higher rates.

I am hopefully getting setup to do ipv6 again soon so as to make sure
all those bugs remain squashed too....

-- 
Dave Täht

Fixing bufferbloat with cerowrt: http://www.teklibre.com/cerowrt/subscribe.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [Cerowrt-devel] instruction traps on tcp_rcv - bug 360 regression - reopened
  2013-12-22 21:35 [Cerowrt-devel] instruction traps on tcp_rcv - bug 360 regression - reopened Dave Taht
@ 2013-12-22 22:26 ` Robert Bradley
  0 siblings, 0 replies; 2+ messages in thread
From: Robert Bradley @ 2013-12-22 22:26 UTC (permalink / raw)
  To: cerowrt-devel

On 22/12/13 21:35, Dave Taht wrote:
> http://www.bufferbloat.net/issues/360
>
> I would be very reluctant to take seriously any benchmarks going to
> the router until this one is quashed. Really bad behavior (I put a
> plot up on the bug)
>
> Stuff going through the router appears to be sane, but if you are monitoring
> the router at the same time (like via ssh or a web page) there are
> side effects visible at higher rates.
>
> I am hopefully getting setup to do ipv6 again soon so as to make sure
> all those bugs remain squashed too....
>

It looks like tcp_parse_aligned_timestamp() in net/ipv4/tcp_input.c has
acquired an if(*ptr) statement between 3.8 and 3.10.  That bug had been
annoying me for a while...

diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c
index 3d67689..c6496eb 100644
--- a/net/ipv4/tcp_input.c
+++ b/net/ipv4/tcp_input.c
@@ -3632,7 +3632,7 @@ static bool tcp_parse_aligned_timestamp(struct
tcp_sock *tp, const struct tcphdr
                ++ptr;
                tp->rx_opt.rcv_tsval = get_unaligned_be32(ptr);
                ++ptr;
-               if (*ptr)
+               if (net_hdr_word(ptr))
                        tp->rx_opt.rcv_tsecr = get_unaligned_be32(ptr) -
                                               tp->tsoffset;
                else

-- 
Robert Bradley


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-12-22 22:26 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-12-22 21:35 [Cerowrt-devel] instruction traps on tcp_rcv - bug 360 regression - reopened Dave Taht
2013-12-22 22:26 ` Robert Bradley

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox