From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=74.125.82.197 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1794709569; b=yv65js86S8nZIEFutYiKVT8Hr+2dHuH7bSDR1Hx+c7e5Rg5gjMlK8Q7SVDl75LO84D1m UgGq/NWAq1uVabRY1UHqsvZB2PO3ZRauttp1qnSsDzKym3nOQ/VMTCl5eEKCAAKrBxUyH qI8SbaNuqVVfak3zOPgc3poQiYPQqsHaYb3ZhZWRmV5GoEeTtOq6+1LMh7cS52bkbqNlD EM/H/OTiIrS3yTOwe6m2ZWyg3+CuWIfFf8kxWsnVqf/2cS6plMIgGKGtaAI6xpSxXKokr ttlX8+Ylom6o1sN3QUHVgknq0OUJLx96AX8E3aqoYJQYTXvOfYEJ56guUzRoE+TKBvw== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1794709569; h=DMARC-Filter:Received:DKIM-Signature:X-Google-DKIM-Signature: X-Forwarded-Encrypted:X-Gm-Message-State:X-Received:Date:Mime-Version: X-Mailer:Message-ID:Subject:From:To:Cc:Content-Type; bh=2gE/Hbfi8pPeMmmTUakexK4mxqaVFBWDBadj/KmoNzw=; b=1sQbO85AnU09M45t2jpNxvutqqKOZ+Fa0YGduQjVcQ4aXbITXpo3oGQjdiNzLf/u1ujf gd9/NOToH1Hztq83vcWA8fbsfOpSw2tHZfjLrAteAlyeRKmqqt4zEQ1iC+hO3vVtHkF8Q EZX67g5jEw2LgUzF78k/zZbTJ6WIvpRCwCSysWlWaGtd6FH+S8rGZM/aOxiNXkjfq+QYp t3FL64ETLDFwNfgjXFxCieHwtaBDa1yIo+aFpdgokSXyjgd+ghXTEPs9NpoYAA6MT0a35 6+dHavD/zZLnbkfjXB3ip+IxecLsaO048KZs+W9Rb3e6zndGssi3h7s9kM/W1ANp3jg== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=google.com; spf=pass smtp.mailfrom=flex--almasrymina.bounces.google.com; dkim=pass header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20260707 header.b=bLswmWZC; arc=none smtp.remote-ip=74.125.82.197 Received: from mail-dy1-f197.google.com (mail-dy1-f197.google.com [74.125.82.197]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 78C7C606A32C for ; Sun, 15 Nov 2026 02:26:09 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org 78C7C606A32C Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--almasrymina.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20260707 header.b=bLswmWZC Received: by mail-dy1-f197.google.com with SMTP id 5a478bee46e88-378722fd4a3so2675281eec.0 for ; Sat, 14 Nov 2026 18:26:09 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20260707; t=1794709568; x=1795314368; darn=vger.kernel.org; h=content-type:cc:to:from:subject:message-id:mime-version:date:from :to:cc:subject:date:message-id:reply-to:content-type; bh=2gE/Hbfi8pPeMmmTUakexK4mxqaVFBWDBadj/KmoNzw=; b=bLswmWZCix3wzY0ceZoRrsWroupiuTdhBqNGCf7lYu77vcSuk9G+C1JP1zVktgBzX1 vCMBis1AAAlI+mbyfLLdlPrUJntv3S89cMAYgemDQ1F3oV290zdOzu497aXYlKWPKzua +sO3ZxRZX8fv3kQ0SHdS4OqhqFFes/9Ky6G+V0Kay8Co+/WipjNUDVc1nx07HlsQE2gK hrz8ISmCyixHggrwzmdlRQzWBNJusjbUtBqAzikBCOP7r0opCguK2HVMt174zFAAyoFq BfvcC0IhSqsnfX89NlAg5UOAmU+POhTu+S/AB6ApualvlRCqTHVoLllCR5LDyAZOKV7g 8p8w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1794709568; x=1795314368; h=content-type:cc:to:from:subject:message-id:mime-version:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=2gE/Hbfi8pPeMmmTUakexK4mxqaVFBWDBadj/KmoNzw=; b=XeRo1MEOoHFH63AElfQf9EoDKSZTNrAyfO8cTiV1G2a1PyaSoibFjuJ2DREVRXewLD kOO088++W6bSQfSY7PmKh5f2EzNMKN0uOUB+XjAwpgpP/QjV98Eeq6szt1M/NXNDHlps Vb/7t+RAUHODluzZCRd7MSc12am3HiKm3XgSa27wSdX4+uLez3QMnKF3kyT383Imb677 MjXF5uhxEL/bBw220L5dT89O06cc6rQsWQ1aBjMgLtZ1oAKFI+NY9YFkFt7aLDcX9LNO 4EbUoP8MMygPxPNeZ553oHNwanuApfTnHNYRSyLwaMfsBnn4pH2QFV7VaKsNLL18HXIP /EBA== X-Forwarded-Encrypted: i=1; AL+hErHsEvVH1NrYuOhyfXX71mTdzu278hhZ+6/kXKi7hikMU0BevRKd32A9Ja9SdXoSIe8TGSDC+6rzkigMf3c=@vger.kernel.org X-Gm-Message-State: APuU4Z+y624tnBeuHeQoglJas53/fHPpjcfImI/dun76cbzeaVHvB7gP hDXHR2JusHVs26y5Hd/e5hIcy+IbbwHaOsD288D9UcHZDE4iPfMdJKhTLt0Ev64g096KloPcXeP YqxN40beBmNC8gk8upXoPdP1OBA== X-Received: from dyah18-n1.prod.google.com ([2002:a05:693c:83d2:10b0:37a:8a0f:3aba]) (user=almasrymina job=prod-delivery.src-stubby-dispatcher) by 2002:a05:693c:414d:20b0:36c:d1fe:84ce with SMTP id 5a478bee46e88-3796740e475mr7608941eec.33.1794709568218; Sat, 14 Nov 2026 18:26:08 -0800 (PST) Date: Sun, 15 Nov 2026 02:26:01 +0000 X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list Mime-Version: 1.0 X-Mailer: git-send-email 2.56.0.611.g1030f0b940-goog Message-ID: <20261115022607.348112-1-almasrymina@google.com> Subject: [PATCH net-next v3 0/6] net: devmem: support leased RX and TX queues on netdev unregister From: Mina Almasry To: netdev@vger.kernel.org Cc: Mina Almasry , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Shuah Khan , linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org, Weiming Shi , Daniel Borkmann , Nikolay Aleksandrov , David Wei , Stanislav Fomichev , Bobby Eshleman , Dragos Tatulea Content-Type: text/plain; charset="UTF-8" Support leased RX and TX devmem bindings when either the virtual device (e.g. netkit) or the physical backing net_device is unregistered: 1. Pass struct net_device **dev_ptr to net_devmem_lock_dev() so callers can lock either binding->map_dev or binding->bind_dev. 2. Rename binding->dev and binding->vdev to binding->map_dev (physical DMA device) and binding->bind_dev (device passed to bind-rx/bind-tx). 3. Introduce netif_lease_valid() and guard netif_mp_open_rxq(), netif_mp_close_rxq(), and netif_rxq_cleanup_unlease() against an unregistering physical device reached through a queue lease. 4. Always populate bind_dev on RX and TX bindings and handle teardown and netdev unregistration of both bind_dev and map_dev. 5. Resolve the physical backing map_dev in netdev_nl_get_dma_dev() for leased RX bindings, lock map_dev around net_devmem_bind_dmabuf() on bind-rx when map_dev differs from netdev, assert both devices are locked, and check netif_lease_valid(map_dev). 6. Add kselftests in nl_netdev.py covering leased RX and TX devmem bindings across both virtual-first and physical-first netdev unregistration orders. v3: - Rebase onto latest net-next/main. - Add netif_lease_valid() helper and keep lease validity checks in the lease-aware callers (netif_mp_open_rxq(), netif_mp_close_rxq(), netif_rxq_cleanup_unlease(), and net_devmem_bind_dmabuf()) rather than inside __netif_mp_open_rxq() / __netif_mp_close_rxq(), and document why mp_params is cleared directly when !netif_lease_valid(dev) (Stanislav, Sashiko). - Guard the bound_rxqs loop in net_devmem_unbind_dmabuf() with if (binding->bind_dev), keep the !netif_is_queue_leasee(dev) guard in net_devmem_dev_unregister() in patch 4/6 until patch 5/6 resolves binding->map_dev to the physical device so the series is bisectable at every commit, and document the use-after-free in dma_unmap_sg_attrs() on physical netdev teardown in patch 5/6 (Sashiko). - In nl_netdev.py, verify dmabuf and lease on the physical queue after binding, skip when queue_create returns EOPNOTSUPP, probe the physical device on EOPNOTSUPP so a broken lease lookup is not masked as SKIP, and clean up nk_ifname between loop iterations (Sashiko). - v2: https://lore.kernel.org/netdev/20261108055809.2122934-1-almasrymina@google.com/ v2: - Split patch 3/4 from v1 into 3 smaller commits (patches 3/6..5/6). - Clear rxq->mp_params and return 0 in __netif_mp_close_rxq() whenever dev is already unregistering, avoiding queue reconfiguration on an already-shut-down map_dev during cascaded lease teardown (Sashiko). - Drop unnecessary xa_erase() in net_devmem_unbind_dmabuf() and use a local map_dev variable in net_devmem_get_binding() (Stanislav). - Restore ASSERT_EXCLUSIVE_WRITER(binding->bind_dev) in mp_dmabuf_devmem_uninstall(), drop redundant bind_dev->reg_state check in net_devmem_bind_dmabuf(), and expand commit message rationale (Dragos, Stanislav). - v1: https://lore.kernel.org/netdev/20261103232659.1340283-1-almasrymina@google.com/ Mina Almasry (6): net: devmem: pass net_device pointer to net_devmem_lock_dev() net: devmem: rename binding dev and vdev to map_dev and bind_dev net: guard memory provider RX queue open and close on unregister net: devmem: always populate bind_dev and detach both devices on unregister net: devmem: resolve physical map_dev for leased RX bindings selftests: net: add leased devmem RX and TX netdev unregister tests net/core/dev.c | 2 +- net/core/dev.h | 7 + net/core/devmem.c | 123 ++++++++++-------- net/core/devmem.h | 25 ++-- net/core/netdev-genl.c | 58 +++++---- net/core/netdev_rx_queue.c | 31 ++++- tools/testing/selftests/net/nl_netdev.py | 159 ++++++++++++++++++++++- 7 files changed, 304 insertions(+), 101 deletions(-) base-commit: 6964ff21c5d8b75649211a52dbfd92753e605649 -- 2.56.0.611.g1030f0b940-goog