Re: [PATCH 3/3] compiler.h: add __chkp_no_side_effects() empty hint/assertion macro
Linus Torvalds <[email protected]>
| Newsgroups | org.kernel.vger.linux-sparse,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <CAHk-=wiwpTUr8keTinnPU8kTN9dpYgDtwM4wONRF_j=1gvo3MQ@mail.gmail.com> |
On Sat, 25 Oct 2025 at 14:15, Jim Cromie <[email protected]> wrote: > > Allow an author to suppress these warnings on _var by adding > '__chkp_no_side_effects(_var)' to the body of macros which trigger > that warning. This may reduce blowouts in CI pipelines. How about just not doing that checkpatch thing at all if it causes problems? Seriously, I think checkpatch often causes more problems than it fixes. If it then causes us to write even uglier macros - and it's not like our macros are pretty in general - it really is just causing pain. I think we should cut down checkpatch to things that are obvious and real issues. Not things that then cause people to make code worse. Linus