From: Nikolay Aleksandrov <nikolay@redhat.com>
To: Santiago Garcia Mantinan <manty@manty.net>
Cc: netdev <netdev@vger.kernel.org>
Subject: Re: bonding + arp monitoring fails if interface is a vlan
Date: Mon, 05 Aug 2013 12:26:14 +0200 [thread overview]
Message-ID: <51FF7DC6.7080602@redhat.com> (raw)
In-Reply-To: <CAJk_L2FU6GcHUoC+UdNJoULN9KewoUQAMhapC0k3RcTqNOymnw@mail.gmail.com>
On 08/05/2013 12:26 PM, Santiago Garcia Mantinan wrote:
> 2013/8/4 Santiago Garcia Mantinan <manty@manty.net>:
>> 2013/8/2 Nikolay Aleksandrov <nikolay@redhat.com>:
>>> I believe that it is because dev_trans_start() returns 0 for 8021q devices and
>>> so the calculations if the slave has transmitted are wrong, and the flip-flop
>>> happens.
>>> Please try the attached patch, it should resolve your issue (basically it gets
>>> the dev_trans_start of the vlan's underlying device if a vlan is found).
>>
>> Thanks, patched and compiling, I'll try today with my laptops and
>> tomorrow at the lab I had setup and then at the original machine.
>>
>> I'll let you know how things go.
>
> Ok, initial tests seem to show that a bonding defined like I had on my
> very basic setup that I sent to the list is now working.
>
> What doesn't seem to be working is if I set it up using bonding under
> the vlans and then doing a bond of those, I mean:
>
> iface bond0 inet manual
> bond-slaves eth0
> bond-mode 802.3ad
> bond-miimon 100
> ...
> iface bond2 inet static
> address 192.168.1.2
> netmask 255.255.255.0
> bond-slaves bond0.1001 bond0.1002
> bond-mode active-backup
> bond-arp_validate 0
> bond-arp_interval 2000
> bond-arp_ip_target 192.168.1.1
> ...
>
> Should this bond of bonds work?
>
No, because we take the first non-vlan's interface trans_start after the patch
which in this case is a bonding interface which also doesn't update its
trans_start, i.e. bond over bond (or over vlans over bond) with arp monitoring
shouldn't work.
> I'm doing more tests to make sure that the basic eth0.1001 and
> eth0.1002 works 100% after finding that the bond of bonds wasn't
> working ok, just in case the basic was also failing, but at least the
> double bond is failing and basic bond seems to work ok.
>
> Regards.
>
next prev parent reply other threads:[~2013-08-05 10:45 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-01 12:11 bonding + arp monitoring fails if interface is a vlan Santiago Garcia Mantinan
2013-08-01 13:00 ` Erik Hugne
2013-08-02 7:26 ` Santiago Garcia Mantinan
2013-08-02 9:33 ` Santiago Garcia Mantinan
2013-08-01 20:21 ` Veaceslav Falico
2013-08-02 7:30 ` Santiago Garcia Mantinan
2013-08-02 11:58 ` Nikolay Aleksandrov
2013-08-02 15:49 ` Jay Vosburgh
2013-08-02 16:13 ` Nikolay Aleksandrov
2013-08-04 10:45 ` Santiago Garcia Mantinan
2013-08-05 10:26 ` Santiago Garcia Mantinan
2013-08-05 10:26 ` Nikolay Aleksandrov [this message]
2013-08-07 7:26 ` Santiago Garcia Mantinan
2013-08-07 7:39 ` Nikolay Aleksandrov
2013-08-07 10:44 ` Santiago Garcia Mantinan
2013-08-20 8:05 ` Santiago Garcia Mantinan
2013-08-20 10:11 ` Nikolay Aleksandrov
2013-08-21 7:39 ` Santiago Garcia Mantinan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51FF7DC6.7080602@redhat.com \
--to=nikolay@redhat.com \
--cc=manty@manty.net \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).