Re: [dpdk-dev] [PATCH v2] net/hns3: fix compile error with gcc8.3 and thunderx2

[David Marchand <david.marchand@redhat.com>]

On Wed, May 19, 2021 at 2:30 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> On 5/19/2021 12:54 PM, Chengwen Feng wrote:
> > Compile error with gcc8.3 and crossfile arm64_thunderx2_linux_gcc:
> >       ../drivers/net/hns3/hns3_rxtx_vec_sve.c
> >       cc1: error: switch ‘-mcpu=armv8.1-a’ conflicts with
> >       ‘-march=armv8.2-a’ switch [-Werror]
> >       ../drivers/net/hns3/hns3_rxtx_vec_sve.c:5:10: fatal error:
> >       arm_sve.h: No such file or directory
> >           5 | #include <arm_sve.h>
> >
> > The root cause is that gcc8.3 supports SVE, but it doesn't support
> > compile ACLE[1] SVE code, and the hns3_rxtx_vec_sve.c was written
> > by ACLE SVE code.
> >
> > This patch also filters out '-march=' '-mcpu' '-mtune' when compile
> > with hns3_rxtx_vec_sve.c.
> >
> > [1] ACLE: Arm C Language Extensions, user should include <arm_sve.h>
> > when writing ACLE SVE code.
> >
> > Fixes: 203fbaf8813d ("net/hns3: refactor SVE code compile method")
> >
> > Signed-off-by: Chengwen Feng <fengchengwen@huawei.com>
>
> Hi Chengwen,
>
> We need CI result to proceed with the patch, since I don't have arm platfrom to
> test this properly.
>
> But in the CI applying patch fails, I guess that is because the original patch
> has been dropped in the next-net.
>
> Can you please send a new version of the patch rebased on top of latest next-net?

We are almost done for 21.05.
Is this issue a blocker?


-- 
David Marchand