* [PATCHv2 4/8] Fix name schema for tx-rx sock struct
@ 2007-12-21 4:33 Leandro
2007-12-21 9:53 ` Gerrit Renker
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Leandro @ 2007-12-21 4:33 UTC (permalink / raw)
To: dccp
[DCCP][TFRC_CCIDS] Fix name schema for tx-rx sock struct
Besides the dccp/ccids directory files, this patch also fixes the name schema for include/linux/tfrc.h.
Signed-off-by: Leandro Melo de Sales <leandro@embedded.ufcg.edu.br>
--------------------> Patch v2 <-------------------------
The first version of this patch doest apply properly in the current ccid4 branch. This new version it is ok.
Index: ccid4.latest/include/linux/tfrc.h
=================================--- /dev/null
+++ ccid4.latest/include/linux/tfrc.h
@@ -0,0 +1,55 @@
+#ifndef _LINUX_TFRC_H_
+#define _LINUX_TFRC_H_
+/*
+ * TFRC - Data Structures for the TCP-Friendly Rate Control congestion
+ * control mechanism as specified in RFC 3448.
+ *
+ * Copyright (c) 2005 The University of Waikato, Hamilton, New Zealand.
+ * Copyright (c) 2005 Ian McDonald <iam4@cs.waikato.ac.nz>
+ * Copyright (c) 2005 Arnaldo Carvalho de Melo <acme@conectiva.com.br>
+ * Copyright (c) 2003 Nils-Erik Mattsson, Joacim Haggmark, Magnus Erixzon
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
+#include <linux/types.h>
+
+/** tfrc_rx_info - TFRC Receiver Data Structure
+ *
+ * @x_recv: receiver estimate of sending rate (3.2.2)
+ * @rtt: round-trip-time (communicated by sender)
+ * @p: current estimate of loss event rate (3.2.2)
+ */
+struct tfrc_rx_info {
+ __u32 x_recv;
+ __u32 rtt;
+ __u32 p;
+};
+
+/** tfrc_tx_info - TFRC Sender Data Structure
+ *
+ * @x_x: computed transmit rate (4.3 (4))
+ * @x_recv: receiver estimate of send rate (4.3)
+ * @x_calc: return value of throughput equation (3.1)
+ * @rtt: (moving average) estimate of RTT (4.3)
+ * @p: current loss event rate (5.4)
+ * @rto: estimate of RTO, equals 4*RTT (4.3)
+ * @ipi: inter-packet interval (4.6)
+ *
+ * Note: X and X_recv are both maintained in units of 64 * bytes/second. This
+ * enables a finer resolution of sending rates and avoids problems with
+ * integer arithmetic; u32 is not sufficient as scaling consumes 6 bits.
+ */
+struct tfrc_tx_info {
+ __u64 x;
+ __u64 x_recv;
+ __u32 x_calc;
+ __u32 rtt;
+ __u32 p;
+ __u32 rto;
+ __u32 ipi;
+};
+
+#endif /* _LINUX_TFRC_H_ */
Index: ccid4.latest/net/dccp/ccids/ccid3.c
=================================--- ccid4.latest.orig/net/dccp/ccids/ccid3.c
+++ ccid4.latest/net/dccp/ccids/ccid3.c
@@ -48,13 +48,13 @@ static void ccid3_hc_tx_set_state(struct
enum tfrc_hc_tx_states state)
{
struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
- enum tfrc_hc_tx_states oldstate = hctx->tfrchctx_state;
+ enum tfrc_hc_tx_states oldstate = hctx->ttx_state;
ccid3_pr_debug("%s(%p) %-8.8s -> %s\n",
dccp_role(sk), sk, tfrc_tx_state_name(oldstate),
tfrc_tx_state_name(state));
WARN_ON(state = oldstate);
- hctx->tfrchctx_state = state;
+ hctx->ttx_state = state;
}
/*
@@ -63,16 +63,15 @@ static void ccid3_hc_tx_set_state(struct
static inline void ccid3_update_send_interval(struct tfrc_hc_tx_sock *hctx)
{
/* Calculate new t_ipi = s / X_inst (X_inst is in 64 * bytes/second) */
- hctx->tfrchctx_t_ipi = scaled_div32(((u64)hctx->tfrchctx_s) << 6,
- hctx->tfrchctx_x);
+ hctx->ttx_t_ipi = scaled_div32(((u64)hctx->ttx_s) << 6, hctx->ttx_x);
/* Calculate new delta by delta = min(t_ipi / 2, t_gran / 2) */
- hctx->tfrchctx_delta = min_t(u32, hctx->tfrchctx_t_ipi / 2,
+ hctx->ttx_delta = min_t(u32, hctx->ttx_t_ipi / 2,
TFRC_OPSYS_HALF_TIME_GRAN);
ccid3_pr_debug("t_ipi=%u, delta=%u, s=%u, X=%u\n",
- hctx->tfrchctx_t_ipi, hctx->tfrchctx_delta,
- hctx->tfrchctx_s, (unsigned)(hctx->tfrchctx_x >> 6));
+ hctx->ttx_t_ipi, hctx->ttx_delta,
+ hctx->ttx_s, (unsigned)(hctx->ttx_x >> 6));
}
@@ -89,8 +88,8 @@ static inline void ccid3_update_send_int
static void ccid3_hc_tx_update_x(struct sock *sk, ktime_t *stamp)
{
struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
- __u64 min_rate = 2 * hctx->tfrchctx_x_recv;
- const __u64 old_x = hctx->tfrchctx_x;
+ __u64 min_rate = 2 * hctx->ttx_x_recv;
+ const __u64 old_x = hctx->ttx_x;
ktime_t now = stamp ? *stamp : ktime_get_real();
/*
@@ -101,33 +100,29 @@ static void ccid3_hc_tx_update_x(struct
*/
if (tfrc_hc_tx_idle_rtt(hctx, now) >= 2) {
min_rate = rfc3390_initial_rate(sk);
- min_rate = max(min_rate, 2 * hctx->tfrchctx_x_recv);
+ min_rate = max(min_rate, 2 * hctx->ttx_x_recv);
}
- if (hctx->tfrchctx_p > 0) {
+ if (hctx->ttx_p > 0) {
- hctx->tfrchctx_x = min(((__u64)hctx->tfrchctx_x_calc) << 6,
- min_rate);
- hctx->tfrchctx_x = max(hctx->tfrchctx_x,
- (((__u64)hctx->tfrchctx_s) << 6) /
- TFRC_T_MBI);
+ hctx->ttx_x = min(((__u64)hctx->ttx_x_calc) << 6, min_rate);
+ hctx->ttx_x = max(hctx->ttx_x,
+ (((__u64)hctx->ttx_s) << 6) / TFRC_T_MBI);
- } else if (ktime_us_delta(now, hctx->tfrchctx_t_ld)
- - (s64)hctx->tfrchctx_rtt >= 0) {
+ } else if (ktime_us_delta(now, hctx->ttx_t_ld)
+ - (s64)hctx->ttx_rtt >= 0) {
- hctx->tfrchctx_x - max(min(2 * hctx->tfrchctx_x, min_rate),
- scaled_div(((__u64)hctx->tfrchctx_s) << 6,
- hctx->tfrchctx_rtt));
- hctx->tfrchctx_t_ld = now;
+ hctx->ttx_x = max(min(2 * hctx->ttx_x, min_rate),
+ scaled_div(((__u64)hctx->ttx_s) << 6,
+ hctx->ttx_rtt));
+ hctx->ttx_t_ld = now;
}
- if (hctx->tfrchctx_x != old_x) {
+ if (hctx->ttx_x != old_x) {
ccid3_pr_debug("X_prev=%u, X_now=%u, X_calc=%u, "
"X_recv=%u\n", (unsigned)(old_x >> 6),
- (unsigned)(hctx->tfrchctx_x >> 6),
- hctx->tfrchctx_x_calc,
- (unsigned)(hctx->tfrchctx_x_recv >> 6));
+ (unsigned)(hctx->ttx_x >> 6), hctx->ttx_x_calc,
+ (unsigned)(hctx->ttx_x_recv >> 6));
ccid3_update_send_interval(hctx);
}
@@ -139,11 +134,11 @@ static void ccid3_hc_tx_update_x(struct
*/
static inline void ccid3_hc_tx_update_s(struct tfrc_hc_tx_sock *hctx, int len)
{
- const u16 old_s = hctx->tfrchctx_s;
+ const u16 old_s = hctx->ttx_s;
- hctx->tfrchctx_s = tfrc_ewma(hctx->tfrchctx_s, len, 9);
+ hctx->ttx_s = tfrc_ewma(hctx->ttx_s, len, 9);
- if (hctx->tfrchctx_s != old_s)
+ if (hctx->ttx_s != old_s)
ccid3_update_send_interval(hctx);
}
@@ -161,22 +156,22 @@ static void ccid3_hc_tx_no_feedback_time
}
ccid3_pr_debug("%s(%p, state=%s) - entry \n", dccp_role(sk), sk,
- tfrc_tx_state_name(hctx->tfrchctx_state));
+ tfrc_tx_state_name(hctx->ttx_state));
- if (hctx->tfrchctx_state = TTX_STATE_FBACK)
+ if (hctx->ttx_state = TTX_STATE_FBACK)
ccid3_hc_tx_set_state(sk, TTX_STATE_NO_FBACK);
- else if (hctx->tfrchctx_state != TTX_STATE_NO_FBACK)
+ else if (hctx->ttx_state != TTX_STATE_NO_FBACK)
goto out;
/*
* Determine new allowed sending rate X as per draft rfc3448bis-00, 4.4
*/
- if (hctx->tfrchctx_t_rto = 0 || /* no feedback received yet */
- hctx->tfrchctx_p = 0) {
+ if (hctx->ttx_t_rto = 0 || /* no feedback received yet */
+ hctx->ttx_p = 0) {
/* halve send rate directly */
- hctx->tfrchctx_x = max(hctx->tfrchctx_x / 2,
- (((__u64)hctx->tfrchctx_s) << 6) /
+ hctx->ttx_x = max(hctx->ttx_x / 2,
+ (((__u64)hctx->ttx_s) << 6) /
TFRC_T_MBI);
ccid3_update_send_interval(hctx);
} else {
@@ -190,33 +185,32 @@ static void ccid3_hc_tx_no_feedback_time
*
* Note that X_recv is scaled by 2^6 while X_calc is not
*/
- BUG_ON(hctx->tfrchctx_p && !hctx->tfrchctx_x_calc);
+ BUG_ON(hctx->ttx_p && !hctx->ttx_x_calc);
- if (hctx->tfrchctx_x_calc > (hctx->tfrchctx_x_recv >> 5))
- hctx->tfrchctx_x_recv - max(hctx->tfrchctx_x_recv / 2,
- (((__u64)hctx->tfrchctx_s) << 6) /
+ if (hctx->ttx_x_calc > (hctx->ttx_x_recv >> 5))
+ hctx->ttx_x_recv = max(hctx->ttx_x_recv / 2,
+ (((__u64)hctx->ttx_s) << 6) /
(2 * TFRC_T_MBI));
else {
- hctx->tfrchctx_x_recv = hctx->tfrchctx_x_calc;
- hctx->tfrchctx_x_recv <<= 4;
+ hctx->ttx_x_recv = hctx->ttx_x_calc;
+ hctx->ttx_x_recv <<= 4;
}
ccid3_hc_tx_update_x(sk, NULL);
}
ccid3_pr_debug("Reduced X to %llu/64 bytes/sec\n",
- (unsigned long long)hctx->tfrchctx_x);
+ (unsigned long long)hctx->ttx_x);
/*
* Set new timeout for the nofeedback timer.
* See comments in packet_recv() regarding the value of t_RTO.
*/
- if (unlikely(hctx->tfrchctx_t_rto = 0)) /* no feedback yet */
+ if (unlikely(hctx->ttx_t_rto = 0)) /* no feedback yet */
t_nfb = TFRC_INITIAL_TIMEOUT;
else
- t_nfb = max(hctx->tfrchctx_t_rto, 2 * hctx->tfrchctx_t_ipi);
+ t_nfb = max(hctx->ttx_t_rto, 2 * hctx->ttx_t_ipi);
restart_timer:
- sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
+ sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
jiffies + usecs_to_jiffies(t_nfb));
out:
bh_unlock_sock(sk);
@@ -244,18 +238,18 @@ static int ccid3_hc_tx_send_packet(struc
if (unlikely(skb->len = 0))
return -EBADMSG;
- switch (hctx->tfrchctx_state) {
+ switch (hctx->ttx_state) {
case TTX_STATE_NO_SENT:
- sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
+ sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
(jiffies +
usecs_to_jiffies(TFRC_INITIAL_TIMEOUT)));
- hctx->tfrchctx_last_win_count = 0;
- hctx->tfrchctx_t_last_win_count = now;
+ hctx->ttx_last_win_count = 0;
+ hctx->ttx_t_last_win_count = now;
/* Set t_0 for initial packet */
- hctx->tfrchctx_t_nom = now;
+ hctx->ttx_t_nom = now;
- hctx->tfrchctx_s = skb->len;
+ hctx->ttx_s = skb->len;
/*
* Use initial RTT sample when available: recommended by erratum
@@ -264,13 +258,13 @@ static int ccid3_hc_tx_send_packet(struc
*/
if (dp->dccps_syn_rtt) {
ccid3_pr_debug("SYN RTT = %uus\n", dp->dccps_syn_rtt);
- hctx->tfrchctx_rtt = dp->dccps_syn_rtt;
- hctx->tfrchctx_x = rfc3390_initial_rate(sk);
- hctx->tfrchctx_t_ld = now;
+ hctx->ttx_rtt = dp->dccps_syn_rtt;
+ hctx->ttx_x = rfc3390_initial_rate(sk);
+ hctx->ttx_t_ld = now;
} else {
/* Sender does not have RTT sample: X_pps = 1 pkt/sec */
- hctx->tfrchctx_x = hctx->tfrchctx_s;
- hctx->tfrchctx_x <<= 6;
+ hctx->ttx_x = hctx->ttx_s;
+ hctx->ttx_x <<= 6;
}
ccid3_update_send_interval(hctx);
@@ -278,7 +272,7 @@ static int ccid3_hc_tx_send_packet(struc
break;
case TTX_STATE_NO_FBACK:
case TTX_STATE_FBACK:
- delay = ktime_us_delta(hctx->tfrchctx_t_nom, now);
+ delay = ktime_us_delta(hctx->ttx_t_nom, now);
ccid3_pr_debug("delay=%ld\n", (long)delay);
/*
* Scheduling of packet transmissions [RFC 3448, 4.6]
@@ -288,7 +282,7 @@ static int ccid3_hc_tx_send_packet(struc
* else
* // send the packet in (t_nom - t_now) milliseconds.
*/
- if (delay - (s64)hctx->tfrchctx_delta >= 1000)
+ if (delay - (s64)hctx->ttx_delta >= 1000)
return (u32)delay / 1000L;
tfrc_hc_tx_update_win_count(hctx, now);
@@ -300,11 +294,10 @@ static int ccid3_hc_tx_send_packet(struc
/* prepare to send now (add options etc.) */
dp->dccps_hc_tx_insert_options = 1;
- DCCP_SKB_CB(skb)->dccpd_ccval = hctx->tfrchctx_last_win_count;
+ DCCP_SKB_CB(skb)->dccpd_ccval = hctx->ttx_last_win_count;
/* set the nominal send time for the next following packet */
- hctx->tfrchctx_t_nom = ktime_add_us(hctx->tfrchctx_t_nom,
- hctx->tfrchctx_t_ipi);
+ hctx->ttx_t_nom = ktime_add_us(hctx->ttx_t_nom, hctx->ttx_t_ipi);
return 0;
}
@@ -315,7 +308,7 @@ static void ccid3_hc_tx_packet_sent(stru
ccid3_hc_tx_update_s(hctx, len);
- if (tfrc_tx_hist_add(&hctx->tfrchctx_hist, dccp_sk(sk)->dccps_gss))
+ if (tfrc_tx_hist_add(&hctx->ttx_hist, dccp_sk(sk)->dccps_gss))
DCCP_CRIT("packet history - out of memory!");
}
@@ -332,15 +325,15 @@ static void ccid3_hc_tx_packet_recv(stru
DCCP_SKB_CB(skb)->dccpd_type = DCCP_PKT_DATAACK))
return;
/* ... and only in the established state */
- if (hctx->tfrchctx_state != TTX_STATE_FBACK &&
- hctx->tfrchctx_state != TTX_STATE_NO_FBACK)
+ if (hctx->ttx_state != TTX_STATE_FBACK &&
+ hctx->ttx_state != TTX_STATE_NO_FBACK)
return;
- opt_recv = &hctx->tfrchctx_options_received;
+ opt_recv = &hctx->ttx_options_received;
now = ktime_get_real();
/* Estimate RTT from history if ACK number is valid */
- r_sample = tfrc_tx_hist_rtt(hctx->tfrchctx_hist,
+ r_sample = tfrc_tx_hist_rtt(hctx->ttx_hist,
DCCP_SKB_CB(skb)->dccpd_ack_seq, now);
if (r_sample = 0) {
DCCP_WARN("%s(%p): %s with bogus ACK-%llu\n", dccp_role(sk), sk,
@@ -350,37 +343,37 @@ static void ccid3_hc_tx_packet_recv(stru
}
/* Update receive rate in units of 64 * bytes/second */
- hctx->tfrchctx_x_recv = opt_recv->tfrcor_receive_rate;
- hctx->tfrchctx_x_recv <<= 6;
+ hctx->ttx_x_recv = opt_recv->tfrcor_receive_rate;
+ hctx->ttx_x_recv <<= 6;
/* Update loss event rate (which is scaled by 1e6) */
pinv = opt_recv->tfrcor_loss_event_rate;
if (pinv = ~0U || pinv = 0) /* see RFC 4342, 8.5 */
- hctx->tfrchctx_p = 0;
+ hctx->ttx_p = 0;
else /* can not exceed 100% */
- hctx->tfrchctx_p = scaled_div(1, pinv);
+ hctx->ttx_p = scaled_div(1, pinv);
/*
* Validate new RTT sample and update moving average
*/
r_sample = dccp_sample_rtt(sk, r_sample);
- hctx->tfrchctx_rtt = tfrc_ewma(hctx->tfrchctx_rtt, r_sample, 9);
+ hctx->ttx_rtt = tfrc_ewma(hctx->ttx_rtt, r_sample, 9);
/*
* Update allowed sending rate X as per draft rfc3448bis-00, 4.2/3
*/
- if (hctx->tfrchctx_state = TTX_STATE_NO_FBACK) {
+ if (hctx->ttx_state = TTX_STATE_NO_FBACK) {
ccid3_hc_tx_set_state(sk, TTX_STATE_FBACK);
- if (hctx->tfrchctx_t_rto = 0) {
+ if (hctx->ttx_t_rto = 0) {
/*
* Initial feedback packet: Larger Initial Windows (4.2)
*/
- hctx->tfrchctx_x = rfc3390_initial_rate(sk);
- hctx->tfrchctx_t_ld = now;
+ hctx->ttx_x = rfc3390_initial_rate(sk);
+ hctx->ttx_t_ld = now;
ccid3_update_send_interval(hctx);
goto done_computing_x;
- } else if (hctx->tfrchctx_p = 0) {
+ } else if (hctx->ttx_p = 0) {
/*
* First feedback after nofeedback timer expiry (4.3)
*/
@@ -389,25 +382,23 @@ static void ccid3_hc_tx_packet_recv(stru
}
/* Update sending rate (step 4 of [RFC 3448, 4.3]) */
- if (hctx->tfrchctx_p > 0)
- hctx->tfrchctx_x_calc - tfrc_calc_x(hctx->tfrchctx_s,
- hctx->tfrchctx_rtt,
- hctx->tfrchctx_p);
+ if (hctx->ttx_p > 0)
+ hctx->ttx_x_calc = tfrc_calc_x(hctx->ttx_s, hctx->ttx_rtt,
+ hctx->ttx_p);
ccid3_hc_tx_update_x(sk, &now);
done_computing_x:
ccid3_pr_debug("%s(%p), RTT=%uus (sample=%uus), s=%u, "
"p=%u, X_calc=%u, X_recv=%u, X=%u\n",
dccp_role(sk),
- sk, hctx->tfrchctx_rtt, r_sample,
- hctx->tfrchctx_s, hctx->tfrchctx_p,
- hctx->tfrchctx_x_calc,
- (unsigned)(hctx->tfrchctx_x_recv >> 6),
- (unsigned)(hctx->tfrchctx_x >> 6));
+ sk, hctx->ttx_rtt, r_sample,
+ hctx->ttx_s, hctx->ttx_p,
+ hctx->ttx_x_calc,
+ (unsigned)(hctx->ttx_x_recv >> 6),
+ (unsigned)(hctx->ttx_x >> 6));
/* unschedule no feedback timer */
- sk_stop_timer(sk, &hctx->tfrchctx_no_feedback_timer);
+ sk_stop_timer(sk, &hctx->ttx_no_feedback_timer);
/*
* As we have calculated new ipi, delta, t_nom it is possible
@@ -421,21 +412,21 @@ done_computing_x:
* This can help avoid triggering the nofeedback timer too
* often ('spinning') on LANs with small RTTs.
*/
- hctx->tfrchctx_t_rto = max_t(u32, 4 * hctx->tfrchctx_rtt,
+ hctx->ttx_t_rto = max_t(u32, 4 * hctx->ttx_rtt,
(CONFIG_IP_DCCP_CCID3_RTO *
(USEC_PER_SEC / 1000)));
/*
* Schedule no feedback timer to expire in
* max(t_RTO, 2 * s/X) = max(t_RTO, 2 * t_ipi)
*/
- t_nfb = max(hctx->tfrchctx_t_rto, 2 * hctx->tfrchctx_t_ipi);
+ t_nfb = max(hctx->ttx_t_rto, 2 * hctx->ttx_t_ipi);
ccid3_pr_debug("%s(%p), Scheduled no feedback timer to "
"expire in %lu jiffies (%luus)\n",
dccp_role(sk),
sk, usecs_to_jiffies(t_nfb), t_nfb);
- sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
+ sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
jiffies + usecs_to_jiffies(t_nfb));
}
@@ -449,7 +440,7 @@ static int ccid3_hc_tx_parse_options(str
struct tfrc_options_received *opt_recv;
__be32 opt_val;
- opt_recv = &hctx->tfrchctx_options_received;
+ opt_recv = &hctx->ttx_options_received;
if (opt_recv->tfrcor_seqno != dp->dccps_gsr) {
opt_recv->tfrcor_seqno = dp->dccps_gsr;
@@ -505,10 +496,10 @@ static int ccid3_hc_tx_init(struct ccid
{
struct tfrc_hc_tx_sock *hctx = ccid_priv(ccid);
- hctx->tfrchctx_state = TTX_STATE_NO_SENT;
- hctx->tfrchctx_hist = NULL;
- setup_timer(&hctx->tfrchctx_no_feedback_timer,
- ccid3_hc_tx_no_feedback_timer, (unsigned long)sk);
+ hctx->ttx_state = TTX_STATE_NO_SENT;
+ hctx->ttx_hist = NULL;
+ setup_timer(&hctx->ttx_no_feedback_timer, ccid3_hc_tx_no_feedback_timer,
+ (unsigned long)sk);
return 0;
}
@@ -518,15 +509,15 @@ static void ccid3_hc_tx_exit(struct sock
struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
ccid3_hc_tx_set_state(sk, TTX_STATE_TERM);
- sk_stop_timer(sk, &hctx->tfrchctx_no_feedback_timer);
+ sk_stop_timer(sk, &hctx->ttx_no_feedback_timer);
- tfrc_tx_hist_purge(&hctx->tfrchctx_hist);
+ tfrc_tx_hist_purge(&hctx->ttx_hist);
}
static void ccid3_hc_tx_get_info(struct sock *sk, struct tcp_info *info)
{
- info->tcpi_rto = tfrc_hc_tx_sk(sk)->tfrchctx_t_rto;
- info->tcpi_rtt = tfrc_hc_tx_sk(sk)->tfrchctx_rtt;
+ info->tcpi_rto = tfrc_hc_tx_sk(sk)->ttx_t_rto;
+ info->tcpi_rtt = tfrc_hc_tx_sk(sk)->ttx_rtt;
}
static int ccid3_hc_tx_getsockopt(struct sock *sk, const int optname, int len,
@@ -537,10 +528,10 @@ static int ccid3_hc_tx_getsockopt(struct
switch (optname) {
case DCCP_SOCKOPT_CCID_TX_INFO:
- if (len < sizeof(hctx->tfrchctx_tfrc))
+ if (len < sizeof(hctx->ttx_tfrc))
return -EINVAL;
- len = sizeof(hctx->tfrchctx_tfrc);
- val = &hctx->tfrchctx_tfrc;
+ len = sizeof(hctx->ttx_tfrc);
+ val = &hctx->ttx_tfrc;
break;
default:
return -ENOPROTOOPT;
@@ -556,13 +547,13 @@ static void ccid3_hc_rx_set_state(struct
enum tfrc_hc_rx_states state)
{
struct tfrc_hc_rx_sock *hcrx = tfrc_hc_rx_sk(sk);
- enum tfrc_hc_rx_states oldstate = hcrx->tfrchcrx_state;
+ enum tfrc_hc_rx_states oldstate = hcrx->trx_state;
ccid3_pr_debug("%s(%p) %-8.8s -> %s\n",
dccp_role(sk), sk, tfrc_rx_state_name(oldstate),
tfrc_rx_state_name(state));
WARN_ON(state = oldstate);
- hcrx->tfrchcrx_state = state;
+ hcrx->trx_state = state;
}
static void ccid3_hc_rx_send_feedback(struct sock *sk,
@@ -574,15 +565,15 @@ static void ccid3_hc_rx_send_feedback(st
ktime_t now;
s64 delta = 0;
- if (unlikely(hcrx->tfrchcrx_state = TRX_STATE_TERM))
+ if (unlikely(hcrx->trx_state = TRX_STATE_TERM))
return;
now = ktime_get_real();
switch (fbtype) {
case TFRC_FBACK_INITIAL:
- hcrx->tfrchcrx_x_recv = 0;
- hcrx->tfrchcrx_pinv = ~0U; /* see RFC 4342, 8.5 */
+ hcrx->trx_x_recv = 0;
+ hcrx->trx_pinv = ~0U; /* see RFC 4342, 8.5 */
break;
case TFRC_FBACK_PARAM_CHANGE:
/*
@@ -595,27 +586,27 @@ static void ccid3_hc_rx_send_feedback(st
* the number of bytes since last feedback.
* This is a safe fallback, since X is bounded above by X_calc.
*/
- if (hcrx->tfrchcrx_x_recv > 0)
+ if (hcrx->trx_x_recv > 0)
break;
/* fall through */
case TFRC_FBACK_PERIODIC:
- delta = ktime_us_delta(now, hcrx->tfrchcrx_last_feedback);
+ delta = ktime_us_delta(now, hcrx->trx_last_feedback);
if (delta <= 0)
DCCP_BUG("delta (%ld) <= 0", (long)delta);
else
- hcrx->tfrchcrx_x_recv - scaled_div32(hcrx->tfrchcrx_bytes_recv, delta);
+ hcrx->trx_x_recv + scaled_div32(hcrx->trx_bytes_recv, delta);
break;
default:
return;
}
ccid3_pr_debug("Interval %ldusec, X_recv=%u, 1/p=%u\n", (long)delta,
- hcrx->tfrchcrx_x_recv, hcrx->tfrchcrx_pinv);
+ hcrx->trx_x_recv, hcrx->trx_pinv);
- hcrx->tfrchcrx_last_feedback = now;
- hcrx->tfrchcrx_last_counter = dccp_hdr(skb)->dccph_ccval;
- hcrx->tfrchcrx_bytes_recv = 0;
+ hcrx->trx_last_feedback = now;
+ hcrx->trx_last_counter = dccp_hdr(skb)->dccph_ccval;
+ hcrx->trx_bytes_recv = 0;
dp->dccps_hc_rx_insert_options = 1;
dccp_send_ack(sk);
@@ -632,8 +623,8 @@ static int ccid3_hc_rx_insert_options(st
if (dccp_packet_without_ack(skb))
return 0;
- x_recv = htonl(hcrx->tfrchcrx_x_recv);
- pinv = htonl(hcrx->tfrchcrx_pinv);
+ x_recv = htonl(hcrx->trx_x_recv);
+ pinv = htonl(hcrx->trx_pinv);
if (dccp_insert_option(sk, skb, TFRC_OPT_LOSS_EVENT_RATE,
&pinv, sizeof(pinv)) ||
@@ -659,22 +650,22 @@ static u32 ccid3_first_li(struct sock *s
u32 x_recv, p, delta;
u64 fval;
- if (hcrx->tfrchcrx_rtt = 0) {
+ if (hcrx->trx_rtt = 0) {
DCCP_WARN("No RTT estimate available, using fallback RTT\n");
- hcrx->tfrchcrx_rtt = DCCP_FALLBACK_RTT;
+ hcrx->trx_rtt = DCCP_FALLBACK_RTT;
}
- delta = ktime_to_us(net_timedelta(hcrx->tfrchcrx_last_feedback));
- x_recv = scaled_div32(hcrx->tfrchcrx_bytes_recv, delta);
+ delta = ktime_to_us(net_timedelta(hcrx->trx_last_feedback));
+ x_recv = scaled_div32(hcrx->trx_bytes_recv, delta);
if (x_recv = 0) { /* would also trigger divide-by-zero */
DCCP_WARN("X_recv=0\n");
- if ((x_recv = hcrx->tfrchcrx_x_recv) = 0) {
+ if ((x_recv = hcrx->trx_x_recv) = 0) {
DCCP_BUG("stored value of X_recv is zero");
return ~0U;
}
}
- fval = scaled_div(hcrx->tfrchcrx_s, hcrx->tfrchcrx_rtt);
+ fval = scaled_div(hcrx->trx_s, hcrx->trx_rtt);
fval = scaled_div32(fval, x_recv);
p = tfrc_calc_x_reverse_lookup(fval);
@@ -691,14 +682,14 @@ static void ccid3_hc_rx_packet_recv(stru
const u64 ndp = dccp_sk(sk)->dccps_options_received.dccpor_ndp;
const bool is_data_packet = dccp_data_packet(skb);
- if (unlikely(hcrx->tfrchcrx_state = TRX_STATE_NO_DATA)) {
+ if (unlikely(hcrx->trx_state = TRX_STATE_NO_DATA)) {
if (is_data_packet) {
const u32 payload = skb->len - dccp_hdr(skb)->dccph_doff * 4;
do_feedback = TFRC_FBACK_INITIAL;
ccid3_hc_rx_set_state(sk, TRX_STATE_DATA);
- hcrx->tfrchcrx_s = payload;
+ hcrx->trx_s = payload;
/*
- * Not necessary to update tfrchcrx_bytes_recv here,
+ * Not necessary to update trx_bytes_recv here,
* since X_recv = 0 for the first feedback packet (cf.
* RFC 3448, 6.3) -- gerrit
*/
@@ -706,7 +697,7 @@ static void ccid3_hc_rx_packet_recv(stru
goto update_records;
}
- if (tfrc_rx_hist_duplicate(&hcrx->tfrchcrx_hist, skb))
+ if (tfrc_rx_hist_duplicate(&hcrx->trx_hist, skb))
return; /* done receiving */
if (is_data_packet) {
@@ -714,22 +705,21 @@ static void ccid3_hc_rx_packet_recv(stru
/*
* Update moving-average of s and the sum of received payload bytes
*/
- hcrx->tfrchcrx_s = tfrc_ewma(hcrx->tfrchcrx_s, payload, 9);
- hcrx->tfrchcrx_bytes_recv += payload;
+ hcrx->trx_s = tfrc_ewma(hcrx->trx_s, payload, 9);
+ hcrx->trx_bytes_recv += payload;
}
/*
* Handle pending losses and otherwise check for new loss
*/
- if (tfrc_rx_hist_loss_pending(&hcrx->tfrchcrx_hist) &&
- tfrc_rx_handle_loss(&hcrx->tfrchcrx_hist,
- &hcrx->tfrchcrx_li_hist,
+ if (tfrc_rx_hist_loss_pending(&hcrx->trx_hist) &&
+ tfrc_rx_handle_loss(&hcrx->trx_hist, &hcrx->trx_li_hist,
skb, ndp, ccid3_first_li, sk) ) {
do_feedback = TFRC_FBACK_PARAM_CHANGE;
goto done_receiving;
}
- if (tfrc_rx_hist_new_loss_indicated(&hcrx->tfrchcrx_hist, skb, ndp))
+ if (tfrc_rx_hist_new_loss_indicated(&hcrx->trx_hist, skb, ndp))
goto update_records;
/*
@@ -738,17 +728,17 @@ static void ccid3_hc_rx_packet_recv(stru
if (unlikely(!is_data_packet))
goto update_records;
- if (!tfrc_lh_is_initialised(&hcrx->tfrchcrx_li_hist)) {
- const u32 sample = tfrc_rx_hist_sample_rtt(&hcrx->tfrchcrx_hist, skb);
+ if (!tfrc_lh_is_initialised(&hcrx->trx_li_hist)) {
+ const u32 sample = tfrc_rx_hist_sample_rtt(&hcrx->trx_hist, skb);
/*
* Empty loss history: no loss so far, hence p stays 0.
* Sample RTT values, since an RTT estimate is required for the
* computation of p when the first loss occurs; RFC 3448, 6.3.1.
*/
if (sample != 0)
- hcrx->tfrchcrx_rtt = tfrc_ewma(hcrx->tfrchcrx_rtt, sample, 9);
+ hcrx->trx_rtt = tfrc_ewma(hcrx->trx_rtt, sample, 9);
- } else if (tfrc_lh_update_i_mean(&hcrx->tfrchcrx_li_hist, skb)) {
+ } else if (tfrc_lh_update_i_mean(&hcrx->trx_li_hist, skb)) {
/*
* Step (3) of [RFC 3448, 6.1]: Recompute I_mean and, if I_mean
* has decreased (resp. p has increased), send feedback now.
@@ -759,11 +749,11 @@ static void ccid3_hc_rx_packet_recv(stru
/*
* Check if the periodic once-per-RTT feedback is due; RFC 4342, 10.3
*/
- if (SUB16(dccp_hdr(skb)->dccph_ccval, hcrx->tfrchcrx_last_counter) > 3)
+ if (SUB16(dccp_hdr(skb)->dccph_ccval, hcrx->trx_last_counter) > 3)
do_feedback = TFRC_FBACK_PERIODIC;
update_records:
- tfrc_rx_hist_add_packet(&hcrx->tfrchcrx_hist, skb, ndp);
+ tfrc_rx_hist_add_packet(&hcrx->trx_hist, skb, ndp);
done_receiving:
if (do_feedback)
@@ -774,9 +764,9 @@ static int ccid3_hc_rx_init(struct ccid
{
struct tfrc_hc_rx_sock *hcrx = ccid_priv(ccid);
- hcrx->tfrchcrx_state = TRX_STATE_NO_DATA;
- tfrc_lh_init(&hcrx->tfrchcrx_li_hist);
- return tfrc_rx_hist_alloc(&hcrx->tfrchcrx_hist);
+ hcrx->trx_state = TRX_STATE_NO_DATA;
+ tfrc_lh_init(&hcrx->trx_li_hist);
+ return tfrc_rx_hist_alloc(&hcrx->trx_hist);
}
static void ccid3_hc_rx_exit(struct sock *sk)
@@ -785,15 +775,15 @@ static void ccid3_hc_rx_exit(struct sock
ccid3_hc_rx_set_state(sk, TRX_STATE_TERM);
- tfrc_rx_hist_purge(&hcrx->tfrchcrx_hist);
- tfrc_lh_cleanup(&hcrx->tfrchcrx_li_hist);
+ tfrc_rx_hist_purge(&hcrx->trx_hist);
+ tfrc_lh_cleanup(&hcrx->trx_li_hist);
}
static void ccid3_hc_rx_get_info(struct sock *sk, struct tcp_info *info)
{
- info->tcpi_ca_state = tfrc_hc_rx_sk(sk)->tfrchcrx_state;
+ info->tcpi_ca_state = tfrc_hc_rx_sk(sk)->trx_state;
info->tcpi_options |= TCPI_OPT_TIMESTAMPS;
- info->tcpi_rcv_rtt = tfrc_hc_rx_sk(sk)->tfrchcrx_rtt;
+ info->tcpi_rcv_rtt = tfrc_hc_rx_sk(sk)->trx_rtt;
}
static int ccid3_hc_rx_getsockopt(struct sock *sk, const int optname, int len,
@@ -807,10 +797,10 @@ static int ccid3_hc_rx_getsockopt(struct
case DCCP_SOCKOPT_CCID_RX_INFO:
if (len < sizeof(rx_info))
return -EINVAL;
- rx_info.tfrcrx_x_recv = hcrx->tfrchcrx_x_recv;
- rx_info.tfrcrx_rtt = hcrx->tfrchcrx_rtt;
- rx_info.tfrcrx_p = hcrx->tfrchcrx_pinv = 0 ? ~0U :
- scaled_div(1, hcrx->tfrchcrx_pinv);
+ rx_info.tfrcrx_x_recv = hcrx->trx_x_recv;
+ rx_info.tfrcrx_rtt = hcrx->trx_rtt;
+ rx_info.tfrcrx_p = hcrx->trx_pinv = 0 ? ~0U :
+ scaled_div(1, hcrx->trx_pinv);
len = sizeof(rx_info);
val = &rx_info;
break;
Index: ccid4.latest/net/dccp/ccids/ccid4.c
=================================--- ccid4.latest.orig/net/dccp/ccids/ccid4.c
+++ ccid4.latest/net/dccp/ccids/ccid4.c
@@ -58,13 +58,13 @@ static void ccid4_hc_tx_set_state(struct
enum tfrc_hc_tx_states state)
{
struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
- enum tfrc_hc_tx_states oldstate = hctx->tfrchctx_state;
+ enum tfrc_hc_tx_states oldstate = hctx->ttx_state;
ccid4_pr_debug("%s(%p) %-8.8s -> %s\n",
dccp_role(sk), sk, tfrc_tx_state_name(oldstate),
tfrc_tx_state_name(state));
WARN_ON(state = oldstate);
- hctx->tfrchctx_state = state;
+ hctx->ttx_state = state;
}
/*
@@ -76,17 +76,17 @@ static inline void ccid4_update_send_int
* Calculate new t_ipi = s / X_inst (X_inst is in 64 * bytes/second).
* TFRC-SP enforces a minimum interval of 10 milliseconds.
*/
- hctx->tfrchctx_t_ipi - max_t(u32, scaled_div32(((u64)hctx->tfrchctx_s) << 6,
- hctx->tfrchctx_x), MIN_SEND_RATE);
+ hctx->ttx_t_ipi + max_t(u32, scaled_div32(((u64)hctx->ttx_s) << 6,
+ hctx->ttx_x), MIN_SEND_RATE);
/* Calculate new delta by delta = min(t_ipi / 2, t_gran / 2) */
- hctx->tfrchctx_delta = min_t(u32, hctx->tfrchctx_t_ipi / 2,
+ hctx->ttx_delta = min_t(u32, hctx->ttx_t_ipi / 2,
TFRC_OPSYS_HALF_TIME_GRAN);
ccid4_pr_debug("t_ipi=%u, delta=%u, s=%u, X=%u\n",
- hctx->tfrchctx_t_ipi, hctx->tfrchctx_delta,
- hctx->tfrchctx_s, (unsigned)(hctx->tfrchctx_x >> 6));
+ hctx->ttx_t_ipi, hctx->ttx_delta,
+ hctx->ttx_s, (unsigned)(hctx->ttx_x >> 6));
}
@@ -97,8 +97,8 @@ static inline void ccid4_update_send_int
*/
static inline void ccid4_hc_tx_x_header_penalty(struct tfrc_hc_tx_sock *hctx)
{
- hctx->tfrchctx_x *= hctx->tfrchctx_s;
- do_div(hctx->tfrchctx_x, (hctx->tfrchctx_s + CCID4HCTX_H));
+ hctx->ttx_x *= hctx->ttx_s;
+ do_div(hctx->ttx_x, (hctx->ttx_s + CCID4HCTX_H));
}
/**
@@ -114,8 +114,8 @@ static inline void ccid4_hc_tx_x_header_
static void ccid4_hc_tx_update_x(struct sock *sk, ktime_t *stamp)
{
struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
- __u64 min_rate = 2 * hctx->tfrchctx_x_recv;
- const __u64 old_x = hctx->tfrchctx_x;
+ __u64 min_rate = 2 * hctx->ttx_x_recv;
+ const __u64 old_x = hctx->ttx_x;
ktime_t now = stamp ? *stamp : ktime_get_real();
/*
@@ -126,36 +126,36 @@ static void ccid4_hc_tx_update_x(struct
*/
if (tfrc_hc_tx_idle_rtt(hctx, now) >= 2) {
min_rate = rfc3390_initial_rate(sk);
- min_rate = max(min_rate, 2 * hctx->tfrchctx_x_recv);
+ min_rate = max(min_rate, 2 * hctx->ttx_x_recv);
}
- if (hctx->tfrchctx_p > 0) {
+ if (hctx->ttx_p > 0) {
- hctx->tfrchctx_x = min(((__u64)hctx->tfrchctx_x_calc) << 6,
+ hctx->ttx_x = min(((__u64)hctx->ttx_x_calc) << 6,
min_rate);
- hctx->tfrchctx_x = max(hctx->tfrchctx_x,
- (((__u64)hctx->tfrchctx_s) << 6) /
+ hctx->ttx_x = max(hctx->ttx_x,
+ (((__u64)hctx->ttx_s) << 6) /
TFRC_T_MBI);
ccid4_hc_tx_x_header_penalty(hctx);
- } else if (ktime_us_delta(now, hctx->tfrchctx_t_ld)
- - (s64)hctx->tfrchctx_rtt >= 0) {
+ } else if (ktime_us_delta(now, hctx->ttx_t_ld)
+ - (s64)hctx->ttx_rtt >= 0) {
- hctx->tfrchctx_x - max(min(2 * hctx->tfrchctx_x, min_rate),
- scaled_div(((__u64)hctx->tfrchctx_s) << 6,
- hctx->tfrchctx_rtt));
+ hctx->ttx_x + max(min(2 * hctx->ttx_x, min_rate),
+ scaled_div(((__u64)hctx->ttx_s) << 6,
+ hctx->ttx_rtt));
ccid4_hc_tx_x_header_penalty(hctx);
- hctx->tfrchctx_t_ld = now;
+ hctx->ttx_t_ld = now;
}
- if (hctx->tfrchctx_x != old_x) {
+ if (hctx->ttx_x != old_x) {
ccid4_pr_debug("X_prev=%u, X_now=%u, X_calc=%u, "
"X_recv=%u\n", (unsigned)(old_x >> 6),
- (unsigned)(hctx->tfrchctx_x >> 6),
- hctx->tfrchctx_x_calc,
- (unsigned)(hctx->tfrchctx_x_recv >> 6));
+ (unsigned)(hctx->ttx_x >> 6),
+ hctx->ttx_x_calc,
+ (unsigned)(hctx->ttx_x_recv >> 6));
ccid4_update_send_interval(hctx);
}
@@ -167,11 +167,11 @@ static void ccid4_hc_tx_update_x(struct
*/
static inline void ccid4_hc_tx_update_s(struct tfrc_hc_tx_sock *hctx, int len)
{
- const u16 old_s = hctx->tfrchctx_s;
+ const u16 old_s = hctx->ttx_s;
- hctx->tfrchctx_s = tfrc_ewma(hctx->tfrchctx_s, len, 9);
+ hctx->ttx_s = tfrc_ewma(hctx->ttx_s, len, 9);
- if (hctx->tfrchctx_s != old_s)
+ if (hctx->ttx_s != old_s)
ccid4_update_send_interval(hctx);
}
@@ -189,22 +189,22 @@ static void ccid4_hc_tx_no_feedback_time
}
ccid4_pr_debug("%s(%p, state=%s) - entry \n", dccp_role(sk), sk,
- tfrc_tx_state_name(hctx->tfrchctx_state));
+ tfrc_tx_state_name(hctx->ttx_state));
- if (hctx->tfrchctx_state = TTX_STATE_FBACK)
+ if (hctx->ttx_state = TTX_STATE_FBACK)
ccid4_hc_tx_set_state(sk, TTX_STATE_NO_FBACK);
- else if (hctx->tfrchctx_state != TTX_STATE_NO_FBACK)
+ else if (hctx->ttx_state != TTX_STATE_NO_FBACK)
goto out;
/*
* Determine new allowed sending rate X as per draft rfc3448bis-00, 4.4
*/
- if (hctx->tfrchctx_t_rto = 0 || /* no feedback received yet */
- hctx->tfrchctx_p = 0) {
+ if (hctx->ttx_t_rto = 0 || /* no feedback received yet */
+ hctx->ttx_p = 0) {
/* halve send rate directly */
- hctx->tfrchctx_x = max(hctx->tfrchctx_x / 2,
- (((__u64)hctx->tfrchctx_s) << 6) /
+ hctx->ttx_x = max(hctx->ttx_x / 2,
+ (((__u64)hctx->ttx_s) << 6) /
TFRC_T_MBI);
ccid4_update_send_interval(hctx);
} else {
@@ -218,33 +218,33 @@ static void ccid4_hc_tx_no_feedback_time
*
* Note that X_recv is scaled by 2^6 while X_calc is not
*/
- BUG_ON(hctx->tfrchctx_p && !hctx->tfrchctx_x_calc);
+ BUG_ON(hctx->ttx_p && !hctx->ttx_x_calc);
- if (hctx->tfrchctx_x_calc > (hctx->tfrchctx_x_recv >> 5))
- hctx->tfrchctx_x_recv - max(hctx->tfrchctx_x_recv / 2,
- (((__u64)hctx->tfrchctx_s) << 6) /
+ if (hctx->ttx_x_calc > (hctx->ttx_x_recv >> 5))
+ hctx->ttx_x_recv + max(hctx->ttx_x_recv / 2,
+ (((__u64)hctx->ttx_s) << 6) /
(2 * TFRC_T_MBI));
else {
- hctx->tfrchctx_x_recv = hctx->tfrchctx_x_calc;
- hctx->tfrchctx_x_recv <<= 4;
+ hctx->ttx_x_recv = hctx->ttx_x_calc;
+ hctx->ttx_x_recv <<= 4;
}
ccid4_hc_tx_update_x(sk, NULL);
}
ccid4_pr_debug("Reduced X to %llu/64 bytes/sec\n",
- (unsigned long long)hctx->tfrchctx_x);
+ (unsigned long long)hctx->ttx_x);
/*
* Set new timeout for the nofeedback timer.
* See comments in packet_recv() regarding the value of t_RTO.
*/
- if (unlikely(hctx->tfrchctx_t_rto = 0)) /* no feedback yet */
+ if (unlikely(hctx->ttx_t_rto = 0)) /* no feedback yet */
t_nfb = TFRC_INITIAL_TIMEOUT;
else
- t_nfb = max(hctx->tfrchctx_t_rto, 2 * hctx->tfrchctx_t_ipi);
+ t_nfb = max(hctx->ttx_t_rto, 2 * hctx->ttx_t_ipi);
restart_timer:
- sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
+ sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
jiffies + usecs_to_jiffies(t_nfb));
out:
bh_unlock_sock(sk);
@@ -272,18 +272,18 @@ static int ccid4_hc_tx_send_packet(struc
if (unlikely(skb->len = 0))
return -EBADMSG;
- switch (hctx->tfrchctx_state) {
+ switch (hctx->ttx_state) {
case TTX_STATE_NO_SENT:
- sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
+ sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
(jiffies +
usecs_to_jiffies(TFRC_INITIAL_TIMEOUT)));
- hctx->tfrchctx_last_win_count = 0;
- hctx->tfrchctx_t_last_win_count = now;
+ hctx->ttx_last_win_count = 0;
+ hctx->ttx_t_last_win_count = now;
/* Set t_0 for initial packet */
- hctx->tfrchctx_t_nom = now;
+ hctx->ttx_t_nom = now;
- hctx->tfrchctx_s = skb->len;
+ hctx->ttx_s = skb->len;
/*
* Use initial RTT sample when available: recommended by erratum
@@ -292,13 +292,13 @@ static int ccid4_hc_tx_send_packet(struc
*/
if (dp->dccps_syn_rtt) {
ccid4_pr_debug("SYN RTT = %uus\n", dp->dccps_syn_rtt);
- hctx->tfrchctx_rtt = dp->dccps_syn_rtt;
- hctx->tfrchctx_x = rfc3390_initial_rate(sk);
- hctx->tfrchctx_t_ld = now;
+ hctx->ttx_rtt = dp->dccps_syn_rtt;
+ hctx->ttx_x = rfc3390_initial_rate(sk);
+ hctx->ttx_t_ld = now;
} else {
/* Sender does not have RTT sample: X_pps = 1 pkt/sec */
- hctx->tfrchctx_x = hctx->tfrchctx_s;
- hctx->tfrchctx_x <<= 6;
+ hctx->ttx_x = hctx->ttx_s;
+ hctx->ttx_x <<= 6;
}
ccid4_update_send_interval(hctx);
@@ -306,7 +306,7 @@ static int ccid4_hc_tx_send_packet(struc
break;
case TTX_STATE_NO_FBACK:
case TTX_STATE_FBACK:
- delay = ktime_us_delta(hctx->tfrchctx_t_nom, now);
+ delay = ktime_us_delta(hctx->ttx_t_nom, now);
ccid4_pr_debug("delay=%ld\n", (long)delay);
/*
* Scheduling of packet transmissions [RFC 3448, 4.6]
@@ -316,7 +316,7 @@ static int ccid4_hc_tx_send_packet(struc
* else
* // send the packet in (t_nom - t_now) milliseconds.
*/
- if (delay - (s64)hctx->tfrchctx_delta >= 1000)
+ if (delay - (s64)hctx->ttx_delta >= 1000)
return (u32)delay / 1000L;
tfrc_hc_tx_update_win_count(hctx, now);
@@ -328,11 +328,11 @@ static int ccid4_hc_tx_send_packet(struc
/* prepare to send now (add options etc.) */
dp->dccps_hc_tx_insert_options = 1;
- DCCP_SKB_CB(skb)->dccpd_ccval = hctx->tfrchctx_last_win_count;
+ DCCP_SKB_CB(skb)->dccpd_ccval = hctx->ttx_last_win_count;
/* set the nominal send time for the next following packet */
- hctx->tfrchctx_t_nom = ktime_add_us(hctx->tfrchctx_t_nom,
- hctx->tfrchctx_t_ipi);
+ hctx->ttx_t_nom = ktime_add_us(hctx->ttx_t_nom,
+ hctx->ttx_t_ipi);
return 0;
}
@@ -343,7 +343,7 @@ static void ccid4_hc_tx_packet_sent(stru
ccid4_hc_tx_update_s(hctx, len);
- if (tfrc_tx_hist_add(&hctx->tfrchctx_hist, dccp_sk(sk)->dccps_gss))
+ if (tfrc_tx_hist_add(&hctx->ttx_hist, dccp_sk(sk)->dccps_gss))
DCCP_CRIT("packet history - out of memory!");
}
@@ -360,15 +360,15 @@ static void ccid4_hc_tx_packet_recv(stru
DCCP_SKB_CB(skb)->dccpd_type = DCCP_PKT_DATAACK))
return;
/* ... and only in the established state */
- if (hctx->tfrchctx_state != TTX_STATE_FBACK &&
- hctx->tfrchctx_state != TTX_STATE_NO_FBACK)
+ if (hctx->ttx_state != TTX_STATE_FBACK &&
+ hctx->ttx_state != TTX_STATE_NO_FBACK)
return;
- opt_recv = &hctx->tfrchctx_options_received;
+ opt_recv = &hctx->ttx_options_received;
now = ktime_get_real();
/* Estimate RTT from history if ACK number is valid */
- r_sample = tfrc_tx_hist_rtt(hctx->tfrchctx_hist,
+ r_sample = tfrc_tx_hist_rtt(hctx->ttx_hist,
DCCP_SKB_CB(skb)->dccpd_ack_seq, now);
if (r_sample = 0) {
DCCP_WARN("%s(%p): %s with bogus ACK-%llu\n", dccp_role(sk), sk,
@@ -378,37 +378,37 @@ static void ccid4_hc_tx_packet_recv(stru
}
/* Update receive rate in units of 64 * bytes/second */
- hctx->tfrchctx_x_recv = opt_recv->tfrcor_receive_rate;
- hctx->tfrchctx_x_recv <<= 6;
+ hctx->ttx_x_recv = opt_recv->tfrcor_receive_rate;
+ hctx->ttx_x_recv <<= 6;
/* Update loss event rate (which is scaled by 1e6) */
pinv = opt_recv->tfrcor_loss_event_rate;
if (pinv = ~0U || pinv = 0) /* see RFC 4342, 8.5 */
- hctx->tfrchctx_p = 0;
+ hctx->ttx_p = 0;
else /* can not exceed 100% */
- hctx->tfrchctx_p = scaled_div(1, pinv);
+ hctx->ttx_p = scaled_div(1, pinv);
/*
* Validate new RTT sample and update moving average
*/
r_sample = dccp_sample_rtt(sk, r_sample);
- hctx->tfrchctx_rtt = tfrc_ewma(hctx->tfrchctx_rtt, r_sample, 9);
+ hctx->ttx_rtt = tfrc_ewma(hctx->ttx_rtt, r_sample, 9);
/*
* Update allowed sending rate X as per draft rfc3448bis-00, 4.2/3
*/
- if (hctx->tfrchctx_state = TTX_STATE_NO_FBACK) {
+ if (hctx->ttx_state = TTX_STATE_NO_FBACK) {
ccid4_hc_tx_set_state(sk, TTX_STATE_FBACK);
- if (hctx->tfrchctx_t_rto = 0) {
+ if (hctx->ttx_t_rto = 0) {
/*
* Initial feedback packet: Larger Initial Windows (4.2)
*/
- hctx->tfrchctx_x = rfc3390_initial_rate(sk);
- hctx->tfrchctx_t_ld = now;
+ hctx->ttx_x = rfc3390_initial_rate(sk);
+ hctx->ttx_t_ld = now;
ccid4_update_send_interval(hctx);
goto done_computing_x;
- } else if (hctx->tfrchctx_p = 0) {
+ } else if (hctx->ttx_p = 0) {
/*
* First feedback after nofeedback timer expiry (4.3)
*/
@@ -417,25 +417,25 @@ static void ccid4_hc_tx_packet_recv(stru
}
/* Update sending rate (step 4 of [RFC 3448, 4.3]) */
- if (hctx->tfrchctx_p > 0)
- hctx->tfrchctx_x_calc + if (hctx->ttx_p > 0)
+ hctx->ttx_x_calc tfrc_calc_x(NOM_PACKET_SIZE,
- hctx->tfrchctx_rtt,
- hctx->tfrchctx_p);
+ hctx->ttx_rtt,
+ hctx->ttx_p);
ccid4_hc_tx_update_x(sk, &now);
done_computing_x:
ccid4_pr_debug("%s(%p), RTT=%uus (sample=%uus), s=%u, "
"p=%u, X_calc=%u, X_recv=%u, X=%u\n",
dccp_role(sk),
- sk, hctx->tfrchctx_rtt, r_sample,
- hctx->tfrchctx_s, hctx->tfrchctx_p,
- hctx->tfrchctx_x_calc,
- (unsigned)(hctx->tfrchctx_x_recv >> 6),
- (unsigned)(hctx->tfrchctx_x >> 6));
+ sk, hctx->ttx_rtt, r_sample,
+ hctx->ttx_s, hctx->ttx_p,
+ hctx->ttx_x_calc,
+ (unsigned)(hctx->ttx_x_recv >> 6),
+ (unsigned)(hctx->ttx_x >> 6));
/* unschedule no feedback timer */
- sk_stop_timer(sk, &hctx->tfrchctx_no_feedback_timer);
+ sk_stop_timer(sk, &hctx->ttx_no_feedback_timer);
/*
* As we have calculated new ipi, delta, t_nom it is possible
@@ -449,21 +449,21 @@ done_computing_x:
* This can help avoid triggering the nofeedback timer too
* often ('spinning') on LANs with small RTTs.
*/
- hctx->tfrchctx_t_rto = max_t(u32, 4 * hctx->tfrchctx_rtt,
+ hctx->ttx_t_rto = max_t(u32, 4 * hctx->ttx_rtt,
(CONFIG_IP_DCCP_CCID4_RTO *
(USEC_PER_SEC / 1000)));
/*
* Schedule no feedback timer to expire in
* max(t_RTO, 2 * s/X) = max(t_RTO, 2 * t_ipi)
*/
- t_nfb = max(hctx->tfrchctx_t_rto, 2 * hctx->tfrchctx_t_ipi);
+ t_nfb = max(hctx->ttx_t_rto, 2 * hctx->ttx_t_ipi);
ccid4_pr_debug("%s(%p), Scheduled no feedback timer to "
"expire in %lu jiffies (%luus)\n",
dccp_role(sk),
sk, usecs_to_jiffies(t_nfb), t_nfb);
- sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
+ sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
jiffies + usecs_to_jiffies(t_nfb));
}
@@ -477,14 +477,14 @@ static int ccid4_hc_tx_parse_options(str
struct tfrc_options_received *opt_recv;
u32 opt_val;
- opt_recv = &hctx->tfrchctx_options_received;
+ opt_recv = &hctx->ttx_options_received;
if (opt_recv->tfrcor_seqno != dp->dccps_gsr) {
- opt_recv->tfrcor_seqno = dp->dccps_gsr;
+ opt_recv->tfrcor_seqno = dp->dccps_gsr;
opt_recv->tfrcor_loss_event_rate = ~0;
opt_recv->tfrcor_loss_intervals_idx = 0;
opt_recv->tfrcor_loss_intervals_len = 0;
- opt_recv->tfrcor_receive_rate = 0;
+ opt_recv->tfrcor_receive_rate = 0;
}
switch (option) {
@@ -536,9 +536,9 @@ static int ccid4_hc_tx_init(struct ccid
{
struct tfrc_hc_tx_sock *hctx = ccid_priv(ccid);
- hctx->tfrchctx_state = TTX_STATE_NO_SENT;
- hctx->tfrchctx_hist = NULL;
- setup_timer(&hctx->tfrchctx_no_feedback_timer,
+ hctx->ttx_state = TTX_STATE_NO_SENT;
+ hctx->ttx_hist = NULL;
+ setup_timer(&hctx->ttx_no_feedback_timer,
ccid4_hc_tx_no_feedback_timer, (unsigned long)sk);
return 0;
@@ -549,15 +549,15 @@ static void ccid4_hc_tx_exit(struct sock
struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
ccid4_hc_tx_set_state(sk, TTX_STATE_TERM);
- sk_stop_timer(sk, &hctx->tfrchctx_no_feedback_timer);
+ sk_stop_timer(sk, &hctx->ttx_no_feedback_timer);
- tfrc_tx_hist_purge(&hctx->tfrchctx_hist);
+ tfrc_tx_hist_purge(&hctx->ttx_hist);
}
static void ccid4_hc_tx_get_info(struct sock *sk, struct tcp_info *info)
{
- info->tcpi_rto = tfrc_hc_tx_sk(sk)->tfrchctx_t_rto;
- info->tcpi_rtt = tfrc_hc_tx_sk(sk)->tfrchctx_rtt;
+ info->tcpi_rto = tfrc_hc_tx_sk(sk)->ttx_t_rto;
+ info->tcpi_rtt = tfrc_hc_tx_sk(sk)->ttx_rtt;
}
static int ccid4_hc_tx_getsockopt(struct sock *sk, const int optname, int len,
@@ -568,10 +568,10 @@ static int ccid4_hc_tx_getsockopt(struct
switch (optname) {
case DCCP_SOCKOPT_CCID_TX_INFO:
- if (len < sizeof(hctx->tfrchctx_tfrc))
+ if (len < sizeof(hctx->ttx_tfrc))
return -EINVAL;
- len = sizeof(hctx->tfrchctx_tfrc);
- val = &hctx->tfrchctx_tfrc;
+ len = sizeof(hctx->ttx_tfrc);
+ val = &hctx->ttx_tfrc;
break;
default:
return -ENOPROTOOPT;
@@ -587,13 +587,13 @@ static void ccid4_hc_rx_set_state(struct
enum tfrc_hc_rx_states state)
{
struct tfrc_hc_rx_sock *hcrx = tfrc_hc_rx_sk(sk);
- enum tfrc_hc_rx_states oldstate = hcrx->tfrchcrx_state;
+ enum tfrc_hc_rx_states oldstate = hcrx->trx_state;
ccid4_pr_debug("%s(%p) %-8.8s -> %s\n",
dccp_role(sk), sk, tfrc_rx_state_name(oldstate),
tfrc_rx_state_name(state));
WARN_ON(state = oldstate);
- hcrx->tfrchcrx_state = state;
+ hcrx->trx_state = state;
}
static void ccid4_hc_rx_send_feedback(struct sock *sk,
@@ -605,13 +605,13 @@ static void ccid4_hc_rx_send_feedback(st
ktime_t now = ktime_get_real();
s64 delta = 0;
- if (unlikely(hcrx->tfrchcrx_state = TRX_STATE_TERM))
+ if (unlikely(hcrx->trx_state = TRX_STATE_TERM))
return;
switch (fbtype) {
case TFRC_FBACK_INITIAL:
- hcrx->tfrchcrx_x_recv = 0;
- hcrx->tfrchcrx_pinv = ~0U; /* see RFC 4342, 8.5 */
+ hcrx->trx_x_recv = 0;
+ hcrx->trx_pinv = ~0U; /* see RFC 4342, 8.5 */
break;
case TFRC_FBACK_PARAM_CHANGE:
/*
@@ -624,26 +624,26 @@ static void ccid4_hc_rx_send_feedback(st
* the number of bytes since last feedback.
* This is a safe fallback, since X is bounded above by X_calc.
*/
- if (hcrx->tfrchcrx_x_recv > 0)
+ if (hcrx->trx_x_recv > 0)
break;
/* fall through */
case TFRC_FBACK_PERIODIC:
- delta = ktime_us_delta(now, hcrx->tfrchcrx_last_feedback);
+ delta = ktime_us_delta(now, hcrx->trx_last_feedback);
if (delta <= 0)
DCCP_BUG("delta (%ld) <= 0", (long)delta);
else
- hcrx->tfrchcrx_x_recv - scaled_div32(hcrx->tfrchcrx_bytes_recv, delta);
+ hcrx->trx_x_recv + scaled_div32(hcrx->trx_bytes_recv, delta);
break;
default:
return;
}
ccid4_pr_debug("Interval %ldusec, X_recv=%u, 1/p=%u\n", (long)delta,
- hcrx->tfrchcrx_x_recv, hcrx->tfrchcrx_pinv);
+ hcrx->trx_x_recv, hcrx->trx_pinv);
- hcrx->tfrchcrx_last_feedback = now;
- hcrx->tfrchcrx_last_counter = dccp_hdr(skb)->dccph_ccval;
- hcrx->tfrchcrx_bytes_recv = 0;
+ hcrx->trx_last_feedback = now;
+ hcrx->trx_last_counter = dccp_hdr(skb)->dccph_ccval;
+ hcrx->trx_bytes_recv = 0;
dp->dccps_hc_rx_insert_options = 1;
dccp_send_ack(sk);
@@ -660,8 +660,8 @@ static int ccid4_hc_rx_insert_options(st
if (dccp_packet_without_ack(skb))
return 0;
- x_recv = htonl(hcrx->tfrchcrx_x_recv);
- pinv = htonl(hcrx->tfrchcrx_pinv);
+ x_recv = htonl(hcrx->trx_x_recv);
+ pinv = htonl(hcrx->trx_pinv);
if (dccp_insert_option(sk, skb, TFRC_OPT_LOSS_EVENT_RATE,
&pinv, sizeof(pinv)) ||
@@ -687,22 +687,22 @@ static u32 ccid4_first_li(struct sock *s
u32 x_recv, p, delta;
u64 fval;
- if (hcrx->tfrchcrx_rtt = 0) {
+ if (hcrx->trx_rtt = 0) {
DCCP_WARN("No RTT estimate available, using fallback RTT\n");
- hcrx->tfrchcrx_rtt = DCCP_FALLBACK_RTT;
+ hcrx->trx_rtt = DCCP_FALLBACK_RTT;
}
- delta = ktime_to_us(net_timedelta(hcrx->tfrchcrx_last_feedback));
- x_recv = scaled_div32(hcrx->tfrchcrx_bytes_recv, delta);
+ delta = ktime_to_us(net_timedelta(hcrx->trx_last_feedback));
+ x_recv = scaled_div32(hcrx->trx_bytes_recv, delta);
if (x_recv = 0) { /* would also trigger divide-by-zero */
DCCP_WARN("X_recv=0\n");
- if ((x_recv = hcrx->tfrchcrx_x_recv) = 0) {
+ if ((x_recv = hcrx->trx_x_recv) = 0) {
DCCP_BUG("stored value of X_recv is zero");
return ~0U;
}
}
- fval = scaled_div(NOM_PACKET_SIZE, hcrx->tfrchcrx_rtt);
+ fval = scaled_div(NOM_PACKET_SIZE, hcrx->trx_rtt);
fval = scaled_div32(fval, x_recv);
p = tfrc_calc_x_reverse_lookup(fval);
@@ -719,14 +719,14 @@ static void ccid4_hc_rx_packet_recv(stru
const u64 ndp = dccp_sk(sk)->dccps_options_received.dccpor_ndp;
const bool is_data_packet = dccp_data_packet(skb);
- if (unlikely(hcrx->tfrchcrx_state = TRX_STATE_NO_DATA)) {
+ if (unlikely(hcrx->trx_state = TRX_STATE_NO_DATA)) {
if (is_data_packet) {
const u32 payload = skb->len - dccp_hdr(skb)->dccph_doff * 4;
do_feedback = TFRC_FBACK_INITIAL;
ccid4_hc_rx_set_state(sk, TRX_STATE_DATA);
- hcrx->tfrchcrx_s = payload;
+ hcrx->trx_s = payload;
/*
- * Not necessary to update tfrchcrx_bytes_recv here,
+ * Not necessary to update trx_bytes_recv here,
* since X_recv = 0 for the first feedback packet (cf.
* RFC 3448, 6.3) -- gerrit
*/
@@ -735,7 +735,7 @@ static void ccid4_hc_rx_packet_recv(stru
goto update_records;
}
- if (tfrc_rx_hist_duplicate(&hcrx->tfrchcrx_hist, skb))
+ if (tfrc_rx_hist_duplicate(&hcrx->trx_hist, skb))
return; /* done receiving */
if (is_data_packet) {
@@ -743,22 +743,21 @@ static void ccid4_hc_rx_packet_recv(stru
/*
* Update moving-average of s and the sum of received payload bytes
*/
- hcrx->tfrchcrx_s = tfrc_ewma(hcrx->tfrchcrx_s, payload, 9);
- hcrx->tfrchcrx_bytes_recv += payload;
+ hcrx->trx_s = tfrc_ewma(hcrx->trx_s, payload, 9);
+ hcrx->trx_bytes_recv += payload;
}
/*
* Handle pending losses and otherwise check for new loss
*/
- if (tfrc_rx_hist_loss_pending(&hcrx->tfrchcrx_hist) &&
- tfrc_rx_handle_loss(&hcrx->tfrchcrx_hist,
- &hcrx->tfrchcrx_li_hist,
+ if (tfrc_rx_hist_loss_pending(&hcrx->trx_hist) &&
+ tfrc_rx_handle_loss(&hcrx->trx_hist, &hcrx->trx_li_hist,
skb, ndp, ccid4_first_li, sk) ) {
do_feedback = TFRC_FBACK_PARAM_CHANGE;
goto done_receiving;
}
- if (tfrc_rx_hist_new_loss_indicated(&hcrx->tfrchcrx_hist, skb, ndp))
+ if (tfrc_rx_hist_new_loss_indicated(&hcrx->trx_hist, skb, ndp))
goto update_records;
/*
@@ -767,17 +766,17 @@ static void ccid4_hc_rx_packet_recv(stru
if (unlikely(!is_data_packet))
goto update_records;
- if (!tfrc_lh_is_initialised(&hcrx->tfrchcrx_li_hist)) {
- const u32 sample = tfrc_rx_hist_sample_rtt(&hcrx->tfrchcrx_hist, skb);
+ if (!tfrc_lh_is_initialised(&hcrx->trx_li_hist)) {
+ const u32 sample = tfrc_rx_hist_sample_rtt(&hcrx->trx_hist, skb);
/*
* Empty loss history: no loss so far, hence p stays 0.
* Sample RTT values, since an RTT estimate is required for the
* computation of p when the first loss occurs; RFC 3448, 6.3.1.
*/
if (sample != 0)
- hcrx->tfrchcrx_rtt = tfrc_ewma(hcrx->tfrchcrx_rtt, sample, 9);
+ hcrx->trx_rtt = tfrc_ewma(hcrx->trx_rtt, sample, 9);
- } else if (tfrc_lh_update_i_mean(&hcrx->tfrchcrx_li_hist, skb))
+ } else if (tfrc_lh_update_i_mean(&hcrx->trx_li_hist, skb))
/*
* Step (3) of [RFC 3448, 6.1]: Recompute I_mean and, if I_mean
* has decreased (resp. p has increased), send feedback now.
@@ -787,11 +786,11 @@ static void ccid4_hc_rx_packet_recv(stru
/*
* Check if the periodic once-per-RTT feedback is due; RFC 4342, 10.3
*/
- if (SUB16(dccp_hdr(skb)->dccph_ccval, hcrx->tfrchcrx_last_counter) > 3)
+ if (SUB16(dccp_hdr(skb)->dccph_ccval, hcrx->trx_last_counter) > 3)
do_feedback = TFRC_FBACK_PERIODIC;
update_records:
- tfrc_rx_hist_add_packet(&hcrx->tfrchcrx_hist, skb, ndp);
+ tfrc_rx_hist_add_packet(&hcrx->trx_hist, skb, ndp);
done_receiving:
if (do_feedback)
@@ -802,9 +801,9 @@ static int ccid4_hc_rx_init(struct ccid
{
struct tfrc_hc_rx_sock *hcrx = ccid_priv(ccid);
- hcrx->tfrchcrx_state = TRX_STATE_NO_DATA;
- tfrc_lh_init(&hcrx->tfrchcrx_li_hist);
- return tfrc_rx_hist_alloc(&hcrx->tfrchcrx_hist);
+ hcrx->trx_state = TRX_STATE_NO_DATA;
+ tfrc_lh_init(&hcrx->trx_li_hist);
+ return tfrc_rx_hist_alloc(&hcrx->trx_hist);
}
static void ccid4_hc_rx_exit(struct sock *sk)
@@ -813,15 +812,15 @@ static void ccid4_hc_rx_exit(struct sock
ccid4_hc_rx_set_state(sk, TRX_STATE_TERM);
- tfrc_rx_hist_purge(&hcrx->tfrchcrx_hist);
- tfrc_lh_cleanup(&hcrx->tfrchcrx_li_hist);
+ tfrc_rx_hist_purge(&hcrx->trx_hist);
+ tfrc_lh_cleanup(&hcrx->trx_li_hist);
}
static void ccid4_hc_rx_get_info(struct sock *sk, struct tcp_info *info)
{
- info->tcpi_ca_state = tfrc_hc_rx_sk(sk)->tfrchcrx_state;
+ info->tcpi_ca_state = tfrc_hc_rx_sk(sk)->trx_state;
info->tcpi_options |= TCPI_OPT_TIMESTAMPS;
- info->tcpi_rcv_rtt = tfrc_hc_rx_sk(sk)->tfrchcrx_rtt;
+ info->tcpi_rcv_rtt = tfrc_hc_rx_sk(sk)->trx_rtt;
}
static int ccid4_hc_rx_getsockopt(struct sock *sk, const int optname, int len,
@@ -835,10 +834,10 @@ static int ccid4_hc_rx_getsockopt(struct
case DCCP_SOCKOPT_CCID_RX_INFO:
if (len < sizeof(rx_info))
return -EINVAL;
- rx_info.tfrcrx_x_recv = hcrx->tfrchcrx_x_recv;
- rx_info.tfrcrx_rtt = hcrx->tfrchcrx_rtt;
- rx_info.tfrcrx_p = hcrx->tfrchcrx_pinv = 0 ? ~0U :
- scaled_div(1, hcrx->tfrchcrx_pinv);
+ rx_info.tfrcrx_x_recv = hcrx->trx_x_recv;
+ rx_info.tfrcrx_rtt = hcrx->trx_rtt;
+ rx_info.tfrcrx_p = hcrx->trx_pinv = 0 ? ~0U :
+ scaled_div(1, hcrx->trx_pinv);
len = sizeof(rx_info);
val = &rx_info;
break;
Index: ccid4.latest/net/dccp/ccids/lib/tfrc_ccids.c
=================================--- ccid4.latest.orig/net/dccp/ccids/lib/tfrc_ccids.c
+++ ccid4.latest/net/dccp/ccids/lib/tfrc_ccids.c
@@ -23,19 +23,19 @@
inline u64 rfc3390_initial_rate(struct sock *sk)
{
const struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
- const __u32 w_init = min_t(__u32, 4 * hctx->tfrchctx_s,
- max_t(__u32, 2 * hctx->tfrchctx_s, 4380));
+ const __u32 w_init = min_t(__u32, 4 * hctx->ttx_s,
+ max_t(__u32, 2 * hctx->ttx_s, 4380));
- return scaled_div(w_init << 6, hctx->tfrchctx_rtt);
+ return scaled_div(w_init << 6, hctx->ttx_rtt);
}
EXPORT_SYMBOL_GPL(rfc3390_initial_rate);
u32 tfrc_hc_tx_idle_rtt(struct tfrc_hc_tx_sock *hctx, ktime_t now)
{
- u32 delta = ktime_us_delta(now, hctx->tfrchctx_t_last_win_count);
+ u32 delta = ktime_us_delta(now, hctx->ttx_t_last_win_count);
- return delta / hctx->tfrchctx_rtt;
+ return delta / hctx->ttx_rtt;
}
EXPORT_SYMBOL_GPL(tfrc_hc_tx_idle_rtt);
@@ -49,16 +49,16 @@ inline void tfrc_hc_tx_update_win_count(
{
u32 quarter_rtts;
- if (unlikely(hctx->tfrchctx_rtt < 4)) /* avoid divide-by-zero */
+ if (unlikely(hctx->ttx_rtt < 4)) /* avoid divide-by-zero */
return;
- quarter_rtts = ktime_us_delta(now, hctx->tfrchctx_t_last_win_count);
- quarter_rtts /= hctx->tfrchctx_rtt / 4;
+ quarter_rtts = ktime_us_delta(now, hctx->ttx_t_last_win_count);
+ quarter_rtts /= hctx->ttx_rtt / 4;
if (quarter_rtts > 0) {
- hctx->tfrchctx_t_last_win_count = now;
- hctx->tfrchctx_last_win_count += min_t(u32, quarter_rtts, 5);
- hctx->tfrchctx_last_win_count &= 0xF; /* mod 16 */
+ hctx->ttx_t_last_win_count = now;
+ hctx->ttx_last_win_count += min_t(u32, quarter_rtts, 5);
+ hctx->ttx_last_win_count &= 0xF; /* mod 16 */
}
}
Index: ccid4.latest/net/dccp/ccids/lib/tfrc_ccids.h
=================================--- ccid4.latest.orig/net/dccp/ccids/lib/tfrc_ccids.h
+++ ccid4.latest/net/dccp/ccids/lib/tfrc_ccids.h
@@ -73,44 +73,44 @@ enum tfrc_fback_type {
/** struct tfrc_hc_tx_sock - CCID3/4 sender half-connection socket
*
- * @tfrchctx_x - Current sending rate in 64 * bytes per second
- * @tfrchctx_x_recv - Receive rate in 64 * bytes per second
- * @tfrchctx_x_calc - Calculated rate in bytes per second
- * @tfrchctx_rtt - Estimate of current round trip time in usecs
- * @tfrchctx_p - Current loss event rate (0-1) scaled by 1000000
- * @tfrchctx_s - Packet size in bytes
- * @tfrchctx_t_rto - Nofeedback Timer setting in usecs
- * @tfrchctx_t_ipi - Interpacket (send) interval (RFC 3448, 4.6) in usecs
- * @tfrchctx_state - Sender state, one of %tfrc_hc_tx_states
- * @tfrchctx_last_win_count - Last window counter sent
- * @tfrchctx_t_last_win_count - Timestamp of earliest packet
+ * @ttx_x - Current sending rate in 64 * bytes per second
+ * @ttx_x_recv - Receive rate in 64 * bytes per second
+ * @ttx_x_calc - Calculated rate in bytes per second
+ * @ttx_rtt - Estimate of current round trip time in usecs
+ * @ttx_p - Current loss event rate (0-1) scaled by 1000000
+ * @ttx_s - Packet size in bytes
+ * @ttx_t_rto - Nofeedback Timer setting in usecs
+ * @ttx_t_ipi - Interpacket (send) interval (RFC 3448, 4.6) in usecs
+ * @ttx_state - Sender state, one of %tfrc_hc_tx_states
+ * @ttx_last_win_count - Last window counter sent
+ * @ttx_t_last_win_count - Timestamp of earliest packet
* with last_win_count value sent
- * @tfrchctx_no_feedback_timer - Handle to no feedback timer
- * @tfrchctx_t_ld - Time last doubled during slow start
- * @tfrchctx_t_nom - Nominal send time of next packet
- * @tfrchctx_delta - Send timer delta (RFC 3448, 4.6) in usecs
- * @tfrchctx_hist - Packet history
- * @tfrchctx_options_received - Parsed set of retrieved options
+ * @ttx_no_feedback_timer - Handle to no feedback timer
+ * @ttx_t_ld - Time last doubled during slow start
+ * @ttx_t_nom - Nominal send time of next packet
+ * @ttx_delta - Send timer delta (RFC 3448, 4.6) in usecs
+ * @ttx_hist - Packet history
+ * @ttx_options_received - Parsed set of retrieved options
*/
struct tfrc_hc_tx_sock {
- struct tfrc_tx_info tfrchctx_tfrc;
-#define tfrchctx_x tfrchctx_tfrc.tfrctx_x
-#define tfrchctx_x_recv tfrchctx_tfrc.tfrctx_x_recv
-#define tfrchctx_x_calc tfrchctx_tfrc.tfrctx_x_calc
-#define tfrchctx_rtt tfrchctx_tfrc.tfrctx_rtt
-#define tfrchctx_p tfrchctx_tfrc.tfrctx_p
-#define tfrchctx_t_rto tfrchctx_tfrc.tfrctx_rto
-#define tfrchctx_t_ipi tfrchctx_tfrc.tfrctx_ipi
- u16 tfrchctx_s;
- enum tfrc_hc_tx_states tfrchctx_state:8;
- u8 tfrchctx_last_win_count;
- ktime_t tfrchctx_t_last_win_count;
- struct timer_list tfrchctx_no_feedback_timer;
- ktime_t tfrchctx_t_ld;
- ktime_t tfrchctx_t_nom;
- u32 tfrchctx_delta;
- struct tfrc_tx_hist_entry *tfrchctx_hist;
- struct tfrc_options_received tfrchctx_options_received;
+ struct tfrc_tx_info ttx_tfrc;
+#define ttx_x ttx_tfrc.x
+#define ttx_x_recv ttx_tfrc.x_recv
+#define ttx_x_calc ttx_tfrc.x_calc
+#define ttx_rtt ttx_tfrc.rtt
+#define ttx_p ttx_tfrc.p
+#define ttx_t_rto ttx_tfrc.rto
+#define ttx_t_ipi ttx_tfrc.ipi
+ u16 ttx_s;
+ enum tfrc_hc_tx_states ttx_state:8;
+ u8 ttx_last_win_count;
+ ktime_t ttx_t_last_win_count;
+ struct timer_list ttx_no_feedback_timer;
+ ktime_t ttx_t_ld;
+ ktime_t ttx_t_nom;
+ u32 ttx_delta;
+ struct tfrc_tx_hist_entry *ttx_hist;
+ struct tfrc_options_received ttx_options_received;
};
static inline struct tfrc_hc_tx_sock *tfrc_hc_tx_sk(const struct sock *sk)
@@ -122,28 +122,28 @@ static inline struct tfrc_hc_tx_sock *tf
/** struct tfrc_hc_rx_sock - CCID3/4 receiver half-connection socket
*
- * @tfrchcrx_last_counter - Tracks window counter (RFC 4342, 8.1)
- * @tfrchcrx_state - Receiver state, one of %tfrc_hc_rx_states
- * @tfrchcrx_bytes_recv - Total sum of DCCP payload bytes
- * @tfrchcrx_x_recv - Receiver estimate of send rate (RFC 3448, sec. 4.3)
- * @tfrchcrx_rtt - Receiver estimate of RTT
- * @tfrchcrx_last_feedback - Time at which last feedback was sent
- * @tfrchcrx_hist - Packet history, exported by TFRC module
- * @tfrchcrx_li_hist - Loss Interval database, exported by TFRC module
- * @tfrchcrx_s - Received packet size in bytes
- * @tfrchcrx_pinv - Inverse of Loss Event Rate (RFC 4342, sec. 8.5)
+ * @trx_last_counter - Tracks window counter (RFC 4342, 8.1)
+ * @trx_state - Receiver state, one of %tfrc_hc_rx_states
+ * @trx_bytes_recv - Total sum of DCCP payload bytes
+ * @trx_x_recv - Receiver estimate of send rate (RFC 3448, sec. 4.3)
+ * @trx_rtt - Receiver estimate of RTT
+ * @trx_last_feedback - Time at which last feedback was sent
+ * @trx_hist - Packet history, exported by TFRC module
+ * @trx_li_hist - Loss Interval database, exported by TFRC module
+ * @trx_s - Received packet size in bytes
+ * @trx_pinv - Inverse of Loss Event Rate (RFC 4342, sec. 8.5)
*/
struct tfrc_hc_rx_sock {
- u8 tfrchcrx_last_counter:4;
- enum tfrc_hc_rx_states tfrchcrx_state:8;
- u32 tfrchcrx_bytes_recv;
- u32 tfrchcrx_x_recv;
- u32 tfrchcrx_rtt;
- ktime_t tfrchcrx_last_feedback;
- struct tfrc_rx_hist tfrchcrx_hist;
- struct tfrc_loss_hist tfrchcrx_li_hist;
- u16 tfrchcrx_s;
-#define tfrchcrx_pinv tfrchcrx_li_hist.i_mean
+ u8 trx_last_counter:4;
+ enum tfrc_hc_rx_states trx_state:8;
+ u32 trx_bytes_recv;
+ u32 trx_x_recv;
+ u32 trx_rtt;
+ ktime_t trx_last_feedback;
+ struct tfrc_rx_hist trx_hist;
+ struct tfrc_loss_hist trx_li_hist;
+ u16 trx_s;
+#define trx_pinv trx_li_hist.i_mean
};
static inline struct tfrc_hc_rx_sock *tfrc_hc_rx_sk(const struct sock *sk)
@@ -193,4 +193,3 @@ extern u32 tfrc_hc_tx_idle_rtt(struct tf
extern void tfrc_hc_tx_update_win_count(struct tfrc_hc_tx_sock *hctx,
ktime_t now);
-
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCHv2 4/8] Fix name schema for tx-rx sock struct
2007-12-21 4:33 [PATCHv2 4/8] Fix name schema for tx-rx sock struct Leandro
@ 2007-12-21 9:53 ` Gerrit Renker
2007-12-26 18:01 ` Leandro Sales
2008-01-07 13:22 ` Gerrit Renker
2 siblings, 0 replies; 4+ messages in thread
From: Gerrit Renker @ 2007-12-21 9:53 UTC (permalink / raw)
To: dccp
|
| The first version of this patch doest apply properly in the current ccid4 branch. This new version it is ok.
|
I don't know what happened but unfortunately this patch does not apply,
since below an existing file (include/linux/tfrc.h) is created.
Which is a pity -- I fully agree with the naming scheme, since IMO we
don't need to repeat identifiers in field names. I think that this
is a helpful patch - the only suggestion is to change `x_x' to simply
`x'.
| Index: ccid4.latest/include/linux/tfrc.h
| =================================| --- /dev/null
| +++ ccid4.latest/include/linux/tfrc.h
| @@ -0,0 +1,55 @@
| +#ifndef _LINUX_TFRC_H_
| +#define _LINUX_TFRC_H_
| +/*
| + * TFRC - Data Structures for the TCP-Friendly Rate Control congestion
| + * control mechanism as specified in RFC 3448.
| + *
| + * Copyright (c) 2005 The University of Waikato, Hamilton, New Zealand.
| + * Copyright (c) 2005 Ian McDonald <iam4@cs.waikato.ac.nz>
| + * Copyright (c) 2005 Arnaldo Carvalho de Melo <acme@conectiva.com.br>
| + * Copyright (c) 2003 Nils-Erik Mattsson, Joacim Haggmark, Magnus Erixzon
| + *
| + * This program is free software; you can redistribute it and/or modify
| + * it under the terms of the GNU General Public License as published by
| + * the Free Software Foundation; either version 2 of the License, or
| + * (at your option) any later version.
| + */
| +#include <linux/types.h>
| +
| +/** tfrc_rx_info - TFRC Receiver Data Structure
| + *
| + * @x_recv: receiver estimate of sending rate (3.2.2)
| + * @rtt: round-trip-time (communicated by sender)
| + * @p: current estimate of loss event rate (3.2.2)
| + */
| +struct tfrc_rx_info {
| + __u32 x_recv;
| + __u32 rtt;
| + __u32 p;
| +};
| +
| +/** tfrc_tx_info - TFRC Sender Data Structure
| + *
| + * @x_x: computed transmit rate (4.3 (4))
| + * @x_recv: receiver estimate of send rate (4.3)
| + * @x_calc: return value of throughput equation (3.1)
| + * @rtt: (moving average) estimate of RTT (4.3)
| + * @p: current loss event rate (5.4)
| + * @rto: estimate of RTO, equals 4*RTT (4.3)
| + * @ipi: inter-packet interval (4.6)
| + *
| + * Note: X and X_recv are both maintained in units of 64 * bytes/second. This
| + * enables a finer resolution of sending rates and avoids problems with
| + * integer arithmetic; u32 is not sufficient as scaling consumes 6 bits.
| + */
| +struct tfrc_tx_info {
| + __u64 x;
| + __u64 x_recv;
| + __u32 x_calc;
| + __u32 rtt;
| + __u32 p;
| + __u32 rto;
| + __u32 ipi;
| +};
| +
| +#endif /* _LINUX_TFRC_H_ */
| Index: ccid4.latest/net/dccp/ccids/ccid3.c
| =================================| --- ccid4.latest.orig/net/dccp/ccids/ccid3.c
| +++ ccid4.latest/net/dccp/ccids/ccid3.c
| @@ -48,13 +48,13 @@ static void ccid3_hc_tx_set_state(struct
| enum tfrc_hc_tx_states state)
| {
| struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
| - enum tfrc_hc_tx_states oldstate = hctx->tfrchctx_state;
| + enum tfrc_hc_tx_states oldstate = hctx->ttx_state;
|
| ccid3_pr_debug("%s(%p) %-8.8s -> %s\n",
| dccp_role(sk), sk, tfrc_tx_state_name(oldstate),
| tfrc_tx_state_name(state));
| WARN_ON(state = oldstate);
| - hctx->tfrchctx_state = state;
| + hctx->ttx_state = state;
| }
|
| /*
| @@ -63,16 +63,15 @@ static void ccid3_hc_tx_set_state(struct
| static inline void ccid3_update_send_interval(struct tfrc_hc_tx_sock *hctx)
| {
| /* Calculate new t_ipi = s / X_inst (X_inst is in 64 * bytes/second) */
| - hctx->tfrchctx_t_ipi = scaled_div32(((u64)hctx->tfrchctx_s) << 6,
| - hctx->tfrchctx_x);
| + hctx->ttx_t_ipi = scaled_div32(((u64)hctx->ttx_s) << 6, hctx->ttx_x);
|
| /* Calculate new delta by delta = min(t_ipi / 2, t_gran / 2) */
| - hctx->tfrchctx_delta = min_t(u32, hctx->tfrchctx_t_ipi / 2,
| + hctx->ttx_delta = min_t(u32, hctx->ttx_t_ipi / 2,
| TFRC_OPSYS_HALF_TIME_GRAN);
|
| ccid3_pr_debug("t_ipi=%u, delta=%u, s=%u, X=%u\n",
| - hctx->tfrchctx_t_ipi, hctx->tfrchctx_delta,
| - hctx->tfrchctx_s, (unsigned)(hctx->tfrchctx_x >> 6));
| + hctx->ttx_t_ipi, hctx->ttx_delta,
| + hctx->ttx_s, (unsigned)(hctx->ttx_x >> 6));
|
| }
|
| @@ -89,8 +88,8 @@ static inline void ccid3_update_send_int
| static void ccid3_hc_tx_update_x(struct sock *sk, ktime_t *stamp)
| {
| struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
| - __u64 min_rate = 2 * hctx->tfrchctx_x_recv;
| - const __u64 old_x = hctx->tfrchctx_x;
| + __u64 min_rate = 2 * hctx->ttx_x_recv;
| + const __u64 old_x = hctx->ttx_x;
| ktime_t now = stamp ? *stamp : ktime_get_real();
|
| /*
| @@ -101,33 +100,29 @@ static void ccid3_hc_tx_update_x(struct
| */
| if (tfrc_hc_tx_idle_rtt(hctx, now) >= 2) {
| min_rate = rfc3390_initial_rate(sk);
| - min_rate = max(min_rate, 2 * hctx->tfrchctx_x_recv);
| + min_rate = max(min_rate, 2 * hctx->ttx_x_recv);
| }
|
| - if (hctx->tfrchctx_p > 0) {
| + if (hctx->ttx_p > 0) {
|
| - hctx->tfrchctx_x = min(((__u64)hctx->tfrchctx_x_calc) << 6,
| - min_rate);
| - hctx->tfrchctx_x = max(hctx->tfrchctx_x,
| - (((__u64)hctx->tfrchctx_s) << 6) /
| - TFRC_T_MBI);
| + hctx->ttx_x = min(((__u64)hctx->ttx_x_calc) << 6, min_rate);
| + hctx->ttx_x = max(hctx->ttx_x,
| + (((__u64)hctx->ttx_s) << 6) / TFRC_T_MBI);
|
| - } else if (ktime_us_delta(now, hctx->tfrchctx_t_ld)
| - - (s64)hctx->tfrchctx_rtt >= 0) {
| + } else if (ktime_us_delta(now, hctx->ttx_t_ld)
| + - (s64)hctx->ttx_rtt >= 0) {
|
| - hctx->tfrchctx_x | - max(min(2 * hctx->tfrchctx_x, min_rate),
| - scaled_div(((__u64)hctx->tfrchctx_s) << 6,
| - hctx->tfrchctx_rtt));
| - hctx->tfrchctx_t_ld = now;
| + hctx->ttx_x = max(min(2 * hctx->ttx_x, min_rate),
| + scaled_div(((__u64)hctx->ttx_s) << 6,
| + hctx->ttx_rtt));
| + hctx->ttx_t_ld = now;
| }
|
| - if (hctx->tfrchctx_x != old_x) {
| + if (hctx->ttx_x != old_x) {
| ccid3_pr_debug("X_prev=%u, X_now=%u, X_calc=%u, "
| "X_recv=%u\n", (unsigned)(old_x >> 6),
| - (unsigned)(hctx->tfrchctx_x >> 6),
| - hctx->tfrchctx_x_calc,
| - (unsigned)(hctx->tfrchctx_x_recv >> 6));
| + (unsigned)(hctx->ttx_x >> 6), hctx->ttx_x_calc,
| + (unsigned)(hctx->ttx_x_recv >> 6));
|
| ccid3_update_send_interval(hctx);
| }
| @@ -139,11 +134,11 @@ static void ccid3_hc_tx_update_x(struct
| */
| static inline void ccid3_hc_tx_update_s(struct tfrc_hc_tx_sock *hctx, int len)
| {
| - const u16 old_s = hctx->tfrchctx_s;
| + const u16 old_s = hctx->ttx_s;
|
| - hctx->tfrchctx_s = tfrc_ewma(hctx->tfrchctx_s, len, 9);
| + hctx->ttx_s = tfrc_ewma(hctx->ttx_s, len, 9);
|
| - if (hctx->tfrchctx_s != old_s)
| + if (hctx->ttx_s != old_s)
| ccid3_update_send_interval(hctx);
| }
|
| @@ -161,22 +156,22 @@ static void ccid3_hc_tx_no_feedback_time
| }
|
| ccid3_pr_debug("%s(%p, state=%s) - entry \n", dccp_role(sk), sk,
| - tfrc_tx_state_name(hctx->tfrchctx_state));
| + tfrc_tx_state_name(hctx->ttx_state));
|
| - if (hctx->tfrchctx_state = TTX_STATE_FBACK)
| + if (hctx->ttx_state = TTX_STATE_FBACK)
| ccid3_hc_tx_set_state(sk, TTX_STATE_NO_FBACK);
| - else if (hctx->tfrchctx_state != TTX_STATE_NO_FBACK)
| + else if (hctx->ttx_state != TTX_STATE_NO_FBACK)
| goto out;
|
| /*
| * Determine new allowed sending rate X as per draft rfc3448bis-00, 4.4
| */
| - if (hctx->tfrchctx_t_rto = 0 || /* no feedback received yet */
| - hctx->tfrchctx_p = 0) {
| + if (hctx->ttx_t_rto = 0 || /* no feedback received yet */
| + hctx->ttx_p = 0) {
|
| /* halve send rate directly */
| - hctx->tfrchctx_x = max(hctx->tfrchctx_x / 2,
| - (((__u64)hctx->tfrchctx_s) << 6) /
| + hctx->ttx_x = max(hctx->ttx_x / 2,
| + (((__u64)hctx->ttx_s) << 6) /
| TFRC_T_MBI);
| ccid3_update_send_interval(hctx);
| } else {
| @@ -190,33 +185,32 @@ static void ccid3_hc_tx_no_feedback_time
| *
| * Note that X_recv is scaled by 2^6 while X_calc is not
| */
| - BUG_ON(hctx->tfrchctx_p && !hctx->tfrchctx_x_calc);
| + BUG_ON(hctx->ttx_p && !hctx->ttx_x_calc);
|
| - if (hctx->tfrchctx_x_calc > (hctx->tfrchctx_x_recv >> 5))
| - hctx->tfrchctx_x_recv | - max(hctx->tfrchctx_x_recv / 2,
| - (((__u64)hctx->tfrchctx_s) << 6) /
| + if (hctx->ttx_x_calc > (hctx->ttx_x_recv >> 5))
| + hctx->ttx_x_recv = max(hctx->ttx_x_recv / 2,
| + (((__u64)hctx->ttx_s) << 6) /
| (2 * TFRC_T_MBI));
| else {
| - hctx->tfrchctx_x_recv = hctx->tfrchctx_x_calc;
| - hctx->tfrchctx_x_recv <<= 4;
| + hctx->ttx_x_recv = hctx->ttx_x_calc;
| + hctx->ttx_x_recv <<= 4;
| }
| ccid3_hc_tx_update_x(sk, NULL);
| }
| ccid3_pr_debug("Reduced X to %llu/64 bytes/sec\n",
| - (unsigned long long)hctx->tfrchctx_x);
| + (unsigned long long)hctx->ttx_x);
|
| /*
| * Set new timeout for the nofeedback timer.
| * See comments in packet_recv() regarding the value of t_RTO.
| */
| - if (unlikely(hctx->tfrchctx_t_rto = 0)) /* no feedback yet */
| + if (unlikely(hctx->ttx_t_rto = 0)) /* no feedback yet */
| t_nfb = TFRC_INITIAL_TIMEOUT;
| else
| - t_nfb = max(hctx->tfrchctx_t_rto, 2 * hctx->tfrchctx_t_ipi);
| + t_nfb = max(hctx->ttx_t_rto, 2 * hctx->ttx_t_ipi);
|
| restart_timer:
| - sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
| + sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
| jiffies + usecs_to_jiffies(t_nfb));
| out:
| bh_unlock_sock(sk);
| @@ -244,18 +238,18 @@ static int ccid3_hc_tx_send_packet(struc
| if (unlikely(skb->len = 0))
| return -EBADMSG;
|
| - switch (hctx->tfrchctx_state) {
| + switch (hctx->ttx_state) {
| case TTX_STATE_NO_SENT:
| - sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
| + sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
| (jiffies +
| usecs_to_jiffies(TFRC_INITIAL_TIMEOUT)));
| - hctx->tfrchctx_last_win_count = 0;
| - hctx->tfrchctx_t_last_win_count = now;
| + hctx->ttx_last_win_count = 0;
| + hctx->ttx_t_last_win_count = now;
|
| /* Set t_0 for initial packet */
| - hctx->tfrchctx_t_nom = now;
| + hctx->ttx_t_nom = now;
|
| - hctx->tfrchctx_s = skb->len;
| + hctx->ttx_s = skb->len;
|
| /*
| * Use initial RTT sample when available: recommended by erratum
| @@ -264,13 +258,13 @@ static int ccid3_hc_tx_send_packet(struc
| */
| if (dp->dccps_syn_rtt) {
| ccid3_pr_debug("SYN RTT = %uus\n", dp->dccps_syn_rtt);
| - hctx->tfrchctx_rtt = dp->dccps_syn_rtt;
| - hctx->tfrchctx_x = rfc3390_initial_rate(sk);
| - hctx->tfrchctx_t_ld = now;
| + hctx->ttx_rtt = dp->dccps_syn_rtt;
| + hctx->ttx_x = rfc3390_initial_rate(sk);
| + hctx->ttx_t_ld = now;
| } else {
| /* Sender does not have RTT sample: X_pps = 1 pkt/sec */
| - hctx->tfrchctx_x = hctx->tfrchctx_s;
| - hctx->tfrchctx_x <<= 6;
| + hctx->ttx_x = hctx->ttx_s;
| + hctx->ttx_x <<= 6;
| }
| ccid3_update_send_interval(hctx);
|
| @@ -278,7 +272,7 @@ static int ccid3_hc_tx_send_packet(struc
| break;
| case TTX_STATE_NO_FBACK:
| case TTX_STATE_FBACK:
| - delay = ktime_us_delta(hctx->tfrchctx_t_nom, now);
| + delay = ktime_us_delta(hctx->ttx_t_nom, now);
| ccid3_pr_debug("delay=%ld\n", (long)delay);
| /*
| * Scheduling of packet transmissions [RFC 3448, 4.6]
| @@ -288,7 +282,7 @@ static int ccid3_hc_tx_send_packet(struc
| * else
| * // send the packet in (t_nom - t_now) milliseconds.
| */
| - if (delay - (s64)hctx->tfrchctx_delta >= 1000)
| + if (delay - (s64)hctx->ttx_delta >= 1000)
| return (u32)delay / 1000L;
|
| tfrc_hc_tx_update_win_count(hctx, now);
| @@ -300,11 +294,10 @@ static int ccid3_hc_tx_send_packet(struc
|
| /* prepare to send now (add options etc.) */
| dp->dccps_hc_tx_insert_options = 1;
| - DCCP_SKB_CB(skb)->dccpd_ccval = hctx->tfrchctx_last_win_count;
| + DCCP_SKB_CB(skb)->dccpd_ccval = hctx->ttx_last_win_count;
|
| /* set the nominal send time for the next following packet */
| - hctx->tfrchctx_t_nom = ktime_add_us(hctx->tfrchctx_t_nom,
| - hctx->tfrchctx_t_ipi);
| + hctx->ttx_t_nom = ktime_add_us(hctx->ttx_t_nom, hctx->ttx_t_ipi);
| return 0;
| }
|
| @@ -315,7 +308,7 @@ static void ccid3_hc_tx_packet_sent(stru
|
| ccid3_hc_tx_update_s(hctx, len);
|
| - if (tfrc_tx_hist_add(&hctx->tfrchctx_hist, dccp_sk(sk)->dccps_gss))
| + if (tfrc_tx_hist_add(&hctx->ttx_hist, dccp_sk(sk)->dccps_gss))
| DCCP_CRIT("packet history - out of memory!");
| }
|
| @@ -332,15 +325,15 @@ static void ccid3_hc_tx_packet_recv(stru
| DCCP_SKB_CB(skb)->dccpd_type = DCCP_PKT_DATAACK))
| return;
| /* ... and only in the established state */
| - if (hctx->tfrchctx_state != TTX_STATE_FBACK &&
| - hctx->tfrchctx_state != TTX_STATE_NO_FBACK)
| + if (hctx->ttx_state != TTX_STATE_FBACK &&
| + hctx->ttx_state != TTX_STATE_NO_FBACK)
| return;
|
| - opt_recv = &hctx->tfrchctx_options_received;
| + opt_recv = &hctx->ttx_options_received;
| now = ktime_get_real();
|
| /* Estimate RTT from history if ACK number is valid */
| - r_sample = tfrc_tx_hist_rtt(hctx->tfrchctx_hist,
| + r_sample = tfrc_tx_hist_rtt(hctx->ttx_hist,
| DCCP_SKB_CB(skb)->dccpd_ack_seq, now);
| if (r_sample = 0) {
| DCCP_WARN("%s(%p): %s with bogus ACK-%llu\n", dccp_role(sk), sk,
| @@ -350,37 +343,37 @@ static void ccid3_hc_tx_packet_recv(stru
| }
|
| /* Update receive rate in units of 64 * bytes/second */
| - hctx->tfrchctx_x_recv = opt_recv->tfrcor_receive_rate;
| - hctx->tfrchctx_x_recv <<= 6;
| + hctx->ttx_x_recv = opt_recv->tfrcor_receive_rate;
| + hctx->ttx_x_recv <<= 6;
|
| /* Update loss event rate (which is scaled by 1e6) */
| pinv = opt_recv->tfrcor_loss_event_rate;
| if (pinv = ~0U || pinv = 0) /* see RFC 4342, 8.5 */
| - hctx->tfrchctx_p = 0;
| + hctx->ttx_p = 0;
| else /* can not exceed 100% */
| - hctx->tfrchctx_p = scaled_div(1, pinv);
| + hctx->ttx_p = scaled_div(1, pinv);
| /*
| * Validate new RTT sample and update moving average
| */
| r_sample = dccp_sample_rtt(sk, r_sample);
| - hctx->tfrchctx_rtt = tfrc_ewma(hctx->tfrchctx_rtt, r_sample, 9);
| + hctx->ttx_rtt = tfrc_ewma(hctx->ttx_rtt, r_sample, 9);
| /*
| * Update allowed sending rate X as per draft rfc3448bis-00, 4.2/3
| */
| - if (hctx->tfrchctx_state = TTX_STATE_NO_FBACK) {
| + if (hctx->ttx_state = TTX_STATE_NO_FBACK) {
| ccid3_hc_tx_set_state(sk, TTX_STATE_FBACK);
|
| - if (hctx->tfrchctx_t_rto = 0) {
| + if (hctx->ttx_t_rto = 0) {
| /*
| * Initial feedback packet: Larger Initial Windows (4.2)
| */
| - hctx->tfrchctx_x = rfc3390_initial_rate(sk);
| - hctx->tfrchctx_t_ld = now;
| + hctx->ttx_x = rfc3390_initial_rate(sk);
| + hctx->ttx_t_ld = now;
|
| ccid3_update_send_interval(hctx);
|
| goto done_computing_x;
| - } else if (hctx->tfrchctx_p = 0) {
| + } else if (hctx->ttx_p = 0) {
| /*
| * First feedback after nofeedback timer expiry (4.3)
| */
| @@ -389,25 +382,23 @@ static void ccid3_hc_tx_packet_recv(stru
| }
|
| /* Update sending rate (step 4 of [RFC 3448, 4.3]) */
| - if (hctx->tfrchctx_p > 0)
| - hctx->tfrchctx_x_calc | - tfrc_calc_x(hctx->tfrchctx_s,
| - hctx->tfrchctx_rtt,
| - hctx->tfrchctx_p);
| + if (hctx->ttx_p > 0)
| + hctx->ttx_x_calc = tfrc_calc_x(hctx->ttx_s, hctx->ttx_rtt,
| + hctx->ttx_p);
| ccid3_hc_tx_update_x(sk, &now);
|
| done_computing_x:
| ccid3_pr_debug("%s(%p), RTT=%uus (sample=%uus), s=%u, "
| "p=%u, X_calc=%u, X_recv=%u, X=%u\n",
| dccp_role(sk),
| - sk, hctx->tfrchctx_rtt, r_sample,
| - hctx->tfrchctx_s, hctx->tfrchctx_p,
| - hctx->tfrchctx_x_calc,
| - (unsigned)(hctx->tfrchctx_x_recv >> 6),
| - (unsigned)(hctx->tfrchctx_x >> 6));
| + sk, hctx->ttx_rtt, r_sample,
| + hctx->ttx_s, hctx->ttx_p,
| + hctx->ttx_x_calc,
| + (unsigned)(hctx->ttx_x_recv >> 6),
| + (unsigned)(hctx->ttx_x >> 6));
|
| /* unschedule no feedback timer */
| - sk_stop_timer(sk, &hctx->tfrchctx_no_feedback_timer);
| + sk_stop_timer(sk, &hctx->ttx_no_feedback_timer);
|
| /*
| * As we have calculated new ipi, delta, t_nom it is possible
| @@ -421,21 +412,21 @@ done_computing_x:
| * This can help avoid triggering the nofeedback timer too
| * often ('spinning') on LANs with small RTTs.
| */
| - hctx->tfrchctx_t_rto = max_t(u32, 4 * hctx->tfrchctx_rtt,
| + hctx->ttx_t_rto = max_t(u32, 4 * hctx->ttx_rtt,
| (CONFIG_IP_DCCP_CCID3_RTO *
| (USEC_PER_SEC / 1000)));
| /*
| * Schedule no feedback timer to expire in
| * max(t_RTO, 2 * s/X) = max(t_RTO, 2 * t_ipi)
| */
| - t_nfb = max(hctx->tfrchctx_t_rto, 2 * hctx->tfrchctx_t_ipi);
| + t_nfb = max(hctx->ttx_t_rto, 2 * hctx->ttx_t_ipi);
|
| ccid3_pr_debug("%s(%p), Scheduled no feedback timer to "
| "expire in %lu jiffies (%luus)\n",
| dccp_role(sk),
| sk, usecs_to_jiffies(t_nfb), t_nfb);
|
| - sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
| + sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
| jiffies + usecs_to_jiffies(t_nfb));
| }
|
| @@ -449,7 +440,7 @@ static int ccid3_hc_tx_parse_options(str
| struct tfrc_options_received *opt_recv;
| __be32 opt_val;
|
| - opt_recv = &hctx->tfrchctx_options_received;
| + opt_recv = &hctx->ttx_options_received;
|
| if (opt_recv->tfrcor_seqno != dp->dccps_gsr) {
| opt_recv->tfrcor_seqno = dp->dccps_gsr;
| @@ -505,10 +496,10 @@ static int ccid3_hc_tx_init(struct ccid
| {
| struct tfrc_hc_tx_sock *hctx = ccid_priv(ccid);
|
| - hctx->tfrchctx_state = TTX_STATE_NO_SENT;
| - hctx->tfrchctx_hist = NULL;
| - setup_timer(&hctx->tfrchctx_no_feedback_timer,
| - ccid3_hc_tx_no_feedback_timer, (unsigned long)sk);
| + hctx->ttx_state = TTX_STATE_NO_SENT;
| + hctx->ttx_hist = NULL;
| + setup_timer(&hctx->ttx_no_feedback_timer, ccid3_hc_tx_no_feedback_timer,
| + (unsigned long)sk);
|
| return 0;
| }
| @@ -518,15 +509,15 @@ static void ccid3_hc_tx_exit(struct sock
| struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
|
| ccid3_hc_tx_set_state(sk, TTX_STATE_TERM);
| - sk_stop_timer(sk, &hctx->tfrchctx_no_feedback_timer);
| + sk_stop_timer(sk, &hctx->ttx_no_feedback_timer);
|
| - tfrc_tx_hist_purge(&hctx->tfrchctx_hist);
| + tfrc_tx_hist_purge(&hctx->ttx_hist);
| }
|
| static void ccid3_hc_tx_get_info(struct sock *sk, struct tcp_info *info)
| {
| - info->tcpi_rto = tfrc_hc_tx_sk(sk)->tfrchctx_t_rto;
| - info->tcpi_rtt = tfrc_hc_tx_sk(sk)->tfrchctx_rtt;
| + info->tcpi_rto = tfrc_hc_tx_sk(sk)->ttx_t_rto;
| + info->tcpi_rtt = tfrc_hc_tx_sk(sk)->ttx_rtt;
| }
|
| static int ccid3_hc_tx_getsockopt(struct sock *sk, const int optname, int len,
| @@ -537,10 +528,10 @@ static int ccid3_hc_tx_getsockopt(struct
|
| switch (optname) {
| case DCCP_SOCKOPT_CCID_TX_INFO:
| - if (len < sizeof(hctx->tfrchctx_tfrc))
| + if (len < sizeof(hctx->ttx_tfrc))
| return -EINVAL;
| - len = sizeof(hctx->tfrchctx_tfrc);
| - val = &hctx->tfrchctx_tfrc;
| + len = sizeof(hctx->ttx_tfrc);
| + val = &hctx->ttx_tfrc;
| break;
| default:
| return -ENOPROTOOPT;
| @@ -556,13 +547,13 @@ static void ccid3_hc_rx_set_state(struct
| enum tfrc_hc_rx_states state)
| {
| struct tfrc_hc_rx_sock *hcrx = tfrc_hc_rx_sk(sk);
| - enum tfrc_hc_rx_states oldstate = hcrx->tfrchcrx_state;
| + enum tfrc_hc_rx_states oldstate = hcrx->trx_state;
|
| ccid3_pr_debug("%s(%p) %-8.8s -> %s\n",
| dccp_role(sk), sk, tfrc_rx_state_name(oldstate),
| tfrc_rx_state_name(state));
| WARN_ON(state = oldstate);
| - hcrx->tfrchcrx_state = state;
| + hcrx->trx_state = state;
| }
|
| static void ccid3_hc_rx_send_feedback(struct sock *sk,
| @@ -574,15 +565,15 @@ static void ccid3_hc_rx_send_feedback(st
| ktime_t now;
| s64 delta = 0;
|
| - if (unlikely(hcrx->tfrchcrx_state = TRX_STATE_TERM))
| + if (unlikely(hcrx->trx_state = TRX_STATE_TERM))
| return;
|
| now = ktime_get_real();
|
| switch (fbtype) {
| case TFRC_FBACK_INITIAL:
| - hcrx->tfrchcrx_x_recv = 0;
| - hcrx->tfrchcrx_pinv = ~0U; /* see RFC 4342, 8.5 */
| + hcrx->trx_x_recv = 0;
| + hcrx->trx_pinv = ~0U; /* see RFC 4342, 8.5 */
| break;
| case TFRC_FBACK_PARAM_CHANGE:
| /*
| @@ -595,27 +586,27 @@ static void ccid3_hc_rx_send_feedback(st
| * the number of bytes since last feedback.
| * This is a safe fallback, since X is bounded above by X_calc.
| */
| - if (hcrx->tfrchcrx_x_recv > 0)
| + if (hcrx->trx_x_recv > 0)
| break;
| /* fall through */
| case TFRC_FBACK_PERIODIC:
| - delta = ktime_us_delta(now, hcrx->tfrchcrx_last_feedback);
| + delta = ktime_us_delta(now, hcrx->trx_last_feedback);
| if (delta <= 0)
| DCCP_BUG("delta (%ld) <= 0", (long)delta);
| else
| - hcrx->tfrchcrx_x_recv | - scaled_div32(hcrx->tfrchcrx_bytes_recv, delta);
| + hcrx->trx_x_recv | + scaled_div32(hcrx->trx_bytes_recv, delta);
| break;
| default:
| return;
| }
|
| ccid3_pr_debug("Interval %ldusec, X_recv=%u, 1/p=%u\n", (long)delta,
| - hcrx->tfrchcrx_x_recv, hcrx->tfrchcrx_pinv);
| + hcrx->trx_x_recv, hcrx->trx_pinv);
|
| - hcrx->tfrchcrx_last_feedback = now;
| - hcrx->tfrchcrx_last_counter = dccp_hdr(skb)->dccph_ccval;
| - hcrx->tfrchcrx_bytes_recv = 0;
| + hcrx->trx_last_feedback = now;
| + hcrx->trx_last_counter = dccp_hdr(skb)->dccph_ccval;
| + hcrx->trx_bytes_recv = 0;
|
| dp->dccps_hc_rx_insert_options = 1;
| dccp_send_ack(sk);
| @@ -632,8 +623,8 @@ static int ccid3_hc_rx_insert_options(st
| if (dccp_packet_without_ack(skb))
| return 0;
|
| - x_recv = htonl(hcrx->tfrchcrx_x_recv);
| - pinv = htonl(hcrx->tfrchcrx_pinv);
| + x_recv = htonl(hcrx->trx_x_recv);
| + pinv = htonl(hcrx->trx_pinv);
|
| if (dccp_insert_option(sk, skb, TFRC_OPT_LOSS_EVENT_RATE,
| &pinv, sizeof(pinv)) ||
| @@ -659,22 +650,22 @@ static u32 ccid3_first_li(struct sock *s
| u32 x_recv, p, delta;
| u64 fval;
|
| - if (hcrx->tfrchcrx_rtt = 0) {
| + if (hcrx->trx_rtt = 0) {
| DCCP_WARN("No RTT estimate available, using fallback RTT\n");
| - hcrx->tfrchcrx_rtt = DCCP_FALLBACK_RTT;
| + hcrx->trx_rtt = DCCP_FALLBACK_RTT;
| }
|
| - delta = ktime_to_us(net_timedelta(hcrx->tfrchcrx_last_feedback));
| - x_recv = scaled_div32(hcrx->tfrchcrx_bytes_recv, delta);
| + delta = ktime_to_us(net_timedelta(hcrx->trx_last_feedback));
| + x_recv = scaled_div32(hcrx->trx_bytes_recv, delta);
| if (x_recv = 0) { /* would also trigger divide-by-zero */
| DCCP_WARN("X_recv=0\n");
| - if ((x_recv = hcrx->tfrchcrx_x_recv) = 0) {
| + if ((x_recv = hcrx->trx_x_recv) = 0) {
| DCCP_BUG("stored value of X_recv is zero");
| return ~0U;
| }
| }
|
| - fval = scaled_div(hcrx->tfrchcrx_s, hcrx->tfrchcrx_rtt);
| + fval = scaled_div(hcrx->trx_s, hcrx->trx_rtt);
| fval = scaled_div32(fval, x_recv);
| p = tfrc_calc_x_reverse_lookup(fval);
|
| @@ -691,14 +682,14 @@ static void ccid3_hc_rx_packet_recv(stru
| const u64 ndp = dccp_sk(sk)->dccps_options_received.dccpor_ndp;
| const bool is_data_packet = dccp_data_packet(skb);
|
| - if (unlikely(hcrx->tfrchcrx_state = TRX_STATE_NO_DATA)) {
| + if (unlikely(hcrx->trx_state = TRX_STATE_NO_DATA)) {
| if (is_data_packet) {
| const u32 payload = skb->len - dccp_hdr(skb)->dccph_doff * 4;
| do_feedback = TFRC_FBACK_INITIAL;
| ccid3_hc_rx_set_state(sk, TRX_STATE_DATA);
| - hcrx->tfrchcrx_s = payload;
| + hcrx->trx_s = payload;
| /*
| - * Not necessary to update tfrchcrx_bytes_recv here,
| + * Not necessary to update trx_bytes_recv here,
| * since X_recv = 0 for the first feedback packet (cf.
| * RFC 3448, 6.3) -- gerrit
| */
| @@ -706,7 +697,7 @@ static void ccid3_hc_rx_packet_recv(stru
| goto update_records;
| }
|
| - if (tfrc_rx_hist_duplicate(&hcrx->tfrchcrx_hist, skb))
| + if (tfrc_rx_hist_duplicate(&hcrx->trx_hist, skb))
| return; /* done receiving */
|
| if (is_data_packet) {
| @@ -714,22 +705,21 @@ static void ccid3_hc_rx_packet_recv(stru
| /*
| * Update moving-average of s and the sum of received payload bytes
| */
| - hcrx->tfrchcrx_s = tfrc_ewma(hcrx->tfrchcrx_s, payload, 9);
| - hcrx->tfrchcrx_bytes_recv += payload;
| + hcrx->trx_s = tfrc_ewma(hcrx->trx_s, payload, 9);
| + hcrx->trx_bytes_recv += payload;
| }
|
| /*
| * Handle pending losses and otherwise check for new loss
| */
| - if (tfrc_rx_hist_loss_pending(&hcrx->tfrchcrx_hist) &&
| - tfrc_rx_handle_loss(&hcrx->tfrchcrx_hist,
| - &hcrx->tfrchcrx_li_hist,
| + if (tfrc_rx_hist_loss_pending(&hcrx->trx_hist) &&
| + tfrc_rx_handle_loss(&hcrx->trx_hist, &hcrx->trx_li_hist,
| skb, ndp, ccid3_first_li, sk) ) {
| do_feedback = TFRC_FBACK_PARAM_CHANGE;
| goto done_receiving;
| }
|
| - if (tfrc_rx_hist_new_loss_indicated(&hcrx->tfrchcrx_hist, skb, ndp))
| + if (tfrc_rx_hist_new_loss_indicated(&hcrx->trx_hist, skb, ndp))
| goto update_records;
|
| /*
| @@ -738,17 +728,17 @@ static void ccid3_hc_rx_packet_recv(stru
| if (unlikely(!is_data_packet))
| goto update_records;
|
| - if (!tfrc_lh_is_initialised(&hcrx->tfrchcrx_li_hist)) {
| - const u32 sample = tfrc_rx_hist_sample_rtt(&hcrx->tfrchcrx_hist, skb);
| + if (!tfrc_lh_is_initialised(&hcrx->trx_li_hist)) {
| + const u32 sample = tfrc_rx_hist_sample_rtt(&hcrx->trx_hist, skb);
| /*
| * Empty loss history: no loss so far, hence p stays 0.
| * Sample RTT values, since an RTT estimate is required for the
| * computation of p when the first loss occurs; RFC 3448, 6.3.1.
| */
| if (sample != 0)
| - hcrx->tfrchcrx_rtt = tfrc_ewma(hcrx->tfrchcrx_rtt, sample, 9);
| + hcrx->trx_rtt = tfrc_ewma(hcrx->trx_rtt, sample, 9);
|
| - } else if (tfrc_lh_update_i_mean(&hcrx->tfrchcrx_li_hist, skb)) {
| + } else if (tfrc_lh_update_i_mean(&hcrx->trx_li_hist, skb)) {
| /*
| * Step (3) of [RFC 3448, 6.1]: Recompute I_mean and, if I_mean
| * has decreased (resp. p has increased), send feedback now.
| @@ -759,11 +749,11 @@ static void ccid3_hc_rx_packet_recv(stru
| /*
| * Check if the periodic once-per-RTT feedback is due; RFC 4342, 10.3
| */
| - if (SUB16(dccp_hdr(skb)->dccph_ccval, hcrx->tfrchcrx_last_counter) > 3)
| + if (SUB16(dccp_hdr(skb)->dccph_ccval, hcrx->trx_last_counter) > 3)
| do_feedback = TFRC_FBACK_PERIODIC;
|
| update_records:
| - tfrc_rx_hist_add_packet(&hcrx->tfrchcrx_hist, skb, ndp);
| + tfrc_rx_hist_add_packet(&hcrx->trx_hist, skb, ndp);
|
| done_receiving:
| if (do_feedback)
| @@ -774,9 +764,9 @@ static int ccid3_hc_rx_init(struct ccid
| {
| struct tfrc_hc_rx_sock *hcrx = ccid_priv(ccid);
|
| - hcrx->tfrchcrx_state = TRX_STATE_NO_DATA;
| - tfrc_lh_init(&hcrx->tfrchcrx_li_hist);
| - return tfrc_rx_hist_alloc(&hcrx->tfrchcrx_hist);
| + hcrx->trx_state = TRX_STATE_NO_DATA;
| + tfrc_lh_init(&hcrx->trx_li_hist);
| + return tfrc_rx_hist_alloc(&hcrx->trx_hist);
| }
|
| static void ccid3_hc_rx_exit(struct sock *sk)
| @@ -785,15 +775,15 @@ static void ccid3_hc_rx_exit(struct sock
|
| ccid3_hc_rx_set_state(sk, TRX_STATE_TERM);
|
| - tfrc_rx_hist_purge(&hcrx->tfrchcrx_hist);
| - tfrc_lh_cleanup(&hcrx->tfrchcrx_li_hist);
| + tfrc_rx_hist_purge(&hcrx->trx_hist);
| + tfrc_lh_cleanup(&hcrx->trx_li_hist);
| }
|
| static void ccid3_hc_rx_get_info(struct sock *sk, struct tcp_info *info)
| {
| - info->tcpi_ca_state = tfrc_hc_rx_sk(sk)->tfrchcrx_state;
| + info->tcpi_ca_state = tfrc_hc_rx_sk(sk)->trx_state;
| info->tcpi_options |= TCPI_OPT_TIMESTAMPS;
| - info->tcpi_rcv_rtt = tfrc_hc_rx_sk(sk)->tfrchcrx_rtt;
| + info->tcpi_rcv_rtt = tfrc_hc_rx_sk(sk)->trx_rtt;
| }
|
| static int ccid3_hc_rx_getsockopt(struct sock *sk, const int optname, int len,
| @@ -807,10 +797,10 @@ static int ccid3_hc_rx_getsockopt(struct
| case DCCP_SOCKOPT_CCID_RX_INFO:
| if (len < sizeof(rx_info))
| return -EINVAL;
| - rx_info.tfrcrx_x_recv = hcrx->tfrchcrx_x_recv;
| - rx_info.tfrcrx_rtt = hcrx->tfrchcrx_rtt;
| - rx_info.tfrcrx_p = hcrx->tfrchcrx_pinv = 0 ? ~0U :
| - scaled_div(1, hcrx->tfrchcrx_pinv);
| + rx_info.tfrcrx_x_recv = hcrx->trx_x_recv;
| + rx_info.tfrcrx_rtt = hcrx->trx_rtt;
| + rx_info.tfrcrx_p = hcrx->trx_pinv = 0 ? ~0U :
| + scaled_div(1, hcrx->trx_pinv);
| len = sizeof(rx_info);
| val = &rx_info;
| break;
| Index: ccid4.latest/net/dccp/ccids/ccid4.c
| =================================| --- ccid4.latest.orig/net/dccp/ccids/ccid4.c
| +++ ccid4.latest/net/dccp/ccids/ccid4.c
| @@ -58,13 +58,13 @@ static void ccid4_hc_tx_set_state(struct
| enum tfrc_hc_tx_states state)
| {
| struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
| - enum tfrc_hc_tx_states oldstate = hctx->tfrchctx_state;
| + enum tfrc_hc_tx_states oldstate = hctx->ttx_state;
|
| ccid4_pr_debug("%s(%p) %-8.8s -> %s\n",
| dccp_role(sk), sk, tfrc_tx_state_name(oldstate),
| tfrc_tx_state_name(state));
| WARN_ON(state = oldstate);
| - hctx->tfrchctx_state = state;
| + hctx->ttx_state = state;
| }
|
| /*
| @@ -76,17 +76,17 @@ static inline void ccid4_update_send_int
| * Calculate new t_ipi = s / X_inst (X_inst is in 64 * bytes/second).
| * TFRC-SP enforces a minimum interval of 10 milliseconds.
| */
| - hctx->tfrchctx_t_ipi | - max_t(u32, scaled_div32(((u64)hctx->tfrchctx_s) << 6,
| - hctx->tfrchctx_x), MIN_SEND_RATE);
| + hctx->ttx_t_ipi | + max_t(u32, scaled_div32(((u64)hctx->ttx_s) << 6,
| + hctx->ttx_x), MIN_SEND_RATE);
|
| /* Calculate new delta by delta = min(t_ipi / 2, t_gran / 2) */
| - hctx->tfrchctx_delta = min_t(u32, hctx->tfrchctx_t_ipi / 2,
| + hctx->ttx_delta = min_t(u32, hctx->ttx_t_ipi / 2,
| TFRC_OPSYS_HALF_TIME_GRAN);
|
| ccid4_pr_debug("t_ipi=%u, delta=%u, s=%u, X=%u\n",
| - hctx->tfrchctx_t_ipi, hctx->tfrchctx_delta,
| - hctx->tfrchctx_s, (unsigned)(hctx->tfrchctx_x >> 6));
| + hctx->ttx_t_ipi, hctx->ttx_delta,
| + hctx->ttx_s, (unsigned)(hctx->ttx_x >> 6));
|
| }
|
| @@ -97,8 +97,8 @@ static inline void ccid4_update_send_int
| */
| static inline void ccid4_hc_tx_x_header_penalty(struct tfrc_hc_tx_sock *hctx)
| {
| - hctx->tfrchctx_x *= hctx->tfrchctx_s;
| - do_div(hctx->tfrchctx_x, (hctx->tfrchctx_s + CCID4HCTX_H));
| + hctx->ttx_x *= hctx->ttx_s;
| + do_div(hctx->ttx_x, (hctx->ttx_s + CCID4HCTX_H));
| }
|
| /**
| @@ -114,8 +114,8 @@ static inline void ccid4_hc_tx_x_header_
| static void ccid4_hc_tx_update_x(struct sock *sk, ktime_t *stamp)
| {
| struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
| - __u64 min_rate = 2 * hctx->tfrchctx_x_recv;
| - const __u64 old_x = hctx->tfrchctx_x;
| + __u64 min_rate = 2 * hctx->ttx_x_recv;
| + const __u64 old_x = hctx->ttx_x;
| ktime_t now = stamp ? *stamp : ktime_get_real();
|
| /*
| @@ -126,36 +126,36 @@ static void ccid4_hc_tx_update_x(struct
| */
| if (tfrc_hc_tx_idle_rtt(hctx, now) >= 2) {
| min_rate = rfc3390_initial_rate(sk);
| - min_rate = max(min_rate, 2 * hctx->tfrchctx_x_recv);
| + min_rate = max(min_rate, 2 * hctx->ttx_x_recv);
| }
|
| - if (hctx->tfrchctx_p > 0) {
| + if (hctx->ttx_p > 0) {
|
| - hctx->tfrchctx_x = min(((__u64)hctx->tfrchctx_x_calc) << 6,
| + hctx->ttx_x = min(((__u64)hctx->ttx_x_calc) << 6,
| min_rate);
| - hctx->tfrchctx_x = max(hctx->tfrchctx_x,
| - (((__u64)hctx->tfrchctx_s) << 6) /
| + hctx->ttx_x = max(hctx->ttx_x,
| + (((__u64)hctx->ttx_s) << 6) /
| TFRC_T_MBI);
| ccid4_hc_tx_x_header_penalty(hctx);
|
| - } else if (ktime_us_delta(now, hctx->tfrchctx_t_ld)
| - - (s64)hctx->tfrchctx_rtt >= 0) {
| + } else if (ktime_us_delta(now, hctx->ttx_t_ld)
| + - (s64)hctx->ttx_rtt >= 0) {
|
| - hctx->tfrchctx_x | - max(min(2 * hctx->tfrchctx_x, min_rate),
| - scaled_div(((__u64)hctx->tfrchctx_s) << 6,
| - hctx->tfrchctx_rtt));
| + hctx->ttx_x | + max(min(2 * hctx->ttx_x, min_rate),
| + scaled_div(((__u64)hctx->ttx_s) << 6,
| + hctx->ttx_rtt));
|
| ccid4_hc_tx_x_header_penalty(hctx);
| - hctx->tfrchctx_t_ld = now;
| + hctx->ttx_t_ld = now;
| }
|
| - if (hctx->tfrchctx_x != old_x) {
| + if (hctx->ttx_x != old_x) {
| ccid4_pr_debug("X_prev=%u, X_now=%u, X_calc=%u, "
| "X_recv=%u\n", (unsigned)(old_x >> 6),
| - (unsigned)(hctx->tfrchctx_x >> 6),
| - hctx->tfrchctx_x_calc,
| - (unsigned)(hctx->tfrchctx_x_recv >> 6));
| + (unsigned)(hctx->ttx_x >> 6),
| + hctx->ttx_x_calc,
| + (unsigned)(hctx->ttx_x_recv >> 6));
|
| ccid4_update_send_interval(hctx);
| }
| @@ -167,11 +167,11 @@ static void ccid4_hc_tx_update_x(struct
| */
| static inline void ccid4_hc_tx_update_s(struct tfrc_hc_tx_sock *hctx, int len)
| {
| - const u16 old_s = hctx->tfrchctx_s;
| + const u16 old_s = hctx->ttx_s;
|
| - hctx->tfrchctx_s = tfrc_ewma(hctx->tfrchctx_s, len, 9);
| + hctx->ttx_s = tfrc_ewma(hctx->ttx_s, len, 9);
|
| - if (hctx->tfrchctx_s != old_s)
| + if (hctx->ttx_s != old_s)
| ccid4_update_send_interval(hctx);
| }
|
| @@ -189,22 +189,22 @@ static void ccid4_hc_tx_no_feedback_time
| }
|
| ccid4_pr_debug("%s(%p, state=%s) - entry \n", dccp_role(sk), sk,
| - tfrc_tx_state_name(hctx->tfrchctx_state));
| + tfrc_tx_state_name(hctx->ttx_state));
|
| - if (hctx->tfrchctx_state = TTX_STATE_FBACK)
| + if (hctx->ttx_state = TTX_STATE_FBACK)
| ccid4_hc_tx_set_state(sk, TTX_STATE_NO_FBACK);
| - else if (hctx->tfrchctx_state != TTX_STATE_NO_FBACK)
| + else if (hctx->ttx_state != TTX_STATE_NO_FBACK)
| goto out;
|
| /*
| * Determine new allowed sending rate X as per draft rfc3448bis-00, 4.4
| */
| - if (hctx->tfrchctx_t_rto = 0 || /* no feedback received yet */
| - hctx->tfrchctx_p = 0) {
| + if (hctx->ttx_t_rto = 0 || /* no feedback received yet */
| + hctx->ttx_p = 0) {
|
| /* halve send rate directly */
| - hctx->tfrchctx_x = max(hctx->tfrchctx_x / 2,
| - (((__u64)hctx->tfrchctx_s) << 6) /
| + hctx->ttx_x = max(hctx->ttx_x / 2,
| + (((__u64)hctx->ttx_s) << 6) /
| TFRC_T_MBI);
| ccid4_update_send_interval(hctx);
| } else {
| @@ -218,33 +218,33 @@ static void ccid4_hc_tx_no_feedback_time
| *
| * Note that X_recv is scaled by 2^6 while X_calc is not
| */
| - BUG_ON(hctx->tfrchctx_p && !hctx->tfrchctx_x_calc);
| + BUG_ON(hctx->ttx_p && !hctx->ttx_x_calc);
|
| - if (hctx->tfrchctx_x_calc > (hctx->tfrchctx_x_recv >> 5))
| - hctx->tfrchctx_x_recv | - max(hctx->tfrchctx_x_recv / 2,
| - (((__u64)hctx->tfrchctx_s) << 6) /
| + if (hctx->ttx_x_calc > (hctx->ttx_x_recv >> 5))
| + hctx->ttx_x_recv | + max(hctx->ttx_x_recv / 2,
| + (((__u64)hctx->ttx_s) << 6) /
| (2 * TFRC_T_MBI));
| else {
| - hctx->tfrchctx_x_recv = hctx->tfrchctx_x_calc;
| - hctx->tfrchctx_x_recv <<= 4;
| + hctx->ttx_x_recv = hctx->ttx_x_calc;
| + hctx->ttx_x_recv <<= 4;
| }
| ccid4_hc_tx_update_x(sk, NULL);
| }
| ccid4_pr_debug("Reduced X to %llu/64 bytes/sec\n",
| - (unsigned long long)hctx->tfrchctx_x);
| + (unsigned long long)hctx->ttx_x);
|
| /*
| * Set new timeout for the nofeedback timer.
| * See comments in packet_recv() regarding the value of t_RTO.
| */
| - if (unlikely(hctx->tfrchctx_t_rto = 0)) /* no feedback yet */
| + if (unlikely(hctx->ttx_t_rto = 0)) /* no feedback yet */
| t_nfb = TFRC_INITIAL_TIMEOUT;
| else
| - t_nfb = max(hctx->tfrchctx_t_rto, 2 * hctx->tfrchctx_t_ipi);
| + t_nfb = max(hctx->ttx_t_rto, 2 * hctx->ttx_t_ipi);
|
| restart_timer:
| - sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
| + sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
| jiffies + usecs_to_jiffies(t_nfb));
| out:
| bh_unlock_sock(sk);
| @@ -272,18 +272,18 @@ static int ccid4_hc_tx_send_packet(struc
| if (unlikely(skb->len = 0))
| return -EBADMSG;
|
| - switch (hctx->tfrchctx_state) {
| + switch (hctx->ttx_state) {
| case TTX_STATE_NO_SENT:
| - sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
| + sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
| (jiffies +
| usecs_to_jiffies(TFRC_INITIAL_TIMEOUT)));
| - hctx->tfrchctx_last_win_count = 0;
| - hctx->tfrchctx_t_last_win_count = now;
| + hctx->ttx_last_win_count = 0;
| + hctx->ttx_t_last_win_count = now;
|
| /* Set t_0 for initial packet */
| - hctx->tfrchctx_t_nom = now;
| + hctx->ttx_t_nom = now;
|
| - hctx->tfrchctx_s = skb->len;
| + hctx->ttx_s = skb->len;
|
| /*
| * Use initial RTT sample when available: recommended by erratum
| @@ -292,13 +292,13 @@ static int ccid4_hc_tx_send_packet(struc
| */
| if (dp->dccps_syn_rtt) {
| ccid4_pr_debug("SYN RTT = %uus\n", dp->dccps_syn_rtt);
| - hctx->tfrchctx_rtt = dp->dccps_syn_rtt;
| - hctx->tfrchctx_x = rfc3390_initial_rate(sk);
| - hctx->tfrchctx_t_ld = now;
| + hctx->ttx_rtt = dp->dccps_syn_rtt;
| + hctx->ttx_x = rfc3390_initial_rate(sk);
| + hctx->ttx_t_ld = now;
| } else {
| /* Sender does not have RTT sample: X_pps = 1 pkt/sec */
| - hctx->tfrchctx_x = hctx->tfrchctx_s;
| - hctx->tfrchctx_x <<= 6;
| + hctx->ttx_x = hctx->ttx_s;
| + hctx->ttx_x <<= 6;
| }
| ccid4_update_send_interval(hctx);
|
| @@ -306,7 +306,7 @@ static int ccid4_hc_tx_send_packet(struc
| break;
| case TTX_STATE_NO_FBACK:
| case TTX_STATE_FBACK:
| - delay = ktime_us_delta(hctx->tfrchctx_t_nom, now);
| + delay = ktime_us_delta(hctx->ttx_t_nom, now);
| ccid4_pr_debug("delay=%ld\n", (long)delay);
| /*
| * Scheduling of packet transmissions [RFC 3448, 4.6]
| @@ -316,7 +316,7 @@ static int ccid4_hc_tx_send_packet(struc
| * else
| * // send the packet in (t_nom - t_now) milliseconds.
| */
| - if (delay - (s64)hctx->tfrchctx_delta >= 1000)
| + if (delay - (s64)hctx->ttx_delta >= 1000)
| return (u32)delay / 1000L;
|
| tfrc_hc_tx_update_win_count(hctx, now);
| @@ -328,11 +328,11 @@ static int ccid4_hc_tx_send_packet(struc
|
| /* prepare to send now (add options etc.) */
| dp->dccps_hc_tx_insert_options = 1;
| - DCCP_SKB_CB(skb)->dccpd_ccval = hctx->tfrchctx_last_win_count;
| + DCCP_SKB_CB(skb)->dccpd_ccval = hctx->ttx_last_win_count;
|
| /* set the nominal send time for the next following packet */
| - hctx->tfrchctx_t_nom = ktime_add_us(hctx->tfrchctx_t_nom,
| - hctx->tfrchctx_t_ipi);
| + hctx->ttx_t_nom = ktime_add_us(hctx->ttx_t_nom,
| + hctx->ttx_t_ipi);
| return 0;
| }
|
| @@ -343,7 +343,7 @@ static void ccid4_hc_tx_packet_sent(stru
|
| ccid4_hc_tx_update_s(hctx, len);
|
| - if (tfrc_tx_hist_add(&hctx->tfrchctx_hist, dccp_sk(sk)->dccps_gss))
| + if (tfrc_tx_hist_add(&hctx->ttx_hist, dccp_sk(sk)->dccps_gss))
| DCCP_CRIT("packet history - out of memory!");
| }
|
| @@ -360,15 +360,15 @@ static void ccid4_hc_tx_packet_recv(stru
| DCCP_SKB_CB(skb)->dccpd_type = DCCP_PKT_DATAACK))
| return;
| /* ... and only in the established state */
| - if (hctx->tfrchctx_state != TTX_STATE_FBACK &&
| - hctx->tfrchctx_state != TTX_STATE_NO_FBACK)
| + if (hctx->ttx_state != TTX_STATE_FBACK &&
| + hctx->ttx_state != TTX_STATE_NO_FBACK)
| return;
|
| - opt_recv = &hctx->tfrchctx_options_received;
| + opt_recv = &hctx->ttx_options_received;
| now = ktime_get_real();
|
| /* Estimate RTT from history if ACK number is valid */
| - r_sample = tfrc_tx_hist_rtt(hctx->tfrchctx_hist,
| + r_sample = tfrc_tx_hist_rtt(hctx->ttx_hist,
| DCCP_SKB_CB(skb)->dccpd_ack_seq, now);
| if (r_sample = 0) {
| DCCP_WARN("%s(%p): %s with bogus ACK-%llu\n", dccp_role(sk), sk,
| @@ -378,37 +378,37 @@ static void ccid4_hc_tx_packet_recv(stru
| }
|
| /* Update receive rate in units of 64 * bytes/second */
| - hctx->tfrchctx_x_recv = opt_recv->tfrcor_receive_rate;
| - hctx->tfrchctx_x_recv <<= 6;
| + hctx->ttx_x_recv = opt_recv->tfrcor_receive_rate;
| + hctx->ttx_x_recv <<= 6;
|
| /* Update loss event rate (which is scaled by 1e6) */
| pinv = opt_recv->tfrcor_loss_event_rate;
| if (pinv = ~0U || pinv = 0) /* see RFC 4342, 8.5 */
| - hctx->tfrchctx_p = 0;
| + hctx->ttx_p = 0;
| else /* can not exceed 100% */
| - hctx->tfrchctx_p = scaled_div(1, pinv);
| + hctx->ttx_p = scaled_div(1, pinv);
| /*
| * Validate new RTT sample and update moving average
| */
| r_sample = dccp_sample_rtt(sk, r_sample);
| - hctx->tfrchctx_rtt = tfrc_ewma(hctx->tfrchctx_rtt, r_sample, 9);
| + hctx->ttx_rtt = tfrc_ewma(hctx->ttx_rtt, r_sample, 9);
| /*
| * Update allowed sending rate X as per draft rfc3448bis-00, 4.2/3
| */
| - if (hctx->tfrchctx_state = TTX_STATE_NO_FBACK) {
| + if (hctx->ttx_state = TTX_STATE_NO_FBACK) {
| ccid4_hc_tx_set_state(sk, TTX_STATE_FBACK);
|
| - if (hctx->tfrchctx_t_rto = 0) {
| + if (hctx->ttx_t_rto = 0) {
| /*
| * Initial feedback packet: Larger Initial Windows (4.2)
| */
| - hctx->tfrchctx_x = rfc3390_initial_rate(sk);
| - hctx->tfrchctx_t_ld = now;
| + hctx->ttx_x = rfc3390_initial_rate(sk);
| + hctx->ttx_t_ld = now;
|
| ccid4_update_send_interval(hctx);
|
| goto done_computing_x;
| - } else if (hctx->tfrchctx_p = 0) {
| + } else if (hctx->ttx_p = 0) {
| /*
| * First feedback after nofeedback timer expiry (4.3)
| */
| @@ -417,25 +417,25 @@ static void ccid4_hc_tx_packet_recv(stru
| }
|
| /* Update sending rate (step 4 of [RFC 3448, 4.3]) */
| - if (hctx->tfrchctx_p > 0)
| - hctx->tfrchctx_x_calc | + if (hctx->ttx_p > 0)
| + hctx->ttx_x_calc | tfrc_calc_x(NOM_PACKET_SIZE,
| - hctx->tfrchctx_rtt,
| - hctx->tfrchctx_p);
| + hctx->ttx_rtt,
| + hctx->ttx_p);
| ccid4_hc_tx_update_x(sk, &now);
|
| done_computing_x:
| ccid4_pr_debug("%s(%p), RTT=%uus (sample=%uus), s=%u, "
| "p=%u, X_calc=%u, X_recv=%u, X=%u\n",
| dccp_role(sk),
| - sk, hctx->tfrchctx_rtt, r_sample,
| - hctx->tfrchctx_s, hctx->tfrchctx_p,
| - hctx->tfrchctx_x_calc,
| - (unsigned)(hctx->tfrchctx_x_recv >> 6),
| - (unsigned)(hctx->tfrchctx_x >> 6));
| + sk, hctx->ttx_rtt, r_sample,
| + hctx->ttx_s, hctx->ttx_p,
| + hctx->ttx_x_calc,
| + (unsigned)(hctx->ttx_x_recv >> 6),
| + (unsigned)(hctx->ttx_x >> 6));
|
| /* unschedule no feedback timer */
| - sk_stop_timer(sk, &hctx->tfrchctx_no_feedback_timer);
| + sk_stop_timer(sk, &hctx->ttx_no_feedback_timer);
|
| /*
| * As we have calculated new ipi, delta, t_nom it is possible
| @@ -449,21 +449,21 @@ done_computing_x:
| * This can help avoid triggering the nofeedback timer too
| * often ('spinning') on LANs with small RTTs.
| */
| - hctx->tfrchctx_t_rto = max_t(u32, 4 * hctx->tfrchctx_rtt,
| + hctx->ttx_t_rto = max_t(u32, 4 * hctx->ttx_rtt,
| (CONFIG_IP_DCCP_CCID4_RTO *
| (USEC_PER_SEC / 1000)));
| /*
| * Schedule no feedback timer to expire in
| * max(t_RTO, 2 * s/X) = max(t_RTO, 2 * t_ipi)
| */
| - t_nfb = max(hctx->tfrchctx_t_rto, 2 * hctx->tfrchctx_t_ipi);
| + t_nfb = max(hctx->ttx_t_rto, 2 * hctx->ttx_t_ipi);
|
| ccid4_pr_debug("%s(%p), Scheduled no feedback timer to "
| "expire in %lu jiffies (%luus)\n",
| dccp_role(sk),
| sk, usecs_to_jiffies(t_nfb), t_nfb);
|
| - sk_reset_timer(sk, &hctx->tfrchctx_no_feedback_timer,
| + sk_reset_timer(sk, &hctx->ttx_no_feedback_timer,
| jiffies + usecs_to_jiffies(t_nfb));
| }
|
| @@ -477,14 +477,14 @@ static int ccid4_hc_tx_parse_options(str
| struct tfrc_options_received *opt_recv;
| u32 opt_val;
|
| - opt_recv = &hctx->tfrchctx_options_received;
| + opt_recv = &hctx->ttx_options_received;
|
| if (opt_recv->tfrcor_seqno != dp->dccps_gsr) {
| - opt_recv->tfrcor_seqno = dp->dccps_gsr;
| + opt_recv->tfrcor_seqno = dp->dccps_gsr;
| opt_recv->tfrcor_loss_event_rate = ~0;
| opt_recv->tfrcor_loss_intervals_idx = 0;
| opt_recv->tfrcor_loss_intervals_len = 0;
| - opt_recv->tfrcor_receive_rate = 0;
| + opt_recv->tfrcor_receive_rate = 0;
| }
|
| switch (option) {
| @@ -536,9 +536,9 @@ static int ccid4_hc_tx_init(struct ccid
| {
| struct tfrc_hc_tx_sock *hctx = ccid_priv(ccid);
|
| - hctx->tfrchctx_state = TTX_STATE_NO_SENT;
| - hctx->tfrchctx_hist = NULL;
| - setup_timer(&hctx->tfrchctx_no_feedback_timer,
| + hctx->ttx_state = TTX_STATE_NO_SENT;
| + hctx->ttx_hist = NULL;
| + setup_timer(&hctx->ttx_no_feedback_timer,
| ccid4_hc_tx_no_feedback_timer, (unsigned long)sk);
|
| return 0;
| @@ -549,15 +549,15 @@ static void ccid4_hc_tx_exit(struct sock
| struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
|
| ccid4_hc_tx_set_state(sk, TTX_STATE_TERM);
| - sk_stop_timer(sk, &hctx->tfrchctx_no_feedback_timer);
| + sk_stop_timer(sk, &hctx->ttx_no_feedback_timer);
|
| - tfrc_tx_hist_purge(&hctx->tfrchctx_hist);
| + tfrc_tx_hist_purge(&hctx->ttx_hist);
| }
|
| static void ccid4_hc_tx_get_info(struct sock *sk, struct tcp_info *info)
| {
| - info->tcpi_rto = tfrc_hc_tx_sk(sk)->tfrchctx_t_rto;
| - info->tcpi_rtt = tfrc_hc_tx_sk(sk)->tfrchctx_rtt;
| + info->tcpi_rto = tfrc_hc_tx_sk(sk)->ttx_t_rto;
| + info->tcpi_rtt = tfrc_hc_tx_sk(sk)->ttx_rtt;
| }
|
| static int ccid4_hc_tx_getsockopt(struct sock *sk, const int optname, int len,
| @@ -568,10 +568,10 @@ static int ccid4_hc_tx_getsockopt(struct
|
| switch (optname) {
| case DCCP_SOCKOPT_CCID_TX_INFO:
| - if (len < sizeof(hctx->tfrchctx_tfrc))
| + if (len < sizeof(hctx->ttx_tfrc))
| return -EINVAL;
| - len = sizeof(hctx->tfrchctx_tfrc);
| - val = &hctx->tfrchctx_tfrc;
| + len = sizeof(hctx->ttx_tfrc);
| + val = &hctx->ttx_tfrc;
| break;
| default:
| return -ENOPROTOOPT;
| @@ -587,13 +587,13 @@ static void ccid4_hc_rx_set_state(struct
| enum tfrc_hc_rx_states state)
| {
| struct tfrc_hc_rx_sock *hcrx = tfrc_hc_rx_sk(sk);
| - enum tfrc_hc_rx_states oldstate = hcrx->tfrchcrx_state;
| + enum tfrc_hc_rx_states oldstate = hcrx->trx_state;
|
| ccid4_pr_debug("%s(%p) %-8.8s -> %s\n",
| dccp_role(sk), sk, tfrc_rx_state_name(oldstate),
| tfrc_rx_state_name(state));
| WARN_ON(state = oldstate);
| - hcrx->tfrchcrx_state = state;
| + hcrx->trx_state = state;
| }
|
| static void ccid4_hc_rx_send_feedback(struct sock *sk,
| @@ -605,13 +605,13 @@ static void ccid4_hc_rx_send_feedback(st
| ktime_t now = ktime_get_real();
| s64 delta = 0;
|
| - if (unlikely(hcrx->tfrchcrx_state = TRX_STATE_TERM))
| + if (unlikely(hcrx->trx_state = TRX_STATE_TERM))
| return;
|
| switch (fbtype) {
| case TFRC_FBACK_INITIAL:
| - hcrx->tfrchcrx_x_recv = 0;
| - hcrx->tfrchcrx_pinv = ~0U; /* see RFC 4342, 8.5 */
| + hcrx->trx_x_recv = 0;
| + hcrx->trx_pinv = ~0U; /* see RFC 4342, 8.5 */
| break;
| case TFRC_FBACK_PARAM_CHANGE:
| /*
| @@ -624,26 +624,26 @@ static void ccid4_hc_rx_send_feedback(st
| * the number of bytes since last feedback.
| * This is a safe fallback, since X is bounded above by X_calc.
| */
| - if (hcrx->tfrchcrx_x_recv > 0)
| + if (hcrx->trx_x_recv > 0)
| break;
| /* fall through */
| case TFRC_FBACK_PERIODIC:
| - delta = ktime_us_delta(now, hcrx->tfrchcrx_last_feedback);
| + delta = ktime_us_delta(now, hcrx->trx_last_feedback);
| if (delta <= 0)
| DCCP_BUG("delta (%ld) <= 0", (long)delta);
| else
| - hcrx->tfrchcrx_x_recv | - scaled_div32(hcrx->tfrchcrx_bytes_recv, delta);
| + hcrx->trx_x_recv | + scaled_div32(hcrx->trx_bytes_recv, delta);
| break;
| default:
| return;
| }
| ccid4_pr_debug("Interval %ldusec, X_recv=%u, 1/p=%u\n", (long)delta,
| - hcrx->tfrchcrx_x_recv, hcrx->tfrchcrx_pinv);
| + hcrx->trx_x_recv, hcrx->trx_pinv);
|
| - hcrx->tfrchcrx_last_feedback = now;
| - hcrx->tfrchcrx_last_counter = dccp_hdr(skb)->dccph_ccval;
| - hcrx->tfrchcrx_bytes_recv = 0;
| + hcrx->trx_last_feedback = now;
| + hcrx->trx_last_counter = dccp_hdr(skb)->dccph_ccval;
| + hcrx->trx_bytes_recv = 0;
|
| dp->dccps_hc_rx_insert_options = 1;
| dccp_send_ack(sk);
| @@ -660,8 +660,8 @@ static int ccid4_hc_rx_insert_options(st
| if (dccp_packet_without_ack(skb))
| return 0;
|
| - x_recv = htonl(hcrx->tfrchcrx_x_recv);
| - pinv = htonl(hcrx->tfrchcrx_pinv);
| + x_recv = htonl(hcrx->trx_x_recv);
| + pinv = htonl(hcrx->trx_pinv);
|
| if (dccp_insert_option(sk, skb, TFRC_OPT_LOSS_EVENT_RATE,
| &pinv, sizeof(pinv)) ||
| @@ -687,22 +687,22 @@ static u32 ccid4_first_li(struct sock *s
| u32 x_recv, p, delta;
| u64 fval;
|
| - if (hcrx->tfrchcrx_rtt = 0) {
| + if (hcrx->trx_rtt = 0) {
| DCCP_WARN("No RTT estimate available, using fallback RTT\n");
| - hcrx->tfrchcrx_rtt = DCCP_FALLBACK_RTT;
| + hcrx->trx_rtt = DCCP_FALLBACK_RTT;
| }
|
| - delta = ktime_to_us(net_timedelta(hcrx->tfrchcrx_last_feedback));
| - x_recv = scaled_div32(hcrx->tfrchcrx_bytes_recv, delta);
| + delta = ktime_to_us(net_timedelta(hcrx->trx_last_feedback));
| + x_recv = scaled_div32(hcrx->trx_bytes_recv, delta);
| if (x_recv = 0) { /* would also trigger divide-by-zero */
| DCCP_WARN("X_recv=0\n");
| - if ((x_recv = hcrx->tfrchcrx_x_recv) = 0) {
| + if ((x_recv = hcrx->trx_x_recv) = 0) {
| DCCP_BUG("stored value of X_recv is zero");
| return ~0U;
| }
| }
|
| - fval = scaled_div(NOM_PACKET_SIZE, hcrx->tfrchcrx_rtt);
| + fval = scaled_div(NOM_PACKET_SIZE, hcrx->trx_rtt);
| fval = scaled_div32(fval, x_recv);
| p = tfrc_calc_x_reverse_lookup(fval);
|
| @@ -719,14 +719,14 @@ static void ccid4_hc_rx_packet_recv(stru
| const u64 ndp = dccp_sk(sk)->dccps_options_received.dccpor_ndp;
| const bool is_data_packet = dccp_data_packet(skb);
|
| - if (unlikely(hcrx->tfrchcrx_state = TRX_STATE_NO_DATA)) {
| + if (unlikely(hcrx->trx_state = TRX_STATE_NO_DATA)) {
| if (is_data_packet) {
| const u32 payload = skb->len - dccp_hdr(skb)->dccph_doff * 4;
| do_feedback = TFRC_FBACK_INITIAL;
| ccid4_hc_rx_set_state(sk, TRX_STATE_DATA);
| - hcrx->tfrchcrx_s = payload;
| + hcrx->trx_s = payload;
| /*
| - * Not necessary to update tfrchcrx_bytes_recv here,
| + * Not necessary to update trx_bytes_recv here,
| * since X_recv = 0 for the first feedback packet (cf.
| * RFC 3448, 6.3) -- gerrit
| */
| @@ -735,7 +735,7 @@ static void ccid4_hc_rx_packet_recv(stru
| goto update_records;
| }
|
| - if (tfrc_rx_hist_duplicate(&hcrx->tfrchcrx_hist, skb))
| + if (tfrc_rx_hist_duplicate(&hcrx->trx_hist, skb))
| return; /* done receiving */
|
| if (is_data_packet) {
| @@ -743,22 +743,21 @@ static void ccid4_hc_rx_packet_recv(stru
| /*
| * Update moving-average of s and the sum of received payload bytes
| */
| - hcrx->tfrchcrx_s = tfrc_ewma(hcrx->tfrchcrx_s, payload, 9);
| - hcrx->tfrchcrx_bytes_recv += payload;
| + hcrx->trx_s = tfrc_ewma(hcrx->trx_s, payload, 9);
| + hcrx->trx_bytes_recv += payload;
| }
|
| /*
| * Handle pending losses and otherwise check for new loss
| */
| - if (tfrc_rx_hist_loss_pending(&hcrx->tfrchcrx_hist) &&
| - tfrc_rx_handle_loss(&hcrx->tfrchcrx_hist,
| - &hcrx->tfrchcrx_li_hist,
| + if (tfrc_rx_hist_loss_pending(&hcrx->trx_hist) &&
| + tfrc_rx_handle_loss(&hcrx->trx_hist, &hcrx->trx_li_hist,
| skb, ndp, ccid4_first_li, sk) ) {
| do_feedback = TFRC_FBACK_PARAM_CHANGE;
| goto done_receiving;
| }
|
| - if (tfrc_rx_hist_new_loss_indicated(&hcrx->tfrchcrx_hist, skb, ndp))
| + if (tfrc_rx_hist_new_loss_indicated(&hcrx->trx_hist, skb, ndp))
| goto update_records;
|
| /*
| @@ -767,17 +766,17 @@ static void ccid4_hc_rx_packet_recv(stru
| if (unlikely(!is_data_packet))
| goto update_records;
|
| - if (!tfrc_lh_is_initialised(&hcrx->tfrchcrx_li_hist)) {
| - const u32 sample = tfrc_rx_hist_sample_rtt(&hcrx->tfrchcrx_hist, skb);
| + if (!tfrc_lh_is_initialised(&hcrx->trx_li_hist)) {
| + const u32 sample = tfrc_rx_hist_sample_rtt(&hcrx->trx_hist, skb);
| /*
| * Empty loss history: no loss so far, hence p stays 0.
| * Sample RTT values, since an RTT estimate is required for the
| * computation of p when the first loss occurs; RFC 3448, 6.3.1.
| */
| if (sample != 0)
| - hcrx->tfrchcrx_rtt = tfrc_ewma(hcrx->tfrchcrx_rtt, sample, 9);
| + hcrx->trx_rtt = tfrc_ewma(hcrx->trx_rtt, sample, 9);
|
| - } else if (tfrc_lh_update_i_mean(&hcrx->tfrchcrx_li_hist, skb))
| + } else if (tfrc_lh_update_i_mean(&hcrx->trx_li_hist, skb))
| /*
| * Step (3) of [RFC 3448, 6.1]: Recompute I_mean and, if I_mean
| * has decreased (resp. p has increased), send feedback now.
| @@ -787,11 +786,11 @@ static void ccid4_hc_rx_packet_recv(stru
| /*
| * Check if the periodic once-per-RTT feedback is due; RFC 4342, 10.3
| */
| - if (SUB16(dccp_hdr(skb)->dccph_ccval, hcrx->tfrchcrx_last_counter) > 3)
| + if (SUB16(dccp_hdr(skb)->dccph_ccval, hcrx->trx_last_counter) > 3)
| do_feedback = TFRC_FBACK_PERIODIC;
|
| update_records:
| - tfrc_rx_hist_add_packet(&hcrx->tfrchcrx_hist, skb, ndp);
| + tfrc_rx_hist_add_packet(&hcrx->trx_hist, skb, ndp);
|
| done_receiving:
| if (do_feedback)
| @@ -802,9 +801,9 @@ static int ccid4_hc_rx_init(struct ccid
| {
| struct tfrc_hc_rx_sock *hcrx = ccid_priv(ccid);
|
| - hcrx->tfrchcrx_state = TRX_STATE_NO_DATA;
| - tfrc_lh_init(&hcrx->tfrchcrx_li_hist);
| - return tfrc_rx_hist_alloc(&hcrx->tfrchcrx_hist);
| + hcrx->trx_state = TRX_STATE_NO_DATA;
| + tfrc_lh_init(&hcrx->trx_li_hist);
| + return tfrc_rx_hist_alloc(&hcrx->trx_hist);
| }
|
| static void ccid4_hc_rx_exit(struct sock *sk)
| @@ -813,15 +812,15 @@ static void ccid4_hc_rx_exit(struct sock
|
| ccid4_hc_rx_set_state(sk, TRX_STATE_TERM);
|
| - tfrc_rx_hist_purge(&hcrx->tfrchcrx_hist);
| - tfrc_lh_cleanup(&hcrx->tfrchcrx_li_hist);
| + tfrc_rx_hist_purge(&hcrx->trx_hist);
| + tfrc_lh_cleanup(&hcrx->trx_li_hist);
| }
|
| static void ccid4_hc_rx_get_info(struct sock *sk, struct tcp_info *info)
| {
| - info->tcpi_ca_state = tfrc_hc_rx_sk(sk)->tfrchcrx_state;
| + info->tcpi_ca_state = tfrc_hc_rx_sk(sk)->trx_state;
| info->tcpi_options |= TCPI_OPT_TIMESTAMPS;
| - info->tcpi_rcv_rtt = tfrc_hc_rx_sk(sk)->tfrchcrx_rtt;
| + info->tcpi_rcv_rtt = tfrc_hc_rx_sk(sk)->trx_rtt;
| }
|
| static int ccid4_hc_rx_getsockopt(struct sock *sk, const int optname, int len,
| @@ -835,10 +834,10 @@ static int ccid4_hc_rx_getsockopt(struct
| case DCCP_SOCKOPT_CCID_RX_INFO:
| if (len < sizeof(rx_info))
| return -EINVAL;
| - rx_info.tfrcrx_x_recv = hcrx->tfrchcrx_x_recv;
| - rx_info.tfrcrx_rtt = hcrx->tfrchcrx_rtt;
| - rx_info.tfrcrx_p = hcrx->tfrchcrx_pinv = 0 ? ~0U :
| - scaled_div(1, hcrx->tfrchcrx_pinv);
| + rx_info.tfrcrx_x_recv = hcrx->trx_x_recv;
| + rx_info.tfrcrx_rtt = hcrx->trx_rtt;
| + rx_info.tfrcrx_p = hcrx->trx_pinv = 0 ? ~0U :
| + scaled_div(1, hcrx->trx_pinv);
| len = sizeof(rx_info);
| val = &rx_info;
| break;
| Index: ccid4.latest/net/dccp/ccids/lib/tfrc_ccids.c
| =================================| --- ccid4.latest.orig/net/dccp/ccids/lib/tfrc_ccids.c
| +++ ccid4.latest/net/dccp/ccids/lib/tfrc_ccids.c
| @@ -23,19 +23,19 @@
| inline u64 rfc3390_initial_rate(struct sock *sk)
| {
| const struct tfrc_hc_tx_sock *hctx = tfrc_hc_tx_sk(sk);
| - const __u32 w_init = min_t(__u32, 4 * hctx->tfrchctx_s,
| - max_t(__u32, 2 * hctx->tfrchctx_s, 4380));
| + const __u32 w_init = min_t(__u32, 4 * hctx->ttx_s,
| + max_t(__u32, 2 * hctx->ttx_s, 4380));
|
| - return scaled_div(w_init << 6, hctx->tfrchctx_rtt);
| + return scaled_div(w_init << 6, hctx->ttx_rtt);
| }
|
| EXPORT_SYMBOL_GPL(rfc3390_initial_rate);
|
| u32 tfrc_hc_tx_idle_rtt(struct tfrc_hc_tx_sock *hctx, ktime_t now)
| {
| - u32 delta = ktime_us_delta(now, hctx->tfrchctx_t_last_win_count);
| + u32 delta = ktime_us_delta(now, hctx->ttx_t_last_win_count);
|
| - return delta / hctx->tfrchctx_rtt;
| + return delta / hctx->ttx_rtt;
| }
|
| EXPORT_SYMBOL_GPL(tfrc_hc_tx_idle_rtt);
| @@ -49,16 +49,16 @@ inline void tfrc_hc_tx_update_win_count(
| {
| u32 quarter_rtts;
|
| - if (unlikely(hctx->tfrchctx_rtt < 4)) /* avoid divide-by-zero */
| + if (unlikely(hctx->ttx_rtt < 4)) /* avoid divide-by-zero */
| return;
|
| - quarter_rtts = ktime_us_delta(now, hctx->tfrchctx_t_last_win_count);
| - quarter_rtts /= hctx->tfrchctx_rtt / 4;
| + quarter_rtts = ktime_us_delta(now, hctx->ttx_t_last_win_count);
| + quarter_rtts /= hctx->ttx_rtt / 4;
|
| if (quarter_rtts > 0) {
| - hctx->tfrchctx_t_last_win_count = now;
| - hctx->tfrchctx_last_win_count += min_t(u32, quarter_rtts, 5);
| - hctx->tfrchctx_last_win_count &= 0xF; /* mod 16 */
| + hctx->ttx_t_last_win_count = now;
| + hctx->ttx_last_win_count += min_t(u32, quarter_rtts, 5);
| + hctx->ttx_last_win_count &= 0xF; /* mod 16 */
| }
| }
|
| Index: ccid4.latest/net/dccp/ccids/lib/tfrc_ccids.h
| =================================| --- ccid4.latest.orig/net/dccp/ccids/lib/tfrc_ccids.h
| +++ ccid4.latest/net/dccp/ccids/lib/tfrc_ccids.h
| @@ -73,44 +73,44 @@ enum tfrc_fback_type {
|
| /** struct tfrc_hc_tx_sock - CCID3/4 sender half-connection socket
| *
| - * @tfrchctx_x - Current sending rate in 64 * bytes per second
| - * @tfrchctx_x_recv - Receive rate in 64 * bytes per second
| - * @tfrchctx_x_calc - Calculated rate in bytes per second
| - * @tfrchctx_rtt - Estimate of current round trip time in usecs
| - * @tfrchctx_p - Current loss event rate (0-1) scaled by 1000000
| - * @tfrchctx_s - Packet size in bytes
| - * @tfrchctx_t_rto - Nofeedback Timer setting in usecs
| - * @tfrchctx_t_ipi - Interpacket (send) interval (RFC 3448, 4.6) in usecs
| - * @tfrchctx_state - Sender state, one of %tfrc_hc_tx_states
| - * @tfrchctx_last_win_count - Last window counter sent
| - * @tfrchctx_t_last_win_count - Timestamp of earliest packet
| + * @ttx_x - Current sending rate in 64 * bytes per second
| + * @ttx_x_recv - Receive rate in 64 * bytes per second
| + * @ttx_x_calc - Calculated rate in bytes per second
| + * @ttx_rtt - Estimate of current round trip time in usecs
| + * @ttx_p - Current loss event rate (0-1) scaled by 1000000
| + * @ttx_s - Packet size in bytes
| + * @ttx_t_rto - Nofeedback Timer setting in usecs
| + * @ttx_t_ipi - Interpacket (send) interval (RFC 3448, 4.6) in usecs
| + * @ttx_state - Sender state, one of %tfrc_hc_tx_states
| + * @ttx_last_win_count - Last window counter sent
| + * @ttx_t_last_win_count - Timestamp of earliest packet
| * with last_win_count value sent
| - * @tfrchctx_no_feedback_timer - Handle to no feedback timer
| - * @tfrchctx_t_ld - Time last doubled during slow start
| - * @tfrchctx_t_nom - Nominal send time of next packet
| - * @tfrchctx_delta - Send timer delta (RFC 3448, 4.6) in usecs
| - * @tfrchctx_hist - Packet history
| - * @tfrchctx_options_received - Parsed set of retrieved options
| + * @ttx_no_feedback_timer - Handle to no feedback timer
| + * @ttx_t_ld - Time last doubled during slow start
| + * @ttx_t_nom - Nominal send time of next packet
| + * @ttx_delta - Send timer delta (RFC 3448, 4.6) in usecs
| + * @ttx_hist - Packet history
| + * @ttx_options_received - Parsed set of retrieved options
| */
| struct tfrc_hc_tx_sock {
| - struct tfrc_tx_info tfrchctx_tfrc;
| -#define tfrchctx_x tfrchctx_tfrc.tfrctx_x
| -#define tfrchctx_x_recv tfrchctx_tfrc.tfrctx_x_recv
| -#define tfrchctx_x_calc tfrchctx_tfrc.tfrctx_x_calc
| -#define tfrchctx_rtt tfrchctx_tfrc.tfrctx_rtt
| -#define tfrchctx_p tfrchctx_tfrc.tfrctx_p
| -#define tfrchctx_t_rto tfrchctx_tfrc.tfrctx_rto
| -#define tfrchctx_t_ipi tfrchctx_tfrc.tfrctx_ipi
| - u16 tfrchctx_s;
| - enum tfrc_hc_tx_states tfrchctx_state:8;
| - u8 tfrchctx_last_win_count;
| - ktime_t tfrchctx_t_last_win_count;
| - struct timer_list tfrchctx_no_feedback_timer;
| - ktime_t tfrchctx_t_ld;
| - ktime_t tfrchctx_t_nom;
| - u32 tfrchctx_delta;
| - struct tfrc_tx_hist_entry *tfrchctx_hist;
| - struct tfrc_options_received tfrchctx_options_received;
| + struct tfrc_tx_info ttx_tfrc;
| +#define ttx_x ttx_tfrc.x
| +#define ttx_x_recv ttx_tfrc.x_recv
| +#define ttx_x_calc ttx_tfrc.x_calc
| +#define ttx_rtt ttx_tfrc.rtt
| +#define ttx_p ttx_tfrc.p
| +#define ttx_t_rto ttx_tfrc.rto
| +#define ttx_t_ipi ttx_tfrc.ipi
| + u16 ttx_s;
| + enum tfrc_hc_tx_states ttx_state:8;
| + u8 ttx_last_win_count;
| + ktime_t ttx_t_last_win_count;
| + struct timer_list ttx_no_feedback_timer;
| + ktime_t ttx_t_ld;
| + ktime_t ttx_t_nom;
| + u32 ttx_delta;
| + struct tfrc_tx_hist_entry *ttx_hist;
| + struct tfrc_options_received ttx_options_received;
| };
|
| static inline struct tfrc_hc_tx_sock *tfrc_hc_tx_sk(const struct sock *sk)
| @@ -122,28 +122,28 @@ static inline struct tfrc_hc_tx_sock *tf
|
| /** struct tfrc_hc_rx_sock - CCID3/4 receiver half-connection socket
| *
| - * @tfrchcrx_last_counter - Tracks window counter (RFC 4342, 8.1)
| - * @tfrchcrx_state - Receiver state, one of %tfrc_hc_rx_states
| - * @tfrchcrx_bytes_recv - Total sum of DCCP payload bytes
| - * @tfrchcrx_x_recv - Receiver estimate of send rate (RFC 3448, sec. 4.3)
| - * @tfrchcrx_rtt - Receiver estimate of RTT
| - * @tfrchcrx_last_feedback - Time at which last feedback was sent
| - * @tfrchcrx_hist - Packet history, exported by TFRC module
| - * @tfrchcrx_li_hist - Loss Interval database, exported by TFRC module
| - * @tfrchcrx_s - Received packet size in bytes
| - * @tfrchcrx_pinv - Inverse of Loss Event Rate (RFC 4342, sec. 8.5)
| + * @trx_last_counter - Tracks window counter (RFC 4342, 8.1)
| + * @trx_state - Receiver state, one of %tfrc_hc_rx_states
| + * @trx_bytes_recv - Total sum of DCCP payload bytes
| + * @trx_x_recv - Receiver estimate of send rate (RFC 3448, sec. 4.3)
| + * @trx_rtt - Receiver estimate of RTT
| + * @trx_last_feedback - Time at which last feedback was sent
| + * @trx_hist - Packet history, exported by TFRC module
| + * @trx_li_hist - Loss Interval database, exported by TFRC module
| + * @trx_s - Received packet size in bytes
| + * @trx_pinv - Inverse of Loss Event Rate (RFC 4342, sec. 8.5)
| */
| struct tfrc_hc_rx_sock {
| - u8 tfrchcrx_last_counter:4;
| - enum tfrc_hc_rx_states tfrchcrx_state:8;
| - u32 tfrchcrx_bytes_recv;
| - u32 tfrchcrx_x_recv;
| - u32 tfrchcrx_rtt;
| - ktime_t tfrchcrx_last_feedback;
| - struct tfrc_rx_hist tfrchcrx_hist;
| - struct tfrc_loss_hist tfrchcrx_li_hist;
| - u16 tfrchcrx_s;
| -#define tfrchcrx_pinv tfrchcrx_li_hist.i_mean
| + u8 trx_last_counter:4;
| + enum tfrc_hc_rx_states trx_state:8;
| + u32 trx_bytes_recv;
| + u32 trx_x_recv;
| + u32 trx_rtt;
| + ktime_t trx_last_feedback;
| + struct tfrc_rx_hist trx_hist;
| + struct tfrc_loss_hist trx_li_hist;
| + u16 trx_s;
| +#define trx_pinv trx_li_hist.i_mean
| };
|
| static inline struct tfrc_hc_rx_sock *tfrc_hc_rx_sk(const struct sock *sk)
| @@ -193,4 +193,3 @@ extern u32 tfrc_hc_tx_idle_rtt(struct tf
|
| extern void tfrc_hc_tx_update_win_count(struct tfrc_hc_tx_sock *hctx,
| ktime_t now);
| -
|
--
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCHv2 4/8] Fix name schema for tx-rx sock struct
2007-12-21 4:33 [PATCHv2 4/8] Fix name schema for tx-rx sock struct Leandro
2007-12-21 9:53 ` Gerrit Renker
@ 2007-12-26 18:01 ` Leandro Sales
2008-01-07 13:22 ` Gerrit Renker
2 siblings, 0 replies; 4+ messages in thread
From: Leandro Sales @ 2007-12-26 18:01 UTC (permalink / raw)
To: dccp
Hi Gerrit,
the patches was applied against:
# git-describe
v2.6.23-rc6-10457-g5ff93cc
SHA1 ID: 5ff93cc505b6ced6aa6d4c65df6d9beb6ae10fac
I cloned it at 2007-12-20, 18:19:49 (GMT -3).
Leandro.
2007/12/21, Gerrit Renker <gerrit@erg.abdn.ac.uk>:
> And your naming scheme changed from version 1 to version 2:
>
> This is patch v2:
> | switch (optname) {
> | case DCCP_SOCKOPT_CCID_TX_INFO:
> | - if (len < sizeof(hctx->tfrchctx_tfrc))
> | + if (len < sizeof(hctx->ttx_tfrc))
> | return -EINVAL;
> | - len = sizeof(hctx->tfrchctx_tfrc);
> | - val = &hctx->tfrchctx_tfrc;
> | + len = sizeof(hctx->ttx_tfrc);
> | + val = &hctx->ttx_tfrc;
> | break;
>
> ... and this is patch v1:
>
> | switch (optname) {
> | case DCCP_SOCKOPT_CCID_TX_INFO:
> | - if (len < sizeof(hctx->tfrchctx_tfrc))
> | + if (len < sizeof(hctx->tfrc_tx))
> | return -EINVAL;
> | - len = sizeof(hctx->tfrchctx_tfrc);
> | - val = &hctx->tfrchctx_tfrc;
> | + len = sizeof(hctx->tfrc_tx);
> | + val = &hctx->tfrc_tx;
> | break;
> 'tfrc_tx' becomes 'ttx_tfrc'. I think that is all needs more work and in
> particular discussion with Arnaldo.
>
> With the given inconsistencies I can't put the set into the test tree.
> Can you please resolve the naming scheme inconsistencies and the version
> problems and resubmit a fresh patch set after discussing with Arnaldo.
>
--
Leandro Melo de Sales.
Pervasive and Embedded Computing Laboratory
BRisa and E-Phone Projects Manager
Network Admin @ http://embedded.ufcg.edu.br/indexen.html
+55 83 3310-1404 (extension 208)
"There are 10 types of people in this world: those who understand
binary, those who don't"
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCHv2 4/8] Fix name schema for tx-rx sock struct
2007-12-21 4:33 [PATCHv2 4/8] Fix name schema for tx-rx sock struct Leandro
2007-12-21 9:53 ` Gerrit Renker
2007-12-26 18:01 ` Leandro Sales
@ 2008-01-07 13:22 ` Gerrit Renker
2 siblings, 0 replies; 4+ messages in thread
From: Gerrit Renker @ 2008-01-07 13:22 UTC (permalink / raw)
To: dccp
Hi Leandro,
can you please re-send the patches from before Christmas and try a new
clone first? I have just switched the git-server back on again, today is
the first day we have electricity back. That is,
git://eden-feed.erg.abdn.ac.uk/dccp_exp {dccp,ccid4}
is up again; the CCID4 patches are exactly as before Christmas.
Gerrit
Quoting Leandro:
| Hi Gerrit,
| the patches was applied against:
|
| # git-describe
| v2.6.23-rc6-10457-g5ff93cc
|
| SHA1 ID: 5ff93cc505b6ced6aa6d4c65df6d9beb6ae10fac
|
| I cloned it at 2007-12-20, 18:19:49 (GMT -3).
|
| Leandro.
|
| 2007/12/21, Gerrit Renker <gerrit@erg.abdn.ac.uk>:
| > And your naming scheme changed from version 1 to version 2:
| >
| > This is patch v2:
| > | switch (optname) {
| > | case DCCP_SOCKOPT_CCID_TX_INFO:
| > | - if (len < sizeof(hctx->tfrchctx_tfrc))
| > | + if (len < sizeof(hctx->ttx_tfrc))
| > | return -EINVAL;
| > | - len = sizeof(hctx->tfrchctx_tfrc);
| > | - val = &hctx->tfrchctx_tfrc;
| > | + len = sizeof(hctx->ttx_tfrc);
| > | + val = &hctx->ttx_tfrc;
| > | break;
| >
| > ... and this is patch v1:
| >
| > | switch (optname) {
| > | case DCCP_SOCKOPT_CCID_TX_INFO:
| > | - if (len < sizeof(hctx->tfrchctx_tfrc))
| > | + if (len < sizeof(hctx->tfrc_tx))
| > | return -EINVAL;
| > | - len = sizeof(hctx->tfrchctx_tfrc);
| > | - val = &hctx->tfrchctx_tfrc;
| > | + len = sizeof(hctx->tfrc_tx);
| > | + val = &hctx->tfrc_tx;
| > | break;
| > 'tfrc_tx' becomes 'ttx_tfrc'. I think that is all needs more work and in
| > particular discussion with Arnaldo.
| >
| > With the given inconsistencies I can't put the set into the test tree.
| > Can you please resolve the naming scheme inconsistencies and the version
| > problems and resubmit a fresh patch set after discussing with Arnaldo.
| >
|
|
| --
| Leandro Melo de Sales.
| Pervasive and Embedded Computing Laboratory
| BRisa and E-Phone Projects Manager
| Network Admin @ http://embedded.ufcg.edu.br/indexen.html
| +55 83 3310-1404 (extension 208)
|
| "There are 10 types of people in this world: those who understand
| binary, those who don't"
|
--
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2008-01-07 13:22 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-12-21 4:33 [PATCHv2 4/8] Fix name schema for tx-rx sock struct Leandro
2007-12-21 9:53 ` Gerrit Renker
2007-12-26 18:01 ` Leandro Sales
2008-01-07 13:22 ` Gerrit Renker
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.