From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id 3B828B0FD for ; Fri, 16 May 2014 20:54:15 +0200 (CEST) Received: from hmsreliant.think-freely.org ([2001:470:8:a08:7aac:c0ff:fec2:933b] helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from ) id 1WlNGs-0000M6-OF; Fri, 16 May 2014 14:54:22 -0400 Date: Fri, 16 May 2014 14:54:17 -0400 From: Neil Horman To: Bruce Richardson Message-ID: <20140516185417.GC5432@hmsreliant.think-freely.org> References: <1400264114-28455-1-git-send-email-bruce.richardson@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1400264114-28455-1-git-send-email-bruce.richardson@intel.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Spam-Score: -2.9 (--) X-Spam-Status: No Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH 0/3] ring: provide rte_ring_as_ethdev API 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: Fri, 16 May 2014 18:54:15 -0000 On Fri, May 16, 2014 at 07:15:11PM +0100, Bruce Richardson wrote: > This patch set aims to provide a shorter simpler alternative the public API functions for using rings as ethdevs provided by the librte_pmd_ring library. This alternative just provides simple RX and TX burst functions and a conversion API, without any of the complexities present in the pmd_ring version. This replacement should allow the public APIs in the pmd_ring library to be deprecated in the future. > > Bruce Richardson (3): > ethdev: Remove ethdev.h dependency on mbuf + mempool > ring: add support for converting a ring to ethdev > ring: autotest for using ring as ethdev > > app/test-pmd/cmdline.c | 1 + > app/test/test_pmd_ring.c | 1 + > app/test/test_ring.c | 25 ++++++++++++++++++++ > lib/librte_ether/rte_ethdev.h | 4 +++- > lib/librte_pmd_vmxnet3/vmxnet3_ethdev.c | 1 + > lib/librte_ring/Makefile | 1 + > lib/librte_ring/rte_ring.c | 42 +++++++++++++++++++++++++++++++++ > lib/librte_ring/rte_ring.h | 11 +++++++++ > 8 files changed, 85 insertions(+), 1 deletion(-) > > -- > 1.9.0 > > NAK, I don't think this makes sense. If you want to encapsulate a ring pair as an ethdev, then write a pmd that does so. That will give you a standardized ethdev that you can create using the existing --vdev librte_eal command line options without having to widen your API surface, or having to write applications that specifically know about the fact that your ethdev is composed of rings under the covers. Neil