From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 407FFA04DD for ; Wed, 27 Nov 2019 21:46:02 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id EE0CC1BEAE; Wed, 27 Nov 2019 21:46:01 +0100 (CET) Received: from us-smtp-1.mimecast.com (us-smtp-delivery-1.mimecast.com [207.211.31.120]) by dpdk.org (Postfix) with ESMTP id 40A841BEA7 for ; Wed, 27 Nov 2019 21:45:59 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1574887559; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=ANdRNVENBbsJ2h1pY9vr1CG/3ON+r2E5CJvrFVKn/O4=; b=IQHfn7Xx2aYlUv88SxWE3keVnnnBRCwnGZcF/Ic8vcUALgyzZbTqcK89gEECSYzQnpISt3 IV808vLKrWTl4mJw+GtaVnFQpNmsbOhzgLrEOSyw5hF+xLk6DVD7I6EBMgRNq9D+VnENHc kM8az+dyA9V/ULJMWyIWRYE3eDLLEsI= Received: from mail-vk1-f198.google.com (mail-vk1-f198.google.com [209.85.221.198]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-355-jkOv8oBZOUC6i1z4flrdSg-1; Wed, 27 Nov 2019 15:45:58 -0500 Received: by mail-vk1-f198.google.com with SMTP id b1so11051631vkk.1 for ; Wed, 27 Nov 2019 12:45:58 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=/82TgGR2Vd6Sz6VWcaoZW/PYv6qk7Wj/Pd60wHDlArY=; b=E/r200MwV5rWq5JE5ML/yTtCUBwUnZjaU60zsGam927z7818fIq13n1rDDH7+tfwHI mFrYGuQ+DDkRxT/XJA3fta/ydL4tr6duBl+C6sPy1GXaLt+mYGg5/9Vf8vzF+LbRBFv0 xVokCbRFDD9+oqt0xlsNMXbiqnuNqpsnMNHaGMQgRnHfOfNo5qvU6Xn4ukvT1wBzLAYh ob7ThNVGiVoZNCiQ2sMX84q5SSbkXFMrfGJXL92jyHjVCPedePLZ0p6rCkayvELCxSyE QT0enk3cs3Sti2E9+6gVmEIcyYI4a3kO5pw1mynauh0+sOSHLNyqlP+dqEFRrJ4vkRB5 8alA== X-Gm-Message-State: APjAAAVnKamJBcnJcMsgdBgD4iiclHhnak8G76uv/+sBN9NJ8MHJeJMZ 0hNM5ujZ8FTEAMGOBGCCrVxbGGRH/Vdyokf7gCiKCVJ1gQeKg/FVZutqWagZdK/+dV7VWBrUfeP 1R1EaGcSd5HPIX139vE4O7Bc= X-Received: by 2002:ab0:2716:: with SMTP id s22mr4141860uao.86.1574887557633; Wed, 27 Nov 2019 12:45:57 -0800 (PST) X-Google-Smtp-Source: APXvYqxkO8txHqFF0Pt4oyN3l151Qy6rqnlrD8+MX6yufWfv/QNEmLYKvme7qwxwQrpeO2Qk/zvp7jF+wkh/UdZoAA8= X-Received: by 2002:ab0:2716:: with SMTP id s22mr4141836uao.86.1574887557212; Wed, 27 Nov 2019 12:45:57 -0800 (PST) MIME-Version: 1.0 References: <20191125161314.18804-1-david.marchand@redhat.com> <20191126142212.GC21200@hmswarspite.think-freely.org> In-Reply-To: <20191126142212.GC21200@hmswarspite.think-freely.org> From: David Marchand Date: Wed, 27 Nov 2019 21:45:46 +0100 Message-ID: To: Neil Horman Cc: dev , Thomas Monjalon , Andrew Rybchenko , dpdk stable , Ray Kinsella , John McNamara , Marko Kovacevic , Qiming Yang , Wenzhuo Lu , Declan Doherty , Adrien Mazarguil , Ferruh Yigit , Cristian Dumitrescu X-MC-Unique: jkOv8oBZOUC6i1z4flrdSg-1 X-Mimecast-Spam-Score: 0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Subject: Re: [dpdk-stable] [RFC PATCH] mark experimental variables X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On Tue, Nov 26, 2019 at 3:22 PM Neil Horman wrote: > On Mon, Nov 25, 2019 at 05:13:14PM +0100, David Marchand wrote: > > So far, we did not pay attention to direct access to variables but they > > are part of the API/ABI too and should be clearly identified. > > > > Introduce a __rte_experimental_var tag and mark existing variables. > > > > Fixes: a4bcd61de82d ("buildtools: add script to check experimental API = exports") > > Cc: stable@dpdk.org > > > > Signed-off-by: David Marchand > > --- > > Quick patch to try to catch experimental variables. > > Not sure if we could use a single section, so please advise if there is > > better to do about this. > > > I don't see any definition of __rte_experimental_var here, won't the > preprocessor choke on this when you try to compile without that? Sorry, not getting your point. If there is an issue, then it is the same as __rte_experimental. [snip] > > diff --git a/lib/librte_eal/common/include/rte_compat.h b/lib/librte_ea= l/common/include/rte_compat.h > > index 3eb33784b..3fd05179f 100644 > > --- a/lib/librte_eal/common/include/rte_compat.h > > +++ b/lib/librte_eal/common/include/rte_compat.h > > @@ -11,11 +11,16 @@ > > #define __rte_experimental \ > > __attribute__((deprecated("Symbol is not yet part of stable ABI"), \ > > section(".text.experimental"))) > > +#define __rte_experimental_var \ > > +__attribute__((deprecated("Symbol is not yet part of stable ABI"), \ > > +section(".data.experimental"))) > > > > #else > > > > #define __rte_experimental \ > > __attribute__((section(".text.experimental"))) > > +#define __rte_experimental_var \ > > +__attribute__((section(".data.experimental"))) > > > > #endif > > --=20 David Marchand