Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Building with W=1? #60

Open
aleksamagicka opened this issue Dec 3, 2023 · 1 comment
Open

Building with W=1? #60

aleksamagicka opened this issue Dec 3, 2023 · 1 comment

Comments

@aleksamagicka
Copy link
Member

Just found a very interesting discussion starting here, related to the warnings that the 0-day bot finds and how to find them before sending the patch upstream.

Guenter then summed up his requirements. We already have checkpatch CI here, so adding W=1 seems like a good thing to add as well.

Thoughts? Maybe also add C=1 (that invokes sparse)?

@jonasmalacofilho
Copy link
Member

[... rant omitted ...]

In that case, we should add both W=1 and C=1... and the other checkers too, if possible.

You must build your new driver with W=1, smatch, sparse and coccinelle before sending upstream.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants