X-Git-Url: http://v3vee.org/palacios/gitweb/gitweb.cgi?a=blobdiff_plain;f=palacios%2Fsrc%2Fvnet%2Fvnet_core.c;h=c2ebaf0b2a0694896233ea4bbad4c73541491108;hb=b7093fd3602ef2c796a1f8a0daded9d6aad0b756;hp=e7b37f9d86efc337e4343633b2655e929d1b8ace;hpb=0975ec64a5a81c1c51bc49791b40e785d1679d28;p=palacios.releases.git diff --git a/palacios/src/vnet/vnet_core.c b/palacios/src/vnet/vnet_core.c index e7b37f9..c2ebaf0 100644 --- a/palacios/src/vnet/vnet_core.c +++ b/palacios/src/vnet/vnet_core.c @@ -27,10 +27,12 @@ #include #ifndef V3_CONFIG_DEBUG_VNET -#undef Vnet_Debug -#define Vnet_Debug(fmt, args...) +#undef PrintDebug +#define PrintDebug(fmt, args...) #endif +#define VNET_YIELD_USEC 1000 + int net_debug = 0; struct eth_hdr { @@ -133,15 +135,15 @@ static void print_route(struct v3_vnet_route * route){ char str[50]; mac2str(route->src_mac, str); - Vnet_Debug("Src Mac (%s), src_qual (%d)\n", + PrintDebug("Src Mac (%s), src_qual (%d)\n", str, route->src_mac_qual); mac2str(route->dst_mac, str); - Vnet_Debug("Dst Mac (%s), dst_qual (%d)\n", + PrintDebug("Dst Mac (%s), dst_qual (%d)\n", str, route->dst_mac_qual); - Vnet_Debug("Src dev id (%d), src type (%d)", + PrintDebug("Src dev id (%d), src type (%d)", route->src_id, route->src_type); - Vnet_Debug("Dst dev id (%d), dst type (%d)\n", + PrintDebug("Dst dev id (%d), dst type (%d)\n", route->dst_id, route->dst_type); } @@ -149,13 +151,13 @@ static void print_route(struct v3_vnet_route * route){ static void dump_routes(){ struct vnet_route_info *route; - Vnet_Debug("\n========Dump routes starts ============\n"); + PrintDebug("\n========Dump routes starts ============\n"); list_for_each_entry(route, &(vnet_state.routes), node) { - Vnet_Debug("\nroute %d:\n", route->idx); + PrintDebug("\nroute %d:\n", route->idx); print_route(&(route->route_def)); if (route->route_def.dst_type == LINK_INTERFACE) { - Vnet_Debug("dst_dev (%p), dst_dev_id (%d), dst_dev_ops(%p), dst_dev_data (%p)\n", + PrintDebug("dst_dev (%p), dst_dev_id (%d), dst_dev_ops(%p), dst_dev_data (%p)\n", route->dst_dev, route->dst_dev->dev_id, (void *)&(route->dst_dev->dev_ops), @@ -163,7 +165,7 @@ static void dump_routes(){ } } - Vnet_Debug("\n========Dump routes end ============\n"); + PrintDebug("\n========Dump routes end ============\n"); } #endif @@ -252,10 +254,16 @@ int v3_vnet_add_route(struct v3_vnet_route route) { unsigned long flags; new_route = (struct vnet_route_info *)Vnet_Malloc(sizeof(struct vnet_route_info)); + + if (!new_route) { + PrintError("Cannot allocate new route\n"); + return -1; + } + memset(new_route, 0, sizeof(struct vnet_route_info)); #ifdef V3_CONFIG_DEBUG_VNET - Vnet_Debug("VNET/P Core: add_route_entry:\n"); + PrintDebug("VNET/P Core: add_route_entry:\n"); print_route(&route); #endif @@ -361,7 +369,7 @@ static struct route_list * match_route(const struct v3_vnet_pkt * pkt) { mac2str(hdr->src_mac, src_str); mac2str(hdr->dst_mac, dst_str); - Vnet_Debug("VNET/P Core: match_route. pkt: SRC(%s), DEST(%s)\n", src_str, dst_str); + PrintDebug("VNET/P Core: match_route. pkt: SRC(%s), DEST(%s)\n", src_str, dst_str); } #endif @@ -448,15 +456,21 @@ static struct route_list * match_route(const struct v3_vnet_pkt * pkt) { } } - Vnet_Debug("VNET/P Core: match_route: Matches=%d\n", num_matches); + PrintDebug("VNET/P Core: match_route: Matches=%d\n", num_matches); - if (num_matches == 0) { + if (num_matches <= 0) { return NULL; } matches = (struct route_list *)Vnet_Malloc(sizeof(struct route_list) + (sizeof(struct vnet_route_info *) * num_matches)); + + if (!matches) { + PrintError("VNET/P Core: Unable to allocate matches\n"); + return NULL; + } + matches->num_routes = num_matches; { @@ -476,9 +490,11 @@ int v3_vnet_send_pkt(struct v3_vnet_pkt * pkt, void * private_data) { int i; int cpu = V3_Get_CPU(); + Vnet_Print(2, "VNET/P Core: cpu %d: pkt (size %d, src_id:%d, src_type: %d, dst_id: %d, dst_type: %d)\n", cpu, pkt->size, pkt->src_id, pkt->src_type, pkt->dst_id, pkt->dst_type); + if(net_debug >= 4){ v3_hexdump(pkt->data, pkt->size, NULL, 0); } @@ -489,15 +505,16 @@ int v3_vnet_send_pkt(struct v3_vnet_pkt * pkt, void * private_data) { vnet_state.stats.rx_pkts++; look_into_cache(pkt, &matched_routes); + if (matched_routes == NULL) { - Vnet_Debug("VNET/P Core: send pkt Looking into routing table\n"); + PrintDebug("VNET/P Core: sending pkt - matching route\n"); matched_routes = match_route(pkt); if (matched_routes) { add_route_to_cache(pkt, matched_routes); } else { - Vnet_Debug("VNET/P Core: Could not find route for packet... discards packet\n"); + PrintDebug("VNET/P Core: Could not find route for packet... discarding packet\n"); vnet_unlock_irqrestore(vnet_state.lock, flags); return 0; /* do we return -1 here?*/ } @@ -505,7 +522,7 @@ int v3_vnet_send_pkt(struct v3_vnet_pkt * pkt, void * private_data) { vnet_unlock_irqrestore(vnet_state.lock, flags); - Vnet_Debug("VNET/P Core: send pkt route matches %d\n", matched_routes->num_routes); + PrintDebug("VNET/P Core: send pkt route matches %d\n", matched_routes->num_routes); for (i = 0; i < matched_routes->num_routes; i++) { struct vnet_route_info * route = matched_routes->routes[i]; @@ -556,7 +573,7 @@ int v3_vnet_add_dev(struct v3_vm_info * vm, uint8_t * mac, new_dev = (struct vnet_dev *)Vnet_Malloc(sizeof(struct vnet_dev)); if (new_dev == NULL) { - Vnet_Print(0, "Malloc fails\n"); + Vnet_Print(0, "VNET/P Core: Unable to allocate a new device\n"); return -1; } @@ -566,7 +583,7 @@ int v3_vnet_add_dev(struct v3_vm_info * vm, uint8_t * mac, new_dev->private_data = priv_data; new_dev->vm = vm; new_dev->dev_id = 0; - new_dev->quote = quotequote = quotepoll = poll_state; flags = vnet_lock_irqsave(vnet_state.lock); @@ -580,7 +597,7 @@ int v3_vnet_add_dev(struct v3_vm_info * vm, uint8_t * mac, v3_enqueue(vnet_state.poll_devs, (addr_t)new_dev); } } else { - PrintError("VNET/P: Device with the same MAC is already there\n"); + PrintError("VNET/P: Device with the same MAC has already been added\n"); } vnet_unlock_irqrestore(vnet_state.lock, flags); @@ -591,7 +608,7 @@ int v3_vnet_add_dev(struct v3_vm_info * vm, uint8_t * mac, return -1; } - Vnet_Debug("VNET/P Core: Add Device: dev_id %d\n", new_dev->dev_id); + PrintDebug("VNET/P Core: Add Device: dev_id %d\n", new_dev->dev_id); return new_dev->dev_id; } @@ -614,7 +631,7 @@ int v3_vnet_del_dev(int dev_id){ Vnet_Free(dev); - Vnet_Debug("VNET/P Core: Remove Device: dev_id %d\n", dev_id); + PrintDebug("VNET/P Core: Removed Device: dev_id %d\n", dev_id); return 0; } @@ -671,7 +688,7 @@ int v3_vnet_add_bridge(struct v3_vm_info * vm, tmp_bridge = (struct vnet_brg_dev *)Vnet_Malloc(sizeof(struct vnet_brg_dev)); if (tmp_bridge == NULL) { - PrintError("Malloc Fails\n"); + PrintError("VNET/P Core: Unable to allocate new bridge\n"); vnet_state.bridge = NULL; return -1; } @@ -699,7 +716,7 @@ void v3_vnet_del_bridge(uint8_t type) { if (vnet_state.bridge != NULL && vnet_state.bridge->type == type) { tmp_bridge = vnet_state.bridge; - vnet_state.bridge = NULL; + vnet_state.bridge = NULL; } vnet_unlock_irqrestore(vnet_state.lock, flags); @@ -716,27 +733,67 @@ void v3_vnet_del_bridge(uint8_t type) { */ static int vnet_tx_flush(void * args){ struct vnet_dev * dev = NULL; - int ret; + int more; + int rc; Vnet_Print(0, "VNET/P Polling Thread Starting ....\n"); - while(!vnet_thread_should_stop()){ - dev = (struct vnet_dev *)v3_dequeue(vnet_state.poll_devs); - if(dev != NULL){ - if(dev->poll && dev->dev_ops.poll != NULL){ - ret = dev->dev_ops.poll(dev->vm, dev->quote, dev->private_data); - - if (ret < 0){ - Vnet_Print(0, "VNET/P: poll from device %p error!\n", dev); + // since there are multiple instances of this thread, and only + // one queue of pollable devices, our model here will be to synchronize + // on that queue, removing devices as we go, and keeping them + // then putting them back on the queue when we are done + // in this way, multiple instances of this function will never + // be polling the same device at the same time + + struct v3_queue * tq = v3_create_queue(); + + if (!tq) { + PrintError("VNET/P polling thread cannot allocate queue\n"); + return -1; + } + + + while (!vnet_thread_should_stop()) { + + more=0; // will indicate if any device has more work for us to do + + while ((dev = (struct vnet_dev *)v3_dequeue(vnet_state.poll_devs))) { + // we are handling this device + v3_enqueue(tq,(addr_t)dev); + + if (dev->poll && dev->dev_ops.poll) { + // The device's poll function MUST NOT BLOCK + rc = dev->dev_ops.poll(dev->vm, dev->quote, dev->private_data); + + if (rc<0) { + Vnet_Print(0, "VNET/P: poll from device %p error (ignoring) !\n", dev); + } else { + more |= rc; } } + } + + while ((dev = (struct vnet_dev *)v3_dequeue(tq))) { + // now someone else can handle it v3_enqueue(vnet_state.poll_devs, (addr_t)dev); - }else { /* no device needs to be polled */ - /* sleep here? */ - Vnet_Yield(); } + + // Yield regardless of whether we handled any devices - need + // to allow other threads to run + if (more) { + // we have more to do, so we want to get back asap + V3_Yield(); + } else { + // put ourselves briefly to sleep if we we don't have more + V3_Yield_Timed(VNET_YIELD_USEC); + } + } + Vnet_Free(tq); + + Vnet_Print(0, "VNET/P Polling Thread Done.\n"); + return 0; } @@ -763,7 +820,7 @@ int v3_init_vnet() { vnet_state.pkt_flush_thread = vnet_start_thread(vnet_tx_flush, NULL, "vnetd-1"); - Vnet_Debug("VNET/P is initiated\n"); + PrintDebug("VNET/P is initiated\n"); return 0; } @@ -771,13 +828,37 @@ int v3_init_vnet() { void v3_deinit_vnet(){ - vnet_lock_deinit(&(vnet_state.lock)); + PrintDebug("Stopping flush thread\n"); + // This will pause until the flush thread is gone + vnet_thread_stop(vnet_state.pkt_flush_thread); + // At this point there should be no lock-holder + + Vnet_Free(vnet_state.poll_devs); + - deinit_devices_list(); + PrintDebug("Deiniting Device List\n"); + // close any devices we have open + deinit_devices_list(); + + PrintDebug("Deiniting Route List\n"); + // remove any routes we have deinit_routes_list(); + PrintDebug("Freeing hash table\n"); + // remove the hash table vnet_free_htable(vnet_state.route_cache, 1, 1); - Vnet_Free(vnet_state.bridge); + + + PrintDebug("Removing Bridge\n"); + // remove bridge if it was added + if (vnet_state.bridge) { + Vnet_Free(vnet_state.bridge); + } + + PrintDebug("Deleting lock\n"); + // eliminate the lock + vnet_lock_deinit(&(vnet_state.lock)); + }