All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 00/13] replace print_symbol() with printk()-s
Date: Wed, 20 Dec 2017 10:20:34 +0000	[thread overview]
Message-ID: <20171220102034.GG11774@jagdpanzerIV> (raw)
In-Reply-To: <20171211125025.2270-1-sergey.senozhatsky@gmail.com>

On (12/11/17 21:50), Sergey Senozhatsky wrote:
> 
> 	A rather automatic replacement of print_symbol()
> with direct printk() calls. print_symbol() uses extra stack
> buffer (KSYM_SYMBOL_LEN 128 bytes) and, basically, should
> be identical to printk(%pS).
> 
> 	I can't test all of the patches, because I don't
> own any of those exotic arch-s. Sorry for the inconvenience.
> 
> Sergey Senozhatsky (13):
>   arm: do not use print_symbol()
>   arm64: do not use print_symbol()
>   c6x: do not use print_symbol()
>   ia64: do not use print_symbol()
>   mn10300: do not use print_symbol()
>   sh: do not use print_symbol()
>   unicore32: do not use print_symbol()
>   x86: do not use print_symbol()
>   drivers: do not use print_symbol()
>   sysfs: do not use print_symbol()
>   irq debug: do not use print_symbol()
>   lib: do not use print_symbol()
>   arc: do not use __print_symbol()

Hello,

can we please have more reviews/acks/etc?

	-ss

WARNING: multiple messages have this Message-ID (diff)
From: sergey.senozhatsky.work@gmail.com (Sergey Senozhatsky)
To: linux-snps-arc@lists.infradead.org
Subject: [PATCH 00/13] replace print_symbol() with printk()-s
Date: Wed, 20 Dec 2017 19:20:34 +0900	[thread overview]
Message-ID: <20171220102034.GG11774@jagdpanzerIV> (raw)
In-Reply-To: <20171211125025.2270-1-sergey.senozhatsky@gmail.com>

On (12/11/17 21:50), Sergey Senozhatsky wrote:
> 
> 	A rather automatic replacement of print_symbol()
> with direct printk() calls. print_symbol() uses extra stack
> buffer (KSYM_SYMBOL_LEN 128 bytes) and, basically, should
> be identical to printk(%pS).
> 
> 	I can't test all of the patches, because I don't
> own any of those exotic arch-s. Sorry for the inconvenience.
> 
> Sergey Senozhatsky (13):
>   arm: do not use print_symbol()
>   arm64: do not use print_symbol()
>   c6x: do not use print_symbol()
>   ia64: do not use print_symbol()
>   mn10300: do not use print_symbol()
>   sh: do not use print_symbol()
>   unicore32: do not use print_symbol()
>   x86: do not use print_symbol()
>   drivers: do not use print_symbol()
>   sysfs: do not use print_symbol()
>   irq debug: do not use print_symbol()
>   lib: do not use print_symbol()
>   arc: do not use __print_symbol()

Hello,

can we please have more reviews/acks/etc?

	-ss

WARNING: multiple messages have this Message-ID (diff)
From: sergey.senozhatsky.work@gmail.com (Sergey Senozhatsky)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 00/13] replace print_symbol() with printk()-s
Date: Wed, 20 Dec 2017 19:20:34 +0900	[thread overview]
Message-ID: <20171220102034.GG11774@jagdpanzerIV> (raw)
In-Reply-To: <20171211125025.2270-1-sergey.senozhatsky@gmail.com>

On (12/11/17 21:50), Sergey Senozhatsky wrote:
> 
> 	A rather automatic replacement of print_symbol()
> with direct printk() calls. print_symbol() uses extra stack
> buffer (KSYM_SYMBOL_LEN 128 bytes) and, basically, should
> be identical to printk(%pS).
> 
> 	I can't test all of the patches, because I don't
> own any of those exotic arch-s. Sorry for the inconvenience.
> 
> Sergey Senozhatsky (13):
>   arm: do not use print_symbol()
>   arm64: do not use print_symbol()
>   c6x: do not use print_symbol()
>   ia64: do not use print_symbol()
>   mn10300: do not use print_symbol()
>   sh: do not use print_symbol()
>   unicore32: do not use print_symbol()
>   x86: do not use print_symbol()
>   drivers: do not use print_symbol()
>   sysfs: do not use print_symbol()
>   irq debug: do not use print_symbol()
>   lib: do not use print_symbol()
>   arc: do not use __print_symbol()

Hello,

can we please have more reviews/acks/etc?

	-ss

WARNING: multiple messages have this Message-ID (diff)
From: Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com>
To: Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Russell King <linux@armlinux.org.uk>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Mark Salter <msalter@redhat.com>, Tony Luck <tony.luck@intel.com>,
	David Howells <dhowells@redhat.com>,
	Yoshinori Sato <ysato@users.sourceforge.jp>,
	Guan Xuetao <gxt@mprc.pku.edu.cn>, Borislav Petkov <bp@alien8.de>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Peter Zijlstra <peterz@infradead.org>,
	Vineet Gupta <vgupta@synopsys.com>,
	Fengguang Wu <fengguang.wu@intel.com>,
	Steven Rostedt <rostedt@goodmis.org>,
	Petr Mladek <pmladek@suse.com>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-arm-kernel@lists.infradead.org,
	linux-c6x-dev@linux-c6x.org, linux-ia64@vger.kernel.org,
	linux-am33-list@redhat.com, linux-sh@vger.kernel.org,
	linux-edac@vger.kernel.org, x86@kernel.org,
	linux-snps-arc@lists.infradead.org,
	Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Subject: Re: [PATCH 00/13] replace print_symbol() with printk()-s
Date: Wed, 20 Dec 2017 19:20:34 +0900	[thread overview]
Message-ID: <20171220102034.GG11774@jagdpanzerIV> (raw)
In-Reply-To: <20171211125025.2270-1-sergey.senozhatsky@gmail.com>

On (12/11/17 21:50), Sergey Senozhatsky wrote:
> 
> 	A rather automatic replacement of print_symbol()
> with direct printk() calls. print_symbol() uses extra stack
> buffer (KSYM_SYMBOL_LEN 128 bytes) and, basically, should
> be identical to printk(%pS).
> 
> 	I can't test all of the patches, because I don't
> own any of those exotic arch-s. Sorry for the inconvenience.
> 
> Sergey Senozhatsky (13):
>   arm: do not use print_symbol()
>   arm64: do not use print_symbol()
>   c6x: do not use print_symbol()
>   ia64: do not use print_symbol()
>   mn10300: do not use print_symbol()
>   sh: do not use print_symbol()
>   unicore32: do not use print_symbol()
>   x86: do not use print_symbol()
>   drivers: do not use print_symbol()
>   sysfs: do not use print_symbol()
>   irq debug: do not use print_symbol()
>   lib: do not use print_symbol()
>   arc: do not use __print_symbol()

Hello,

can we please have more reviews/acks/etc?

	-ss

  parent reply	other threads:[~2017-12-20 10:20 UTC|newest]

Thread overview: 180+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-11 12:50 [PATCH 00/13] replace print_symbol() with printk()-s Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 16:26 ` Joe Perches
2017-12-11 16:26   ` Joe Perches
2017-12-11 16:26   ` Joe Perches
2017-12-11 16:26   ` Joe Perches
2017-12-20 10:20 ` Sergey Senozhatsky [this message]
2017-12-20 10:20   ` Sergey Senozhatsky
2017-12-20 10:20   ` Sergey Senozhatsky
2017-12-20 10:20   ` Sergey Senozhatsky
2018-01-05 10:03 ` Petr Mladek
2018-01-05 10:03   ` Petr Mladek
2018-01-05 10:03   ` Petr Mladek
2018-01-05 10:03   ` Petr Mladek
2018-01-05 10:21   ` Sergey Senozhatsky
2018-01-05 10:21     ` Sergey Senozhatsky
2018-01-05 10:21     ` Sergey Senozhatsky
2018-01-05 10:21     ` Sergey Senozhatsky
2018-01-05 11:38     ` Petr Mladek
2018-01-05 11:38       ` Petr Mladek
2018-01-05 11:38       ` Petr Mladek
2018-01-05 11:38       ` Petr Mladek
2018-01-05 12:01     ` Sergey Senozhatsky
2018-01-05 12:01       ` Sergey Senozhatsky
2018-01-05 12:01       ` Sergey Senozhatsky
2018-01-05 12:01       ` Sergey Senozhatsky
2018-01-05 12:23       ` Sergey Senozhatsky
2018-01-05 12:23         ` Sergey Senozhatsky
2018-01-05 12:23         ` Sergey Senozhatsky
2018-01-05 12:23         ` Sergey Senozhatsky
2018-01-05 13:09         ` Petr Mladek
2018-01-05 13:09           ` Petr Mladek
2018-01-05 13:09           ` Petr Mladek
2018-01-05 13:09           ` Petr Mladek
  -- strict thread matches above, loose matches on Subject: below --
2017-12-11 12:50 [01/13] arm: do not use print_symbol() Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 01/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [02/13] arm64: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 02/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [03/13] c6x: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 03/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [04/13] ia64: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 04/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [05/13] mn10300: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 05/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [06/13] sh: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 06/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [07/13] unicore32: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 07/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [08/13] x86: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 08/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [09/13] drivers: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 09/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [10/13] sysfs: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 10/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [11/13] irq debug: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 11/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [12/13] lib: " Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 12/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 [13/13] arc: do not use __print_symbol() Sergey Senozhatsky
2017-12-11 12:50 ` [PATCH 13/13] " Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:50 ` Sergey Senozhatsky
2017-12-11 12:55 [11/13] irq debug: do not use print_symbol() David Laight
2017-12-11 12:55 ` [PATCH 11/13] " David Laight
2017-12-11 12:55 ` David Laight
2017-12-11 12:55 ` David Laight
2017-12-11 12:55 ` David Laight
2017-12-11 16:28 [13/13] arc: do not use __print_symbol() Vineet Gupta
2017-12-11 16:28 ` [PATCH 13/13] " Vineet Gupta
2017-12-11 16:28 ` Vineet Gupta
2017-12-11 16:28 ` Vineet Gupta
2017-12-11 16:28 ` Vineet Gupta
2017-12-11 17:45 [08/13] x86: do not use print_symbol() Borislav Petkov
2017-12-11 17:45 ` [PATCH 08/13] " Borislav Petkov
2017-12-11 17:45 ` Borislav Petkov
2017-12-11 17:45 ` Borislav Petkov
2017-12-11 17:45 ` Borislav Petkov
2017-12-12  2:41 [13/13] arc: do not use __print_symbol() Sergey Senozhatsky
2017-12-12  2:41 ` [PATCH 13/13] " Sergey Senozhatsky
2017-12-12  2:41 ` Sergey Senozhatsky
2017-12-12  2:41 ` Sergey Senozhatsky
2017-12-12  2:41 ` Sergey Senozhatsky
2017-12-12  2:41 [08/13] x86: do not use print_symbol() Sergey Senozhatsky
2017-12-12  2:41 ` [PATCH 08/13] " Sergey Senozhatsky
2017-12-12  2:41 ` Sergey Senozhatsky
2017-12-12  2:41 ` Sergey Senozhatsky
2017-12-12  2:41 ` Sergey Senozhatsky
2017-12-12  2:47 [00/13] replace print_symbol() with printk()-s Sergey Senozhatsky
2017-12-12  2:47 ` [PATCH 00/13] " Sergey Senozhatsky
2017-12-12  2:47 ` Sergey Senozhatsky
2017-12-12  2:47 ` Sergey Senozhatsky
2017-12-12  2:47 ` Sergey Senozhatsky
2017-12-12  2:50 [11/13] irq debug: do not use print_symbol() Sergey Senozhatsky
2017-12-12  2:50 ` [PATCH 11/13] " Sergey Senozhatsky
2017-12-12  2:50 ` Sergey Senozhatsky
2017-12-12  2:50 ` Sergey Senozhatsky
2017-12-12  2:50 ` Sergey Senozhatsky
2017-12-12  3:10 [00/13] replace print_symbol() with printk()-s Joe Perches
2017-12-12  3:10 ` [PATCH 00/13] " Joe Perches
2017-12-12  3:10 ` Joe Perches
2017-12-12  3:10 ` Joe Perches
2017-12-12  3:10 ` Joe Perches
2017-12-12  7:34 [PATCHv2,11/13] irq debug: do not use print_symbol() Sergey Senozhatsky
2017-12-12  7:34 ` [PATCHv2 11/13] " Sergey Senozhatsky
2017-12-12  7:34 ` Sergey Senozhatsky
2017-12-12  7:34 ` Sergey Senozhatsky
2017-12-12  7:34 ` Sergey Senozhatsky
2017-12-21  5:54 [00/13] replace print_symbol() with printk()-s Sergey Senozhatsky
2017-12-21  5:54 ` [PATCH 00/13] " Sergey Senozhatsky
2017-12-21  5:54 ` Sergey Senozhatsky
2017-12-21  5:54 ` Sergey Senozhatsky
2017-12-21  5:54 ` Sergey Senozhatsky
2018-01-05 10:25 [00/13] " Sergey Senozhatsky
2018-01-05 10:25 ` [PATCH 00/13] " Sergey Senozhatsky
2018-01-05 10:25 ` Sergey Senozhatsky
2018-01-05 10:25 ` Sergey Senozhatsky
2018-01-05 10:25 ` Sergey Senozhatsky
2018-01-05 14:42 [00/13] " Petr Mladek
2018-01-05 14:42 ` [PATCH 00/13] " Petr Mladek
2018-01-05 14:42 ` Petr Mladek
2018-01-05 14:42 ` Petr Mladek
2018-01-05 14:42 ` Petr Mladek
2018-01-05 14:57 [00/13] " Sergey Senozhatsky
2018-01-05 14:57 ` [PATCH 00/13] " Sergey Senozhatsky
2018-01-05 14:57 ` Sergey Senozhatsky
2018-01-05 14:57 ` Sergey Senozhatsky
2018-01-05 14:57 ` Sergey Senozhatsky
2018-01-08  2:09 [00/13] " Sergey Senozhatsky
2018-01-08  2:09 ` [PATCH 00/13] " Sergey Senozhatsky
2018-01-08  2:09 ` Sergey Senozhatsky
2018-01-08  2:09 ` Sergey Senozhatsky
2018-01-08  2:09 ` Sergey Senozhatsky
2018-01-16 16:33 [00/13] " Petr Mladek
2018-01-16 16:33 ` [PATCH 00/13] " Petr Mladek
2018-01-16 16:33 ` Petr Mladek
2018-01-16 16:33 ` Petr Mladek
2018-01-17  2:36 [00/13] " Sergey Senozhatsky
2018-01-17  2:36 ` [PATCH 00/13] " Sergey Senozhatsky
2018-01-17  2:36 ` Sergey Senozhatsky
2018-01-17  2:36 ` Sergey Senozhatsky
2018-01-17  2:36 ` Sergey Senozhatsky

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171220102034.GG11774@jagdpanzerIV \
    --to=sergey.senozhatsky.work@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.