* [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