1. 26 Jan, 2015 1 commit
  2. 25 Jan, 2015 39 commits
    • Harout Hedeshian's avatar
      net: ipv6: Add sysctl entry to disable MTU updates from RA · c2943f14
      Harout Hedeshian authored
      The kernel forcefully applies MTU values received in router
      advertisements provided the new MTU is less than the current. This
      behavior is undesirable when the user space is managing the MTU. Instead
      a sysctl flag 'accept_ra_mtu' is introduced such that the user space
      can control whether or not RA provided MTU updates should be applied. The
      default behavior is unchanged; user space must explicitly set this flag
      to 0 for RA MTUs to be ignored.
      Signed-off-by: default avatarHarout Hedeshian <harouth@codeaurora.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      c2943f14
    • David S. Miller's avatar
      Merge branch 'fib_trie_next' · 46a93af2
      David S. Miller authored
      Alexander Duyck says:
      
      ====================
      Fixes and improvements for recent fib_trie updates
      
      While performing testing and prepping the next round of patches I found a
      few minor issues and improvements that could be made.
      
      These changes should help to reduce the overall code size and improve the
      performance slighlty as I noticed a 20ns or so improvement in my worst-case
      testing which will likely only result in a 1ns difference with a standard
      sized trie.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      46a93af2
    • Alexander Duyck's avatar
      fib_trie: Various clean-ups for handling slen · 64c62723
      Alexander Duyck authored
      While doing further work on the fib_trie I noted a few items.
      
      First I was using calls that were far more complicated than they needed to
      be for determining when to push/pull the suffix length.  I have updated the
      code to reflect the simplier logic.
      
      The second issue is that I realised we weren't necessarily handling the
      case of a leaf_info struct surviving a flush.  I have updated the logic so
      that now we will call pull_suffix in the event of having a leaf info value
      left in the leaf after flushing it.
      Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@redhat.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      64c62723
    • Alexander Duyck's avatar
      fib_trie: Move fib_find_alias to file where it is used · 02525368
      Alexander Duyck authored
      The function fib_find_alias is only accessed by functions in fib_trie.c as
      such it makes sense to relocate it and cast it as static so that the
      compiler can take advantage of optimizations it can do to it as a local
      function.
      Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@redhat.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      02525368
    • Alexander Duyck's avatar
      fib_trie: Use empty_children instead of counting empty nodes in stats collection · 30cfe7c9
      Alexander Duyck authored
      It doesn't make much sense to count the pointers ourselves when
      empty_children already has a count for the number of NULL pointers stored
      in the tnode.  As such save ourselves the cycles and just use
      empty_children.
      Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@redhat.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      30cfe7c9
    • Alexander Duyck's avatar
      fib_trie: Add collapse() and should_collapse() to resize · 95f60ea3
      Alexander Duyck authored
      This patch really does two things.
      
      First it pulls the logic for determining if we should collapse one node out
      of the tree and the actual code doing the collapse into a separate pair of
      functions.  This helps to make the changes to these areas more readable.
      
      Second it encodes the upper 32b of the empty_children value onto the
      full_children value in the case of bits == KEYLENGTH.  By doing this we are
      able to handle the case of a 32b node where empty_children would appear to
      be 0 when it was actually 1ul << 32.
      Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@redhat.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      95f60ea3
    • Alexander Duyck's avatar
      fib_trie: Fall back to slen update on inflate/halve failure · a80e89d4
      Alexander Duyck authored
      This change corrects an issue where if inflate or halve fails we were
      exiting the resize function without at least updating the slen for the
      node.  To correct this I have moved the update of max_size into the while
      loop so that it is only decremented on a successful call to either inflate
      or halve.
      Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@redhat.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      a80e89d4
    • Alexander Duyck's avatar
      fib_trie: Fix RCU bug and merge similar bits of inflate/halve · 69fa57b1
      Alexander Duyck authored
      This patch addresses two issues.
      
      The first issue is the fact that I believe I had the RCU freeing sequence
      slightly out of order.  As a result we could get into an issue if a caller
      went into a child of a child of the new node, then backtraced into the to be
      freed parent, and then attempted to access a child of a child that may have
      been consumed in a resize of one of the new nodes children.  To resolve this I
      have moved the resize after we have freed the oldtnode.  The only side effect
      of this is that we will now be calling resize on more nodes in the case of
      inflate due to the fact that we don't have a good way to test to see if a
      full_tnode on the new node was there before or after the allocation.  This
      should have minimal impact however since the node should already be
      correctly size so it is just the cost of calling should_inflate that we
      will be taking on the node which is only a couple of cycles.
      
      The second issue is the fact that inflate and halve were essentially doing
      the same thing after the new node was added to the trie replacing the old
      one.  As such it wasn't really necessary to keep the code in both functions
      so I have split it out into two other functions, called replace and
      update_children.
      Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@redhat.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      69fa57b1
    • Alexander Duyck's avatar
      fib_trie: Use index & (~0ul << n->bits) instead of index >> n->bits · b3832117
      Alexander Duyck authored
      In doing performance testing and analysis of the changes I recently found
      that by shifting the index I had created an unnecessary dependency.
      
      I have updated the code so that we instead shift a mask by bits and then
      just test against that as that should save us about 2 CPU cycles since we
      can generate the mask while the key and pos are being processed.
      Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@redhat.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      b3832117
    • David S. Miller's avatar
      Merge branch 'mlx4-next' · bc579ae5
      David S. Miller authored
      Or Gerlitz says:
      
      ====================
      mlx4: Fix and enhance the device reset flow
      
      This series from Yishai Hadas fixes the device reset flow and adds SRIOV support.
      
      Reset flows are required whenever a device experiences errors, is unresponsive,
      or is not in a deterministic state. In such cases, the driver is expected to
      reset the HW and continue operation. When SRIOV is enabled, these requirements
      apply both to PF and VF devices.
      
      Currently, the mlx4 reset flow doesn't work properly: when a fatal error is
      detected on the FW internal buffer the chip is not reset and stays in its
      bad state. There are cases that assumed to be fatal such as non-responsive FW,
      errors via closing commands but are not handled today.
      
      The AER mechanism should also be fixed:
      - It should use mlx4_load_one instead of __mlx4_init_one which is done
        upon HCA probing.
      - It must be aligned with concurrent catas flow, mark device to be in
        an error state, reset chip, etc.
      - Port types should be restored to their original values before error occurred.
      
      In addition, there the SRIOV use-case isn't supported.
      
      In above cases when the device state becomes fatal we must act as follows:
      1) Reset the chip and mark the HW device state as in fatal error.
      2) Wake up any pending commands, preventing new ones to come in.
      3) Restart the software stack.
      
      We also address the SRIOV mode as follows: In case the PF detects a fatal error,
      it lets VFs know about that, then both itself and VFs are restarted asynchronously.
      However, in case only the VF encountered a fatal case or forced to be reset, they
      reset the VF stuff and then restart software.
      
      changes from V0:
      
      No need to call pci_disable_device upon permanent PCI error. This will
      be done as part of mlx4_remove_one which is called later once we
      return PCI_ERS_RESULT_DISCONNECT from the pci error handler.
      
      Initial toggle value should use only the T bit and not the whole byte value.
      Not doing so sometimes broke SRIOV as of junky value seen by the VF as a
      non-ready comm channel
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      bc579ae5
    • Yishai Hadas's avatar
      net/mlx4_core: Reset flow activation upon SRIOV fatal command cases · 0cd93027
      Yishai Hadas authored
      When SRIOV commands are executed over the comm-channel and get
      a fatal error (e.g. timeout, closing command failure) the VF enters
      into error state and reset flow is activated.
      
      To be able to recognize whether the failure was on a closing command, the
      operational code for the given VHCR command is used. Once the device entered
      into an error state we prevent redundant error messages from being printed.
      Signed-off-by: default avatarYishai Hadas <yishaih@mellanox.com>
      Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      0cd93027
    • Yishai Hadas's avatar
      net/mlx4_core: Enable device recovery flow with SRIOV · 55ad3592
      Yishai Hadas authored
      In SRIOV, both the PF and the VF may attempt device recovery whenever they
      assume that the device is not functioning.  When the PF driver resets the
      device, the VF should detect this and attempt to reinitialize itself.
      
      The VF must be able to reset itself under all circumstances, even
      if the PF is not responsive.
      
      The VF shall reset itself in the following cases:
      
      1. Commands are not processed within reasonable time over the communication channel.
      This is done considering device state and the correct return code based on
      the command as was done in the native mode, done in the next patch.
      
      2. The VF driver receives an internal error event reported by the PF on the
      communication channel. This occurs when the PF driver resets the device or
      when VF is out of sync with the PF.
      
      Add 'VF reset' capability, which allows the VF to reinitialize itself even when the
      PF is not responsive.
      
      As PF and VF may run their reset flow simulantanisly, there are several cases
      that are handled:
      - Prevent freeing VF resources upon FLR, when PF is in its unloading stage.
      - Prevent PF getting VF commands before it has finished initializing its resources.
      - Upon VF startup, check that comm-channel is online before sending
        commands to the PF and getting timed-out.
      Signed-off-by: default avatarYishai Hadas <yishaih@mellanox.com>
      Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      55ad3592
    • Yishai Hadas's avatar
      net/mlx4_core: Handle AER flow properly · 2ba5fbd6
      Yishai Hadas authored
      Fix AER callbacks to work properly, it includes:
      - Refractoring AER to be aligned with Reset flow support.
      - Sync with concurrent catas flow.
      
      In addition, fix the shutdown PCI callback to sync with
      concurrent catas flow.
      Signed-off-by: default avatarYishai Hadas <yishaih@mellanox.com>
      Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      2ba5fbd6
    • Yishai Hadas's avatar
      net/mlx4_core: Manage interface state for Reset flow cases · c69453e2
      Yishai Hadas authored
      We need to manage interface state to sync between reset flow and some other
      relative cases such as remove_one. This has to be done to prevent certain
      races. For example in case software stack is down as a result of unload call,
      the remove_one should skip the unload phase.
      
      Implement the remove_one case, handling AER and other cases comes next.
      
      The interface can be up/down, upon remove_one, the state will include an extra
      bit indicating that the device is cleaned-up, forcing other tasks to finish
      before the final cleanup.
      Signed-off-by: default avatarYishai Hadas <yishaih@mellanox.com>
      Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      c69453e2
    • Yishai Hadas's avatar
      net/mlx4_core: Activate reset flow upon fatal command cases · f5aef5aa
      Yishai Hadas authored
      We activate reset flow upon command fatal errors, when the device enters an
      erroneous state, and must be reset.
      
      The cases below are assumed to be fatal: FW command timed-out, an error from FW
      on closing commands, pci is offline when posting/pending a command.
      
      In those cases we place the device into an error state: chip is reset, pending
      commands are awakened and completed immediately. Subsequent commands will
      return immediately.
      
      The return code in the above cases will depend on the command. Commands which
      free and close resources will return success (because the chip was reset, so
      callers may safely free their kernel resources). Other commands will return -EIO.
      
      Since the device's state was marked as error, the catas poller will
      detect this and restart the device's software stack (as is done when a FW
      internal error is directly detected). The device state is protected by a
      persistent mutex lives on its mlx4_dev, as such no need any more for the
      hcr_mutex which is removed.
      Signed-off-by: default avatarYishai Hadas <yishaih@mellanox.com>
      Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      f5aef5aa
    • Yishai Hadas's avatar
      net/mlx4_core: Enhance the catas flow to support device reset · f6bc11e4
      Yishai Hadas authored
      This includes:
      
      - resetting the chip when a fatal error is detected (the current code
        does not do this).
      
      - exposing the ability to enter error state from outside the catas code
        by calling its functionality. (E.g. FW Command timeout, AER error).
      
      - managing a persistent device state. This is needed to sync between
        reset flow cases.
      Signed-off-by: default avatarYishai Hadas <yishaih@mellanox.com>
      Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      f6bc11e4
    • Yishai Hadas's avatar
      net/mlx4_core: Refactor the catas flow to work per device · ad9a0bf0
      Yishai Hadas authored
      Using a WQ per device instead of a single global WQ, this allows
      independent reset handling per device even when SRIOV is used.
      
      This comes as a pre-patch for supporting chip reset
      for both native and SRIOV.
      Signed-off-by: default avatarYishai Hadas <yishaih@mellanox.com>
      Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      ad9a0bf0
    • Yishai Hadas's avatar
      net/mlx4_core: Set device configuration data to be persistent across reset · dd0eefe3
      Yishai Hadas authored
      When an HCA enters an internal error state, this is detected by the driver.
      The driver then should reset the HCA and restart the software stack.
      
      Keep ports information and some SRIOV configuration in a persistent area
      to have it valid across reset.
      Signed-off-by: default avatarYishai Hadas <yishaih@mellanox.com>
      Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      dd0eefe3
    • Yishai Hadas's avatar
      net/mlx4_core: Maintain a persistent memory for mlx4 device · 872bf2fb
      Yishai Hadas authored
      Maintain a persistent memory that should survive reset flow/PCI error.
      This comes as a preparation for coming series to support above flows.
      Signed-off-by: default avatarYishai Hadas <yishaih@mellanox.com>
      Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      872bf2fb
    • Andy Shevchenko's avatar
      cxgb3: re-use native hex2bin() · 7aee42c6
      Andy Shevchenko authored
      Call hex2bin() library function instead of doing conversion here.
      Signed-off-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      7aee42c6
    • Andy Shevchenko's avatar
      usbnet: re-use native hex2bin() · 51487ae7
      Andy Shevchenko authored
      Call hex2bin() library function, instead of doing conversion here.
      Signed-off-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
      Acked-by: default avatarOliver Neukum <oneukum@suse.de>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      51487ae7
    • David S. Miller's avatar
      Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next · bc0247a4
      David S. Miller authored
      Jeff Kirsher says:
      
      ====================
      Intel Wired LAN Driver Updates 2015-01-22
      
      This series contains updates to e1000, e1000e, igb, fm10k and virtio_net.
      
      Asaf Vertz provides a fix for e1000 to future-proof the time comparisons
      by using time_after_eq() instead of plain math.
      
      Mathias Koehrer provides a fix for e1000e to add a check to e1000_xmit_frame()
      to ensure a work queue will not be scheduled that has not been initialized.
      
      Jacob adds the use of software timestamping via the virtio_net driver.
      
      Alex Duyck cleans up page reuse code in igb and fm10k.  Cleans up the
      page reuse code from getting into a state where all the workarounds
      needed are in place as well as cleaning up oversights, such as using
      __free_pages instead of put_page to drop a locally allocated page.
      
      Richard Cochran provides 4 patches for igb dealing with time sync.
      First provides a helper function since the code that handles the time
      sync interrupt is repeated in three different places.  Then serializes
      the access to the time sync interrupt since the registers may be
      manipulated from different contexts.  Enables the use of i210 device
      interrupt to generate an internal PPS event for adjusting the kernel
      system time.  The i210 device offers a number of special PTP hardware
      clock features on the Software Defined Pins (SDPs), so added support for
      two of the possible functions (time stamping external events and
      periodic output signals).
      
      Or Gerlitz fixes fm10k from double setting of NETIF_F_SG since the
      networking core does it for the driver during registration time.
      
      Joe Stringer adds support for up to 104 bytes of inner+outer headers in
      fm10k and adds an initial check to fail encapsulation offload if these
      are too large.
      
      Matthew increases the timeout for the data path reset based on feedback
      from the hardware team, since 100us is too short of a time to wait for
      the data path reset to complete.
      
      Alexander Graf provides a fix for igb to indicate failure on VF reset
      for an empty MAC address, to mirror the behavior of ixgbe.
      
      Florian Westphal updates e1000 and e1000e to support txtd update delay
      via xmit_more, this way we won't update the Tx tail descriptor if the
      queue has not been stopped and we know at least one more skb will be
      sent right away.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      bc0247a4
    • David S. Miller's avatar
      Merge branch 'vxlan_tx' · 86b368b4
      David S. Miller authored
      Tom Herbert says:
      
      ====================
      vxlan: Don't use UDP socket for transmit
      
      UDP socket is not pertinent to transmit for UDP tunnels, checksum
      enablement can be done without a socket. This patch set eliminates
      reference to a socket in udp_tunnel_xmit functions and in VXLAN
      transmit.
      
      Also, make GBP, RCO, can CSUM6_RX flags visible to receive socket
      and only match these for shareable socket.
      
      v2: Fix geneve to call udp_tunnel_xmit with good arguments.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      86b368b4
    • Tom Herbert's avatar
      vxlan: Eliminate dependency on UDP socket in transmit path · af33c1ad
      Tom Herbert authored
      In the vxlan transmit path there is no need to reference the socket
      for a tunnel which is needed for the receive side. We do, however,
      need the vxlan_dev flags. This patch eliminate references
      to the socket in the transmit path, and changes VXLAN_F_UNSHAREABLE
      to be VXLAN_F_RCV_FLAGS. This mask is used to store the flags
      applicable to receive (GBP, CSUM6_RX, and REMCSUM_RX) in the
      vxlan_sock flags.
      Signed-off-by: default avatarTom Herbert <therbert@google.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      af33c1ad
    • Tom Herbert's avatar
      udp: Do not require sock in udp_tunnel_xmit_skb · d998f8ef
      Tom Herbert authored
      The UDP tunnel transmit functions udp_tunnel_xmit_skb and
      udp_tunnel6_xmit_skb include a socket argument. The socket being
      passed to the functions (from VXLAN) is a UDP created for receive
      side. The only thing that the socket is used for in the transmit
      functions is to get the setting for checksum (enabled or zero).
      This patch removes the argument and and adds a nocheck argument
      for checksum setting. This eliminates the unnecessary dependency
      on a UDP socket for UDP tunnel transmit.
      Signed-off-by: default avatarTom Herbert <therbert@google.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      d998f8ef
    • Nimrod Andy's avatar
      net: fec: fix the warning found by dma debug · 2b995f63
      Nimrod Andy authored
      Enable kernel config "CONFIG_HAVE_DMA_API_DEBUG", FEC have kernel warning:
      [    6.650444] fec 2188000.ethernet: DMA-API: device driver tries to free DMA memory it has not allocated
      [    6.664289] Modules linked in:
      [    6.667378] CPU: 0 PID: 3 Comm: ksoftirqd/0 Not tainted 3.19.0-rc4-00688-g88340160-dirty #150
      [    6.675841] Hardware name: Freescale i.MX6 SoloX (Device Tree)
      [    6.681698] Backtrace:
      [    6.684189] [<80011e3c>] (dump_backtrace) from [<80011fdc>] (show_stack+0x18/0x1c)
      [    6.691789]  r6:80890154 r5:00000000 r4:00000000 r3:00000000
      [    6.697533] [<80011fc4>] (show_stack) from [<806d2d88>] (dump_stack+0x80/0x9c)
      [    6.704799] [<806d2d08>] (dump_stack) from [<8002a4e4>] (warn_slowpath_common+0x7c/0xb4)
      [    6.712917]  r5:00000445 r4:00000000
      [    6.716544] [<8002a468>] (warn_slowpath_common) from [<8002a5c0>] (warn_slowpath_fmt+0x38/0x40)
      [    6.725265]  r8:809a2ee8 r7:00000000 r6:00000000 r5:00000000 r4:00000042
      [    6.732087] [<8002a58c>] (warn_slowpath_fmt) from [<802d6268>] (check_unmap+0x86c/0x98c)
      [    6.740202]  r3:808c79bc r2:8089060c
      [    6.743826] [<802d59fc>] (check_unmap) from [<802d65e4>] (debug_dma_unmap_page+0x80/0x88)
      [    6.752029]  r10:00000000 r9:00000000 r8:00000000 r7:00000001 r6:be12a410 r5:00000000
      [    6.759967]  r4:00000042
      [    6.762538] [<802d6564>] (debug_dma_unmap_page) from [<80440248>] (fec_enet_rx_napi+0x7ec/0xb9c)
      [    6.771345]  r7:00000400 r6:be3e4000 r5:bf08fa20 r4:be036000
      [    6.777094] [<8043fa5c>] (fec_enet_rx_napi) from [<8056ae24>] (net_rx_action+0x134/0x324)
      [    6.785297]  r10:be089e60 r9:80998180 r8:ffff8d68 r7:0000012c r6:00000040 r5:00000001
      [    6.793239]  r4:be036718
      [    6.795801] [<8056acf0>] (net_rx_action) from [<8002db24>] (__do_softirq+0x138/0x2d0)
      [    6.803655]  r10:00000003 r9:00000003 r8:80996378 r7:8099c080 r6:00000100 r5:8099c08c
      [    6.811593]  r4:00000000
      [    6.814157] [<8002d9ec>] (__do_softirq) from [<8002dd00>] (run_ksoftirqd+0x44/0x5c)
      [    6.821836]  r10:00000000 r9:00000000 r8:809b133c r7:00000000 r6:00000001 r5:00000000
      [    6.829775]  r4:be027e80
      [    6.832346] [<8002dcbc>] (run_ksoftirqd) from [<80048290>] (smpboot_thread_fn+0x154/0x1c4)
      [    6.840649] [<8004813c>] (smpboot_thread_fn) from [<80044780>] (kthread+0xdc/0xf8)
      [    6.848224]  r10:00000000 r8:00000000 r7:8004813c r6:be027e80 r5:be027ec0 r4:00000000
      [    6.856179] [<800446a4>] (kthread) from [<8000ebc8>] (ret_from_fork+0x14/0x2c)
      [    6.863425]  r7:00000000 r6:00000000 r5:800446a4 r4:be027ec0
      [    6.869156] ---[ end trace 861cf914d2461a8b ]---
      
      There have one bug in .fec_enet_tx_queue() function to unmap the DMA memory:
      For SG or TSO, get one buffer descriptor and then unmap the related DMA memory, and then
      get the next buffer descriptor, loop to while() to check "TX_READY". If "TX_READY" bit
      still __IS__ existed in the BD (The next fraglist or next TSO packet is not transmited
      complitely), exit the current clean work. When the next work is triggered, it still repeat
      above step with the same BD. The potential issue is that unmap the same DMA memory for
      multiple times.
      
      The patch fix the clean work for SG and TSO packet.
      Reported-by: default avatarAnand Moon <moon.linux@yahoo.com>
      Reported-by: default avatarChristian Gmeiner <christian.gmeiner@gmail.com>
      Signed-off-by: default avatarFugang Duan <B38611@freescale.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      2b995f63
    • David S. Miller's avatar
      Merge branch 'be2net-next' · 58d94227
      David S. Miller authored
      Sathya Perla says:
      
      ====================
      be2net: patch set
      
      Hi David, as the below patch-set includes minor bug fixes and some
      code re-org, pls consider applying it to the "net-next" tree. Thanks!
      
      Patch 1 fixes a bit of code duplication involving interface object
      creation code.
      
      Patch 2 ensures that when a flow-control FW cmd fails, the adapter state
      continues to reflect the old values. This allows for correct reporting
      on subsequent ethtool "get".
      
      Patch 3 returns proper error for link config change on BE3/Lancer
      
      Patch 4 adds a kernel log message for FW boot error on Lancer
      
      Patch 5 adds a function reset on Lancer as a part of the function init
      sequence.
      
      Patch 6 moves some FW-cmd definitions that belong in be_cmds.h, but
      were placed in be_hw.h
      
      Patch 7 resets the "bw_min" field while configuring "bw_max" needed
      for TX rate limiting config.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      58d94227
    • Kalesh AP's avatar
      be2net: Fix TX rate limiting on Lancer/Skyhawk-R VFs · 58bdeaa6
      Kalesh AP authored
      When max_tx_rate is set via bw_max in the NIC resource desc, bw_min must be
      set to 0.
      Signed-off-by: default avatarKalesh AP <kalesh.purayil@emulex.com>
      Signed-off-by: default avatarSathya Perla <sathya.perla@emulex.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      58bdeaa6
    • Vasundhara Volam's avatar
      be2net: move definitions related to FW cmdsfrom be_hw.h to be_cmds.h · e02cfd96
      Vasundhara Volam authored
      Some FW cmd related definitions were included in be_hw.h
      Signed-off-by: default avatarVasundhara Volam <vasundhara.volam@emulex.com>
      Signed-off-by: default avatarSathya Perla <sathya.perla@emulex.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      e02cfd96
    • Kalesh AP's avatar
      be2net: issue function reset cmd in resume path · 9a6d73d9
      Kalesh AP authored
      The Lancer FW is picky about requiring a function reset FW cmd as a part
      of the initialization sequence.
      Signed-off-by: default avatarKalesh AP <kalesh.purayil@emulex.com>
      Signed-off-by: default avatarSathya Perla <sathya.perla@emulex.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      9a6d73d9
    • Kalesh AP's avatar
      be2net: add a log message for POST timeout in Lancer · e673244a
      Kalesh AP authored
      This patch adds a log message in case of POST timeout in Lancer to
      help debugging failure cases. It also logs sliport_status register value in
      case of POST timeout.
      Signed-off-by: default avatarKalesh AP <kalesh.purayil@emulex.com>
      Signed-off-by: default avatarSathya Perla <sathya.perla@emulex.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      e673244a
    • Kalesh AP's avatar
      be2net: fail VF link config change via ndo_set_vf_link_state() on BE3/Lancer · 18fd6025
      Kalesh AP authored
      The support for this exists only in skyhawk FW.
      Signed-off-by: default avatarKalesh AP <kalesh.purayil@emulex.com>
      Signed-off-by: default avatarSathya Perla <sathya.perla@emulex.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      18fd6025
    • Kalesh AP's avatar
      be2net: fix failure case in setting flow control · 00d594c3
      Kalesh AP authored
      When the FW cmd to set flow control fails, the adapter state must simply
      reflect the old values.
      Signed-off-by: default avatarKalesh AP <kalesh.purayil@emulex.com>
      Signed-off-by: default avatarSathya Perla <sathya.perla@emulex.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      00d594c3
    • Kalesh AP's avatar
      be2net: move interface create code to a separate routine · 0700d816
      Kalesh AP authored
      This removes a bit of duplication of code that initializes the en_flags.
      Signed-off-by: default avatarKalesh AP <kalesh.purayil@emulex.com>
      Signed-off-by: default avatarSathya Perla <sathya.perla@emulex.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      0700d816
    • David S. Miller's avatar
      Merge branch 'stmmac-dwmac-rk' · 5ec27425
      David S. Miller authored
      Romain Perier says:
      
      ====================
      net: stmmac: dwmac-rk: Fix phy regulator issues
      
      This series fixes few issues in dwmac-rk:
      
      1. Voltage settings was hardcoded into the driver for the phy regulator.
         The driver now uses the default voltage settings found in the devicetree,
         which are applied throught the regulator framework.
      2. The regulator name used to power on or power off the phy was put in the devicetree
         variable "phy_regulator", which is not standard and added a lot of code for nothing.
         The driver now uses the devicetree property "phy-supply" and the corresponding functions
         to manipulate this regulator.
      
      The corresponding devicetree files are also updated. As this new binding for
      rk3288 has not been released with any official kernel yet (not until 3.20),
      I don't need to care about keeping compatibility with the old non standard property.
      ====================
      Tested-by: default avatarHeiko Stuebner <heiko@sntech.de>
      Reviewed-by: default avatarHeiko Stuebner <heiko@sntech.de>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      5ec27425
    • Romain Perier's avatar
      dt-bindings: Document phy-supply property in rockchip-dwmac · 384dd55d
      Romain Perier authored
      As no property for phy regulator was documented in this dt-binding documentation,
      this commit adds an entry for the optional property phy-supply.
      Signed-off-by: default avatarRomain Perier <romain.perier@gmail.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      384dd55d
    • Romain Perier's avatar
      net: stmmac: dwmac-rk: Use standard devicetree property for phy regulator · 2e12f536
      Romain Perier authored
      Currently, dwmac-rk uses a custom propety "phy_regulator" to get the name of the
      right regulator to use to power on or power off the phy. This commit converts the
      driver to use phy-supply devicetree property and the corresponding API, it cleans
      the code a bit and make it simpler to maintain. This also replaces the property
      phy_regulator by the standard property phy-supply in rk3288-evb-rk808.dts.
      Signed-off-by: default avatarRomain Perier <romain.perier@gmail.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      2e12f536
    • Romain Perier's avatar
    • Romain Perier's avatar
      net: stmmac: dwmac-rk: Don't set the regulator voltage for phy from the driver · 4cc8bfb9
      Romain Perier authored
      As these settings can be directly expressed from devicetree for both fixed
      regulators and pmic-integrated regulators, it is more standard to set them
      from dts and let the regulator framework use the right voltage informations
      when it is used in the driver.
      Signed-off-by: default avatarRomain Perier <romain.perier@gmail.com>
      Tested-by: default avatarHeiko Stuebner <heiko@sntech.de>
      Reviewed-by: default avatarHeiko Stuebner <heiko@sntech.de>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      4cc8bfb9