1. 23 10月, 2013 3 次提交
    • V
      bonding: move bond-specific init after enslave happens · 5378c2e6
      Veaceslav Falico 提交于
      As Jiri noted, currently we first do all bonding-specific initialization
      (specifically - bond_select_active_slave(bond)) before we actually attach
      the slave (so that it becomes visible through bond_for_each_slave() and
      friends). This might result in bond_select_active_slave() not seeing the
      first/new slave and, thus, not actually selecting an active slave.
      
      Fix this by moving all the bond-related init part after we've actually
      completely initialized and linked (via bond_master_upper_dev_link()) the
      new slave.
      
      Also, remove the bond_(de/a)ttach_slave(), it's useless to have functions
      to ++/-- one int.
      
      After this we have all the initialization of the new slave *before*
      linking, and all the stuff that needs to be done on bonding *after* it. It
      has also a bonus effect - we can remove the locking on the new slave init
      completely, and only use it for bond_select_active_slave().
      Reported-by: NJiri Pirko <jiri@resnulli.us>
      CC: Jay Vosburgh <fubar@us.ibm.com>
      CC: Andy Gospodarek <andy@greyhouse.net>
      Signed-off-by: NVeaceslav Falico <vfalico@redhat.com>
      Acked-by: Ding Tianhong@huawei.com
      Reviewed-by: NJiri Pirko <jiri@resnulli.us>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      5378c2e6
    • D
      Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next · 666d1008
      David S. Miller 提交于
      Jeff Kirsher says:
      
      ====================
      Intel Wired LAN Driver Updates
      
      This series contains updates to i40e only.
      
      Jesse provides 6 patches against i40e.  First is a patch to reduce
      CPU utilization by reducing read-flush to read in the hot path.  Next
      couple of patches resolve coverity issues reported by Hannes Frederic
      Sowa <hannes@stressinduktion.org>.  Then Jesse refactored i40e to cleanup
      functions which used cpu_to_xxx(foo) which caused a lot of line wrapping.
      
      Mitch provides 2 i40e patches.  First fixes a panic when tx_rings[0]
      are not allocated, his second patch corrects a math error when
      assigning MSI-X vectors to VFs.  The vectors-per-vf value reported
      by the hardware already conveniently reports one less than the actual
      value.
      
      Shannon provides 5 patches against i40e.  His first patch corrects a
      number of little bugs in the error handling of irq setup, most of
      which ended up panicing the kernel.  Next he fixes the overactive
      IRQ issue seen in testing and allows the use of the legacy interrupt.
      Shannon then provides a cleanup of the arguments declared at the
      beginning of each function.  Then he provides a patch to make sure
      that there are really rings and queues before trying to dump
      information in them.  Lastly he simplifies the code by using an
      already existing variable.
      
      Catherine provides an i40e patch to bump the version.
      
      v2:
       - Remove unneeded parenthesis in patch 3 based on feedback from
         Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
       - Fix patch description for patch 11 based on feedback from
         Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
      ====================
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      666d1008
    • Z
      net: remove function sk_reset_txq() · 0a6957e7
      ZHAO Gang 提交于
      What sk_reset_txq() does is just calls function sk_tx_queue_reset(),
      and sk_reset_txq() is used only in sock.h, by dst_negative_advice().
      Let dst_negative_advice() calls sk_tx_queue_reset() directly so we
      can remove unneeded sk_reset_txq().
      Signed-off-by: NZHAO Gang <gamerh2o@gmail.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      0a6957e7
  2. 22 10月, 2013 37 次提交