From patchwork Mon Jan 7 15:38:22 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Varghese, Vipin" X-Patchwork-Id: 49464 Return-Path: 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 071C31B3A2; Mon, 7 Jan 2019 10:41:43 +0100 (CET) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 8F3301B1F2 for ; Mon, 7 Jan 2019 10:41:41 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 Jan 2019 01:41:40 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,450,1539673200"; d="scan'208";a="289444969" Received: from unknown (HELO saesrv02-S2600CWR.intel.com) ([10.224.122.203]) by orsmga005.jf.intel.com with ESMTP; 07 Jan 2019 01:41:37 -0800 From: Vipin Varghese To: dev@dpdk.org, thomas@monjalon.net Cc: reshma.pattan@intel.com, john.mcnamara@intel.com, amol.patel@intel.com, stephen@networkplumber.org, konstantin.ananyev@intel.com, jasvinder.singh@intel.com, Vipin Varghese Date: Mon, 7 Jan 2019 21:08:22 +0530 Message-Id: <20190107153829.34047-1-vipin.varghese@intel.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20181213050842.64587-1-vipin.varghese@intel.com> References: <20181213050842.64587-1-vipin.varghese@intel.com> Subject: [dpdk-dev] [PATCH v8 0/7] app/proc-info: enhance 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 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. The patch series tries to address the missing debug information for PMD by adding debug for port, traffic manager, crypto, ring and mempool. With these additional information collected, it becomes easy to analyze issues and performance variance. Status ====== Reviews and changes accomadated. ACK received for documenation and code. Patch Information ================= Patch series has 7 patches as shared below: Vipin Varghese (7): app/procinfo: prepare for new debug functions app/procinfo: add support for show port app/procinfo: add support for show tm app/procinfo: add support for show crypto app/procinfo: add support for debug ring app/procinfo: add support for show mempool app/procinfo: add support for iter mempool app/proc-info/main.c | 658 ++++++++++++++++++++++++++++++++- app/proc-info/meson.build | 2 +- doc/guides/tools/proc_info.rst | 32 +- 3 files changed, 688 insertions(+), 4 deletions(-) Change Log: ========== v8: - merge documenation with code - Thomas Monjalon - split patch for iter_mempool - Reshma Pattan - add ack for code - Reshma Pattan V7: - update usage text with token ":" - Reshma Pattan - reword usage for 'iter-mempool' - Reshma Pattan V6: - add mempool iterate elements option - Vipin Varghese - check for iter mempool - Vipin Varghese - add call to iter mempool - Vipin Varghese - replace or with logical or - Reshma Pathan & Konstantin Ananyev - split iter mempool - Vipin Varghese - update entry for mempool iter - Vipin Varghese - add ack for v5 - John Mcnamara - update entry for mempool iter - Vipin Varghese V5: - convert 100 to MAX_STRING_LEN - Reshma Pattan - RSS made for port - Reshma Pattan - update bdr_str to static - Stephen Hemminger - remove memset for link, dev_info, queue, stats - Stephen Hemminger - update ret to uint32_t - Reshma Pattan v4: - add space to compare - Vipin Varghese - add spacing for flag compare - Vipin Varghese V3: - change the usage details - Vipin Varghese - variables from debug to show - Vipin Varghese - update function names from debug to show - Vipin Varghese - fix meson build - Reshma Pattan - change 100 to MAX_STRING_LEN - Reshma Pattan - memset to struct elements - Reshma Pattan - printf tab space - Reshma Pattan - remove 'drop packet information' - Vipin Varghese - memset for struct elements - Vipin Varghese - code cleanup for TM - Vipin Varghese - fetch for leaf nodes if node exist - Jasvinder Singh - display MARCO to function - Reshma Pathan & Stephen Hemminger - add memset for struct elements - Reshma Pathan - change display formating of flags - Vipin Varghese - use MACRO for string - Vipin Varghese - replace space to tab in printf - Reshma Pathan - change ring display information - Vipin Varghese - update document from dbg to show - Vipin Varghese V2: - change word dbg to show - Stephen Hemminger - compare string from dbg to show - Stephen Hemminger - removed if else ladder - Vipin Varghese - redefine code format - Vipin Varghese - MACRO for display node|level - cap - Vipin Varghese - update word style for content - Vipin Varghese