Re: [PATCH] tcp: assign the sock correctly to an outgoing SYNACK packet

Eric Dumazet
2013-04-09T15:32:56Z

On Tue, 2013-04-09 at 11:17 -0400, Paul Moore wrote:

> The "blob" is a void pointer, so 8 bytes. We're talking about removing the
> "secmark" field (4 bytes) and adding a void pointer (8 bytes). I've shown
> several different approaches that make this change without increasing the
> overall size of the sk_buff struct.

You want to use 4 extra bytes in sk_buff. You'll have to show us why we
close the way for other valid uses of the current holes.

I have no idea why its needed, and why it can't be solved in another
way.

It looks like _I_ have to do your work. Sorry, I have no more time to
spend on this topic. You'll have to convince David, not me.