Re: Re: Re: Re: [PATCH v2] treewide: const qualify ctl_tables where applicable
Joel Granados <[email protected]>
| Newsgroups | gmane.comp.freedesktop.xorg.drivers.intel,gmane.linux.ports.arm.kernel,gmane.linux.kernel,gmane.linux.ports.ppc64.devel,gmane.linux.ports.riscv,gmane.linux.kernel.cryptoapi,gmane.linux.kernel.openipmi,gmane.comp.video.dri.devel,gmane.linux.drivers.rdma,gmane.linux.raid,gmane.linux.scsi,gmane.linux.serial,gmane.comp.emulators.xen.devel,gmane.linux.kernel.aio.general,gmane.linux.file-systems,gmane.comp.file-systems.coda.general,gmane.linux.kernel.mm,gmane.linux.nfs,gmane.linux.kernel.io-uring,gmane.linux.kernel.bpf,gmane.linux.kernel.kexec,gmane.comp.security.apparmor,gmane.linux.kernel.lsm |
|---|---|
| Message-ID | <u2fwibsnbfvulxj6adigla6geiafh2vuve4hcyo4vmeytwjl7p@oz6xonrq5225> |
On Mon, Jan 27, 2025 at 03:42:39PM +0000, Matthew Wilcox wrote: > On Mon, Jan 27, 2025 at 04:55:58PM +0200, Jani Nikula wrote: > > You could have static const within functions too. You get the rodata > > protection and function local scope, best of both worlds? > > timer_active is on the stack, so it can't be static const. > > Does this really need to be cc'd to such a wide distribution list? That is a very good question. I removed 160 people from the original e-mail and left the ones that where previously involved with this patch and left all the lists for good measure. But it seems I can reduce it even more. How about this: For these treewide efforts I just leave the people that are/were involved in the series and add two lists: linux-kernel and linux-hardening. Unless someone screams, I'll try this out on my next treewide. Thx for the feedback Best -- Joel Granados