From: Joshua Washington <joshwash@google.com>
To: David Marchand <david.marchand@redhat.com>
Cc: Jeroen de Borst <jeroendb@google.com>,
Rushil Gupta <rushilg@google.com>, dev <dev@dpdk.org>
Subject: Re: Build issue with Fedora Rawhide
Date: Thu, 17 Oct 2024 15:59:51 -0700 [thread overview]
Message-ID: <CALuQH+VYihPOXbhhyoEwq1EpmP5f5S86X7xRWGR8oKYLRYvuRA@mail.gmail.com> (raw)
In-Reply-To: <CAJFAV8xcAEGSfgepSD+_EY2Bv+7hYnG4bGhRzpoey2j9raDZDw@mail.gmail.com>
Hello, I have a fix that changes all typedefs in gve_osdep.h to
#defines. It should be going out shortly.
On Thu, Oct 17, 2024 at 5:12 AM David Marchand
<david.marchand@redhat.com> wrote:
>
> Hello guys,
>
> I am not clear if this issue is new (and what caused it), but
> compilation fails on Rawhide for the net/gve driver (see below for the
> log).
>
> Afaics, Intel drivers wrapped __le16 types (and friends) using macros
> (example: https://git.dpdk.org/dpdk/tree/drivers/net/i40e/base/i40e_osdep.h#n47),
> and I suspect it was to avoid such conflicts.
>
> Can you send a fix please?
>
> [ 224s] FAILED: drivers/libtmp_rte_net_gve.a.p/net_gve_base_gve_adminq.c.o
> [ 224s] cc -Idrivers/libtmp_rte_net_gve.a.p -Idrivers -I../drivers
> -Idrivers/net/gve -I../drivers/net/gve -I../drivers/net/gve/base
> -Ilib/ethdev -I../lib/ethdev -I. -I.. -Iconfig -I../config
> -Ilib/eal/include -I../lib/eal/include -Ilib/eal/linux/include
> -I../lib/eal/linux/include -Ilib/eal/x86/include
> -I../lib/eal/x86/include -Ilib/eal/common -I../lib/eal/common
> -Ilib/eal -I../lib/eal -Ilib/kvargs -I../lib/kvargs -Ilib/log
> -I../lib/log -Ilib/metrics -I../lib/metrics -Ilib/telemetry
> -I../lib/telemetry -Ilib/net -I../lib/net -Ilib/mbuf -I../lib/mbuf
> -Ilib/mempool -I../lib/mempool -Ilib/ring -I../lib/ring -Ilib/meter
> -I../lib/meter -Idrivers/bus/pci -I../drivers/bus/pci
> -I../drivers/bus/pci/linux -Ilib/pci -I../lib/pci -Idrivers/bus/vdev
> -I../drivers/bus/vdev -fdiagnostics-color=always
> -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=c11 -O3
> -include rte_config.h -Wcast-qual -Wdeprecated -Wformat
> -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
> -fcommon -Werror -fPIC -march=corei7 -mrtm -DALLOW_EXPERIMENTAL_API
> -DALLOW_INTERNAL_API -Wno-format-truncation
> -DRTE_LOG_DEFAULT_LOGTYPE=pmd.net.gve -MD -MQ
> drivers/libtmp_rte_net_gve.a.p/net_gve_base_gve_adminq.c.o -MF
> drivers/libtmp_rte_net_gve.a.p/net_gve_base_gve_adminq.c.o.d -o
> drivers/libtmp_rte_net_gve.a.p/net_gve_base_gve_adminq.c.o -c
> ../drivers/net/gve/base/gve_adminq.c
> [ 224s] In file included from ../drivers/net/gve/base/../base/gve_desc.h:11,
> [ 224s] from ../drivers/net/gve/base/../base/gve.h:9,
> [ 224s] from ../drivers/net/gve/base/../gve_ethdev.h:13,
> [ 224s] from ../drivers/net/gve/base/gve_adminq.c:6:
> [ 224s] ../drivers/net/gve/base/../base/gve_osdep.h:41:20: error:
> conflicting types for ‘__be64’; have ‘rte_be64_t’ {aka ‘long
> unsigned int’}
> [ 224s] 41 | typedef rte_be64_t __be64;
> [ 224s] | ^~~~~~
> [ 224s] In file included from /usr/include/linux/sched/types.h:5,
> [ 224s] from /usr/include/bits/sched.h:60,
> [ 224s] from /usr/include/sched.h:43,
> [ 224s] from /usr/include/pthread.h:22,
> [ 224s] from ../lib/ethdev/ethdev_driver.h:17,
> [ 224s] from ../drivers/net/gve/base/../gve_ethdev.h:8:
> [ 224s] /usr/include/linux/types.h:36:25: note: previous declaration
> of ‘__be64’ with type ‘__be64’ {aka ‘long long unsigned
> int’}
> [ 224s] 36 | typedef __u64 __bitwise __be64;
> [ 224s] | ^~~~~~
> [ 224s] ../drivers/net/gve/base/../base/gve_osdep.h:45:20: error:
> conflicting types for ‘__le64’; have ‘rte_le64_t’ {aka ‘long
> unsigned int’}
> [ 224s] 45 | typedef rte_le64_t __le64;
> [ 224s] | ^~~~~~
> [ 224s] /usr/include/linux/types.h:35:25: note: previous declaration
> of ‘__le64’ with type ‘__le64’ {aka ‘long long unsigned
> int’}
> [ 224s] 35 | typedef __u64 __bitwise __le64;
> [ 224s] | ^~~~~~
>
>
>
> --
> David Marchand
>
--
Joshua Washington | Software Engineer | joshwash@google.com | (414) 366-4423
prev parent reply other threads:[~2024-10-17 23:00 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-17 12:12 David Marchand
2024-10-17 22:59 ` Joshua Washington [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CALuQH+VYihPOXbhhyoEwq1EpmP5f5S86X7xRWGR8oKYLRYvuRA@mail.gmail.com \
--to=joshwash@google.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=jeroendb@google.com \
--cc=rushilg@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).