From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alejandro Lucero Subject: Re: [PATCH] drivers/net: document missing speed capabilities feature Date: Tue, 16 May 2017 11:17:11 +0100 Message-ID: References: <20170510131053.75843-1-ferruh.yigit@intel.com> <17406199.hqjMraK9H2@xps> <3454007.3ZDP6eXIq0@xps> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: Ferruh Yigit , dev , Shepard Siegel , Ed Czeck , John Miller , John McNamara , Harish Patil , Rasesh Mody , Rahul Lakkireddy , Hemant Agrawal , Shreyansh Jain , Wenzhuo Lu , Marcin Wojtas , Michal Krawczyk , Guy Tzalik , Evgeny Schemeilin , Jing Chen , Helin Zhang , Jingjing Wu , Konstantin Ananyev , Adrien Mazarguil , Nelio Laranjeiro , M To: Thomas Monjalon Return-path: Received: from mail-vk0-f53.google.com (mail-vk0-f53.google.com [209.85.213.53]) by dpdk.org (Postfix) with ESMTP id 4D3AE20F for ; Tue, 16 May 2017 12:17:13 +0200 (CEST) Received: by mail-vk0-f53.google.com with SMTP id y190so63064879vkc.1 for ; Tue, 16 May 2017 03:17:13 -0700 (PDT) In-Reply-To: <3454007.3ZDP6eXIq0@xps> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, May 16, 2017 at 11:02 AM, Thomas Monjalon wrote: > 16/05/2017 11:39, Alejandro Lucero: > > Maybe we need another feature for detected link speed report capability. > > > > There is already support for ETH_SPEED_NUM_NONE if the speed is not > defined > > or it can not be specified by the device. Netronome has some firmwares > > without that link speed report capability, although maybe we are not the > > only ones with that problem. > > > > > > What do you think about this? > > Yes, you mean you can advertise a speed capability, > but cannot report the real negotiated speed, right? > So we could add this feature. > Perfect. > There are also virtual drivers which cannot report some real speed. > > Yes, that is right. Virtual drivers can not report speed. We have a problem with a client using pktgen which assumes a device will report the link speed always. By the way, I will send a patch for avoiding this issue with pktgen. Thanks