X-Git-Url: http://v3vee.org/palacios/gitweb/gitweb.cgi?a=blobdiff_plain;f=palacios%2Fsrc%2Fdevices%2Fvnet_nic.c;h=c1f6dc90311b61acedcbf98327ba0d0ea87b479d;hb=c4fa59ab9c6963f1b7d8ab56718513070f92f768;hp=8b1280de8203605e7cf00ed6cf76fc10b22c0af1;hpb=17f8752ad440dba7d414d0812e3680066dcb8c9e;p=palacios.releases.git

diff --git a/palacios/src/devices/vnet_nic.c b/palacios/src/devices/vnet_nic.c
index 8b1280d..c1f6dc9 100644
--- a/palacios/src/devices/vnet_nic.c
+++ b/palacios/src/devices/vnet_nic.c
@@ -39,14 +39,17 @@ struct vnet_nic_state {
     struct v3_dev_net_ops net_ops;
 };
 
+#if 0
 //used when virtio_nic get a packet from guest and send it to the backend
 static int vnet_send(uint8_t * buf, uint32_t len, void * private_data, struct vm_device *dest_dev){
     struct v3_vnet_pkt * pkt = NULL;
     struct guest_info *core  = (struct guest_info *)private_data; 
 
-    PrintDebug("Virtio VNET-NIC: send pkt size: %d\n", len);
-#ifdef CONFIG_DEBUG_VNET_NIC
-    v3_hexdump(buf, len, NULL, 0);
+#ifdef WCONFIG_DEBUG_VNET_NIC
+    {
+    	PrintDebug("Virtio VNET-NIC: send pkt size: %d\n", len);
+    	v3_hexdump(buf, len, NULL, 0);
+    }
 #endif
 
 #ifdef CONFIG_VNET_PROFILE
@@ -70,36 +73,109 @@ static int vnet_send(uint8_t * buf, uint32_t len, void * private_data, struct vm
 
 
     pkt->size = len;
+    pkt->src_type = LINK_INTERFACE;
+    pkt->src_id = 0;
     memcpy(pkt->data, buf, pkt->size);
 
 #ifdef CONFIG_VNET_PROFILE
     {
     	rdtscll(start);
-    	core->vnet_times.time_copy_from_guest = start - end;
+    	core->vnet_times.memcpy_time = start - end;
+	core->vnet_times.time_copy_from_guest = start - core->vnet_times.virtio_handle_start - core->vnet_times.time_mallocfree;
     }
 #endif
 
     v3_vnet_send_pkt(pkt, (void *)core);
 
 #ifdef CONFIG_VNET_PROFILE
+    rdtscll(start);
+#endif
+
+    V3_Free(pkt);
+
+#ifdef CONFIG_VNET_PROFILE
     {
     	rdtscll(end);
-    	core->vnet_times.vnet_handle_time = end - start;
+    	core->vnet_times.time_mallocfree += end - start;
     }
 #endif
 
-    V3_Free(pkt);
+    return 0;
+}
+
+#endif
+
+#if 1
+//alternative way, no malloc/free
+static int vnet_send(uint8_t * buf, uint32_t len, void * private_data, struct vm_device *dest_dev){
+    struct v3_vnet_pkt pkt;
+    struct guest_info *core  = (struct guest_info *)private_data; 
+
+#ifdef CONFIG_DEBUG_VNET_NIC
+    {
+    	PrintDebug("Virtio VNET-NIC: send pkt size: %d\n", len);
+    	v3_hexdump(buf, len, NULL, 0);
+    }
+#endif
 
 #ifdef CONFIG_VNET_PROFILE
+    uint64_t start, end;
+    rdtscll(start);
+#endif
+
+    pkt.size = len;
+    pkt.src_type = LINK_INTERFACE;
+    pkt.src_id = 0;
+    memcpy(pkt.data, buf, pkt.size);
+
+#ifdef CONFIG_VNET_PROFILE
+    rdtscll(end);
+    core->vnet_times.time_copy_from_guest = end - core->vnet_times.virtio_handle_start;
+    core->vnet_times.memcpy_time = end - start;
+#endif
+
+    v3_vnet_send_pkt(&pkt, (void *)core);
+
+    return 0;
+}
+#endif
+
+#if 0
+//alternative way, no malloc/free, no copy
+//need to change pkt format
+static int vnet_send(uint8_t * buf, uint32_t len, void * private_data, struct vm_device *dest_dev){
+    struct v3_vnet_pkt pkt;
+    struct guest_info *core  = (struct guest_info *)private_data; 
+
+    
+#ifdef CONFIG_DEBUG_VNET_NIC
     {
-    	rdtscll(start);
-    	core->vnet_times.time_mallocfree += start - end;
+    	PrintDebug("Virtio VNET-NIC: send pkt size: %d\n", len);
+    	v3_hexdump(buf, len, NULL, 0);
+    }
+#endif
+
+    pkt.size = len;
+    pkt.data = buf;
+
+#ifdef CONFIG_VNET_PROFILE
+    uint64_t start, end;
+    rdtscll(start);
+#endif
+
+    v3_vnet_send_pkt(&pkt, (void *)core);
+
+#ifdef CONFIG_VNET_PROFILE
+    {
+    	rdtscll(end);
+    	core->vnet_times.vnet_handle_time = end - start;
     }
 #endif
 
     return 0;
 }
 
+#endif
 
 static int virtio_input(struct v3_vm_info *info, struct v3_vnet_pkt * pkt, void * private_data){
     struct vnet_nic_state *vnetnic = (struct vnet_nic_state *)private_data;
@@ -149,13 +225,18 @@ static int str2mac(char *macstr, char mac[6]){
 static int vnet_nic_init(struct v3_vm_info * vm, v3_cfg_tree_t * cfg) {
     struct vnet_nic_state * vnetnic = NULL;
     char * name = v3_cfg_val(cfg, "name");
-    char * macstr;
+    char * macstr = NULL;
     char mac[6];
     int vnet_dev_id;
 
     v3_cfg_tree_t * frontend_cfg = v3_cfg_subtree(cfg, "frontend");
     macstr = v3_cfg_val(frontend_cfg, "mac");
-    str2mac(macstr, mac);
+
+    if (macstr == NULL) {
+	PrintDebug("Vnet-nic: No Mac specified\n");
+    } else {
+    	str2mac(macstr, mac);
+    }
 
     vnetnic = (struct vnet_nic_state *)V3_Malloc(sizeof(struct vnet_nic_state));
     memset(vnetnic, 0, sizeof(struct vnet_nic_state));
@@ -181,22 +262,22 @@ static int vnet_nic_init(struct v3_vm_info * vm, v3_cfg_tree_t * cfg) {
 		   name, v3_cfg_val(frontend_cfg, "tag"));
 
     if ((vnet_dev_id = register_to_vnet(vm, vnetnic, name, vnetnic->mac)) == -1) {
-	return -1;
+	PrintError("Vnet-nic device %s (mac: %s) fails to registered to VNET\n", name, macstr);
     }
 
-    PrintDebug("Vnet-nic device %s (mac: %s, %ld) initialized\n", name, macstr, *((ulong_t *)vnetnic->mac));
+    PrintDebug("Vnet-nic device %s (mac: %s, %ld) registered to VNET\n", name, macstr, *((ulong_t *)vnetnic->mac));
 
 
 //for temporary hack
 #if 1	
     {
-	 //uchar_t tapmac[6] = {0x00,0x02,0x55,0x67,0x42,0x39}; //for Intel-VT test HW
-    	uchar_t tapmac[6] = {0x6e,0xa8,0x75,0xf4,0x82,0x95};
+	uchar_t tapmac[6] = {0x00,0x02,0x55,0x67,0x42,0x39}; //for Intel-VT test HW
+    	//uchar_t tapmac[6] = {0x6e,0xa8,0x75,0xf4,0x82,0x95};
     	uchar_t dstmac[6] = {0xff,0xff,0xff,0xff,0xff,0xff};
     	uchar_t zeromac[6] = {0,0,0,0,0,0};
 
 	struct v3_vnet_route route;
-       route.dst_id = vnet_dev_id;
+        route.dst_id = vnet_dev_id;
 	route.dst_type = LINK_INTERFACE;
 	route.src_id = -1;
 	route.src_type = LINK_ANY;