lkml.org 
[lkml]   [2011]   [Sep]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH V2] preempt-rt: allow immediate Magic SysRq output for PREEMPT_RT_FULL
On 09/23/11 10:36, Valdis.Kletnieks@vt.edu wrote:
> On Thu, 22 Sep 2011 14:05:56 PDT, Frank Rowand said:
>> Change from V1 to V2: add drivers/tty/serial/8250.c
>
>> drivers/tty/serial/cpm_uart/cpm_uart_core.c | 2 1 + 1 - 0 !
>> drivers/tty/serial/mfd.c | 2 1 + 1 - 0 !
>> drivers/tty/serial/nwpserial.c | 2 1 + 1 - 0 !
>> drivers/tty/serial/omap-serial.c | 2 1 + 1 - 0 !
>> drivers/tty/serial/sunhv.c | 4 2 + 2 - 0 !
>> drivers/tty/serial/sunsab.c | 2 1 + 1 - 0 !
>> drivers/tty/serial/sunsu.c | 2 1 + 1 - 0 !
>> drivers/tty/serial/sunzilog.c | 2 1 + 1 - 0 !
>> drivers/tty/serial/uartlite.c | 2 1 + 1 - 0 !
>> drivers/tty/serial/xilinx_uartps.c | 2 1 + 1 - 0 !
>
>> if (up->port.sysrq)
>> locked = 0;
>> - else if (oops_in_progress) {
>> + else if (oops_in_progress || sysrq_in_progress) {
>> locked = spin_trylock(&up->port.lock);
>> } else
>> spin_lock(&up->port.lock);
>
> Is it time to create a helper function for this chunk of code that appears
> in almost identical form in a dozen drivers?

Thanks for pointing that out. The "if (up->port.sysrq)" in this pattern
already handles the case that I was trying to deal with in all of those
drivers, except cpm_uart_core.c, so I was able to remove all the other
drivers from V3 of the patch.

-Frank



\
 
 \ /
  Last update: 2011-09-23 22:19    [W:0.049 / U:0.964 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site