Re: [PATCH 11/13] irq debug: do not use print_symbol()

2017-12-11 Thread Sergey Senozhatsky
On (12/11/17 12:55), David Laight wrote: > > kernel/irq/debug.h | 8 +++- > > 1 file changed, 3 insertions(+), 5 deletions(-) > > > > diff --git a/kernel/irq/debug.h b/kernel/irq/debug.h > > index 17f05ef8f575..5766e15c1160 100644 > > --- a/kernel/irq/debug.h > > +++ b/kernel/irq/debug.h >

RE: [PATCH 11/13] irq debug: do not use print_symbol()

2017-12-11 Thread David Laight
From: Sergey Senozhatsky > Sent: 11 December 2017 12:50 > print_symbol() uses extra stack space to sprintf() symbol > information and then to feed that buffer to printk() > > char buffer[KSYM_SYMBOL_LEN]; > > sprint_symbol(buffer, address); > printk(fmt, buffer); > > Replace

[PATCH 11/13] irq debug: do not use print_symbol()

2017-12-11 Thread Sergey Senozhatsky
print_symbol() uses extra stack space to sprintf() symbol information and then to feed that buffer to printk() char buffer[KSYM_SYMBOL_LEN]; sprint_symbol(buffer, address); printk(fmt, buffer); Replace print_symbol() with a direct printk("%pS") call. Signed-off-by: Sergey Senozhatsky