From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id C344A1C329 for ; Fri, 11 May 2018 14:28:49 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 May 2018 05:28:49 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,388,1520924400"; d="scan'208";a="40999452" Received: from rhorton-mobl1.ger.corp.intel.com (HELO FC23.ir.intel.com) ([163.33.230.68]) by orsmga006.jf.intel.com with ESMTP; 11 May 2018 05:28:47 -0700 From: Remy Horton To: dev@dpdk.org Cc: Ferruh Yigit , Konstantin Ananyev , Wenzhuo Lu , Qi Zhang , Beilei Xing Date: Fri, 11 May 2018 13:28:44 +0100 Message-Id: <20180511122845.22757-2-remy.horton@intel.com> X-Mailer: git-send-email 2.9.5 In-Reply-To: <20180511122845.22757-1-remy.horton@intel.com> References: <20180511122845.22757-1-remy.horton@intel.com> Subject: [dpdk-dev] [PATCH v2 1/2] net/i40e: fix missing Port Representor data-path 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: Fri, 11 May 2018 12:28:50 -0000 This patch adds Rx and Tx burst functions to the i40e Port Representors, so that the implementation within this PMD can be tested using applications such as testpmd which require data-path functionality. Fixes: e0cb96204b71 ("net/i40e: add support for representor ports") Signed-off-by: Remy Horton --- drivers/net/i40e/i40e_vf_representor.c | 22 +++++++++++++++++++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git a/drivers/net/i40e/i40e_vf_representor.c b/drivers/net/i40e/i40e_vf_representor.c index 96b3787..6026ec9 100644 --- a/drivers/net/i40e/i40e_vf_representor.c +++ b/drivers/net/i40e/i40e_vf_representor.c @@ -337,6 +337,20 @@ struct eth_dev_ops i40e_representor_dev_ops = { }; +static uint16_t +i40e_vf_representor_rx_burst(__rte_unused void *rx_queue, + __rte_unused struct rte_mbuf **rx_pkts, __rte_unused uint16_t nb_pkts) +{ + return 0; +} + +static uint16_t +i40e_vf_representor_tx_burst(__rte_unused void *tx_queue, + __rte_unused struct rte_mbuf **tx_pkts, __rte_unused uint16_t nb_pkts) +{ + return 0; +} + int i40e_vf_representor_init(struct rte_eth_dev *ethdev, void *init_params) { @@ -365,9 +379,11 @@ i40e_vf_representor_init(struct rte_eth_dev *ethdev, void *init_params) /* Set representor device ops */ ethdev->dev_ops = &i40e_representor_dev_ops; - /* No data-path so no RX/TX functions */ - ethdev->rx_pkt_burst = NULL; - ethdev->tx_pkt_burst = NULL; + /* No data-path, but need stub Rx/Tx functions to avoid crash + * when testing with the likes of testpmd. + */ + ethdev->rx_pkt_burst = i40e_vf_representor_rx_burst; + ethdev->tx_pkt_burst = i40e_vf_representor_tx_burst; vf = &pf->vfs[representor->vf_id]; -- 2.9.5