[PATCH] PATCH v3 Convert multiple netdev_info messages to netdev_dbg

2017-06-20 Thread Michael J Dilmore
The bond_options.c file contains multiple netdev_info messages that clutter kernel output. This patches replaces these with netdev_dbg messages and adds a netdev_dbg for packets for slave. Signed-off-by: Michael J Dilmore <michael.j.dilm...@gmail.com> Suggested-by: Joe Perches <j...@pe

[PATCH] [PATCH v2 net-next] bonding: Convert multiple netdev_info messages to netdev_dbg

2017-06-20 Thread Michael J Dilmore
The bond_options.c file contains several netdev_info messages that clutter kernel output. This patch changes all netdev_info messages to netdev_dbg and adds a netdev debug for the packets per slave parameter. Suggested-by: Joe Perches <j...@perches.com> Signed-off-by: Michael J D

Re: [PATCH] Convert BUG_ON to WARN_ON in bond_options.c

2017-06-21 Thread Michael J Dilmore
On 21/06/17 23:39, Jay Vosburgh wrote: Michael J Dilmore <michael.j.dilm...@gmail.com> wrote: On 21/06/17 22:56, David Miller wrote: From: Michael D <michael.j.dilm...@gmail.com> Date: Wed, 21 Jun 2017 22:41:07 +0100 I don't think you can stop it being dereferenced... y

Re: [PATCH] Convert BUG_ON to WARN_ON in bond_options.c

2017-06-21 Thread Michael J Dilmore
On 21/06/17 22:56, David Miller wrote: From: Michael D Date: Wed, 21 Jun 2017 22:41:07 +0100 I don't think you can stop it being dereferenced... you just need to prevent an attacker from exploiting the null pointer dereference vulnerability right? And this is

[PATCH] Convert BUG_ON to WARN_ON in bond_options.c

2017-06-21 Thread Michael J Dilmore
The function below contains a BUG_ON where no active slave is detected. The patch converts this to a WARN_ON to avoid crashing the kernel. Signed-off-by: Michael J Dilmore <michael.j.dilm...@gmail.com> --- drivers/net/bonding/bond_options.c | 2 +- 1 file changed, 1 insertion(+), 1 de

[PATCH net-next] Bonding: Convert multiple netdev_info messages to netdev_dbg

2017-06-26 Thread Michael J Dilmore
com> Signed-off-by: Michael J Dilmore <michael.j.dilm...@gmail.com> --- drivers/net/bonding/bond_options.c | 132 +++-- 1 file changed, 67 insertions(+), 65 deletions(-) diff --git a/drivers/net/bonding/bond_options.c b/drivers/net/bonding/bond_options.c i

[PATCH] Convert multiple netdev_info messages to netdev_dbg

2017-06-15 Thread Michael J Dilmore
Multiple netdev_info messages clutter kernel output. Also add netdev_dbg for packets per slave. Suggested-by: Joe Perches <j...@perches.com> Signed-off-by: Michael J Dilmore <michael.j.dilm...@gmail.com> cc: Veaceslav Falico <vfal...@gmail.com>, Andy Gospodarek <a...

[PATCH net-next] Bonding: Convert multiple netdev_info messages to netdev_dbg

2017-06-26 Thread Michael J Dilmore
-by: Michael J Dilmore --- drivers/net/bonding/bond_options.c | 132 +++-- 1 file changed, 67 insertions(+), 65 deletions(-) diff --git a/drivers/net/bonding/bond_options.c b/drivers/net/bonding/bond_options.c index 1bcbb89..b1e5b8a 100644 --- a/drivers/net/bonding

[PATCH] [PATCH v2 net-next] bonding: Convert multiple netdev_info messages to netdev_dbg

2017-06-20 Thread Michael J Dilmore
The bond_options.c file contains several netdev_info messages that clutter kernel output. This patch changes all netdev_info messages to netdev_dbg and adds a netdev debug for the packets per slave parameter. Suggested-by: Joe Perches Signed-off-by: Michael J Dilmore --- drivers/net/bonding

[PATCH] PATCH v3 Convert multiple netdev_info messages to netdev_dbg

2017-06-20 Thread Michael J Dilmore
The bond_options.c file contains multiple netdev_info messages that clutter kernel output. This patches replaces these with netdev_dbg messages and adds a netdev_dbg for packets for slave. Signed-off-by: Michael J Dilmore Suggested-by: Joe Perches --- drivers/net/bonding/bond_options.c | 54

[PATCH] Convert BUG_ON to WARN_ON in bond_options.c

2017-06-21 Thread Michael J Dilmore
The function below contains a BUG_ON where no active slave is detected. The patch converts this to a WARN_ON to avoid crashing the kernel. Signed-off-by: Michael J Dilmore --- drivers/net/bonding/bond_options.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net

Re: [PATCH] Convert BUG_ON to WARN_ON in bond_options.c

2017-06-21 Thread Michael J Dilmore
On 21/06/17 22:56, David Miller wrote: From: Michael D Date: Wed, 21 Jun 2017 22:41:07 +0100 I don't think you can stop it being dereferenced... you just need to prevent an attacker from exploiting the null pointer dereference vulnerability right? And this is done by returning the function

Re: [PATCH] Convert BUG_ON to WARN_ON in bond_options.c

2017-06-21 Thread Michael J Dilmore
On 21/06/17 23:39, Jay Vosburgh wrote: Michael J Dilmore wrote: On 21/06/17 22:56, David Miller wrote: From: Michael D Date: Wed, 21 Jun 2017 22:41:07 +0100 I don't think you can stop it being dereferenced... you just need to prevent an attacker from exploiting the null pointer

[PATCH] Convert multiple netdev_info messages to netdev_dbg

2017-06-15 Thread Michael J Dilmore
Multiple netdev_info messages clutter kernel output. Also add netdev_dbg for packets per slave. Suggested-by: Joe Perches Signed-off-by: Michael J Dilmore cc: Veaceslav Falico , Andy Gospodarek , net...@vger.kernel.org, linux-kernel@vger.kernel.org --- drivers/net/bonding/bond_options.c