1. 11 Nov, 2022 36 commits
  2. 10 Nov, 2022 4 commits
    • Florian Fainelli's avatar
      MAINTAINERS: Move Vivien to CREDITS · 6ce3df59
      Florian Fainelli authored
      Last patch from Vivien was nearly 3 years ago and he has not reviewed or
      responded to DSA patches since then, move to CREDITS.
      Signed-off-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
      Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
      Link: https://lore.kernel.org/r/20221109231907.621678-1-f.fainelli@gmail.comSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
      6ce3df59
    • Jakub Kicinski's avatar
      genetlink: fix single op policy dump when do is present · c1b05105
      Jakub Kicinski authored
      Jonathan reports crashes when running net-next in Meta's fleet.
      Stats collection uses ethtool -I which does a per-op policy dump
      to check if stats are supported. We don't initialize the dumpit
      information if doit succeeds due to evaluation short-circuiting.
      
      The crash may look like this:
      
         BUG: kernel NULL pointer dereference, address: 0000000000000cc0
         RIP: 0010:netlink_policy_dump_add_policy+0x174/0x2a0
           ctrl_dumppolicy_start+0x19f/0x2f0
           genl_start+0xe7/0x140
      
      Or we may trigger a warning:
      
         WARNING: CPU: 1 PID: 785 at net/netlink/policy.c:87 netlink_policy_dump_get_policy_idx+0x79/0x80
         RIP: 0010:netlink_policy_dump_get_policy_idx+0x79/0x80
           ctrl_dumppolicy_put_op+0x214/0x360
      
      depending on what garbage we pick up from the stack.
      Reported-by: default avatarJonathan Lemon <bsd@meta.com>
      Fixes: 26588edb ("genetlink: support split policies in ctrl_dumppolicy_put_op()")
      Reviewed-by: default avatarJacob Keller <jacob.e.keller@intel.com>
      Tested-by: default avatarLeon Romanovsky <leonro@nvidia.com>
      Link: https://lore.kernel.org/r/20221109183254.554051-1-kuba@kernel.orgSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
      c1b05105
    • Ido Schimmel's avatar
      devlink: Fix warning when unregistering a port · 1fb22ed6
      Ido Schimmel authored
      When a devlink port is unregistered, its type is expected to be unset or
      otherwise a WARNING is generated [1]. This was supposed to be handled by
      cited commit by clearing the type upon 'NETDEV_PRE_UNINIT'.
      
      The assumption was that no other events can be generated for the netdev
      after this event, but this proved to be wrong. After the event is
      generated, netdev_wait_allrefs_any() will rebroadcast a
      'NETDEV_UNREGISTER' until the netdev's reference count drops to 1. This
      causes devlink to set the port type back to Ethernet.
      
      Fix by only setting and clearing the port type upon 'NETDEV_POST_INIT'
      and 'NETDEV_PRE_UNINIT', respectively. For all other events, preserve
      the port type.
      
      [1]
      WARNING: CPU: 0 PID: 11 at net/core/devlink.c:9998 devl_port_unregister+0x2f6/0x390 net/core/devlink.c:9998
      Modules linked in:
      CPU: 1 PID: 11 Comm: kworker/u4:1 Not tainted 6.1.0-rc3-next-20221107-syzkaller #0
      Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 10/26/2022
      Workqueue: netns cleanup_net
      RIP: 0010:devl_port_unregister+0x2f6/0x390 net/core/devlink.c:9998
      [...]
      Call Trace:
       <TASK>
       __nsim_dev_port_del+0x1bb/0x240 drivers/net/netdevsim/dev.c:1433
       nsim_dev_port_del_all drivers/net/netdevsim/dev.c:1443 [inline]
       nsim_dev_reload_destroy+0x171/0x510 drivers/net/netdevsim/dev.c:1660
       nsim_dev_reload_down+0x6b/0xd0 drivers/net/netdevsim/dev.c:968
       devlink_reload+0x1c2/0x6b0 net/core/devlink.c:4501
       devlink_pernet_pre_exit+0x104/0x1c0 net/core/devlink.c:12609
       ops_pre_exit_list net/core/net_namespace.c:159 [inline]
       cleanup_net+0x451/0xb10 net/core/net_namespace.c:594
       process_one_work+0x9bf/0x1710 kernel/workqueue.c:2289
       worker_thread+0x665/0x1080 kernel/workqueue.c:2436
       kthread+0x2e4/0x3a0 kernel/kthread.c:376
       ret_from_fork+0x1f/0x30 arch/x86/entry/entry_64.S:308
       </TASK>
      
      Fixes: 02a68a47 ("net: devlink: track netdev with devlink_port assigned")
      Reported-by: syzbot+85e47e1a08b3e159b159@syzkaller.appspotmail.com
      Reported-by: syzbot+c2ca18f0fccdd1f09c66@syzkaller.appspotmail.com
      Reviewed-by: default avatarJiri Pirko <jiri@nvidia.com>
      Signed-off-by: default avatarIdo Schimmel <idosch@nvidia.com>
      Link: https://lore.kernel.org/r/20221110085150.520800-1-idosch@nvidia.comSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
      1fb22ed6
    • Jakub Kicinski's avatar
      Merge branch 'mana-shared-6.2' of https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma · 79b0872b
      Jakub Kicinski authored
      Long Li says:
      
      ====================
      Introduce Microsoft Azure Network Adapter (MANA) RDMA driver [netdev prep]
      
      The first 11 patches which modify the MANA Ethernet driver to support
      RDMA driver.
      
      * 'mana-shared-6.2' of https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma:
        net: mana: Define data structures for protection domain and memory registration
        net: mana: Define data structures for allocating doorbell page from GDMA
        net: mana: Define and process GDMA response code GDMA_STATUS_MORE_ENTRIES
        net: mana: Define max values for SGL entries
        net: mana: Move header files to a common location
        net: mana: Record port number in netdev
        net: mana: Export Work Queue functions for use by RDMA driver
        net: mana: Set the DMA device max segment size
        net: mana: Handle vport sharing between devices
        net: mana: Record the physical address for doorbell page region
        net: mana: Add support for auxiliary device
      ====================
      
      Link: https://lore.kernel.org/all/1667502990-2559-1-git-send-email-longli@linuxonhyperv.com/Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
      79b0872b