]> pilppa.org Git - linux-2.6-omap-h63xx.git/blobdiff - net/dccp/ccids/ccid2.c
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6
[linux-2.6-omap-h63xx.git] / net / dccp / ccids / ccid2.c
index c7d83e3c1648ceaf63d49d9867f554cbda042fa7..9a430734530c6d8bd2079ca7c1662d03b221ad73 100644 (file)
@@ -25,7 +25,7 @@
 /*
  * This implementation should follow RFC 4341
  */
-#include "../feat.h"
+
 #include "../ccid.h"
 #include "../dccp.h"
 #include "ccid2.h"
@@ -39,16 +39,16 @@ static void ccid2_hc_tx_check_sanity(const struct ccid2_hc_tx_sock *hctx)
 {
        int len = 0;
        int pipe = 0;
-       struct ccid2_seq *seqp = hctx->seqh;
+       struct ccid2_seq *seqp = hctx->ccid2hctx_seqh;
 
        /* there is data in the chain */
-       if (seqp != hctx->seqt) {
+       if (seqp != hctx->ccid2hctx_seqt) {
                seqp = seqp->ccid2s_prev;
                len++;
                if (!seqp->ccid2s_acked)
                        pipe++;
 
-               while (seqp != hctx->seqt) {
+               while (seqp != hctx->ccid2hctx_seqt) {
                        struct ccid2_seq *prev = seqp->ccid2s_prev;
 
                        len++;
@@ -65,16 +65,16 @@ static void ccid2_hc_tx_check_sanity(const struct ccid2_hc_tx_sock *hctx)
                }
        }
 
-       BUG_ON(pipe != hctx->pipe);
+       BUG_ON(pipe != hctx->ccid2hctx_pipe);
        ccid2_pr_debug("len of chain=%d\n", len);
 
        do {
                seqp = seqp->ccid2s_prev;
                len++;
-       } while (seqp != hctx->seqh);
+       } while (seqp != hctx->ccid2hctx_seqh);
 
        ccid2_pr_debug("total len=%d\n", len);
-       BUG_ON(len != hctx->seqbufc * CCID2_SEQBUF_LEN);
+       BUG_ON(len != hctx->ccid2hctx_seqbufc * CCID2_SEQBUF_LEN);
 }
 #else
 #define ccid2_pr_debug(format, a...)
@@ -87,7 +87,8 @@ static int ccid2_hc_tx_alloc_seq(struct ccid2_hc_tx_sock *hctx)
        int i;
 
        /* check if we have space to preserve the pointer to the buffer */
-       if (hctx->seqbufc >= sizeof(hctx->seqbuf) / sizeof(struct ccid2_seq *))
+       if (hctx->ccid2hctx_seqbufc >= (sizeof(hctx->ccid2hctx_seqbuf) /
+                                       sizeof(struct ccid2_seq*)))
                return -ENOMEM;
 
        /* allocate buffer and initialize linked list */
@@ -103,35 +104,38 @@ static int ccid2_hc_tx_alloc_seq(struct ccid2_hc_tx_sock *hctx)
        seqp->ccid2s_prev = &seqp[CCID2_SEQBUF_LEN - 1];
 
        /* This is the first allocation.  Initiate the head and tail.  */
-       if (hctx->seqbufc == 0)
-               hctx->seqh = hctx->seqt = seqp;
+       if (hctx->ccid2hctx_seqbufc == 0)
+               hctx->ccid2hctx_seqh = hctx->ccid2hctx_seqt = seqp;
        else {
                /* link the existing list with the one we just created */
-               hctx->seqh->ccid2s_next = seqp;
-               seqp->ccid2s_prev = hctx->seqh;
+               hctx->ccid2hctx_seqh->ccid2s_next = seqp;
+               seqp->ccid2s_prev = hctx->ccid2hctx_seqh;
 
-               hctx->seqt->ccid2s_prev = &seqp[CCID2_SEQBUF_LEN - 1];
-               seqp[CCID2_SEQBUF_LEN - 1].ccid2s_next = hctx->seqt;
+               hctx->ccid2hctx_seqt->ccid2s_prev = &seqp[CCID2_SEQBUF_LEN - 1];
+               seqp[CCID2_SEQBUF_LEN - 1].ccid2s_next = hctx->ccid2hctx_seqt;
        }
 
        /* store the original pointer to the buffer so we can free it */
-       hctx->seqbuf[hctx->seqbufc] = seqp;
-       hctx->seqbufc++;
+       hctx->ccid2hctx_seqbuf[hctx->ccid2hctx_seqbufc] = seqp;
+       hctx->ccid2hctx_seqbufc++;
 
        return 0;
 }
 
 static int ccid2_hc_tx_send_packet(struct sock *sk, struct sk_buff *skb)
 {
-       if (ccid2_cwnd_network_limited(ccid2_hc_tx_sk(sk)))
-               return CCID_PACKET_WILL_DEQUEUE_LATER;
-       return CCID_PACKET_SEND_AT_ONCE;
+       struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
+
+       if (hctx->ccid2hctx_pipe < hctx->ccid2hctx_cwnd)
+               return 0;
+
+       return 1; /* XXX CCID should dequeue when ready instead of polling */
 }
 
 static void ccid2_change_l_ack_ratio(struct sock *sk, u32 val)
 {
        struct dccp_sock *dp = dccp_sk(sk);
-       u32 max_ratio = DIV_ROUND_UP(ccid2_hc_tx_sk(sk)->cwnd, 2);
+       u32 max_ratio = DIV_ROUND_UP(ccid2_hc_tx_sk(sk)->ccid2hctx_cwnd, 2);
 
        /*
         * Ensure that Ack Ratio does not exceed ceil(cwnd/2), which is (2) from
@@ -143,8 +147,8 @@ static void ccid2_change_l_ack_ratio(struct sock *sk, u32 val)
                DCCP_WARN("Limiting Ack Ratio (%u) to %u\n", val, max_ratio);
                val = max_ratio;
        }
-       if (val > DCCPF_ACK_RATIO_MAX)
-               val = DCCPF_ACK_RATIO_MAX;
+       if (val > 0xFFFF)               /* RFC 4340, 11.3 */
+               val = 0xFFFF;
 
        if (val == dp->dccps_l_ack_ratio)
                return;
@@ -156,7 +160,7 @@ static void ccid2_change_l_ack_ratio(struct sock *sk, u32 val)
 static void ccid2_change_srtt(struct ccid2_hc_tx_sock *hctx, long val)
 {
        ccid2_pr_debug("change SRTT to %ld\n", val);
-       hctx->srtt = val;
+       hctx->ccid2hctx_srtt = val;
 }
 
 static void ccid2_start_rto_timer(struct sock *sk);
@@ -165,12 +169,12 @@ static void ccid2_hc_tx_rto_expire(unsigned long data)
 {
        struct sock *sk = (struct sock *)data;
        struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
-       const bool sender_was_blocked = ccid2_cwnd_network_limited(hctx);
        long s;
 
        bh_lock_sock(sk);
        if (sock_owned_by_user(sk)) {
-               sk_reset_timer(sk, &hctx->rtotimer, jiffies + HZ / 5);
+               sk_reset_timer(sk, &hctx->ccid2hctx_rtotimer,
+                              jiffies + HZ / 5);
                goto out;
        }
 
@@ -179,33 +183,30 @@ static void ccid2_hc_tx_rto_expire(unsigned long data)
        ccid2_hc_tx_check_sanity(hctx);
 
        /* back-off timer */
-       hctx->rto <<= 1;
+       hctx->ccid2hctx_rto <<= 1;
 
-       s = hctx->rto / HZ;
+       s = hctx->ccid2hctx_rto / HZ;
        if (s > 60)
-               hctx->rto = 60 * HZ;
+               hctx->ccid2hctx_rto = 60 * HZ;
+
+       ccid2_start_rto_timer(sk);
 
        /* adjust pipe, cwnd etc */
-       hctx->ssthresh = hctx->cwnd / 2;
-       if (hctx->ssthresh < 2)
-               hctx->ssthresh = 2;
-       hctx->cwnd = 1;
-       hctx->pipe = 0;
+       hctx->ccid2hctx_ssthresh = hctx->ccid2hctx_cwnd / 2;
+       if (hctx->ccid2hctx_ssthresh < 2)
+               hctx->ccid2hctx_ssthresh = 2;
+       hctx->ccid2hctx_cwnd     = 1;
+       hctx->ccid2hctx_pipe     = 0;
 
        /* clear state about stuff we sent */
-       hctx->seqt = hctx->seqh;
-       hctx->packets_acked = 0;
+       hctx->ccid2hctx_seqt = hctx->ccid2hctx_seqh;
+       hctx->ccid2hctx_packets_acked = 0;
 
        /* clear ack ratio state. */
-       hctx->rpseq    = 0;
-       hctx->rpdupack = -1;
+       hctx->ccid2hctx_rpseq    = 0;
+       hctx->ccid2hctx_rpdupack = -1;
        ccid2_change_l_ack_ratio(sk, 1);
        ccid2_hc_tx_check_sanity(hctx);
-
-       /* if we were blocked before, we may now send cwnd=1 packet */
-       if (sender_was_blocked)
-               tasklet_schedule(&dccp_sk(sk)->dccps_xmitlet);
-       ccid2_start_rto_timer(sk);
 out:
        bh_unlock_sock(sk);
        sock_put(sk);
@@ -215,39 +216,40 @@ static void ccid2_start_rto_timer(struct sock *sk)
 {
        struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
 
-       ccid2_pr_debug("setting RTO timeout=%ld\n", hctx->rto);
+       ccid2_pr_debug("setting RTO timeout=%ld\n", hctx->ccid2hctx_rto);
 
-       BUG_ON(timer_pending(&hctx->rtotimer));
-       sk_reset_timer(sk, &hctx->rtotimer,
-                      jiffies + hctx->rto);
+       BUG_ON(timer_pending(&hctx->ccid2hctx_rtotimer));
+       sk_reset_timer(sk, &hctx->ccid2hctx_rtotimer,
+                      jiffies + hctx->ccid2hctx_rto);
 }
 
-static void ccid2_hc_tx_packet_sent(struct sock *sk, unsigned int len)
+static void ccid2_hc_tx_packet_sent(struct sock *sk, int more, unsigned int len)
 {
        struct dccp_sock *dp = dccp_sk(sk);
        struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
        struct ccid2_seq *next;
 
-       hctx->pipe++;
+       hctx->ccid2hctx_pipe++;
 
-       hctx->seqh->ccid2s_seq   = dp->dccps_gss;
-       hctx->seqh->ccid2s_acked = 0;
-       hctx->seqh->ccid2s_sent  = jiffies;
+       hctx->ccid2hctx_seqh->ccid2s_seq   = dp->dccps_gss;
+       hctx->ccid2hctx_seqh->ccid2s_acked = 0;
+       hctx->ccid2hctx_seqh->ccid2s_sent  = jiffies;
 
-       next = hctx->seqh->ccid2s_next;
+       next = hctx->ccid2hctx_seqh->ccid2s_next;
        /* check if we need to alloc more space */
-       if (next == hctx->seqt) {
+       if (next == hctx->ccid2hctx_seqt) {
                if (ccid2_hc_tx_alloc_seq(hctx)) {
                        DCCP_CRIT("packet history - out of memory!");
                        /* FIXME: find a more graceful way to bail out */
                        return;
                }
-               next = hctx->seqh->ccid2s_next;
-               BUG_ON(next == hctx->seqt);
+               next = hctx->ccid2hctx_seqh->ccid2s_next;
+               BUG_ON(next == hctx->ccid2hctx_seqt);
        }
-       hctx->seqh = next;
+       hctx->ccid2hctx_seqh = next;
 
-       ccid2_pr_debug("cwnd=%d pipe=%d\n", hctx->cwnd, hctx->pipe);
+       ccid2_pr_debug("cwnd=%d pipe=%d\n", hctx->ccid2hctx_cwnd,
+                      hctx->ccid2hctx_pipe);
 
        /*
         * FIXME: The code below is broken and the variables have been removed
@@ -270,12 +272,12 @@ static void ccid2_hc_tx_packet_sent(struct sock *sk, unsigned int len)
         */
 #if 0
        /* Ack Ratio.  Need to maintain a concept of how many windows we sent */
-       hctx->arsent++;
+       hctx->ccid2hctx_arsent++;
        /* We had an ack loss in this window... */
-       if (hctx->ackloss) {
-               if (hctx->arsent >= hctx->cwnd) {
-                       hctx->arsent  = 0;
-                       hctx->ackloss = 0;
+       if (hctx->ccid2hctx_ackloss) {
+               if (hctx->ccid2hctx_arsent >= hctx->ccid2hctx_cwnd) {
+                       hctx->ccid2hctx_arsent  = 0;
+                       hctx->ccid2hctx_ackloss = 0;
                }
        } else {
                /* No acks lost up to now... */
@@ -285,28 +287,28 @@ static void ccid2_hc_tx_packet_sent(struct sock *sk, unsigned int len)
                        int denom = dp->dccps_l_ack_ratio * dp->dccps_l_ack_ratio -
                                    dp->dccps_l_ack_ratio;
 
-                       denom = hctx->cwnd * hctx->cwnd / denom;
+                       denom = hctx->ccid2hctx_cwnd * hctx->ccid2hctx_cwnd / denom;
 
-                       if (hctx->arsent >= denom) {
+                       if (hctx->ccid2hctx_arsent >= denom) {
                                ccid2_change_l_ack_ratio(sk, dp->dccps_l_ack_ratio - 1);
-                               hctx->arsent = 0;
+                               hctx->ccid2hctx_arsent = 0;
                        }
                } else {
                        /* we can't increase ack ratio further [1] */
-                       hctx->arsent = 0; /* or maybe set it to cwnd*/
+                       hctx->ccid2hctx_arsent = 0; /* or maybe set it to cwnd*/
                }
        }
 #endif
 
        /* setup RTO timer */
-       if (!timer_pending(&hctx->rtotimer))
+       if (!timer_pending(&hctx->ccid2hctx_rtotimer))
                ccid2_start_rto_timer(sk);
 
 #ifdef CONFIG_IP_DCCP_CCID2_DEBUG
        do {
-               struct ccid2_seq *seqp = hctx->seqt;
+               struct ccid2_seq *seqp = hctx->ccid2hctx_seqt;
 
-               while (seqp != hctx->seqh) {
+               while (seqp != hctx->ccid2hctx_seqh) {
                        ccid2_pr_debug("out seq=%llu acked=%d time=%lu\n",
                                       (unsigned long long)seqp->ccid2s_seq,
                                       seqp->ccid2s_acked, seqp->ccid2s_sent);
@@ -318,11 +320,73 @@ static void ccid2_hc_tx_packet_sent(struct sock *sk, unsigned int len)
 #endif
 }
 
+/* XXX Lame code duplication!
+ * returns -1 if none was found.
+ * else returns the next offset to use in the function call.
+ */
+static int ccid2_ackvector(struct sock *sk, struct sk_buff *skb, int offset,
+                          unsigned char **vec, unsigned char *veclen)
+{
+       const struct dccp_hdr *dh = dccp_hdr(skb);
+       unsigned char *options = (unsigned char *)dh + dccp_hdr_len(skb);
+       unsigned char *opt_ptr;
+       const unsigned char *opt_end = (unsigned char *)dh +
+                                       (dh->dccph_doff * 4);
+       unsigned char opt, len;
+       unsigned char *value;
+
+       BUG_ON(offset < 0);
+       options += offset;
+       opt_ptr = options;
+       if (opt_ptr >= opt_end)
+               return -1;
+
+       while (opt_ptr != opt_end) {
+               opt   = *opt_ptr++;
+               len   = 0;
+               value = NULL;
+
+               /* Check if this isn't a single byte option */
+               if (opt > DCCPO_MAX_RESERVED) {
+                       if (opt_ptr == opt_end)
+                               goto out_invalid_option;
+
+                       len = *opt_ptr++;
+                       if (len < 3)
+                               goto out_invalid_option;
+                       /*
+                        * Remove the type and len fields, leaving
+                        * just the value size
+                        */
+                       len     -= 2;
+                       value   = opt_ptr;
+                       opt_ptr += len;
+
+                       if (opt_ptr > opt_end)
+                               goto out_invalid_option;
+               }
+
+               switch (opt) {
+               case DCCPO_ACK_VECTOR_0:
+               case DCCPO_ACK_VECTOR_1:
+                       *vec    = value;
+                       *veclen = len;
+                       return offset + (opt_ptr - options);
+               }
+       }
+
+       return -1;
+
+out_invalid_option:
+       DCCP_BUG("Invalid option - this should not happen (previous parsing)!");
+       return -1;
+}
+
 static void ccid2_hc_tx_kill_rto_timer(struct sock *sk)
 {
        struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
 
-       sk_stop_timer(sk, &hctx->rtotimer);
+       sk_stop_timer(sk, &hctx->ccid2hctx_rtotimer);
        ccid2_pr_debug("deleted RTO timer\n");
 }
 
@@ -332,73 +396,73 @@ static inline void ccid2_new_ack(struct sock *sk,
 {
        struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
 
-       if (hctx->cwnd < hctx->ssthresh) {
-               if (*maxincr > 0 && ++hctx->packets_acked == 2) {
-                       hctx->cwnd += 1;
-                       *maxincr   -= 1;
-                       hctx->packets_acked = 0;
+       if (hctx->ccid2hctx_cwnd < hctx->ccid2hctx_ssthresh) {
+               if (*maxincr > 0 && ++hctx->ccid2hctx_packets_acked == 2) {
+                       hctx->ccid2hctx_cwnd += 1;
+                       *maxincr             -= 1;
+                       hctx->ccid2hctx_packets_acked = 0;
                }
-       } else if (++hctx->packets_acked >= hctx->cwnd) {
-                       hctx->cwnd += 1;
-                       hctx->packets_acked = 0;
+       } else if (++hctx->ccid2hctx_packets_acked >= hctx->ccid2hctx_cwnd) {
+                       hctx->ccid2hctx_cwnd += 1;
+                       hctx->ccid2hctx_packets_acked = 0;
        }
 
        /* update RTO */
-       if (hctx->srtt == -1 ||
-           time_after(jiffies, hctx->lastrtt + hctx->srtt)) {
+       if (hctx->ccid2hctx_srtt == -1 ||
+           time_after(jiffies, hctx->ccid2hctx_lastrtt + hctx->ccid2hctx_srtt)) {
                unsigned long r = (long)jiffies - (long)seqp->ccid2s_sent;
                int s;
 
                /* first measurement */
-               if (hctx->srtt == -1) {
+               if (hctx->ccid2hctx_srtt == -1) {
                        ccid2_pr_debug("R: %lu Time=%lu seq=%llu\n",
                                       r, jiffies,
                                       (unsigned long long)seqp->ccid2s_seq);
                        ccid2_change_srtt(hctx, r);
-                       hctx->rttvar = r >> 1;
+                       hctx->ccid2hctx_rttvar = r >> 1;
                } else {
                        /* RTTVAR */
-                       long tmp = hctx->srtt - r;
+                       long tmp = hctx->ccid2hctx_srtt - r;
                        long srtt;
 
                        if (tmp < 0)
                                tmp *= -1;
 
                        tmp >>= 2;
-                       hctx->rttvar *= 3;
-                       hctx->rttvar >>= 2;
-                       hctx->rttvar += tmp;
+                       hctx->ccid2hctx_rttvar *= 3;
+                       hctx->ccid2hctx_rttvar >>= 2;
+                       hctx->ccid2hctx_rttvar += tmp;
 
                        /* SRTT */
-                       srtt = hctx->srtt;
+                       srtt = hctx->ccid2hctx_srtt;
                        srtt *= 7;
                        srtt >>= 3;
                        tmp = r >> 3;
                        srtt += tmp;
                        ccid2_change_srtt(hctx, srtt);
                }
-               s = hctx->rttvar << 2;
+               s = hctx->ccid2hctx_rttvar << 2;
                /* clock granularity is 1 when based on jiffies */
                if (!s)
                        s = 1;
-               hctx->rto = hctx->srtt + s;
+               hctx->ccid2hctx_rto = hctx->ccid2hctx_srtt + s;
 
                /* must be at least a second */
-               s = hctx->rto / HZ;
+               s = hctx->ccid2hctx_rto / HZ;
                /* DCCP doesn't require this [but I like it cuz my code sux] */
 #if 1
                if (s < 1)
-                       hctx->rto = HZ;
+                       hctx->ccid2hctx_rto = HZ;
 #endif
                /* max 60 seconds */
                if (s > 60)
-                       hctx->rto = HZ * 60;
+                       hctx->ccid2hctx_rto = HZ * 60;
 
-               hctx->lastrtt = jiffies;
+               hctx->ccid2hctx_lastrtt = jiffies;
 
                ccid2_pr_debug("srtt: %ld rttvar: %ld rto: %ld (HZ=%d) R=%lu\n",
-                              hctx->srtt, hctx->rttvar,
-                              hctx->rto, HZ, r);
+                              hctx->ccid2hctx_srtt, hctx->ccid2hctx_rttvar,
+                              hctx->ccid2hctx_rto, HZ, r);
        }
 
        /* we got a new ack, so re-start RTO timer */
@@ -410,12 +474,12 @@ static void ccid2_hc_tx_dec_pipe(struct sock *sk)
 {
        struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
 
-       if (hctx->pipe == 0)
+       if (hctx->ccid2hctx_pipe == 0)
                DCCP_BUG("pipe == 0");
        else
-               hctx->pipe--;
+               hctx->ccid2hctx_pipe--;
 
-       if (hctx->pipe == 0)
+       if (hctx->ccid2hctx_pipe == 0)
                ccid2_hc_tx_kill_rto_timer(sk);
 }
 
@@ -423,43 +487,30 @@ static void ccid2_congestion_event(struct sock *sk, struct ccid2_seq *seqp)
 {
        struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
 
-       if (time_before(seqp->ccid2s_sent, hctx->last_cong)) {
+       if (time_before(seqp->ccid2s_sent, hctx->ccid2hctx_last_cong)) {
                ccid2_pr_debug("Multiple losses in an RTT---treating as one\n");
                return;
        }
 
-       hctx->last_cong = jiffies;
+       hctx->ccid2hctx_last_cong = jiffies;
 
-       hctx->cwnd     = hctx->cwnd / 2 ? : 1U;
-       hctx->ssthresh = max(hctx->cwnd, 2U);
+       hctx->ccid2hctx_cwnd     = hctx->ccid2hctx_cwnd / 2 ? : 1U;
+       hctx->ccid2hctx_ssthresh = max(hctx->ccid2hctx_cwnd, 2U);
 
        /* Avoid spurious timeouts resulting from Ack Ratio > cwnd */
-       if (dccp_sk(sk)->dccps_l_ack_ratio > hctx->cwnd)
-               ccid2_change_l_ack_ratio(sk, hctx->cwnd);
-}
-
-static int ccid2_hc_tx_parse_options(struct sock *sk, u8 packet_type,
-                                    u8 option, u8 *optval, u8 optlen)
-{
-       struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
-
-       switch (option) {
-       case DCCPO_ACK_VECTOR_0:
-       case DCCPO_ACK_VECTOR_1:
-               return dccp_ackvec_parsed_add(&hctx->av_chunks, optval, optlen,
-                                             option - DCCPO_ACK_VECTOR_0);
-       }
-       return 0;
+       if (dccp_sk(sk)->dccps_l_ack_ratio > hctx->ccid2hctx_cwnd)
+               ccid2_change_l_ack_ratio(sk, hctx->ccid2hctx_cwnd);
 }
 
 static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
 {
        struct dccp_sock *dp = dccp_sk(sk);
        struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk);
-       const bool sender_was_blocked = ccid2_cwnd_network_limited(hctx);
-       struct dccp_ackvec_parsed *avp;
        u64 ackno, seqno;
        struct ccid2_seq *seqp;
+       unsigned char *vector;
+       unsigned char veclen;
+       int offset = 0;
        int done = 0;
        unsigned int maxincr = 0;
 
@@ -472,21 +523,21 @@ static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
         * -sorbo.
         */
        /* need to bootstrap */
-       if (hctx->rpdupack == -1) {
-               hctx->rpdupack = 0;
-               hctx->rpseq = seqno;
+       if (hctx->ccid2hctx_rpdupack == -1) {
+               hctx->ccid2hctx_rpdupack = 0;
+               hctx->ccid2hctx_rpseq = seqno;
        } else {
                /* check if packet is consecutive */
-               if (dccp_delta_seqno(hctx->rpseq, seqno) == 1)
-                       hctx->rpseq = seqno;
+               if (dccp_delta_seqno(hctx->ccid2hctx_rpseq, seqno) == 1)
+                       hctx->ccid2hctx_rpseq = seqno;
                /* it's a later packet */
-               else if (after48(seqno, hctx->rpseq)) {
-                       hctx->rpdupack++;
+               else if (after48(seqno, hctx->ccid2hctx_rpseq)) {
+                       hctx->ccid2hctx_rpdupack++;
 
                        /* check if we got enough dupacks */
-                       if (hctx->rpdupack >= NUMDUPACK) {
-                               hctx->rpdupack = -1; /* XXX lame */
-                               hctx->rpseq = 0;
+                       if (hctx->ccid2hctx_rpdupack >= NUMDUPACK) {
+                               hctx->ccid2hctx_rpdupack = -1; /* XXX lame */
+                               hctx->ccid2hctx_rpseq = 0;
 
                                ccid2_change_l_ack_ratio(sk, 2 * dp->dccps_l_ack_ratio);
                        }
@@ -494,22 +545,27 @@ static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
        }
 
        /* check forward path congestion */
-       if (dccp_packet_without_ack(skb))
+       /* still didn't send out new data packets */
+       if (hctx->ccid2hctx_seqh == hctx->ccid2hctx_seqt)
                return;
 
-       /* still didn't send out new data packets */
-       if (hctx->seqh == hctx->seqt)
-               goto done;
+       switch (DCCP_SKB_CB(skb)->dccpd_type) {
+       case DCCP_PKT_ACK:
+       case DCCP_PKT_DATAACK:
+               break;
+       default:
+               return;
+       }
 
        ackno = DCCP_SKB_CB(skb)->dccpd_ack_seq;
-       if (after48(ackno, hctx->high_ack))
-               hctx->high_ack = ackno;
+       if (after48(ackno, hctx->ccid2hctx_high_ack))
+               hctx->ccid2hctx_high_ack = ackno;
 
-       seqp = hctx->seqt;
+       seqp = hctx->ccid2hctx_seqt;
        while (before48(seqp->ccid2s_seq, ackno)) {
                seqp = seqp->ccid2s_next;
-               if (seqp == hctx->seqh) {
-                       seqp = hctx->seqh->ccid2s_prev;
+               if (seqp == hctx->ccid2hctx_seqh) {
+                       seqp = hctx->ccid2hctx_seqh->ccid2s_prev;
                        break;
                }
        }
@@ -519,26 +575,26 @@ static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
         * packets per acknowledgement. Rounding up avoids that cwnd is not
         * advanced when Ack Ratio is 1 and gives a slight edge otherwise.
         */
-       if (hctx->cwnd < hctx->ssthresh)
+       if (hctx->ccid2hctx_cwnd < hctx->ccid2hctx_ssthresh)
                maxincr = DIV_ROUND_UP(dp->dccps_l_ack_ratio, 2);
 
        /* go through all ack vectors */
-       list_for_each_entry(avp, &hctx->av_chunks, node) {
+       while ((offset = ccid2_ackvector(sk, skb, offset,
+                                        &vector, &veclen)) != -1) {
                /* go through this ack vector */
-               for (; avp->len--; avp->vec++) {
-                       u64 ackno_end_rl = SUB48(ackno,
-                                                dccp_ackvec_runlen(avp->vec));
+               while (veclen--) {
+                       const u8 rl = *vector & DCCP_ACKVEC_LEN_MASK;
+                       u64 ackno_end_rl = SUB48(ackno, rl);
 
-                       ccid2_pr_debug("ackvec %llu |%u,%u|\n",
+                       ccid2_pr_debug("ackvec start:%llu end:%llu\n",
                                       (unsigned long long)ackno,
-                                      dccp_ackvec_state(avp->vec) >> 6,
-                                      dccp_ackvec_runlen(avp->vec));
+                                      (unsigned long long)ackno_end_rl);
                        /* if the seqno we are analyzing is larger than the
                         * current ackno, then move towards the tail of our
                         * seqnos.
                         */
                        while (after48(seqp->ccid2s_seq, ackno)) {
-                               if (seqp == hctx->seqt) {
+                               if (seqp == hctx->ccid2hctx_seqt) {
                                        done = 1;
                                        break;
                                }
@@ -551,15 +607,17 @@ static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
                         * run length
                         */
                        while (between48(seqp->ccid2s_seq,ackno_end_rl,ackno)) {
-                               const u8 state = dccp_ackvec_state(avp->vec);
+                               const u8 state = *vector &
+                                                DCCP_ACKVEC_STATE_MASK;
 
                                /* new packet received or marked */
-                               if (state != DCCPAV_NOT_RECEIVED &&
+                               if (state != DCCP_ACKVEC_STATE_NOT_RECEIVED &&
                                    !seqp->ccid2s_acked) {
-                                       if (state == DCCPAV_ECN_MARKED)
+                                       if (state ==
+                                           DCCP_ACKVEC_STATE_ECN_MARKED) {
                                                ccid2_congestion_event(sk,
                                                                       seqp);
-                                       else
+                                       else
                                                ccid2_new_ack(sk, seqp,
                                                              &maxincr);
 
@@ -568,7 +626,7 @@ static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
                                                       (unsigned long long)seqp->ccid2s_seq);
                                        ccid2_hc_tx_dec_pipe(sk);
                                }
-                               if (seqp == hctx->seqt) {
+                               if (seqp == hctx->ccid2hctx_seqt) {
                                        done = 1;
                                        break;
                                }
@@ -578,6 +636,7 @@ static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
                                break;
 
                        ackno = SUB48(ackno_end_rl, 1);
+                       vector++;
                }
                if (done)
                        break;
@@ -586,11 +645,11 @@ static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
        /* The state about what is acked should be correct now
         * Check for NUMDUPACK
         */
-       seqp = hctx->seqt;
-       while (before48(seqp->ccid2s_seq, hctx->high_ack)) {
+       seqp = hctx->ccid2hctx_seqt;
+       while (before48(seqp->ccid2s_seq, hctx->ccid2hctx_high_ack)) {
                seqp = seqp->ccid2s_next;
-               if (seqp == hctx->seqh) {
-                       seqp = hctx->seqh->ccid2s_prev;
+               if (seqp == hctx->ccid2hctx_seqh) {
+                       seqp = hctx->ccid2hctx_seqh->ccid2s_prev;
                        break;
                }
        }
@@ -601,7 +660,7 @@ static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
                        if (done == NUMDUPACK)
                                break;
                }
-               if (seqp == hctx->seqt)
+               if (seqp == hctx->ccid2hctx_seqt)
                        break;
                seqp = seqp->ccid2s_prev;
        }
@@ -624,28 +683,23 @@ static void ccid2_hc_tx_packet_recv(struct sock *sk, struct sk_buff *skb)
                                ccid2_congestion_event(sk, seqp);
                                ccid2_hc_tx_dec_pipe(sk);
                        }
-                       if (seqp == hctx->seqt)
+                       if (seqp == hctx->ccid2hctx_seqt)
                                break;
                        seqp = seqp->ccid2s_prev;
                }
 
-               hctx->seqt = last_acked;
+               hctx->ccid2hctx_seqt = last_acked;
        }
 
        /* trim acked packets in tail */
-       while (hctx->seqt != hctx->seqh) {
-               if (!hctx->seqt->ccid2s_acked)
+       while (hctx->ccid2hctx_seqt != hctx->ccid2hctx_seqh) {
+               if (!hctx->ccid2hctx_seqt->ccid2s_acked)
                        break;
 
-               hctx->seqt = hctx->seqt->ccid2s_next;
+               hctx->ccid2hctx_seqt = hctx->ccid2hctx_seqt->ccid2s_next;
        }
 
        ccid2_hc_tx_check_sanity(hctx);
-done:
-       /* check if incoming Acks allow pending packets to be sent */
-       if (sender_was_blocked && !ccid2_cwnd_network_limited(hctx))
-               tasklet_schedule(&dccp_sk(sk)->dccps_xmitlet);
-       dccp_ackvec_parsed_cleanup(&hctx->av_chunks);
 }
 
 static int ccid2_hc_tx_init(struct ccid *ccid, struct sock *sk)
@@ -655,17 +709,17 @@ static int ccid2_hc_tx_init(struct ccid *ccid, struct sock *sk)
        u32 max_ratio;
 
        /* RFC 4341, 5: initialise ssthresh to arbitrarily high (max) value */
-       hctx->ssthresh = ~0U;
+       hctx->ccid2hctx_ssthresh  = ~0U;
 
        /*
         * RFC 4341, 5: "The cwnd parameter is initialized to at most four
         * packets for new connections, following the rules from [RFC3390]".
         * We need to convert the bytes of RFC3390 into the packets of RFC 4341.
         */
-       hctx->cwnd = clamp(4380U / dp->dccps_mss_cache, 2U, 4U);
+       hctx->ccid2hctx_cwnd = clamp(4380U / dp->dccps_mss_cache, 2U, 4U);
 
        /* Make sure that Ack Ratio is enabled and within bounds. */
-       max_ratio = DIV_ROUND_UP(hctx->cwnd, 2);
+       max_ratio = DIV_ROUND_UP(hctx->ccid2hctx_cwnd, 2);
        if (dp->dccps_l_ack_ratio == 0 || dp->dccps_l_ack_ratio > max_ratio)
                dp->dccps_l_ack_ratio = max_ratio;
 
@@ -673,13 +727,13 @@ static int ccid2_hc_tx_init(struct ccid *ccid, struct sock *sk)
        if (ccid2_hc_tx_alloc_seq(hctx))
                return -ENOMEM;
 
-       hctx->rto        = 3 * HZ;
+       hctx->ccid2hctx_rto      = 3 * HZ;
        ccid2_change_srtt(hctx, -1);
-       hctx->rttvar    = -1;
-       hctx->rpdupack  = -1;
-       hctx->last_cong = jiffies;
-       setup_timer(&hctx->rtotimer, ccid2_hc_tx_rto_expire, (unsigned long)sk);
-       INIT_LIST_HEAD(&hctx->av_chunks);
+       hctx->ccid2hctx_rttvar   = -1;
+       hctx->ccid2hctx_rpdupack = -1;
+       hctx->ccid2hctx_last_cong = jiffies;
+       setup_timer(&hctx->ccid2hctx_rtotimer, ccid2_hc_tx_rto_expire,
+                       (unsigned long)sk);
 
        ccid2_hc_tx_check_sanity(hctx);
        return 0;
@@ -692,9 +746,9 @@ static void ccid2_hc_tx_exit(struct sock *sk)
 
        ccid2_hc_tx_kill_rto_timer(sk);
 
-       for (i = 0; i < hctx->seqbufc; i++)
-               kfree(hctx->seqbuf[i]);
-       hctx->seqbufc = 0;
+       for (i = 0; i < hctx->ccid2hctx_seqbufc; i++)
+               kfree(hctx->ccid2hctx_seqbuf[i]);
+       hctx->ccid2hctx_seqbufc = 0;
 }
 
 static void ccid2_hc_rx_packet_recv(struct sock *sk, struct sk_buff *skb)
@@ -705,28 +759,27 @@ static void ccid2_hc_rx_packet_recv(struct sock *sk, struct sk_buff *skb)
        switch (DCCP_SKB_CB(skb)->dccpd_type) {
        case DCCP_PKT_DATA:
        case DCCP_PKT_DATAACK:
-               hcrx->data++;
-               if (hcrx->data >= dp->dccps_r_ack_ratio) {
+               hcrx->ccid2hcrx_data++;
+               if (hcrx->ccid2hcrx_data >= dp->dccps_r_ack_ratio) {
                        dccp_send_ack(sk);
-                       hcrx->data = 0;
+                       hcrx->ccid2hcrx_data = 0;
                }
                break;
        }
 }
 
 static struct ccid_operations ccid2 = {
-       .ccid_id                  = DCCPC_CCID2,
-       .ccid_name                = "TCP-like",
-       .ccid_owner               = THIS_MODULE,
-       .ccid_hc_tx_obj_size      = sizeof(struct ccid2_hc_tx_sock),
-       .ccid_hc_tx_init          = ccid2_hc_tx_init,
-       .ccid_hc_tx_exit          = ccid2_hc_tx_exit,
-       .ccid_hc_tx_send_packet   = ccid2_hc_tx_send_packet,
-       .ccid_hc_tx_packet_sent   = ccid2_hc_tx_packet_sent,
-       .ccid_hc_tx_parse_options = ccid2_hc_tx_parse_options,
-       .ccid_hc_tx_packet_recv   = ccid2_hc_tx_packet_recv,
-       .ccid_hc_rx_obj_size      = sizeof(struct ccid2_hc_rx_sock),
-       .ccid_hc_rx_packet_recv   = ccid2_hc_rx_packet_recv,
+       .ccid_id                = DCCPC_CCID2,
+       .ccid_name              = "TCP-like",
+       .ccid_owner             = THIS_MODULE,
+       .ccid_hc_tx_obj_size    = sizeof(struct ccid2_hc_tx_sock),
+       .ccid_hc_tx_init        = ccid2_hc_tx_init,
+       .ccid_hc_tx_exit        = ccid2_hc_tx_exit,
+       .ccid_hc_tx_send_packet = ccid2_hc_tx_send_packet,
+       .ccid_hc_tx_packet_sent = ccid2_hc_tx_packet_sent,
+       .ccid_hc_tx_packet_recv = ccid2_hc_tx_packet_recv,
+       .ccid_hc_rx_obj_size    = sizeof(struct ccid2_hc_rx_sock),
+       .ccid_hc_rx_packet_recv = ccid2_hc_rx_packet_recv,
 };
 
 #ifdef CONFIG_IP_DCCP_CCID2_DEBUG