From: Nelson Escobar Date: Tue, 28 Jun 2016 18:49:11 +0000 (-0700) Subject: net/enic: fix Rx scatter with multiple queues X-Git-Tag: spdx-start~6341 X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=16dba07133bd354adc9e99223595dffbbe8f1649;p=dpdk.git net/enic: fix Rx scatter with multiple queues The Rx scatter patch failed to make a few changes and resulted in problems when using multiple receive queues (RQs) in DPDK (ie RSS) since the wrong adapter resources were being used. - get and use the correct completion queue index associated with a receive queue. - set the correct receive queue index when using RSS Fixes: 856d7ba7ed22 ("net/enic: support scattered Rx") Signed-off-by: Nelson Escobar Reviewed-by: John Daley --- diff --git a/drivers/net/enic/enic.h b/drivers/net/enic/enic.h index 060fe61854..53fed0b8c3 100644 --- a/drivers/net/enic/enic.h +++ b/drivers/net/enic/enic.h @@ -175,7 +175,11 @@ static inline unsigned int enic_vnic_rq_count(struct enic *enic) static inline unsigned int enic_cq_rq(__rte_unused struct enic *enic, unsigned int rq) { - return rq; + /* Scatter rx uses two receive queues together with one + * completion queue, so the completion queue number is no + * longer the same as the rq number. + */ + return rq / 2; } static inline unsigned int enic_cq_wq(struct enic *enic, unsigned int wq) diff --git a/drivers/net/enic/enic_main.c b/drivers/net/enic/enic_main.c index 4797fce8b9..dc831b48fb 100644 --- a/drivers/net/enic/enic_main.c +++ b/drivers/net/enic/enic_main.c @@ -238,19 +238,20 @@ void enic_init_vnic_resources(struct enic *enic) struct vnic_rq *data_rq; for (index = 0; index < enic->rq_count; index++) { + cq_idx = enic_cq_rq(enic, enic_sop_rq(index)); + vnic_rq_init(&enic->rq[enic_sop_rq(index)], - enic_cq_rq(enic, index), + cq_idx, error_interrupt_enable, error_interrupt_offset); data_rq = &enic->rq[enic_data_rq(index)]; if (data_rq->in_use) vnic_rq_init(data_rq, - enic_cq_rq(enic, index), + cq_idx, error_interrupt_enable, error_interrupt_offset); - cq_idx = enic_cq_rq(enic, index); vnic_cq_init(&enic->cq[cq_idx], 0 /* flow_control_enable */, 1 /* color_enable */, @@ -899,7 +900,8 @@ static int enic_set_rsscpu(struct enic *enic, u8 rss_hash_bits) return -ENOMEM; for (i = 0; i < (1 << rss_hash_bits); i++) - (*rss_cpu_buf_va).cpu[i/4].b[i%4] = i % enic->rq_count; + (*rss_cpu_buf_va).cpu[i / 4].b[i % 4] = + enic_sop_rq(i % enic->rq_count); err = enic_set_rss_cpu(enic, rss_cpu_buf_pa,