Re: [PATCH net-next 0/2] bonding: add its own Kconfig

On Tue, Jul 15, 2014 at 06:10:56PM -0700, David Miller wrote:

From: Veaceslav Falico <redacted>
Date: Tue, 15 Jul 2014 12:15:39 +0200

quoted

Currently to change some built-in defaults it's necessary to change the
code, and to maintain these patches on each update.

Create a bonding's Kconfig to make it possible to configure them at build
time.

There is really no reason these values can't be set at run time, and that
is my only, but firm, objection to this patch series.

Yeah, MIIMON default value doesn't really need to exist, vlan depth can be
configured dynamically ...

For the per-slave things, make new netlink attributes that can be provided
when the slave is added, using the existing default if not specified.

This means the fixed sized arrays involved here need to become
dynamically allocated.

... and arp ip targets can be also made into lists.

I'll try to change it and come up with split patchsets.

Thank you!