aboutsummaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorGerrit Renker <gerrit@erg.abdn.ac.uk>2008-09-04 01:30:19 -0400
committerGerrit Renker <gerrit@erg.abdn.ac.uk>2008-09-04 01:45:28 -0400
commit17c30b40ed79e9f3955e884632c8f01e577b204a (patch)
tree41b91885f808f0b1d48b8b87a303ea7891565a46 /include
parent20f41eee82864e308a5499308a1722dc3181cc3a (diff)
dccp: Deprecate Ack Ratio sysctl
This patch deprecates the Ack Ratio sysctl, since * Ack Ratio is entirely ignored by CCID-3 and CCID-4, * Ack Ratio currently doesn't work in CCID-2 (i.e. is always set to 1); * even if it would work in CCID-2, there is no point for a user to change it: - Ack Ratio is constrained by cwnd (RFC 4341, 6.1.2), - if Ack Ratio > cwnd, the system resorts to spurious RTO timeouts (since waiting for Acks which will never arrive in this window), - cwnd is not a user-configurable value. The only reasonable place for Ack Ratio is to print it for debugging. It is planned to do this later on, as part of e.g. dccp_probe. With this patch Ack Ratio is now under full control of feature negotiation: * Ack Ratio is resolved as a dependency of the selected CCID; * if the chosen CCID supports it (i.e. CCID == CCID-2), Ack Ratio is set to the default of 2, following RFC 4340, 11.3 - "New connections start with Ack Ratio 2 for both endpoints"; * what happens then is part of another patch set, since it concerns the dynamic update of Ack Ratio while the connection is in full flight. Thanks to Tomasz Grobelny for discussion leading up to this patch. Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk> Acked-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'include')
-rw-r--r--include/linux/dccp.h2
1 files changed, 0 insertions, 2 deletions
diff --git a/include/linux/dccp.h b/include/linux/dccp.h
index 5a5a89935dbc..eda389ce04f4 100644
--- a/include/linux/dccp.h
+++ b/include/linux/dccp.h
@@ -368,7 +368,6 @@ static inline unsigned int dccp_hdr_len(const struct sk_buff *skb)
368 * @dccpms_ccid - Congestion Control Id (CCID) (section 10) 368 * @dccpms_ccid - Congestion Control Id (CCID) (section 10)
369 * @dccpms_send_ack_vector - Send Ack Vector Feature (section 11.5) 369 * @dccpms_send_ack_vector - Send Ack Vector Feature (section 11.5)
370 * @dccpms_send_ndp_count - Send NDP Count Feature (7.7.2) 370 * @dccpms_send_ndp_count - Send NDP Count Feature (7.7.2)
371 * @dccpms_ack_ratio - Ack Ratio Feature (section 11.3)
372 * @dccpms_pending - List of features being negotiated 371 * @dccpms_pending - List of features being negotiated
373 * @dccpms_conf - 372 * @dccpms_conf -
374 */ 373 */
@@ -378,7 +377,6 @@ struct dccp_minisock {
378 __u8 dccpms_tx_ccid; 377 __u8 dccpms_tx_ccid;
379 __u8 dccpms_send_ack_vector; 378 __u8 dccpms_send_ack_vector;
380 __u8 dccpms_send_ndp_count; 379 __u8 dccpms_send_ndp_count;
381 __u8 dccpms_ack_ratio;
382 struct list_head dccpms_pending; 380 struct list_head dccpms_pending;
383 struct list_head dccpms_conf; 381 struct list_head dccpms_conf;
384}; 382};