From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 092CF4C9D for ; Wed, 13 Mar 2019 12:15:42 +0100 (CET) X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 13 Mar 2019 04:15:42 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.58,474,1544515200"; d="scan'208";a="131268884" Received: from yexl-server.sh.intel.com (HELO localhost) ([10.67.110.206]) by fmsmga008.fm.intel.com with ESMTP; 13 Mar 2019 04:15:41 -0700 Date: Wed, 13 Mar 2019 19:12:10 +0800 From: Ye Xiaolong To: Ferruh Yigit Cc: dev@dpdk.org, Qi Zhang Message-ID: <20190313111210.GA51517@intel.com> References: <20190301080947.91086-1-xiaolong.ye@intel.com> <20190301080947.91086-2-xiaolong.ye@intel.com> <20190312155430.GC46228@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) Subject: Re: [dpdk-dev] [PATCH v1 1/6] net/af_xdp: introduce AF_XDP PMD driver X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 13 Mar 2019 11:15:43 -0000 On 03/13, Ferruh Yigit wrote: > >>> <...> >>> >>>> +Prerequisites >>>> +------------- >>>> + >>>> +This is a Linux-specific PMD, thus the following prerequisites apply: >>>> + >>>> +* A Linux Kernel with XDP sockets configuration enabled; >>> >>> Can you please give more details of what exact vanilla kernel version? >> >> Do you mean I should write more details about AF_XDP in kernel in this introduction >> document? > >I think it is good to document the exact version information instead of saying >"Linux Kernel with af_xdp". Get your point now, will add the exact kernel info. > >>>> +* libbpf with latest af_xdp support installed >>> >>> Is there a specific version of libbpf for this? >> >> I'm not aware that there is specific version number for libbpf, it's part of linux >> kernel src code. > >If it is coming with Linux kernel, which version of Linux kernel? Will add the kernel version info. > >>> <...> >>> >>>> + >>>> +#include >>>> +#include >>>> +#include >>>> +#include >>> >>> Getting an build error for this [1], can there be any include path param missing? >>> >>> [1] >>> drivers/net/af_xdp/rte_eth_af_xdp.c:15:10: fatal error: asm/barrier.h: No such >>> file or directory >> >> Yes, it need something like >> >> CFLAGS += -I/root/yexl/shared_mks0/linux/tools/include >> >> as in above Makefile currently. > >I see, assuming you will be booting up with that kernel, can something like >below work: > >CFLAGS += -I/lib/modules/$(shell uname -r)/build/tools/include/ > I'll have a try. >>> <...> >>> >>>> +static void >>>> +eth_dev_info(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info) >>>> +{ >>>> + struct pmd_internals *internals = dev->data->dev_private; >>>> + >>>> + dev_info->if_index = internals->if_index; >>>> + dev_info->max_mac_addrs = 1; >>>> + dev_info->max_rx_pktlen = (uint32_t)ETH_FRAME_LEN; >>>> + dev_info->max_rx_queues = 1; >>>> + dev_info->max_tx_queues = 1; >>> >>> 'ETH_AF_XDP_MAX_QUEUE_PAIRS' is '16' but you are forcing the max Rx/Tx queue >>> number to be '1', intentional? >> >> Yes, current implementation is single queue only, we plan to support muli-queues >> in futher. > >OK, Can you please document this information? Sure. Thanks, Xiaolong >