vmbus: remove unused kickq argument to sendpacket
authorStephen Hemminger <stephen@networkplumber.org>
Mon, 6 Feb 2017 00:20:34 +0000 (17:20 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 10 Feb 2017 14:45:07 +0000 (15:45 +0100)
Since sendpacket no longer uses kickq argument remove it.
Remove it no longer used xmit_more in sendpacket in netvsc as well.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/hv/channel.c
drivers/net/hyperv/netvsc.c
include/linux/hyperv.h

index e26285c..789c75f 100644 (file)
@@ -643,8 +643,8 @@ void vmbus_close(struct vmbus_channel *channel)
 EXPORT_SYMBOL_GPL(vmbus_close);
 
 int vmbus_sendpacket_ctl(struct vmbus_channel *channel, void *buffer,
-                          u32 bufferlen, u64 requestid,
-                          enum vmbus_packet_type type, u32 flags, bool kick_q)
+                        u32 bufferlen, u64 requestid,
+                        enum vmbus_packet_type type, u32 flags)
 {
        struct vmpacket_descriptor desc;
        u32 packetlen = sizeof(struct vmpacket_descriptor) + bufferlen;
@@ -693,7 +693,7 @@ int vmbus_sendpacket(struct vmbus_channel *channel, void *buffer,
                           enum vmbus_packet_type type, u32 flags)
 {
        return vmbus_sendpacket_ctl(channel, buffer, bufferlen, requestid,
-                                   type, flags, true);
+                                   type, flags);
 }
 EXPORT_SYMBOL(vmbus_sendpacket);
 
@@ -705,11 +705,9 @@ EXPORT_SYMBOL(vmbus_sendpacket);
  * explicitly.
  */
 int vmbus_sendpacket_pagebuffer_ctl(struct vmbus_channel *channel,
-                                    struct hv_page_buffer pagebuffers[],
-                                    u32 pagecount, void *buffer, u32 bufferlen,
-                                    u64 requestid,
-                                    u32 flags,
-                                    bool kick_q)
+                                   struct hv_page_buffer pagebuffers[],
+                                   u32 pagecount, void *buffer, u32 bufferlen,
+                                   u64 requestid, u32 flags)
 {
        int i;
        struct vmbus_channel_packet_page_buffer desc;
@@ -769,9 +767,10 @@ int vmbus_sendpacket_pagebuffer(struct vmbus_channel *channel,
                                     u64 requestid)
 {
        u32 flags = VMBUS_DATA_PACKET_FLAG_COMPLETION_REQUESTED;
+
        return vmbus_sendpacket_pagebuffer_ctl(channel, pagebuffers, pagecount,
-                                              buffer, bufferlen, requestid,
-                                              flags, true);
+                                              buffer, bufferlen,
+                                              requestid, flags);
 
 }
 EXPORT_SYMBOL_GPL(vmbus_sendpacket_pagebuffer);
index 86e5749..3726030 100644 (file)
@@ -723,8 +723,6 @@ static u32 netvsc_copy_to_send_buf(struct netvsc_device *net_device,
        char *dest = start + (section_index * net_device->send_section_size)
                     + pend_size;
        int i;
-       bool is_data_pkt = (skb != NULL) ? true : false;
-       bool xmit_more = (skb != NULL) ? skb->xmit_more : false;
        u32 msg_size = 0;
        u32 padding = 0;
        u32 remain = packet->total_data_buflen % net_device->pkt_align;
@@ -732,7 +730,7 @@ static u32 netvsc_copy_to_send_buf(struct netvsc_device *net_device,
                packet->page_buf_cnt;
 
        /* Add padding */
-       if (is_data_pkt && xmit_more && remain &&
+       if (skb && skb->xmit_more && remain &&
            !packet->cp_partial) {
                padding = net_device->pkt_align - remain;
                rndis_msg->msg_len += padding;
@@ -772,7 +770,6 @@ static inline int netvsc_send_pkt(
        int ret;
        struct hv_page_buffer *pgbuf;
        u32 ring_avail = hv_ringbuf_avail_percent(&out_channel->outbound);
-       bool xmit_more = (skb != NULL) ? skb->xmit_more : false;
 
        nvmsg.hdr.msg_type = NVSP_MSG1_TYPE_SEND_RNDIS_PKT;
        if (skb != NULL) {
@@ -796,16 +793,6 @@ static inline int netvsc_send_pkt(
        if (out_channel->rescind)
                return -ENODEV;
 
-       /*
-        * It is possible that once we successfully place this packet
-        * on the ringbuffer, we may stop the queue. In that case, we want
-        * to notify the host independent of the xmit_more flag. We don't
-        * need to be precise here; in the worst case we may signal the host
-        * unnecessarily.
-        */
-       if (ring_avail < (RING_AVAIL_PERCENT_LOWATER + 1))
-               xmit_more = false;
-
        if (packet->page_buf_cnt) {
                pgbuf = packet->cp_partial ? (*pb) +
                        packet->rmsg_pgcnt : (*pb);
@@ -815,15 +802,13 @@ static inline int netvsc_send_pkt(
                                                      &nvmsg,
                                                      sizeof(struct nvsp_message),
                                                      req_id,
-                                                     VMBUS_DATA_PACKET_FLAG_COMPLETION_REQUESTED,
-                                                     !xmit_more);
+                                                     VMBUS_DATA_PACKET_FLAG_COMPLETION_REQUESTED);
        } else {
                ret = vmbus_sendpacket_ctl(out_channel, &nvmsg,
                                           sizeof(struct nvsp_message),
                                           req_id,
                                           VM_PKT_DATA_INBAND,
-                                          VMBUS_DATA_PACKET_FLAG_COMPLETION_REQUESTED,
-                                          !xmit_more);
+                                          VMBUS_DATA_PACKET_FLAG_COMPLETION_REQUESTED);
        }
 
        if (ret == 0) {
index 7795966..e208e64 100644 (file)
@@ -1037,8 +1037,7 @@ extern int vmbus_sendpacket_ctl(struct vmbus_channel *channel,
                                  u32 bufferLen,
                                  u64 requestid,
                                  enum vmbus_packet_type type,
-                                 u32 flags,
-                                 bool kick_q);
+                                 u32 flags);
 
 extern int vmbus_sendpacket_pagebuffer(struct vmbus_channel *channel,
                                            struct hv_page_buffer pagebuffers[],
@@ -1053,8 +1052,7 @@ extern int vmbus_sendpacket_pagebuffer_ctl(struct vmbus_channel *channel,
                                           void *buffer,
                                           u32 bufferlen,
                                           u64 requestid,
-                                          u32 flags,
-                                          bool kick_q);
+                                          u32 flags);
 
 extern int vmbus_sendpacket_multipagebuffer(struct vmbus_channel *channel,
                                        struct hv_multipage_buffer *mpb,