aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWeongyo Jeong <weongyo.linux@gmail.com>2016-03-31 12:15:03 -0700
committerThomas Gleixner <tglx@linutronix.de>2016-06-10 17:07:50 +0200
commitff5b706f5189fe8d2a6fd576b491b769ec1d29d3 (patch)
treeb7361198a3786138d9eeab3bd25a80b5f77b2c17
parentfe3464ca8710012a247bb4586dde21b080f88514 (diff)
downloadlinux-stericsson-ff5b706f5189fe8d2a6fd576b491b769ec1d29d3.tar.gz
genirq: Remove unnecessary memset() calls
sprintf() and snprintf() implementation of kernel guarantees that its result is terminated with null byte if size is larger than 0. So we don't need to call memset() at all. Signed-off-by: Weongyo Jeong <weongyo.linux@gmail.com> Link: http://lkml.kernel.org/r/1459451703-5744-1-git-send-email-weongyo.linux@gmail.com Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
-rw-r--r--kernel/irq/proc.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/kernel/irq/proc.c b/kernel/irq/proc.c
index 50a8f28be247..f30425dce9dd 100644
--- a/kernel/irq/proc.c
+++ b/kernel/irq/proc.c
@@ -311,7 +311,6 @@ void register_handler_proc(unsigned int irq, struct irqaction *action)
!name_unique(irq, action))
return;
- memset(name, 0, MAX_NAMELEN);
snprintf(name, MAX_NAMELEN, "%s", action->name);
/* create /proc/irq/1234/handler/ */
@@ -340,7 +339,6 @@ void register_irq_proc(unsigned int irq, struct irq_desc *desc)
if (desc->dir)
goto out_unlock;
- memset(name, 0, MAX_NAMELEN);
sprintf(name, "%d", irq);
/* create /proc/irq/1234 */
@@ -386,7 +384,6 @@ void unregister_irq_proc(unsigned int irq, struct irq_desc *desc)
#endif
remove_proc_entry("spurious", desc->dir);
- memset(name, 0, MAX_NAMELEN);
sprintf(name, "%u", irq);
remove_proc_entry(name, root_irq_dir);
}