From: Frank Rowand <frank.rowand@am.sony.com>
To: Michael Thalmeier <michael.thalmeier@hale.at>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
<rostedt@goodmis.org>, <stable-rt@vger.kernel.org>
Subject: Re: [PATCH RT 2/2] fix printk flush of messages
Date: Tue, 21 Aug 2012 15:56:16 -0700 [thread overview]
Message-ID: <50341210.9040803@am.sony.com> (raw)
In-Reply-To: <loom.20120821T161458-427@post.gmane.org>
added recipients...
On 08/21/12 07:30, Michael Thalmeier wrote:
> Frank Rowand <frank.rowand <at> am.sony.com> writes:
>
>>
>>
>> Updates console-make-rt-friendly.patch
>>
>> #ifdef CONFIG_PREEMPT_RT_FULL, printk() output is never flushed by
>> printk() because:
>> ...
>>
>> On system boot some printk() output is flushed because register_console()
>> and tty_open() call console_unlock().
>>
>> This change also fixes the problem that was previously fixed by
>> preempt-rt-allow-immediate-magic-sysrq-output-for-preempt_rt_full.patch
>>
>> Signed-off-by: Frank Rowand <frank.rowand <at> am.sony.com>
>>
>> ---
>> kernel/printk.c | 2 1 + 1 - 0 !
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> Index: b/kernel/printk.c
>> ===================================================================
>> --- a/kernel/printk.c
>> +++ b/kernel/printk.c
>> @@ -847,7 +847,7 @@ static int console_trylock_for_printk(un
>> int retval = 0, wake = 0;
>> #ifdef CONFIG_PREEMPT_RT_FULL
>> int lock = !early_boot_irqs_disabled && !irqs_disabled_flags(flags) &&
>> - !preempt_count();
>> + (preempt_count() <= 1);
>> #else
>> int lock = 1;
>> #endif
>>
>>
>
> I have seen that this patch is applied in the 3.4 stable rt series.
> As we are using the 3.0 stable rt kernel I have tested this patch on this
> kernel series (on a Freescale i.MX31 based board) and have not found any
> problems so far.
> Is there something I have missed why this patch has not found its way
> in the 3.0 series ?
>
> Thanks in advance,
> Michael
prev parent reply other threads:[~2012-08-21 22:56 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-17 1:05 [PATCH RT 1/2] fix printk flush of messages Frank Rowand
2012-05-17 1:09 ` [PATCH RT 2/2] " Frank Rowand
2012-05-17 1:17 ` Frank Rowand
2012-05-21 20:10 ` Venkat Subbiah
2012-05-21 20:59 ` Frank Rowand
2012-08-21 14:30 ` Michael Thalmeier
2012-08-21 22:56 ` Frank Rowand [this message]
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=50341210.9040803@am.sony.com \
--to=frank.rowand@am.sony.com \
--cc=linux-kernel@vger.kernel.org \
--cc=michael.thalmeier@hale.at \
--cc=rostedt@goodmis.org \
--cc=stable-rt@vger.kernel.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.