From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 05719A04B6; Tue, 13 Oct 2020 10:31:22 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id DB0161DA6A; Tue, 13 Oct 2020 10:31:20 +0200 (CEST) Received: from shelob.oktetlabs.ru (shelob.oktetlabs.ru [91.220.146.113]) by dpdk.org (Postfix) with ESMTP id 2F1F31BACC; Tue, 13 Oct 2020 10:31:18 +0200 (CEST) Received: from [192.168.38.17] (aros.oktetlabs.ru [192.168.38.17]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by shelob.oktetlabs.ru (Postfix) with ESMTPSA id CA3997F572; Tue, 13 Oct 2020 11:31:16 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 shelob.oktetlabs.ru CA3997F572 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=oktetlabs.ru; s=default; t=1602577876; bh=4N1kHUgeMvPGaUlxzn6Jpq9/WtatvRKVTijhIRn5za8=; h=Subject:To:Cc:References:From:Date:In-Reply-To; b=gjb96bIi82Anv2cYozaH0cvH8o6ivm9JQibvULA3w9g3SOf84gqr8mDFCndvjI51y QHCmKz5jknp358GIu7ZmSB3yMaj/AzzL+nWSS3ImqHpRjyXgPUgR4WKE9Ao5BESjeh 8FHYvPx1Bg2RaiOcRXIiDpAqPst77EweLQCcE4yE= To: Thomas Monjalon , Ferruh Yigit Cc: dev@dpdk.org, techboard@dpdk.org, Min Hu , Stephen Hemminger References: <20201012164602.1965694-1-ferruh.yigit@intel.com> <3382071.xncihUvvZe@thomas> From: Andrew Rybchenko Organization: OKTET Labs Message-ID: Date: Tue, 13 Oct 2020 11:31:16 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.1 MIME-Version: 1.0 In-Reply-To: <3382071.xncihUvvZe@thomas> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [RFC 1/2] ethdev: move queue stats to xstats 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" On 10/13/20 12:53 AM, Thomas Monjalon wrote: > 12/10/2020 18:46, Ferruh Yigit: >> Queue stats are stored in 'struct rte_eth_stats' as array and array size >> is defined by 'RTE_ETHDEV_QUEUE_STAT_CNTRS' compile time flag. >> >> As a result of technical board discussion, decided to remove the queue >> statistics from 'struct rte_eth_stats' in the long term. >> >> Instead PMDs should represent the queue statistics via xstats, this >> gives more flexibility on the number of the queues supported. > > Yes I like Stephen's idea to have dedicated API to get stats per queue. Of course it is doable via getting xstats by IDs, but IMHO it is over-complicated. From the other hand it sounds like a duplication to have it in xstats and dedicated API (basically the same as we have for basic stats and xstats).