Re: [PATCH] 8021q/vlan: process NETDEV_GOING_DOWN
From: Eldad Zack
Date: Wed May 30 2012 - 17:47:48 EST
On Wed, 30 May 2012, David Miller wrote:
> From: Eldad Zack <eldad@xxxxxxxxxxxxxxx>
> Date: Wed, 30 May 2012 21:11:02 +0200 (CEST)
>
> > In case a certain protocol needs to send a "dying gasp" packet, when you
> > administrativly shutdown the port (which is also what happens when you
> > restart the machine).
>
> No in tree users have this requirement, therefore your patch is
> inappropriate.
You are right in that, that no in tree users have this requirement
(yet), but in the same time it doesn't harm any existing code.
Don't you agree that it's the right order to do the notifications?
And if so, isn't that enough, considering that there are no side
effects and it's a tiny change?
Eldad
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/