On 20.02.2017 18:18, David Miller wrote: > This would be so much easier to understand if it were coded as: > if (!tp->repair) { > seq = secure_tcp_sequence_number(...); > if (!tp->write_seq) > tp->write_seq = seq; > }
Hi David, Thought about not adding extra variable here... Agree,it looks much better and easier to follow. Should I send a patch with this version modified as you suggested or we can inherit ts offsetfrom tcptw->tw_ts_offset in tcp_twsk_unique()? Thanks, Alexey