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 76E61A0C45; Mon, 13 Sep 2021 12:54:00 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 5DAB540151; Mon, 13 Sep 2021 12:54:00 +0200 (CEST) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mails.dpdk.org (Postfix) with ESMTP id E481C4014F for ; Mon, 13 Sep 2021 12:53:58 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10105"; a="282640182" X-IronPort-AV: E=Sophos;i="5.85,288,1624345200"; d="scan'208";a="282640182" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Sep 2021 03:53:58 -0700 X-IronPort-AV: E=Sophos;i="5.85,288,1624345200"; d="scan'208";a="543117955" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.30.142]) by fmsmga003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 13 Sep 2021 03:53:56 -0700 Date: Mon, 13 Sep 2021 11:53:53 +0100 From: Bruce Richardson To: David Hunt Cc: dev@dpdk.org, ciara.power@intel.com Message-ID: References: <20210909155625.24581-1-david.hunt@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [dpdk-dev] [PATCH v1] usertools/telemetry: add non-interactive 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 Mon, Sep 13, 2021 at 11:43:25AM +0100, Bruce Richardson wrote: > On Thu, Sep 09, 2021 at 04:56:25PM +0100, David Hunt wrote: > > Add non-interactive mode to dpdk-telemetry.py so that a query string > > can be supplied on the command line, and script dumps out data and > > exits. Handing for calling from scripts. > > > > Signed-off-by: David Hunt > > --- > Hi Dave, > > I'm not sure I like the use of "-q" for adding a query mode - it's more a > shortcut parameter for a "quiet" mode. If I may, I'd suggest an alternative > approach here might be to improve support for piping the input commands to > the script instead so that you can do e.g. > > "echo /ethdev/stats,0 | dpdk-telemetry.py" > > and have that work well in a script. > > I'll do up a patchset for improving that and upstream it for feedback. > Now at: http://patches.dpdk.org/project/dpdk/list/?series=18867 /Bruce