1. 08 May, 2020 22 commits
    • Eric Dumazet's avatar
      netpoll: netpoll_send_skb() returns transmit status · 1ddabdfa
      Eric Dumazet authored
      Some callers want to know if the packet has been sent or
      dropped, to inform upper stacks.
      Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      1ddabdfa
    • Eric Dumazet's avatar
      netpoll: move netpoll_send_skb() out of line · fb1eee47
      Eric Dumazet authored
      There is no need to inline this helper, as we intend to add more
      code in this function.
      Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      fb1eee47
    • Eric Dumazet's avatar
      netpoll: remove dev argument from netpoll_send_skb_on_dev() · 307f660d
      Eric Dumazet authored
      netpoll_send_skb_on_dev() can get the device pointer directly from np->dev
      
      Rename it to __netpoll_send_skb()
      
      Following patch will move netpoll_send_skb() out-of-line.
      Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      307f660d
    • Colin Ian King's avatar
      net: phy: fix less than zero comparison with unsigned variable val · 3a13f98b
      Colin Ian King authored
      The unsigned variable val is being checked for an error by checking
      if it is less than zero. This can never occur because val is unsigned.
      Fix this by making val a plain int.
      
      Addresses-Coverity: ("Unsigned compared against zero")
      Fixes: bdbdac76 ("ethtool: provide UAPI for PHY master/slave configuration.")
      Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      3a13f98b
    • YueHaibing's avatar
      net/smc: remove set but not used variables 'del_llc, del_llc_resp' · ca7e3edc
      YueHaibing authored
      Fixes gcc '-Wunused-but-set-variable' warning:
      
      net/smc/smc_llc.c: In function 'smc_llc_cli_conf_link':
      net/smc/smc_llc.c:753:31: warning:
       variable 'del_llc' set but not used [-Wunused-but-set-variable]
        struct smc_llc_msg_del_link *del_llc;
                                     ^
      net/smc/smc_llc.c: In function 'smc_llc_process_srv_delete_link':
      net/smc/smc_llc.c:1311:33: warning:
       variable 'del_llc_resp' set but not used [-Wunused-but-set-variable]
          struct smc_llc_msg_del_link *del_llc_resp;
                                       ^
      Signed-off-by: default avatarYueHaibing <yuehaibing@huawei.com>
      Signed-off-by: default avatarKarsten Graul <kgraul@linux.ibm.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      ca7e3edc
    • zhang kai's avatar
      tcp: tcp_mark_head_lost is only valid for sack-tcp · 636ef28d
      zhang kai authored
      so tcp_is_sack/reno checks are removed from tcp_mark_head_lost.
      Signed-off-by: default avatarzhang kai <zhangkaiheb@126.com>
      Acked-by: default avatarNeal Cardwell <ncardwell@google.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      636ef28d
    • Jacob Keller's avatar
      net: remove newlines in NL_SET_ERR_MSG_MOD · c75a33c8
      Jacob Keller authored
      The NL_SET_ERR_MSG_MOD macro is used to report a string describing an
      error message to userspace via the netlink extended ACK structure. It
      should not have a trailing newline.
      
      Add a cocci script which catches cases where the newline marker is
      present. Using this script, fix the handful of cases which accidentally
      included a trailing new line.
      
      I couldn't figure out a way to get a patch mode working, so this script
      only implements context, report, and org.
      Signed-off-by: default avatarJacob Keller <jacob.e.keller@intel.com>
      Cc: Jakub Kicinski <kuba@kernel.org>
      Cc: Andy Whitcroft <apw@canonical.com>
      Cc: Joe Perches <joe@perches.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      c75a33c8
    • David S. Miller's avatar
      Merge branch 'ti-am65x-cpts-follow-up-dt-bindings-update' · 57ea8506
      David S. Miller authored
      Grygorii Strashko says:
      
      ====================
      net: ethernet: ti: am65x-cpts: follow up dt bindings update
      
      This series is follow update for  TI A65x/J721E Common platform time sync (CPTS)
      driver [1] to implement  DT bindings review comments from
      Rob Herring <robh@kernel.org> [2].
       - "reg" and "compatible" properties are made required for CPTS DT nodes which
         also required to change K3 CPSW driver to use of_platform_device_create()
         instead of of_platform_populate() for proper CPTS and MDIO initialization
       - minor DT bindings format changes
       - K3 CPTS example added to K3 MCU CPSW bindings
      
      [1] https://lwn.net/Articles/819313/
      [2] https://lwn.net/ml/linux-kernel/20200505040419.GA8509@bogus/
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      57ea8506
    • Grygorii Strashko's avatar
      arm64: dts: ti: k3-am65/j721e-mcu: update cpts node · ef2d1363
      Grygorii Strashko authored
      Update CPTS node following DT binding update:
       - add reg and compatible properties
       - fix node name
      Signed-off-by: default avatarGrygorii Strashko <grygorii.strashko@ti.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      ef2d1363
    • Grygorii Strashko's avatar
      dt-binding: net: ti: am65x-cpts: make reg and compatible required · 4786f4a0
      Grygorii Strashko authored
      This patch follows K3 CPTS review comments from Rob Herring
      <robh@kernel.org>.
       - "reg" and "compatible" properties are required now
       - minor format changes
       - K3 CPTS example added to K3 MCU CPSW bindings
      
      Cc: Rob Herring <robh@kernel.org>
      Signed-off-by: default avatarGrygorii Strashko <grygorii.strashko@ti.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      4786f4a0
    • Grygorii Strashko's avatar
      net: ethernet: ti: am65-cpsw-nuss: use of_platform_device_create() for mdio · a45cfcc6
      Grygorii Strashko authored
      The MCU CPSW expected to populate only MDIO device, but follow up patches
      will add "compatible" property to the MCU CPSW CPTS node which will cause
      creation of CPTS device and MCU CPSW init failure. Hence, switch to use
      of_platform_device_create() instead of of_platform_populate() for MDIO
      device population.
      Signed-off-by: default avatarGrygorii Strashko <grygorii.strashko@ti.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      a45cfcc6
    • David S. Miller's avatar
      Merge branch 'hsr-hsr-code-refactoring' · a8c9baf2
      David S. Miller authored
      Taehee Yoo says:
      
      ====================
      hsr: hsr code refactoring
      
      There are some unnecessary routine in the hsr module.
      This patch removes these routines.
      
      The first patch removes incorrect comment.
      The second patch removes unnecessary WARN_ONCE() macro.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      a8c9baf2
    • Ioana Ciornei's avatar
      dpaa2-eth: create a function to flush the XDP fds · 38c440b2
      Ioana Ciornei authored
      Create an independent function that takes a particular frame queue and
      an array of frame descriptors and tries to enqueue them until it hits
      the maximum number fo retries. The same function will be used in the
      next patch also on the XDP_TX path.
      
      Also, create the dpaa2_eth_xdp_fds structure to incorporate the array of
      FDs as well as the number of FDs already populated.
      Signed-off-by: default avatarIoana Ciornei <ioana.ciornei@nxp.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      38c440b2
    • Taehee Yoo's avatar
      hsr: remove WARN_ONCE() in hsr_fill_frame_info() · f96e8717
      Taehee Yoo authored
      When VLAN frame is being sent, hsr calls WARN_ONCE() because hsr doesn't
      support VLAN. But using WARN_ONCE() is overdoing.
      Using netdev_warn_once() is enough.
      Signed-off-by: default avatarTaehee Yoo <ap420073@gmail.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      f96e8717
    • Ioana Ciornei's avatar
      soc: fsl: dpio: properly compute the consumer index · 7596ac9d
      Ioana Ciornei authored
      Mask the consumer index before using it. Without this, we would be
      writing frame descriptors beyond the ring size supported by the QBMAN
      block.
      
      Fixes: 3b2abda7 ("soc: fsl: dpio: Replace QMAN array mode with ring mode enqueue")
      Signed-off-by: default avatarIoana Ciornei <ioana.ciornei@nxp.com>
      Acked-by: default avatarLi Yang <leoyang.li@nxp.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      7596ac9d
    • David S. Miller's avatar
      Merge branch 'tc-gate-offload-for-SJA1105-DSA-switch' · eb55d7b6
      David S. Miller authored
      Vladimir Oltean says:
      
      ====================
      tc-gate offload for SJA1105 DSA switch
      
      Expose the TTEthernet hardware features of the switch using standard
      tc-flower actions: trap, drop, redirect and gate.
      
      v1 was submitted at:
      https://patchwork.ozlabs.org/project/netdev/cover/20200503211035.19363-1-olteanv@gmail.com/
      
      v2 was submitted at:
      https://patchwork.ozlabs.org/project/netdev/cover/20200503211035.19363-1-olteanv@gmail.com/
      
      Changes in v3:
      Made sure there are no compilation warnings when
      CONFIG_NET_DSA_SJA1105_TAS or CONFIG_NET_DSA_SJA1105_VL are disabled.
      
      Changes in v2:
      Using a newly introduced dsa_port_from_netdev public helper.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      eb55d7b6
    • Vladimir Oltean's avatar
      docs: net: dsa: sja1105: document intended usage of virtual links · 47cfa3af
      Vladimir Oltean authored
      Add some verbiage describing how the hardware features of the switch are
      exposed to users through tc-flower.
      Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      47cfa3af
    • Vladimir Oltean's avatar
      net: dsa: sja1105: implement tc-gate using time-triggered virtual links · 834f8933
      Vladimir Oltean authored
      Restrict the TTEthernet hardware support on this switch to operate as
      closely as possible to IEEE 802.1Qci as possible. This means that it can
      perform PTP-time-based ingress admission control on streams identified
      by {DMAC, VID, PCP}, which is useful when trying to ensure the
      determinism of traffic scheduled via IEEE 802.1Qbv.
      
      The oddity comes from the fact that in hardware (and in TTEthernet at
      large), virtual links always need a full-blown action, including not
      only the type of policing, but also the list of destination ports. So in
      practice, a single tc-gate action will result in all packets getting
      dropped. Additional actions (either "trap" or "redirect") need to be
      specified in the same filter rule such that the conforming packets are
      actually forwarded somewhere.
      
      Apart from the VL Lookup, Policing and Forwarding tables which need to
      be programmed for each flow (virtual link), the Schedule engine also
      needs to be told to open/close the admission gates for each individual
      virtual link. A fairly accurate (and detailed) description of how that
      works is already present in sja1105_tas.c, since it is already used to
      trigger the egress gates for the tc-taprio offload (IEEE 802.1Qbv). Key
      point here, we remember that the schedule engine supports 8
      "subschedules" (execution threads that iterate through the global
      schedule in parallel, and that no 2 hardware threads must execute a
      schedule entry at the same time). For tc-taprio, each egress port used
      one of these 8 subschedules, leaving a total of 4 subschedules unused.
      In principle we could have allocated 1 subschedule for the tc-gate
      offload of each ingress port, but actually the schedules of all virtual
      links installed on each ingress port would have needed to be merged
      together, before they could have been programmed to hardware. So
      simplify our life and just merge the entire tc-gate configuration, for
      all virtual links on all ingress ports, into a single subschedule. Be
      sure to check that against the usual hardware scheduling conflicts, and
      program it to hardware alongside any tc-taprio subschedule that may be
      present.
      
      The following scenarios were tested:
      
      1. Quantitative testing:
      
         tc qdisc add dev swp2 clsact
         tc filter add dev swp2 ingress flower skip_sw \
                 dst_mac 42:be:24:9b:76:20 \
                 action gate index 1 base-time 0 \
                 sched-entry OPEN 1200 -1 -1 \
                 sched-entry CLOSE 1200 -1 -1 \
                 action trap
      
         ping 192.168.1.2 -f
         PING 192.168.1.2 (192.168.1.2) 56(84) bytes of data.
         .............................
         --- 192.168.1.2 ping statistics ---
         948 packets transmitted, 467 received, 50.7384% packet loss, time 9671ms
      
      2. Qualitative testing (with a phase-aligned schedule - the clocks are
         synchronized by ptp4l, not shown here):
      
         Receiver (sja1105):
      
         tc qdisc add dev swp2 clsact
         now=$(phc_ctl /dev/ptp1 get | awk '/clock time is/ {print $5}') && \
                 sec=$(echo $now | awk -F. '{print $1}') && \
                 base_time="$(((sec + 2) * 1000000000))" && \
                 echo "base time ${base_time}"
         tc filter add dev swp2 ingress flower skip_sw \
                 dst_mac 42:be:24:9b:76:20 \
                 action gate base-time ${base_time} \
                 sched-entry OPEN  60000 -1 -1 \
                 sched-entry CLOSE 40000 -1 -1 \
                 action trap
      
         Sender (enetc):
         now=$(phc_ctl /dev/ptp0 get | awk '/clock time is/ {print $5}') && \
                 sec=$(echo $now | awk -F. '{print $1}') && \
                 base_time="$(((sec + 2) * 1000000000))" && \
                 echo "base time ${base_time}"
         tc qdisc add dev eno0 parent root taprio \
                 num_tc 8 \
                 map 0 1 2 3 4 5 6 7 \
                 queues 1@0 1@1 1@2 1@3 1@4 1@5 1@6 1@7 \
                 base-time ${base_time} \
                 sched-entry S 01  50000 \
                 sched-entry S 00  50000 \
                 flags 2
      
         ping -A 192.168.1.1
         PING 192.168.1.1 (192.168.1.1): 56 data bytes
         ...
         ^C
         --- 192.168.1.1 ping statistics ---
         1425 packets transmitted, 1424 packets received, 0% packet loss
         round-trip min/avg/max = 0.322/0.361/0.990 ms
      
         And just for comparison, with the tc-taprio schedule deleted:
      
         ping -A 192.168.1.1
         PING 192.168.1.1 (192.168.1.1): 56 data bytes
         ...
         ^C
         --- 192.168.1.1 ping statistics ---
         33 packets transmitted, 19 packets received, 42% packet loss
         round-trip min/avg/max = 0.336/0.464/0.597 ms
      Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      834f8933
    • Vladimir Oltean's avatar
      net: dsa: sja1105: support flow-based redirection via virtual links · dfacc5a2
      Vladimir Oltean authored
      Implement tc-flower offloads for redirect, trap and drop using
      non-critical virtual links.
      
      Commands which were tested to work are:
      
        # Send frames received on swp2 with a DA of 42:be:24:9b:76:20 to the
        # CPU and to swp3. This type of key (DA only) when the port's VLAN
        # awareness state is off.
        tc qdisc add dev swp2 clsact
        tc filter add dev swp2 ingress flower skip_sw dst_mac 42:be:24:9b:76:20 \
                action mirred egress redirect dev swp3 \
                action trap
      
        # Drop frames received on swp2 with a DA of 42:be:24:9b:76:20, a VID
        # of 100 and a PCP of 0.
        tc filter add dev swp2 ingress protocol 802.1Q flower skip_sw \
                dst_mac 42:be:24:9b:76:20 vlan_id 100 vlan_prio 0 action drop
      
      Under the hood, all rules match on DMAC, VID and PCP, but when VLAN
      filtering is disabled, those are set internally by the driver to the
      port-based defaults. Because we would be put in an awkward situation if
      the user were to change the VLAN filtering state while there are active
      rules (packets would no longer match on the specified keys), we simply
      deny changing vlan_filtering unless the list of flows offloaded via
      virtual links is empty. Then the user can re-add new rules.
      Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      dfacc5a2
    • Vladimir Oltean's avatar
      net: dsa: sja1105: make room for virtual link parsing in flower offload · b70bb8d4
      Vladimir Oltean authored
      Virtual links are a sja1105 hardware concept of executing various flow
      actions based on a key extracted from the frame's DMAC, VID and PCP.
      
      Currently the tc-flower offload code supports only parsing the DMAC if
      that is the broadcast MAC address, and the VLAN PCP. Extract the key
      parsing logic from the L2 policers functionality and move it into its
      own function, after adding extra logic for matching on any DMAC and VID.
      Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      b70bb8d4
    • Vladimir Oltean's avatar
      net: dsa: sja1105: add static tables for virtual links · 94f94d4a
      Vladimir Oltean authored
      This patch adds the register definitions for the:
      - VL Lookup Table
      - VL Policing Table
      - VL Forwarding Table
      - VL Forwarding Parameters Table
      
      These are needed in order to perform TTEthernet operations: QoS
      classification, flow-based policing and/or frame redirecting with the
      switch.
      Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      94f94d4a
    • Vladimir Oltean's avatar
      net: dsa: introduce a dsa_port_from_netdev public helper · e1eea811
      Vladimir Oltean authored
      As its implementation shows, this is synonimous with calling
      dsa_slave_dev_check followed by dsa_slave_to_port, so it is quite simple
      already and provides functionality which is already there.
      
      However there is now a need for these functions outside dsa_priv.h, for
      example in drivers that perform mirroring and redirection through
      tc-flower offloads (they are given raw access to the flow_cls_offload
      structure), where they need to call this function on act->dev.
      
      But simply exporting dsa_slave_to_port would make it non-inline and
      would result in an extra function call in the hotpath, as can be seen
      for example in sja1105:
      
      Before:
      
      000006dc <sja1105_xmit>:
      {
       6dc:	e92d4ff0 	push	{r4, r5, r6, r7, r8, r9, sl, fp, lr}
       6e0:	e1a04000 	mov	r4, r0
       6e4:	e591958c 	ldr	r9, [r1, #1420]	; 0x58c <- Inline dsa_slave_to_port
       6e8:	e1a05001 	mov	r5, r1
       6ec:	e24dd004 	sub	sp, sp, #4
      	u16 tx_vid = dsa_8021q_tx_vid(dp->ds, dp->index);
       6f0:	e1c901d8 	ldrd	r0, [r9, #24]
       6f4:	ebfffffe 	bl	0 <dsa_8021q_tx_vid>
      			6f4: R_ARM_CALL	dsa_8021q_tx_vid
      	u8 pcp = netdev_txq_to_tc(netdev, queue_mapping);
       6f8:	e1d416b0 	ldrh	r1, [r4, #96]	; 0x60
      	u16 tx_vid = dsa_8021q_tx_vid(dp->ds, dp->index);
       6fc:	e1a08000 	mov	r8, r0
      
      After:
      
      000006e4 <sja1105_xmit>:
      {
       6e4:	e92d4ff0 	push	{r4, r5, r6, r7, r8, r9, sl, fp, lr}
       6e8:	e1a04000 	mov	r4, r0
       6ec:	e24dd004 	sub	sp, sp, #4
      	struct dsa_port *dp = dsa_slave_to_port(netdev);
       6f0:	e1a00001 	mov	r0, r1
      {
       6f4:	e1a05001 	mov	r5, r1
      	struct dsa_port *dp = dsa_slave_to_port(netdev);
       6f8:	ebfffffe 	bl	0 <dsa_slave_to_port>
      			6f8: R_ARM_CALL	dsa_slave_to_port
       6fc:	e1a09000 	mov	r9, r0
      	u16 tx_vid = dsa_8021q_tx_vid(dp->ds, dp->index);
       700:	e1c001d8 	ldrd	r0, [r0, #24]
       704:	ebfffffe 	bl	0 <dsa_8021q_tx_vid>
      			704: R_ARM_CALL	dsa_8021q_tx_vid
      
      Because we want to avoid possible performance regressions, introduce
      this new function which is designed to be public.
      Suggested-by: default avatarVivien Didelot <vivien.didelot@gmail.com>
      Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
      Reviewed-by: default avatarVivien Didelot <vivien.didelot@gmail.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      e1eea811
  2. 07 May, 2020 18 commits
    • Chen Zhou's avatar
      cxgb4: remove duplicate headers · bb206a08
      Chen Zhou authored
      Remove duplicate headers which are included twice.
      Signed-off-by: default avatarChen Zhou <chenzhou10@huawei.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      bb206a08
    • David S. Miller's avatar
      Merge tag 'wireless-drivers-next-2020-05-07' of... · 5d9e4722
      David S. Miller authored
      Merge tag 'wireless-drivers-next-2020-05-07' of git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next
      
      Kalle Valo says:
      
      ====================
      wireless-drivers-next patches for v5.8
      
      First set of patches for v5.8. Changes all over, ath10k apparently
      seeing most new features this time. rtw88 also had lots of changes due
      to preparation for new hardware support.
      
      In this pull request there's also a new macro to include/linux/iopoll:
      read_poll_timeout_atomic(). This is needed by rtw88 for atomic
      polling.
      
      Major changes:
      
      ath11k
      
      * add debugfs file for testing ADDBA and DELBA
      
      * add 802.11 encapsulation offload on hardware support
      
      * add htt_peer_stats_reset debugfs file
      
      ath10k
      
      * enable VHT160 and VHT80+80 modes
      
      * enable radar detection in secondary segment
      
      * sdio: disable TX complete indication to improve throughput
      
      * sdio: decrease power consumption
      
      * sdio: add HTT TX bundle support to increase throughput
      
      * sdio: add rx bitrate reporting
      
      ath9k
      
      * improvements to AR9002 calibration logic
      
      carl9170
      
      * remove buggy P2P_GO support
      
      p54usb
      
      * add support for AirVasT USB stick
      
      rtw88
      
      * add support for antenna configuration
      
      ti wlcore
      
      * add support for AES_CMAC cipher
      
      iwlwifi
      
      * support for a few new FW API versions
      
      * new hw configs
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      5d9e4722
    • David S. Miller's avatar
      Merge branch 'Add-QRTR-MHI-client-driver' · 3031a86e
      David S. Miller authored
      Manivannan Sadhasivam says:
      
      ====================
      Add QRTR MHI client driver
      
      Here is the series adding MHI client driver support to Qualcomm IPC router
      protocol. MHI is a newly added bus to kernel which is used to communicate to
      external modems over a physical interface like PCI-E. This driver is used to
      transfer the QMI messages between the host processor and external modems over
      the "IPCR" channel.
      
      For QRTR, this driver is just another driver acting as a transport layer like
      SMD.
      
      Currently this driver is needed to control the QCA6390 WLAN device from ath11k.
      The ath11k MHI controller driver will take care of booting up QCA6390 and
      bringing it to operating state. Later, this driver will be used to transfer QMI
      messages over the MHI-IPCR channel.
      
      The second patch of this series removes the ARCH_QCOM dependency for QRTR. This
      is needed because the QRTR driver will be used with x86 machines as well to talk
      to devices like QCA6390.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      3031a86e
    • Manivannan Sadhasivam's avatar
      net: qrtr: Do not depend on ARCH_QCOM · e4267108
      Manivannan Sadhasivam authored
      IPC Router protocol is also used by external modems for exchanging the QMI
      messages. Hence, it doesn't always depend on Qualcomm platforms. One such
      instance is the QCA6390 WLAN device connected to x86 machine.
      Reviewed-by: default avatarBjorn Andersson <bjorn.andersson@linaro.org>
      Signed-off-by: default avatarManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      e4267108
    • Manivannan Sadhasivam's avatar
      net: qrtr: Add MHI transport layer · 6e728f32
      Manivannan Sadhasivam authored
      MHI is the transport layer used for communicating to the external modems.
      Hence, this commit adds MHI transport layer support to QRTR for
      transferring the QMI messages over IPC Router.
      Reviewed-by: default avatarBjorn Andersson <bjorn.andersson@linaro.org>
      Signed-off-by: default avatarManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      6e728f32
    • Geert Uytterhoeven's avatar
      via-rhine: Add platform dependencies · 0932969e
      Geert Uytterhoeven authored
      The VIA Rhine Ethernet interface is only present on PCI devices or
      VIA/WonderMedia VT8500/WM85xx SoCs.  Add platform dependencies to the
      VIA_RHINE config symbol, to avoid asking the user about it when
      configuring a kernel without PCI or VT8500/WM85xx support.
      Signed-off-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
      Acked-by: default avatarArnd Bergmann <arnd@arndb.de>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      0932969e
    • Po Liu's avatar
      net:enetc: bug fix for qos sfi operate space after freed · 571cf296
      Po Liu authored
      'Dan Carpenter' reported:
      This code frees "sfi" and then dereferences it on the next line:
      >                 kfree(sfi);
      >                 clear_bit(sfi->index, epsfp.psfp_sfi_bitmap);
      
      This "sfi->index" should be "index".
      Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
      Signed-off-by: default avatarPo Liu <Po.Liu@nxp.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      571cf296
    • Jason Yan's avatar
      net: encx24j600: make encx24j600_hw_init() return void · 7f960633
      Jason Yan authored
      This function always return 0 now, we can make it return void to
      simplify the code. This fixes the following coccicheck warning:
      
      drivers/net/ethernet/microchip/encx24j600.c:609:5-8: Unneeded variable:
      "ret". Return "0" on line 653
      Signed-off-by: default avatarJason Yan <yanaijie@huawei.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      7f960633
    • Jason Yan's avatar
      net: mlx4: remove unneeded variable "err" in mlx4_en_ethtool_add_mac_rule() · 5a7c4509
      Jason Yan authored
      Fix the following coccicheck warning:
      
      drivers/net/ethernet/mellanox/mlx4/en_ethtool.c:1396:5-8: Unneeded
      variable: "err". Return "0" on line 1411
      Signed-off-by: default avatarJason Yan <yanaijie@huawei.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      5a7c4509
    • Jason Yan's avatar
      net: tulip: de4x5: make PCI_signature() return void · 27ad2cd2
      Jason Yan authored
      This function always return 0 now, we can make it return void to
      simplify the code. This fixes the following coccicheck warning:
      
      drivers/net/ethernet/dec/tulip/de4x5.c:3908:11-17: Unneeded variable:
      "status". Return "0" on line 3912
      Signed-off-by: default avatarJason Yan <yanaijie@huawei.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      27ad2cd2
    • Jason Yan's avatar
      net: atheros: remove dead code in atl1c_resume() · f9a98f90
      Jason Yan authored
      This code has been marked dead for nearly 10 years. Remove it.
      Signed-off-by: default avatarJason Yan <yanaijie@huawei.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      f9a98f90
    • Jason Yan's avatar
      b43: remove dead function b43_rssinoise_postprocess() · 49d4c392
      Jason Yan authored
      This function is dead for more than 10 years. Remove it.
      Signed-off-by: default avatarJason Yan <yanaijie@huawei.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      49d4c392
    • Zheng Zengkai's avatar
      net: phy: Make iproc_mdio_resume static · 31e2d067
      Zheng Zengkai authored
      Fix sparse warnings:
      
      drivers/net/phy/mdio-bcm-iproc.c:182:5: warning:
       symbol 'iproc_mdio_resume' was not declared. Should it be static?
      Reported-by: default avatarHulk Robot <hulkci@huawei.com>
      Signed-off-by: default avatarZheng Zengkai <zhengzengkai@huawei.com>
      Acked-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      31e2d067
    • David S. Miller's avatar
      Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net · 3793faad
      David S. Miller authored
      Conflicts were all overlapping changes.
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      3793faad
    • Linus Torvalds's avatar
      Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net · a811c1fa
      Linus Torvalds authored
      Pull networking fixes from David Miller:
      
       1) Fix reference count leaks in various parts of batman-adv, from Xiyu
          Yang.
      
       2) Update NAT checksum even when it is zero, from Guillaume Nault.
      
       3) sk_psock reference count leak in tls code, also from Xiyu Yang.
      
       4) Sanity check TCA_FQ_CODEL_DROP_BATCH_SIZE netlink attribute in
          fq_codel, from Eric Dumazet.
      
       5) Fix panic in choke_reset(), also from Eric Dumazet.
      
       6) Fix VLAN accel handling in bnxt_fix_features(), from Michael Chan.
      
       7) Disallow out of range quantum values in sch_sfq, from Eric Dumazet.
      
       8) Fix crash in x25_disconnect(), from Yue Haibing.
      
       9) Don't pass pointer to local variable back to the caller in
          nf_osf_hdr_ctx_init(), from Arnd Bergmann.
      
      10) Wireguard should use the ECN decap helper functions, from Toke
          Høiland-Jørgensen.
      
      11) Fix command entry leak in mlx5 driver, from Moshe Shemesh.
      
      12) Fix uninitialized variable access in mptcp's
          subflow_syn_recv_sock(), from Paolo Abeni.
      
      13) Fix unnecessary out-of-order ingress frame ordering in macsec, from
          Scott Dial.
      
      14) IPv6 needs to use a global serial number for dst validation just
          like ipv4, from David Ahern.
      
      15) Fix up PTP_1588_CLOCK deps, from Clay McClure.
      
      16) Missing NLM_F_MULTI flag in gtp driver netlink messages, from
          Yoshiyuki Kurauchi.
      
      17) Fix a regression in that dsa user port errors should not be fatal,
          from Florian Fainelli.
      
      18) Fix iomap leak in enetc driver, from Dejin Zheng.
      
      19) Fix use after free in lec_arp_clear_vccs(), from Cong Wang.
      
      20) Initialize protocol value earlier in neigh code paths when
          generating events, from Roman Mashak.
      
      21) netdev_update_features() must be called with RTNL mutex in macsec
          driver, from Antoine Tenart.
      
      22) Validate untrusted GSO packets even more strictly, from Willem de
          Bruijn.
      
      23) Wireguard decrypt worker needs a cond_resched(), from Jason
          Donenfeld.
      
      * git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net: (111 commits)
        net: flow_offload: skip hw stats check for FLOW_ACTION_HW_STATS_DONT_CARE
        MAINTAINERS: put DYNAMIC INTERRUPT MODERATION in proper order
        wireguard: send/receive: use explicit unlikely branch instead of implicit coalescing
        wireguard: selftests: initalize ipv6 members to NULL to squelch clang warning
        wireguard: send/receive: cond_resched() when processing worker ringbuffers
        wireguard: socket: remove errant restriction on looping to self
        wireguard: selftests: use normal kernel stack size on ppc64
        net: ethernet: ti: am65-cpsw-nuss: fix irqs type
        ionic: Use debugfs_create_bool() to export bool
        net: dsa: Do not leave DSA master with NULL netdev_ops
        net: dsa: remove duplicate assignment in dsa_slave_add_cls_matchall_mirred
        net: stricter validation of untrusted gso packets
        seg6: fix SRH processing to comply with RFC8754
        net: mscc: ocelot: ANA_AUTOAGE_AGE_PERIOD holds a value in seconds, not ms
        net: dsa: ocelot: the MAC table on Felix is twice as large
        net: dsa: sja1105: the PTP_CLK extts input reacts on both edges
        selftests: net: tcp_mmap: fix SO_RCVLOWAT setting
        net: hsr: fix incorrect type usage for protocol variable
        net: macsec: fix rtnl locking issue
        net: mvpp2: cls: Prevent buffer overflow in mvpp2_ethtool_cls_rule_del()
        ...
      a811c1fa
    • Pablo Neira Ayuso's avatar
      net: flow_offload: skip hw stats check for FLOW_ACTION_HW_STATS_DONT_CARE · 16f80360
      Pablo Neira Ayuso authored
      This patch adds FLOW_ACTION_HW_STATS_DONT_CARE which tells the driver
      that the frontend does not need counters, this hw stats type request
      never fails. The FLOW_ACTION_HW_STATS_DISABLED type explicitly requests
      the driver to disable the stats, however, if the driver cannot disable
      counters, it bails out.
      
      TCA_ACT_HW_STATS_* maintains the 1:1 mapping with FLOW_ACTION_HW_STATS_*
      except by disabled which is mapped to FLOW_ACTION_HW_STATS_DISABLED
      (this is 0 in tc). Add tc_act_hw_stats() to perform the mapping between
      TCA_ACT_HW_STATS_* and FLOW_ACTION_HW_STATS_*.
      
      Fixes: 319a1d19 ("flow_offload: check for basic action hw stats type")
      Signed-off-by: default avatarPablo Neira Ayuso <pablo@netfilter.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      16f80360
    • Lukas Bulwahn's avatar
      MAINTAINERS: put DYNAMIC INTERRUPT MODERATION in proper order · b0956956
      Lukas Bulwahn authored
      Commit 9b038086 ("docs: networking: convert DIM to RST") added a new
      file entry to DYNAMIC INTERRUPT MODERATION to the end, and not following
      alphabetical order.
      
      So, ./scripts/checkpatch.pl -f MAINTAINERS complains:
      
        WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic
        order
        #5966: FILE: MAINTAINERS:5966:
        +F:      lib/dim/
        +F:      Documentation/networking/net_dim.rst
      
      Reorder the file entries to keep MAINTAINERS nicely ordered.
      Signed-off-by: default avatarLukas Bulwahn <lukas.bulwahn@gmail.com>
      Acked-by: default avatarJakub Kicinski <kuba@kernel.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      b0956956
    • David S. Miller's avatar
      Merge branch 'wireguard-fixes' · d3f3e6ac
      David S. Miller authored
      Jason A. Donenfeld says:
      
      ====================
      wireguard fixes for 5.7-rc5
      
      With Ubuntu and Debian having backported this into their kernels, we're
      finally seeing testing from places we hadn't seen prior, which is nice.
      With that comes more fixes:
      
      1) The CI for PPC64 was running with extremely small stacks for 64-bit,
         causing spurious crashes in surprising places.
      
      2) There's was an old leftover routing loop restriction, which no longer
         makes sense given the queueing architecture, and was causing problems
         for people who really did want nested routing.
      
      3) Not yielding our kthread on CONFIG_PREEMPT_VOLUNTARY systems caused
         RCU stalls and other issues, reported by Wang Jian, with the fix
         suggested by Sultan Alsawaf.
      
      4) Clang spewed warnings in a selftest for CONFIG_IPV6=n, reported by
         Arnd Bergmann.
      
      5) A complicated if statement was simplified to an assignment while also
         making the likely/unlikely hinting more correct and simple, and
         increasing readability, suggested by Sultan.
      
      Patches (2) and (3) have Fixes: lines and are probably good candidates
      for stable.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      d3f3e6ac