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 65140B441 for ; Wed, 18 Feb 2015 15:12:09 +0100 (CET) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga103.fm.intel.com with ESMTP; 18 Feb 2015 06:03:19 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,602,1418112000"; d="scan'208";a="679775491" Received: from bricha3-mobl3.ger.corp.intel.com ([10.243.20.37]) by fmsmga002.fm.intel.com with SMTP; 18 Feb 2015 06:10:39 -0800 Received: by (sSMTP sendmail emulation); Wed, 18 Feb 2015 14:10:38 +0025 Date: Wed, 18 Feb 2015 14:10:37 +0000 From: Bruce Richardson To: "Wodkowski, PawelX" Message-ID: <20150218141037.GE5960@bricha3-MOBL3> References: <1424257369-19130-1-git-send-email-mukawa@igel.co.jp> <20150218123006.GB5960@bricha3-MOBL3> <20150218123159.GC5960@bricha3-MOBL3> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [RFC PATCH] lib/librte_ethdev: Expand port identifier 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: Wed, 18 Feb 2015 14:12:09 -0000 On Wed, Feb 18, 2015 at 01:05:10PM +0000, Wodkowski, PawelX wrote: > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Bruce Richardson > > Sent: Wednesday, February 18, 2015 1:32 PM > > To: Tetsuya Mukawa > > Cc: dev@dpdk.org > > Subject: Re: [dpdk-dev] [RFC PATCH] lib/librte_ethdev: Expand port identifier > > > > On Wed, Feb 18, 2015 at 12:30:07PM +0000, Bruce Richardson wrote: > > > On Wed, Feb 18, 2015 at 08:02:49PM +0900, Tetsuya Mukawa wrote: > > > > Currently uint8_t is used for port identifier. This patch changes it, > > > > and use uint16_t as port identifier. > > > > This patch only changes ethdev library. ABI of the library will be > > > > kept even after applying it. > > > > > > > > Also, this patch involves following fixes. > > > > - Use "port_id" as variable name instead of "port". > > > > > > > > > > > > Signed-off-by: Tetsuya Mukawa > > > > --- > > > > lib/librte_ether/rte_ethdev.c | 212 +- > > > > lib/librte_ether/rte_ethdev_internal.h | 3672 > > ++++++++++++++++++++++++++++++++ > > > > 2 files changed, 3778 insertions(+), 106 deletions(-) > > > > create mode 100644 lib/librte_ether/rte_ethdev_internal.h > > > > > > > I'm not sure I follow why we need a new header file for this. > > > Also, thinking about this change, a more fundamental problem is going to be > > > the mbuf structure, which stores a port id inside it in an 8-bit value. > > > Upgrading that to a 16-bit value requires some thought, and verification to > > > ensure any adjustment of fields does not lead to serious performance issues. > > > > > > Therefore, I suggest we leave the port id values as 8-bits until such time > > > as we need greater than 255 port values in a real-world use case. > > > Out of interest - anyone have a DPDK app where they use >16 port id values? If > > > so, how high does the port id value get? > > Not real application but simple example of setup: > 4 Niantic x 2 ports x 64 VF = 512 port id However, I'd find it hard to see why you would split a single port into 64 within a *single* application, let alone do that with 8 of them :-) > > I don't know what would be the real need/advantage of such setup (bonding?) but > you see, in theory it is already insufficient. Well, in theory any number of ports is insufficient, as you can create thousands and thousands of pcap or ring ethdevs in an app if you really want. Hence the question about any actual use cases which use a large number of ports. :-) /Bruce > > > > > > > Regards, > > > /Bruce > > > > > > > Resending with correct email addr for Neil. > > > > /Bruce