Zephyr SDK 0.15.0 Migration Issues #530

stephanosio · 2022-07-27T15:56:50Z

stephanosio
Jul 27, 2022
Maintainer

Please report any issues you find migrating to the Zephyr SDK 0.15.0 here.

galak · 2022-07-27T15:59:06Z

galak
Jul 27, 2022
Collaborator

A number of networking test/samples show this issue (not specific SDK 0.15.0, but native posix on fedora with gcc-12.x):

/home/galak/git/zephyr/subsys/net/ip/net_if.c:566:32: error: comparison between two arrays [-Werror=array-compare]
  566 |         if (_net_if_list_start == _net_if_list_end) {
      |                                ^~
/home/galak/git/zephyr/subsys/net/ip/net_if.c:566:32: note: use ‘&_net_if_list_start[0] == &_net_if_list_end[0]’ to compare the addresses
cc1: all warnings being treated as errors

2 replies

stephanosio Jul 29, 2022
Maintainer Author

Fixed by zephyrproject-rtos/zephyr@c8933d0

stephanosio Jul 29, 2022
Maintainer Author

zephyrproject-rtos/zephyr#48455

galak · 2022-07-27T16:01:50Z

galak
Jul 27, 2022
Collaborator

net_core.c:

In file included from /home/galak/git/zephyr/include/zephyr/toolchain.h:50,
                 from /home/galak/git/zephyr/include/zephyr/sys/cbprintf.h:13,
                 from /home/galak/git/zephyr/include/zephyr/logging/log_msg.h:11,
                 from /home/galak/git/zephyr/include/zephyr/logging/log_core.h:9,
                 from /home/galak/git/zephyr/include/zephyr/logging/log.h:11,
                 from /home/galak/git/zephyr/subsys/net/ip/net_core.c:14:
/home/galak/git/zephyr/include/zephyr/toolchain/gcc.h: In function ‘net_send_data’:
/home/galak/git/zephyr/include/zephyr/toolchain/gcc.h:127:12: error: ‘*(struct <anonymous> *)((char *)&addr + offsetof(struct in_addr, <U3428>)).__v’ may be used uninitialized [-Werror=maybe-uninitialized]
  127 |         __p->__v;                                                       \
      |            ^~
/home/galak/git/zephyr/subsys/net/ip/net_core.c:281:40: note: ‘addr’ declared here
  281 |                         struct in_addr addr;
      |                                        ^~~~
cc1: all warnings being treated as errors

Seen with test: tests/net/dhcpv4/net.dhcp

2 replies

stephanosio Jul 29, 2022
Maintainer Author

The warning looks very cryptic and makes zero sense to me ... (maybe a GCC bug?)

So far, I have been unable to reproduce this using the GCC 12.1 included in the Zephyr SDK 0.15.0-rc1.

stephanosio Jul 29, 2022
Maintainer Author

I was able to reproduce this on native_posix by installing GCC 12.1 from source locally (I'm still on Ubuntu 20.04 which comes with GCC 9.4).

Will fix these native_posix-specific issues later.

galak · 2022-07-27T16:10:14Z

galak
Jul 27, 2022
Collaborator

tests: tests/net/trickle/net.trickle on native_posix:

zephyr/drivers/net/CMakeFiles/drivers__net.dir/loopback.c.obj -c /home/galak/git/zephyr/drivers/net/loopback.c
In file included from /home/galak/git/zephyr/include/zephyr/toolchain.h:50,
                 from /home/galak/git/zephyr/include/zephyr/sys/cbprintf.h:13,
                 from /home/galak/git/zephyr/include/zephyr/logging/log_msg.h:11,
                 from /home/galak/git/zephyr/include/zephyr/logging/log_core.h:9,
                 from /home/galak/git/zephyr/include/zephyr/logging/log.h:11,
                 from /home/galak/git/zephyr/drivers/net/loopback.c:17:
/home/galak/git/zephyr/include/zephyr/toolchain/gcc.h: In function ‘loopback_send’:
/home/galak/git/zephyr/include/zephyr/toolchain/gcc.h:127:12: error: ‘*(struct <anonymous> *)((char *)&addr + offsetof(struct in_addr, <U31c8>)).__v
’ may be used uninitialized [-Werror=maybe-uninitialized]
  127 |         __p->__v;                                                       \
      |            ^~
/home/galak/git/zephyr/drivers/net/loopback.c:109:32: note: ‘addr’ declared here
  109 |                 struct in_addr addr;
      |                                ^~~~
cc1: all warnings being treated as errors

1 reply

stephanosio Jul 29, 2022
Maintainer Author

same deal as #530 (reply in thread)

galak · 2022-07-27T16:12:41Z

galak
Jul 27, 2022
Collaborator

test: tests/drivers/build_all/sensor/sensors.build.trigger on native_posix:

/home/galak/git/modules/hal/st/sensor/stmemsc/lsm6dso_STdC/driver/lsm6dso_reg.c
In function ‘bytecpy’,
    inlined from ‘lsm6dso_mode_set’ at /home/galak/git/modules/hal/st/sensor/stmemsc/lsm6dso_STdC/driver/lsm6dso_reg.c:12217:5:
/home/galak/git/modules/hal/st/sensor/stmemsc/lsm6dso_STdC/driver/lsm6dso_reg.c:98:13: error: ‘*(uint8_t *)(&ctrl2_ois + offsetof(lsm6dso_ctrl2_ois_t, hp_en_ois))’ may be used uninitialized [-Werror=maybe-uninitialized]
   98 |     *target = *source;
      |     ~~~~~~~~^~~~~~~~~
/home/galak/git/modules/hal/st/sensor/stmemsc/lsm6dso_STdC/driver/lsm6dso_reg.c: In function ‘lsm6dso_mode_set’:
/home/galak/git/modules/hal/st/sensor/stmemsc/lsm6dso_STdC/driver/lsm6dso_reg.c:11788:23: note: ‘*(uint8_t *)(&ctrl2_ois + offsetof(lsm6dso_ctrl2_ois_t, hp_en_ois))’ was declared here
11788 |   lsm6dso_ctrl2_ois_t ctrl2_ois;
      |                       ^~~~~~~~~
cc1: all warnings being treated as errors

4 replies

This comment has been hidden.

Sign in to view

stephanosio Aug 3, 2022
Maintainer Author

also observed while building samples/shields/x_nucleo_iks01a3/standard/sample.shields.x_nucleo_iks01a3.standard for nrf52840dk_nrf52840

stephanosio Aug 4, 2022
Maintainer Author

another false positive.

ctrl2_ois is always initialised when aux_ctx != NULL and it is accessed if and only if aux_ctx != NULL, so ctrl2_ois is never uninitialised when it is accessed.

stephanosio Aug 4, 2022
Maintainer Author

Fixed by zephyrproject-rtos/zephyr#48686

galak · 2022-07-27T16:14:02Z

galak
Jul 27, 2022
Collaborator

test: tests/lib/spsc_pbuf/lib.spsc_pbuf on native_posix:

/home/galak/git/zephyr/subsys/testsuite/ztest/src/ztress.c: In function ‘ztress_init’:
/home/galak/git/zephyr/subsys/testsuite/ztest/src/ztress.c:293:22: error: the comparison will always evaluate as ‘false’ for the address of ‘idle_tid’ will never be NULL [-Werror=address]
  293 |         if (idle_tid == NULL) {
      |                      ^~
/home/galak/git/zephyr/subsys/testsuite/ztest/src/ztress.c:49:16: note: ‘idle_tid’ declared here
   49 | static k_tid_t idle_tid[CONFIG_MP_NUM_CPUS];
      |                ^~~~~~~~
cc1: all warnings being treated as errors

2 replies

stephanosio Jul 29, 2022
Maintainer Author

Fixed by zephyrproject-rtos/zephyr@33cbd84

stephanosio Jul 29, 2022
Maintainer Author

zephyrproject-rtos/zephyr#48456

galak · 2022-07-27T16:14:54Z

galak
Jul 27, 2022
Collaborator

test: tests/kernel/sleep/kernel.common.timing on native_posix:

In file included from /home/galak/git/zephyr/subsys/testsuite/ztest/include/zephyr/ztest.h:61,
                 from /home/galak/git/zephyr/tests/kernel/sleep/src/usleep.c:7:
/home/galak/git/zephyr/subsys/testsuite/ztest/include/zephyr/ztest_assert.h: In function ‘_sleep_test_usleep_wrapper’:
/home/galak/git/zephyr/subsys/testsuite/ztest/include/zephyr/ztest_assert.h:135:29: error: ‘elapsed_ms’ may be used uninitialized [-Werror=maybe-uninitialized]
  135 |                 bool _ret = z_zassert(cond, msg ? ("(" default_msg ")") : (default_msg), __FILE__, \
      |                             ^~~~~~~~~
/home/galak/git/zephyr/tests/kernel/sleep/src/usleep.c:68:17: note: ‘elapsed_ms’ was declared here
   68 |         int64_t elapsed_ms;
      |                 ^~~~~~~~~~

2 replies

stephanosio Jul 29, 2022
Maintainer Author

Fixed by zephyrproject-rtos/zephyr@73dccf6

stephanosio Jul 29, 2022
Maintainer Author

zephyrproject-rtos/zephyr#48457

galak · 2022-07-27T16:16:01Z

galak
Jul 27, 2022
Collaborator

test: tests/lib/notify/libraries.data_structures on native_posix:

/home/galak/git/zephyr/subsys/testsuite/ztest/include/zephyr/ztest_assert.h:135:29: error: ‘res’ may be used uninitialized [-Werror=maybe-uninitialized]
  135 |                 bool _ret = z_zassert(cond, msg ? ("(" default_msg ")") : (default_msg), __FILE__, \
      |                             ^~~~~~~~~
/home/galak/git/zephyr/tests/lib/notify/src/main.c:48:13: note: ‘res’ was declared here
   48 |         int res;
      |             ^~~
cc1: all warnings being treated as errors

2 replies

stephanosio Jul 29, 2022
Maintainer Author

Fixed by zephyrproject-rtos/zephyr@dd36024

stephanosio Jul 29, 2022
Maintainer Author

zephyrproject-rtos/zephyr#48458

galak · 2022-07-27T16:17:09Z

galak
Jul 27, 2022
Collaborator

tests tests/net/context/net.context on native_posix:

In file included from /home/galak/git/zephyr/include/zephyr/toolchain.h:50,
                 from /home/galak/git/zephyr/include/zephyr/sys/cbprintf.h:13,
                 from /home/galak/git/zephyr/include/zephyr/logging/log_msg.h:11,
                 from /home/galak/git/zephyr/include/zephyr/logging/log_core.h:9,
                 from /home/galak/git/zephyr/include/zephyr/logging/log.h:11,
                 from /home/galak/git/zephyr/tests/net/context/src/main.c:9:
/home/galak/git/zephyr/include/zephyr/toolchain/gcc.h: In function ‘tester_send’:
/home/galak/git/zephyr/include/zephyr/toolchain/gcc.h:127:12: error: ‘*(struct <anonymous> *)((char *)&addr + offsetof(struct in_addr, <U2ed8>)).__v
’ may be used uninitialized [-Werror=maybe-uninitialized]
  127 |         __p->__v;                                                       \
      |            ^~
/home/galak/git/zephyr/tests/net/context/src/main.c:922:40: note: ‘addr’ declared here
  922 |                         struct in_addr addr;
      |                                        ^~~~

2 replies

galak Jul 27, 2022
Collaborator

See similar issue on tests/net/checksum_offload/net.offload :

/home/galak/git/zephyr/include/zephyr/toolchain/gcc.h: In function ‘eth_tx_offloading_disabled’:
/home/galak/git/zephyr/include/zephyr/toolchain/gcc.h:127:12: error: ‘*(struct <anonymous> *)((char *)&addr + offsetof(struct in_addr, <U   0>)).__v’ may be used uninitialized [-Werror=maybe-uninitialized]
  127 |         __p->__v;                                                       \
      |            ^~
/home/galak/git/zephyr/tests/net/checksum_offload/src/main.c:171:40: note: ‘addr’ declared here
  171 |                         struct in_addr addr;
      |                                        ^~~~
cc1: all warnings being treated as errors

stephanosio Jul 29, 2022
Maintainer Author

native_posix-specific issues, will fix later.

galak · 2022-07-27T16:18:02Z

galak
Jul 27, 2022
Collaborator

test: tests/bluetooth/host_long_adv_recv/bluetooth.host_long_adv_recv on native_posix

/home/stephanos/Dev/zephyrproject/zephyr/subsys/bluetooth/host/scan.c: In function ‘le_adv_recv’:
/home/stephanos/Dev/zephyrproject/zephyr/subsys/bluetooth/host/scan.c:463:20: error: storing the address of local variable ‘id_addr’ in ‘*info.addr’ [-Werror=dangling-pointer=]
  463 |         info->addr = &id_addr;
      |         ~~~~~~~~~~~^~~~~~~~~~
/home/stephanos/Dev/zephyrproject/zephyr/subsys/bluetooth/host/scan.c:439:22: note: ‘id_addr’ declared here
  439 |         bt_addr_le_t id_addr;
      |                      ^~~~~~~
/home/stephanos/Dev/zephyrproject/zephyr/subsys/bluetooth/host/scan.c:439:22: note: ‘info’ declared here
cc1: all warnings being treated as errors

4 replies

stephanosio Jul 29, 2022
Maintainer Author

This is an actual dangling pointer usage.

Issue filed: zephyrproject-rtos/zephyr#48459

stephanosio Jul 29, 2022
Maintainer Author

~~Note that this is native_posix-specific issue (because the test only runs on that platform), so not a blocker for the Zephyr SDK 0.15.0.~~

stephanosio Aug 3, 2022
Maintainer Author

This issue is also observed while building tests/bluetooth/init/bluetooth.init.test_ctlr_oberver_ext for nrf52840dk_nrf52840.

Now this is a blocker.

stephanosio Aug 4, 2022
Maintainer Author

Fixed by ~~zephyrproject-rtos/zephyr#48690~~ zephyrproject-rtos/zephyr#48727

galak · 2022-07-27T16:19:59Z

galak
Jul 27, 2022
Collaborator

test tests/net/lib/lwm2m/content_plain_text/net.lwm2m.content_plain_text on native_posix:

/home/galak/git/zephyr/subsys/net/lib/coap/coap_link_format.c: In function ‘coap_well_known_core_get’:
/home/galak/git/zephyr/subsys/net/lib/coap/coap_link_format.c:690:21: error: the comparison will always evaluate as ‘true’ for the pointer operand in ‘resource + 52’ must not be NULL [-Werror=address]
  690 |                 if ((resource + 1) && (resource + 1)->path) {
      |                     ^
cc1: all warnings being treated as errors

2 replies

stephanosio Jul 29, 2022
Maintainer Author

Fixed by zephyrproject-rtos/zephyr@388023f

stephanosio Jul 29, 2022
Maintainer Author

zephyrproject-rtos/zephyr#48483

carlocaione · 2022-07-28T14:48:33Z

carlocaione
Jul 28, 2022
Collaborator

result of west build -b rv32m1_vega_ri5cy samples/hello_world --pristine

https://pastebin.com/raw/bP9ydEhf

2 replies

stephanosio Jul 28, 2022
Maintainer Author

This happens because -march is specified twice, where the latter is -march=rv32imc (i.e. missing Zicsr).

https://github.com/zephyrproject-rtos/zephyr/blob/21d1ad3762302b3e461953df59430c77e0709274/soc/riscv/openisa_rv32m1/CMakeLists.txt#L5-L13

stephanosio Aug 1, 2022
Maintainer Author

Fixed by zephyrproject-rtos/zephyr#48526

stephanosio · 2022-07-29T05:27:23Z

stephanosio
Jul 29, 2022
Maintainer Author

samples/posix/eventfd/sample.posix.eventfd

ccache /opt/toolchains/zephyr-sdk-0.15.0-rc1/arc64-zephyr-elf/bin/arc64-zephyr-elf-gcc -DKERNEL -D_FORTIFY_SOURCE=2 -D__LINUX_ERRNO_EXTENSIONS__ -D__ZEPHYR__=1 -I../../../../../../subsys/net/ip/. -I../../../../../../kernel/include -I../../../../../../arch/arc/include -I../../../../../../include/zephyr -I../../../../../../include -Izephyr/include/generated -I../../../../../../soc/arc/snps_qemu -I../../../../../../lib/libc/newlib/include -I../../../../../../subsys/net/l2 -Os -imacros /__w/zephyr-testing/zephyr-testing/twister-out/qemu_arc_hs6x/samples/posix/eventfd/sample.posix.eventfd/zephyr/include/generated/autoconf.h -ffreestanding -fno-common -g -gdwarf-4 -fdiagnostics-color=always -mcpu=hs6x --sysroot=/opt/toolchains/zephyr-sdk-0.15.0-rc1/arc64-zephyr-elf/arc64-zephyr-elf -imacros /__w/zephyr-testing/zephyr-testing/include/zephyr/toolchain/zephyr_stdint.h -Wall -Wformat -Wformat-security -Wno-format-zero-length -Wno-main -Wno-pointer-sign -Wpointer-arith -Wexpansion-to-defined -Wno-unused-but-set-variable -Werror=implicit-int -Werror -fno-asynchronous-unwind-tables -fno-pie -fno-pic -fno-reorder-functions -fno-defer-pop -fmacro-prefix-map=/__w/zephyr-testing/zephyr-testing/samples/posix/eventfd=CMAKE_SOURCE_DIR -fmacro-prefix-map=/__w/zephyr-testing/zephyr-testing=ZEPHYR_BASE -fmacro-prefix-map=/__w/zephyr-testing=WEST_TOPDIR -ffunction-sections -fdata-sections -g3 -gdwarf-2 -fno-delete-null-pointer-checks -munaligned-access -std=c99 -MD -MT zephyr/subsys/net/ip/CMakeFiles/subsys__net__ip.dir/net_if.c.obj -MF zephyr/subsys/net/ip/CMakeFiles/subsys__net__ip.dir/net_if.c.obj.d -o zephyr/subsys/net/ip/CMakeFiles/subsys__net__ip.dir/net_if.c.obj -c /__w/zephyr-testing/zephyr-testing/subsys/net/ip/net_if.c
/__w/zephyr-testing/zephyr-testing/subsys/net/ip/net_if.c: In function 'net_if_get_default':
/__w/zephyr-testing/zephyr-testing/subsys/net/ip/net_if.c:566:32: error: comparison between two arrays [-Werror=array-compare]
  566 |         if (_net_if_list_start == _net_if_list_end) {
      |                                ^~
/__w/zephyr-testing/zephyr-testing/subsys/net/ip/net_if.c:566:32: note: use '&_net_if_list_start[0] == &_net_if_list_end[0]' to compare the addresses
cc1: all warnings being treated as errors

1 reply

stephanosio Aug 1, 2022
Maintainer Author

Duplicate of #530 (comment)

stephanosio · 2022-07-29T05:28:44Z

stephanosio
Jul 29, 2022
Maintainer Author

tests/kernel/common/kernel.common and other kernel tests on pan1770_evb (Nordic platforms)

FAILED: modules/hal_nordic/nrfx/CMakeFiles/modules__hal_nordic__nrfx.dir/__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c.obj 
ccache /opt/toolchains/zephyr-sdk-0.15.0-rc1/arm-zephyr-eabi/bin/arm-zephyr-eabi-gcc -DKERNEL -DNRF52840_XXAA -D_FORTIFY_SOURCE=2 -D__PROGRAM_START -D__ZEPHYR__=1 -I../../../../../include/zephyr -I../../../../../include -Izephyr/include/generated -I../../../../../soc/arm/nordic_nrf/nrf52 -I../../../../../soc/arm/nordic_nrf/common/. -I/__w/zephyr-testing/modules/hal/cmsis/CMSIS/Core/Include -I/__w/zephyr-testing/modules/hal/nordic/nrfx -I/__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/include -I/__w/zephyr-testing/modules/hal/nordic/nrfx/mdk -I../../../../../modules/hal_nordic/nrfx/. -I/__w/zephyr-testing/modules/debug/segger/SEGGER -I/__w/zephyr-testing/modules/debug/segger/Config -I../../../../../modules/segger/. -isystem ../../../../../lib/libc/minimal/include -isystem /opt/toolchains/zephyr-sdk-0.15.0-rc1/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/12.1.0/include -isystem /opt/toolchains/zephyr-sdk-0.15.0-rc1/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/12.1.0/include-fixed -Os -imacros /__w/zephyr-testing/zephyr-testing/twister-out/pan1770_evb/samples/synchronization/sample.kernel.synchronization/zephyr/include/generated/autoconf.h -ffreestanding -fno-common -g -gdwarf-4 -fdiagnostics-color=always -mcpu=cortex-m4 -mthumb -mabi=aapcs -mfp16-format=ieee --sysroot=/opt/toolchains/zephyr-sdk-0.15.0-rc1/arm-zephyr-eabi/arm-zephyr-eabi -imacros /__w/zephyr-testing/zephyr-testing/include/zephyr/toolchain/zephyr_stdint.h -Wall -Wformat -Wformat-security -Wno-format-zero-length -Wno-main -Wno-pointer-sign -Wpointer-arith -Wexpansion-to-defined -Wno-unused-but-set-variable -Werror=implicit-int -Werror -fno-asynchronous-unwind-tables -fno-pie -fno-pic -fno-reorder-functions -fno-defer-pop -fmacro-prefix-map=/__w/zephyr-testing/zephyr-testing/samples/synchronization=CMAKE_SOURCE_DIR -fmacro-prefix-map=/__w/zephyr-testing/zephyr-testing=ZEPHYR_BASE -fmacro-prefix-map=/__w/zephyr-testing=WEST_TOPDIR -ffunction-sections -fdata-sections -std=c99 -nostdinc -MD -MT modules/hal_nordic/nrfx/CMakeFiles/modules__hal_nordic__nrfx.dir/__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c.obj -MF modules/hal_nordic/nrfx/CMakeFiles/modules__hal_nordic__nrfx.dir/__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c.obj.d -o modules/hal_nordic/nrfx/CMakeFiles/modules__hal_nordic__nrfx.dir/__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c.obj -c /__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c
In file included from /__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/include/nrfx_clock.h:38,
                 from /__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c:38:
In function 'nrf_clock_is_running',
    inlined from 'nrfx_clock_is_running' at /__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/include/nrfx_clock.h:326:12,
    inlined from 'clock_stop' at /__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c:223:5,
    inlined from 'clock_stop' at /__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c:187:13:
/__w/zephyr-testing/modules/hal/nordic/nrfx/hal/nrf_clock.h:784:51: error: array subscript 'nrf_clock_lfclk_t[0]' is partly outside array bounds of 'nrf_clock_hfclk_t[1]' [-Werror=array-bounds]
  784 |                 (*(nrf_clock_lfclk_t *)p_clk_src) =
      |                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
  785 |                     (nrf_clock_lfclk_t)((p_reg->LFCLKSTAT & CLOCK_LFCLKSTAT_SRC_Msk)
      |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  786 |                                         >> CLOCK_LFCLKSTAT_SRC_Pos);
      |                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~
/__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c: In function 'clock_stop':
/__w/zephyr-testing/modules/hal/nordic/nrfx/drivers/src/nrfx_clock.c:221:23: note: object 'clk_src' of size 1
  221 |     nrf_clock_hfclk_t clk_src = NRF_CLOCK_HFCLK_HIGH_ACCURACY;
      |                       ^~~~~~~
cc1: all warnings being treated as errors

1 reply

stephanosio Aug 3, 2022
Maintainer Author

Fixed by zephyrproject-rtos/zephyr#48661

stephanosio · 2022-07-29T05:30:24Z

stephanosio
Jul 29, 2022
Maintainer Author

tests/kernel/common/kernel.common on olimex_esp32_evb (ESP32 platforms)

FAILED: zephyr/CMakeFiles/zephyr.dir/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c.obj 
ccache /opt/toolchains/zephyr-sdk-0.15.0-rc1/xtensa-espressif_esp32_zephyr-elf/bin/xtensa-espressif_esp32_zephyr-elf-gcc -DCONFIG_SPI_FLASH_ROM_DRIVER_PATCH -DCONFIG_SPI_FLASH_USE_LEGACY_IMPL -DKERNEL -DTC_RUNID=ae6971f94c1f45b14e4c434b3449c35d -D_FORTIFY_SOURCE=2 -D__ZEPHYR__=1 -I../../../../../../kernel/include -I../../../../../../arch/xtensa/include -I../../../../../../include/zephyr -I../../../../../../include -Izephyr/include/generated -I../../../../../../soc/xtensa/esp32 -I../../../../../../soc/xtensa/esp32/include -I../../../../../../subsys/testsuite/include -I../../../../../../subsys/testsuite/ztest/include -I../../../../../../subsys/testsuite/include/zephyr -I../../../../../../subsys/testsuite/ztest/include/zephyr -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/include/bt -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/include/crypto -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../esp_shared/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../esp_shared/include/wifi -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/hal/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/hal/platform_port/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/hal/esp32/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_hw_support/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_hw_support/port/esp32 -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_hw_support/port/esp32/private_include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_common/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_rom/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_rom/esp32/ld -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp32/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/soc/esp32/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/xtensa/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/xtensa/esp32/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/soc/include/soc -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/soc/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/soc/src/esp32/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/driver/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_wifi/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/efuse/esp32/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/efuse/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_system/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_wifi/esp32/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_timer/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_timer/private_include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_netif/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/esp_event/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/log/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/wpa_supplicant/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/wpa_supplicant/port/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/wpa_supplicant/src -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/wpa_supplicant/include/esp_supplicant -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/wpa_supplicant/src/crypto -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/spi_flash/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/spi_flash/private_include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/bootloader_support/include -I/__w/zephyr-testing/modules/hal/espressif/zephyr/esp32/../../components/bootloader_support/include_bootloader -isystem ../../../../../../lib/libc/minimal/include -Os -imacros /__w/zephyr-testing/zephyr-testing/twister-out/olimex_esp32_evb/tests/kernel/common/kernel.common/zephyr/include/generated/autoconf.h -ffreestanding -fno-common -g -gdwarf-4 -fdiagnostics-color=always --sysroot=/opt/toolchains/zephyr-sdk-0.15.0-rc1/xtensa-espressif_esp32_zephyr-elf/xtensa-espressif_esp32_zephyr-elf -imacros /__w/zephyr-testing/zephyr-testing/include/zephyr/toolchain/zephyr_stdint.h -Wall -Wformat -Wformat-security -Wno-format-zero-length -Wno-main -Wno-pointer-sign -Wpointer-arith -Wexpansion-to-defined -Wno-unused-but-set-variable -Werror=implicit-int -Werror -fno-asynchronous-unwind-tables -fno-pie -fno-pic -fno-reorder-functions -fno-defer-pop -fmacro-prefix-map=/__w/zephyr-testing/zephyr-testing/tests/kernel/common=CMAKE_SOURCE_DIR -fmacro-prefix-map=/__w/zephyr-testing/zephyr-testing=ZEPHYR_BASE -fmacro-prefix-map=/__w/zephyr-testing=WEST_TOPDIR -ffunction-sections -fdata-sections -mlongcalls -Wno-unused-variable -Wno-maybe-uninitialized -std=c99 -MD -MT zephyr/CMakeFiles/zephyr.dir/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c.obj -MF zephyr/CMakeFiles/zephyr.dir/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c.obj.d -o zephyr/CMakeFiles/zephyr.dir/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c.obj -c /__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c
/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c: In function 'periph_module_enable':
/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c:28:19: error: array subscript 35 is above array bounds of 'uint8_t[35]' {aka 'unsigned char[35]'} [-Werror=array-bounds]
   28 |     if (ref_counts[periph] == 0) {
      |         ~~~~~~~~~~^~~~~~~~
/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c:22:16: note: while referencing 'ref_counts'
   22 | static uint8_t ref_counts[PERIPH_MODULE_MAX] = {0};
      |                ^~~~~~~~~~
/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c: In function 'periph_module_disable':
/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c:39:15: error: array subscript 35 is above array bounds of 'uint8_t[35]' {aka 'unsigned char[35]'} [-Werror=array-bounds]
   39 |     ref_counts[periph]--;
      |     ~~~~~~~~~~^~~~~~~~
/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c:22:16: note: while referencing 'ref_counts'
   22 | static uint8_t ref_counts[PERIPH_MODULE_MAX] = {0};
      |                ^~~~~~~~~~
/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c:39:15: error: array subscript 35 is above array bounds of 'uint8_t[35]' {aka 'unsigned char[35]'} [-Werror=array-bounds]
   39 |     ref_counts[periph]--;
      |     ~~~~~~~~~~^~~~~~~~
/__w/zephyr-testing/modules/hal/espressif/components/driver/periph_ctrl.c:22:16: note: while referencing 'ref_counts'
   22 | static uint8_t ref_counts[PERIPH_MODULE_MAX] = {0};
      |                ^~~~~~~~~~
cc1: all warnings being treated as errors

1 reply

stephanosio Aug 4, 2022
Maintainer Author

This was caused by the compiler analysing the assertion failure execution path, because the assert() maps to __ASSERT_NO_MSG() (when the minimal libc is selected), which does not indicate that the function does not return in case of an assertion failure.

Fixed by zephyrproject-rtos/zephyr#48710

stephanosio · 2022-07-29T05:31:44Z

stephanosio
Jul 29, 2022
Maintainer Author

tests/kernel/common/kernel.common.picolibc

FAILED: modules/picolibc/CMakeFiles/c.dir/newlib/libc/stdlib/nano-malloc-memalign.c.obj 
ccache /opt/toolchains/zephyr-sdk-0.15.0-rc1/arc-zephyr-elf/bin/arc-zephyr-elf-gcc  -I/__w/zephyr-testing/modules/lib/picolibc/newlib/libm/common -isystem modules/picolibc/picolibc/include --include /__w/zephyr-testing/zephyr-testing/twister-out/qemu_arc_em/tests/kernel/common/kernel.common.picolibc/modules/picolibc/picolibc/include/picolibc.h -nostdlib -fno-common -frounding-math -fsignaling-nans -fno-stack-protector -ffunction-sections -fdata-sections -Wall -Wextra -Werror=implicit-function-declaration -Werror=vla -Warray-bounds -Wold-style-definition -ftls-model=local-exec -Os -imacros /__w/zephyr-testing/zephyr-testing/twister-out/qemu_arc_em/tests/kernel/common/kernel.common.picolibc/zephyr/include/generated/autoconf.h -g -gdwarf-4 -fdiagnostics-color=always -mcpu=arcem --sysroot=/opt/toolchains/zephyr-sdk-0.15.0-rc1/arc-zephyr-elf/arc-zephyr-elf -imacros /__w/zephyr-testing/zephyr-testing/include/zephyr/toolchain/zephyr_stdint.h -Wformat -Wformat-security -Wno-format-zero-length -Wno-main -Wno-pointer-sign -Wpointer-arith -Wexpansion-to-defined -Wno-unused-but-set-variable -Werror=implicit-int -Werror -fno-asynchronous-unwind-tables -fno-pie -fno-pic -fno-reorder-functions -fno-defer-pop -fmacro-prefix-map=/__w/zephyr-testing/zephyr-testing/tests/kernel/common=CMAKE_SOURCE_DIR -fmacro-prefix-map=/__w/zephyr-testing/zephyr-testing=ZEPHYR_BASE -fmacro-prefix-map=/__w/zephyr-testing=WEST_TOPDIR -g3 -gdwarf-2 -fno-delete-null-pointer-checks -mtp-regno=26 -mno-sdata -MD -MT modules/picolibc/CMakeFiles/c.dir/newlib/libc/stdlib/nano-malloc-memalign.c.obj -MF modules/picolibc/CMakeFiles/c.dir/newlib/libc/stdlib/nano-malloc-memalign.c.obj.d -o modules/picolibc/CMakeFiles/c.dir/newlib/libc/stdlib/nano-malloc-memalign.c.obj -c /__w/zephyr-testing/modules/lib/picolibc/newlib/libc/stdlib/nano-malloc-memalign.c
In file included from /__w/zephyr-testing/modules/lib/picolibc/newlib/libc/stdlib/nano-malloc-memalign.c:34:
/__w/zephyr-testing/modules/lib/picolibc/newlib/libc/stdlib/nano-mallocr.c: In function 'memalign':
/__w/zephyr-testing/modules/lib/picolibc/newlib/libc/stdlib/nano-mallocr.c:761:36: error: array subscript 0 is outside array bounds of 'char[2147483647]' [-Werror=array-bounds]
  761 |         new_chunk_p->size = chunk_p->size - offset;
      |                                    ^~
/__w/zephyr-testing/modules/lib/picolibc/newlib/libc/stdlib/nano-mallocr.c:743:17: note: at offset -4 into object of size [0, 2147483647] allocated by '__malloc_malloc'
  743 |     allocated = __malloc_malloc(size_with_padding);
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In function 'make_free_chunk',
    inlined from 'memalign' at /__w/zephyr-testing/modules/lib/picolibc/newlib/libc/stdlib/nano-mallocr.c:763:2:
/__w/zephyr-testing/modules/lib/picolibc/newlib/libc/stdlib/nano-mallocr.c:208:13: error: array subscript 0 is outside array bounds of 'char[2147483647]' [-Werror=array-bounds]
  208 |     c->size = size;
      |     ~~~~~~~~^~~~~~
/__w/zephyr-testing/modules/lib/picolibc/newlib/libc/stdlib/nano-mallocr.c: In function 'memalign':
/__w/zephyr-testing/modules/lib/picolibc/newlib/libc/stdlib/nano-mallocr.c:743:17: note: at offset -4 into object of size [0, 2147483647] allocated by '__malloc_malloc'
  743 |     allocated = __malloc_malloc(size_with_padding);
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

6 replies

stephanosio Aug 4, 2022
Maintainer Author

@keith-packard re: zephyrproject-rtos/zephyr#46967 (comment), have you had a chance to take a look at this?

keith-packard Aug 5, 2022
Collaborator

just looking at the error messages, we probably just need to disable these warnings around these code fragments. Any concern about doing that? I'll look at using the new SDK on upstream picolibc testing and see how that goes.

stephanosio Aug 5, 2022
Maintainer Author

we probably just need to disable these warnings around these code fragments

yes, I think that is the right thing to do as well, since there is nothing wrong with the implementation in itself.

keith-packard Aug 5, 2022
Collaborator

PR to picolibc is picolibc/picolibc#316 -- this uses the same mechanism as on meson -- -fno-builtin-malloc -fno-builtin-free.

When that gets merged upstream, I'll move the zephyr hash to include that fix.

keith-packard Aug 13, 2022
Collaborator

Zephyr PR zephyrproject-rtos/zephyr#49042 moves the picolibc hash to version 1.7.8 with fixes for SDK 0.15.0-rc1

stephanosio · 2022-08-10T10:40:35Z

stephanosio
Aug 10, 2022
Maintainer Author

cbprintf and logging test failures on qemu_nios2

e.g. tests/lib/cbprintf_package/libraries.cbprintf_package_cpp

START - test_cbprintf_package
-----------------------------------------
test long b1b2b3b4 bdc5ca39 e4e3e2e1
runtime package 0x401068:
05 00 00 00 d0 ab 42 00 b4 b3 b2 b1 39 ca c5 bd e1 e2 e3 e4 
static package 0x4010d0:
05 00 00 00 d0 ab 42 00 b4 b3 b2 b1 39 ca c5 bd e1 e2 e3 e4 
-----------------------------------------
test long long b1b2b3b4 1122334455667788 e4e3e2e1
runtime package 0x401060:
06 00 00 00 ec ac 42 00 b4 b3 b2 b1 88 77 66 55 44 33 22 11 e1 e2 e3 e4 
static package 0x4010d0:
06 00 00 00 ec ac 42 00 b4 b3 b2 b1 b4 b3 b2 b1 64 00 00 00 e1 e2 e3 e4 

    Assertion failed at WEST_TOPDIR/zephyr-testing/tests/lib/cbprintf_package/src/main.c:55: unpack: (strcmp(buf->buf, compare_buf) not equal to 0)
Strings differ
exp: |test long long b1b2b3b4 1122334455667788 e4e3e2e1|
got: |test long long b1b2b3b4 64b1b2b3b4 e4e3e2e1|

 FAIL - test_cbprintf_package in 0.0 seconds

0 replies

stephanosio · 2022-08-10T10:58:37Z

stephanosio
Aug 10, 2022
Maintainer Author

samples/philosophers/sample.kernel.philosopher.coop_only on qemu_riscv32

Philosopher 5 [C:-7]        STARVING       
Philosopher 5 [C:-7]    HOLDING ONE FORK   
Philosopher 5 [C:-7]   EATING  [  25 ms ] 
Philosopher 4 [C:-6]        STARVING       
Philosopher 4 [C:-6]    HOLDING ONE FORK   
Philosopher 3 [C:-5]        STARVING       
Philosopher 3 [C:-5]    HOLDING ONE FORK   
Philosopher 3 [C:-5]   EATING  [  25 ms ]

1 reply

stephanosio Aug 10, 2022
Maintainer Author

NOTE: only the coop_only variant fails:

INFO    - 1/9 qemu_riscv32              samples/philosophers/sample.kernel.philosopher.preempt_only PASSED (qemu 2.055s)
INFO    - 2/9 qemu_riscv32              samples/philosophers/sample.kernel.philosopher.static PASSED (qemu 2.078s)
INFO    - 3/9 qemu_riscv32              samples/philosophers/sample.kernel.philosopher.fifos PASSED (qemu 2.037s)
INFO    - 4/9 qemu_riscv32              samples/philosophers/sample.kernel.philosopher.semaphores PASSED (qemu 2.062s)
INFO    - 5/9 qemu_riscv32              samples/philosophers/sample.kernel.philosopher.stacks PASSED (qemu 2.054s)
INFO    - 6/9 qemu_riscv32              samples/philosophers/sample.kernel.philosopher     PASSED (qemu 2.035s)
INFO    - 7/9 qemu_riscv32              samples/philosophers/sample.kernel.philosopher.lifos PASSED (qemu 2.036s)
INFO    - 8/9 qemu_riscv32              samples/philosophers/sample.kernel.philosopher.same_prio PASSED (qemu 2.050s)
INFO    - 9/9 qemu_riscv32              samples/philosophers/sample.kernel.philosopher.coop_only FAILED Timeout (qemu 60.010s)

The timeout is due to the output not matching the regex.

stephanosio · 2022-08-10T11:00:23Z

stephanosio
Aug 10, 2022
Maintainer Author

tests/subsys/portability/cmsis_rtos_v2/portability.cmsis_rtos_v2 on qemu_nios2

START - test_thread_detached
 * Thread B started.
ASSERTION FAIL [!z_is_thread_state_set(_kernel.cpus[0].current, ((1UL << (4))))] @ WEST_TOPDIR/zephyr-testing/kernel/sched.c:1434
	
@ WEST_TOPDIR/zephyr-testing/lib/os/assert.c:43
E: >>> ZEPHYR FATAL ERROR 4: Kernel panic on CPU 0
E: Current thread: 0x401c08 (ZephyrThread)
E: Halting system
E: Exception cause: 0 ECCFTL: 0x0
E: reason: reset
E: Faulting instruction: 0x00426d84
E:   r1: 0x00409338  r2: 0x00000012  r3: 0x00401c08  r4: 0x00000004
E:   r5: 0x0000000a  r6: 0x00406fda  r7: 0x00406fdb  r8: 0x00000000
E:   r9: 0x00000000 r10: 0x00402380 r11: 0x00401c08 r12: 0x004093e8
E:  r13: 0x00000000 r14: 0x00000000 r15: 0x00000001  ra: 0x0042a43c
E: estatus: 00000001
E: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
E: Current thread: 0x401c08 (ZephyrThread)
E: Halting system
E: Exception cause: 0 ECCFTL: 0x0
E: reason: reset
E: Faulting instruction: 0x00426d84
E:   r1: 0x00409338  r2: 0x00000012  r3: 0x00401c08  r4: 0x00000000
E:   r5: 0x0000000a  r6: 0x00406f52  r7: 0x00406f53  r8: 0x00000000
E:   r9: 0x00000000 r10: 0x00400710 r11: 0x00000000 r12: 0x004093e8
E:  r13: 0x00000000 r14: 0x00000000 r15: 0x00426d84  ra: 0x0042a43c
E: estatus: 00000001

0 replies

stephanosio · 2022-08-10T11:04:30Z

stephanosio
Aug 10, 2022
Maintainer Author

tests/kernel/condvar/condvar_api/kernel.condvar and many other kernel tests on qemu_x86_nopae, qemu_x86_tiny, qemu_x86_virt,

START - test_condvar_multiple_threads_wait_wake
E: Page fault at address 0x141000 (error code 0x3)
E: Access violation: supervisor thread not allowed to write
E: PTE: 0x141000 -> 0x00141000: 
E: EAX: 0x00116700, EBX: 0x00000448, ECX: 0x00116500, EDX: 0x00116400
E: ESI: 0x0013fee4, EDI: 0x001167bc, EBP: 0x0013fedc, ESP: 0x0013fecc
E: EFLAGS: 0x00000006 CS: 0x0008 CR3: 0x00149000
E: call trace:
E: EIP: 0x00101402
E:      0x0010159e (0x1165c0)
E:      0x00116700 (0x1165c0)
E:      corrupted? (bp=0x1165c0)
E: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
E: Current thread: 0x1165c0 (test_condvar_multiple_threads_w)
Caught system error -- reason 0 0
Fatal error was unexpected, aborting...

0 replies

stephanosio · 2022-08-10T11:06:49Z

stephanosio
Aug 10, 2022
Maintainer Author

samples/net/openthread/coprocessor/sample.net.openthread.coprocessor and many other tests on tlsr9518adk80d

/opt/toolchains/zephyr-sdk-0.15.0-rc1/riscv64-zephyr-elf/bin/../lib/gcc/riscv64-zephyr-elf/12.1.0/../../../../riscv64-zephyr-elf/bin/ld.bfd: section vector LMA [0000000020000000,0000000020000017] overlaps section vector LMA [0000000020000000,00000000200000bb]
collect2: error: ld returned 1 exit status

0 replies

stephanosio · 2022-08-10T11:10:24Z

stephanosio
Aug 10, 2022
Maintainer Author

samples/synchronization/sample.kernel.synchronization on rpi_pico

In file included from /__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/bootrom.c:7:
In function 'rom_func_lookup_inline',
    inlined from 'rom_func_lookup' at /__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/bootrom.c:19:12:
/__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/include/pico/bootrom.h:120:59: error: array subscript 0 is outside array bounds of 'uint16_t[0]' {aka 'short unsigned int[]'} [-Werror=array-bounds]
  120 | #define rom_hword_as_ptr(rom_address) (void *)(uintptr_t)(*(uint16_t *)rom_address)
      |                                                          ~^~~~~~~~~~~~~~~~~~~~~~~~~
/__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/include/pico/bootrom.h:129:66: note: in expansion of macro 'rom_hword_as_ptr'
  129 |     rom_table_lookup_fn rom_table_lookup = (rom_table_lookup_fn) rom_hword_as_ptr(0x18);
      |                                                                  ^~~~~~~~~~~~~~~~
/__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/include/pico/bootrom.h:120:59: error: array subscript 0 is outside array bounds of 'uint16_t[0]' {aka 'short unsigned int[]'} [-Werror=array-bounds]
  120 | #define rom_hword_as_ptr(rom_address) (void *)(uintptr_t)(*(uint16_t *)rom_address)
      |                                                          ~^~~~~~~~~~~~~~~~~~~~~~~~~
/__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/include/pico/bootrom.h:130:41: note: in expansion of macro 'rom_hword_as_ptr'
  130 |     uint16_t *func_table = (uint16_t *) rom_hword_as_ptr(0x14);
      |                                         ^~~~~~~~~~~~~~~~
/__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/bootrom.c: In function 'rom_data_lookup':
/__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/bootrom.c:16:59: error: array subscript 0 is outside array bounds of 'uint16_t[0]' {aka 'short unsigned int[]'} [-Werror=array-bounds]
   16 | #define rom_hword_as_ptr(rom_address) (void *)(uintptr_t)(*(uint16_t *)rom_address)
      |                                                          ~^~~~~~~~~~~~~~~~~~~~~~~~~
/__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/bootrom.c:23:66: note: in expansion of macro 'rom_hword_as_ptr'
   23 |     rom_table_lookup_fn rom_table_lookup = (rom_table_lookup_fn) rom_hword_as_ptr(0x18);
      |                                                                  ^~~~~~~~~~~~~~~~
/__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/bootrom.c:16:59: error: array subscript 0 is outside array bounds of 'uint16_t[0]' {aka 'short unsigned int[]'} [-Werror=array-bounds]
   16 | #define rom_hword_as_ptr(rom_address) (void *)(uintptr_t)(*(uint16_t *)rom_address)
      |                                                          ~^~~~~~~~~~~~~~~~~~~~~~~~~
/__w/zephyr-testing/modules/hal/rpi_pico/src/rp2_common/pico_bootrom/bootrom.c:24:41: note: in expansion of macro 'rom_hword_as_ptr'
   24 |     uint16_t *data_table = (uint16_t *) rom_hword_as_ptr(0x16);
      |                                         ^~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

0 replies

stephanosio · 2022-08-10T11:13:15Z

stephanosio
Aug 10, 2022
Maintainer Author

samples/philosophers/sample.kernel.philosopher.coop_only on qemu_xtensa

Philosopher 5 [C:-7]        STARVING       
Philosopher 5 [C:-7]    HOLDING ONE FORK   
Philosopher 5 [C:-7]   EATING  [  25 ms ] 
Philosopher 4 [C:-6]        STARVING       
Philosopher 4 [C:-6]    HOLDING ONE FORK   
Philosopher 3 [C:-5]        STARVING       
Philosopher 3 [C:-5]    HOLDING ONE FORK   
Philosopher 3 [C:-5]   EATING  [  25 ms ]

0 replies

stephanosio · 2022-08-10T11:13:54Z

stephanosio
Aug 10, 2022
Maintainer Author

cbprintf and logging test failures on qemu_xtensa

e.g. tests/lib/cbprintf_package/libraries.cbprintf_package_cpp

START - test_cbprintf_package
-----------------------------------------
test long b1b2b3b4 bdc5ca39 e4e3e2e1
runtime package 0x600088c0:
05 00 00 00 05 0b 00 60 b4 b3 b2 b1 39 ca c5 bd e1 e2 e3 e4 
static package 0x600088e0:
05 00 00 00 05 0b 00 60 b4 b3 b2 b1 39 ca c5 bd e1 e2 e3 e4 
-----------------------------------------
test long long b1b2b3b4 1122334455667788 e4e3e2e1
runtime package 0x600088c0:
07 00 00 00 17 0c 00 60 b4 b3 b2 b1 00 00 00 00 88 77 66 55 44 33 22 11 e1 e2 e3 e4 
static package 0x600088a0:
07 00 00 00 17 0c 00 60 b4 b3 b2 b1 44 33 22 11 b4 b3 b2 b1 24 00 00 00 e1 e2 e3 e4 

    Assertion failed at WEST_TOPDIR/zephyr-testing/tests/lib/cbprintf_package/src/main.c:55: unpack: (strcmp(buf->buf, compare_buf) not equal to 0)
Strings differ
exp: |test long long b1b2b3b4 1122334455667788 e4e3e2e1|
got: |test long long b1b2b3b4 24b1b2b3b4 e4e3e2e1|

 FAIL - test_cbprintf_package in 0.9 seconds

0 replies

stephanosio · 2022-08-10T11:14:56Z

stephanosio
Aug 10, 2022
Maintainer Author

tests/subsys/portability/cmsis_rtos_v2/portability.cmsis_rtos_v2 on qemu_xtensa

START - test_thread_detached
 * Thread B started.
ASSERTION FAIL [!z_is_thread_state_set(_kernel.cpus[0].current, ((1UL << (4))))] @ WEST_TOPDIR/zephyr-testing/kernel/sched.c:1434
	
E:  ** FATAL EXCEPTION
E:  ** CPU 0 EXCCAUSE 63 (zephyr exception)
E:  **  PC 0x60008e2a VADDR (nil)
E:  **  PS 0x60620
E:  **    (INTLEVEL:0 EXCM: 0 UM:1 RING:0 WOE:1 OWB:6 CALLINC:2)
E:  **  A0 0xa0008cdc  SP 0x600131c0  A2 0x4  A3 0x600131d0
E:  **  A4 0x600131b0  A5 0x4  A6 (nil)  A7 (nil)
E:  **  A8 0xa000767c  A9 0x600130d0 A10 0x2 A11 0x2
E:  ** A12 0x4 A13 0x25 A14 0x600131b0 A15 0x4
E:  ** SAR 0x4
E: >>> ZEPHYR FATAL ERROR 4: Kernel panic on CPU 0
E: Current thread: 0x600182a8 (ZephyrThread)
E: Halting system

0 replies

stephanosio · 2022-08-10T11:15:14Z

stephanosio
Aug 10, 2022
Maintainer Author

samples/philosophers/sample.kernel.philosopher.coop_only on qemu_cortex_a53

*** Booting Zephyr OS build v2.2.0-rc1-30438-g861850467394  ***
Philosopher 5 [C:-7]        STARVING       
Philosopher 5 [C:-7]    HOLDING ONE FORK   
Philosopher 5 [C:-7]   EATING  [  25 ms ] 
Philosopher 4 [C:-6]        STARVING       
Philosopher 4 [C:-6]    HOLDING ONE FORK   
Philosopher 3 [C:-5]        STARVING       
Philosopher 3 [C:-5]    HOLDING ONE FORK   
Philosopher 3 [C:-5]   EATING  [  25 ms ]

1 reply

stephanosio Aug 10, 2022
Maintainer Author

This sample only gets stuck when compiled with -Os (optimise for size), which is the default setting. When compiled with -Og (optimise for debugging; i.e. with less optimisation), it works.

The problem is likely related to the compiler optimising out something that it should not have.

stephanosio · 2022-08-10T11:16:01Z

stephanosio
Aug 10, 2022
Maintainer Author

samples/posix/eventfd/sample.posix.eventfd on qemu_cortex_a53

/opt/toolchains/zephyr-sdk-0.15.0-rc1/aarch64-zephyr-elf/bin/../lib/gcc/aarch64-zephyr-elf/12.1.0/../../../../aarch64-zephyr-elf/bin/ld.bfd: discarded output section: `.got.plt'
collect2: error: ld returned 1 exit status

0 replies

stephanosio · 2022-08-10T11:16:47Z

stephanosio
Aug 10, 2022
Maintainer Author

tests/kernel/interrupt/arch.interrupt on qemu_cortex_a53

START - test_nested_isr
Triggering irq : 14
isr0: Enter
Triggering irq : 15

    Assertion failed at WEST_TOPDIR/zephyr-testing/tests/kernel/interrupt/src/nested_irq.c:106: isr0: (isr1_result not equal to ISR1_TOKEN)
isr1 did not execute
isr0: Leave
isr1: Enter
isr1: Leave
 FAIL - test_nested_isr in 0.6 seconds

0 replies

stephanosio · 2022-08-10T11:17:54Z

stephanosio
Aug 10, 2022
Maintainer Author

cbprintf and logging test failures on qemu_cortex_a53

e.g. tests/lib/cbprintf_package/libraries.cbprintf_package_cpp

START - test_cbprintf_package
-----------------------------------------
test long b1b2b3b4 ffffffffbdc5ca39 e4e3e2e1
runtime package 0x40063640:
09 00 00 00 00 00 00 00 3b fb 00 40 00 00 00 00 b4 b3 b2 b1 00 00 00 00 39 ca c5 bd ff ff ff ff e1 e2 e3 e4 
static package 0x40063610:
09 00 00 00 00 00 00 00 3b fb 00 40 00 00 00 00 b4 b3 b2 b1 00 00 00 00 b4 b3 b2 b1 00 00 00 00 e1 e2 e3 e4 

    Assertion failed at WEST_TOPDIR/zephyr-testing/tests/lib/cbprintf_package/src/main.c:55: unpack: (strcmp(buf->buf, compare_buf) not equal to 0)
Strings differ
exp: |test long b1b2b3b4 ffffffffbdc5ca39 e4e3e2e1|
got: |test long b1b2b3b4 b1b2b3b4 e4e3e2e1|

 FAIL - test_cbprintf_package in 0.5 seconds

0 replies

stephanosio · 2022-08-10T11:22:08Z

stephanosio
Aug 10, 2022
Maintainer Author

samples/boards/esp32/flash_encryption/sample.board.esp32 on esp32

/__w/zephyr-testing/modules/hal/espressif/zephyr/esp_shared/src/host_flash/cache_utils.c:31:1: error: ignoring attribute 'section (".iram1.7")' because it conflicts with previous 'section (".iram1.5")' [-Werror=attributes]
   31 | {
      | ^
/__w/zephyr-testing/modules/hal/espressif/zephyr/esp_shared/src/host_flash/cache_utils.c:23:23: note: previous declaration here
   23 | static void IRAM_ATTR spi_flash_disable_cache(uint32_t cpuid, uint32_t *saved_state);
      |                       ^~~~~~~~~~~~~~~~~~~~~~~
/__w/zephyr-testing/modules/hal/espressif/zephyr/esp_shared/src/host_flash/cache_utils.c:67:1: error: ignoring attribute 'section (".iram1.8")' because it conflicts with previous 'section (".iram1.6")' [-Werror=attributes]
   67 | {
      | ^
/__w/zephyr-testing/modules/hal/espressif/zephyr/esp_shared/src/host_flash/cache_utils.c:24:23: note: previous declaration here
   24 | static void IRAM_ATTR spi_flash_restore_cache(uint32_t cpuid, uint32_t saved_state);
      |                       ^~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

0 replies

sh-one · 2022-10-09T10:05:37Z

sh-one
Oct 9, 2022

setup.sh fails on SUSE Leap 15.4. $OSTYPE=linux not linux-gnu. Changing line 188 to linux*) worked.

1 reply

stephanosio Nov 4, 2022
Maintainer Author

Thanks for reporting, I will update the setup script accordingly.

Zephyr SDK 0.15.0 Migration Issues #530

stephanosio Jul 27, 2022 Maintainer

Replies: 40 comments · 47 replies

galak Jul 27, 2022 Collaborator

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

galak Jul 27, 2022 Collaborator

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

galak Jul 27, 2022 Collaborator

stephanosio Jul 29, 2022 Maintainer Author

galak Jul 27, 2022 Collaborator

This comment has been hidden.

stephanosio Aug 3, 2022 Maintainer Author

stephanosio Aug 4, 2022 Maintainer Author

stephanosio Aug 4, 2022 Maintainer Author

galak Jul 27, 2022 Collaborator

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

galak Jul 27, 2022 Collaborator

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

galak Jul 27, 2022 Collaborator

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

galak Jul 27, 2022 Collaborator

galak Jul 27, 2022 Collaborator

stephanosio Jul 29, 2022 Maintainer Author

galak Jul 27, 2022 Collaborator

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Aug 3, 2022 Maintainer Author

stephanosio Aug 4, 2022 Maintainer Author

galak Jul 27, 2022 Collaborator

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

carlocaione Jul 28, 2022 Collaborator

stephanosio Jul 28, 2022 Maintainer Author

stephanosio Aug 1, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Aug 1, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Aug 3, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Aug 4, 2022 Maintainer Author

stephanosio Jul 29, 2022 Maintainer Author

stephanosio Aug 4, 2022 Maintainer Author

keith-packard Aug 5, 2022 Collaborator

stephanosio Aug 5, 2022 Maintainer Author

keith-packard Aug 5, 2022 Collaborator

keith-packard Aug 13, 2022 Collaborator

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio Aug 10, 2022 Maintainer Author

stephanosio
Jul 27, 2022
Maintainer

Replies: 40 comments 47 replies

galak
Jul 27, 2022
Collaborator

stephanosio Jul 29, 2022
Maintainer Author

stephanosio Jul 29, 2022
Maintainer Author

galak
Jul 27, 2022
Collaborator

stephanosio Jul 29, 2022
Maintainer Author

stephanosio Jul 29, 2022
Maintainer Author

galak
Jul 27, 2022
Collaborator

stephanosio Jul 29, 2022
Maintainer Author

galak
Jul 27, 2022
Collaborator

stephanosio Aug 3, 2022
Maintainer Author

stephanosio Aug 4, 2022
Maintainer Author

stephanosio Aug 4, 2022
Maintainer Author

galak
Jul 27, 2022
Collaborator

stephanosio Jul 29, 2022
Maintainer Author

stephanosio Jul 29, 2022
Maintainer Author

galak
Jul 27, 2022
Collaborator

stephanosio Jul 29, 2022
Maintainer Author

stephanosio Jul 29, 2022
Maintainer Author

galak
Jul 27, 2022
Collaborator

stephanosio Jul 29, 2022
Maintainer Author

stephanosio Jul 29, 2022
Maintainer Author

galak
Jul 27, 2022
Collaborator

galak Jul 27, 2022
Collaborator

stephanosio Jul 29, 2022
Maintainer Author

galak
Jul 27, 2022
Collaborator

stephanosio Jul 29, 2022
Maintainer Author

stephanosio Jul 29, 2022
Maintainer Author

stephanosio Aug 3, 2022
Maintainer Author

stephanosio Aug 4, 2022
Maintainer Author

galak
Jul 27, 2022
Collaborator

stephanosio Jul 29, 2022
Maintainer Author

stephanosio Jul 29, 2022
Maintainer Author

carlocaione
Jul 28, 2022
Collaborator

stephanosio Jul 28, 2022
Maintainer Author

stephanosio Aug 1, 2022
Maintainer Author

stephanosio
Jul 29, 2022
Maintainer Author

stephanosio Aug 1, 2022
Maintainer Author

stephanosio
Jul 29, 2022
Maintainer Author

stephanosio Aug 3, 2022
Maintainer Author

stephanosio
Jul 29, 2022
Maintainer Author

stephanosio Aug 4, 2022
Maintainer Author

stephanosio
Jul 29, 2022
Maintainer Author

stephanosio Aug 4, 2022
Maintainer Author

keith-packard Aug 5, 2022
Collaborator

stephanosio Aug 5, 2022
Maintainer Author

keith-packard Aug 5, 2022
Collaborator

keith-packard Aug 13, 2022
Collaborator

stephanosio
Aug 10, 2022
Maintainer Author

stephanosio
Aug 10, 2022
Maintainer Author

stephanosio Aug 10, 2022
Maintainer Author

stephanosio
Aug 10, 2022
Maintainer Author

stephanosio
Aug 10, 2022
Maintainer Author

stephanosio
Aug 10, 2022
Maintainer Author

stephanosio
Aug 10, 2022
Maintainer Author

stephanosio
Aug 10, 2022
Maintainer Author

stephanosio
Aug 10, 2022
Maintainer Author

stephanosio
Aug 10, 2022
Maintainer Author

stephanosio
Aug 10, 2022
Maintainer Author