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 218471B957 for ; Sat, 22 Dec 2018 22:02:47 +0100 (CET) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id 41B1E213D0; Sat, 22 Dec 2018 16:02:46 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute1.internal (MEProxy); Sat, 22 Dec 2018 16:02:46 -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=9DNUtE2nXvx28RwSM+9Ro+7CIYF7nOpJndiZuhjikTU=; b=XbHhxkHdxzvx XcitrXpG6KIoVEZB+y+iSBbMDxW/teae4eGNqv/xPjcJuJH9+eaa0XdtIiVgu9tO qzZ4623Uppn9Su6QLOcfce5rGPjDkh2+81mvqyNFWVPNGSGghu65yRP9Z+M2V099 kHoBXrwtaMIoboEcvqR/wFsOjtLhQCk= 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=9DNUtE2nXvx28RwSM+9Ro+7CIYF7nOpJndiZuhjik TU=; b=QgvgLrVDDzM608rysNzcOdYSiDfGzIbipzdKXydy/eThgvvKKpbmWBGH1 d4d8lA/uhpSwRz9zHjxAPtVnIgXRCvCYhgWviIzAxPGVmB8j2Ke4KaLptdh8top2 mnwvBc3E16ZLDyT33E42WCUopmE4BVORGxkDnHClawLbvGDWmJkmt6SD+fbl+8rX o32kOEUwiGVX4QsJ6ullhcqRSNvTE0HW+NoDK0cVtb2xFFKgq9ppvxMAiGRLPeKe BgvXFFhw4ZnwNKmStKjzeIUUswwrHjHQReMwwkzw3akEgVLhNQO3PqWr/JQv5YWS v3IYgJ+57QzFvoEWGFmpjAYsZYVbA== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedtkedrudejjedgudeggecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfhuthenuceurghilhhouhhtmecu fedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmnecujfgurhephffvufffkf gjfhgggfgtsehtufertddttddvnecuhfhrohhmpefvhhhomhgrshcuofhonhhjrghlohhn uceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecukfhppeejjedrudefgedrvd dtfedrudekgeenucfrrghrrghmpehmrghilhhfrhhomhepthhhohhmrghssehmohhnjhgr lhhonhdrnhgvthenucevlhhushhtvghrufhiiigvpedt 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 9FE1510085; Sat, 22 Dec 2018 16:02:44 -0500 (EST) From: Thomas Monjalon To: "Varghese, Vipin" Cc: dev@dpdk.org, "Pattan, Reshma" , "Ananyev, Konstantin" , "stephen@networkplumber.org" , "Mcnamara, John" , "Byrne, Stephen1" , "Patel, Amol" Date: Sat, 22 Dec 2018 22:02:42 +0100 Message-ID: <1651608.RmD6NkZceM@xps> In-Reply-To: <4C9E0AB70F954A408CC4ADDBF0F8FA7D4D2DA895@BGSMSX101.gar.corp.intel.com> References: <20181203055000.39012-2-vipin.varghese@intel.com> <3AEA2BF9852C6F48A459DA490692831F2A3EA1F0@irsmsx110.ger.corp.intel.com> <4C9E0AB70F954A408CC4ADDBF0F8FA7D4D2DA895@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 v7 0/9] app/proc-info: improve debug of proc-info tool 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: Sat, 22 Dec 2018 21:02:47 -0000 13/12/2018 17:21, Varghese, Vipin: > HI Reshma > > From: Pattan, Reshma > > From: Varghese, Vipin > > > > > > This patch adds new debug functions to existing proc-info tool. > > > > > > Motivation > > > ========== > > > > > > DPDK proc-info tool is been widely used as secondary process to > > > collect stats for any primary process. But these are limited to DPDK > > > NIC ports and basic memory usage. > > > > > > Motivation > > > ========== > > > > > > This patch tries to address the short coming by adding debug for port, > > > traffic manager, crypto, ring and mempool. With these additional > > > information in tool will be able to deliver helpful data to debug issues and > > performance variance. > > > > > > Status > > > ====== > > > > > > With the following patch set debug data has been collected from > > > customers using DPDK instances. Analysing the information helped to > > > suggest the next debug steps and solutions for fixing the issues. > > > > > > Next Steps > > > ========== > > > > > > - add event dev debug information. > > > - enhance iter_mempool for INLINE crypto entries. > > > - add debug for libraries like hash and acl. > > > - explore debug possibility for ring and list. > > > - add links and reference to 'how to guide' for 'debug and troubleshoot'. > > > > > > Patch set Information > > > ===================== > > > > > > Patch includes set of 9 patches: > > > * 0 : to introduce the debug function enhancement for proc-info > > > * 1 : MACRO and help usage for new functions > > > * 2 : string compare for new functions > > > * 3 : invocation for the new prototypes > > > * 4 : show port > > > * 5 : show tm > > > * 6 : show crypto > > > * 7 : ring element debug > > > * 8 : mempool element debug > > > * 9 : iterate mempoool elements > > > > > > > Small nits > > 9th patch in this set is doc. So above info need to be corrected. > > if you are addressing my earlier comment of separating out mempool element > > iteration changes in to separate new patch 9/10 .Please keep my ack in next > > version > > Thanks for pointing this out, Like updated in email and chat I am not planning to split it. Hence no version 8. So, no ack and no merge? Looking at the first patches + doc patch, the split is not meaningful. You should merge doc and option parsing in the related patches. For instance, parsing and doc of "tm" option should be in the "tm" patch.