All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Nikita V. Youshchenko" <yoush@cs.msu.su>
To: Thomas Gleixner <tglx@linutronix.de>
Cc: Sujit K M <sjt.kar@gmail.com>,
	linux-rt-users@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: PREEMPT_RT (2.6.33-rt17) disabled printk-to-console after console_init
Date: Thu, 20 May 2010 14:23:07 +0400	[thread overview]
Message-ID: <201005201423.09075@zigzag.lvk.cs.msu.su> (raw)
In-Reply-To: <alpine.LFD.2.00.1005201142360.3368@localhost.localdomain>

> > >>> Well, obviously it's unsafe if you remove safety checks. And if
> > >>> you care to look at the changelog of kernel/printk.c you'll find
> > >>> out why.
> > >>
> > >> Hmm... did a quick look and could not find anything related there.
> > >> Could you please give a pointer?
>
> Gah, yes. The changelog of the commit is not really helpful. Let me
> explain:
>
> The console drivers might take locks, which are converted to "sleeping
> locks" in preempt-rt. As a result we cannot call into those drivers
> from atomic contexts. And that's what the checks in the printk code
> prevent.

I've already understood that when looking at that code some weeks ago.

Still questions:

1) why does that prevent klogd from working?

2) why does print not pass to non-CON_ATOMIC even if called from non-atomic 
context?

3) I believe that 8250 serial driver is aware of preempt-rt.
Could you please comment on my "2.6.33.2-rt13: RFC: fix serial console" 
post to linux-rt-users list 
(http://eeek.borgchat.net/lists/linux-rt-users/msg05569.html)

Nikita

  reply	other threads:[~2010-05-20 10:23 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-18 13:13 PREEMPT_RT (2.6.33-rt17) disabled printk-to-console after console_init Xianghua Xiao
2010-05-18 13:13 ` Xianghua Xiao
2010-05-18 19:28 ` Xianghua Xiao
2010-05-18 19:28   ` Xianghua Xiao
2010-05-18 19:47   ` Thomas Gleixner
2010-05-18 20:49     ` Xianghua Xiao
2010-05-18 20:49       ` Xianghua Xiao
2010-05-20  8:33     ` Nikita V. Youshchenko
2010-05-20  9:25       ` Sujit K M
2010-05-20  9:25         ` Sujit K M
2010-05-20  9:32         ` Sujit K M
2010-05-20  9:32           ` Sujit K M
2010-05-20  9:47           ` Thomas Gleixner
2010-05-20 10:23             ` Nikita V. Youshchenko [this message]
2010-05-20 10:52               ` Thomas Gleixner
2010-05-20 11:12                 ` Nikita V. Youshchenko
2010-05-20 11:23                   ` Thomas Gleixner
2010-05-20 11:40                     ` Nikita V. Youshchenko

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=201005201423.09075@zigzag.lvk.cs.msu.su \
    --to=yoush@cs.msu.su \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rt-users@vger.kernel.org \
    --cc=sjt.kar@gmail.com \
    --cc=tglx@linutronix.de \
    /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.