From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f180.google.com (mail-wr0-f180.google.com [209.85.128.180]) by dpdk.org (Postfix) with ESMTP id CEA792BB9 for ; Thu, 8 Jun 2017 17:02:22 +0200 (CEST) Received: by mail-wr0-f180.google.com with SMTP id v111so19760358wrc.3 for ; Thu, 08 Jun 2017 08:02:22 -0700 (PDT) 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-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=zbbiuyXSMgIOmUtqrtSDrYMFsWYS6pquJtFiEzcBpWM=; b=mKUfxFw3/BzEjmNcfRpmRZou/R30Eb94ihcr4k60+Hulw8jEO/szw18Uux3niRwU/Z Cp7P++kDrwk5bKdsuCnnv28r1ww91gRf36uvCKEwJ0SY2WQus2VTfaEYjGZ5RyVWQYIZ 4upsQCpZz1KwW7CdeUSs8o88YM0xWxwYoZgpf5IAbgRdBPeNYh3q4LLzmLtwNPM8PLd6 hM01tXN8VtGHEi9sXXwOKJ85xBJ3IAzT181SkhMB7JieuEfa+jBeTIl09BSaAkva+ITv VIslPJquQop3QZ/K8UkB3uxbg4IbTQ8tXj5fyOY6t+3qlyhQ87/Opw+9SIwyf5PFVg6D 4ZaA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=zbbiuyXSMgIOmUtqrtSDrYMFsWYS6pquJtFiEzcBpWM=; b=luKC1W+bOlIA3ca9wzPQNqztYJNWWWTuMfv5dkRQ39Tb4I1sLEMuA8nv2DyeZUbnru v42lybDI9bd4xaPsjoqoonf/PfCsQ8IAFca1fQB0cXjdtK16dtduT7wzbyTDhD7odER0 Eh7Hn+n6v0fUsnkobAnCbjYypyiBKMA2Md49cH4QFmsgaw9S6mb7KmqYF2zAK4xFO31z EX41AbbrL0uylHnZA0DlAsk2lXpJWIyjTz2IeF43w562cfiMK7COVo/Y2ffFgempFvT/ OgyYTWaqCTltTL5tWg7qYclR6QE/mmT7BVinYO68xpsYsrSAssHX4cy1f8veAwgLDlN1 MEPw== X-Gm-Message-State: AODbwcDtllHNPnIb8iuy3JqjsfIST3F5qttowaNFE3fBoKwsHhrHz54x +YyET0n+ke+DajQdZYw= X-Received: by 10.223.139.152 with SMTP id o24mr15948597wra.146.1496934141480; Thu, 08 Jun 2017 08:02:21 -0700 (PDT) Received: from bidouze.vm.6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id h73sm9446591wma.10.2017.06.08.08.02.20 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 08 Jun 2017 08:02:20 -0700 (PDT) Date: Thu, 8 Jun 2017 17:02:13 +0200 From: =?iso-8859-1?Q?Ga=EBtan?= Rivet To: Ferruh Yigit Cc: dev@dpdk.org Message-ID: <20170608150213.GG18840@bidouze.vm.6wind.com> References: <85d051e9-626a-9228-3ee0-a098be2586f4@intel.com> <20170608133004.GE18840@bidouze.vm.6wind.com> <971db9c7-8ea9-3e79-a4f5-cf391226c83d@intel.com> <10800fbf-40ba-d3a2-d0c9-277980f44cd9@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <10800fbf-40ba-d3a2-d0c9-277980f44cd9@intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH v5 00/12] introduce fail-safe PMD 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: Thu, 08 Jun 2017 15:02:23 -0000 On Thu, Jun 08, 2017 at 03:25:03PM +0100, Ferruh Yigit wrote: > On 6/8/2017 3:20 PM, Ferruh Yigit wrote: > > On 6/8/2017 2:30 PM, Gaëtan Rivet wrote: > >> On Thu, Jun 08, 2017 at 11:56:37AM +0100, Ferruh Yigit wrote: > >>> On 6/8/2017 12:59 AM, Gaetan Rivet wrote: > >>>> eal: complete attach / detach support > >>>> http://dpdk.org/ml/archives/dev/2017-May/066366.html > >>>> http://dpdk.org/dev/patchwork/patch/24522/ > >>>> > >>>> ethdev: add flow API rule copy function > >>>> http://dpdk.org/ml/archives/dev/2017-May/066145.html > >>>> http://dpdk.org/dev/patchwork/patch/24406/ > >>>> > >>>> ethdev: add isolated mode to flow API > >>>> http://dpdk.org/ml/archives/dev/2017-April/064327.html > >>>> http://dpdk.org/dev/patchwork/patch/23741/ > >>> > >>> Hi Gaetan, > >>> > >>> Above dependencies are not enough to apply the PMD, it seems above ones > >>> also has dependencies to other patchsets, specially eal one that I am > >>> not able trace its dependencies from mails. > >>> > >>> Since most of the eal patches are also from you, can you please list > >>> dependency tree to apply the PMD? > >>> > >>> Thanks, > >>> ferruh > >>> > >> > >> Hi Ferruh, > >> > >> here is the full list of patchsets that I have currently in my tree, in > >> order, up until the fail-safe PMD: > >> > >> --- > >> > >> [0] ethdev: add isolated mode to flow API > >> http://dpdk.org/ml/archives/dev/2017-April/064327.html > >> > >> [1] ethdev: add flow API rule copy function > >> http://dpdk.org/ml/archives/dev/2017-May/066145.html > >> > >> [2] Remove PCI and VDEV dependency from eventdev library > >> http://dpdk.org/ml/archives/dev/2017-June/067402.html > >> > >> [3] Remove cryptodev driver > >> http://dpdk.org/ml/archives/dev/2017-May/066382.html > >> > >> [4] bus: attach / detach API > >> http://dpdk.org/ml/archives/dev/2017-June/067474.html > >> > >> [5] pci: implement find_device bus operation > >> http://dpdk.org/ml/archives/dev/2017-June/067485.html > >> > >> [6] rte_bus parse API > >> http://dpdk.org/ml/archives/dev/2017-June/067486.html > >> > >> [7] Generic devargs parsing > >> http://dpdk.org/ml/archives/dev/2017-June/067496.html > >> > >> [8] eal: complete attach / detach support > >> http://dpdk.org/ml/archives/dev/2017-June/067516.html > >> > >> [9] bus/pci: remove PCI bus from EAL > >> http://dpdk.org/ml/archives/dev/2017-June/067520.html > >> > >> [10] introduce fail-safe PMD > >> http://dpdk.org/ml/archives/dev/2017-June/067533.html > >> > >> --- > >> > >> [2] has a slight compilation issue that I fixed with > >> https://github.com/grivet/dpdk/commit/1af1272bd3129e90ae7c4ab005e85d5f8a2f95fe > >> However Jerin has not yet answered to this issue (see: > >> http://dpdk.org/ml/archives/dev/2017-June/067412.html ) > >> > >> The failsafe PMD only depends on 0, 1, 4, 5, 6, 7, 8. All others are > >> left there for consistency. > > > > Thanks Gaetan, > > > > I did able to apply the patchset, I think it is good idea to wait for > > PMD until things settle down in dependent patches side. > Sure, it won't compile without it anyway. > btw, first three patches of the failsafe patchset are more generic > ethdev layer patches. I think they shouldn't be merged as part of PMD > patchset. > > What do you think separating them into separate patchset (or perhaps > individual patches, since they looks independent from eachother)? > Why not, but I am juggling (read: struggling) with a few patchsets already. I will wait for the dust to settle and look into it. > Thanks, > ferruh > > > > > > > btw,getting build error [1], CFLAGS += -D_DEFAULT_SOURCE may help. > > > > [1] > > /usr/include/features.h:148:3: error: #warning "_BSD_SOURCE and > > _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE" [-Werror=cpp] > > # warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use > > _DEFAULT_SOURCE" > > > -- Gaëtan Rivet 6WIND