]> git.openfabrics.org - ~emulex/infiniband.git/commit
bonding: use __dev_get_by_name instead of dev_get_by_name to find interface
authorYing Xue <ying.xue@windriver.com>
Wed, 15 Jan 2014 02:23:37 +0000 (10:23 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 15 Jan 2014 02:50:46 +0000 (18:50 -0800)
commit0917b9334b1cc130e91a53b2e3bbaae760fc50ee
tree5572e041ce2e4e1e2dca2d3e9d9178c386bfae53
parentbdffbb8e2630feb258d848cdfabaebbdb8105864
bonding: use __dev_get_by_name instead of dev_get_by_name to find interface

The following call chain indicates that bond_do_ioctl() is protected
under rtnl_lock. If we use __dev_get_by_name() instead of
dev_get_by_name() to find interface handler in it, this would
help us avoid to change reference counter of interface once.

dev_ioctl()
  rtnl_lock()
  dev_ifsioc()
    bond_do_ioctl()
  rtnl_unlock()

Additionally we also change the coding style in bond_do_ioctl(),
letting it more readable for us.

Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: Ying Xue <ying.xue@windriver.com>
Acked-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c