From: Jarod Wilson <jarod@redhat.com>
To: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Cc: linux-kernel@vger.kernel.org,
Uwe Koziolek <uwe.koziolek@redknee.com>,
Jay Vosburgh <jay.vosburgh@canonical.com>,
Andy Gospodarek <gospo@cumulusnetworks.com>,
Veaceslav Falico <vfalico@gmail.com>,
netdev@vger.kernel.org
Subject: Re: [PATCH v4] net/bonding: send arp in interval if no active slave
Date: Fri, 09 Oct 2015 10:36:03 -0400 [thread overview]
Message-ID: <5617D0D3.90208@redhat.com> (raw)
In-Reply-To: <56151E4A.2000503@redhat.com>
Jarod Wilson wrote:
...
> As Andy already stated I'm not a fan of such workarounds either but it's
> necessary sometimes so if this is going to be actually considered then a
> few things need to be fixed. Please make this a proper bonding option
> which can be changed at runtime and not only via a module parameter.
Is there any particular userspace tool that would need some updating, or
is adding the sysfs knobs sufficient here? I think I've got all the
sysfs stuff thrown together now, but still need to test.
>> Now, I saw that you've only tested with 500 ms, can't this be fixed by
>> using
>> a different interval ? This seems like a very specific problem to have a
>> whole new option for.
>
> ...I'll wait until we've heard confirmation from Uwe that intervals
> other than 500ms don't fix things.
Okay, so I believe the "only tested with 500ms" was in reference to
testing with Uwe's initial patch. I do have supporting evidence in a
bugzilla report that shows upwards of 5000ms still experience the
problem here.
--
Jarod Wilson
jarod@redhat.com
next prev parent reply other threads:[~2015-10-09 14:36 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-17 16:23 [PATCH] net/bonding: send arp in interval if no active slave Jarod Wilson
2015-08-17 16:55 ` Veaceslav Falico
2015-08-17 17:12 ` Jarod Wilson
2015-08-17 18:56 ` Uwe Koziolek
2015-08-17 19:14 ` Jay Vosburgh
2015-08-17 20:51 ` Uwe Koziolek
2015-08-31 22:21 ` Jarod Wilson
2015-09-01 23:15 ` Uwe Koziolek
2015-09-01 15:41 ` Andy Gospodarek
2015-09-01 23:10 ` Uwe Koziolek
2015-09-03 15:05 ` Jay Vosburgh
2015-09-04 11:04 ` Uwe Koziolek
2015-09-28 13:31 ` Jarod Wilson
2015-10-06 19:53 ` [PATCH v4] " Jarod Wilson
2015-10-06 19:58 ` Jarod Wilson
2015-10-07 12:03 ` Nikolay Aleksandrov
2015-10-07 13:29 ` Jarod Wilson
2015-10-09 14:36 ` Jarod Wilson [this message]
2015-10-09 15:25 ` Nikolay Aleksandrov
2015-10-09 15:31 ` Jay Vosburgh
2015-10-12 15:33 ` Jarod Wilson
2015-10-30 18:59 ` Uwe Koziolek
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=5617D0D3.90208@redhat.com \
--to=jarod@redhat.com \
--cc=gospo@cumulusnetworks.com \
--cc=jay.vosburgh@canonical.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=nikolay@cumulusnetworks.com \
--cc=uwe.koziolek@redknee.com \
--cc=vfalico@gmail.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.