1. 06 Sep, 2016 27 commits
    • Arnd Bergmann's avatar
      ptp: ixp46x: remove NO_IRQ handling · cf86799e
      Arnd Bergmann authored
      gpio_to_irq does not return NO_IRQ but instead returns a negative
      error code on failure. Returning NO_IRQ from the function has no
      negative effects as we only compare the result to the expected
      interrupt number, but it's better to return a proper failure
      code for consistency, and we should remove NO_IRQ from the kernel
      entirely.
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Acked-by: default avatarRichard Cochran <richardcochran@gmail.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      cf86799e
    • Bert Kenward's avatar
      72a31d85
    • Arnd Bergmann's avatar
      perf, bpf: fix conditional call to bpf_overflow_handler · f1e4ba5b
      Arnd Bergmann authored
      The newly added bpf_overflow_handler function is only built of both
      CONFIG_EVENT_TRACING and CONFIG_BPF_SYSCALL are enabled, but the caller
      only checks the latter:
      
      kernel/events/core.c: In function 'perf_event_alloc':
      kernel/events/core.c:9106:27: error: 'bpf_overflow_handler' undeclared (first use in this function)
      
      This changes the caller so we also skip this call if CONFIG_EVENT_TRACING
      is disabled entirely.
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Fixes: aa6a5f3c ("perf, bpf: add perf events core support for BPF_PROG_TYPE_PERF_EVENT programs")
      Acked-by: default avatarAlexei Starovoitov <ast@kernel.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      f1e4ba5b
    • Baoyou Xie's avatar
      net: arc_emac: mark arc_mdio_reset() static · 3f591997
      Baoyou Xie authored
      We get 1 warning when building kernel with W=1:
      drivers/net/ethernet/arc/emac_mdio.c:107:5: warning: no previous prototype for 'arc_mdio_reset' [-Wmissing-prototypes]
      
      In fact, this function is only used in the file in which it is
      declared and don't need a declaration, but can be made static.
      so this patch marks this function with 'static'.
      Signed-off-by: default avatarBaoyou Xie <baoyou.xie@linaro.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      3f591997
    • Baoyou Xie's avatar
      lan78xx: mark symbols static where possible · e0c79ff6
      Baoyou Xie authored
      We get a few warnings when building kernel with W=1:
      drivers/net/usb/lan78xx.c:1182:6: warning: no previous prototype for 'lan78xx_defer_kevent' [-Wmissing-prototypes]
      drivers/net/usb/lan78xx.c:1409:5: warning: no previous prototype for 'lan78xx_nway_reset' [-Wmissing-prototypes]
      drivers/net/usb/lan78xx.c:2000:5: warning: no previous prototype for 'lan78xx_set_mac_addr' [-Wmissing-prototypes]
      ....
      
      In fact, these functions are only used in the file in which they are
      declared and don't need a declaration, but can be made static.
      so this patch marks these functions with 'static'.
      Signed-off-by: default avatarBaoyou Xie <baoyou.xie@linaro.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      e0c79ff6
    • David S. Miller's avatar
      Merge branch 'qed-get_regs' · c63ad410
      David S. Miller authored
      c63ad410
    • Tomer Tayar's avatar
      qed: Add infrastructure for debug data collection · 4102426f
      Tomer Tayar authored
      Adds support for several infrastructure operations that are done as part of
      debug data collection.
      Signed-off-by: default avatarTomer Tayar <Tomer.Tayar@qlogic.com>
      Signed-off-by: default avatarYuval Mintz <Yuval.Mintz@qlogic.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      4102426f
    • Alexander Duyck's avatar
    • Joe Perches's avatar
      qed: Remove OOM messages · 2591c280
      Joe Perches authored
      These messages are unnecessary as OOM allocation failures already do
      a dump_stack() giving more or less the same information.
      
      $ size drivers/net/ethernet/qlogic/qed/built-in.o* (defconfig x86-64)
         text	   data	    bss	    dec	    hex	filename
       127817	  27969	  32800	 188586	  2e0aa	drivers/net/ethernet/qlogic/qed/built-in.o.new
       132474	  27969	  32800	 193243	  2f2db	drivers/net/ethernet/qlogic/qed/built-in.o.old
      
      Miscellanea:
      
      o Change allocs to the generally preferred forms where possible.
      Signed-off-by: default avatarJoe Perches <joe@perches.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      2591c280
    • David S. Miller's avatar
      Merge tag 'rxrpc-rewrite-20160904-2' of... · c7ee5672
      David S. Miller authored
      Merge tag 'rxrpc-rewrite-20160904-2' of git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs
      
      David Howells says:
      
      ====================
      rxrpc: Split output code from sendmsg code
      
      Here's a set of small patches that split the packet transmission code from
      the sendmsg code and simply rearrange the new file to make it more
      logically laid out ready for being rewritten.  An enum is also moved out of
      the header file to there as it's only used there.  This needs to be applied
      on top of the just-posted fixes patch set.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      c7ee5672
    • David S. Miller's avatar
      Merge tag 'rxrpc-rewrite-20160904-1' of... · 0122c6d5
      David S. Miller authored
      Merge tag 'rxrpc-rewrite-20160904-1' of git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs
      
      David Howells says:
      
      ====================
      rxrpc: Small fixes
      
      Here's a set of small fix patches:
      
       (1) Fix some uninitialised variables.
      
       (2) Set the client call state before making it live by attaching it to the
           conn struct.
      
       (3) Randomise the epoch and starting client conn ID values, and don't
           change the epoch when the client conn ID rolls round.
      
       (4) Replace deprecated create_singlethread_workqueue() calls.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      0122c6d5
    • Haishuang Yan's avatar
      vxlan: Update tx_errors statistics if vxlan_build_skb return err. · 5e1e61a3
      Haishuang Yan authored
      If vxlan_build_skb return err < 0, tx_errors should be also increased.
      Signed-off-by: default avatarHaishuang Yan <yanhaishuang@cmss.chinamobile.com>
      Acked-by: default avatarJiri Benc <jbenc@redhat.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      5e1e61a3
    • Brenden Blanco's avatar
      net/mlx4_en: protect ring->xdp_prog with rcu_read_lock · 326fe02d
      Brenden Blanco authored
      Depending on the preempt mode, the bpf_prog stored in xdp_prog may be
      freed despite the use of call_rcu inside bpf_prog_put. The situation is
      possible when running in PREEMPT_RCU=y mode, for instance, since the rcu
      callback for destroying the bpf prog can run even during the bh handling
      in the mlx4 rx path.
      
      Several options were considered before this patch was settled on:
      
      Add a napi_synchronize loop in mlx4_xdp_set, which would occur after all
      of the rings are updated with the new program.
      This approach has the disadvantage that as the number of rings
      increases, the speed of update will slow down significantly due to
      napi_synchronize's msleep(1).
      
      Add a new rcu_head in bpf_prog_aux, to be used by a new bpf_prog_put_bh.
      The action of the bpf_prog_put_bh would be to then call bpf_prog_put
      later. Those drivers that consume a bpf prog in a bh context (like mlx4)
      would then use the bpf_prog_put_bh instead when the ring is up. This has
      the problem of complexity, in maintaining proper refcnts and rcu lists,
      and would likely be harder to review. In addition, this approach to
      freeing must be exclusive with other frees of the bpf prog, for instance
      a _bh prog must not be referenced from a prog array that is consumed by
      a non-_bh prog.
      
      The placement of rcu_read_lock in this patch is functionally the same as
      putting an rcu_read_lock in napi_poll. Actually doing so could be a
      potentially controversial change, but would bring the implementation in
      line with sk_busy_loop (though of course the nature of those two paths
      is substantially different), and would also avoid future copy/paste
      problems with future supporters of XDP. Still, this patch does not take
      that opinionated option.
      
      Testing was done with kernels in either PREEMPT_RCU=y or
      CONFIG_PREEMPT_VOLUNTARY=y+PREEMPT_RCU=n modes, with neither exhibiting
      any drawback. With PREEMPT_RCU=n, the extra call to rcu_read_lock did
      not show up in the perf report whatsoever, and with PREEMPT_RCU=y the
      overhead of rcu_read_lock (according to perf) was the same before/after.
      In the rx path, rcu_read_lock is eventually called for every packet
      from netif_receive_skb_internal, so the napi poll call's rcu_read_lock
      is easily amortized.
      
      v2:
      Remove extra rcu_read_lock in mlx4_en_process_rx_cq body
      Annotate xdp_prog with __rcu, and convert all usages to rcu_assign or
      rcu_dereference[_protected] as appropriate.
      Add explicit mutex lock around rcu_assign instead of xchg loop.
      
      Fixes: d576acf0 ("net/mlx4_en: add page recycle to prepare rx ring for tx support")
      Acked-by: default avatarDaniel Borkmann <daniel@iogearbox.net>
      Acked-by: default avatarAlexei Starovoitov <alexei.starovoitov@gmail.com>
      Signed-off-by: default avatarBrenden Blanco <bblanco@plumgrid.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      326fe02d
    • David S. Miller's avatar
      Merge branch 'mediatek-rx-path-enhancements' · bc58493b
      David S. Miller authored
      Sean Wang says:
      
      ====================
      net: ethernet: mediatek: add enhancements to RX path
      
      Changes since v1:
      - fix message typos and add coverletter
      
      Changes since v2:
      - split from the previous series for submitting add enhancements as
      a series targeting 'net-next' and add indents before comments.
      
      Changes since v3:
      - merge the patch using PDMA RX path
      - fixed the input of mtk_poll_rx is with the remaining budget
      
      Changes since v4:
      - save one wmb and register update when no packet is being handled
      inside mtk_poll_rx call
      - fixed incorrect return packet count from mtk_napi_rx
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      bc58493b
    • Sean Wang's avatar
      net: ethernet: mediatek: enhance RX path by aggregating more SKBs into NAPI · 41156cea
      Sean Wang authored
      The patch adds support for aggregating more SKBs feed into NAPI in
      order to get more benefits from generic receive offload (GRO) by
      peeking at the RX ring status and moving more packets right before
      returning from NAPI RX polling handler if NAPI budgets are still
      available and some packets already present in hardware.
      Signed-off-by: default avatarSean Wang <sean.wang@mediatek.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      41156cea
    • Sean Wang's avatar
      net: ethernet: mediatek: enhance RX path by reducing the frequency of the memory barrier used · 635372ad
      Sean Wang authored
      The patch makes move wmb() to outside the loop that could help
      RX path handling more faster although that RX descriptors aren't
      freed for DMA to use as soon as possible, but based on my experiment
      and the result shows it still can reach about 943Mbpis without
      performance drop that is tested based on the setup with one port
      using Giga PHY and 256 RX descriptors for DMA to move.
      Signed-off-by: default avatarSean Wang <sean.wang@mediatek.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      635372ad
    • David S. Miller's avatar
      Merge branch 'hso-neatening' · 0da4d283
      David S. Miller authored
      Joe Perches says:
      
      ====================
      hso: neatening
      
      This seems to be the only code in the kernel that uses
      macro defines with a trailing underscore.  Fix that.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      0da4d283
    • Joe Perches's avatar
      hso: Convert printk to pr_<level> · 3981cce6
      Joe Perches authored
      Use a more common logging style
      
      Miscellanea:
      
      o Add pr_fmt to prefix each output message
      o Realign arguments
      Signed-off-by: default avatarJoe Perches <joe@perches.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      3981cce6
    • Joe Perches's avatar
      hso: Use a more common logging style · 95a69117
      Joe Perches authored
      Macros that end in an underscore are just odd.
      Add hso_dbg(level, fmt, ...) and use it everwhere instead.
      
      Several uses had additional unnecessary newlines as the
      macro added a newline.  Remove the newline from the macro
      and add newlines to each use as appropriate.
      
      Remove now unused D<digit> macros.
      Signed-off-by: default avatarJoe Perches <joe@perches.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      95a69117
    • Woojung Huh's avatar
      smsc95xx: Add mdix control via ethtool · 13722bbe
      Woojung Huh authored
      Add mdix control through ethtool.
      Signed-off-by: default avatarWoojung Huh <Woojung.huh@gmail.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      13722bbe
    • Woojung Huh's avatar
      smsc95xx: Add register define · 273bf288
      Woojung Huh authored
      Add STRAP_STATUS defines.
      Signed-off-by: default avatarWoojung Huh <Woojung.huh@gmail.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      273bf288
    • Woojung Huh's avatar
      smsc95xx: Add maintainer · 983ccd74
      Woojung Huh authored
      Add Microchip Linux Driver Support as maintainer
      because this driver is maintaining by Microchip.
      Signed-off-by: default avatarWoojung Huh <Woojung.huh@gmail.com>
      Acked-by: default avatarSteve Glendinning <steve.glendinning@shawell.net>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      983ccd74
    • David S. Miller's avatar
      Merge branch 'mv88e6xxx-isolate-Global2' · 464520a1
      David S. Miller authored
      Vivien Didelot says:
      
      ====================
      net: dsa: mv88e6xxx: isolate Global2 support
      
      Registers of Marvell chips are organized in internal SMI devices.
      
      One of them at address 0x1C is called Global2. It provides an extended
      set of registers, used for interrupt control, EEPROM access, indirect
      PHY access (to bypass the PHY Polling Unit) and cross-chip setup.
      
      Most chips have it, but some others don't (older ones such as 6060).
      
      Now that its related code is isolated in mv88e6xxx_g2_* functions, move
      it to its own global2.c file, making most of its setup code static.
      
      Then make its compilation optional, which allows to reduce the size of
      the mv88e6xxx driver for devices such as home routers embedding Ethernet
      chips without Global2 support.
      
      It is present on most recent chips, thus enable its support by default.
      
      Changes in v2: fail probe if GLOBAL2 is required but not enabled.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      464520a1
    • Vivien Didelot's avatar
      net: dsa: mv88e6xxx: make global2 code optional · ca070c10
      Vivien Didelot authored
      Since not every chip has a Global2 set of registers, make its support
      optional, in which case the related functions will return -EOPNOTSUPP.
      
      This also allows to reduce the size of the mv88e6xxx driver for devices
      such as home routers embedding Ethernet chips without Global2 support.
      
      It is present on most recent chips, thus enable its support by default.
      Signed-off-by: default avatarVivien Didelot <vivien.didelot@savoirfairelinux.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      ca070c10
    • Vivien Didelot's avatar
      net: dsa: mv88e6xxx: move Global2 code · ec561276
      Vivien Didelot authored
      Marvell chips are composed of multiple SMI devices. One of them at
      address 0x1C is called Global2. It provides an extended set of
      registers, used for interrupt control, EEPROM access, indirect PHY
      access (to bypass the PHY Polling Unit) and cross-chip related setup.
      
      Most chips have it, but some others don't (older ones such as 6060).
      
      Now that its related code is isolated in mv88e6xxx_g2_* functions, move
      it to its own global2.c file, making most of its setup code static.
      Document each registers in the meantime.
      
      Its compilation can be later avoided for chips without such registers.
      Signed-off-by: default avatarVivien Didelot <vivien.didelot@savoirfairelinux.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      ec561276
    • Vivien Didelot's avatar
      net: dsa: mv88e6xxx: fix module naming · 6654d0bf
      Vivien Didelot authored
      Since the mv88e6xxx.c file has been renamed, the driver compiled as a
      module is called chip.ko instead of mv88e6xxx.ko. Fix this.
      
      Fixes: fad09c73 ("net: dsa: mv88e6xxx: rename single-chip support")
      Signed-off-by: default avatarVivien Didelot <vivien.didelot@savoirfairelinux.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      6654d0bf
    • David S. Miller's avatar
      Merge git://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next · 60175ccd
      David S. Miller authored
      Pablo Neira Ayuso says:
      
      ====================
      Netfilter updates for net-next
      
      The following patchset contains Netfilter updates for your net-next
      tree.  Most relevant updates are the removal of per-conntrack timers to
      use a workqueue/garbage collection approach instead from Florian
      Westphal, the hash and numgen expression for nf_tables from Laura
      Garcia, updates on nf_tables hash set to honor the NLM_F_EXCL flag,
      removal of ip_conntrack sysctl and many other incremental updates on our
      Netfilter codebase.
      
      More specifically, they are:
      
      1) Retrieve only 4 bytes to fetch ports in case of non-linear skb
         transport area in dccp, sctp, tcp, udp and udplite protocol
         conntrackers, from Gao Feng.
      
      2) Missing whitespace on error message in physdev match, from Hangbin Liu.
      
      3) Skip redundant IPv4 checksum calculation in nf_dup_ipv4, from Liping Zhang.
      
      4) Add nf_ct_expires() helper function and use it, from Florian Westphal.
      
      5) Replace opencoded nf_ct_kill() call in IPVS conntrack support, also
         from Florian.
      
      6) Rename nf_tables set implementation to nft_set_{name}.c
      
      7) Introduce the hash expression to allow arbitrary hashing of selector
         concatenations, from Laura Garcia Liebana.
      
      8) Remove ip_conntrack sysctl backward compatibility code, this code has
         been around for long time already, and we have two interfaces to do
         this already: nf_conntrack sysctl and ctnetlink.
      
      9) Use nf_conntrack_get_ht() helper function whenever possible, instead
         of opencoding fetch of hashtable pointer and size, patch from Liping Zhang.
      
      10) Add quota expression for nf_tables.
      
      11) Add number generator expression for nf_tables, this supports
          incremental and random generators that can be combined with maps,
          very useful for load balancing purpose, again from Laura Garcia Liebana.
      
      12) Fix a typo in a debug message in FTP conntrack helper, from Colin Ian King.
      
      13) Introduce a nft_chain_parse_hook() helper function to parse chain hook
          configuration, this is used by a follow up patch to perform better chain
          update validation.
      
      14) Add rhashtable_lookup_get_insert_key() to rhashtable and use it from the
          nft_set_hash implementation to honor the NLM_F_EXCL flag.
      
      15) Missing nulls check in nf_conntrack from nf_conntrack_tuple_taken(),
          patch from Florian Westphal.
      
      16) Don't use the DYING bit to know if the conntrack event has been already
          delivered, instead a state variable to track event re-delivery
          states, also from Florian.
      
      17) Remove the per-conntrack timer, use the workqueue approach that was
          discussed during the NFWS, from Florian Westphal.
      
      18) Use the netlink conntrack table dump path to kill stale entries,
          again from Florian.
      
      19) Add a garbage collector to get rid of stale conntracks, from
          Florian.
      
      20) Reschedule garbage collector if eviction rate is high.
      
      21) Get rid of the __nf_ct_kill_acct() helper.
      
      22) Use ARPHRD_ETHER instead of hardcoded 1 from ARP logger.
      
      23) Make nf_log_set() interface assertive on unsupported families.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      60175ccd
  2. 04 Sep, 2016 13 commits