Re: [dpdk-dev] [dpdk-ci] Fedora Rawhide Meson Compile Testing - DPDK Compile Warning

Re: [dpdk-dev] [dpdk-ci] Fedora Rawhide Meson Compile Testing - DPDK Compile Warning

[David Marchand]

On Thu, Oct 1, 2020 at 8:24 PM Brandon Lo <blo@iol.unh.edu> wrote:
> We have successfully added Fedora Rawhide to our production pipeline
> for Meson compile testing.
> The image for the container will be updated on a weekly basis.
>
> The version of GCC that it is currently running (10.2) catches that
> the drivers/vdpa/ifc/base/ifcvf.h file redefines
> VIRTIO_F_IOMMU_PLATFORM, originally from
> /usr/include/linux/virtio_config.h.
> I am just giving you guys a heads-up before the failure report catches
> anyone off guard.

Brandon,

Before putting this new job online, the build issue should have been
fixed on the dpdk side.
All new submitted series are now getting a fail flag that we must
inspect to check whether it is because of this known issue or
something else.

Please, disable this job.

There is also the OpenSuse job failing.
Can you investigate?


Xiao, Maxime,

Anyone free to have a look on the vdpa build issue on Fedora Rawhide?

FAILED: drivers/libtmp_rte_pmd_ifc.a.p/vdpa_ifc_ifcvf_vdpa.c.o
cc -Idrivers/libtmp_rte_pmd_ifc.a.p -Idrivers -I../drivers
-Idrivers/vdpa/ifc -I../drivers/vdpa/ifc -I../drivers/vdpa/ifc/base
-Idrivers/bus/pci -I../drivers/bus/pci -I../drivers/bus/pci/linux -I.
-I.. -Iconfig -I../config -Ilib/librte_eal/include
-I../lib/librte_eal/include -Ilib/librte_eal/linux/include
-I../lib/librte_eal/linux/include -Ilib/librte_eal/x86/include
-I../lib/librte_eal/x86/include -Ilib/librte_eal/common
-I../lib/librte_eal/common -Ilib/librte_eal -I../lib/librte_eal
-Ilib/librte_kvargs -I../lib/librte_kvargs -Ilib/librte_metrics
-I../lib/librte_metrics -Ilib/librte_telemetry
-I../lib/librte_telemetry -Ilib/librte_pci -I../lib/librte_pci
-Ilib/librte_vhost -I../lib/librte_vhost -Ilib/librte_ethdev
-I../lib/librte_ethdev -Ilib/librte_net -I../lib/librte_net
-Ilib/librte_mbuf -I../lib/librte_mbuf -Ilib/librte_mempool
-I../lib/librte_mempool -Ilib/librte_ring -I../lib/librte_ring
-Ilib/librte_meter -I../lib/librte_meter -Ilib/librte_cryptodev
-I../lib/librte_cryptodev -Ilib/librte_hash -I../lib/librte_hash
-fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall
-Winvalid-pch -Werror -O3 -include rte_config.h -Wextra -Wcast-qual
-Wdeprecated -Wformat-nonliteral -Wformat-security
-Wmissing-declarations -Wmissing-prototypes -Wnested-externs
-Wold-style-definition -Wpointer-arith -Wsign-compare
-Wstrict-prototypes -Wundef -Wwrite-strings
-Wno-address-of-packed-member -Wno-packed-not-aligned
-Wno-missing-field-initializers -Wno-zero-length-bounds -D_GNU_SOURCE
-fPIC -march=native -DALLOW_EXPERIMENTAL_API -DALLOW_INTERNAL_API
-Wno-format-truncation -MD -MQ
drivers/libtmp_rte_pmd_ifc.a.p/vdpa_ifc_ifcvf_vdpa.c.o -MF
drivers/libtmp_rte_pmd_ifc.a.p/vdpa_ifc_ifcvf_vdpa.c.o.d -o
drivers/libtmp_rte_pmd_ifc.a.p/vdpa_ifc_ifcvf_vdpa.c.o -c
../drivers/vdpa/ifc/ifcvf_vdpa.c
In file included from ../drivers/vdpa/ifc/ifcvf_vdpa.c:26:
../drivers/vdpa/ifc/base/ifcvf.h:16: error: "VIRTIO_F_IOMMU_PLATFORM"
redefined [-Werror]
   16 | #define VIRTIO_F_IOMMU_PLATFORM  33
      |
In file included from /usr/include/linux/virtio_net.h:30,
                 from ../drivers/vdpa/ifc/ifcvf_vdpa.c:11:
/usr/include/linux/virtio_config.h:78: note: this is the location of
the previous definition
   78 | #define VIRTIO_F_IOMMU_PLATFORM  VIRTIO_F_ACCESS_PLATFORM
      |
cc1: all warnings being treated as errors


Thanks guys.


-- 
David Marchand

Re: [dpdk-dev] [dpdk-ci] Fedora Rawhide Meson Compile Testing - DPDK Compile Warning

[David Marchand]

On Fri, Oct 2, 2020 at 8:51 AM David Marchand <david.marchand@redhat.com> wrote:
>
> On Thu, Oct 1, 2020 at 8:24 PM Brandon Lo <blo@iol.unh.edu> wrote:
> > We have successfully added Fedora Rawhide to our production pipeline
> > for Meson compile testing.
> > The image for the container will be updated on a weekly basis.
> >
> > The version of GCC that it is currently running (10.2) catches that
> > the drivers/vdpa/ifc/base/ifcvf.h file redefines
> > VIRTIO_F_IOMMU_PLATFORM, originally from
> > /usr/include/linux/virtio_config.h.
> > I am just giving you guys a heads-up before the failure report catches
> > anyone off guard.
>
> Brandon,
>
> Before putting this new job online, the build issue should have been
> fixed on the dpdk side.
> All new submitted series are now getting a fail flag that we must
> inspect to check whether it is because of this known issue or
> something else.
>
> Please, disable this job.

The vdpa/ifc issue should be fixed in the main branch now (thanks to Maxime).
But next-net and other subtrees will still have the issue until they
catch on this fix.


>
> There is also the OpenSuse job failing.
> Can you investigate?

Still failing.


-- 
David Marchand