Re: [PATCH] fix ifenslave to not fail on lack of IP information

[Willy Tarreau <willy@w.ods.org>]

Hi,

On Fri, Nov 04, 2005 at 11:54:34AM -0500, Neil Horman wrote:
> The current version of ifenslave fails to attach slave interfaces to a bond if
> the masters doesn't have appropriate IP information.  While its common for
> bonded interface to have IP information its not required (bond as part of a
> bridge for instance).  This patch modifies ifenslave to not fail if IP
> information is not available in the master at the time of enslaving.
> 
> Regards
> Neil
> 
> Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
> 
> 
>  ifenslave.c |   10 ++++------
>  1 files changed, 4 insertions(+), 6 deletions(-)
> 
> 
> diff --git a/Documentation/networking/ifenslave.c b/Documentation/networking/ifenslave.c
> --- a/Documentation/networking/ifenslave.c
> +++ b/Documentation/networking/ifenslave.c
(...)

I find it annoying that ifenslave is still hosted by the kernel. I made
this mistake years ago because it was not hosted anywhere and I needed
to make changes available somewhere, but it should move to somewhere
else, either as a standalone package on sf.net, or added to an existing
package such as ifconfig, or better merged with it. I even think that
if ifconfig included all ifenslave, vconfig, ethtool and brctl functions,
it wouldn't be abandonned as it is now, but this is another problem.

Regards,
Willy