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
| ||
|
Date: Thu, 24 Mar 2022 07:38:08 +0100 From: Helge Deller <deller@....de> To: Christophe JAILLET <christophe.jaillet@...adoo.fr> Cc: linux-kernel@...r.kernel.org, kernel-janitors@...r.kernel.org, linux-fbdev@...r.kernel.org, dri-devel@...ts.freedesktop.org Subject: Re: [PATCH] video: of: display_timing: Remove a redundant zeroing of memory On 3/22/22 21:33, Christophe JAILLET wrote: > of_parse_display_timing() already call memset(0) on its 2nd argument, so > there is no need to clear it explicitly before calling this function. > > Use kmalloc() instead of kzalloc() to save a few cycles. > > Signed-off-by: Christophe JAILLET <christophe.jaillet@...adoo.fr> applied. Thanks! Helge > --- > drivers/video/of_display_timing.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/of_display_timing.c b/drivers/video/of_display_timing.c > index f93b6abbe258..bebd371c6b93 100644 > --- a/drivers/video/of_display_timing.c > +++ b/drivers/video/of_display_timing.c > @@ -199,7 +199,7 @@ struct display_timings *of_get_display_timings(const struct device_node *np) > struct display_timing *dt; > int r; > > - dt = kzalloc(sizeof(*dt), GFP_KERNEL); > + dt = kmalloc(sizeof(*dt), GFP_KERNEL); > if (!dt) { > pr_err("%pOF: could not allocate display_timing struct\n", > np);
Powered by blists - more mailing lists