1. 27 Jan, 2022 18 commits
    • Lorenzo Bianconi's avatar
      net: ethernet: mtk_star_emac: fix unused variable · a9c5eb64
      Lorenzo Bianconi authored
      Fix the following warning in mtk_star_emac.c if CONFIG_OF is not set:
      
      drivers/net/ethernet/mediatek/mtk_star_emac.c:1559:34:
          warning: unused variable 'mtk_star_of_match' [-Wunused-const-variable]
          static const struct of_device_id mtk_star_of_match[] = {
      Signed-off-by: default avatarLorenzo Bianconi <lorenzo@kernel.org>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      a9c5eb64
    • David S. Miller's avatar
      Merge branch 'xgmac_mdio-preamble-suppression-and-custom-MDC-frequerncies' · f0a910dd
      David S. Miller authored
      Tobias Waldekranz says:
      
      ====================
      net/fsl: xgmac_mdio: Preamble suppression and custom MDC frequencies
      
      The first patch removes the docs for a binding that has never been
      supported by the driver as far as I can see. This is a bit of a
      mystery to me, maybe Freescale/NXP had/has support for it in an
      internal version?
      
      We then start working on the xgmac_mdio driver, converting the driver
      to exclusively use managed resources, thereby simplifying the error
      paths. Suggested by Andrew.
      
      Preamble suppression is then added, followed by MDC frequency
      customization. Neither code will change any bits if the corresponding
      dt properties are not specified, so as to not trample on any setup
      done by the bootloader, which boards might have relied on up to now.
      
      Finally, we document the new bindings.
      
      Tested on a T1023 based board.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      f0a910dd
    • Tobias Waldekranz's avatar
      dt-bindings: net: xgmac_mdio: Add "clock-frequency" and "suppress-preamble" · f7af8fe8
      Tobias Waldekranz authored
      The driver now supports the standard "clock-frequency" and
      "suppress-preamble" properties, do document them in the binding
      description.
      Signed-off-by: default avatarTobias Waldekranz <tobias@waldekranz.com>
      Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      f7af8fe8
    • Tobias Waldekranz's avatar
      net/fsl: xgmac_mdio: Support setting the MDC frequency · dd8f467e
      Tobias Waldekranz authored
      Support the standard "clock-frequency" attribute to set the generated
      MDC frequency. If not specified, the driver will leave the divisor
      bits untouched.
      Signed-off-by: default avatarTobias Waldekranz <tobias@waldekranz.com>
      Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      dd8f467e
    • Tobias Waldekranz's avatar
      net/fsl: xgmac_mdio: Support preamble suppression · 909bea73
      Tobias Waldekranz authored
      Support the standard "suppress-preamble" attribute to disable preamble
      generation.
      Signed-off-by: default avatarTobias Waldekranz <tobias@waldekranz.com>
      Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      909bea73
    • Tobias Waldekranz's avatar
      net/fsl: xgmac_mdio: Use managed device resources · 1d14eb15
      Tobias Waldekranz authored
      All of the resources used by this driver has managed interfaces, so
      use them. Heed the warning in the comment before platform_get_resource
      and use a bare devm_ioremap to allow for non-exclusive access to the
      IO memory.
      Signed-off-by: default avatarTobias Waldekranz <tobias@waldekranz.com>
      Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      1d14eb15
    • Tobias Waldekranz's avatar
      dt-bindings: net: xgmac_mdio: Remove unsupported "bus-frequency" · 15ca0518
      Tobias Waldekranz authored
      This property has never been supported by the driver. The kernel has
      settled on "clock-frequency" as the standard name for this binding, so
      once that is supported we will document that instead.
      
      Fixes: 7f93c9d9 ("power/fsl: add MDIO dt binding for FMan")
      Signed-off-by: default avatarTobias Waldekranz <tobias@waldekranz.com>
      Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      15ca0518
    • xu xin's avatar
      ipv4: Namespaceify min_adv_mss sysctl knob · 2e9589ff
      xu xin authored
      Different netns has different requirement on the setting of min_adv_mss
      sysctl which the advertised MSS will be never lower than.
      
      Enable min_adv_mss to be configured per network namespace.
      Signed-off-by: default avatarxu xin <xu.xin16@zte.com.cn>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      2e9589ff
    • David S. Miller's avatar
      Merge branch 'mvneta-mac_select_pcs' · 349fdca6
      David S. Miller authored
      Russell King says:
      
      ====================
      net: mvneta: use .mac_select_pcs()
      
      This series converts mvneta to use the .mac_select_pcs() like eventually
      everything else will be. mvneta is slightly more involved because we
      need to rearrange the initialisation first to ensure everything required
      is initialised prior to phylink_create() being called.
      
      Tested locally on SolidRun Clearfog.
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      349fdca6
    • Russell King (Oracle)'s avatar
      net: mvneta: use .mac_select_pcs() interface · 0ac4a71f
      Russell King (Oracle) authored
      Convert mvneta to use the mac_select_interface rather than using
      phylink_set_pcs(). The intention here is to unify the approach for
      PCS and eventually remove phylink_set_pcs().
      Signed-off-by: default avatarRussell King (Oracle) <rmk+kernel@armlinux.org.uk>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      0ac4a71f
    • Russell King (Oracle)'s avatar
      net: mvneta: reorder initialisation · 72bb9531
      Russell King (Oracle) authored
      Re-order the mvneta initialisation to move devm based resources and
      easy setup earlier in the probe function. The primary reason for this
      is to allow us to switch the driver to use phylink's mac_select_pcs()
      callback.
      Signed-off-by: default avatarRussell King (Oracle) <rmk+kernel@armlinux.org.uk>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      72bb9531
    • David S. Miller's avatar
      Merge branch 'at803x-sfp-fiber' · 7ded129b
      David S. Miller authored
      Robert Hancock says:
      
      ====================
      at803x fiber/SFP support
      
      Add support for 1000Base-X fiber modes to the at803x PHY driver, as
      well as support for connecting a downstream SFP cage.
      
      Changes since v3:
      -Renamed some constants with OHM suffix for clarity
      
      Changes since v2:
      -fixed tabs/spaces issue in one patch
      
      Changes since v1:
      -moved page selection to config_init so it is handled properly
      after suspend/resume
      -added explicit check for empty sfp_support bitmask
      ====================
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      7ded129b
    • Robert Hancock's avatar
      net: phy: at803x: Support downstream SFP cage · dc4d5fcc
      Robert Hancock authored
      Add support for downstream SFP cages for AR8031 and AR8033. This is
      primarily intended for fiber modules or direct-attach cables, however
      copper modules which work in 1000Base-X mode may also function. Such
      modules are allowed with a warning.
      Signed-off-by: default avatarRobert Hancock <robert.hancock@calian.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      dc4d5fcc
    • Robert Hancock's avatar
      net: phy: at803x: add fiber support · 3265f421
      Robert Hancock authored
      Previously this driver always forced the copper page to be selected,
      however for AR8031 in 100Base-FX or 1000Base-X modes, the fiber page
      needs to be selected. Set the appropriate mode based on the hardware
      mode_cfg strap selection.
      
      Enable the appropriate interrupt bits to detect fiber-side link up
      or down events.
      
      Update config_aneg and read_status methods to use the appropriate
      Clause 37 calls when fiber mode is in use.
      Signed-off-by: default avatarRobert Hancock <robert.hancock@calian.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      3265f421
    • Robert Hancock's avatar
      net: phy: at803x: move page selection fix to config_init · 4f3a00c7
      Robert Hancock authored
      The fix to select the copper page on AR8031 was being done in the probe
      function rather than config_init, so it would not be redone after resume
      from suspend. Move this to config_init so it is always redone when
      needed.
      
      Fixes: c329e5af ("net: phy: at803x: select correct page on config init")
      Signed-off-by: default avatarRobert Hancock <robert.hancock@calian.com>
      Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
      4f3a00c7
    • Eric Dumazet's avatar
      tcp: allocate tcp_death_row outside of struct netns_ipv4 · fbb82952
      Eric Dumazet authored
      I forgot tcp had per netns tracking of timewait sockets,
      and their sysctl to change the limit.
      
      After 0dad4087 ("tcp/dccp: get rid of inet_twsk_purge()"),
      whole struct net can be freed before last tw socket is freed.
      
      We need to allocate a separate struct inet_timewait_death_row
      object per netns.
      
      tw_count becomes a refcount and gains associated debugging infrastructure.
      
      BUG: KASAN: use-after-free in inet_twsk_kill+0x358/0x3c0 net/ipv4/inet_timewait_sock.c:46
      Read of size 8 at addr ffff88807d5f9f40 by task kworker/1:7/3690
      
      CPU: 1 PID: 3690 Comm: kworker/1:7 Not tainted 5.16.0-syzkaller #0
      Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
      Workqueue: events pwq_unbound_release_workfn
      Call Trace:
       <IRQ>
       __dump_stack lib/dump_stack.c:88 [inline]
       dump_stack_lvl+0xcd/0x134 lib/dump_stack.c:106
       print_address_description.constprop.0.cold+0x8d/0x336 mm/kasan/report.c:255
       __kasan_report mm/kasan/report.c:442 [inline]
       kasan_report.cold+0x83/0xdf mm/kasan/report.c:459
       inet_twsk_kill+0x358/0x3c0 net/ipv4/inet_timewait_sock.c:46
       call_timer_fn+0x1a5/0x6b0 kernel/time/timer.c:1421
       expire_timers kernel/time/timer.c:1466 [inline]
       __run_timers.part.0+0x67c/0xa30 kernel/time/timer.c:1734
       __run_timers kernel/time/timer.c:1715 [inline]
       run_timer_softirq+0xb3/0x1d0 kernel/time/timer.c:1747
       __do_softirq+0x29b/0x9c2 kernel/softirq.c:558
       invoke_softirq kernel/softirq.c:432 [inline]
       __irq_exit_rcu+0x123/0x180 kernel/softirq.c:637
       irq_exit_rcu+0x5/0x20 kernel/softirq.c:649
       sysvec_apic_timer_interrupt+0x93/0xc0 arch/x86/kernel/apic/apic.c:1097
       </IRQ>
       <TASK>
       asm_sysvec_apic_timer_interrupt+0x12/0x20 arch/x86/include/asm/idtentry.h:638
      RIP: 0010:lockdep_unregister_key+0x1c9/0x250 kernel/locking/lockdep.c:6328
      Code: 00 00 00 48 89 ee e8 46 fd ff ff 4c 89 f7 e8 5e c9 ff ff e8 09 cc ff ff 9c 58 f6 c4 02 75 26 41 f7 c4 00 02 00 00 74 01 fb 5b <5d> 41 5c 41 5d 41 5e 41 5f e9 19 4a 08 00 0f 0b 5b 5d 41 5c 41 5d
      RSP: 0018:ffffc90004077cb8 EFLAGS: 00000206
      RAX: 0000000000000046 RBX: ffff88807b61b498 RCX: 0000000000000001
      RDX: dffffc0000000000 RSI: 0000000000000000 RDI: 0000000000000000
      RBP: ffff888077027128 R08: 0000000000000001 R09: ffffffff8f1ea4fc
      R10: fffffbfff1ff93ee R11: 000000000000af1e R12: 0000000000000246
      R13: 0000000000000000 R14: ffffffff8ffc89b8 R15: ffffffff90157fb0
       wq_unregister_lockdep kernel/workqueue.c:3508 [inline]
       pwq_unbound_release_workfn+0x254/0x340 kernel/workqueue.c:3746
       process_one_work+0x9ac/0x1650 kernel/workqueue.c:2307
       worker_thread+0x657/0x1110 kernel/workqueue.c:2454
       kthread+0x2e9/0x3a0 kernel/kthread.c:377
       ret_from_fork+0x1f/0x30 arch/x86/entry/entry_64.S:295
       </TASK>
      
      Allocated by task 3635:
       kasan_save_stack+0x1e/0x50 mm/kasan/common.c:38
       kasan_set_track mm/kasan/common.c:46 [inline]
       set_alloc_info mm/kasan/common.c:437 [inline]
       __kasan_slab_alloc+0x90/0xc0 mm/kasan/common.c:470
       kasan_slab_alloc include/linux/kasan.h:260 [inline]
       slab_post_alloc_hook mm/slab.h:732 [inline]
       slab_alloc_node mm/slub.c:3230 [inline]
       slab_alloc mm/slub.c:3238 [inline]
       kmem_cache_alloc+0x202/0x3a0 mm/slub.c:3243
       kmem_cache_zalloc include/linux/slab.h:705 [inline]
       net_alloc net/core/net_namespace.c:407 [inline]
       copy_net_ns+0x125/0x760 net/core/net_namespace.c:462
       create_new_namespaces+0x3f6/0xb20 kernel/nsproxy.c:110
       unshare_nsproxy_namespaces+0xc1/0x1f0 kernel/nsproxy.c:226
       ksys_unshare+0x445/0x920 kernel/fork.c:3048
       __do_sys_unshare kernel/fork.c:3119 [inline]
       __se_sys_unshare kernel/fork.c:3117 [inline]
       __x64_sys_unshare+0x2d/0x40 kernel/fork.c:3117
       do_syscall_x64 arch/x86/entry/common.c:50 [inline]
       do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
       entry_SYSCALL_64_after_hwframe+0x44/0xae
      
      The buggy address belongs to the object at ffff88807d5f9a80
       which belongs to the cache net_namespace of size 6528
      The buggy address is located 1216 bytes inside of
       6528-byte region [ffff88807d5f9a80, ffff88807d5fb400)
      The buggy address belongs to the page:
      page:ffffea0001f57e00 refcount:1 mapcount:0 mapping:0000000000000000 index:0xffff88807d5f9a80 pfn:0x7d5f8
      head:ffffea0001f57e00 order:3 compound_mapcount:0 compound_pincount:0
      memcg:ffff888070023001
      flags: 0xfff00000010200(slab|head|node=0|zone=1|lastcpupid=0x7ff)
      raw: 00fff00000010200 ffff888010dd4f48 ffffea0001404e08 ffff8880118fd000
      raw: ffff88807d5f9a80 0000000000040002 00000001ffffffff ffff888070023001
      page dumped because: kasan: bad access detected
      page_owner tracks the page as allocated
      page last allocated via order 3, migratetype Unmovable, gfp_mask 0xd20c0(__GFP_IO|__GFP_FS|__GFP_NOWARN|__GFP_NORETRY|__GFP_COMP|__GFP_NOMEMALLOC), pid 3634, ts 119694798460, free_ts 119693556950
       prep_new_page mm/page_alloc.c:2434 [inline]
       get_page_from_freelist+0xa72/0x2f50 mm/page_alloc.c:4165
       __alloc_pages+0x1b2/0x500 mm/page_alloc.c:5389
       alloc_pages+0x1aa/0x310 mm/mempolicy.c:2271
       alloc_slab_page mm/slub.c:1799 [inline]
       allocate_slab mm/slub.c:1944 [inline]
       new_slab+0x28a/0x3b0 mm/slub.c:2004
       ___slab_alloc+0x87c/0xe90 mm/slub.c:3018
       __slab_alloc.constprop.0+0x4d/0xa0 mm/slub.c:3105
       slab_alloc_node mm/slub.c:3196 [inline]
       slab_alloc mm/slub.c:3238 [inline]
       kmem_cache_alloc+0x35c/0x3a0 mm/slub.c:3243
       kmem_cache_zalloc include/linux/slab.h:705 [inline]
       net_alloc net/core/net_namespace.c:407 [inline]
       copy_net_ns+0x125/0x760 net/core/net_namespace.c:462
       create_new_namespaces+0x3f6/0xb20 kernel/nsproxy.c:110
       unshare_nsproxy_namespaces+0xc1/0x1f0 kernel/nsproxy.c:226
       ksys_unshare+0x445/0x920 kernel/fork.c:3048
       __do_sys_unshare kernel/fork.c:3119 [inline]
       __se_sys_unshare kernel/fork.c:3117 [inline]
       __x64_sys_unshare+0x2d/0x40 kernel/fork.c:3117
       do_syscall_x64 arch/x86/entry/common.c:50 [inline]
       do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
       entry_SYSCALL_64_after_hwframe+0x44/0xae
      page last free stack trace:
       reset_page_owner include/linux/page_owner.h:24 [inline]
       free_pages_prepare mm/page_alloc.c:1352 [inline]
       free_pcp_prepare+0x374/0x870 mm/page_alloc.c:1404
       free_unref_page_prepare mm/page_alloc.c:3325 [inline]
       free_unref_page+0x19/0x690 mm/page_alloc.c:3404
       skb_free_head net/core/skbuff.c:655 [inline]
       skb_release_data+0x65d/0x790 net/core/skbuff.c:677
       skb_release_all net/core/skbuff.c:742 [inline]
       __kfree_skb net/core/skbuff.c:756 [inline]
       consume_skb net/core/skbuff.c:914 [inline]
       consume_skb+0xc2/0x160 net/core/skbuff.c:908
       skb_free_datagram+0x1b/0x1f0 net/core/datagram.c:325
       netlink_recvmsg+0x636/0xea0 net/netlink/af_netlink.c:1998
       sock_recvmsg_nosec net/socket.c:948 [inline]
       sock_recvmsg net/socket.c:966 [inline]
       sock_recvmsg net/socket.c:962 [inline]
       ____sys_recvmsg+0x2c4/0x600 net/socket.c:2632
       ___sys_recvmsg+0x127/0x200 net/socket.c:2674
       __sys_recvmsg+0xe2/0x1a0 net/socket.c:2704
       do_syscall_x64 arch/x86/entry/common.c:50 [inline]
       do_syscall_64+0x35/0xb0 arch/x86/entry/common.c:80
       entry_SYSCALL_64_after_hwframe+0x44/0xae
      
      Memory state around the buggy address:
       ffff88807d5f9e00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
       ffff88807d5f9e80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
      >ffff88807d5f9f00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
                                                 ^
       ffff88807d5f9f80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
       ffff88807d5fa000: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
      
      Fixes: 0dad4087 ("tcp/dccp: get rid of inet_twsk_purge()")
      Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
      Reported-by: default avatarsyzbot <syzkaller@googlegroups.com>
      Reported-by: default avatarPaolo Abeni <pabeni@redhat.com>
      Tested-by: default avatarPaolo Abeni <pabeni@redhat.com>
      Link: https://lore.kernel.org/r/20220126180714.845362-1-eric.dumazet@gmail.comSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
      fbb82952
    • Simon Horman's avatar
      nfp: only use kdoc style comments for kdoc · 49db8a70
      Simon Horman authored
      Update comments to only use kdoc style comments, starting with '/**',
      for kdoc.
      
      Flagged by ./scripts/kernel-doc
      Signed-off-by: default avatarSimon Horman <simon.horman@corigine.com>
      Acked-by: default avatarRandy Dunlap <rdunlap@infradead.org>
      Tested-by: default avatarRandy Dunlap <rdunlap@infradead.org>
      Link: https://lore.kernel.org/r/20220126090803.5582-1-simon.horman@corigine.comSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
      49db8a70
    • Corentin Labbe's avatar
      net: ethernet: cortina: permit to set mac address in DT · 15f75fd3
      Corentin Labbe authored
      Add ability of setting mac address in DT for cortina ethernet driver.
      Signed-off-by: default avatarCorentin Labbe <clabbe@baylibre.com>
      Reviewed-by: default avatarLinus Walleij <linus.walleij@linaro.org>
      Link: https://lore.kernel.org/r/20220125210811.54350-1-clabbe@baylibre.comSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
      15f75fd3
  2. 26 Jan, 2022 22 commits