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

Reply via email to