From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id A0C1FC6E2 for ; Thu, 18 Jun 2015 16:13:07 +0200 (CEST) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga103.jf.intel.com with ESMTP; 18 Jun 2015 07:13:06 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,638,1427785200"; d="scan'208";a="713303874" Received: from bricha3-mobl3.ger.corp.intel.com ([10.243.20.21]) by orsmga001.jf.intel.com with SMTP; 18 Jun 2015 07:13:04 -0700 Received: by (sSMTP sendmail emulation); Thu, 18 Jun 2015 15:13:03 +0025 Date: Thu, 18 Jun 2015 15:13:03 +0100 From: Bruce Richardson To: "Walukiewicz, Miroslaw" Message-ID: <20150618141303.GB5344@bricha3-MOBL3> References: <1434560084-21237-2-git-send-email-konstantin.ananyev@intel.com> <1434633528-23329-1-git-send-email-konstantin.ananyev@intel.com> <7C4248CAE043B144B1CD242D27562653454ABD61@IRSMSX104.ger.corp.intel.com> <20150618133900.GD7972@bricha3-MOBL3> <7C4248CAE043B144B1CD242D27562653454ABDA5@IRSMSX104.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <7C4248CAE043B144B1CD242D27562653454ABDA5@IRSMSX104.ger.corp.intel.com> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCHv2 0/5] ethdev: add new API to retrieve RX/TX queue information X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 18 Jun 2015 14:13:08 -0000 On Thu, Jun 18, 2015 at 02:56:52PM +0100, Walukiewicz, Miroslaw wrote: > Yes, really I need a high performance function - it is especially useful on TX path. > > Which queue query function are you talking here to be enhanced? > There are already functions for querying the RX queue - so as to enable power management/frequency scaling. rte_eth_rx_queue_count() rte_eth_rX_descriptor_done() You could write equivalents for the TX side that only do the minimum needed. Descriptor-done is probably the simplest and most useful for you, as in you can query if you have enough space for a single operation, rather than having the code scan arbitrarily far ahead. /Bruce > Mirek > > > -----Original Message----- > > From: Richardson, Bruce > > Sent: Thursday, June 18, 2015 3:39 PM > > To: Walukiewicz, Miroslaw > > Cc: Ananyev, Konstantin; dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCHv2 0/5] ethdev: add new API to retrieve RX/TX > > queue information > > > > On Thu, Jun 18, 2015 at 01:30:33PM +0000, Walukiewicz, Miroslaw wrote: > > > Konstantin, > > > > > > Is there a possibility to read information about available space in NIC queue > > for TX. > > > > > > It is quite easy to compute (or even available directly) and very useful > > especially for application sending multi-descriptor packets like TCP TSO. > > > > > > Now there is no access to such information and the transmit packet function > > must be called to > > > be sure that there is available space. > > > > > > Mirek > > > > > > > Hi Mirek, > > > > I'm not sure this function is the right place for such a query. It seems that > > a function like you require needs to be used on the datapath, while this API > > is not planned to be high-performance. I think that it might be better to > > add an equivalent function to the existing RX descriptor querying function. > > > > /Bruce > > > > > > -----Original Message----- > > > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Konstantin > > Ananyev > > > > Sent: Thursday, June 18, 2015 3:19 PM > > > > To: dev@dpdk.org > > > > Subject: [dpdk-dev] [PATCHv2 0/5] ethdev: add new API to retrieve RX/TX > > > > queue information > > > > > > > > Add the ability for the upper layer to query RX/TX queue information. > > > > Right now supported for: > > > > ixgbe, i40e, e1000 PMDs. > > > > > > > > Konstantin Ananyev (5): > > > > ethdev: add new API to retrieve RX/TX queue information > > > > i40e: add support for eth_(rxq|txq)_info_get > > > > ixgbe: add support for eth_(rxq|txq)_info_get > > > > e1000: add support for eth_(rxq|txq)_info_get > > > > testpmd: add new command to display RX/TX queue information > > > > > > > > app/test-pmd/cmdline.c | 48 +++++++++++++++++++++++++ > > > > app/test-pmd/config.c | 67 > > > > ++++++++++++++++++++++++++++++++++ > > > > app/test-pmd/testpmd.h | 2 ++ > > > > drivers/net/e1000/e1000_ethdev.h | 12 +++++++ > > > > drivers/net/e1000/em_ethdev.c | 2 ++ > > > > drivers/net/e1000/em_rxtx.c | 38 ++++++++++++++++++++ > > > > drivers/net/e1000/igb_ethdev.c | 4 +++ > > > > drivers/net/e1000/igb_rxtx.c | 36 +++++++++++++++++++ > > > > drivers/net/i40e/i40e_ethdev.c | 2 ++ > > > > drivers/net/i40e/i40e_ethdev.h | 5 +++ > > > > drivers/net/i40e/i40e_rxtx.c | 42 ++++++++++++++++++++++ > > > > drivers/net/ixgbe/ixgbe_ethdev.c | 4 +++ > > > > drivers/net/ixgbe/ixgbe_ethdev.h | 6 ++++ > > > > drivers/net/ixgbe/ixgbe_rxtx.c | 42 ++++++++++++++++++++++ > > > > lib/librte_ether/rte_ethdev.c | 54 ++++++++++++++++++++++++++++ > > > > lib/librte_ether/rte_ethdev.h | 77 > > > > +++++++++++++++++++++++++++++++++++++++- > > > > 16 files changed, 440 insertions(+), 1 deletion(-) > > > > > > > > -- > > > > 1.8.5.3 > > >