From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id E57FA2BC8 for ; Mon, 21 Nov 2016 11:57:10 +0100 (CET) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga104.jf.intel.com with ESMTP; 21 Nov 2016 02:57:09 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,674,1473145200"; d="scan'208";a="7347150" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.57]) ([10.237.220.57]) by orsmga002.jf.intel.com with ESMTP; 21 Nov 2016 02:57:05 -0800 To: Alejandro Lucero , Thomas Monjalon References: <1479485218-11931-1-git-send-email-alejandro.lucero@netronome.com> <1901770.8floaBijqc@xps13> Cc: dev From: Ferruh Yigit Message-ID: <553f5073-1d92-994c-7d3c-b27c4d9c1ce3@intel.com> Date: Mon, 21 Nov 2016 10:57:04 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2] nfp: report link speed using hardware info 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: Mon, 21 Nov 2016 10:57:11 -0000 On 11/18/2016 4:50 PM, Alejandro Lucero wrote: > On Fri, Nov 18, 2016 at 4:29 PM, Thomas Monjalon > wrote: > >> 2016-11-18 16:06, Alejandro Lucero: >>> Previous reported speed was hardcoded. >>> >>> Signed-off-by: Alejandro Lucero >>> --- >>> drivers/net/nfp/nfp_net.c | 28 ++++++++++++++++++++++++++-- >>> drivers/net/nfp/nfp_net_ctrl.h | 13 +++++++++++++ >>> 2 files changed, 39 insertions(+), 2 deletions(-) >> >> You should update the doc in the same patch: >> doc/guides/nics/features/nfp.ini >> It will be the first feature as the file appears to be empty. >> So you will need another patch to fill other existing features. >> > > Yes. I'm just working on updating that file properly. > May I delay this doc change for including it with that other one? > It will be a bit weird to just have one feature there. I think yes, but can you please send the documentation patch before integration deadline of this release? > > >> >> I have an unrelated question: why nfp is disabled in the default build? >> >> > Because NFP PMD can just work if Netronome BSP is installed in the system. Is this BSP open source and freely available? If so, can you update the document with details (how/where to get it) please? > We do not support PF with the PMD, so it requires a Linux PF driver which > comes with the BSP. > > The compilation has no dependencies, but we had our own UIO driver before > (now using igb_uio). So basically, we wanted the people aware of this > dependency and to specifically configure this option. > > I know what you are surely going to ask about DPDK in Linux distributions, > and that this being a bad idea. The fact is, we have people using NFP PMD > as part of a product, so installing that product implies to (automatically) > install the BSP and a specific DPDK version with the NFP PMD enabled. But > yes, maybe we should modify this and to add some sort of BSP check inside > the PMD. Is there a DPDK version <-> BSP version dependency? > > So, thanks for the heads up. I will think about this. >