From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergey Senozhatsky Date: Tue, 12 Dec 2017 02:41:09 +0000 Subject: Re: [PATCH 13/13] arc: do not use __print_symbol() Message-Id: <20171212024109.GA7798@jagdpanzerIV> List-Id: References: <20171211125025.2270-1-sergey.senozhatsky@gmail.com> <20171211125025.2270-14-sergey.senozhatsky@gmail.com> <12708a61-ebc7-8f3f-cef6-4a725047de6d@synopsys.com> In-Reply-To: <12708a61-ebc7-8f3f-cef6-4a725047de6d@synopsys.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On (12/11/17 08:28), Vineet Gupta wrote: > On 12/11/2017 04:53 AM, Sergey Senozhatsky wrote: > > __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 > > Cc: Vineet Gupta > > Applied to arc for-curr thanks. -ss