tools: Fix rtla and rv problems (found) with clang
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest6 patches 2 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/6 | tools/rtla: Fix Makefile compiler options for clangtools/ | +6 | 2 replies | |
| 2/6 | tools/rtla: Fix uninitialized bucket/data->bucket_size warningtools/ tools/ | +2 | 0 replies | |
| 3/6 | tools/rtla: Fix clang warning about mount_point var sizetools/ | +1 | 0 replies | |
| 4/6 | tools/rtla: Remove unused sched_getattr() functiontools/ | +0 | 0 replies | |
| 5/6 | tools/rv: Fix Makefile compiler options for clangtools/ | +6 | 0 replies | |
| 6/6 | tools/rv: Fix curr_reactor uninitialized variabletools/ | +1 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/6] tools: Fix rtla and rv problems (found) with clang
RHEL people reported some errors when compiling rtla and rv with clang. The command line used to compile the tools is:
$ make HOSTCC=clang CC=clang LLVM_IAS=1
The first problem is two unsupported flags passed to the compiler:
-ffat-lto-objects and -Wno-maybe-uninitialized. They will be removed if the compile is clang.
Also, the clang linker does not automatically recognize the -flto=auto option used at compilation time, so it is explicitly set.
With the compiler working, it starts pointing to some warnings and errors about uninitialized variables, variable size, and an unused function. These problems are also fixed.
Daniel Bristot de Oliveira (6):
tools/rtla: Fix Makefile compiler options for clang tools/rtla: Fix uninitialized bucket/data->bucket_size warning tools/rtla: Fix clang warning about mount_point var size tools/rtla: Remove unused sched_getattr() function tools/rv: Fix Makefile compiler options for clang tools/rv: Fix curr_reactor uninitialized variable
tools/tracing/rtla/Makefile | 7 ++++++- tools/tracing/rtla/src/osnoise_hist.c | 3 +-- tools/tracing/rtla/src/timerlat_hist.c | 3 +-- tools/tracing/rtla/src/utils.c | 8 +------- tools/verification/rv/Makefile | 7 ++++++- tools/verification/rv/src/in_kernel.c | 2 +- 6 files changed, 16 insertions(+), 14 deletions(-)
Signature · 1 line
2.43.0