Messages in this thread | | | Subject | Re: [PATCH] net: add support for netdev notifier error injection | From | Nikolay Aleksandrov <> | Date | Tue, 1 Dec 2015 00:49:55 +0100 |
| |
On 12/01/2015 12:48 AM, Akinobu Mita wrote: > 2015-11-28 21:45 GMT+09:00 Nikolay Aleksandrov <razor@blackwall.org>: >> From: Nikolay Aleksandrov <nikolay@cumulusnetworks.com> >> >> This module allows to insert errors in some of netdevice's notifier >> events. All network drivers use these notifiers to signal various events >> and to check if they are allowed, e.g. PRECHANGEMTU and CHANGEMTU >> afterwards. Until recently I had to run failure tests by injecting >> a custom module, but now this infrastructure makes it trivial to test >> these failure paths. Some of the recent bugs I fixed were found using >> this module. >> Here's an example: >> $ cd /sys/kernel/debug/notifier-error-inject/netdev >> $ echo -22 > actions/NETDEV_CHANGEMTU/error >> $ ip link set eth0 mtu 1024 >> RTNETLINK answers: Invalid argument > > This patch looks good to me. It would be nice if you add tools like > tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh for this feature. >
Thanks for reviewing and noted. I'll take a look and make some tests. Do you mind if I post it as a follow-up ?
Cheers, Nik
| |