Re: [PATCH] fixes the type of 'index' in struct open_request

[kuznet@xxxxxxxxxxxxx]

Hello!

> Looks like this field is only set, but not being read anywhere within TCP 
> currently, 

Yes, it was used only in some tests with timing out by calendar
rather then scanning hash table.


>       probably the reason why it was not noticed till now.

No, I remember this. It is an explicit padding now, though
initialization should be deleted yet.


> field to get the syntable hash bucket index of the openreq instead of 
> recalculating the hash.

Is this really necessary? Not a big deal, of course, but it adds another
4 bytes and it is really useless. To _find_ an open request you need
to calculate hash function...

Alexey