From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?iso-8859-1?Q?Ga=EBtan?= Rivet Subject: Re: [PATCH v2 06/12] cryptodev: disabled by default Date: Wed, 14 Jun 2017 10:00:33 +0200 Message-ID: <20170614080033.GU29091@bidouze.vm.6wind.com> References: <1b9434af8872a0fa468618835c34e255e9c76c36.1496877060.git.gaetan.rivet@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Cc: "dev@dpdk.org" To: "De Lara Guarch, Pablo" Return-path: Received: from mail-wr0-f176.google.com (mail-wr0-f176.google.com [209.85.128.176]) by dpdk.org (Postfix) with ESMTP id 443427CBB for ; Wed, 14 Jun 2017 10:00:43 +0200 (CEST) Received: by mail-wr0-f176.google.com with SMTP id q97so177148091wrb.2 for ; Wed, 14 Jun 2017 01:00:43 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Pablo, On Fri, Jun 09, 2017 at 03:03:08PM +0000, De Lara Guarch, Pablo wrote: > > > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Gaetan Rivet > > Sent: Thursday, June 08, 2017 12:59 AM > > To: dev@dpdk.org > > Cc: Gaetan Rivet > > Subject: [dpdk-dev] [PATCH v2 06/12] cryptodev: disabled by default > > > > Signed-off-by: Gaetan Rivet > > --- > > config/common_base | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/config/common_base b/config/common_base > > index 0e0b732..cade611 100644 > > --- a/config/common_base > > +++ b/config/common_base > > @@ -430,7 +430,7 @@ CONFIG_RTE_PMD_PACKET_PREFETCH=y > > # > > # Compile generic crypto device library > > # > > -CONFIG_RTE_LIBRTE_CRYPTODEV=y > > +CONFIG_RTE_LIBRTE_CRYPTODEV=n > > CONFIG_RTE_LIBRTE_CRYPTODEV_DEBUG=n > > CONFIG_RTE_CRYPTO_MAX_DEVS=64 > > CONFIG_RTE_CRYPTODEV_NAME_LEN=64 > > -- > > 2.1.4 > > As already discussed, there should be no need to disable this, > after applying this: http://dpdk.org/ml/archives/dev/2017-May/066382.html > Just to confirm, once properly applied, your patchset fixes the issue. I will remove this patch from the next version and update the dependencies to include your patchset. -- Gaëtan Rivet 6WIND