On 10/16/2018 10:47 PM, Timothy Redaelli wrote:
Currently it's not possible to build DPDK as shared library with cryptodev disabled since vhost is trying to link with rte_crypto, but rte_crypto and rte_hash are only needed when you build vhost_crypto and so only when cryptodev is enabled. This patch fix this by linking rte_vhost with rte_crypto and rte_hash only when cryptodev is enabled. Fixes: b4ca81298613 ("vhost/crypto: fix build without cryptodev") Fixes: 939066d96563 ("vhost/crypto: add public function implementation") Cc: sta...@dpdk.org Signed-off-by: Timothy Redaelli <tredae...@redhat.com> Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com> --- v2: Fix the second 'Fixes' line. I wrongly copied that commit id from commit b4ca81298613, but it seems it's a commit not contained in any branch (as reported by Maxime and check-git-log.sh). --- lib/librte_vhost/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Applied to dpdk-next-virtio/master Thanks, Maxime