* [PATCH] ci: switch to Fedora 37
@ 2023-04-21 14:34 David Marchand
2023-04-21 21:06 ` Aaron Conole
0 siblings, 1 reply; 6+ messages in thread
From: David Marchand @ 2023-04-21 14:34 UTC (permalink / raw)
To: dev; +Cc: Aaron Conole, Michael Santana
Fedora 35 has been declared EOL in 2022/12 (see [1]).
Fedora 36 will soon be EOL too.
Move to Fedora 37.
Fedora 37 libbpf does not support AF_XDP anymore, now provided by
libxdp.
1: https://docs.fedoraproject.org/en-US/releases/eol/
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
.github/workflows/build.yml | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index edd39cbd62..7322eb59b7 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -149,7 +149,7 @@ jobs:
fail-fast: false
matrix:
config:
- - image: fedora:35
+ - image: fedora:37
steps:
- name: Generate various keys
@@ -198,9 +198,9 @@ jobs:
fail-fast: false
matrix:
config:
- - image: fedora:35
+ - image: fedora:37
compiler: gcc
- - image: fedora:35
+ - image: fedora:37
compiler: clang
steps:
@@ -250,7 +250,7 @@ jobs:
run: docker exec -i dpdk dnf update -y || true
- name: Install packages
run: docker exec -i dpdk dnf install -y ccache jansson-devel
- libarchive-devel libatomic libbsd-devel libbpf-devel libfdt-devel
+ libarchive-devel libatomic libbsd-devel libxdp-devel libfdt-devel
libpcap-devel ninja-build numactl-devel openssl-devel python3-pip
python3-pyelftools python3-setuptools python3-wheel rdma-core-devel
zlib-devel
--
2.40.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ci: switch to Fedora 37
2023-04-21 14:34 [PATCH] ci: switch to Fedora 37 David Marchand
@ 2023-04-21 21:06 ` Aaron Conole
2023-04-25 13:13 ` David Marchand
0 siblings, 1 reply; 6+ messages in thread
From: Aaron Conole @ 2023-04-21 21:06 UTC (permalink / raw)
To: David Marchand; +Cc: dev, Michael Santana
David Marchand <david.marchand@redhat.com> writes:
> Fedora 35 has been declared EOL in 2022/12 (see [1]).
> Fedora 36 will soon be EOL too.
>
> Move to Fedora 37.
> Fedora 37 libbpf does not support AF_XDP anymore, now provided by
> libxdp.
>
> 1: https://docs.fedoraproject.org/en-US/releases/eol/
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
Acked-by: Aaron Conole <aconole@redhat.com>
FYI, Fedora 38 also just got released. Perhaps that can be a candidate
as well, but I didn't try it out.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ci: switch to Fedora 37
2023-04-21 21:06 ` Aaron Conole
@ 2023-04-25 13:13 ` David Marchand
2023-04-25 23:20 ` Stephen Hemminger
2023-05-23 9:49 ` Thomas Monjalon
0 siblings, 2 replies; 6+ messages in thread
From: David Marchand @ 2023-04-25 13:13 UTC (permalink / raw)
To: Aaron Conole; +Cc: dev, Michael Santana
Hello Aaron,
On Fri, Apr 21, 2023 at 11:06 PM Aaron Conole <aconole@redhat.com> wrote:
> David Marchand <david.marchand@redhat.com> writes:
> > Fedora 35 has been declared EOL in 2022/12 (see [1]).
> > Fedora 36 will soon be EOL too.
> >
> > Move to Fedora 37.
> > Fedora 37 libbpf does not support AF_XDP anymore, now provided by
> > libxdp.
> >
> > 1: https://docs.fedoraproject.org/en-US/releases/eol/
> >
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > ---
>
> Acked-by: Aaron Conole <aconole@redhat.com>
>
> FYI, Fedora 38 also just got released. Perhaps that can be a candidate
> as well, but I didn't try it out.
At a first glance, gcc 13 raises some new warnings, at least for
examples (ip-pipeline and ntb).
We can switch to f38 once builds are fine with gcc 13.
--
David Marchand
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ci: switch to Fedora 37
2023-04-25 13:13 ` David Marchand
@ 2023-04-25 23:20 ` Stephen Hemminger
2023-04-26 5:45 ` David Marchand
2023-05-23 9:49 ` Thomas Monjalon
1 sibling, 1 reply; 6+ messages in thread
From: Stephen Hemminger @ 2023-04-25 23:20 UTC (permalink / raw)
To: David Marchand; +Cc: Aaron Conole, dev, Michael Santana
On Tue, 25 Apr 2023 15:13:50 +0200
David Marchand <david.marchand@redhat.com> wrote:
> Hello Aaron,
>
> On Fri, Apr 21, 2023 at 11:06 PM Aaron Conole <aconole@redhat.com> wrote:
> > David Marchand <david.marchand@redhat.com> writes:
> > > Fedora 35 has been declared EOL in 2022/12 (see [1]).
> > > Fedora 36 will soon be EOL too.
> > >
> > > Move to Fedora 37.
> > > Fedora 37 libbpf does not support AF_XDP anymore, now provided by
> > > libxdp.
> > >
> > > 1: https://docs.fedoraproject.org/en-US/releases/eol/
> > >
> > > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > > ---
> >
> > Acked-by: Aaron Conole <aconole@redhat.com>
> >
> > FYI, Fedora 38 also just got released. Perhaps that can be a candidate
> > as well, but I didn't try it out.
>
> At a first glance, gcc 13 raises some new warnings, at least for
> examples (ip-pipeline and ntb).
> We can switch to f38 once builds are fine with gcc 13.
What errors, just tried and current main branch builds clean on new F38 VM.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ci: switch to Fedora 37
2023-04-25 23:20 ` Stephen Hemminger
@ 2023-04-26 5:45 ` David Marchand
0 siblings, 0 replies; 6+ messages in thread
From: David Marchand @ 2023-04-26 5:45 UTC (permalink / raw)
To: Stephen Hemminger; +Cc: Aaron Conole, dev, Michael Santana
On Wed, Apr 26, 2023 at 1:20 AM Stephen Hemminger
<stephen@networkplumber.org> wrote:
> > > FYI, Fedora 38 also just got released. Perhaps that can be a candidate
> > > as well, but I didn't try it out.
> >
> > At a first glance, gcc 13 raises some new warnings, at least for
> > examples (ip-pipeline and ntb).
> > We can switch to f38 once builds are fine with gcc 13.
>
> What errors, just tried and current main branch builds clean on new F38 VM.
Fixes welcome:
[root@7e1b3663cd03 dpdk]# cat /etc/redhat-release
Fedora release 38 (Thirty Eight)
[root@7e1b3663cd03 dpdk]# rpm -q gcc
gcc-13.0.1-0.12.fc38.x86_64
[root@7e1b3663cd03 dpdk]# ninja -C build -vvvv
ninja: Entering directory `build'
[1/4] cc -Iexamples/dpdk-ip_pipeline.p -Iexamples -I../examples
-Iexamples/ip_pipeline -I../examples/ip_pipeline -I../examples/common
-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/metrics -I../lib/metrics -Ilib/telemetry
-I../lib/telemetry -Ilib/mempool -I../lib/mempool -Ilib/ring
-I../lib/ring -Ilib/net -I../lib/net -Ilib/mbuf -I../lib/mbuf
-Ilib/ethdev -I../lib/ethdev -Ilib/meter -I../lib/meter -Ilib/cmdline
-I../lib/cmdline -Ilib/pipeline -I../lib/pipeline -Ilib/port
-I../lib/port -Ilib/sched -I../lib/sched -Ilib/ip_frag
-I../lib/ip_frag -Ilib/hash -I../lib/hash -Ilib/rcu -I../lib/rcu
-Ilib/cryptodev -I../lib/cryptodev -Ilib/eventdev -I../lib/eventdev
-Ilib/timer -I../lib/timer -Ilib/table -I../lib/table -Ilib/lpm
-I../lib/lpm -Ilib/acl -I../lib/acl -Ilib/ipsec -I../lib/ipsec
-Ilib/security -I../lib/security -Idrivers/bus/pci
-I../drivers/bus/pci -I../drivers/bus/pci/linux -Ilib/pci -I../lib/pci
-fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch
-Wextra -O3 -include rte_config.h -Wcast-qual -Wdeprecated -Wformat
-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
-march=native -Wno-format-truncation -DALLOW_EXPERIMENTAL_API -MD -MQ
examples/dpdk-ip_pipeline.p/ip_pipeline_thread.c.o -MF
examples/dpdk-ip_pipeline.p/ip_pipeline_thread.c.o.d -o
examples/dpdk-ip_pipeline.p/ip_pipeline_thread.c.o -c
../examples/ip_pipeline/thread.c
In function ‘thread_msg_handle’,
inlined from ‘thread_main’ at ../examples/ip_pipeline/thread.c:3130:6:
../examples/ip_pipeline/thread.c:535:20: warning: ‘req’ may be used
uninitialized [-Wmaybe-uninitialized]
535 | if (req == NULL)
| ^
../examples/ip_pipeline/thread.c: In function ‘thread_main’:
../examples/ip_pipeline/thread.c:433:32: note: ‘req’ was declared here
433 | struct thread_msg_req *req;
| ^~~
[2/4] cc -Iexamples/dpdk-ntb.p -Iexamples -I../examples -Iexamples/ntb
-I../examples/ntb -I../examples/common -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/metrics
-I../lib/metrics -Ilib/telemetry -I../lib/telemetry -Ilib/mempool
-I../lib/mempool -Ilib/ring -I../lib/ring -Ilib/net -I../lib/net
-Ilib/mbuf -I../lib/mbuf -Ilib/ethdev -I../lib/ethdev -Ilib/meter
-I../lib/meter -Ilib/cmdline -I../lib/cmdline -Ilib/rawdev
-I../lib/rawdev -Idrivers/raw/ntb -I../drivers/raw/ntb -Ilib/pci
-I../lib/pci -Idrivers/bus/pci -I../drivers/bus/pci
-I../drivers/bus/pci/linux -fdiagnostics-color=always -Wall
-Winvalid-pch -Wextra -O3 -include rte_config.h -Wcast-qual
-Wdeprecated -Wformat -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
-march=native -Wno-format-truncation -D_FILE_OFFSET_BITS=64
-DALLOW_EXPERIMENTAL_API -MD -MQ examples/dpdk-ntb.p/ntb_ntb_fwd.c.o
-MF examples/dpdk-ntb.p/ntb_ntb_fwd.c.o.d -o
examples/dpdk-ntb.p/ntb_ntb_fwd.c.o -c ../examples/ntb/ntb_fwd.c
../examples/ntb/ntb_fwd.c: In function ‘ntb_stats_display’:
../examples/ntb/ntb_fwd.c:945:23: warning: ‘rte_rawdev_xstats_get’
accessing 8 bytes in a region of size 0 [-Wstringop-overflow=]
945 | if (nb_ids != rte_rawdev_xstats_get(dev_id, ids,
values, nb_ids)) {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../examples/ntb/ntb_fwd.c:945:23: note: referencing argument 3 of type
‘uint64_t[0]’ {aka ‘long unsigned int[]’}
In file included from ../examples/ntb/ntb_fwd.c:17:
../lib/rawdev/rte_rawdev.h:504:1: note: in a call to function
‘rte_rawdev_xstats_get’
504 | rte_rawdev_xstats_get(uint16_t dev_id,
| ^~~~~~~~~~~~~~~~~~~~~
[3/4] cc -o examples/dpdk-ip_pipeline
examples/dpdk-ip_pipeline.p/ip_pipeline_action.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_cli.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_conn.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_cryptodev.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_kni.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_link.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_main.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_mempool.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_parser.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_pipeline.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_swq.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_tap.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_thread.c.o
examples/dpdk-ip_pipeline.p/ip_pipeline_tmgr.c.o -Wl,--as-needed
-Wl,--no-undefined -Wl,-O1 -Wl,--whole-archive -Wl,--start-group
lib/librte_node.a lib/librte_graph.a lib/librte_pipeline.a
lib/librte_table.a lib/librte_pdump.a lib/librte_port.a
lib/librte_fib.a lib/librte_ipsec.a lib/librte_vhost.a
lib/librte_stack.a lib/librte_security.a lib/librte_sched.a
lib/librte_reorder.a lib/librte_rib.a lib/librte_dmadev.a
lib/librte_mldev.a lib/librte_regexdev.a lib/librte_rawdev.a
lib/librte_power.a lib/librte_pcapng.a lib/librte_member.a
lib/librte_lpm.a lib/librte_latencystats.a lib/librte_jobstats.a
lib/librte_ip_frag.a lib/librte_gso.a lib/librte_gro.a
lib/librte_gpudev.a lib/librte_eventdev.a lib/librte_efd.a
lib/librte_distributor.a lib/librte_cryptodev.a
lib/librte_compressdev.a lib/librte_cfgfile.a lib/librte_bpf.a
lib/librte_bitratestats.a lib/librte_bbdev.a lib/librte_acl.a
lib/librte_timer.a lib/librte_hash.a lib/librte_metrics.a
lib/librte_cmdline.a lib/librte_pci.a lib/librte_ethdev.a
lib/librte_meter.a lib/librte_net.a lib/librte_mbuf.a
lib/librte_mempool.a lib/librte_rcu.a lib/librte_ring.a
lib/librte_eal.a lib/librte_telemetry.a lib/librte_kvargs.a
drivers/librte_bus_pci.a drivers/librte_bus_vdev.a
drivers/librte_mempool_ring.a drivers/librte_net_bnxt.a
drivers/librte_net_i40e.a drivers/librte_net_ixgbe.a
drivers/librte_raw_ntb.a -Wl,--no-whole-archive -Wl,--no-as-needed
-pthread -lm -ldl -lnuma -Wl,--end-group -Wl,--export-dynamic
[4/4] cc -o examples/dpdk-ntb examples/dpdk-ntb.p/ntb_ntb_fwd.c.o
-Wl,--as-needed -Wl,--no-undefined -Wl,-O1 -Wl,--whole-archive
-Wl,--start-group lib/librte_node.a lib/librte_graph.a
lib/librte_pipeline.a lib/librte_table.a lib/librte_pdump.a
lib/librte_port.a lib/librte_fib.a lib/librte_ipsec.a
lib/librte_vhost.a lib/librte_stack.a lib/librte_security.a
lib/librte_sched.a lib/librte_reorder.a lib/librte_rib.a
lib/librte_dmadev.a lib/librte_mldev.a lib/librte_regexdev.a
lib/librte_rawdev.a lib/librte_power.a lib/librte_pcapng.a
lib/librte_member.a lib/librte_lpm.a lib/librte_latencystats.a
lib/librte_jobstats.a lib/librte_ip_frag.a lib/librte_gso.a
lib/librte_gro.a lib/librte_gpudev.a lib/librte_eventdev.a
lib/librte_efd.a lib/librte_distributor.a lib/librte_cryptodev.a
lib/librte_compressdev.a lib/librte_cfgfile.a lib/librte_bpf.a
lib/librte_bitratestats.a lib/librte_bbdev.a lib/librte_acl.a
lib/librte_timer.a lib/librte_hash.a lib/librte_metrics.a
lib/librte_cmdline.a lib/librte_pci.a lib/librte_ethdev.a
lib/librte_meter.a lib/librte_net.a lib/librte_mbuf.a
lib/librte_mempool.a lib/librte_rcu.a lib/librte_ring.a
lib/librte_eal.a lib/librte_telemetry.a lib/librte_kvargs.a
drivers/librte_bus_pci.a drivers/librte_bus_vdev.a
drivers/librte_mempool_ring.a drivers/librte_net_bnxt.a
drivers/librte_net_i40e.a drivers/librte_net_ixgbe.a
drivers/librte_raw_ntb.a -Wl,--no-whole-archive -Wl,--no-as-needed
-pthread -lm -ldl -lnuma -Wl,--end-group -Wl,--export-dynamic
--
David Marchand
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ci: switch to Fedora 37
2023-04-25 13:13 ` David Marchand
2023-04-25 23:20 ` Stephen Hemminger
@ 2023-05-23 9:49 ` Thomas Monjalon
1 sibling, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2023-05-23 9:49 UTC (permalink / raw)
To: David Marchand; +Cc: Aaron Conole, dev, Michael Santana
25/04/2023 15:13, David Marchand:
> Hello Aaron,
>
> On Fri, Apr 21, 2023 at 11:06 PM Aaron Conole <aconole@redhat.com> wrote:
> > David Marchand <david.marchand@redhat.com> writes:
> > > Fedora 35 has been declared EOL in 2022/12 (see [1]).
> > > Fedora 36 will soon be EOL too.
> > >
> > > Move to Fedora 37.
> > > Fedora 37 libbpf does not support AF_XDP anymore, now provided by
> > > libxdp.
> > >
> > > 1: https://docs.fedoraproject.org/en-US/releases/eol/
> > >
> > > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > > ---
> >
> > Acked-by: Aaron Conole <aconole@redhat.com>
> >
> > FYI, Fedora 38 also just got released. Perhaps that can be a candidate
> > as well, but I didn't try it out.
>
> At a first glance, gcc 13 raises some new warnings, at least for
> examples (ip-pipeline and ntb).
> We can switch to f38 once builds are fine with gcc 13.
Let's switch to Fedora 37 as a first step (we skipped Fedora 36).
Applied, thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-05-23 9:49 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-21 14:34 [PATCH] ci: switch to Fedora 37 David Marchand
2023-04-21 21:06 ` Aaron Conole
2023-04-25 13:13 ` David Marchand
2023-04-25 23:20 ` Stephen Hemminger
2023-04-26 5:45 ` David Marchand
2023-05-23 9:49 ` Thomas Monjalon
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).