1. 26 10月, 2012 15 次提交
  2. 24 10月, 2012 7 次提交
  3. 23 10月, 2012 18 次提交
    • G
      ixgbevf: Update version string · 1b3d2d77
      Greg Rose 提交于
      Signed-off-by: NGreg Rose <gregory.v.rose@intel.com>
      Tested-by: NSibai Li <sibai.li@intel.com>
      Signed-off-by: NJeff Kirsher <jeffrey.t.kirsher@intel.com>
      1b3d2d77
    • J
      ixgbevf: fix softirq-safe to unsafe splat on internal mbx_lock · 55fdd45b
      John Fastabend 提交于
      The lockdep splat below identifies a case where irq safe to unsafe
      lock order is detected. Resolved by making mbx_lock bh.
      
      ======================================================
      [ INFO: SOFTIRQ-safe -> SOFTIRQ-unsafe lock order detected ]
      3.6.0-rc5jk-net-next+ #119 Not tainted
      ------------------------------------------------------
      ip/2608 [HC0[0]:SC0[2]:HE1:SE0] is trying to acquire:
       (&(&adapter->mbx_lock)->rlock){+.+...}, at: [<ffffffffa008114e>] ixgbevf_set_rx_mode+0x36/0xd2 [ixgbevf]
      
      and this task is already holding:
       (_xmit_ETHER){+.....}, at: [<ffffffff814097c8>] dev_set_rx_mode+0x1e/0x33
      which would create a new lock dependency:
       (_xmit_ETHER){+.....} -> (&(&adapter->mbx_lock)->rlock){+.+...}
      
      but this new dependency connects a SOFTIRQ-irq-safe lock:
       (&(&mc->mca_lock)->rlock){+.-...}
      ... which became SOFTIRQ-irq-safe at:
        [<ffffffff81092ee5>] __lock_acquire+0x2f2/0xdf3
        [<ffffffff81093b11>] lock_acquire+0x12b/0x158
        [<ffffffff814bdbcd>] _raw_spin_lock_bh+0x4a/0x7d
        [<ffffffffa011a740>] mld_ifc_timer_expire+0x1b2/0x282 [ipv6]
        [<ffffffff81054580>] run_timer_softirq+0x2a2/0x3ee
        [<ffffffff8104cc42>] __do_softirq+0x161/0x2b9
        [<ffffffff814c6a7c>] call_softirq+0x1c/0x30
        [<ffffffff81011bc7>] do_softirq+0x4b/0xa3
        [<ffffffff8104c8d5>] irq_exit+0x53/0xd7
        [<ffffffff814c734d>] do_IRQ+0x9d/0xb4
        [<ffffffff814be56f>] ret_from_intr+0x0/0x1a
        [<ffffffff813de21c>] cpuidle_enter+0x12/0x14
        [<ffffffff813de235>] cpuidle_enter_state+0x17/0x3f
        [<ffffffff813deb6c>] cpuidle_idle_call+0x140/0x21c
        [<ffffffff8101764c>] cpu_idle+0x79/0xcd
        [<ffffffff814a59f5>] rest_init+0x149/0x150
        [<ffffffff81ca2cbc>] start_kernel+0x37c/0x389
        [<ffffffff81ca22dd>] x86_64_start_reservations+0xb8/0xbd
        [<ffffffff81ca23e3>] x86_64_start_kernel+0x101/0x110
      
      to a SOFTIRQ-irq-unsafe lock:
       (&(&adapter->mbx_lock)->rlock){+.+...}
      ... which became SOFTIRQ-irq-unsafe at:
      ...  [<ffffffff81092f59>] __lock_acquire+0x366/0xdf3
        [<ffffffff81093b11>] lock_acquire+0x12b/0x158
        [<ffffffff814bd862>] _raw_spin_lock+0x45/0x7a
        [<ffffffffa0080fde>] ixgbevf_negotiate_api+0x3d/0x6d [ixgbevf]
        [<ffffffffa008404b>] ixgbevf_open+0x6c/0x43e [ixgbevf]
        [<ffffffff8140b2c1>] __dev_open+0xa0/0xe6
        [<ffffffff814099b6>] __dev_change_flags+0xbe/0x142
        [<ffffffff8140b1eb>] dev_change_flags+0x21/0x57
        [<ffffffff8141a523>] do_setlink+0x2e2/0x7f4
        [<ffffffff8141ad8c>] rtnl_newlink+0x277/0x4bb
        [<ffffffff81419c08>] rtnetlink_rcv_msg+0x236/0x253
        [<ffffffff8142f92d>] netlink_rcv_skb+0x43/0x94
        [<ffffffff814199cb>] rtnetlink_rcv+0x26/0x2d
        [<ffffffff8142f6dc>] netlink_unicast+0xee/0x174
        [<ffffffff8142ff12>] netlink_sendmsg+0x26a/0x288
        [<ffffffff813f5a0d>] __sock_sendmsg_nosec+0x58/0x61
        [<ffffffff813f7d57>] __sock_sendmsg+0x3d/0x48
        [<ffffffff813f7ed9>] sock_sendmsg+0x6e/0x87
        [<ffffffff813f93d4>] __sys_sendmsg+0x206/0x288
        [<ffffffff813f95ce>] sys_sendmsg+0x42/0x60
        [<ffffffff814c57a9>] system_call_fastpath+0x16/0x1b
      
      other info that might help us debug this:
      
      Chain exists of:
        &(&mc->mca_lock)->rlock --> _xmit_ETHER --> &(&adapter->mbx_lock)->rlock
      
       Possible interrupt unsafe locking scenario:
      
             CPU0                    CPU1
             ----                    ----
        lock(&(&adapter->mbx_lock)->rlock);
                                     local_irq_disable();
                                     lock(&(&mc->mca_lock)->rlock);
                                     lock(_xmit_ETHER);
        <Interrupt>
          lock(&(&mc->mca_lock)->rlock);
      
       *** DEADLOCK ***
      Signed-off-by: NJohn Fastabend <john.r.fastabend@intel.com>
      Acked-by: NGreg Rose <gregory.v.rose@intel.com>
      Tested-by: NSibai Li <sibai.li@intel.com>
      Signed-off-by: NJeff Kirsher <jeffrey.t.kirsher@intel.com>
      55fdd45b
    • G
      ixgbevf: Check for error on dma_map_single call · 6132ee8a
      Greg Rose 提交于
      Ignoring the return value from a call to the kernel dma_map API functions
      can cause data corruption and system instability.  Check the return value
      and take appropriate action.
      Signed-off-by: NGreg Rose <gregory.v.rose@intel.com>
      Tested-by: NPhil Schmitt <phillip.j.schmitt@intel.com>
      Tested-by: NSibai Li <sibai.li@intel.com>
      Signed-off-by: NJeff Kirsher <jeffrey.t.kirsher@intel.com>
      6132ee8a
    • J
      ixgbevf: make netif_napi_add and netif_napi_del symmetric · f4477702
      John Fastabend 提交于
      ixgbevf_alloc_q_vectors() calls netif_napi_add for each qvector
      where qvectors is determined by the number of msix vectors. This
      makes perfect sense.
      
      However on cleanup when ixgbevf_free_q_vectors() is called and
      for each qvector we should call netif_napi_del there is some
      extra logic to add a dependency on RX queues. This patch makes
      the add/del operations symmetric by removing the RX queues
      dependency.
      
      Without this if  free_netdev() is called we see the general
      protection fault below in netif_napi_del when list_del_init()
      is called.
      
      # addr2line -e ./vmlinux ffffffff8140810c
      net-next/include/linux/list.h:88
      
      general protection fault: 0000 [#1] SMP
      Modules linked in: bonding ixgbevf ixgbe(-) mdio libfc scsi_transport_fc scsi_tgt 8021q garp stp llc cpufreq_ondemand acpi_cpufreq freq_table mperf ipv6 uinput coretemp lpc_ich i2c_i801 shpchp hwmon i2c_core serio_raw crc32c_intel mfd_core joydev pcspkr microcode ioatdma igb dca pata_acpi ata_generic usb_storage pata_jmicron [last unloaded: bonding]
      CPU 10
      Pid: 4174, comm: rmmod Tainted: G        W    3.6.0-rc3jk-net-next+ #104 Supermicro X8DTN/X8DTN
      RIP: 0010:[<ffffffff8140810c>]  [<ffffffff8140810c>] netif_napi_del+0x24/0x87
      RSP: 0018:ffff88027f5e9b48  EFLAGS: 00010293
      RAX: ffff8806224b4768 RBX: ffff8806224b46e8 RCX: 6b6b6b6b6b6b6b6b
      RDX: 6b6b6b6b6b6b6b6b RSI: ffffffff810bf6c5 RDI: ffff8806224b46e8
      RBP: ffff88027f5e9b58 R08: ffff88033200b180 R09: ffff88027f5e98a8
      R10: ffff88033320b000 R11: ffff88027f5e9ae8 R12: 6b6b6b6b6b6b6aeb
      R13: ffff8806221d11c0 R14: 0000000000000000 R15: ffff88027f5e9cf8
      FS:  00007f5e58b9b700(0000) GS:ffff880333200000(0000) knlGS:0000000000000000
      CS:  0010 DS: 0000 ES: 0000 CR0: 000000008005003b
      CR2: 00000000010ef2b8 CR3: 0000000281fff000 CR4: 00000000000007e0
      DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
      DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
      Process rmmod (pid: 4174, threadinfo ffff88027f5e8000, task ffff88032f888000)
      Stack:
       ffff8806221d1160 6b6b6b6b6b6b6aeb ffff88027f5e9b88 ffffffff81408e46
       ffff8806221d1160 ffff8806221d1160 ffff8806221d1ae0 ffff8806221d5668
       ffff88027f5e9bb8 ffffffffa009153c ffffffffa0092a30 ffff8806221d5700
      Call Trace:
       [<ffffffff81408e46>] free_netdev+0x64/0xd7
       [<ffffffffa009153c>] ixgbevf_remove+0xa6/0xbc [ixgbevf]
       [<ffffffff8127a7a1>] pci_device_remove+0x2d/0x51
       [<ffffffff8131f503>] __device_release_driver+0x6c/0xc2
       [<ffffffff8131f640>] device_release_driver+0x25/0x32
       [<ffffffff8131e821>] bus_remove_device+0x148/0x15d
       [<ffffffff8131cb6b>] device_del+0x130/0x1a4
       [<ffffffff8131cc2a>] device_unregister+0x4b/0x57
       [<ffffffff81275c27>] pci_stop_bus_device+0x63/0x85
       [...]
      Signed-off-by: NJohn Fastabend <john.r.fastabend@intel.com>
      Acked-by: NGreg Rose <gregory.v.rose@intel.com>
      Acked-by: NAlexander Duyck <alexander.h.duyck@intel.com>
      Tested-by: NSibai Li <sibai.li@intel.com>
      Signed-off-by: NJeff Kirsher <jeffrey.t.kirsher@intel.com>
      f4477702
    • C
      igb: Update version · 3db73804
      Carolyn Wyborny 提交于
      This patch updates the igb driver version to 4.0.17.
      Signed-off-by: NCarolyn Wyborny <carolyn.wyborny@intel.com>
      Tested-by: NAaron Brown <aaron.f.brown@intel.com>
      Signed-off-by: NJeff Kirsher <jeffrey.t.kirsher@intel.com>
      3db73804
    • C
      igb: Update get cable length function for i210/i211 · 59f30104
      Carolyn Wyborny 提交于
      There was a problem in the initial implementation of the get cable length
      function for i210 and it did not work properly.  This patch fixes that
      problem for i210/i211 devices.
      Signed-off-by: NCarolyn Wyborny <carolyn.wyborny@intel.com>
      Tested-by: NAaron Brown <aaron.f.brown@intel.com>
      Signed-off-by: NJeff Kirsher <jeffrey.t.kirsher@intel.com>
      59f30104
    • J
      maintainers: update with official intel support link, new maintainer · f6fde11a
      Jesse Brandeburg 提交于
      Add an official link which is designed to guide the user to the appropriate
      support resource (be it community, OEM, Intel phone, Intel email, etc)
      
      Add the current e1000 maintainer to the list of Intel maintainers.
      Signed-off-by: NJesse Brandeburg <jesse.brandeburg@intel.com>
      Tested-by: NAaron Brown <aaron.f.brown@intel.com>
      Signed-off-by: NJeff Kirsher <jeffrey.t.kirsher@intel.com>
      f6fde11a
    • T
      e1000e: Minimum packet size must be 17 bytes · 6e97c170
      Tushar Dave 提交于
      This is a HW requirement. Although a buffer as short as 1 byte is allowed,
      the total length of packet before, padding and CRC insertion, must be at
      least 17 bytes.  So pad all small packets manually up to 17 bytes before
      delivering them to HW.
      Signed-off-by: NTushar Dave <tushar.n.dave@intel.com>
      Tested-by: NJeff Pieper <jeffrey.e.pieper@intel.com>
      Signed-off-by: NJeff Kirsher <jeffrey.t.kirsher@intel.com>
      6e97c170
    • D
      Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-next · 598e74f3
      David S. Miller 提交于
      Jeff Kirsher says:
      
      ====================
      This series contains updates to ixgbe only.  Only change to this series
      is I dropped the "ixgbe: Add support for pipeline reset" due to
      change requested by Martin Josefsson.
      
      Alexander Duyck (7):
        ixgbe: Add support for IPv6 and UDP to ixgbe_get_headlen
        ixgbe: Add support for tracking the default user priority to SR-IOV
        ixgbe: Add support for GET_QUEUES message to get DCB configuration
        ixgbe: Enable support for VF API version 1.1 in the PF.
        ixgbevf: Add VF DCB + SR-IOV support
        ixgbe: Drop unnecessary addition from ixgbe_set_rx_buffer_len
        ixgbe: Fix possible memory leak in ixgbe_set_ringparam
      
      Don Skidmore (1):
        ixgbe: Add function ixgbe_reset_pipeline_82599
      
      Emil Tantilov (1):
        ixgbe: add WOL support for new subdevice id
      
      Jacob Keller (1):
        ixgbe: (PTP) refactor init, cyclecounter and reset
      
      Tushar Dave (1):
        ixgbe: Correcting small packet padding
      
      Wei Yongjun (1):
        ixgbe: using is_zero_ether_addr() to simplify the code
      ====================
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      598e74f3
    • B
      net: cdc_mbim: Device Service Stream support · 9f651cac
      Bjørn Mork 提交于
      MBIM devices can support up to 256 generic streams called
      Device Service Streams (DSS). The MBIM spec says
      
         The format of the Device Service Stream payload depends
         on the device service (as identified by the corresponding
         UUID) that is used when opening the data stream.
      
      Example use cases are serial AT command interfaces and NMEA
      data streams. We cannot make any assumptions about these
      device services.
      
      Adding support for Device Service Stream by extending
      the MBIM session to VLAN mapping scheme, allocating
      VLAN IDs 256 to 511 for DSS, using the DSS SessionID
      as the lower 8bit of the VLAN ID.
      
      Using a netdev for DSS keeps the device framing intact and
      allows userspace to do whatever it want with the streams.
      For example, exporting an AT command interface using DSS
      session #0 to a PTY for use with a terminal application like
      minicom:
      
        vconfig add wwan0 256
        ip link set dev wwan0 up
        ip link set dev wwan0.256 up
        socat INTERFACE:wwan0.256,type=2 PTY:,echo=0,link=/tmp/modem
      
      Device configuration must be done using MBIM control commands
      over the /dev/cdc-wdmx device. The userspace management
      application should coordinate host VLAN configuration and the
      device MBIM configuration using the device capabilities to
      find out if it needs to set up PTY mappings etc.
      Signed-off-by: NBjørn Mork <bjorn@mork.no>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      9f651cac
    • B
      net: cdc_ncm: map MBIM IPS SessionID to VLAN ID · a82c7ce5
      Bjørn Mork 提交于
      MBIM devices can support up to 256 independent IP Streams.
      The main network device will only handle SessionID 0. Mapping
      SessionIDs 1 to 255 to VLANs using the SessionID as VLAN ID
      allow userspace to use these streams with traditional tools
      like vconfig.
      Signed-off-by: NBjørn Mork <bjorn@mork.no>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      a82c7ce5
    • B
      net: cdc_ncm: do not bind to NCM compatible MBIM devices · bd329e12
      Bjørn Mork 提交于
      The MBIM specification allows a MBIM device to disguise
      itself as NCM for backwards compatibility, using additional
      altsettings with different subclass (control) or protocol
      (data):
      
      C:* #Ifs= 2 Cfg#= 1 Atr=80 MxPwr=100mA
      I:  If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0d Prot=00 Driver=cdc_mbim
      E:  Ad=81(I) Atr=03(Int.) MxPS=  64 Ivl=7ms
      I:* If#= 0 Alt= 1 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
      E:  Ad=81(I) Atr=03(Int.) MxPS=  64 Ivl=7ms
      I:  If#= 1 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=01 Driver=cdc_mbim
      I:  If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=01 Driver=cdc_mbim
      E:  Ad=81(I) Atr=02(Bulk) MxPS=  64 Ivl=0ms
      E:  Ad=02(O) Atr=02(Bulk) MxPS=  64 Ivl=0ms
      I:* If#= 1 Alt= 2 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
      E:  Ad=81(I) Atr=02(Bulk) MxPS=  64 Ivl=0ms
      E:  Ad=02(O) Atr=02(Bulk) MxPS=  64 Ivl=0ms
      
      If the MBIM driver is enabled then that should have priority
      for devices providing such a NCM 1.0 backward compatibility
      mode.
      Signed-off-by: NBjørn Mork <bjorn@mork.no>
      Signed-off-by: NGreg Suarez <gsuarez@smithmicro.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      bd329e12
    • B
      net: cdc_mbim: build the MBIM driver · 9084ccf9
      Bjørn Mork 提交于
      Signed-off-by: NBjørn Mork <bjorn@mork.no>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      9084ccf9
    • G
      net: cdc_mbim: adding MBIM driver · 9bf211a3
      Greg Suarez 提交于
      The CDC Mobile Broadband Interface Model (MBIM) specification
      extends CDC NCM by
       - removing the redundant ethernet header from the point-to-point
         USB channel
       - adding support for multiple IP (v4 and/or v6) sessions multiplexed
         on the same USB channel
       - adding a MBIM control channel encapsulated in CDC
       - adding Device Service Streams (DSS), which are non IP generic data
         streams multiplexed on the same USB channel as the IP sessions
      
      MBIM devices are managed using the dedicated control channel, and no
      data will flow on the data channel until a control session has been
      established.  This driver has no knowledge of MBIM control messages.
      It just exports the control channel to a /dev/cdc-wdmX character
      device for userspace management applications. Such an application is
      therefore required to use this driver.
      
      This patch implements basic MBIM support, reusing the NCM and WDM driver
      APIs, currently limited to IP sessions with SessionID 0. DSS and
      multiplexed IP sessions are not yet supported.
      Signed-off-by: NGreg Suarez <gsuarez@smithmicro.com>
      Signed-off-by: NBjørn Mork <bjorn@mork.no>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      9bf211a3
    • B
      net: cdc_ncm: export shared symbols and definitions · c91ce3b6
      Bjørn Mork 提交于
      Move symbols and definitons which can be shared with a
      MBIM driver in a new header.
      Signed-off-by: NBjørn Mork <bjorn@mork.no>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      c91ce3b6
    • B
      net: cdc_ncm: refactoring for tx multiplexing · c78b7c58
      Bjørn Mork 提交于
      Adding multiplexed NDP support to cdc_ncm_fill_tx_frame, allowing
      transmissions of multiple independent sessions within the same NTB.
      
      Refactoring the code quite a bit to avoid having to store copies
      of multiple NDPs being prepared for tx. The old code would still
      reserve enough room for a maximum sized NDP in the skb so we might
      as well keep them in the skb while they are being prepared.
      Signed-off-by: NBjørn Mork <bjorn@mork.no>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      c78b7c58
    • B
      net: cdc_ncm: splitting rx_fixup for code reuse · ff06ab13
      Bjørn Mork 提交于
      Verifying and handling received MBIM and NCM frames will need
      to be different in three areas:
       - verifying the NDP signature
       - checking valid datagram length
       - datagram header manipulation
      
      This makes it inconvenient to share rx_fixup in whole.  But
      some verification parts are common.  Split these out in separate
      functions.
      Signed-off-by: NBjørn Mork <bjorn@mork.no>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      ff06ab13
    • B
      net: cdc_ncm: process chained NDPs · 75d67d35
      Bjørn Mork 提交于
      The NCM 1.0 spefication makes provisions for linking more than
      one NDP into a single NTB. This is important for MBIM support,
      where these NDPs might be of different types.
      
      Following the chain of NDPs is also correct for NCM, and will
      not change anything in the common case where there is only
      one NDP
      Signed-off-by: NBjørn Mork <bjorn@mork.no>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      75d67d35