Re: [PATCH v16 01/18] seccomp: Convert __secure_computing() to return boolean
"H. Peter Anvin" <[email protected]> Fri, 03 Jul 2026 14:01:29 -0700
| Newsgroups | gmane.linux.uml.devel,gmane.linux.ports.alpha,gmane.linux.kernel,gmane.linux.ports.arm.kernel,gmane.linux.kernel.mm,gmane.linux.ports.m68k,gmane.linux.ports.mips,gmane.linux.ports.parisc,gmane.linux.ports.ppc64.devel,gmane.linux.ports.riscv,gmane.linux.ports.sh.devel |
|---|---|
| Message-ID | <[email protected]> |
On July 3, 2026 1:52:43 PM PDT, Thomas Gleixner <[email protected]> wrote: >On Fri, Jul 03 2026 at 13:59, Kevin Brodsky wrote: >> On 03/07/2026 12:27, Michal Suchánek wrote: >>>> If we're changing the return type, it might be worth renaming the >>>> function something like: >>>> >>>> seccomp_permits_syscall() >>> Then not only it is clear which way the boolean value should be >>> interpreted, it also pervents the accidental inversion of existing >>> calls. Overall great. >> >> Totally agreed, if we have the opportunity to rename a completely >> undescriptive function name like "secure_computing" we should take it. > >Absolutely no objections from my side on that. > Hell yes.