drivers: fix the event dependency on crypto

Message ID 20200305120412.16085-1-hemant.agrawal@nxp.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series drivers: fix the event dependency on crypto |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/travis-robot success Travis build: passed
ci/Intel-compilation success Compilation OK

Commit Message

Hemant Agrawal March 5, 2020, 12:04 p.m. UTC
  Since the introduction of crypto event adapters, event drivers
have dependencies on crypto drivers.

/usr/bin/ld: cannot find -lrte_pmd_dpaa_sec
collect2: error: ld returned 1 exit status
make[9]: *** [.../mk/rte.lib.mk:100:
	librte_pmd_dpaa_event.so.20.0.2] Error 1

Fixes: b0f66a68ca74 ("event/dpaa: support crypto adapter")
Fixes: 3721c39f405a ("event/dpaa2: support crypto adapter")
Cc: stable@dpdk.org

Suggested-by: Ferruh Yigit <ferruh.yigit@intel.com>
Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

David Marchand March 5, 2020, 8:04 a.m. UTC | #1
On Thu, Mar 5, 2020 at 7:32 AM Hemant Agrawal <hemant.agrawal@nxp.com> wrote:
>
> Since the introduction of crypto event adapters, event drivers
> have dependencies on crypto drivers.
>
> /usr/bin/ld: cannot find -lrte_pmd_dpaa_sec
> collect2: error: ld returned 1 exit status
> make[9]: *** [.../mk/rte.lib.mk:100:
>         librte_pmd_dpaa_event.so.20.0.2] Error 1
>
> Fixes: b0f66a68ca74 ("event/dpaa: support crypto adapter")
> Fixes: 3721c39f405a ("event/dpaa2: support crypto adapter")
> Cc: stable@dpdk.org
>

Reported-by: David Marchand <david.marchand@redhat.com>

> Suggested-by: Ferruh Yigit <ferruh.yigit@intel.com>
> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
> ---
>  drivers/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/Makefile b/drivers/Makefile
> index 46374ca69..c70bdf9cc 100644
> --- a/drivers/Makefile
> +++ b/drivers/Makefile
> @@ -21,7 +21,7 @@ DEPDIRS-compress := bus mempool
>  DIRS-$(CONFIG_RTE_LIBRTE_VHOST) += vdpa
>  DEPDIRS-vdpa := common bus mempool
>  DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
> -DEPDIRS-event := common bus mempool net
> +DEPDIRS-event := common bus mempool net crypto
>  DIRS-$(CONFIG_RTE_LIBRTE_RAWDEV) += raw
>  DEPDIRS-raw := common bus mempool net event

Acked-by: David Marchand <david.marchand@redhat.com>
  
David Marchand March 5, 2020, 1:56 p.m. UTC | #2
On Thu, Mar 5, 2020 at 9:04 AM David Marchand <david.marchand@redhat.com> wrote:
> On Thu, Mar 5, 2020 at 7:32 AM Hemant Agrawal <hemant.agrawal@nxp.com> wrote:
> >
> > Since the introduction of crypto event adapters, event drivers
> > have dependencies on crypto drivers.
> >
> > /usr/bin/ld: cannot find -lrte_pmd_dpaa_sec
> > collect2: error: ld returned 1 exit status
> > make[9]: *** [.../mk/rte.lib.mk:100:
> >         librte_pmd_dpaa_event.so.20.0.2] Error 1
> >
> > Fixes: b0f66a68ca74 ("event/dpaa: support crypto adapter")
> > Fixes: 3721c39f405a ("event/dpaa2: support crypto adapter")
> > Cc: stable@dpdk.org
> >
>
> Reported-by: David Marchand <david.marchand@redhat.com>
> > Suggested-by: Ferruh Yigit <ferruh.yigit@intel.com>
> > Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
> Acked-by: David Marchand <david.marchand@redhat.com>
>

Applied.
  

Patch

diff --git a/drivers/Makefile b/drivers/Makefile
index 46374ca69..c70bdf9cc 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -21,7 +21,7 @@  DEPDIRS-compress := bus mempool
 DIRS-$(CONFIG_RTE_LIBRTE_VHOST) += vdpa
 DEPDIRS-vdpa := common bus mempool
 DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
-DEPDIRS-event := common bus mempool net
+DEPDIRS-event := common bus mempool net crypto
 DIRS-$(CONFIG_RTE_LIBRTE_RAWDEV) += raw
 DEPDIRS-raw := common bus mempool net event