Phonet: restore flow control credits when sending fails
authorRémi Denis-Courmont <remi.denis-courmont@nokia.com>
Wed, 29 Sep 2010 22:33:50 +0000 (22:33 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 1 Oct 2010 00:57:30 +0000 (17:57 -0700)
This patch restores the below flow control patch submitted by Rémi
Denis-Courmont, which accidentaly got lost due to Pipe controller patch
on Phonet.

commit 1a98214feef2221cd7c24b17cd688a5a9d85b2ea
Author: Rémi Denis-Courmont <remi.denis-courmont@nokia.com>
Date:   Mon Aug 30 12:57:03 2010 +0000

Phonet: restore flow control credits when sending fails

Signed-off-by: Rémi Denis-Courmont <remi.denis-courmont@nokia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Kumar Sanghvi <kumar.sanghvi@stericsson.com>
Acked-by: Linus Walleij <linus.walleij@stericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/phonet/pep.c

index 7bf23cf36b02052396e5c64037ba992f68e47fe7..552fb665645f005d33b7249bf733fb28a3d7092f 100644 (file)
@@ -1288,6 +1288,7 @@ static int pipe_skb_send(struct sock *sk, struct sk_buff *skb)
 {
        struct pep_sock *pn = pep_sk(sk);
        struct pnpipehdr *ph;
+       int err;
 #ifdef CONFIG_PHONET_PIPECTRLR
        struct sockaddr_pn spn = {
                .spn_family = AF_PHONET,
@@ -1314,10 +1315,15 @@ static int pipe_skb_send(struct sock *sk, struct sk_buff *skb)
                ph->message_id = PNS_PIPE_DATA;
        ph->pipe_handle = pn->pipe_handle;
 #ifdef CONFIG_PHONET_PIPECTRLR
-       return pn_skb_send(sk, skb, &spn);
+       err = pn_skb_send(sk, skb, &spn);
 #else
-       return pn_skb_send(sk, skb, &pipe_srv);
+       err = pn_skb_send(sk, skb, &pipe_srv);
 #endif
+
+       if (err && pn_flow_safe(pn->tx_fc))
+               atomic_inc(&pn->tx_credits);
+       return err;
+
 }
 
 static int pep_sendmsg(struct kiocb *iocb, struct sock *sk,