From 8d336ba9cbcb4832b992201497afe07afcd4f2e1 Mon Sep 17 00:00:00 2001 From: John Daley Date: Fri, 1 Jul 2016 12:32:45 -0700 Subject: [PATCH 19/25] enic: fix Rx queue initialization after restarting a device If you stop then start a port that had already received some packets, the NIC could fetch discriptors from the wrong location. This could effectivly reduce the size of the Rx queue by a random amount and cause packet drop or reduced performance. Reset the NIC fetch index to 0 when allocating and posting mbuf addresses to the NIC. Fixes: 947d860c821f ("enic: improve Rx performance") Signed-off-by: John Daley Reviewed-by: Nelson Escobar --- drivers/net/enic/enic_main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/enic/enic_main.c b/drivers/net/enic/enic_main.c index be17707..68532d3 100644 --- a/drivers/net/enic/enic_main.c +++ b/drivers/net/enic/enic_main.c @@ -346,6 +346,7 @@ enic_alloc_rx_queue_mbufs(struct enic *enic, struct vnic_rq *rq) dev_debug(enic, "port=%u, qidx=%u, Write %u posted idx, %u sw held\n", enic->port_id, rq->index, rq->posted_index, rq->rx_nb_hold); iowrite32(rq->posted_index, &rq->ctrl->posted_index); + iowrite32(0, &rq->ctrl->fetch_index); rte_rmb(); // printf("posted %d buffers to %s rq\n", rq->ring.desc_count, -- 2.7.0