diff options
| author | Weongyo Jeong <weongyo.linux@gmail.com> | 2016-03-31 12:15:03 -0700 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2016-06-10 17:07:50 +0200 |
| commit | ff5b706f5189fe8d2a6fd576b491b769ec1d29d3 (patch) | |
| tree | b7361198a3786138d9eeab3bd25a80b5f77b2c17 /kernel/irq | |
| parent | fe3464ca8710012a247bb4586dde21b080f88514 (diff) | |
| download | op-kernel-dev-ff5b706f5189fe8d2a6fd576b491b769ec1d29d3.zip op-kernel-dev-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>
Diffstat (limited to 'kernel/irq')
| -rw-r--r-- | kernel/irq/proc.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/kernel/irq/proc.c b/kernel/irq/proc.c index 50a8f28..f30425d 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); } |
