[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1339600460.17240.69.camel@lb-tlvb-meravs.il.broadcom.com>
Date: Wed, 13 Jun 2012 18:14:20 +0300
From: "Merav Sicron" <meravs@...adcom.com>
To: "Eric Dumazet" <eric.dumazet@...il.com>
cc: eilong@...adcom.com, davem@...emloft.net, netdev@...r.kernel.org
Subject: Re: [net-next patch 8/12] bnx2x: Allow up to 63 RSS queues
default 8 queues
Hi Eric,
On Wed, 2012-06-13 at 11:52 +0200, Eric Dumazet wrote:
> On Wed, 2012-06-13 at 15:44 +0300, Merav Sicron wrote:
> > This patch removed the limitation in the code for 16 RSS queues. The default
> > (without other instruction from the user) number of queues is determined
> > according to the number of MSI-X vectors supported for that function, the number
> > of CPUs in the system, and a maximum of 8 queues.
>
> Thats a very confusing changelog
>
> You meant : " a minimum of 8 queues" ?
>
No, I meant maximum...for example in a system with 16 CPUs we will
allocate 8 RSS queues. We saw that in most scenarios we tested there was
no need for more than 8 queues to get the maximal throughput, and by
limiting to 8 by default we reduce the allocated memory. We do provide
ethtool -L support so that the user could request more RSS queues if
desired.
> You should give more explanations, because its a sensible area for
> performances.
>
Thanks,
Merav
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists