From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f47.google.com (mail-wm0-f47.google.com [74.125.82.47]) by dpdk.org (Postfix) with ESMTP id 2BF45C166 for ; Tue, 16 Feb 2016 16:26:18 +0100 (CET) Received: by mail-wm0-f47.google.com with SMTP id g62so196588696wme.0 for ; Tue, 16 Feb 2016 07:26:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=jrG8XaK0BJ2rpFrn23KjwZuGidXtVt0MTDUA6PBd00I=; b=tYhQsAs5g2o5iLSBNxYnWZ32XxidSGgTVCYU4lY5n1qtcDbdluepmQdNNrRYa2cqZn aN1KmKkAaqRFZpW7OI4+dLraWw6lZnWO/82SuikAofbV5f8cyO+qNAg13Fc9JWpuhZBg xrsh1X4HhMpt+5nMM5XM8sXDSp5cVgMr0d0bgSu7wtZ6GHOW6BvMxvJ8ktSO2QgQqW2T Zhm4QKuFD6v/PS3hvOLyvIb1tiASsgwEe7eBIyZXupd5wnLBRDyWoLKAIuqi/aWJyCUT +L+SNENJS2NL6tw+Br/9tkoEI3A7uZgzQDC9Mh6d3iBpffsLOCbglvGyLL7dObVlQ6af d7Pg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-type:content-disposition :content-transfer-encoding:in-reply-to:user-agent; bh=jrG8XaK0BJ2rpFrn23KjwZuGidXtVt0MTDUA6PBd00I=; b=jcM4CF2LQ8uCpB7BmK0ZAaLWrMP+XfYtDZ7bG5dRJqeX9Ej6Eo6v4Geek6TmSukZRH RidanrKwGDILWp/RMVg/vRD0xUcZUMqVedL6OxfAytceW97lOlND5WPzb9qmvNLq/R2P XkMjalQR+0rAq10Oz67QXoIkYXY0TIAzVQxhFHisVtpfAklrPBNCIbiYIx6t4I7DBZvV EepXQDrNg/yLh1qA29gV09/BJrBed6b3tVUB2B//iEbxXqAuLpXuN7wJ1bVnJtY0FVJy 0GUnjBb90YULOJcZR+hOp+XCdiszPSlK4NFEtUTNYsb5PyiSAqpm8dsy4rOuQqoCBXme l3lA== X-Gm-Message-State: AG10YORMevuWuGg9+Yre0Mj3W8wHSmj0P75hw1yyT9Ng66MZ8Eiwb/LXqwiPVsz92TJamCVL X-Received: by 10.28.107.140 with SMTP id a12mr21057385wmi.77.1455636377528; Tue, 16 Feb 2016 07:26:17 -0800 (PST) Received: from autoinstall.dev.6wind.com (guy78-3-82-239-227-177.fbx.proxad.net. [82.239.227.177]) by smtp.gmail.com with ESMTPSA id gg7sm30743021wjd.10.2016.02.16.07.26.16 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 16 Feb 2016 07:26:17 -0800 (PST) Date: Tue, 16 Feb 2016 16:25:44 +0100 From: =?iso-8859-1?Q?N=E9lio?= Laranjeiro To: Marc Message-ID: <20160216152544.GA22847@autoinstall.dev.6wind.com> References: <1454028127-10401-1-git-send-email-marcdevel@gmail.com> <1455488259-1000-1-git-send-email-marcdevel@gmail.com> <1455488259-1000-3-git-send-email-marcdevel@gmail.com> <20160215144310.GA19031@scalar.blr.asicdesigners.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Cc: dev@dpdk.org, Nirranjan Kirubaharan , Kumar Sanghvi Subject: Re: [dpdk-dev] [PATCH v8 2/4] ethdev: Fill speed capability bitmaps in the PMDs X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 Feb 2016 15:26:18 -0000 Hi Marc, On Mon, Feb 15, 2016 at 06:14:42PM +0100, Marc wrote: > Rahul, Neilo, Jing D, et al > > On 15 February 2016 at 15:43, Rahul Lakkireddy > wrote: > > > Hi Marc, > > > > On Sunday, February 02/14/16, 2016 at 23:17:37 +0100, Marc Sune wrote: > > > Added speed capabilities to all pmds supporting physical NICs: > > > > > > * e1000 > > > * ixgbe > > > * i40 > > > * bnx2x > > > * cxgbe > > > * mlx4 > > > * mlx5 > > > * nfp > > > * fm10k > > > > > > Signed-off-by: Marc Sune > > > --- > > > drivers/net/bnx2x/bnx2x_ethdev.c | 1 + > > > drivers/net/cxgbe/cxgbe_ethdev.c | 1 + > > > drivers/net/e1000/em_ethdev.c | 6 ++++++ > > > drivers/net/e1000/igb_ethdev.c | 6 ++++++ > > > drivers/net/fm10k/fm10k_ethdev.c | 4 ++++ > > > drivers/net/i40e/i40e_ethdev.c | 9 +++++++++ > > > drivers/net/ixgbe/ixgbe_ethdev.c | 10 ++++++++++ > > > drivers/net/mlx4/mlx4.c | 4 ++++ > > > drivers/net/mlx5/mlx5_ethdev.c | 5 +++++ > > > drivers/net/nfp/nfp_net.c | 2 ++ > > > 10 files changed, 48 insertions(+) > > > > > > > [...] > > > > > diff --git a/drivers/net/cxgbe/cxgbe_ethdev.c > > b/drivers/net/cxgbe/cxgbe_ethdev.c > > > index 97ef152..203e119 100644 > > > --- a/drivers/net/cxgbe/cxgbe_ethdev.c > > > +++ b/drivers/net/cxgbe/cxgbe_ethdev.c > > > @@ -171,6 +171,7 @@ static void cxgbe_dev_info_get(struct rte_eth_dev > > *eth_dev, > > > > > > device_info->rx_desc_lim = cxgbe_desc_lim; > > > device_info->tx_desc_lim = cxgbe_desc_lim; > > > + device_info->speed_capa = ETH_SPEED_CAP_10G | ETH_SPEED_CAP_40G; > > > } > > > > > > > Not all Chelsio NICs support _both_ 10G and 40G speed capabilities on a > > single card. You can query pi->link_cfg.supported to get the supported > > speeds. Check out print_port_info() in cxgbe_main.c to help you fill > > in your speed capabilities for Chelsio NICs. > > > > This patch series has been long delayed, and I've been requested to merge > it for next release if possible. Most of the feedback has been coming late > (not for cxgbe, which is introduced in this new v8, but it did for most of > the rest of drivers). > > My proposal is simply to add in this patch series ALL possible speeds for > that driver. Other patches can be later submitted to adjust speeds > according to specific device model. I agree with this. I was asking in order to understand what your were expecting from this API, for me it is clear. You should just maintain the current situation i.e. rte_eth_link.link_speed. This is already what your patches do. For the link speed capability (aka device_info->speed_capa), you should add a new line "speed capability" in the doc/guides/nics/overview.rst. Those who think it is useful will implement it in their PMD. Regards, -- Nélio Laranjeiro 6WIND