hv_netvsc: Eliminatte the data field from struct hv_netvsc_packet
authorKY Srinivasan <kys@microsoft.com>
Wed, 2 Dec 2015 00:43:07 +0000 (16:43 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 3 Dec 2015 04:43:24 +0000 (23:43 -0500)
Eliminatte the data field from struct hv_netvsc_packet.

Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Reviewed-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/hyperv/hyperv_net.h
drivers/net/hyperv/netvsc.c
drivers/net/hyperv/netvsc_drv.c
drivers/net/hyperv/rndis_filter.c

index 14deedd1b4113ee53b0ae5add8d108f2f055258f..62542081a864c6ca3d4ca356b39bb8d5749566b9 100644 (file)
@@ -148,9 +148,6 @@ struct hv_netvsc_packet {
        u64 send_completion_tid;
        void *send_completion_ctx;
        void (*send_completion)(void *context);
-
-       /* Points to the send/receive buffer where the ethernet frame is */
-       void *data;
        struct hv_page_buffer *page_buf;
 };
 
@@ -195,6 +192,7 @@ void netvsc_linkstatus_callback(struct hv_device *device_obj,
 void netvsc_xmit_completion(void *context);
 int netvsc_recv_callback(struct hv_device *device_obj,
                        struct hv_netvsc_packet *packet,
+                       void **data,
                        struct ndis_tcp_ip_checksum_info *csum_info,
                        struct vmbus_channel *channel);
 void netvsc_channel_cb(void *context);
@@ -205,6 +203,7 @@ int rndis_filter_device_add(struct hv_device *dev,
 void rndis_filter_device_remove(struct hv_device *dev);
 int rndis_filter_receive(struct hv_device *dev,
                        struct hv_netvsc_packet *pkt,
+                       void **data,
                        struct vmbus_channel *channel);
 
 int rndis_filter_set_packet_filter(struct rndis_device *dev, u32 new_filter);
index 2de9e7fb4f686bef3f426731e46995421192bcd2..8fbf81626bc3096909c046ab8a4aa77eb7f47948 100644 (file)
@@ -1008,6 +1008,7 @@ static void netvsc_receive(struct netvsc_device *net_device,
        int i;
        int count = 0;
        struct net_device *ndev;
+       void *data;
 
        ndev = net_device->ndev;
 
@@ -1047,13 +1048,13 @@ static void netvsc_receive(struct netvsc_device *net_device,
        for (i = 0; i < count; i++) {
                /* Initialize the netvsc packet */
                netvsc_packet->status = NVSP_STAT_SUCCESS;
-               netvsc_packet->data = (void *)((unsigned long)net_device->
+               data = (void *)((unsigned long)net_device->
                        recv_buf + vmxferpage_packet->ranges[i].byte_offset);
                netvsc_packet->total_data_buflen =
                                        vmxferpage_packet->ranges[i].byte_count;
 
                /* Pass it to the upper layer */
-               rndis_filter_receive(device, netvsc_packet, channel);
+               rndis_filter_receive(device, netvsc_packet, &data, channel);
 
                if (netvsc_packet->status != NVSP_STAT_SUCCESS)
                        status = NVSP_STAT_FAIL;
index e5f81c797e26ec443093e71d1fbbc2d6c3213556..622e62e326363fb5d9f0180428fcf453ec5a3943 100644 (file)
@@ -682,6 +682,7 @@ void netvsc_linkstatus_callback(struct hv_device *device_obj,
  */
 int netvsc_recv_callback(struct hv_device *device_obj,
                                struct hv_netvsc_packet *packet,
+                               void **data,
                                struct ndis_tcp_ip_checksum_info *csum_info,
                                struct vmbus_channel *channel)
 {
@@ -710,7 +711,7 @@ int netvsc_recv_callback(struct hv_device *device_obj,
         * Copy to skb. This copy is needed here since the memory pointed by
         * hv_netvsc_packet cannot be deallocated
         */
-       memcpy(skb_put(skb, packet->total_data_buflen), packet->data,
+       memcpy(skb_put(skb, packet->total_data_buflen), *data,
                packet->total_data_buflen);
 
        skb->protocol = eth_type_trans(skb, net);
index 63584e7a49e37ca2235de297df03f88d76dad952..be0fa9c94f6362100ae4e9cf1e22cfd79bdb4e6e 100644 (file)
@@ -351,6 +351,7 @@ static inline void *rndis_get_ppi(struct rndis_packet *rpkt, u32 type)
 static void rndis_filter_receive_data(struct rndis_device *dev,
                                   struct rndis_message *msg,
                                   struct hv_netvsc_packet *pkt,
+                                  void **data,
                                   struct vmbus_channel *channel)
 {
        struct rndis_packet *rndis_pkt;
@@ -383,7 +384,7 @@ static void rndis_filter_receive_data(struct rndis_device *dev,
         * the data packet to the stack, without the rndis trailer padding
         */
        pkt->total_data_buflen = rndis_pkt->data_len;
-       pkt->data = (void *)((unsigned long)pkt->data + data_offset);
+       *data = (void *)((unsigned long)(*data) + data_offset);
 
        vlan = rndis_get_ppi(rndis_pkt, IEEE_8021Q_INFO);
        if (vlan) {
@@ -394,11 +395,12 @@ static void rndis_filter_receive_data(struct rndis_device *dev,
        }
 
        csum_info = rndis_get_ppi(rndis_pkt, TCPIP_CHKSUM_PKTINFO);
-       netvsc_recv_callback(dev->net_dev->dev, pkt, csum_info, channel);
+       netvsc_recv_callback(dev->net_dev->dev, pkt, data, csum_info, channel);
 }
 
 int rndis_filter_receive(struct hv_device *dev,
                                struct hv_netvsc_packet *pkt,
+                               void **data,
                                struct vmbus_channel *channel)
 {
        struct netvsc_device *net_dev = hv_get_drvdata(dev);
@@ -430,7 +432,7 @@ int rndis_filter_receive(struct hv_device *dev,
                goto exit;
        }
 
-       rndis_msg = pkt->data;
+       rndis_msg = *data;
 
        if (netif_msg_rx_err(net_dev->nd_ctx))
                dump_rndis_message(dev, rndis_msg);
@@ -438,7 +440,8 @@ int rndis_filter_receive(struct hv_device *dev,
        switch (rndis_msg->ndis_msg_type) {
        case RNDIS_MSG_PACKET:
                /* data msg */
-               rndis_filter_receive_data(rndis_dev, rndis_msg, pkt, channel);
+               rndis_filter_receive_data(rndis_dev, rndis_msg, pkt,
+                                         data, channel);
                break;
 
        case RNDIS_MSG_INIT_C: