linux-kernel - Re: [PATCH 12/12] add CPU notifier error injection module

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <961aa3351003230600r45e4579ave945b2dd71d9e4a0@mail.gmail.com>
Date:	Tue, 23 Mar 2010 22:00:09 +0900
From:	Akinobu Mita <akinobu.mita@...il.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 12/12] add CPU notifier error injection module

2010/3/23 Andrew Morton <akpm@...ux-foundation.org>:
> On Thu, 18 Mar 2010 18:05:24 +0900
> Akinobu Mita <akinobu.mita@...il.com> wrote:
>
>> I used this module to test the series of modification to the
>> cpu notifiers code.
>>
>> Example1: inject CPU offline error (-1 == -EPERM)
>>
>>       # modprobe cpu-notifier-error-inject cpu_down_prepare_error=-1
>>       # echo 0 > /sys/devices/system/cpu/cpu1/online0
>>       bash: echo: write error: Operation not permitted
>>
>> Example2: inject CPU online error (-2 == -ENOENT)
>>
>>       # modprobe cpu-notifier-error-inject cpu_up_prepare_error=-2
>>       # echo 1 > /sys/devices/system/cpu/cpu1/online0
>>       bash: echo: write error: No such file or directory
>>
>> ...
>>
>> --- a/lib/Kconfig.debug
>> +++ b/lib/Kconfig.debug
>> @@ -885,6 +885,18 @@ config LKDTM
>>       Documentation on how to use the module can be found in
>>       Documentation/fault-injection/provoke-crashes.txt
>>
>> +config CPU_NOTIFIER_ERROR_INJECT
>> +     tristate "CPU notifier error injection module"
>> +     depends on HOTPLUG_CPU && DEBUG_KERNEL
>> +     help
>> +       This option provides a kernel module that can be used to test
>> +       the error handling of the cpu notifiers
>> +
>> +       To compile this code as a module, choose M here: the module will
>> +       be called cpu-notify-inject.
>
> Should be "cpu-notifier-error-inject", yes?

Yes. Thanks for spotting and fixing.
And I had another typos in the above patch description:

>> Example1: inject CPU offline error (-1 == -EPERM)
>>
>>       # modprobe cpu-notifier-error-inject cpu_down_prepare_error=-1
>>       # echo 0 > /sys/devices/system/cpu/cpu1/online0

# echo 0 > /sys/devices/system/cpu/cpu1/online

(s/online0/online/)

>>       bash: echo: write error: Operation not permitted
>>
>> Example2: inject CPU online error (-2 == -ENOENT)
>>
>>       # modprobe cpu-notifier-error-inject cpu_up_prepare_error=-2
>>       # echo 1 > /sys/devices/system/cpu/cpu1/online0

Ditto.

>>       bash: echo: write error: No such file or directory
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/