All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com>
To: Jiri Kosina <jikos@kernel.org>
Cc: theodore.tso@gmail.com, ksummit-discuss@lists.linuxfoundation.org
Subject: Re: [Ksummit-discuss] Late KS topics
Date: Wed, 28 Oct 2015 08:53:51 +0900	[thread overview]
Message-ID: <20151027235351.GA4885@swordfish> (raw)
In-Reply-To: <alpine.LNX.2.00.1510262348190.8998@pobox.suse.cz>

On (10/26/15 23:51), Jiri Kosina wrote:
> On Mon, 26 Oct 2015, Hannes Reinecke wrote:
> 
> >   The talk/proposal should be used to come to a consensus about
> >   restricting printk() to high-priority, small volume messages,
> >   and implement a different call (like log_printk()) for the
> >   low-priority, high-volume traffic.
> 
> The printk() troubles are not only about the "high volume" vs. "very 
> important", but also about the context of the caller.

sometimes "high volume" and 'unlucky' printk_deferred() dynamic duo
can do dramatic things. printk_deferred() queues a irq work -
wake_up_klogd_work(), which does

wake_up_klogd_work_func():
	..
	console_trylock()
		console_unlock()


so we eventually end up in

	for (;;) {
		..
		call_console_drivers()
		..
	}

	-ss

> The optimistic comment above printk() says that it can be called from any 
> context, but that's not true for quite some time already.
> So if printk() is going to be redesigned to be more lightweight, this 
> aspect definitely needs to be taken into consideration as well.

  reply	other threads:[~2015-10-27 23:52 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-26 14:25 [Ksummit-discuss] Late KS topics Hannes Reinecke
2015-10-26 14:49 ` David Woodhouse
2015-10-26 14:51 ` Jiri Kosina
2015-10-27 23:53   ` Sergey Senozhatsky [this message]
2015-10-26 16:14 ` Luck, Tony
2015-10-27  0:23   ` Hannes Reinecke
2015-10-27  0:12 ` Mark Brown
     [not found]   ` <20151027005950.GA26445@localhost>
2015-10-27  1:39     ` [Ksummit-discuss] Deferred probing Mark Brown

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=20151027235351.GA4885@swordfish \
    --to=sergey.senozhatsky.work@gmail.com \
    --cc=jikos@kernel.org \
    --cc=ksummit-discuss@lists.linuxfoundation.org \
    --cc=theodore.tso@gmail.com \
    /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.