Message ID | 20180404150312.12304-2-adrien.mazarguil@6wind.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B0E901C781; Wed, 4 Apr 2018 17:56:46 +0200 (CEST) Received: from mail-wr0-f196.google.com (mail-wr0-f196.google.com [209.85.128.196]) by dpdk.org (Postfix) with ESMTP id 11F4B1C76D for <dev@dpdk.org>; Wed, 4 Apr 2018 17:56:44 +0200 (CEST) Received: by mail-wr0-f196.google.com with SMTP id d17so8140200wre.1 for <dev@dpdk.org>; Wed, 04 Apr 2018 08:56:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=fo581nqxXpYj/ORBe2OPS14kFvJN7SRp7wJ2AOV30A4=; b=ByQbKOzK+lyTYBEdKinNWm6wBTb+xxC6A9qiQ3yW2Szy6b6DbyF0ck1bR6dVjAguBC Au2SsEdZ2vHCRBF+yXDQsFLzlM/VOtey2846peLQ38ZOnlxuTOA/P9jMKJt1625M1Ctl Gn87iwqDr3pVyVKnQtT845LxI2dKer6G3PRYcqkPNih6dsLcizz2nFTRA/Go/eS/SFbG PQZtoCrv09p6+iu94rzsMimhETeDzRD/Q6Z1mEzw49CjJbbK8KO2khAwBQCTWGvhFncl YQR0lpYyJ0IfB81GWt8yEgYuwI0K/vEtytmqEU5mPLtZMLQZfAL15zuGWQmvQuZRids2 B0cA== 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:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=fo581nqxXpYj/ORBe2OPS14kFvJN7SRp7wJ2AOV30A4=; b=dmoOLRk4Fx7qQaQamo4CfBoEXILZVsXY/yFCxiJr3xLA1nk7owq3IPcv6FPSOxf6mX J5htCee0Y73iQCvCeOLRU8zs/wktn+zmYsGJaAE0ZlDgL6KQs5JlRO7+MO6gJG+l06ED iywMMSsjweJYBZOQeSY82e5+XEtY5pGQK4v973/WySD6Ph4vJC24YBHnX3WvGOmVNaog +xijn5qd1D4foTc3Ccd3eJvSY1PgsHGYt8Yy4JXX1ysRltm4JwPjuACi5zX3fAzqWWSw R/A3m8227hPhfyJF2RJZ8tSXaRlA+X5SRoZkbQvPGPJ3nI6+oIXipZZ5OHVp1SaC57qS VsyQ== X-Gm-Message-State: AElRT7FT8JYaGu7WuxV+OhjjifRAsQyAh6PFQsYepFHy3pj/lk+6LZxW zHbDAelhDBnmJYC8LGdhnbmq1TJr X-Google-Smtp-Source: AIpwx4/LqS5ht4wqAs2RbjGyDCPZXxFt1MN+hNfnj8gaizoALEAZCE0kSVsx1HiuPj44UAYS4DuTQA== X-Received: by 10.223.134.210 with SMTP id 18mr15048356wry.232.1522857404597; Wed, 04 Apr 2018 08:56:44 -0700 (PDT) Received: from 6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id h190sm6468410wmd.22.2018.04.04.08.56.43 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 04 Apr 2018 08:56:44 -0700 (PDT) Date: Wed, 4 Apr 2018 17:56:31 +0200 From: Adrien Mazarguil <adrien.mazarguil@6wind.com> To: Thomas Monjalon <thomas@monjalon.net>, Ferruh Yigit <ferruh.yigit@intel.com>, dev@dpdk.org Message-ID: <20180404150312.12304-2-adrien.mazarguil@6wind.com> References: <20180404150312.12304-1-adrien.mazarguil@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180404150312.12304-1-adrien.mazarguil@6wind.com> X-Mailer: git-send-email 2.11.0 Subject: [dpdk-dev] [PATCH v1 01/16] ethdev: update ABI for flow API functions X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Checks
Context | Check | Description |
---|---|---|
ci/checkpatch | success | coding style OK |
ci/Intel-compilation | fail | apply issues |
Commit Message
Adrien Mazarguil
April 4, 2018, 3:56 p.m. UTC
Subsequent patches will modify existing types and slightly alter the
behavior of the flow API. This warrants a major ABI breakage.
While it is already taken care of for 18.05 (LIBABIVER was updated to
version 9 by a prior commit), this patch explicitly adds the affected flow
API functions as a safety measure.
Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
---
lib/librte_ether/rte_ethdev_version.map | 10 ++++++++++
1 file changed, 10 insertions(+)
Comments
04/04/2018 17:56, Adrien Mazarguil: > Subsequent patches will modify existing types and slightly alter the > behavior of the flow API. This warrants a major ABI breakage. > > While it is already taken care of for 18.05 (LIBABIVER was updated to > version 9 by a prior commit), this patch explicitly adds the affected flow > API functions as a safety measure. I don't understand this patch. If the API is broken, you must move the function from old block to the new one. And it must be done in the patch modifying the function. > --- a/lib/librte_ether/rte_ethdev_version.map > +++ b/lib/librte_ether/rte_ethdev_version.map > +DPDK_18.05 { > + global: > + > + rte_flow_validate; > + rte_flow_create; > + rte_flow_query; > + rte_flow_copy; > + > +} DPDK_18.02;
On Thu, Apr 05, 2018 at 12:06:10PM +0200, Thomas Monjalon wrote: > 04/04/2018 17:56, Adrien Mazarguil: > > Subsequent patches will modify existing types and slightly alter the > > behavior of the flow API. This warrants a major ABI breakage. > > > > While it is already taken care of for 18.05 (LIBABIVER was updated to > > version 9 by a prior commit), this patch explicitly adds the affected flow > > API functions as a safety measure. > > I don't understand this patch. > > If the API is broken, you must move the function from old block to > the new one. Missed that part, I'll update it. > And it must be done in the patch modifying the function. About that, almost each patch in this series breaks the ABI in its own way. This left me with two options: either updating these functions once and for all and explaining why in a dedicated patch, or updating them in the first patch with an ABI impact, with subsequent patches piggybacking on that change. Unless there's a way to update the map file for each patch that breaks ABI, I think the former is more consistent, but I don't mind if you prefer the latter. What do you suggest?
05/04/2018 14:44, Adrien Mazarguil: > On Thu, Apr 05, 2018 at 12:06:10PM +0200, Thomas Monjalon wrote: > > 04/04/2018 17:56, Adrien Mazarguil: > > > Subsequent patches will modify existing types and slightly alter the > > > behavior of the flow API. This warrants a major ABI breakage. > > > > > > While it is already taken care of for 18.05 (LIBABIVER was updated to > > > version 9 by a prior commit), this patch explicitly adds the affected flow > > > API functions as a safety measure. > > > > I don't understand this patch. > > > > If the API is broken, you must move the function from old block to > > the new one. > > Missed that part, I'll update it. > > > And it must be done in the patch modifying the function. > > About that, almost each patch in this series breaks the ABI in its own > way. This left me with two options: either updating these functions once and > for all and explaining why in a dedicated patch, or updating them in the > first patch with an ABI impact, with subsequent patches piggybacking on that > change. > > Unless there's a way to update the map file for each patch that breaks ABI, > I think the former is more consistent, but I don't mind if you prefer the > latter. What do you suggest? The ABI information must be updated when breaking (2nd solution).
diff --git a/lib/librte_ether/rte_ethdev_version.map b/lib/librte_ether/rte_ethdev_version.map index 34df6c8b5..78a6f5afb 100644 --- a/lib/librte_ether/rte_ethdev_version.map +++ b/lib/librte_ether/rte_ethdev_version.map @@ -203,6 +203,16 @@ DPDK_18.02 { } DPDK_17.11; +DPDK_18.05 { + global: + + rte_flow_validate; + rte_flow_create; + rte_flow_query; + rte_flow_copy; + +} DPDK_18.02; + EXPERIMENTAL { global: