lkml.org 
[lkml]   [2013]   [Mar]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] sysrq: allow user trigger sysrq by upper-case character key
On 2013/3/7 5:39, Andrew Morton wrote:
> On Wed, 6 Mar 2013 12:07:34 +0800 "zhangwei(Jovi)" <jovi.zhangwei@huawei.com> wrote:
>
>> See the help text output of /proc/sysrq-trigger:
>>
>> SysRq : HELP : loglevel(0-9) reBoot Crash terminate-all-tasks(E)
>> memory-full-oom-kill(F) kill-all-tasks(I) ...
>>
>> Most command is referenced by responding upper-case character,
>> this would hint user can trigger sysrq by upper-case character key,
>> but that's wrong, sysrq only accept lower-case character currently.
>>
>> It's reasonable to let user trigger sysrq by upper-case character key.
>>
>> Signed-off-by: zhnagwei(Jovi) <jovi.zhangwei@huawei.com>
>> ---
>> drivers/tty/sysrq.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/tty/sysrq.c b/drivers/tty/sysrq.c
>> index 3687f0c..a88908e 100644
>> --- a/drivers/tty/sysrq.c
>> +++ b/drivers/tty/sysrq.c
>> @@ -471,6 +471,8 @@ static int sysrq_key_table_key2index(int key)
>> retval = key - '0';
>> else if ((key >= 'a') && (key <= 'z'))
>> retval = key + 10 - 'a';
>> + else if ((key >= 'A') && (key <= 'Z'))
>> + retval = key + 10 - 'A';
>> else
>> retval = -1;
>> return retval;
>
> Then fix the help message ;)
>
> We're already using 19 of the 26 letters and this change would rule out
> future use of the 26 upper-case letters.
>
> I doubt if anyone is hurting from the inconsistent help message, really.
It's confusing, I always try to trigger sysrq by upper-case character after read help message.

I will resend patch to change the help message instead.

Thanks.

>





\
 
 \ /
  Last update: 2013-03-07 10:21    [W:6.450 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site