From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id A873CA0C4B; Fri, 15 Oct 2021 10:19:04 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8F81A40692; Fri, 15 Oct 2021 10:19:04 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mails.dpdk.org (Postfix) with ESMTP id 908AE40041 for ; Fri, 15 Oct 2021 10:19:02 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10137"; a="228156330" X-IronPort-AV: E=Sophos;i="5.85,375,1624345200"; d="scan'208";a="228156330" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Oct 2021 01:19:01 -0700 X-IronPort-AV: E=Sophos;i="5.85,375,1624345200"; d="scan'208";a="492372055" Received: from rkeegan-mobl1.ger.corp.intel.com (HELO bricha3-MOBL.ger.corp.intel.com) ([10.252.15.29]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 15 Oct 2021 01:18:59 -0700 Date: Fri, 15 Oct 2021 09:18:56 +0100 From: Bruce Richardson To: David Marchand Cc: dev , Ciara Power , Anatoly Burakov , Kevin Traynor Message-ID: References: <20210915141030.23514-1-bruce.richardson@intel.com> <20211014104907.856490-1-bruce.richardson@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [dpdk-dev] [PATCH v9 0/4] improve telemetry support with in-memory mode X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 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 Thu, Oct 14, 2021 at 09:00:09PM +0200, David Marchand wrote: > On Thu, Oct 14, 2021 at 12:50 PM Bruce Richardson > wrote: > > > > This patchset cleans up telemetry support for "in-memory" mode, so that > > multiple independent processes can be run using that mode and still have > > telemetry support. It also removes problems of one process removing the > > socket of another - which was the original issue reported. The main changes > > in this set are to: > > > > * disable telemetry for secondary processes, which prevents any socket > > conflicts in multi-process cases. > > * when multiple processes are run using the same runtime directory (i.e. > > "in-memory" mode or similar), add a counter suffix to the socket names to > > avoid conflicts over the socket. Each process will use the lowest available > > suffix, with the first process using the directory, not adding any suffix. > > * update the telemetry script and documentation to allow it to connect to > > in-memory DPDK processes. > > > > Thanks a lot for working on this. > Reading the updated doc, I fixed some -file-prefix to --file-prefix in doc. > Thanks for the fixups.