On 10/18/2010 04:31 PM, Vladislav Bogdanov wrote:
18.10.2010 17:15, Harald Hoyer wrote:
On 10/18/2010 03:48 PM, Vladislav Bogdanov wrote:
18.10.2010 16:06, Harald Hoyer wrote:
On 10/18/2010 02:04 PM, Vladislav Bogdanov wrote:
Hi all,
I attach patch for bonding support in dracut.
It is against 005 (developed on fedora 13 version).
No documentation yet.
(partially) Tested to work correctly with bond0 alone and bridge on top
of bond0.
Comments are welcome,
Best,
Vladislav
Please patch against the git version...
$ git clone git://dracut.git.sourceforge.net/gitroot/dracut/dracut
http://dracut.git.sourceforge.net/git/gitweb.cgi?p=dracut/dracut
This should be it.
pushed
...
http://dracut.git.sourceforge.net/git/gitweb.cgi?p=dracut/dracut;a=commitdiff;h=96fb9c8dd708dfbab9cf6515d9a450abb060e623
Ahmmm....
So quick :)
I was ready for a long discussion :)
Some notes:
There is no documentation yet :( and it should not be forgotten to add
it. I generally hate to edit XML by hands. Could someone who operates
some magic XML tools do it?
Nice, free, open source xml editor:
http://www.syntext.com/products/serna-free/
Also both original patch (against 005) and commit message has a typo (in
head comments):
bond=bond0:eth0,eth1:balance-rr
should be really
bond=bond0:eth0,eth1:mode=balance-rr
Patch against git has this corrected (together with one more typo -
s/DEVICE/netif/)
Hmm, I took dracut-bonding-support.patch and not
dracut-005-bonding-support.patch... but I might have copied the commit message
from the wrong patch.
Or even better third clause (options) should be omitted from that line:
it remained from a revision where I had mode as a separate clause.
Current version leaves all options defaulting to bonding.ko (and
balance-rr is its default mode btw).
One more is a mode of parse-bond.sh which should be 0755 (for consistency?).
Already done :)
http://dracut.git.sourceforge.net/git/gitweb.cgi?p=dracut/dracut;a=commitdiff;h=57a49c67275c8ec6b3d85dc70433ffc15aec47e4
And last, this should be tested thoroughly. I only did limited testing -
not all use cases, especially not with default values. It seems to be
correct, but I'm not 100% sure.
Best,
Vladislav
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
