[DPDK version]
commit 9bd5ebca59c865aead7096358fb733e53a72007b (HEAD, tag: v23.03-rc1,
origin/main, origin/HEAD, main)
Author: Thomas Monjalon <thomas@monjalon.net>
Date:   Mon Feb 20 16:53:44 2023 +0100

    version: 23.03-rc1

    Signed-off-by: Thomas Monjalon <thomas@monjalon.net>

[OS version]
gcc11.3.0
Ubuntu22.04/5.15.0-58-generic

RHEL8.6/4.18.0-372.9.1.el8.x86_64
gcc8.5.0

[Test setup]
export RTE_TARGET=x86_64-native-linuxapp-gcc
CC=gcc meson --optimization=1 --werror -Denable_kmods=True  -Dlibdir=lib 
--default-library=static x86_64-native-linuxapp-gcc
ninja -C x86_64-native-linuxapp-gcc -j 10

[Error]
Found ninja-1.10.0.git.kitware.jobserver-1 at /usr/local/bin/ninja
ninja: Entering directory `x86_64-native-linuxapp-gcc'
[2900/3094] Compiling C object app/dpdk-testpmd.p/test-pmd_cmdline_flow.c.o
FAILED: app/dpdk-testpmd.p/test-pmd_cmdline_flow.c.o
gcc -Iapp/dpdk-testpmd.p -Iapp -I../app -Iapp/test-pmd -I../app/test-pmd
-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/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 -Ilib/cmdline -I../lib/cmdline
-Idrivers/crypto/scheduler -I../drivers/crypto/scheduler -Ilib/cryptodev
-I../lib/cryptodev -Ilib/rcu -I../lib/rcu -Idrivers/bus/vdev
-I../drivers/bus/vdev -Ilib/reorder -I../lib/reorder -Ilib/bitratestats
-I../lib/bitratestats -Ilib/bpf -I../lib/bpf -Ilib/gro -I../lib/gro -Ilib/gso
-I../lib/gso -Ilib/latencystats -I../lib/latencystats -Ilib/pdump
-I../lib/pdump -Ilib/pcapng -I../lib/pcapng -Idrivers/net/bnxt
-I../drivers/net/bnxt -Idrivers/net/bnxt/tf_ulp -I../drivers/net/bnxt/tf_ulp
-Idrivers/net/bnxt/tf_ulp/generic_templates
-I../drivers/net/bnxt/tf_ulp/generic_templates -Idrivers/net/bnxt/tf_core
-I../drivers/net/bnxt/tf_core -Idrivers/net/bnxt/hcapi/cfa
-I../drivers/net/bnxt/hcapi/cfa -Idrivers/bus/pci -I../drivers/bus/pci
-I../drivers/bus/pci/linux -Ilib/pci -I../lib/pci -Idrivers/net/i40e
-I../drivers/net/i40e -Idrivers/net/i40e/base -I../drivers/net/i40e/base
-Ilib/hash -I../lib/hash -Idrivers/net/ixgbe -I../drivers/net/ixgbe
-Idrivers/net/ixgbe/base -I../drivers/net/ixgbe/base -Ilib/security
-I../lib/security -Idrivers/bus/dpaa -I../drivers/bus/dpaa
-I../drivers/bus/dpaa/include -I../drivers/bus/dpaa/base/qbman
-Idrivers/common/dpaax -I../drivers/common/dpaax
-I../drivers/common/dpaax/caamflib -Ilib/eventdev -I../lib/eventdev -Ilib/timer
-I../lib/timer -Idrivers/mempool/dpaa -I../drivers/mempool/dpaa
-Idrivers/net/dpaa -I../drivers/net/dpaa -Idrivers/net/bonding
-I../drivers/net/bonding -Ilib/sched -I../lib/sched -Ilib/ip_frag
-I../lib/ip_frag -Idrivers/net/ice -I../drivers/net/ice -Idrivers/net/ice/base
-I../drivers/net/ice/base -Idrivers/common/iavf -I../drivers/common/iavf
-fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra
-Werror -O1 -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 -DALLOW_EXPERIMENTAL_API -Wno-deprecated-declarations -MD -MQ
app/dpdk-testpmd.p/test-pmd_cmdline_flow.c.o -MF
app/dpdk-testpmd.p/test-pmd_cmdline_flow.c.o.d -o
app/dpdk-testpmd.p/test-pmd_cmdline_flow.c.o -c ../app/test-pmd/cmdline_flow.c
../app/test-pmd/cmdline_flow.c: In function ‘parse_vc_modify_field_level’:
../app/test-pmd/cmdline_flow.c:8678:53: error: ‘fp’ may be used uninitialized
in this function [-Werror=maybe-uninitialized]
 8678 |                         action->dst.flex_handle = fp->flex_handle;
      |                                                   ~~^~~~~~~~~~~~~
cc1: all warnings being treated as errors
[2909/3094] Compiling C object app/dpdk-testpmd.p/test-pmd_noisy_vnf.c.o
ninja: build stopped: subcommand failed.

[Bad commit]
commit c8e25fbf1440a5eb9c022f021354b7598c60732e
Author: Rongwei Liu <rongweil@nvidia.com>
Date:   Mon Feb 6 05:39:05 2023 +0200

    ethdev: add flow flex modify

    Add flex item as modify field destination.
    Add "struct rte_flow_item_flex_handle *flex_handle" into
    "struct rte_flow_action_modify_data" as union with existed
    "level" member. This new member is dedicated for modifying
    flex item.

    Add flex item modify field cmdline support. Now user can use
    testpmd cli to specify which flex item to be modified, either
    source or destination.

    Syntax is as below:
    modify_field op set dst_type flex_item dst_level 0
    dst_offset 16 src_type value src_value 0x123456781020 width 8

    Signed-off-by: Rongwei Liu <rongweil@nvidia.com>
    Acked-by: Ori Kam <orika@nvidia.com>

 app/test-pmd/cmdline_flow.c            | 89 +++++++++++++++++++++++++++++++---
 doc/guides/prog_guide/rte_flow.rst     | 41 +++++++++-------
 doc/guides/rel_notes/release_23_03.rst |  4 ++
 lib/ethdev/rte_flow.h                  |  8 ++-
 4 files changed, 116 insertions(+), 26 deletions(-)