Skip to content

Commit

Permalink
Removed the enable sanitizers check from qemu dockerfile to fix the Q…
Browse files Browse the repository at this point in the history
…EMU CI failure (#32685)

- The tests before disabling the QEMU tests in CI were failing.
- When we run QEMU with address sanitizers enabled ,we get following warning at the start
WARNING: ASan doesn't fully support makecontext/swapcontext functions and may produce false positives in some cases!.
- For some case, this features works fine with Clang toolchain but has issues with GNU.
- Also it might consume more CPU or memory resources which might affect the CI
- Updated the chip build version and removed the address sanitizers from QEMU docker image.
  • Loading branch information
shripad621git authored and pull[bot] committed Mar 29, 2024
1 parent 88d0b72 commit 3675509
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions integrations/docker/images/base/chip-build/version
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
45 : [TI] Update Sysconfig Version to 1.18.1

46 : [ESP32] Updated the QEMU Dockerfile.
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ WORKDIR /opt/espressif/qemu
# Setup QEMU emulator for ESP32 platform
RUN set -x \
&& git clone --depth 1 -b esp-develop-20210826 https://github.com/espressif/qemu.git ../qemu-src \
&& ../qemu-src/configure --target-list=xtensa-softmmu --enable-debug --enable-sanitizers --disable-strip --disable-user --disable-capstone --disable-vnc --disable-sdl --disable-gtk \
&& ../qemu-src/configure --target-list=xtensa-softmmu --enable-debug --disable-strip --disable-user --disable-capstone --disable-vnc --disable-sdl --disable-gtk \
&& make -j8 \
&& : # last line

Expand Down

0 comments on commit 3675509

Please sign in to comment.