Re: [PATCH] channel bonding: add support for device-indexed parameters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Florin Malita <[email protected]> wrote:

>On Wed, 21 Sep 2005 23:03:53 -0700
>"Jason R. Martin" <[email protected]> wrote:
>> Personally I think working to get the sysfs support finished in
>> bonding and stop relying on module parameters to configure bonds would
>> be better
[...]
>Agreed - that would be a better configuration interface, but I don't see
>why we couldn't support module parameter arrays too. Especially since
>the changes are minimal and don't break the ABI/ifenslave
>compatibility/etc.
>
>IMHO the "primary" semantics are completely broken right now and this
>is a possible fix for it.

	The distro provided network init scripts are, as far as I know,
really the main user of the bonding module parameters.  Right now, the
init scripts will generally load the bonding module multiple times when
creating multiple bonds with differing parameters.  This works tolerably
well, although I recall that some users have run afoul of Fedora Core
kernels that could or would not load any module multiple times.  I don't
know if that's still the case today.

	In any event, your patch does not provide enough flexibility to
allow the distro scripts to switch to it (it omits arp_ip_target), so
the init scripts will be unable to switch.  Given that, I'm not sure I
see the real value.

	Additionally, as Jason mentions, the sysfs API is looming, and
I'd frankly rather have the init scripts switch to sysfs as it provide
greater flexibility (in particular, it removes the dependency on bonding
being compiled as a module).

	-J

---
	-Jay Vosburgh, IBM Linux Technology Center, [email protected]
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]
  Powered by Linux