Techwizz/kernel-fxtec-pro1x
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

[DCCP]: Introducing CCMPS

Browse source 
This introduces a CCMPS field for setting a CCID-specific upper bound on the application payload
size, as is defined in RFC 4340, section 14.

Only the TX CCID is considered in setting this limit, since the RX CCID generates comparatively
small (DCCP-Ack) feedback packets. The CCMPS field includes network and transport layer header
lengths. The only current CCMPS customer is CCID4 (via RFC 4828).

A wrapper is used to allow querying the CCMPS even at times where the CCID modules may not have
been fully negotiated yet.

In dccp_sync_mss() the variable `mss_now' has been renamed into `cur_mps', to reflect that we are
dealing with an MPS, but not an MSS.
Since the DCCP code closely follows the TCP code, the identifiers `dccp_sync_mss' and
`dccps_mss_cache' have been kept, as they have direct TCP counterparts.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>
Signed-off-by: Ian McDonald <ian.mcdonald@jandi.co.nz>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Gerrit Renker 2007-12-13 23:37:55 -02:00 committed by David S. Miller
parent 84a97b0af8
commit 6179983ad3
2 changed files with 25 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
net/dccp/ccid.h
View file

@ -27,6 +27,7 @@ struct tcp_info;
* struct ccid_operations - Interface to Congestion-Control Infrastructure * struct ccid_operations - Interface to Congestion-Control Infrastructure
* *
* @ccid_id: numerical CCID ID (up to %CCID_MAX, cf. table 5 in RFC 4340, 10.) * @ccid_id: numerical CCID ID (up to %CCID_MAX, cf. table 5 in RFC 4340, 10.)
* @ccid_ccmps: the CCMPS including network/transport headers (0 when disabled)
* @ccid_name: alphabetical identifier string for @ccid_id * @ccid_name: alphabetical identifier string for @ccid_id
* @ccid_owner: module which implements/owns this CCID * @ccid_owner: module which implements/owns this CCID
* @ccid_hc_{r,t}x_slab: memory pool for the receiver/sender half-connection * @ccid_hc_{r,t}x_slab: memory pool for the receiver/sender half-connection
@ -45,6 +46,7 @@ struct tcp_info;
*/ */
struct ccid_operations { struct ccid_operations {
unsigned char ccid_id; unsigned char ccid_id;
__u32 ccid_ccmps;
const char *ccid_name; const char *ccid_name;
struct module *ccid_owner; struct module *ccid_owner;
struct kmem_cache *ccid_hc_rx_slab, struct kmem_cache *ccid_hc_rx_slab,

30
net/dccp/output.c
View file

@ -133,15 +133,31 @@ static int dccp_transmit_skb(struct sock *sk, struct sk_buff *skb)
return -ENOBUFS; return -ENOBUFS;
} }
/**
* dccp_determine_ccmps - Find out about CCID-specfic packet-size limits
* We only consider the HC-sender CCID for setting the CCMPS (RFC 4340, 14.),
* since the RX CCID is restricted to feedback packets (Acks), which are small
* in comparison with the data traffic. A value of 0 means "no current CCMPS".
*/
static u32 dccp_determine_ccmps(const struct dccp_sock *dp)
{
const struct ccid *tx_ccid = dp->dccps_hc_tx_ccid;
if (tx_ccid == NULL || tx_ccid->ccid_ops == NULL)
return 0;
return tx_ccid->ccid_ops->ccid_ccmps;
}
unsigned int dccp_sync_mss(struct sock *sk, u32 pmtu) unsigned int dccp_sync_mss(struct sock *sk, u32 pmtu)
{ {
struct inet_connection_sock *icsk = inet_csk(sk); struct inet_connection_sock *icsk = inet_csk(sk);
struct dccp_sock *dp = dccp_sk(sk); struct dccp_sock *dp = dccp_sk(sk);
int mss_now = (pmtu - icsk->icsk_af_ops->net_header_len - u32 ccmps = dccp_determine_ccmps(dp);
sizeof(struct dccp_hdr) - sizeof(struct dccp_hdr_ext)); int cur_mps = ccmps ? min(pmtu, ccmps) : pmtu;
/* Now subtract optional transport overhead */ /* Account for header lengths and IPv4/v6 option overhead */
mss_now -= icsk->icsk_ext_hdr_len; cur_mps -= (icsk->icsk_af_ops->net_header_len + icsk->icsk_ext_hdr_len +
sizeof(struct dccp_hdr) + sizeof(struct dccp_hdr_ext));
/* /*
* FIXME: this should come from the CCID infrastructure, where, say, * FIXME: this should come from the CCID infrastructure, where, say,
@ -151,13 +167,13 @@ unsigned int dccp_sync_mss(struct sock *sk, u32 pmtu)
* make it a multiple of 4 * make it a multiple of 4
*/ */
mss_now -= ((5 + 6 + 10 + 6 + 6 + 6 + 3) / 4) * 4; cur_mps -= ((5 + 6 + 10 + 6 + 6 + 6 + 3) / 4) * 4;
/* And store cached results */ /* And store cached results */
icsk->icsk_pmtu_cookie = pmtu; icsk->icsk_pmtu_cookie = pmtu;
dp->dccps_mss_cache = mss_now; dp->dccps_mss_cache = cur_mps;
return mss_now; return cur_mps;
} }
EXPORT_SYMBOL_GPL(dccp_sync_mss); EXPORT_SYMBOL_GPL(dccp_sync_mss);