Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1281193

Re: [PATCH] net: add support for netdev notifier error injection

From David Miller <davem@davemloft.net>
Newsgroups linux.kernel
Subject Re: [PATCH] net: add support for netdev notifier error injection
Date 2015-12-01 21:40 +0100
Message-ID <qAZSr-622-27@gated-at.bofh.it> (permalink)
References <qzN6V-w7-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


From: Nikolay Aleksandrov <razor@blackwall.org>
Date: Sat, 28 Nov 2015 13:45:28 +0100

> 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
> 
> CC: Akinobu Mita <akinobu.mita@gmail.com>
> CC: "David S. Miller" <davem@davemloft.net>
> CC: netdev <netdev@vger.kernel.org>
> Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>

This looks fine, applied to net-next, thanks!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

Re: [PATCH] net: add support for netdev notifier error injection David Miller <davem@davemloft.net> - 2015-12-01 21:40 +0100

csiph-web