From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 5E0531B599 for ; Thu, 10 Jan 2019 09:39:17 +0100 (CET) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id C3E3024534; Thu, 10 Jan 2019 03:39:16 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute1.internal (MEProxy); Thu, 10 Jan 2019 03:39:16 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=mesmtp; bh=l7S2dlxFTADcTbU4GDFsIil6GGt9Apcu8Z/O8bsnoTQ=; b=lZSgrejeHBS/ mBTBdsRUDxx2IuAqvmcgQ4aTx83cxSlgadlIzjrTb1kwycOW2Ar5oaSK9j5UlEKm te+Ry29IsrVUPjA4z8RTnRlLxzjOSVYFGR96pg2HDfQnL0Z0b55fvjgNekrwiaR/ dhiZFXbH6u8MDFH8nCTztTV670lSyYs= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=l7S2dlxFTADcTbU4GDFsIil6GGt9Apcu8Z/O8bsno TQ=; b=p3gvIQJl5l4byNtYeJAy4ZkPF4okeojyU7A4vdabHt8EiIOtyOTIrg1gf c0Bd7626tkHfyoN0FDWNF3qCNoklE3HkLnrA/ZInlQ2Cl9zpwlTPw5Z5yIwPC9nj 4kbMVOzLQXg3E3gYvVpAkjuntDwI4q6u58i5hqnvfXX+y70SsU96kbZgiTQMPlHm yWicj+zxxpHv38kO58Yl5B9lGTpm+qK/H5Rjo8fGSEJn8KyfkmmWYjPSeCTgQ2xW +tkkJ2JZdr/RIhyGiA4efuLx/ete9yFjgUiEH8dn7mvJu7zQJsUUDgHlK0sGbo3i cuceWpZmNtE+OBFcj7eFDBjTgR0zA== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedtledrfedvgdduvddvucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfquhhtnecuuegrihhlohhuthemucef tddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjughrpefhvffufffkjg hfggfgtgesthfuredttddtvdenucfhrhhomhepvfhhohhmrghsucfoohhnjhgrlhhonhcu oehthhhomhgrshesmhhonhhjrghlohhnrdhnvghtqeenucfkphepjeejrddufeegrddvtd efrddukeegnecurfgrrhgrmhepmhgrihhlfhhrohhmpehthhhomhgrshesmhhonhhjrghl ohhnrdhnvghtnecuvehluhhsthgvrhfuihiivgepud X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id B9F36E4598; Thu, 10 Jan 2019 03:39:14 -0500 (EST) From: Thomas Monjalon To: "Varghese, Vipin" Cc: "dev@dpdk.org" , "Pattan, Reshma" , "Mcnamara, John" , "Patel, Amol" , "stephen@networkplumber.org" , "Ananyev, Konstantin" , "Singh, Jasvinder" Date: Thu, 10 Jan 2019 09:39:13 +0100 Message-ID: <4383677.neL5jlz7Vu@xps> In-Reply-To: <4C9E0AB70F954A408CC4ADDBF0F8FA7D4D2EE451@BGSMSX101.gar.corp.intel.com> References: <20181213050842.64587-1-vipin.varghese@intel.com> <15168357.XSmGzJBqIm@xps> <4C9E0AB70F954A408CC4ADDBF0F8FA7D4D2EE451@BGSMSX101.gar.corp.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v8 1/7] app/procinfo: prepare for new debug functions 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, 10 Jan 2019 08:39:17 -0000 10/01/2019 04:08, Varghese, Vipin: > snipped > > > > Hi Vipin, > > > > The code split in this v8 looks really better, except this patch which makes no > > sense alone. > Hi Thomas, I need this base patch as first one. These include the helper MACRO and include which pans out for next 7 patches. > > > I feel you can move these changes in next patches, where appropriate, isn't it? > Patches v2 to v7 uses common features from v1. At least, you should introduce the includes when needed. If you need this patch for introducing some macros, then rename it and explain the macros in the message. > > 07/01/2019 16:38, Vipin Varghese: > > > Update code base and meson build file to accomadate changes for the > > > new functionality. > > > > > > Signed-off-by: Vipin Varghese > > > Acked-by: Reshma Pattan > > > --- > > > app/proc-info/main.c | 13 +++++++++++++ > > > app/proc-info/meson.build | 2 +- > > > 2 files changed, 14 insertions(+), 1 deletion(-) > > > > > > diff --git a/app/proc-info/main.c b/app/proc-info/main.c index > > > c20effa4f..c7697389c 100644 > > > --- a/app/proc-info/main.c > > > +++ b/app/proc-info/main.c > > > @@ -29,6 +29,9 @@ > > > #include > > > #include > > > #include > > > +#include > > > +#include > > > +#include > > > > > > /* Maximum long option length for option parsing. */ #define > > > MAX_LONG_OPT_SZ 64 @@ -36,6 +39,10 @@ > > > > > > #define MAX_STRING_LEN 256 > > > > > > +#define STATS_BDR_FMT > > "========================================" > > > +#define STATS_BDR_STR(w, s) printf("%.*s%s%.*s\n", w, \ > > > + STATS_BDR_FMT, s, w, STATS_BDR_FMT) > > > + > > > /**< mask of enabled ports */ > > > static uint32_t enabled_port_mask; > > > /**< Enable stats. */ > > > @@ -65,6 +72,9 @@ static char *xstats_name; static uint32_t > > > nb_xstats_ids; static uint64_t xstats_ids[MAX_NB_XSTATS_IDS]; > > > > > > +/* show border */ > > > +static char bdr_str[MAX_STRING_LEN]; > > > + > > > /**< display usage */ > > > static void > > > proc_info_usage(const char *prgname) > > > @@ -668,5 +678,8 @@ main(int argc, char **argv) > > > if (ret) > > > printf("Error from rte_eal_cleanup(), %d\n", ret); > > > > > > + snprintf(bdr_str, MAX_STRING_LEN, " "); > > > + STATS_BDR_STR(50, bdr_str); > > > + > > > return 0; > > > } > > > diff --git a/app/proc-info/meson.build b/app/proc-info/meson.build > > > index a52b2ee4a..866b390d6 100644 > > > --- a/app/proc-info/meson.build > > > +++ b/app/proc-info/meson.build > > > @@ -3,4 +3,4 @@ > > > > > > sources = files('main.c') > > > allow_experimental_apis = true > > > -deps += ['ethdev', 'metrics'] > > > +deps += ['ethdev', 'metrics', 'security']