lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <ZSs4RgtcYqkJc1Px@google.com> Date: Sat, 14 Oct 2023 17:54:30 -0700 From: Joel Becker <jlbec@...lplan.org> To: Breno Leitao <leitao@...ian.org> Cc: kuba@...nel.org, davem@...emloft.net, pabeni@...hat.com, Eric Dumazet <edumazet@...gle.com>, hch@....de, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, horms@...nel.org Subject: Re: [PATCH net-next v4 1/4] netconsole: move init/cleanup functions lower On Thu, Oct 12, 2023 at 04:13:58AM -0700, Breno Leitao wrote: > Move alloc_param_target() and its counterpart (free_param_target()) > to the bottom of the file. These functions are called mostly at > initialization/cleanup of the module, and they should be just above the > callers, at the bottom of the file. > > >From a practical perspective, having alloc_param_target() at the bottom > of the file will avoid forward declaration later (in the following > patch). > > Nothing changed other than the functions location. > > Suggested-by: Jakub Kicinski <kuba@...nel.org> > Signed-off-by: Breno Leitao <leitao@...ian.org> Reviewed-by: Joel Becker <jlbec@...lplan.org> > --- > drivers/net/netconsole.c | 104 +++++++++++++++++++-------------------- > 1 file changed, 52 insertions(+), 52 deletions(-) > > diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c > index 3111e1648592..d609fb59cf99 100644 > --- a/drivers/net/netconsole.c > +++ b/drivers/net/netconsole.c > @@ -192,58 +192,6 @@ static struct netconsole_target *alloc_and_init(void) > return nt; > } > > -/* Allocate new target (from boot/module param) and setup netpoll for it */ > -static struct netconsole_target *alloc_param_target(char *target_config) > -{ > - struct netconsole_target *nt; > - int err; > - > - nt = alloc_and_init(); > - if (!nt) { > - err = -ENOMEM; > - goto fail; > - } > - > - if (*target_config == '+') { > - nt->extended = true; > - target_config++; > - } > - > - if (*target_config == 'r') { > - if (!nt->extended) { > - pr_err("Netconsole configuration error. Release feature requires extended log message"); > - err = -EINVAL; > - goto fail; > - } > - nt->release = true; > - target_config++; > - } > - > - /* Parse parameters and setup netpoll */ > - err = netpoll_parse_options(&nt->np, target_config); > - if (err) > - goto fail; > - > - err = netpoll_setup(&nt->np); > - if (err) > - goto fail; > - > - nt->enabled = true; > - > - return nt; > - > -fail: > - kfree(nt); > - return ERR_PTR(err); > -} > - > -/* Cleanup netpoll for given target (from boot/module param) and free it */ > -static void free_param_target(struct netconsole_target *nt) > -{ > - netpoll_cleanup(&nt->np); > - kfree(nt); > -} > - > #ifdef CONFIG_NETCONSOLE_DYNAMIC > > /* > @@ -938,6 +886,58 @@ static void write_msg(struct console *con, const char *msg, unsigned int len) > spin_unlock_irqrestore(&target_list_lock, flags); > } > > +/* Allocate new target (from boot/module param) and setup netpoll for it */ > +static struct netconsole_target *alloc_param_target(char *target_config) > +{ > + struct netconsole_target *nt; > + int err; > + > + nt = alloc_and_init(); > + if (!nt) { > + err = -ENOMEM; > + goto fail; > + } > + > + if (*target_config == '+') { > + nt->extended = true; > + target_config++; > + } > + > + if (*target_config == 'r') { > + if (!nt->extended) { > + pr_err("Netconsole configuration error. Release feature requires extended log message"); > + err = -EINVAL; > + goto fail; > + } > + nt->release = true; > + target_config++; > + } > + > + /* Parse parameters and setup netpoll */ > + err = netpoll_parse_options(&nt->np, target_config); > + if (err) > + goto fail; > + > + err = netpoll_setup(&nt->np); > + if (err) > + goto fail; > + > + nt->enabled = true; > + > + return nt; > + > +fail: > + kfree(nt); > + return ERR_PTR(err); > +} > + > +/* Cleanup netpoll for given target (from boot/module param) and free it */ > +static void free_param_target(struct netconsole_target *nt) > +{ > + netpoll_cleanup(&nt->np); > + kfree(nt); > +} > + > static struct console netconsole_ext = { > .name = "netcon_ext", > .flags = CON_ENABLED | CON_EXTENDED, > -- > 2.34.1 > -- "There is shadow under this red rock. (Come in under the shadow of this red rock) And I will show you something different from either Your shadow at morning striding behind you Or your shadow at evening rising to meet you. I will show you fear in a handful of dust." http://www.jlbec.org/ jlbec@...lplan.org
Powered by blists - more mailing lists