IB/rxe: Change rxe_rcv to return void
authorYuval Shaia <yuval.shaia@oracle.com>
Fri, 20 Apr 2018 14:05:03 +0000 (17:05 +0300)
committerDoug Ledford <dledford@redhat.com>
Fri, 27 Apr 2018 16:17:56 +0000 (12:17 -0400)
It always returns 0. Change return type to void.

Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
Reviewed-by: Zhu Yanjun <yanjun.zhu@oracle.com>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/sw/rxe/rxe.h
drivers/infiniband/sw/rxe/rxe_loc.h
drivers/infiniband/sw/rxe/rxe_net.c
drivers/infiniband/sw/rxe/rxe_recv.c

index 561ad307c6ecacf39c3adc1e531f1517b21c9d47..1275fde6503a900a7afaf838081e55e3093b0770 100644 (file)
@@ -98,7 +98,7 @@ int rxe_add(struct rxe_dev *rxe, unsigned int mtu);
 void rxe_remove(struct rxe_dev *rxe);
 void rxe_remove_all(void);
 
-int rxe_rcv(struct sk_buff *skb);
+void rxe_rcv(struct sk_buff *skb);
 
 static inline void rxe_dev_put(struct rxe_dev *rxe)
 {
index b71023c1c58bc7eebd3a604d80475f8a4a1bbe46..2f8ab8eebcb1d84360201b5436b5b523197adf18 100644 (file)
@@ -143,7 +143,7 @@ void rxe_mem_cleanup(struct rxe_pool_entry *arg);
 int advance_dma_data(struct rxe_dma_info *dma, unsigned int length);
 
 /* rxe_net.c */
-int rxe_loopback(struct sk_buff *skb);
+void rxe_loopback(struct sk_buff *skb);
 int rxe_send(struct rxe_pkt_info *pkt, struct sk_buff *skb);
 struct sk_buff *rxe_init_packet(struct rxe_dev *rxe, struct rxe_av *av,
                                int paylen, struct rxe_pkt_info *pkt);
@@ -268,7 +268,8 @@ static inline int rxe_xmit_packet(struct rxe_dev *rxe, struct rxe_qp *qp,
 
        if (pkt->mask & RXE_LOOPBACK_MASK) {
                memcpy(SKB_TO_PKT(skb), pkt, sizeof(*pkt));
-               err = rxe_loopback(skb);
+               rxe_loopback(skb);
+               err = 0;
        } else {
                err = rxe_send(pkt, skb);
        }
index 97e128579137ed7074972299e5277cecda8eb311..fca13a6281f057c8c2d9cd2ead56761fd80ed11f 100644 (file)
@@ -276,9 +276,12 @@ static int rxe_udp_encap_recv(struct sock *sk, struct sk_buff *skb)
        pkt->mask = RXE_GRH_MASK;
        pkt->paylen = be16_to_cpu(udph->len) - sizeof(*udph);
 
-       return rxe_rcv(skb);
+       rxe_rcv(skb);
+
+       return 0;
 drop:
        kfree_skb(skb);
+
        return 0;
 }
 
@@ -517,9 +520,9 @@ int rxe_send(struct rxe_pkt_info *pkt, struct sk_buff *skb)
        return 0;
 }
 
-int rxe_loopback(struct sk_buff *skb)
+void rxe_loopback(struct sk_buff *skb)
 {
-       return rxe_rcv(skb);
+       rxe_rcv(skb);
 }
 
 static inline int addr_same(struct rxe_dev *rxe, struct rxe_av *av)
index c4c9f3be33d92fcd823fe4269b86db2bcd3f1985..dfba44a40f0b4a92ae0102e634e693ff81a5a642 100644 (file)
@@ -345,7 +345,7 @@ static int rxe_match_dgid(struct rxe_dev *rxe, struct sk_buff *skb)
 }
 
 /* rxe_rcv is called from the interface driver */
-int rxe_rcv(struct sk_buff *skb)
+void rxe_rcv(struct sk_buff *skb)
 {
        int err;
        struct rxe_pkt_info *pkt = SKB_TO_PKT(skb);
@@ -403,12 +403,11 @@ int rxe_rcv(struct sk_buff *skb)
        else
                rxe_rcv_pkt(rxe, pkt, skb);
 
-       return 0;
+       return;
 
 drop:
        if (pkt->qp)
                rxe_drop_ref(pkt->qp);
 
        kfree_skb(skb);
-       return 0;
 }