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 E536C45A97; Thu, 3 Oct 2024 20:52:02 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id CC0C7402B3; Thu, 3 Oct 2024 20:52:00 +0200 (CEST) Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) by mails.dpdk.org (Postfix) with ESMTP id 74EBE402B1 for ; Thu, 3 Oct 2024 20:51:59 +0200 (CEST) Received: by mail-pl1-f176.google.com with SMTP id d9443c01a7336-208cf673b8dso13598755ad.3 for ; Thu, 03 Oct 2024 11:51:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1727981518; x=1728586318; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=ht6av6cR0vCNx1UyYIJ+jvpdFUoeDn+qjuu0JLzQd8M=; b=O3C722FrfjxBS/zNWJoXsihDxe3WvX278w46Vp0Hw1VPSky8oaj5PAYyHCikPn1TFt yvBircF7B343ThZ1bWbZ4cBI3sAYYrerXxeOhhRt0TvVjK1EvQpW5Oix8Sre2qjMXYLf FgAmAtCC5bGWnyjoFx555r09XArsXownITX+OHkBr4N1MpulfACIe68MO+WU+ql+tN19 DiauP01DQ5V9PHsJmXH9EyjEklwBALj8E5dGUNRVTOMWjQoAYosQouPD2xVeQ/L3HXB+ PZND3iIsDBrf4veaWUmxuTyyZF+LgW8Hx2OjzSLoo0zx+4f2pwPprWaLeHawXXyv30Wo Rl8A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1727981518; x=1728586318; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=ht6av6cR0vCNx1UyYIJ+jvpdFUoeDn+qjuu0JLzQd8M=; b=xQv254ELvIwwP83PFG6RhQ/JXYrHpKkTfTuOJ0UCMOKb6SKz33pxvDOWdPyL3lL0Vu dHl24YZM1nWKwT/nibWcWeT9JCA4h6nQMxDOwrfmur58t4ud6BnuGbMFqQBgIJWYdbBG n4WmokKjA0GfwsI2ivWAkcva6yekUdwDDlexQ+yBEBqCDTmml/XnPpbMgObB/vBddwU1 I9qO+kSovY+oBuJw26A+NEumlgkzI7xH9QtU7LXjeCvh38hv2XOShu6qeSS0DyPCIMFM Uf5jp4b42jwlabENyRhirsvmqQo9IejmQnWk6rMYfal/cmJDKFp2wU5nHvUeHVW2/wf2 HCVQ== X-Forwarded-Encrypted: i=1; AJvYcCVBNaJDZo86mtwRhBJWS71i02KOyq0Hdfe/vkbi3CB96KVtd8t/FE11gxLEcXn3HMhpNN0=@dpdk.org X-Gm-Message-State: AOJu0YzaX0zj2wuW0wPz4pRCBGtZeUUZo6p32QsArWsE7nsIYxj+i3+a FaEvC9w0igiswI3OLR3bm0eRMSpo6G+nuZP1UOdfchTHGpKcshvK4ZWynLakKWw= X-Google-Smtp-Source: AGHT+IHZ2UmTaHM1tfDWtSAdjcVFcaagBFxXKGm+5HVZThPybzWPoZq76EMmUhs+/f+889SgkoZ4fA== X-Received: by 2002:a17:90a:ff14:b0:2e0:875a:f72d with SMTP id 98e67ed59e1d1-2e1e5dc6231mr141602a91.0.1727981518288; Thu, 03 Oct 2024 11:51:58 -0700 (PDT) Received: from hermes.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-20bef7074ddsm11972275ad.243.2024.10.03.11.51.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 03 Oct 2024 11:51:58 -0700 (PDT) Date: Thu, 3 Oct 2024 11:51:56 -0700 From: Stephen Hemminger To: fengchengwen Cc: Morten =?UTF-8?B?QnLDuHJ1cA==?= , Thomas Monjalon , Dongdong Liu , Bruce Richardson , , Ferruh Yigit , , , , Subject: Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command Message-ID: <20241003115156.0c55c54d@hermes.local> In-Reply-To: References: <20221219090723.29356-1-fengchengwen@huawei.com> <12802674.ZYm5mLc6kN@thomas> <18761354.fAMKPKieAE@thomas> <98CBD80474FA8B44BF855DF32C47DC35D87A5C@smartserver.smartshare.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit 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 On Tue, 4 Jul 2023 14:41:27 +0800 fengchengwen wrote: > > A general comment about telemetry: > > > > If an application exposes telemetry through an end user facing API, e.g. http(s) REST, it would be nice if non-read-only telemetry paths are easy to identify by following some DPDK standard convention, so the application does not need to manually maintain an allow-list of read-only paths. > > +1 for this point. > > > > > Bruce's documentation about trace/log/telemetry/dump might also need to be updated regarding non-read-only telemetry actions. > > I just check Bruce's patch [1], and notice that the telemetry callback must be 'read-only': (Telemetry callbacks should not modify any program state, but be "read-only"). > > From internal product usage, we think xstats-reset is valid to identify problem, but this callback is not read-only. > > We think telemetry callback should not limit to 'read-only'. Perhaps we could develop some strategy to better manage non-read-only callbacks (just like Morten's advise). > > [1]: https://patchwork.dpdk.org/project/dpdk/patch/20230620170728.74117-3-bruce.richardson@intel.com/ This patch set had functional test failures, and there never was a good conclusion about allowing telemetry to do reset. If you want to keep going with it, go through the feedback and resubmit.