Netdev List
 help / color / mirror / Atom feed
From: "Sébastien Barré" <sebastien.barre@uclouvain.be>
To: netdev@vger.kernel.org
Cc: alan.p.smith@bt.com
Subject: increase in size for struct request_sock tolerable ?
Date: Wed, 03 Mar 2010 10:35:37 +0100	[thread overview]
Message-ID: <4B8E2D69.5070900@uclouvain.be> (raw)

Hi,

As part of the development of the MPTCP protocol 
(http://inl.info.ucl.ac.be/mptcp), we are wondering whether it is 
tolerable to extend the length of the struct request_sock. In the 
current version of the protocol 
(http://tools.ietf.org/id/draft-ford-mptcp-multiaddressed-02.txt), a 
32bit token is exchanged as TCP options in the SYN and SYN/ACK segments. 
AFAIU, that requires that we store 8 bytes in the request sock for the 
server side, so that it remembers the local and remote tokens until the 
real sock is created.Even worse, since the NULL token is not a reserved 
value, we need to store an additional bit (flag) to know that we 
received that TCP option. Since there are no flags currently stored in 
the request_sock, we need to directly increase the size by 8 additional 
bits.

So with the current protocol design, I arrive to a struct request_sock 
extension of 4 (local token)+4(remote token)+1(1bit flag+7free flags)=9 
bytes.

A minor protocol modification would allow reducing that constraint. 
Basically, we would then differ the token exchange until the real sock 
is created.    The flag would still be needed, though, thus in that case 
we would arrive to 1 byte of extension only for the struct request_sock.

So my question is : Do you think there is a sufficient incentive here to 
adapt our protocol draft ? Or is it tolerable to extend the struct 
request_sock by 9 bytes ?

Thanks in advance !

Sébastien.

-- 
Sébastien Barré
Researcher,
CSE department, UCLouvain, Belgium
http://inl.info.ucl.ac.be/sbarre


                 reply	other threads:[~2010-03-03  9:46 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4B8E2D69.5070900@uclouvain.be \
    --to=sebastien.barre@uclouvain.be \
    --cc=alan.p.smith@bt.com \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox