1. 06 5月, 2015 33 次提交
  2. 05 5月, 2015 7 次提交
    • L
      net: fix two sparse warnings introduced by IGMP/MLD parsing exports · fcba67c9
      Linus Lüssing 提交于
      > net/core/skbuff.c:4108:13: sparse: incorrect type in assignment (different base types)
      > net/ipv6/mcast_snoop.c:63 ipv6_mc_check_exthdrs() warn: unsigned 'offset' is never less than zero.
      
      Introduced by 9afd85c9
      ("net: Export IGMP/MLD message validation code")
      Reported-by: Nkbuild test robot <fengguang.wu@intel.com>
      Signed-off-by: NLinus Lüssing <linus.luessing@c0d3.blue>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      fcba67c9
    • D
      Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/next-queue · f9161585
      David S. Miller 提交于
      Jeff Kirsher says:
      
      ====================
      Intel Wired LAN Driver Updates 2015-05-04
      
      This series contains updates to igb, e100, e1000e and ixgbe.
      
      Todd cleans up igb_enable_mas() since it should only be called for the
      82575 silicon and has no clear return, so modify the function to void.
      
      Jean Sacren found upon inspection that 'err' did not need to be
      initialized, since it is immediately overwritten.
      
      Alex Duyck provides two patches for e1000e, the first cleans up the
      handling VLAN_HLEN as a part of max frame size.  Fixes the issue:
      c751a3d5 ("e1000e: Correctly include VLAN_HLEN when changing
      interface MTU").  The second fixes an issue where the driver was not
      allowing jumbo frames to be enabled when CRC stripping was disabled,
      however it was allowing CRC stripping to be disabled while jumbo frames
      were enabled.
      
      Jeff (me) fixes a warning found on PPC where the use of do_div() needed
      to use u64 arg and not s64.
      
      Mark provides three ixgbe patches, first to fix the Intel On-chip System
      Fabric (IOSF) Sideband message interfaces, to serialize access using both
      PHY bits in the SWFW_SEMAPHORE register.  Then fixes how semaphore bits
      were released, since they should be released in reverse of the order that
      they were taken.  Lastly updates ixgbe to use a signed type to hold
      error codes, since error codes are negative, so consistently use signed
      types when handling them.
      
      v2: dropped the previous #6-#8 patches by Hiroshi Shimanoto based on
          feedback from Or Gerlitz (and David Miller) that it appears there
          needs to be further discussion on how this gets implemented.
      ====================
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      f9161585
    • D
      Merge branch 'tipc-topology-cleanup' · c373dac5
      David S. Miller 提交于
      Ying Xue says:
      
      ====================
      tipc: cleanup topology server
      
      Not only function names declared in subscr.c are very confused, but
      also topology server's locking policy is not designed very well, for
      instance, usually leading to panic in some special corner cases.
      
      In this series, we attempt to eliminate the confusion of function names
      and simplify topology server's locking policy to solve above mentioned
      issues. More importantly, the change will make relevant code easily
      understandable and maintainable.
      ====================
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      c373dac5
    • Y
      tipc: deal with return value of tipc_conn_new callback · 90bdfcb7
      Ying Xue 提交于
      Once tipc_conn_new() returns NULL, the connection should be shut
      down immediately, otherwise, oops may happen due to the NULL pointer.
      Signed-off-by: NYing Xue <ying.xue@windriver.com>
      Reviewed-by: NJon Maloy <jon.maloy@ericson.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      90bdfcb7
    • Y
      tipc: adjust locking policy of subscription · a13683f2
      Ying Xue 提交于
      Currently subscriber's lock protects not only subscriber's subscription
      list but also all subscriptions linked into the list. However, as all
      members of subscription are never changed after they are initialized,
      it's unnecessary for subscription to be protected under subscriber's
      lock. If the lock is used to only protect subscriber's subscription
      list, the adjustment not only makes the locking policy simpler, but
      also helps to avoid a deadlock which may happen once creating a
      subscription is failed.
      Signed-off-by: NYing Xue <ying.xue@windriver.com>
      Reviewed-by: NJon Maloy <jon.maloy@ericson.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      a13683f2
    • Y
      tipc: involve reference counter for subscriber · 00bc00a9
      Ying Xue 提交于
      At present subscriber's lock is used to protect the subscription list
      of subscriber as well as subscriptions linked into the list. While one
      or all subscriptions are deleted through iterating the list, the
      subscriber's lock must be held. Meanwhile, as deletion of subscription
      may happen in subscription timer's handler, the lock must be grabbed
      in the function as well. When subscription's timer is terminated with
      del_timer_sync() during above iteration, subscriber's lock has to be
      temporarily released, otherwise, deadlock may occur. However, the
      temporary release may cause the double free of a subscription as the
      subscription is not disconnected from the subscription list.
      
      Now if a reference counter is introduced to subscriber, subscription's
      timer can be asynchronously stopped with del_timer(). As a result, the
      issue is not only able to be fixed, but also relevant code is pretty
      readable and understandable.
      Signed-off-by: NYing Xue <ying.xue@windriver.com>
      Reviewed-by: NJon Maloy <jon.maloy@ericson.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      00bc00a9
    • Y
      tipc: introduce tipc_subscrb_create routine · 1b764828
      Ying Xue 提交于
      Introducing a new function makes the purpose of tipc_subscrb_connect_cb
      callback routine more clear.
      Signed-off-by: NYing Xue <ying.xue@windriver.com>
      Reviewed-by: NJon Maloy <jon.maloy@ericson.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      1b764828