From: Or Gerlitz <ogerlitz@mellanox.com>
To: Jay Vosburgh <fubar@us.ibm.com>
Cc: <davem@davemloft.net>, <roland@kernel.org>,
<netdev@vger.kernel.org>, Shlomo Pongratz <shlomop@mellanox.com>
Subject: Re: [PATCH net 1/2] net/bonding: emit address change event in more places
Date: Wed, 4 Apr 2012 10:57:03 +0300 [thread overview]
Message-ID: <4F7BFECF.2060001@mellanox.com> (raw)
In-Reply-To: <9859.1333492106@death.nxdomain>
On 4/4/2012 1:28 AM, Jay Vosburgh wrote:
> Is the bond_release_all notifier call actually necessary in your
> testing? The bond itself is destroyed immediately after
> bond_release_all returns, so I would expect the neighbours to be
> deleted then. For that path we probably don't need to zero the mac at
> all (or compute features, for that matter).
yep, you're right, will remove this extra emitting of event and resend,
thanks.
Or.
next prev parent reply other threads:[~2012-04-04 7:57 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-02 16:17 [PATCH net 0/2] net/bonding: fixes related to neigh Or Gerlitz
2012-04-02 16:17 ` [PATCH net 1/2] net/bonding: emit address change event in more places Or Gerlitz
2012-04-03 22:28 ` Jay Vosburgh
2012-04-04 7:57 ` Or Gerlitz [this message]
2012-04-02 16:17 ` [PATCH net 2/2] net/bonding: correctly proxy slave neigh param setup ndo function Or Gerlitz
2012-04-03 22:53 ` Jay Vosburgh
2012-04-04 8:03 ` Or Gerlitz
2012-04-04 16:56 ` Jay Vosburgh
2012-04-03 18:58 ` [PATCH net 0/2] net/bonding: fixes related to neigh Or Gerlitz
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=4F7BFECF.2060001@mellanox.com \
--to=ogerlitz@mellanox.com \
--cc=davem@davemloft.net \
--cc=fubar@us.ibm.com \
--cc=netdev@vger.kernel.org \
--cc=roland@kernel.org \
--cc=shlomop@mellanox.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.