Messages in this thread | | | Date | Mon, 2 Oct 2023 10:17:46 +0200 | Subject | Re: [PATCH 04/15] tty: Remove now superfluous sentinel element from ctl_table array | From | Jiri Slaby <> |
| |
On 28. 09. 23, 15:21, Joel Granados via B4 Relay wrote: > From: Joel Granados <j.granados@samsung.com> > > This commit comes at the tail end of a greater effort to remove the > empty elements at the end of the ctl_table arrays (sentinels) which > will reduce the overall build time size of the kernel and run time > memory bloat by ~64 bytes per sentinel (further information Link : > https://lore.kernel.org/all/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/) > > Remove sentinel from tty_table > > Signed-off-by: Joel Granados <j.granados@samsung.com> > --- > drivers/tty/tty_io.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/tty/tty_io.c b/drivers/tty/tty_io.c > index 8a94e5a43c6d..2f925dc54a20 100644 > --- a/drivers/tty/tty_io.c > +++ b/drivers/tty/tty_io.c > @@ -3607,8 +3607,7 @@ static struct ctl_table tty_table[] = { > .proc_handler = proc_dointvec, > .extra1 = SYSCTL_ZERO, > .extra2 = SYSCTL_ONE, > - }, > - { } > + }
Why to remove the comma? One would need to add one when adding a new entry?
thanks, -- js suse labs
| |