From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tetsuya Mukawa Subject: Re: [PATCH v5 4/9] null: virtual dynamic rss configuration Date: Thu, 15 Oct 2015 16:46:16 +0900 Message-ID: <561F59C8.1070105@igel.co.jp> References: <1436981189-3320-1-git-send-email-tomaszx.kulasek@intel.com> <1443621905-7872-1-git-send-email-tomaszx.kulasek@intel.com> <1443621905-7872-5-git-send-email-tomaszx.kulasek@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-2022-jp Content-Transfer-Encoding: 7bit To: Tomasz Kulasek , dev@dpdk.org Return-path: Received: from mail-pa0-f50.google.com (mail-pa0-f50.google.com [209.85.220.50]) by dpdk.org (Postfix) with ESMTP id 902978E90 for ; Thu, 15 Oct 2015 09:46:20 +0200 (CEST) Received: by payp3 with SMTP id p3so32147460pay.1 for ; Thu, 15 Oct 2015 00:46:19 -0700 (PDT) In-Reply-To: <1443621905-7872-5-git-send-email-tomaszx.kulasek@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 2015/09/30 23:05, Tomasz Kulasek wrote: > This implementation allows to set and read RSS configuration for null > device, and is used to validate right values propagation over the slaves, > in test units for dynamic RSS configuration for bonding. > > v5 changes: > - replaced memcpy with rte_memcpy > > Signed-off-by: Tomasz Kulasek > --- > drivers/net/null/rte_eth_null.c | 116 +++++++++++++++++++++++++++++++++++++++ > 1 file changed, 116 insertions(+) > > diff --git a/drivers/net/null/rte_eth_null.c b/drivers/net/null/rte_eth_null.c > index bf81b1b..b01f647 100644 > --- a/drivers/net/null/rte_eth_null.c > +++ b/drivers/net/null/rte_eth_null.c > @@ -37,6 +37,8 @@ > #include > #include > #include > +#include > +#include > > Hi Tomasz, We don't have "rte_eth_null.h" at this point. (The header file will be added next patch) Probably, we also need "rte_pmd_null_version.map" to compile correctly. (To make sure, please compile DPDK with "CONFIG_RTE_BUILD_SHARED_LIB=y" option.) Also, it seems 'rte_eth_null.h' should be included like below. #include "rte_eth_null.h" Without it, we cannot compile. Thanks, Tetsuya