1. 28 Aug, 2014 8 commits
  2. 27 Aug, 2014 7 commits
  3. 26 Aug, 2014 12 commits
  4. 25 Aug, 2014 13 commits
    • Daniel Borkmann's avatar
      ixgbe: support skb->xmit_more in netdev_ops->ndo_start_xmit() · 9c938cdd
      Daniel Borkmann authored
      This implements the deferred tail pointer flush API for the ixgbe
      driver. Similar version also proposed longer time ago by Alexander Duyck.
      Signed-off-by: default avatarDaniel Borkmann <dborkman@redhat.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      9c938cdd
    • David S. Miller's avatar
      net: Remove ndo_xmit_flush netdev operation, use signalling instead. · 0b725a2c
      David S. Miller authored
      As reported by Jesper Dangaard Brouer, for high packet rates the
      overhead of having another indirect call in the TX path is
      non-trivial.
      
      There is the indirect call itself, and then there is all of the
      reloading of the state to refetch the tail pointer value and
      then write the device register.
      
      Move to a more passive scheme, which requires very light modifications
      to the device drivers.
      
      The signal is a new skb->xmit_more value, if it is non-zero it means
      that more SKBs are pending to be transmitted on the same queue as the
      current SKB.  And therefore, the driver may elide the tail pointer
      update.
      
      Right now skb->xmit_more is always zero.
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      0b725a2c
    • David S. Miller's avatar
      Merge branch 'is_kdump_kernel' · 44a52ffd
      David S. Miller authored
      Amir Vadai says:
      
      ====================
      Make is_kdump_kernel() accessible from modules
      
      I'm re-spinning this patchset. At the begining it was suggested to use a
      different name for the parameter, but at the end [3] the resolution was to
      leave it as it is in this patch.
      
      Drivers need to know if running from kdump kernel in order to change their
      memory profile - since kdump environment is limited by available memory.
      Currently there are drivers that are using reset_devices as suggested in [2].
      In [2] it was suggested to use reset_devices, but the context was, to enable
      driver to know when the hardware device is needed to be reset, and not if this
      is a kdump environment. We think that is_kdump_kernel() is better suited to
      select between different memory profiles.
      
      The first patch in this patchset exports a needed symbol in order to make
      is_kdump_kernel() accessible from the drivers. The rest of the patches change
      from reset_devices to is_kdump_kernel() in 2 networking drivers.
      
      The idea of this patchset was suggested by Vivek Goyal.
      
      Tested (only build) and applied on top of commit 8fc54f68: ("net: use
      reciprocal_scale() helper")
      
      [1] - ea1c1af1: ("net/mlx4_en: Reduce memory consumption on kdump kernel")
      [2] - https://lkml.org/lkml/2011/1/27/341
      [3] - http://www.spinics.net/lists/netdev/msg291492.html
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      44a52ffd
    • Amir Vadai's avatar
      net/bnx2x: Use is_kdump_kernel() to detect kdump kernel · c9931896
      Amir Vadai authored
      Use is_kdump_kernel() to detect kdump kernel, instead of
      reset_devices.
      
      CC: Ariel Elior <ariel.elior@qlogic.com>
      CC: Michal Schmidt <mschmidt@redhat.com>
      Signed-off-by: default avatarAmir Vadai <amirv@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      c9931896
    • Amir Vadai's avatar
      net/mlx4: Use is_kdump_kernel() to detect kdump kernel · 48ea526a
      Amir Vadai authored
      Use is_kdump_kernel() to detect kdump kernel, instead of reset_devices.
      Signed-off-by: default avatarAmir Vadai <amirv@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      48ea526a
    • Amir Vadai's avatar
      crash_dump: Make is_kdump_kernel() accessible from modules · b3292e88
      Amir Vadai authored
      In order to make is_kdump_kernel() accessible from modules, need to
      make elfcorehdr_addr exported.
      This was rejected in the past [1] because reset_devices was prefered in
      that context (reseting the device in kdump kernel), but now there are
      some network drivers that need to reduce memory usage when loaded from
      a kdump kernel.  And in that context, is_kdump_kernel() suits better.
      
      [1] - https://lkml.org/lkml/2011/1/27/341
      
      CC: Vivek Goyal <vgoyal@redhat.com>
      Signed-off-by: default avatarAmir Vadai <amirv@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      b3292e88
    • Pavel Machek's avatar
      stmmac: simple cleanups · a77e4acc
      Pavel Machek authored
      This adds simple cleanups for stmmac, removing test we know is always
      true, fixing whitespace, and moving code out of if().
      Signed-off-by: default avatarPavel Machek <pavel@denx.de>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      a77e4acc
    • hayeswang's avatar
      r8152: check code with checkpatch.pl · b209af99
      hayeswang authored
      626: CHECK: Alignment should match open parenthesis
       646: CHECK: Alignment should match open parenthesis
       655: CHECK: Alignment should match open parenthesis
       695: CHECK: Alignment should match open parenthesis
       729: CHECK: Alignment should match open parenthesis
       739: CHECK: Alignment should match open parenthesis
       976: WARNING: externs should be avoided in .c files
       1314: CHECK: Alignment should match open parenthesis
       1358: WARNING: networking block comments don't use an empty /* line, use /* Comment...
       1402: WARNING: networking block comments don't use an empty /* line, use /* Comment...
       1521: CHECK: multiple assignments should be avoided
       1775: CHECK: Alignment should match open parenthesis
       1838: CHECK: multiple assignments should be avoided
       1843: CHECK: multiple assignments should be avoided
       1847: CHECK: multiple assignments should be avoided
       1850: WARNING: Missing a blank line after declarations
       1864: CHECK: Alignment should match open parenthesis
       1872: CHECK: braces {} should be used on all arms of this statement
       1906: CHECK: usleep_range is preferred over udelay
       2865: WARNING: networking block comments don't use an empty /* line, use /* Comment...
       3088: CHECK: Alignment should match open parenthesis
       total: 0 errors, 5 warnings, 16 checks, 3567 lines checked
      Signed-off-by: default avatarHayes Wang <hayeswang@realtek.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      b209af99
    • David S. Miller's avatar
      Merge branch 'ndo_xmit_flush' · fe88e6dd
      David S. Miller authored
      Basic deferred TX queue flushing infrastructure.
      
      Over time, and specifically and more recently at the Networking
      Workshop during Kernel SUmmit in Chicago, we have discussed the idea
      of having some way to optimize transmits of multiple TX packets at
      a time.
      
      There are several areas of overhead that could be amortized with such
      schemes.  One has to do with locking and transactional overhead, the
      other has to do with device specific costs.
      
      This patch set here is more aimed at device specific costs.
      
      Typically a device queues up a packet in the TX queue and then has to
      do something to have the device start processing that new entry.
      Sometimes this is composed of doing an MMIO write to a "tail"
      register, and in other cases it can involve something as expensive as
      a hypervisor call.
      
      The basic setup defined here is that when the driver supports deferred
      TX queue flushing, ndo_start_xmit should no longer perform that
      operation.  Instead a new operation, ndo_xmit_flush, should do it.
      
      I have converted IGB and virtio_net as example initial users.  The IGB
      conversion is tested, virtio_net is not but it does compile :-)
      
      All ndo_start_xmit call sites have been abstracted behind a new helper
      called netdev_start_xmit().
      
      This just adds the infrastructure, it does not actually add any
      instances of actually doing multiple ndo_start_xmit calls per
      ndo_xmit_flush invocation.
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      fe88e6dd
    • David S. Miller's avatar
      c223a078
    • David S. Miller's avatar
      c1ebf46c
    • David S. Miller's avatar
      net: Add ops->ndo_xmit_flush() · 4798248e
      David S. Miller authored
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      4798248e
    • Ian Morris's avatar
      ipv6: White-space cleansing : gaps between function and symbol export · 4c83acbc
      Ian Morris authored
      This patch makes no changes to the logic of the code but simply addresses
      coding style issues as detected by checkpatch.
      
      Both objdump and diff -w show no differences.
      
      This patch removes some blank lines between the end of a function
      definition and the EXPORT_SYMBOL_GPL macro in order to prevent
      checkpatch warning that EXPORT_SYMBOL must immediately follow
      a function.
      Signed-off-by: default avatarIan Morris <ipm@chirality.org.uk>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      4c83acbc