On Mon, 2019-08-12 at 15:24 -0700, Jakub Kicinski wrote: > On Mon, 12 Aug 2019 15:07:09 +0000, Nguyen, Anthony L wrote: > > On Fri, 2019-08-09 at 14:15 -0700, Jakub Kicinski wrote: > > > On Fri, 9 Aug 2019 11:31:25 -0700, Jeff Kirsher wrote: > > > > From: Henry Tieman <henry.w.tie...@intel.com> > > > > > > > > Add code to query and set the number of queues on the primary > > > > VSI for a PF. This is accessed from the 'ethtool -l' and > > > > 'ethtool > > > > -L' > > > > commands, respectively. > > > > > > > > Signed-off-by: Henry Tieman <henry.w.tie...@intel.com> > > > > Signed-off-by: Tony Nguyen <anthony.l.ngu...@intel.com> > > > > Tested-by: Andrew Bowers <andrewx.bow...@intel.com> > > > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirs...@intel.com> > > > > > > If you're using the same IRQ vector for RX and TX queue the > > > channel > > > counts as combined. Looks like you are counting RX and TX > > > separately > > > here. That's incorrect. > > > > Hi Jakub, > > > > The ice driver can support asymmetric queues. We report these > > seperately, as opposed to combined, so that the user can specify a > > different number of Rx and Tx queues. > > If you have 20 IRQ vectors, 10 TX queues and 20 RX queues, the first > 10 > RX queues share a IRQ vector with TX queues the ethool API counts > them > as 10 combined and 10 rx-only. > > 10 tx-only and 20 rx-only would require 30 IRQ vectors.
Thanks for the feedback Jakub. We are looking into this. -Tony
smime.p7s
Description: S/MIME cryptographic signature
