1. 22 3月, 2018 1 次提交
  2. 21 12月, 2017 4 次提交
  3. 03 12月, 2017 2 次提交
    • J
      s390/qeth: build max size GSO skbs on L2 devices · 0cbff6d4
      Julian Wiedmann 提交于
      The current GSO skb size limit was copy&pasted over from the L3 path,
      where it is needed due to a TSO limitation.
      As L2 devices don't offer TSO support (and thus all GSO skbs are
      segmented before they reach the driver), there's no reason to restrict
      the stack in how large it may build the GSO skbs.
      
      Fixes: d52aec97 ("qeth: enable scatter/gather in layer 2 mode")
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      0cbff6d4
    • J
      s390/qeth: fix GSO throughput regression · 6d69b1f1
      Julian Wiedmann 提交于
      Using GSO with small MTUs currently results in a substantial throughput
      regression - which is caused by how qeth needs to map non-linear skbs
      into its IO buffer elements:
      compared to a linear skb, each GSO-segmented skb effectively consumes
      twice as many buffer elements (ie two instead of one) due to the
      additional header-only part. This causes the Output Queue to be
      congested with low-utilized IO buffers.
      
      Fix this as follows:
      If the MSS is low enough so that a non-SG GSO segmentation produces
      order-0 skbs (currently ~3500 byte), opt out from NETIF_F_SG. This is
      where we anticipate the biggest savings, since an SG-enabled
      GSO segmentation produces skbs that always consume at least two
      buffer elements.
      
      Larger MSS values continue to get a SG-enabled GSO segmentation, since
      1) the relative overhead of the additional header-only buffer element
      becomes less noticeable, and
      2) the linearization overhead increases.
      
      With the throughput regression fixed, re-enable NETIF_F_SG by default to
      reap the significant CPU savings of GSO.
      
      Fixes: 5722963a ("qeth: do not turn on SG per default")
      Reported-by: NNils Hoppmann <niho@de.ibm.com>
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      6d69b1f1
  4. 24 11月, 2017 1 次提交
    • G
      s390: net: add SPDX identifiers to the remaining files · ab9953ff
      Greg Kroah-Hartman 提交于
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Update the drivers/s390/net/ files with the correct SPDX license
      identifier based on the license text in the file itself.  The SPDX
      identifier is a legally binding shorthand, which can be used instead of
      the full boiler plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Julian Wiedmann <jwi@linux.vnet.ibm.com>
      Cc: Ursula Braun <ubraun@linux.vnet.ibm.com>
      Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
      Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      Signed-off-by: NHeiko Carstens <heiko.carstens@de.ibm.com>
      Signed-off-by: NMartin Schwidefsky <schwidefsky@de.ibm.com>
      ab9953ff
  5. 20 10月, 2017 2 次提交
  6. 19 9月, 2017 5 次提交
    • J
      s390/qeth: translate SETVLAN/DELVLAN errors · 2aa48671
      Julian Wiedmann 提交于
      Properly return any error encountered during VLAN processing to the
      the caller.
      Resulting change in behaviour: if SETVLAN fails while registering a
      new VLAN ID, the stack no longer creates the corresponding vlan device.
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Reviewed-by: NUrsula Braun <ubraun@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      2aa48671
    • J
      s390/qeth: don't take queue lock in send_packet_fast() · 7c2e9ba3
      Julian Wiedmann 提交于
      Locking the output queue prior to TX is needed on OSA devices,
      to synchronize against a packing flush from the TX completion code
      (via qeth_check_outbound_queue()).
      But send_packet_fast() is only used for IQDs, which don't do packing.
      So remove the locking, and apply some easy cleanups.
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Reviewed-by: NUrsula Braun <ubraun@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      7c2e9ba3
    • H
      s390/qeth: add VNICC get/set timeout support · 349d13d5
      Hans Wippel 提交于
      HiperSockets allow configuring so called VNIC Characteristics (VNICC)
      that influence how the underlying hardware handles packets. For VNICCs,
      additional commands for getting and setting timeouts are available.
      Currently, the learning VNICC uses these commands.
      
      * Learning VNICC: If learning is enabled on a qeth device, the device
        learns the source MAC addresses of outgoing packets and incoming
        packets to those learned MAC addresses are received.
      
      For learning, the timeout specifies the idle period in seconds, after
      which the underlying hardware removes a learned MAC address again.
      
      This patch adds support for the IPA commands that are required to get
      and set the current timeout values for the learning VNIC characteristic.
      Also, it introduces the sysfs interface that allows users to configure
      the timeout.
      Signed-off-by: NHans Wippel <hwippel@linux.vnet.ibm.com>
      Reviewed-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      349d13d5
    • H
      s390/qeth: add VNICC enable/disable support · caa1f0b1
      Hans Wippel 提交于
      HiperSocket devices allow enabling and disabling so called VNIC
      Characteristics (VNICC) that influence how the underlying hardware
      handles packets. These VNICCs are:
      
      * Flooding VNICC: Flooding allows specifying if packets to unknown
        destination MAC addresses are received by the qeth device.
      
      * Multicast flooding VNICC: Multicast flooding allows specifying if
        packets to multicast MAC addresses are received by the qeth device.
      
      * Learning VNICC: If learning is enabled on a qeth device, the device
        learns the source MAC addresses of outgoing packets and incoming
        packets to those learned MAC addresses are received.
      
      * Takeover setvmac VNICC: If takeover setvmac is configured on a qeth
        device, the MAC address of this device can be configured on a
        different qeth device with the setvmac IPA command.
      
      * Takeover by learning VNICC: If takeover learning is enabled on a qeth
        device, the MAC address of this device can be learned (learning VNICC)
        on a different qeth device.
      
      * BridgePort invisible VNICC: If BridgePort invisible is enabled on a
        qeth device, (1) packets from this device are not sent to a BridgePort
        enabled qeth device and (2) packets coming from a BridgePort enabled
        qeth device are not received by this device.
      
      * Receive broadcast VNICC: Receive broadcast allows configuring if a
        qeth device receives packets with the broadcast destination MAC
        address.
      
      This patch adds support for the IPA commands that are required to enable
      and disable these VNIC characteristics on qeth devices. As a
      prerequisite, it also adds the query commands IPA command.
      
      The query commands IPA command allows requesting the supported commands
      for each characteristic from the underlying hardware.
      
      Additionally, this patch provides users with a sysfs user interface to
      enable/disable the VNICCs mentioned above.
      Signed-off-by: NHans Wippel <hwippel@linux.vnet.ibm.com>
      Reviewed-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      caa1f0b1
    • H
      s390/qeth: add basic VNICC support · a45b3faf
      Hans Wippel 提交于
      VNIC Characteristics (VNICC) are features of HiperSockets that define
      how packets are handled by the underlying network hardware. For example,
      if the VNICC flooding is configured on a qeth device, ethernet frames to
      unknown destination MAC addresses are received.
      
      Currently, there is support for seven VNICCs: flooding, multicast
      flooding, receive broadcast, learning, takeover learning, takeover
      setvmac, bridge invisible. Also, six IPA commands exist for configuring
      VNICCs on a qeth device: query characteristics, query commands, enable
      characteristic, disable characteristic, set timeout, get timeout.
      
      This patch adds the basic code infrastructure for VNICC support to qeth.
      It allows querying VNICC support from the underlying hardware. To this
      end, it adds:
      
      * basic message formats for IPA commands
      * basic data structures
      * basic error handling
      * query characteristics IPA command support
      
      The query characteristics IPA command allows requesting the currently
      supported and currently enabled VNIC characteristics from the underlying
      hardware.
      
      Support for the other IPA commands and for the configuration of VNICCs
      is added in follow-up patches together with the respective user
      interface functions.
      Signed-off-by: NHans Wippel <hwippel@linux.vnet.ibm.com>
      Reviewed-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      a45b3faf
  7. 19 8月, 2017 6 次提交
  8. 16 8月, 2017 5 次提交
  9. 21 6月, 2017 2 次提交
  10. 16 6月, 2017 1 次提交
    • J
      networking: make skb_push & __skb_push return void pointers · d58ff351
      Johannes Berg 提交于
      It seems like a historic accident that these return unsigned char *,
      and in many places that means casts are required, more often than not.
      
      Make these functions return void * and remove all the casts across
      the tree, adding a (u8 *) cast only where the unsigned char pointer
      was used directly, all done with the following spatch:
      
          @@
          expression SKB, LEN;
          typedef u8;
          identifier fn = { skb_push, __skb_push, skb_push_rcsum };
          @@
          - *(fn(SKB, LEN))
          + *(u8 *)fn(SKB, LEN)
      
          @@
          expression E, SKB, LEN;
          identifier fn = { skb_push, __skb_push, skb_push_rcsum };
          type T;
          @@
          - E = ((T *)(fn(SKB, LEN)))
          + E = fn(SKB, LEN)
      
          @@
          expression SKB, LEN;
          identifier fn = { skb_push, __skb_push, skb_push_rcsum };
          @@
          - fn(SKB, LEN)[0]
          + *(u8 *)fn(SKB, LEN)
      
      Note that the last part there converts from push(...)[0] to the
      more idiomatic *(u8 *)push(...).
      Signed-off-by: NJohannes Berg <johannes.berg@intel.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      d58ff351
  11. 07 6月, 2017 1 次提交
  12. 12 5月, 2017 3 次提交
    • J
      s390/qeth: avoid null pointer dereference on OSN · 25e2c341
      Julian Wiedmann 提交于
      Access card->dev only after checking whether's its valid.
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Reviewed-by: NUrsula Braun <ubraun@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      25e2c341
    • J
      s390/qeth: unbreak OSM and OSN support · 2d2ebb3e
      Julian Wiedmann 提交于
      commit b4d72c08 ("qeth: bridgeport support - basic control")
      broke the support for OSM and OSN devices as follows:
      
      As OSM and OSN are L2 only, qeth_core_probe_device() does an early
      setup by loading the l2 discipline and calling qeth_l2_probe_device().
      In this context, adding the l2-specific bridgeport sysfs attributes
      via qeth_l2_create_device_attributes() hits a BUG_ON in fs/sysfs/group.c,
      since the basic sysfs infrastructure for the device hasn't been
      established yet.
      
      Note that OSN actually has its own unique sysfs attributes
      (qeth_osn_devtype), so the additional attributes shouldn't be created
      at all.
      For OSM, add a new qeth_l2_devtype that contains all the common
      and l2-specific sysfs attributes.
      When qeth_core_probe_device() does early setup for OSM or OSN, assign
      the corresponding devtype so that the ccwgroup probe code creates the
      full set of sysfs attributes.
      This allows us to skip qeth_l2_create_device_attributes() in case
      of an early setup.
      
      Any device that can't do early setup will initially have only the
      generic sysfs attributes, and when it's probed later
      qeth_l2_probe_device() adds the l2-specific attributes.
      
      If an early-setup device is removed (by calling ccwgroup_ungroup()),
      device_unregister() will - using the devtype - delete the
      l2-specific attributes before qeth_l2_remove_device() is called.
      So make sure to not remove them twice.
      
      What complicates the issue is that qeth_l2_probe_device() and
      qeth_l2_remove_device() is also called on a device when its
      layer2 attribute changes (ie. its layer mode is switched).
      For early-setup devices this wouldn't work properly - we wouldn't
      remove the l2-specific attributes when switching to L3.
      But switching the layer mode doesn't actually make any sense;
      we already decided that the device can only operate in L2!
      So just refuse to switch the layer mode on such devices. Note that
      OSN doesn't have a layer2 attribute, so we only need to special-case
      OSM.
      
      Based on an initial patch by Ursula Braun.
      
      Fixes: b4d72c08 ("qeth: bridgeport support - basic control")
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      2d2ebb3e
    • U
      s390/qeth: handle sysfs error during initialization · 9111e788
      Ursula Braun 提交于
      When setting up the device from within the layer discipline's
      probe routine, creating the layer-specific sysfs attributes can fail.
      Report this error back to the caller, and handle it by
      releasing the layer discipline.
      Signed-off-by: NUrsula Braun <ubraun@linux.vnet.ibm.com>
      [jwi: updated commit msg, moved an OSN change to a subsequent patch]
      Signed-off-by: NJulian Wiedmann <jwi@linux.vnet.ibm.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      9111e788
  13. 12 4月, 2017 7 次提交