[LTP] [PATCH v2 1/8] Add Sparse based checker and TST_RET/ERR check

Joerg Vehlow lkml@jv-coder.de
Wed Jul 14 13:21:21 CEST 2021


Hi,


On 7/14/2021 9:11 AM, Richard Palethorpe via ltp wrote:
> Vendors in Sparse as a git module. Then uses it to check for stores to
> TST_RET/ERR within the library.
>
> Signed-off-by: Richard Palethorpe <rpalethorpe@suse.com>
> ---
>   .gitmodules             |   3 +
>   tools/Makefile          |   2 +
>   tools/sparse/.gitignore |   1 +
>   tools/sparse/Makefile   |  27 ++++++++
>   tools/sparse/README.md  |  38 +++++++++++
>   tools/sparse/main.c     | 148 ++++++++++++++++++++++++++++++++++++++++
I think the name could be a bit better... sparse_main or ltp_checker or 
something, because a binary just called main could be confusing.

Otherwise good basis for additional checks and fixing the warnings 
emitted by sparse will allow use to raise the overall warning level.
Hopefully at some point at least "-Wall -Werror" is possible.

Joerg


More information about the ltp mailing list