From: Veaceslav Falico <vfalico@redhat.com>
To: David Miller <davem@redhat.com>
Cc: netdev@vger.kernel.org, jiri@resnulli.us, fubar@us.ibm.com,
andy@greyhouse.net
Subject: Re: [PATCH v4 net-next 06/27] bonding: modify bond_get_slave_by_dev() to use neighbours
Date: Tue, 24 Sep 2013 21:39:08 +0200 [thread overview]
Message-ID: <20130924193908.GA7561@redhat.com> (raw)
In-Reply-To: <20130924.151943.1884204173373827600.davem@redhat.com>
On Tue, Sep 24, 2013 at 03:19:43PM -0400, David Miller wrote:
>From: Veaceslav Falico <vfalico@redhat.com>
>Date: Tue, 24 Sep 2013 13:46:46 +0200
>
>> - return NULL;
>> + return (struct slave *)netdev_lower_dev_get_private(bond->dev,
>> + slave_dev);
>> }
>
>void pointers never need to be cast, please remove this "struct slave *"
>cast.
Yep, leftover from the sandbox versions, will change and submit the new
version. Sorry about that :-/.
next prev parent reply other threads:[~2013-09-24 19:41 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-24 11:46 [PATCH v4 net-next 00/27] bonding: use neighbours instead of own lists Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 01/27] net: use lists as arguments instead of bool upper Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 02/27] net: add adj_list to save only neighbours Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 03/27] net: add RCU variant to search for netdev_adjacent link Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 04/27] net: add netdev_adjacent->private and allow to use it Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 05/27] bonding: populate neighbour's private on enslave Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 06/27] bonding: modify bond_get_slave_by_dev() to use neighbours Veaceslav Falico
2013-09-24 19:19 ` David Miller
2013-09-24 19:39 ` Veaceslav Falico [this message]
2013-09-24 11:46 ` [PATCH v4 net-next 07/27] net: add for_each iterators through neighbour lower link's private Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 08/27] bonding: remove bond_for_each_slave_continue_reverse() Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 09/27] bonding: make bond_for_each_slave() use lower neighbour's private Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 10/27] bonding: use bond_for_each_slave() in bond_uninit() Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 11/27] bonding: rework bond_3ad_xmit_xor() to use bond_for_each_slave() only Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 12/27] bonding: rework rlb_next_rx_slave() to use bond_for_each_slave() Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 13/27] bonding: rework bond_find_best_slave() " Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 14/27] bonding: rework bond_ab_arp_probe() " Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 15/27] bonding: remove unused bond_for_each_slave_from() Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 16/27] bonding: add bond_has_slaves() and use it Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 17/27] bonding: convert bond_has_slaves() to use the neighbour list Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 18/27] net: add a possibility to get private from netdev_adjacent->list Veaceslav Falico
2013-09-24 11:46 ` [PATCH v4 net-next 19/27] bonding: convert first/last slave logic to use neighbours Veaceslav Falico
2013-09-24 11:47 ` [PATCH v4 net-next 20/27] bonding: remove bond_prev_slave() Veaceslav Falico
2013-09-24 11:47 ` [PATCH v4 net-next 21/27] bonding: add __bond_next_slave() which uses neighbours Veaceslav Falico
2013-09-24 11:47 ` [PATCH v4 net-next 22/27] bonding: use neighbours for bond_next_slave() Veaceslav Falico
2013-09-24 11:47 ` [PATCH v4 net-next 23/27] bonding: remove slave lists Veaceslav Falico
2013-09-24 11:47 ` [PATCH v4 net-next 24/27] vlan: link the upper neighbour only after registering Veaceslav Falico
2013-09-24 11:47 ` [PATCH v4 net-next 25/27] vlan: unlink the upper neighbour before unregistering Veaceslav Falico
2013-09-24 11:47 ` [PATCH v4 net-next 26/27] net: expose the master link to sysfs, and remove it from bond Veaceslav Falico
2013-09-24 11:47 ` [PATCH v4 net-next 27/27] net: create sysfs symlinks for neighbour devices Veaceslav Falico
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=20130924193908.GA7561@redhat.com \
--to=vfalico@redhat.com \
--cc=andy@greyhouse.net \
--cc=davem@redhat.com \
--cc=fubar@us.ibm.com \
--cc=jiri@resnulli.us \
--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 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.