From: David Miller <davem@davemloft.net>
To: andy@greyhouse.net
Cc: netdev@vger.kernel.org, ralf.zeidler@nsn.com
Subject: Re: [PATCH net-next] [v2] bonding: remove entries for master_ip and vlan_ip and query devices instead
Date: Wed, 21 Mar 2012 22:34:11 -0400 (EDT) [thread overview]
Message-ID: <20120321.223411.1471023630379810739.davem@davemloft.net> (raw)
In-Reply-To: <1332365802-21550-1-git-send-email-andy@greyhouse.net>
From: Andy Gospodarek <andy@greyhouse.net>
Date: Wed, 21 Mar 2012 17:36:42 -0400
> As the Subject indicates this patch drops the master_ip and vlan_ip
> elements from the 'bonding' and 'vlan_entry' structs, respectively.
> This can be done because a device's address-list is now traversed to
> determine the optimal source IP address for ARP requests and for checks
> to see if the bonding device has a particular IP address. This code
> could have all be contained inside the bonding driver, but it made more
> sense to me to EXPORT and call inet_confirm_addr since it did exactly
> what was needed.
I like this patch a lot but you have one little bug that needs to be
fixed:
> + rcu_read_lock();
> + in_dev = __in_dev_get_rcu(dev);
> + rcu_read_unlock();
> +
> + if (in_dev)
> + addr = inet_confirm_addr(in_dev, dst, local, RT_SCOPE_HOST);
If you're going to do an RCU ref-less lookup of in_dev and then use
it, you have to include the "use" inside of the RCU protected section
as well.
Otherwise as soon as you rcu_read_unlock() the in_dev could be freed
up on you.
The only exception would be if you know that all callers of
bond_confirm_addr() ran in an RCU protected section, but I do not
think that is universally the case here.
If you think it might be the case that we are RCU protected in all of
these code paths already, you can remove the RCU locking altogether
from bond_confirm_addr() and run with lockdep enabled while exercising
all of the relevant code paths.
next prev parent reply other threads:[~2012-03-22 2:34 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-21 21:36 [PATCH net-next] [v2] bonding: remove entries for master_ip and vlan_ip and query devices instead Andy Gospodarek
2012-03-22 2:34 ` David Miller [this message]
2012-03-22 20:22 ` Andy Gospodarek
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=20120321.223411.1471023630379810739.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=andy@greyhouse.net \
--cc=netdev@vger.kernel.org \
--cc=ralf.zeidler@nsn.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.