Re: [dpdk-dev] [PATCH v4 10/17] net/axgbe: add transmit and receive data path apis

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 4/5/2018 7:39 AM, Ravi Kumar wrote:
> Supported scalar implementation for RX data path
> Supported scalar and vector implementation for TX data path
> 
> Signed-off-by: Ravi Kumar <Ravi1.kumar@amd.com>
> ---
>  drivers/net/axgbe/Makefile             |   1 +
>  drivers/net/axgbe/axgbe_ethdev.c       |  22 +-
>  drivers/net/axgbe/axgbe_rxtx.c         | 429 +++++++++++++++++++++++++++++++++
>  drivers/net/axgbe/axgbe_rxtx.h         |  19 ++
>  drivers/net/axgbe/axgbe_rxtx_vec_sse.c |  93 +++++++
>  5 files changed, 563 insertions(+), 1 deletion(-)
>  create mode 100644 drivers/net/axgbe/axgbe_rxtx_vec_sse.c
> 
> diff --git a/drivers/net/axgbe/Makefile b/drivers/net/axgbe/Makefile
> index 9fd7b5e..aff7917 100644
> --- a/drivers/net/axgbe/Makefile
> +++ b/drivers/net/axgbe/Makefile
> @@ -24,5 +24,6 @@ SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_mdio.c
>  SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_phy_impl.c
>  SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_i2c.c
>  SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_rxtx.c
> +SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_rxtx_vec_sse.c

This needs to be protected with x86 checks. PMD is enabled by default in config,
which means it will be enabled for other architectures too, like arm and ibm,
and this file will cause build error for them.