From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 9E3F79E7 for ; Thu, 11 Jun 2015 15:25:35 +0200 (CEST) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga103.fm.intel.com with ESMTP; 11 Jun 2015 06:25:34 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,595,1427785200"; d="scan'208";a="586018709" Received: from orsmsx101.amr.corp.intel.com ([10.22.225.128]) by orsmga003.jf.intel.com with ESMTP; 11 Jun 2015 06:25:34 -0700 Received: from orsmsx151.amr.corp.intel.com (10.22.226.38) by ORSMSX101.amr.corp.intel.com (10.22.225.128) with Microsoft SMTP Server (TLS) id 14.3.224.2; Thu, 11 Jun 2015 06:25:25 -0700 Received: from fmsmsx112.amr.corp.intel.com (10.18.116.6) by ORSMSX151.amr.corp.intel.com (10.22.226.38) with Microsoft SMTP Server (TLS) id 14.3.224.2; Thu, 11 Jun 2015 06:25:25 -0700 Received: from FMSMSX110.amr.corp.intel.com ([169.254.14.46]) by FMSMSX112.amr.corp.intel.com ([169.254.5.158]) with mapi id 14.03.0224.002; Thu, 11 Jun 2015 06:25:25 -0700 From: "Wang, Liang-min" To: "Ananyev, Konstantin" , "dev@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH v4 1/4] ethdev: add apis to support access device info Thread-Index: AQHQpEHLQvx5o4rpKEeANuQDErHw+52nPe8AgAB/9gD//4wxAA== Date: Thu, 11 Jun 2015 13:25:23 +0000 Message-ID: References: <1432946276-9424-1-git-send-email-liang-min.wang@intel.com> <1433948996-9716-1-git-send-email-liang-min.wang@intel.com> <1433948996-9716-2-git-send-email-liang-min.wang@intel.com> <2601191342CEEE43887BDE71AB97725836A08BA3@irsmsx105.ger.corp.intel.com> <2601191342CEEE43887BDE71AB97725836A08BEC@irsmsx105.ger.corp.intel.com> In-Reply-To: <2601191342CEEE43887BDE71AB97725836A08BEC@irsmsx105.ger.corp.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.1.200.106] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v4 1/4] ethdev: add apis to support access device info 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, 11 Jun 2015 13:25:36 -0000 > -----Original Message----- > From: Ananyev, Konstantin > Sent: Thursday, June 11, 2015 9:14 AM > To: Wang, Liang-min; dev@dpdk.org > Subject: RE: [dpdk-dev] [PATCH v4 1/4] ethdev: add apis to support access > device info >=20 >=20 > > > > + > > > > +int > > > > +rte_eth_dev_set_ringparam(uint8_t port_id, struct > > > > +rte_dev_ring_info > > > > +*info) { > > > > + struct rte_eth_dev *dev; > > > > + > > > > + if (!rte_eth_dev_is_valid_port(port_id)) { > > > > + PMD_DEBUG_TRACE("Invalid port_id=3D%d\n", port_id); > > > > + return -ENODEV; > > > > + } > > > > + > > > > + if ((dev=3D &rte_eth_devices[port_id]) =3D=3D NULL) { > > > > + PMD_DEBUG_TRACE("Invalid port device\n"); > > > > + return -ENODEV; > > > > + } > > > > + > > > > + FUNC_PTR_OR_ERR_RET(*dev->dev_ops->set_ringparam, - > > > ENOTSUP); > > > > + return (*dev->dev_ops->set_ringparam)(dev, info); } > > > > > > > > > I am a bit confused, what particular value of rte_dev_ring_info will > > > be allowed to change? > > > You can't change number of RXD/TXD at runtime. > > > You have to stop the device first, and then reconfigure particular > queue(s). > > > Again, you can't reset RXF/TXD without stopping RX/TX first. > > > So what that function is supposed to do? > > > As I can see currently, none of the PMD in your patch support it. > > > > > > > Exactly, setting RXD/TXD will require some change on kernel (RX/TX burs= t) > function. > > As you already observe that there is no dev_op registered for set_ring. > > The API is created as a placeholder for future implementation. > > There are a couple of idea thrown around (such as mono- > increase/decrease to avoid buffer overrun). > > Any suggestion is welcome. >=20 > If you don't plan to implement it in current patch-set, then it is better= to > remove it for now. > About suggestions - as I said, I don't think it is doable without stoppin= g (and > reconfiguring) the queue. > Again not sure, why do you want to do that run-tme. > Konstantin That's a very valid point. To support run-time ring descriptor size adjustm= ent, we need to stop traffic in some way.