From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2 06/12] cryptodev: move vdev functions to a separate file Date: Tue, 27 Jun 2017 01:30:59 +0200 Message-ID: <2058861.mNNx7DgSpj@xps> References: <1495639634-74846-1-git-send-email-pablo.de.lara.guarch@intel.com> <20170621062826.21896-7-pablo.de.lara.guarch@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: Declan Doherty , Pablo de Lara Return-path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id 8743A374 for ; Tue, 27 Jun 2017 01:31:01 +0200 (CEST) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 23/06/2017 14:52, Declan Doherty: > On 21/06/2017 7:28 AM, Pablo de Lara wrote: > > Move all functions handling virtual devices to a separate > > header file "rte_cryptodev_vdev.h", in order to leave only > > generic functions for any device in the rest of the files. > > > > Signed-off-by: Pablo de Lara > > Acked-by: Declan Doherty This patch is moving some code from a .c to a .h. The consequence is that driver are now compiling new functions from the .h and require librte_kvargs to link with. It is a build error in shared lib mode. Please keep the code in a .c file if possible.