From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f68.google.com (mail-wm0-f68.google.com [74.125.82.68]) by dpdk.org (Postfix) with ESMTP id 1A5601B73F for ; Mon, 9 Apr 2018 09:20:18 +0200 (CEST) Received: by mail-wm0-f68.google.com with SMTP id o23so16287035wmf.0 for ; Mon, 09 Apr 2018 00:20:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=netronome-com.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=oA0pc4nFnOZ8FEERy4qPfQnhHjUPeaKsWqMckgnUUQs=; b=ra1OmttykufTzhuJvllyTfjNrZLz71PKGV3nTrk2bgzWPf1g0Y3QxJEVyT7MV4oRXy BKAQikbUgBTwK2mzmLr0bYnwC05RT1XSH3TaQGDiTgoXf1L7QjYqp2fDa2xOhSHAkFXA JGJoRpp+y4mKMqdhVVJ7rQKNPlBVocod/qkKd+5cnmx32QdlP99vNAV/yzpBxvgnlD/b JM4CFEb54pf7rsfLcSt52Ha0Q02xi22n88axzzItq7T1PKEQmJ7rkEFd5EgK7RijxV6R AbzZwNPJ8dMZcA0mtoOX1LHgQJzyDCWfpftnkC3HJT4Jzx0bWZ7dEAGDlKUminfE9ONI 77dQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=oA0pc4nFnOZ8FEERy4qPfQnhHjUPeaKsWqMckgnUUQs=; b=qolZ83hEH3GvJPI5ZEKe1toZTj/6G2qFpmkozR95UW5XjMRRpNPfOt600UyrPJCeIX CUGIQELGBidQUZ4V0islb17ZhGWIK6aR4g/44Jhhv8Z6JrW/MsiFG3ezH/OYugSuSe5i y6okZbQlk3EHQ0hE/ApDj5Q6kYYMcMgQhIuEwPWodTgjAAEDKQmqQyf3L4WenOjm5RE1 66WoOs0lQXrrO68l+kerczxTlqV8q68+ihtHaFxPQUmyXmhHuH7tkCSGrlWTs/MsvLl4 s6cviC3wCPmT91pltK4++8v10Qn23fJeoZ1oaE/F6G6AhhHnutLETRkqrSm0MQvtQPPD d6NQ== X-Gm-Message-State: ALQs6tB+IUZcLZTLZztjt+wQcOYChserAhotP2SXoiu6OxZzLv6E1rw3 vdoF/uA9SSs4mtYlF8DwyN4x/Xqol/35UG0zWfAeRg== X-Google-Smtp-Source: AIpwx49n4jyZ5AJmRhSCgg4Bl8muj/edsxWhI4m7YEBFp5YqSzSe7IK/Y6xNzpknjoikna+mbE8vbO1H2rVW6YM73VA= X-Received: by 10.80.234.132 with SMTP id d4mr20074154edo.97.1523258417878; Mon, 09 Apr 2018 00:20:17 -0700 (PDT) MIME-Version: 1.0 Received: by 10.80.203.69 with HTTP; Mon, 9 Apr 2018 00:20:17 -0700 (PDT) In-Reply-To: <30ee09ef-57d7-dea1-d034-81f8f0a2a8f4@intel.com> References: <1522939367-5392-1-git-send-email-alejandro.lucero@netronome.com> <802aee06-1bb5-594d-4834-bb7ef7b61813@intel.com> <30ee09ef-57d7-dea1-d034-81f8f0a2a8f4@intel.com> From: Alejandro Lucero Date: Mon, 9 Apr 2018 09:20:17 +0200 Message-ID: To: Ferruh Yigit Cc: dev Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH v2 0/4] NFP PF support based on new CPP interface X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 09 Apr 2018 07:20:18 -0000 On Fri, Apr 6, 2018 at 4:47 PM, Ferruh Yigit wrote: > On 4/6/2018 3:44 PM, Ferruh Yigit wrote: > > On 4/5/2018 3:42 PM, Alejandro Lucero wrote: > >> NFP PMD PF support requires to access the NFP chip for initialization. > >> Current NFP PMD PF support was added based on the NSPU interface. This > >> implies to do initialization through the NSP, a embedded ARM processor > >> which does initialization tasks on demand. The main problem with this > >> approach is it requires to add support for new NSP commands each time > >> a new functionality is required, which does not scale well and it is > >> not really flexible. > >> > >> Using the new CPP user space interface, the PMD can do whatever could > >> be done by the NSP, this is current commands and any new functionality > >> required. This CPP interface allows to access any single chip component > >> facilitating initialization, firmware uploading, firmware debugging or > >> extended stats. > >> > >> The changes just change the PMD PF initialization and do not touch the > >> datapath at all. No performance changes nor PMD functionalities are > affected. > >> > >> The initial impact using the new CPP interface is the way firmware > upload > >> is handled, which helps the PMD detecting the card type and the > firmware file > >> to upload. Future commits will include extended stats and some sort of > debug > >> channel. > >> > >> The specific CPP code is contained in the first patch, which has not > been > >> splitted up because is completely internal to the NFP functionality. The > >> second patch makes the PMD changes required for using the new interface. > >> > >> v2: > >> - removing unused reference to zlib.h > >> - fix build errors > >> - add SPDX tags in new files > >> - rebase changes nfp.rst > >> > >> Alejandro Lucero (4): > >> net/nfp: add NFP CPP support > >> net/nfp: update PMD for using new CPP interface > >> doc: update NFP guide > >> net/nfp: remove files > > > > Series applied to dpdk-next-net/master, thanks. > > btw, some nfp files seems still missing spdx tags, can you please check > them? > Yes, those are not new files. I will send a patch soon. Thanks!