1. 09 Jul, 2014 25 commits
  2. 08 Jul, 2014 15 commits
    • David S. Miller's avatar
      Merge branch 'bridge_batmanadv_exports' · 9274f9f8
      David S. Miller authored
      Linus Lüssing says:
      
      ====================
      bridge: multicast snooping exports #2
      
      Some people pointed out to me that it might be helpful to add stubs for
      the newly added multicast exports. That way e.g. batman-adv should continue
      to be compile and useable without having to have a kernel compiled
      with bridge code in the future. This is what the first patch is supposed
      to do.
      
      The second patch adds a third multicast export for the bridge which
      e.g. batman-adv is supposed to use, too, soon: Just like the bridge
      disables its multicast snooping activities if no querier is present,
      batman-adv needs to do the same if bridges are involved.
      
      These three exports should be the final ones needed to marry the bridge
      multicast snooping with the batman-adv multicast optimizations recently
      added for the 3.15 kernel, allowing to use these optimzations in common
      setups having a bridge on top of e.g. bat0, too. So far these bridged
      setups would fall back to simple flooding through the batman-adv mesh
      network for any multicast packet entering bat0.
      
      More information about the batman-adv multicast optimizations currently
      implemented can be found here:
      
      http://www.open-mesh.org/projects/batman-adv/wiki/Basic-multicast-optimizations
      
      The integration on the batman-adv side could afterwards look like this,
      for instance (now including the third export):
      
      http://git.open-mesh.org/batman-adv.git/commitdiff/61e4f6af4b7a21ed4040f2e711d50c778e5b6d93?hp=6ae4281474675fbca5bedcf768972a32db586eb6
      ====================
      9274f9f8
    • Linus Lüssing's avatar
      bridge: export knowledge about the presence of IGMP/MLD queriers · c34963e2
      Linus Lüssing authored
      With this patch other modules are able to ask the bridge whether an
      IGMP or MLD querier exists on the according, bridged link layer.
      
      Multicast snooping can only be performed if a valid, selected querier
      exists on a link.
      
      Just like the bridge only enables its multicast snooping if a querier
      exists, e.g. batman-adv too can only activate its multicast
      snooping in bridged scenarios if a querier is present.
      
      For instance this export avoids having to reimplement IGMP/MLD
      querier message snooping and parsing in e.g. batman-adv, when
      multicast optimizations for bridged scenarios are added in the
      future.
      Signed-off-by: default avatarLinus Lüssing <linus.luessing@web.de>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      c34963e2
    • Linus Lüssing's avatar
      bridge: adding stubs for multicast exports · f941a6d9
      Linus Lüssing authored
      To make users (e.g. batman-adv soon) load- and runnable even if the
      bridge was compiled without snooping capabilities - or even if the
      kernel was compiled without any bridge code at all.
      Signed-off-by: default avatarLinus Lüssing <linus.luessing@web.de>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      f941a6d9
    • Erik Hugne's avatar
      tipc: fix a memleak when sending data · 70452dcb
      Erik Hugne authored
      This fixes a regression bug caused by:
      067608e9 ("tipc: introduce direct
      iovec to buffer chain fragmentation function")
      
      If data is sent on a nonblocking socket and the destination link
      is congested, the buffer chain is leaked. We fix this by freeing
      the chain in this case.
      Signed-off-by: default avatarErik Hugne <erik.hugne@ericsson.com>
      Signed-off-by: default avatarJon Maloy <jon.maloy@ericsson.com>
      Acked-by: default avatarYing Xue <ying.xue@windriver.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      70452dcb
    • Maciej W. Rozycki's avatar
      defxx: Fix issues with debug printk calls · 51ba0ed1
      Maciej W. Rozycki authored
      This fixes issues with debug printk calls across the driver, normally
      disabled; first compilation errors:
      
      drivers/net/fddi/defxx.c:676:1: error: pasting "(" and ""In dfx_bus_init...\n"" does not give a valid preprocessing token
      drivers/net/fddi/defxx.c:820:1: error: pasting "(" and ""In dfx_bus_uninit...\n"" does not give a valid preprocessing token
      
      and so on, and then warnings:
      
      drivers/net/fddi/defxx.c: In function 'dfx_driver_init':
      drivers/net/fddi/defxx.c:1132: warning: format '%0X' expects type 'unsigned int', but argument 4 has type 'dma_addr_t'
      drivers/net/fddi/defxx.c:1132: warning: format '%0X' expects type 'unsigned int', but argument 4 has type 'dma_addr_t'
      
      etc.  Additionally casts are removed from virtual addresses and %p used.
      Signed-off-by: default avatarMaciej W. Rozycki <macro@linux-mips.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      51ba0ed1
    • David S. Miller's avatar
      Merge branch 'defxx-next' · 284a83a0
      David S. Miller authored
      Maciej W. Rozycki says:
      
      ====================
      defxx: Fixes for 64-bit host support
      
       This mini patch series addresses issues with 64-bit host support for FDDI
      interface boards supported by the defxx driver where DMA mapping
      synchronisation is required on swiotlb systems.  While PDQ, the DMA engine
      chip used with these boards, supports 48-bit addressing that would
      normally suffice for typical 64-bit systems in existence, the host bus
      interface chips used by individual implementations have their limitations
      as follows:
      
      * DEFTA or DEC FDDIcontroller/TURBOchannel -- there's no host bus
        interface chip, the PDQ connects to TURBOchannel directly; TURBOchannel
        supports DMA addressing of up to 16GB (34-bit addressing), however no
        TURBOchannel system has ever been made that supports more than 1GB of
        RAM, so in reality no remapping is ever required,
      
      * DEFEA or DEC FDDIcontroller/EISA -- the ESIC EISA interface chip only
        supports 32-bit addressing, all accesses beyond 4GB have to be remapped,
      
      * DEFPA or DEC FDDIcontroller/PCI -- the PFI PCI interface chip rev. 1 & 2
        only support 32-bit addressing, they have 32 AD lines only both on the
        PDQ and the PCI side, and consequently no Dual Address Cycle support, so
        all accesses beyond 4GB have to be remapped; the range of addressing
        supported by PFI rev. 3 is currently not certain, however the chip is
        backwards compatible with earlier revisions and will work with code that
        supports them.
      
      Some other issues discovered in the course of correcting 64-bit support
      have been fixed as well.  Each of the patches is functionally
      self-contained and can be applied independentely, although there may be
      mechanical dependencies making it necessary to apply patches in order.
      
       The driver suffers from non-standard formatting and while I did my best
      with these bug fixes to follow our coding style, I found some pieces
      hopeless, checkpatch.pl will complain.  I plan to reformat the whole
      driver, that will inevitably require factoring out some pieces into
      separate functions, but that's going to be a major effort and therefore I
      want to do this separately, with no functional changes made at the same
      time.  If anyone has specific suggestions as to how to reformat any of the
      pieces submitted here for a better layout, then I'll be happy to take them
      into account.
      
       And last but not least many thanks to Robert Coerver, who was the most
      recent person to report this problem with the driver and was kind enough
      to patiently try a few revisions of the driver update on his system as I
      was finding and addressing issues.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      284a83a0
    • Maciej W. Rozycki's avatar
      defxx: Add missing DMA synchronisation calls · 8848761f
      Maciej W. Rozycki authored
      This adds DMA synchronisation calls needed in the receive path:
      
      1. To retrieve the Receive Status word that is prepended by the PDQ DMA
         engine in the receive buffer, and provides information about the
         frame received, including its size and any errors.
      
      2. To make data received available for copying in the small-frame case
         (size <= SKBUFF_RX_COPYBREAK) where the original DMA buffer will be
         returned to the receive descriptor ring and therefore its mapping
         retained.
      
         With DMA mapping error handling in place, added by the other patch,
         this may now also trigger where an attempt to map a newly allocated
         buffer for DMA has failed.  In that case data from the original buffer
         will be copied out and the buffer returned to the DMA descriptor ring.
      
      These calls may do nothing when data is in the host DMA addressing range
      of the FDDI interface, such as always on 32-bit systems, however their
      absence makes frame reception stop functioning reliably on systems that
      have memory beyond the low 4GB of the address space.
      Reported-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Tested-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Signed-off-by: default avatarMaciej W. Rozycki <macro@linux-mips.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      8848761f
    • Maciej W. Rozycki's avatar
      defxx: Handle DMA mapping errors · b37cccf0
      Maciej W. Rozycki authored
      This adds error handling for DMA mapping requests; I think there isn't
      much else to say about it.
      
      A good side-effect is the mapping in the transmit path is now made with
      the board lock released.  Also if DMA mapping fails for a newly
      allocated receive buffer, then data from the old buffer will be copied
      out (as is presently done for small frames only whose size does not
      exceed SKBUFF_RX_COPYBREAK) and the original buffer returned, with its
      mapping unchanged, to the DMA descriptor ring.
      Reported-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Tested-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Signed-off-by: default avatarMaciej W. Rozycki <macro@linux-mips.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      b37cccf0
    • Maciej W. Rozycki's avatar
      defxx: Use netdev_alloc_skb consistently · a630be70
      Maciej W. Rozycki authored
      Switch the two remaining places across the driver that use dev_alloc_skb
      to netdev_alloc_skb.  Another place has already been converted to use
      __netdev_alloc_skb, no idea why these two have been left behind.
      Reported-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Tested-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Signed-off-by: default avatarMaciej W. Rozycki <macro@linux-mips.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      a630be70
    • Maciej W. Rozycki's avatar
      defxx: Discard DMA maps on buffer deallocation · 6329fe5c
      Maciej W. Rozycki authored
      Prearranged receive DMA bounce buffer mappings are not released in the
      card reboot/shutdown path.  That does not affect frame reception, but
      probably explains the random segmentation fault I observed the other day
      on interface shutdown.  Card is rebooted as required by the spec in the
      process of ring fault recovery when a PC Trace signal has been received.
      
      This change fixes the problem in an obvious manner.
      Reported-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Tested-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Signed-off-by: default avatarMaciej W. Rozycki <macro@linux-mips.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      6329fe5c
    • Maciej W. Rozycki's avatar
      defxx: Correct the receive DMA map size · d68ab591
      Maciej W. Rozycki authored
      Receive DMA maps are oversized, they include EISA legacy 128-byte
      alignment padding in size calculation whereas this padding is never used
      for data.  Worse yet, if the skb's data area has been realigned indeed,
      then data beyond the end of the buffer will be synchronised from the
      receive DMA bounce buffer, possibly corrupting data structures residing
      in memory beyond the actual end of this data buffer.
      
      Therefore switch to using PI_RCV_DATA_K_SIZE_MAX rather than NEW_SKB_SIZE
      in DMA mapping, the value the former macro expands to is written to the
      receive ring DMA descriptor of the PDQ DMA chip and determines the
      maximum amount of data PDQ will ever transfer to the corresponding data
      buffer, including all headers and padding.
      Reported-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Tested-by: default avatarRobert Coerver <Robert.Coerver@ll.mit.edu>
      Signed-off-by: default avatarMaciej W. Rozycki <macro@linux-mips.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      d68ab591
    • David S. Miller's avatar
      Merge branch 'sctp_command_queue' · 90fb5679
      David S. Miller authored
      David Laight says:
      
      ====================
      net: sctp: Optimisations to sctp command queue code
      
      These 3 patches optimise the code that processes sctp's command queue.
      (A list of 'tasks' to be performed after the rest of the chunk processing.)
      
      1) Inline all the functions from command.c
      2) Remove the memset() calls used to zero a word-sized union.
      3) Use pointers instead of array indexes.
      
      The combined changes reduce the code size (amd64) by a few kb.
      
      I'm not 100% convinced that the zeroing done in patch 2 is needed at all.
      On BE systems it is likely to generate more code than on LE ones.
      In fact it might be best to change the union to only contain 'long' sized
      items.
      
      Changes for v2:
      	- Add some missing initialisers in patch 2/3 and delete them in 3/3.
      	- Modify the commit message for 2/3 to point out that the union
      	  shouldn't need to be zeroed, but the patches aren't intended to
      	  change the behaviour even if the code is buggy.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      90fb5679
    • David Laight's avatar
      net: sctp: Use pointers (not array indexes) to access sctp_cmd_seq_t.cmds[]. · d1a3fe26
      David Laight authored
      Using pointers into sctp_cmd_seq_t.cmds[] lets the compiler generate much
      better code.
      Use the last entry first to optimise the overflow check.
      Signed-off-by: default avatarDavid Laight <david.laight@aculab.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      d1a3fe26
    • David Laight's avatar
      net: sctp: Optimise the way 'sctp_arg_t' values are initialised. · b9420e1c
      David Laight authored
      Even if memset() is inlined (as on x86) using it to zero the union
      generates a memory word write of zero, followed by a write of the
      smaller field, and then a read of the word.
      As well as being a lot of instructions the sequence is unlikely to
      be optimised by the store-load forward hardware so will be slow.
      
      Instead allocate a field of the union that is the same size as the
      entire union and write a zero value to it. The compiler will then
      generate the required value in a register.
      
      Zeroing the union shouldn't be necessary, but this patch series isn't
      intended to have a behavioural change.
      Signed-off-by: default avatarDavid Laight <david.laight@aculab.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      b9420e1c
    • David Laight's avatar
      net: sctp: Inline the functions from command.c · be1f4f48
      David Laight authored
      sctp_init_cmd_seq() and sctp_next_cmd() are only called from one place.
      The call sequence for sctp_add_cmd_sf() is likely to be longer than
      the inlined code.
      With sctp_add_cmd_sf() inlined the compiler can optimise repeated calls.
      Signed-off-by: default avatarDavid Laight <david.laight@aculab.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      be1f4f48