[v2] port: fix library build order dependency

Message ID 20191106144320.30600-1-rahul.r.shah@intel.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series [v2] port: fix library build order dependency |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-compilation success Compile Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/Intel-compilation fail apply issues

Commit Message

Rahul Shah Nov. 6, 2019, 2:43 p.m. UTC
  The port library should be build after eventdev library.

Fixes: 	5d92c4e5 ("port: add eventdev port type")

Signed-off-by: Rahul Shah <rahul.r.shah@intel.com>
---
 lib/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Cristian Dumitrescu Nov. 7, 2019, 3:22 p.m. UTC | #1
> -----Original Message-----
> From: Shah, Rahul R <rahul.r.shah@intel.com>
> Sent: Wednesday, November 6, 2019 2:43 PM
> To: Dumitrescu, Cristian <cristian.dumitrescu@intel.com>
> Cc: dev@dpdk.org
> Subject: [PATCH v2] port: fix library build order dependency
> 
> The port library should be build after eventdev library.
> 
> Fixes: 	5d92c4e5 ("port: add eventdev port type")
> 
> Signed-off-by: Rahul Shah <rahul.r.shah@intel.com>
> ---
>  lib/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/Makefile b/lib/Makefile
> index 5d04ab915..1ff00ba8c 100644
> --- a/lib/Makefile
> +++ b/lib/Makefile
> @@ -89,7 +89,7 @@ DIRS-$(CONFIG_RTE_LIBRTE_DISTRIBUTOR) +=
> librte_distributor
>  DEPDIRS-librte_distributor := librte_eal librte_mbuf librte_ethdev
>  DIRS-$(CONFIG_RTE_LIBRTE_PORT) += librte_port
>  DEPDIRS-librte_port := librte_eal librte_mempool librte_mbuf librte_ethdev
> -DEPDIRS-librte_port += librte_ip_frag librte_sched
> +DEPDIRS-librte_port += librte_ip_frag librte_sched librte_eventdev
>  ifeq ($(CONFIG_RTE_LIBRTE_KNI),y)
>  DEPDIRS-librte_port += librte_kni
>  endif
> --
> 2.20.1

Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
  
David Marchand Nov. 7, 2019, 4:44 p.m. UTC | #2
On Thu, Nov 7, 2019 at 4:23 PM Dumitrescu, Cristian
<cristian.dumitrescu@intel.com> wrote:
> > -----Original Message-----
> > From: Shah, Rahul R <rahul.r.shah@intel.com>
> > Sent: Wednesday, November 6, 2019 2:43 PM
> > To: Dumitrescu, Cristian <cristian.dumitrescu@intel.com>
> > Cc: dev@dpdk.org
> > Subject: [PATCH v2] port: fix library build order dependency
> >
> > The port library should be build after eventdev library.

built*

> >
> > Fixes:        5d92c4e5 ("port: add eventdev port type")

Fixes: 5d92c4e592c4 ("port: add eventdev port type")
Cc: stable@dpdk.org

> >
> > Signed-off-by: Rahul Shah <rahul.r.shah@intel.com>
Tested-by: Ferruh Yigit <ferruh.yigit@intel.com>
> Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>

Applied, thanks.


--
David Marchand
  

Patch

diff --git a/lib/Makefile b/lib/Makefile
index 5d04ab915..1ff00ba8c 100644
--- a/lib/Makefile
+++ b/lib/Makefile
@@ -89,7 +89,7 @@  DIRS-$(CONFIG_RTE_LIBRTE_DISTRIBUTOR) += librte_distributor
 DEPDIRS-librte_distributor := librte_eal librte_mbuf librte_ethdev
 DIRS-$(CONFIG_RTE_LIBRTE_PORT) += librte_port
 DEPDIRS-librte_port := librte_eal librte_mempool librte_mbuf librte_ethdev
-DEPDIRS-librte_port += librte_ip_frag librte_sched
+DEPDIRS-librte_port += librte_ip_frag librte_sched librte_eventdev
 ifeq ($(CONFIG_RTE_LIBRTE_KNI),y)
 DEPDIRS-librte_port += librte_kni
 endif