X-Git-Url: http://v3vee.org/palacios/gitweb/gitweb.cgi?a=blobdiff_plain;f=palacios%2Fsrc%2Fdevices%2Flnx_virtio_nic.c;h=7f217aed2cb3ba7860313306fe22a97693df6039;hb=7618b2492af387a660879432bcf2d741d546437e;hp=8032b13be23266ded91c5e406b562d5d64f39860;hpb=8d81fe78cacf4b754857fb0c7342a609bd2ef69d;p=palacios.git diff --git a/palacios/src/devices/lnx_virtio_nic.c b/palacios/src/devices/lnx_virtio_nic.c index 8032b13..7f217ae 100644 --- a/palacios/src/devices/lnx_virtio_nic.c +++ b/palacios/src/devices/lnx_virtio_nic.c @@ -199,7 +199,7 @@ static int tx_one_pkt(struct guest_info * core, PrintDebug("Could not translate buffer address\n"); return -1; } - + V3_Net_Print(2, "Virtio-NIC: virtio_tx: size: %d\n", len); if(net_debug >= 4){ v3_hexdump(buf, len, NULL, 0); @@ -209,10 +209,10 @@ static int tx_one_pkt(struct guest_info * core, virtio->stats.tx_dropped ++; return -1; } - + virtio->stats.tx_pkts ++; virtio->stats.tx_bytes += len; - + return 0; } @@ -226,7 +226,7 @@ static inline int copy_data_to_desc(struct guest_info * core, uint_t dst_offset){ uint32_t len; uint8_t * desc_buf = NULL; - + if (v3_gpa_to_hva(core, desc->addr_gpa, (addr_t *)&(desc_buf)) == -1) { PrintDebug("Could not translate buffer address\n"); return -1; @@ -266,79 +266,78 @@ static int handle_pkt_tx(struct guest_info * core, struct virtio_net_state * virtio_state, int quote) { - struct virtio_queue *q = &(virtio_state->tx_vq); + struct virtio_queue * q; int txed = 0, left = 0; unsigned long flags; + q = &(virtio_state->tx_vq); if (!q->ring_avail_addr) { return -1; } - flags = v3_lock_irqsave(virtio_state->tx_lock); - while (q->cur_avail_idx != q->avail->index) { - struct virtio_net_hdr_mrg_rxbuf * hdr = NULL; + while (1) { struct vring_desc * hdr_desc = NULL; addr_t hdr_addr = 0; - uint16_t desc_idx = q->avail->ring[q->cur_avail_idx % q->queue_size]; - int desc_cnt = get_desc_count(q, desc_idx); + uint16_t desc_idx, tmp_idx; + int desc_cnt; + + flags = v3_lock_irqsave(virtio_state->tx_lock); + + if(q->cur_avail_idx == q->avail->index || + (quote > 0 && txed >= quote)) { + left = (q->cur_avail_idx != q->avail->index); + break; + } + + desc_idx = q->avail->ring[q->cur_avail_idx % q->queue_size]; + tmp_idx = q->cur_avail_idx ++; + + v3_unlock_irqrestore(virtio_state->tx_lock, flags); + desc_cnt = get_desc_count(q, desc_idx); if(desc_cnt != 2){ PrintError("VNIC: merged rx buffer not supported, desc_cnt %d\n", desc_cnt); - goto exit_error; } hdr_desc = &(q->desc[desc_idx]); - if (v3_gpa_to_hva(core, hdr_desc->addr_gpa, &(hdr_addr)) == -1) { - PrintError("Could not translate block header address\n"); - goto exit_error; - } - - hdr = (struct virtio_net_hdr_mrg_rxbuf *)hdr_addr; - desc_idx = hdr_desc->next; + if (v3_gpa_to_hva(core, hdr_desc->addr_gpa, &(hdr_addr)) != -1) { + struct virtio_net_hdr_mrg_rxbuf * hdr; + struct vring_desc * buf_desc; - V3_Net_Print(2, "Virtio NIC: TX hdr count : %d\n", hdr->num_buffers); + hdr = (struct virtio_net_hdr_mrg_rxbuf *)hdr_addr; + desc_idx = hdr_desc->next; - /* here we assumed that one ethernet pkt is not splitted into multiple buffer */ - struct vring_desc * buf_desc = &(q->desc[desc_idx]); - if (tx_one_pkt(core, virtio_state, buf_desc) == -1) { - PrintError("Virtio NIC: Error handling nic operation\n"); - goto exit_error; - } - if(buf_desc->next & VIRTIO_NEXT_FLAG){ - PrintError("Virtio NIC: TX more buffer need to read\n"); + /* here we assumed that one ethernet pkt is not splitted into multiple buffer */ + buf_desc = &(q->desc[desc_idx]); + if (tx_one_pkt(core, virtio_state, buf_desc) == -1) { + PrintError("Virtio NIC: Fails to send packet\n"); + } + } else { + PrintError("Could not translate block header address\n"); } - - q->used->ring[q->used->index % q->queue_size].id = q->avail->ring[q->cur_avail_idx % q->queue_size]; - q->used->ring[q->used->index % q->queue_size].length = buf_desc->length; /* What do we set this to???? */ - q->used->index ++; + + flags = v3_lock_irqsave(virtio_state->tx_lock); - q->cur_avail_idx ++; + q->used->ring[q->used->index % q->queue_size].id = + q->avail->ring[tmp_idx % q->queue_size]; - if(++txed >= quote && quote > 0){ - left = (q->cur_avail_idx != q->avail->index); - break; - } - } - - v3_unlock_irqrestore(virtio_state->tx_lock, flags); + //q->used->ring[q->used->index % q->queue_size].length = buf_desc->length; /* What do we set this to???? */ + + q->used->index ++; + + v3_unlock_irqrestore(virtio_state->tx_lock, flags); + txed ++; + } + if (txed && !(q->avail->flags & VIRTIO_NO_IRQ_FLAG)) { v3_pci_raise_irq(virtio_state->virtio_dev->pci_bus, - 0, virtio_state->pci_dev); + virtio_state->pci_dev, 0); virtio_state->virtio_cfg.pci_isr = 0x1; virtio_state->stats.rx_interrupts ++; } - if(txed > 0) { - V3_Net_Print(2, "Virtio Handle TX: txed pkts: %d, left %d\n", txed, left); - } - return left; - -exit_error: - - v3_unlock_irqrestore(virtio_state->tx_lock, flags); - return -1; } @@ -555,7 +554,7 @@ static int virtio_io_read(struct guest_info *core, *(uint8_t *)dst = virtio->virtio_cfg.pci_isr; virtio->virtio_cfg.pci_isr = 0; v3_pci_lower_irq(virtio->virtio_dev->pci_bus, - 0, virtio->pci_dev); + virtio->pci_dev, 0); break; case VIRTIO_NET_CONFIG ... VIRTIO_NET_CONFIG + ETH_ALEN: @@ -710,7 +709,7 @@ static int virtio_rx(uint8_t * buf, uint32_t size, void * private_data) { virtio->pci_dev->config_header.intr_line); virtio->virtio_cfg.pci_isr = 0x1; - v3_pci_raise_irq(virtio->virtio_dev->pci_bus, 0, virtio->pci_dev); + v3_pci_raise_irq(virtio->virtio_dev->pci_bus, virtio->pci_dev, 0); virtio->stats.rx_interrupts ++; } @@ -805,7 +804,7 @@ static int register_dev(struct virtio_dev_state * virtio, pci_dev = v3_pci_register_device(virtio->pci_bus, PCI_STD_DEVICE, 0, PCI_AUTO_DEV_NUM, 0, "LNX_VIRTIO_NIC", bars, - NULL, NULL, NULL, net_state); + NULL, NULL, NULL, NULL, net_state); if (!pci_dev) { PrintError("Virtio NIC: Could not register PCI Device\n"); @@ -840,6 +839,7 @@ static int register_dev(struct virtio_dev_state * virtio, return 0; } +#if 0 #define RATE_UPPER_THRESHOLD 10 /* 10000 pkts per second, around 100Mbits */ #define RATE_LOWER_THRESHOLD 1 #define PROFILE_PERIOD 10000 /*us*/ @@ -907,7 +907,7 @@ static void virtio_nic_timer(struct guest_info * core, static struct v3_timer_ops timer_ops = { .update_timer = virtio_nic_timer, }; - +#endif static int connect_fn(struct v3_vm_info * info, void * frontend_data, @@ -928,8 +928,7 @@ static int connect_fn(struct v3_vm_info * info, net_state->tx_notify = 1; net_state->rx_notify = 1; - net_state->timer = v3_add_timer(&(info->cores[0]), - &timer_ops,net_state); + //net_state->timer = v3_add_timer(&(info->cores[0]), &timer_ops,net_state); ops->recv = virtio_rx; ops->poll = virtio_poll;