summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYann Collet <cyan@fb.com>2022-09-09 18:05:32 (GMT)
committerYann Collet <cyan@fb.com>2022-09-09 18:05:32 (GMT)
commit9bed6b56e9beb1cf820f01e47e70a13442d33544 (patch)
tree6f0506e758c8869cf689c6d63510e84b221d44fa
parent46ecb595d86defab7fa869ebee45b80b189a3331 (diff)
downloadlz4-9bed6b56e9beb1cf820f01e47e70a13442d33544.zip
lz4-9bed6b56e9beb1cf820f01e47e70a13442d33544.tar.gz
lz4-9bed6b56e9beb1cf820f01e47e70a13442d33544.tar.bz2
updated Github Actions tests documentation
notably regarding ubsan test status (now enabled)
-rw-r--r--.github/workflows/README.md23
1 files changed, 12 insertions, 11 deletions
diff --git a/.github/workflows/README.md b/.github/workflows/README.md
index 306d875..9ad2d99 100644
--- a/.github/workflows/README.md
+++ b/.github/workflows/README.md
@@ -4,12 +4,17 @@ This directory contains [GitHub Actions](https://github.com/features/actions) wo
## USAN, ASAN (`lz4-ubsan-x64`, `lz4-ubsan-x86`, `lz4-asan-x64`)
-For now, `lz4-ubsan-*` ignores the exit code of `make usan` and `make usan32`.
-Because there are several issues which may take relatively long time to resolve.
-
-We'll fully enable it when we ensure `make usan` is ready for all commits and PRs.
-
-See [#983](https://github.com/lz4/lz4/pull/983) for details.
+For now, `lz4-ubsan-*` uses the `-fsanitize-recover=pointer-overflow` flag:
+there are known cases of pointer overflow arithmetic within `lz4.c` fast compression.
+These cases are not dangerous with current architecture,
+but they are not guaranteed to work by the C standard,
+which means that, in some future, some new architecture or some new compiler
+may decide to do something funny that would break this behavior.
+Hence it's proper to remove them.
+This has been done in `lz4hc.c`.
+However, the same attempt in `lz4.c` resulted in massive speed loss,
+which is not acceptable to solve a "potential future" problem that does not exist anywhere today.
+Therefore, a better work-around will have to be found.
## C Compilers (`lz4-c-compilers`)
@@ -50,11 +55,7 @@ Also sometimes it reports false positives.
# Difference with `.travis.yml`
-The following tests are not included yet.
-
-- name: Compile OSS-Fuzz targets
-
-The following tests will not be included due to limitation of GH-Actions.
+The following tests are not be included due to limitation of GH-Actions.
- name: aarch64 real-hw tests
- name: PPC64LE real-hw tests