tcp/syncache: Set flowid and hash type properly for SYN|ACK

[sepherosa_gmail.com]

Summary

So the underlying drivers can use it to select the sending queue properly for SYN|ACK instead of rolling their own hash.

[hiren]

I hope you'd write a bit more descriptive commit-log (not just 'what' but also 'why') for the change. Thanks a lot for your work!

Cheers,
Hiren

sys/netinet/tcp_syncache.c

1507 ↗

(On Diff #15647)

Do you mind adding a line or two about what this function does in comments to improve readability? Thanks :-)

[sepherosa_gmail.com]

In D6120#130786, @hiren wrote:

I hope you'd write a bit more descriptive commit-log (not just 'what' but also 'why') for the change. Thanks a lot for your work!

OK :)

sys/netinet/tcp_syncache.c

1507 ↗

(On Diff #15647)

OK, but as a different commit.

[• hselasky]

Basically what you're doing is to loop back the flowid of the received packet - right?

Maybe you could put that in a code comment.

[sepherosa_gmail.com]

In D6120#130844, @hselasky wrote:

Basically what you're doing is to loop back the flowid of the received packet - right?

Maybe you could put that in a code comment.

Yep, since we don't have inp to loopback the SYN's flowid here. So we use the SYN's flowid directly. OK, will add.