net/enic: fix resource check failures when bonding devices
authorNelson Escobar <neescoba@cisco.com>
Wed, 6 Jul 2016 23:21:59 +0000 (16:21 -0700)
committerBruce Richardson <bruce.richardson@intel.com>
Fri, 8 Jul 2016 20:51:03 +0000 (22:51 +0200)
commitce93d3c36db0fef12063bdfaf5030a23277e1fee
tree21d442b82d9597ffd3645c65a033f7a61ff9c855
parentbeae8ab53ce49968f71e01ee5386e3b0a72de863
net/enic: fix resource check failures when bonding devices

The enic PMD was using the same variables in the enic structure to
track two different things.  Initially rq_count, wq_count, cq_count,
and intr_count were set to the values obtained from the VIC adapters
as the maximum resources allocated on the VIC, then in
enic_set_vnic_res(), they were set to the counts of resources actually
used, discarding the initial values. The checks in enic_set_vnic_res()
were technically incorrect if it is called more than once on a port,
which happens when using bonding, but were harmless in practice as the
checks couldn't fail on the second call.

The enic rx-scatter patch misunderstood the subtleties of
enic_set_vnic_res(), and naively added a multiply by two to the
rq_count check. This resulted in the rq_count check failing when
enic_set_vnic_res() was called a second time, ie when using bonding.

This patch adds new variables to the enic structure to track the
maximum resources the VIC is configured to provide so that the
information isn't later lost and calls to enic_set_vnic_res() do
the expected thing.

Fixes: 856d7ba7ed22 ("net/enic: support scattered Rx")

Signed-off-by: Nelson Escobar <neescoba@cisco.com>
drivers/net/enic/enic.h
drivers/net/enic/enic_ethdev.c
drivers/net/enic/enic_main.c
drivers/net/enic/enic_res.c