[Bug 881] [dpdk-21.11] drivers/libtmp_rte_bus_fslmc.a.p build failed on OS Fedora35 with clang 13.0.0

[Bug 881] [dpdk-21.11] drivers/libtmp_rte_bus_fslmc.a.p build failed on OS Fedora35 with clang 13.0.0

[bugzilla]

https://bugs.dpdk.org/show_bug.cgi?id=881

            Bug ID: 881
           Summary: [dpdk-21.11] drivers/libtmp_rte_bus_fslmc.a.p build
                    failed on OS Fedora35 with clang 13.0.0
           Product: DPDK
           Version: 21.11
          Hardware: All
                OS: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: meson
          Assignee: dev@dpdk.org
          Reporter: longfengx.liang@intel.com
  Target Milestone: ---

1.DPDK version:
commit 19d024003dfce0018c9e52635b78efdc6e1c172a (HEAD -> main, origin/main,
origin/HEAD)
Author: David Marchand <david.marchand@redhat.com>
Date:   Mon Nov 8 11:08:08 2021 +0100

    build: factorize jansson availability check

    Since two components wants to know if the jansson library is available,
    move it to config/.

    Signed-off-by: David Marchand <david.marchand@redhat.com>

2.OS version:
  OS: Fedora35 /kernel 5.14.16-301.fc35.x86_64
    gcc version: gcc (GCC) 11.2.1
    clang version: clang version 13.0.0 (Fedora 13.0.0~rc1-1.fc35)
    ninja version: 1.10.2.git.kitware.jobserver-1
    meson version: 0.59.4

3.build cmd:
  #CC=clang meson --werror -Denable_kmods=True -Dlibdir=lib -Dexamples=all
--default-library=static x86_64-native-linuxapp-clang
  #ninja -C x86_64-native-linuxapp-clang/

4.Test Setup:

1)First,the compile error on Fedora35 is "lib/librte_power.a.p",details as
below:

ninja: Entering directory `x86_64-native-linuxapp-clang/'
[387/3129] Compiling C object
lib/librte_power.a.p/power_rte_power_empty_poll.c.o
FAILED: lib/librte_power.a.p/power_rte_power_empty_poll.c.o
clang -Ilib/librte_power.a.p -Ilib -I../lib -Ilib/power -I../lib/power -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/timer -I../lib/timer -Ilib/ethdev
-I../lib/ethdev -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
-fcolor-diagnostics -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Werror -O3
-include rte_config.h -Wextra -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-missing-field-initializers -D_GNU_SOURCE
-fPIC -march=native -DALLOW_EXPERIMENTAL_API -DALLOW_INTERNAL_API
-Wno-cast-qual -DRTE_LOG_DEFAULT_LOGTYPE=lib.power -MD -MQ
lib/librte_power.a.p/power_rte_power_empty_poll.c.o -MF
lib/librte_power.a.p/power_rte_power_empty_poll.c.o.d -o
lib/librte_power.a.p/power_rte_power_empty_poll.c.o -c
../lib/power/rte_power_empty_poll.c
../lib/power/rte_power_empty_poll.c:210:25: error: variable 'tot_ppi' set but
not used [-Werror,-Wunused-but-set-variable]
        uint64_t tot_edpi = 0, tot_ppi = 0;
                               ^
1 error generated.
[392/3129] Compiling C object lib/librte_power.a.p/power_rte_power_pmd_mgmt.c.o
ninja: build stopped: subcommand failed.

2) Then the devlopment provides the fixed patch:
http://patches.dpdk.org/project/dpdk/patch/20211105155351.350403-1-james.r.harris@intel.com/

3)When apply this patches,report the new error
"drivers/libtmp_rte_bus_fslmc.a.p",details as below:

ninja: Entering directory `x86_64-native-linuxapp-clang/'
[685/3129] Compiling C object
drivers/libtmp_rte_bus_fslmc.a.p/bus_fslmc_fslmc_bus.c.o
FAILED: drivers/libtmp_rte_bus_fslmc.a.p/bus_fslmc_fslmc_bus.c.o
clang -Idrivers/libtmp_rte_bus_fslmc.a.p -Idrivers -I../drivers
-Idrivers/bus/fslmc -I../drivers/bus/fslmc -I../drivers/bus/fslmc/mc
-I../drivers/bus/fslmc/qbman/include -I../drivers/bus/fslmc/portal -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 -Idrivers/common/dpaax
-I../drivers/common/dpaax -I../drivers/common/dpaax/caamflib -Ilib/eventdev
-I../lib/eventdev -Ilib/ring -I../lib/ring -Ilib/ethdev -I../lib/ethdev
-Ilib/net -I../lib/net -Ilib/mbuf -I../lib/mbuf -Ilib/mempool -I../lib/mempool
-Ilib/meter -I../lib/meter -Ilib/hash -I../lib/hash -Ilib/rcu -I../lib/rcu
-Ilib/timer -I../lib/timer -Ilib/cryptodev -I../lib/cryptodev
-fcolor-diagnostics -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Werror -O3
-include rte_config.h -Wextra -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-missing-field-initializers -D_GNU_SOURCE
-fPIC -march=native -DALLOW_EXPERIMENTAL_API -DALLOW_INTERNAL_API
-DRTE_LOG_DEFAULT_LOGTYPE=bus.fslmc -MD -MQ
drivers/libtmp_rte_bus_fslmc.a.p/bus_fslmc_fslmc_bus.c.o -MF
drivers/libtmp_rte_bus_fslmc.a.p/bus_fslmc_fslmc_bus.c.o.d -o
drivers/libtmp_rte_bus_fslmc.a.p/bus_fslmc_fslmc_bus.c.o -c
../drivers/bus/fslmc/fslmc_bus.c
../drivers/bus/fslmc/fslmc_bus.c:318:6: error: variable 'device_count' set but
not used [-Werror,-Wunused-but-set-variable]
        int device_count = 0;
            ^
1 error generated.
[690/3129] Generating pipeline.sym_chk with a custom command (wrapped by meson
to capture output)
ninja: build stopped: subcommand failed.


Please take a look,thanks a lot.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 881] [dpdk-21.11] build failure on Fedora35 with clang 13.0.0

[bugzilla]

https://bugs.dpdk.org/show_bug.cgi?id=881

David Marchand (david.marchand@redhat.com) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from David Marchand (david.marchand@redhat.com) ---
Fixes provided by Conor have been merged:
8b46c37be5 - common/qat: remove unused time counter (34 minutes ago) <Conor
Walsh>
b5dedf261c - net/hinic/base: remove some unused variables (34 minutes ago)
<Conor Walsh>
23c97a67e5 - net/octeontx: remove unused packet length (30 minutes ago) <Conor
Walsh>
6dfcdf8fd5 - test/thash: fix build with clang 13 (30 minutes ago) <Conor Walsh>
54d109b564 - bus/fslmc: remove unused device count (30 minutes ago) <Conor
Walsh>
0f66d415c4 - event/sw: remove unused inflight events count (30 minutes ago)
<Conor Walsh>
2d72ec8dab - net/bnxt: remove some unused variables (30 minutes ago) <Conor
Walsh>
52415c6b25 - net/liquidio: remove unused counter (30 minutes ago) <Conor Walsh>
9465a5d5a0 - net/nfp: remove unused message length (30 minutes ago) <Conor
Walsh>
2ccebadab2 - net/qede/base: remove unused message size (30 minutes ago) <Conor
Walsh>
0b62b3c940 - net/txgbe: fix build with clang 13 (30 minutes ago) <Conor Walsh>
e76eb560bc - net/vmxnet3: fix build with clang 13 (30 minutes ago) <Conor
Walsh>
ccbdaaa981 - test/distributor: remove unused counter (30 minutes ago) <Conor
Walsh>
130e99db59 - examples/performance-thread: remove unused hits count (30 minutes
ago) <Conor Walsh>

-- 
You are receiving this mail because:
You are the assignee for the bug.