From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 500019AC8 for ; Wed, 2 Mar 2016 09:18:54 +0100 (CET) Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (Postfix) with ESMTPS id 1F99BAED22; Wed, 2 Mar 2016 08:18:53 +0000 (UTC) Received: from sopuli.koti.laiskiainen.org (vpn1-4-227.ams2.redhat.com [10.36.4.227]) by int-mx14.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u228Ip8M025469; Wed, 2 Mar 2016 03:18:51 -0500 To: Jan Medala , dev@dpdk.org References: <1456850801-3737-1-git-send-email-jan@semihalf.com> <1456850801-3737-5-git-send-email-jan@semihalf.com> From: Panu Matilainen Message-ID: <56D6A1EA.3040604@redhat.com> Date: Wed, 2 Mar 2016 10:18:50 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <1456850801-3737-5-git-send-email-jan@semihalf.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.68 on 10.5.11.27 Cc: Evgeny Schemeilin , matua@amazon.com Subject: Re: [dpdk-dev] [PATCH v4 4/4] ena: DPDK polling-mode driver for Amazon Elastic Network Adapters (ENA) 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: Wed, 02 Mar 2016 08:18:54 -0000 On 03/01/2016 06:46 PM, Jan Medala wrote: > This is a PMD for the Amazon ethernet ENA family. > The driver operates variety of ENA adapters through feature negotiation > with the adapter and upgradable commands set. > ENA driver handles PCI Physical and Virtual ENA functions. > > Signed-off-by: Evgeny Schemeilin > Signed-off-by: Jan Medala > Signed-off-by: Jakub Palider > --- > config/common_linuxapp | 11 + > drivers/net/Makefile | 1 + > drivers/net/ena/Makefile | 65 ++ > drivers/net/ena/ena_ethdev.c | 1311 +++++++++++++++++++++++++++++++ > drivers/net/ena/ena_ethdev.h | 155 ++++ > drivers/net/ena/ena_logs.h | 74 ++ > drivers/net/ena/ena_platform.h | 58 ++ > drivers/net/ena/rte_pmd_ena_version.map | 4 + > mk/rte.app.mk | 1 + > 9 files changed, 1680 insertions(+) > create mode 100644 drivers/net/ena/Makefile > create mode 100644 drivers/net/ena/ena_ethdev.c > create mode 100644 drivers/net/ena/ena_ethdev.h > create mode 100644 drivers/net/ena/ena_logs.h > create mode 100644 drivers/net/ena/ena_platform.h > create mode 100644 drivers/net/ena/rte_pmd_ena_version.map > [...] > diff --git a/drivers/net/ena/Makefile b/drivers/net/ena/Makefile > new file mode 100644 > index 0000000..0beb850 > --- /dev/null > +++ b/drivers/net/ena/Makefile > @@ -0,0 +1,65 @@ > +# [...] > +include $(RTE_SDK)/mk/rte.vars.mk > + > +# > +# library name > +# > +LIB = librte_pmd_ena.a > +CFLAGS += $(WERROR_FLAGS) -O2 > +INCLUDES :=-I$(SRCDIR) -I$(SRCDIR)/base/ena_defs -I$(SRCDIR)/base > + > +EXPORT_MAP := rte_pmd_ena_version.map > +LIBABIVER := 1 > + > +VPATH += $(SRCDIR)/base > +# > +# all source are stored in SRCS-y > +# > +SRCS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += ena_ethdev.c > +SRCS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += ena_com.c > +SRCS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += ena_eth_com.c > + > +# this lib depends upon: > +DEPDIRS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += lib/librte_eal lib/librte_ether > +DEPDIRS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += lib/librte_mempool lib/librte_mbuf > +DEPDIRS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += lib/librte_net lib/librte_malloc > + > +ifeq ($(CONFIG_RTE_EXEC_ENV),"cvos") > +CFLAGS += -Wno-old-style-definition > +endif Hmm, what's this? There's no "cvos" exec env in DPDK that I know of, and while its of course quite harmless I dont think its entirely appropriate to have references to in-house projects and the like (assuming that's what it is) in an OSS project. [...] > diff --git a/drivers/net/ena/rte_pmd_ena_version.map b/drivers/net/ena/rte_pmd_ena_version.map > new file mode 100644 > index 0000000..ef35398 > --- /dev/null > +++ b/drivers/net/ena/rte_pmd_ena_version.map > @@ -0,0 +1,4 @@ > +DPDK_2.0 { > + > + local: *; > +}; The symbol versions should reflect the actual DPDK version where introduced. Its fairly academical here since no actual symbols are exported but for correctness sake, it should say DPDK_16.04 as the version. - Panu -