[Top] [All Lists]

Re: [Bonding-announce] [PATCH SET][bonding] cleanup

To: Jay Vosburgh <fubar@xxxxxxxxxx>, shmulik.hen@xxxxxxxxx, "Chad N. Tindel" <chad@xxxxxxxxxx>, bonding-devel@xxxxxxxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-net@xxxxxxxxxxxxxxx, Jeff Garzik <jgarzik@xxxxxxxxx>, "Noam, Amir" <amir.noam@xxxxxxxxx>, "Mendelson, Tsippy" <tsippy.mendelson@xxxxxxxxx>, "Noam, Marom" <noam.marom@xxxxxxxxx>
Subject: Re: [Bonding-announce] [PATCH SET][bonding] cleanup
From: Willy TARREAU <willy@xxxxxxxxx>
Date: Mon, 6 Oct 2003 00:28:17 +0200
In-reply-to: <>
References: <> <> <> <>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4i
Hi !

On Thu, Sep 25, 2003 at 05:13:00PM -0400, Chad N. Tindel wrote:
> I was specifically told by David Miller that we are not to break binary
> compatibility within a 2.4 release.  Such things had to wait until 2.5 
> or later.  We can not require a user to upgrade their ifenslave within a 2.4
> series kernel just to keep using the same functionality they were using in 
> 2.4.1.

I strongly agree. I have been facing this problem and it was really a pain. I
used the last bonding version which didn't define the ABI version, together
with the associated ifenslave, but when the need to upgrade to plain 2.4.22
came in, I had the surprize of getting a non-working bonding because this
intermediate ifenslave. Well, I upgraded it to latest version, which prevents
me from downgrading to the previous kernel because it has ABIv2 with no version,
so the newer ifenslave thinks it's an ABIv1. So the result is a symlink with
two versions of ifenslave on the disk, just in case I have to downgrade.

Although I agree it's clearly my fault and I should have been more careful, I
prefer to warn everyone about the consequences this might have on production
systems. Schmulik has done quite a great job here, and I believe most of it
should be integrated, but we have to carefully test each combination of old/new
ifenslave with old/new driver if we don't want to break some setups or prevent
admins from downgrading if something goes wrong.


<Prev in Thread] Current Thread [Next in Thread>
  • Re: [Bonding-announce] [PATCH SET][bonding] cleanup, Willy TARREAU <=