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 25664A057B; Wed, 1 Apr 2020 16:12:25 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id A237F2C15; Wed, 1 Apr 2020 16:12:24 +0200 (CEST) Received: from us-smtp-1.mimecast.com (us-smtp-delivery-1.mimecast.com [207.211.31.120]) by dpdk.org (Postfix) with ESMTP id 54F12FFA for ; Wed, 1 Apr 2020 16:12:23 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1585750342; 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=jw3aBCDYCIoPpoVkvu/ky2BfxOl6YGfW8YmNkfnDq7U=; b=K6Lv5Hkiv0fHuRU4b6NVZhNkSfsI/sWkyqK0D9F3ZPAnewsD73s2lfy/HHM9dlNmqtVl3p WGq57rRXwr/xGeDlT1yY02NI4XMJAfxEgqWcXjo4jcWEh45qed9mJLNlVpiGQ4gGCwfS6G rsJL/qChEb0pGLkvFoj4ptFsEs0eifQ= Received: from mail-vs1-f70.google.com (mail-vs1-f70.google.com [209.85.217.70]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-185-E5gfKnrJNYq6_hkgzTE75w-1; Wed, 01 Apr 2020 10:12:20 -0400 X-MC-Unique: E5gfKnrJNYq6_hkgzTE75w-1 Received: by mail-vs1-f70.google.com with SMTP id u203so6159692vsu.1 for ; Wed, 01 Apr 2020 07:12:20 -0700 (PDT) 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=80f3BSYU2jMKImOOoLRCTWdt813s+WlykJoz+CWET60=; b=SV8HFKQO/GhMActIZSRqc3z/CRlmdCjgwBkSCdU8/DPirlKq5F1wasUB9+T7PDwHsg SeEhUdzFrAlapBpPseU0PSBFHWDN4VIr1hUvJ2XR/GK1pmdYVvLZrf25hX/T/9XSchNO 3gcN+uABdSulYD8UMlHbrmYEn6Ma+XO8y2e2b1XZqyrfC/j4jhcAjuNe4S0CdBE32i38 8sil0/aXkGf6cw6cyQdZJo3FD4bnq9hfLjtCExfmpvQ3wZXVuQNmvJCIIAm/nj7YvU7d zzaWyYg5cWdOI1oSK5qQMFzipBMUDBzaXsaodQcQCzD9cxjjBzc55igoQ3OI6WXtVUvZ Ddyw== X-Gm-Message-State: AGi0PuaVJEPA9Tbw8WopeySosYeup3jA0XGeTo+UzjZELgAikv1U4/1/ Cnm300a+qjAik1L5oHk78UdUJOc+k8TV09gKudFlaX5FgZFaBE68fplz7SUhVRBDd6YRzLu3od2 S1cUh2MukN8kZv+4awt8= X-Received: by 2002:a67:26c2:: with SMTP id m185mr16952872vsm.180.1585750340272; Wed, 01 Apr 2020 07:12:20 -0700 (PDT) X-Google-Smtp-Source: APiQypKTATt6A5LKlY6d00WGbx82YbuD9OBdRE79680nKPgVo/l8+kgjkiajhiW3JHo2tD9dLJj+IcBDCL21nvCi4yw= X-Received: by 2002:a67:26c2:: with SMTP id m185mr16952841vsm.180.1585750339892; Wed, 01 Apr 2020 07:12:19 -0700 (PDT) MIME-Version: 1.0 References: <20200325211603.240288-1-jerinj@marvell.com> <20200329144342.1543749-1-jerinj@marvell.com> In-Reply-To: From: David Marchand Date: Wed, 1 Apr 2020 16:12:08 +0200 Message-ID: To: Jerin Jacob Kollanukkaran , Thomas Monjalon Cc: dev , Bruce Richardson , =?UTF-8?Q?Mattias_R=C3=B6nnblom?= , Sunil Kumar Kori , "Yigit, Ferruh" , Andrew Rybchenko , Declan Doherty , Olivier Matz , Neil Horman , Ray Kinsella X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Subject: Re: [dpdk-dev] [PATCH v3 00/33] DPDK Trace support 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Wed, Apr 1, 2020 at 12:05 PM Jerin Jacob wrote: > On Wed, Apr 1, 2020 at 1:49 PM David Marchand = wrote: > > - Regardless of the trace framework, the ALLOW_EXPERIMENTAL_API flag > > gates access to APIs so that external users are aware of their status. > > I have been considering setting this flag unconditionally for internal > > users in the top Makefile/meson for app/ lib/ and drivers/. > > I could look at this and prepare a patch about this, but this is not > > enough here. > > It makes sense to me. Let me know when you are planning to send that patc= h, > I will rebase this series on top of that. Feel free to take over, thanks. > > If you don't have time then I can send the patch too. > I assume the patch content will be: > 1) Removing experimental API from app, lib, drivers, examples with > make and meson > 2) Have it enabled at the global level. examples are a special case as they can be compiled out of the dpdk sources= . This is why I excluded them of the list in my mail before. > > How about: > > * we introduce a global config switch that enables/disables the trace > > framework (off by default): the people who want to test it and help > > stabilize it will have to deal with the experimental flag for now, > > * in 20.11, the trace points are put into the stable ABI, and the > > option is removed, > > IMO, the better alternative would be: > > Since the trace changes in the "inline" functions of the specific > library already > disabled under _compile time_ RTE_ENABLE_TRACE_DP flag and > it is using RTE_TRACE_POINT_DP() to define the trace unlike slow path > trace like RTE_TRACE_POINT(). Ah indeed. Note: RTE_ENABLE_TRACE_DP is not plugged with meson. > So I can improve RTE_TRACE_POINT_DP() to make absolute NOP if > ALLOW_EXPERIMENTAL_API not defined. > > On the upside, > The tracing code will be enabled by default(runtime it is disabled by > default anyway). > If some need to fastpath API tracing then ALLOW_EXPERIMENTAL_API need > to enable. > So this won't break applications. So either keep the RTE_ENABLE_TRACE_DP flag or use ALLOW_EXPERIMENTAL_API... no opinion. Thomas? > > - With the patchset rebased on the current master (could be my fault, > > so take it with a grain of salt), the ALLOW_EXPERIMENTAL_API flag is > > not passed when compiling the l3fwd example against an installed dpdk. > > I will check. We have added ALLOW_EXPERIMENTAL_API flag where we got > compilation issues. No compilation issue, just big fat warnings reporting that l3fwd did not enable ALLOW_EXPERIMENTAL_API and it is not built with -Werror. I caught it with ./devtools/test-*build*.sh scripts. --=20 David Marchand