From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olivier Matz Subject: Re: [PATCH] mlx4: use dummy rxqs when a non-pow2 number is requested Date: Tue, 22 Mar 2016 10:48:51 +0100 Message-ID: <56F11503.8060905@6wind.com> References: <1458576484-28211-1-git-send-email-olivier.matz@6wind.com> <024DB49E-56E0-4550-A981-00CE20E6A23D@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: "dev@dpdk.org" , "adrien.mazarguil@6wind.com" To: "Wiles, Keith" Return-path: Received: from mail-wm0-f52.google.com (mail-wm0-f52.google.com [74.125.82.52]) by dpdk.org (Postfix) with ESMTP id 855352904 for ; Tue, 22 Mar 2016 10:48:54 +0100 (CET) Received: by mail-wm0-f52.google.com with SMTP id l68so143848597wml.0 for ; Tue, 22 Mar 2016 02:48:54 -0700 (PDT) In-Reply-To: <024DB49E-56E0-4550-A981-00CE20E6A23D@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Keith, On 03/21/2016 06:38 PM, Wiles, Keith wrote: >> On Mar 21, 2016, at 11:10 AM, Olivier Matz wrote: >> >> When using RSS, the number of rxqs has to be a power of two. >> This is a problem because there is no API is dpdk that makes >> the application aware of that. >> >> A good compromise is to allow the application to request a >> number of rxqs that is not a power of 2, but having inactive >> queues that will never receive packets. In this configuration, >> a warning will be issued to users to let them know that >> this is not an optimal configuration. > > Not sure I like this solution. I think an error should be returned with a log message instead. What if the next driver needs power of three or must be odd or even number. > > The bigger problem is the application is no longer portable for any given nic configuration. > > We need a method for the application to query the system for these types of information. But as we do not have that API we need to just error the request off. The initial problem is that the driver says "I support a maximum of X queues" and if the application configures a lower number, it gets an error. There is no API in DPDK to tell that only specific number of queues are supported. Adding an API is a solution, but in this case it's probably overkill. With this patch, the driver can present the proper number of queues to the application, knowing that the spreading of the packets won't be ideal (some queues won't receive packets), but it will work. A step further in this direction would be to configure more queues than asked in hardware to do a better spreading, almost similar to what is done with RETA tables in mlx5. But this is more complicated to do, especially if we want it for 16.04. Hope this is clearer with the explanation. Regards, Olivier