Received: by 2002:ab2:7b86:0:b0:1f7:5705:b850 with SMTP id q6csp157793lqh; Fri, 3 May 2024 17:32:28 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCX/4fDLDSbMsBkWgME0qB+VRYoGiOXVdwmEbA9JkM5FNoZJPRQi4vZ9xfgO/EAYUgMsM/eQPy6BrnGlfT2Hbt3FYs9WaqfxT0UIE76hdA== X-Google-Smtp-Source: AGHT+IFuZi+DD9/PglKJPMXLnkXlqnz0XTNkERiTm1qJVAUvSWLsDcirgx7bb8owS+50CdQ/DNhB X-Received: by 2002:ac8:5742:0:b0:436:887c:6ced with SMTP id 2-20020ac85742000000b00436887c6cedmr4937689qtx.63.1714782748622; Fri, 03 May 2024 17:32:28 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1714782748; cv=pass; d=google.com; s=arc-20160816; b=VNlkzWXSigtq5LmnPUd/Zg8JwmNRFp9Odk5iYoNdQnhA8EcaHyqL0a9DRYgTcFIW9V I9mkFgdJfwXWUtDu2FR9nQVfhVtlvtS3UY6o97VURzmYORh5ZHYEhmlXyqZKr6gMJ4In lhsRAXgwmZYD2gmu53hlrN/yIauTAgm0brJJ0O9hCxMomuVOwc5Dz1Eq5236vU4wfJvb UnwOju2++K3bmdaraPQJ48zlE/IRPbp3z7GfYmYagQQbBhIVe3xHLTwF/BKvNWTy876N XSnyZpFmIbMkeizPyD0Wf68Q/HbxsTO9B1bYoRXMfyF6nNybPULAguYVfbVZFTpOwPzI hJgg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=cc:to:in-reply-to:references:date:message-id:from:subject :content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:dkim-signature; bh=9zZRQf/si/ewn5XkxSV9+K/+pH+CT2xxxMXlbIHyDWQ=; fh=LrTCNxTWnu7N0PEoHrlj4Whwb8txt4Ple+UXxNDBNWU=; b=re3cb1WAFYIqwb2Hq1lLJRSsQyczVxPtMv++2ZgMdkmlepCuuM79KKee+vTYFBBU+e 1PEHrcYfGfa4OchwjtQiZyFA5+01SPgWorX8atWwlbsjeFBCs2rgTta9AIL2cUK8j+mu 0eEslb7RPHUCoeJMuEey5D88/urc8QfDwY7YKIc9tNxP3SNCFFB9Geff5wR67oWiViJy UGnH1cfS7We6etrGEmbZnImqASzOTVlfVWFMTv3UzplxX6/dCgbN5KIE+N72V8QZV48v 9yoBDZgeJF12k2Ad5yiMnCwBuuljGDSIyuYKV8Mkp+dE/5n8nxk3q0qqvc2BGPgIfHSr J2pA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=BiAZ+JyP; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-168453-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-168453-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id p17-20020a05622a00d100b0043b086a4861si4512066qtw.64.2024.05.03.17.32.28 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 May 2024 17:32:28 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-168453-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=BiAZ+JyP; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-168453-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-168453-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 334541C20C84 for ; Sat, 4 May 2024 00:32:28 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 1FAAC208A8; Sat, 4 May 2024 00:30:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="BiAZ+JyP" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 46E2322612; Sat, 4 May 2024 00:30:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714782630; cv=none; b=Z+DpmYCq3+IqLfL5UzCeYWlTEaJ/Vk1/LNXQoThcUeRox9dUi6Y4zfCR+ZSo//rDOHIqU2UxPTe3XMKMe4wn/uuNwE7zWlrfeWG+w6nNr7r311BdQDKf03u/DxwdZ7qUokW31JU7w4FZfo02Eu0ZHYlbi4whWEi7HQrn2rvdyb0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714782630; c=relaxed/simple; bh=pZ6zcEY25nwGF9/B+NzjHkiKdVzmaaBaPXGGoBI9U4E=; h=Content-Type:MIME-Version:Subject:From:Message-Id:Date:References: In-Reply-To:To:Cc; b=PWUDJUjJdik59XVWtwOKjQuEC4m+fugdfnryxbIcpDPpfFZmd6uKhwR9qPKE8TV1BZAl3k5KHwR8mdbKGcOIBl4UQwNKvdKETtNQC16XF+yQuoG40s5li9WYMPux5SrbzY7+OKaNUuoBDUEGm1irNoTJyXnuUaCfDZs2r5vzcJI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=BiAZ+JyP; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPS id E5CCEC4AF1D; Sat, 4 May 2024 00:30:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1714782629; bh=pZ6zcEY25nwGF9/B+NzjHkiKdVzmaaBaPXGGoBI9U4E=; h=Subject:From:Date:References:In-Reply-To:To:Cc:From; b=BiAZ+JyPb88j3zTLxWy7gKEpPwO6S25+SMDHwavBOWtr0pkF0MMYdo5N+O8q34bNa 1SbgGLeCbxLbF8aplHso7bLLhehVEMKwYKes0YMuzae7Ee2GjmuwwJFg42d6gyGIez 2sg4PBb/8D5hEmd4emr4CrX3KDnb7uPK3Oo3EL32Mj2uJR5pffkjOAg/zbD4SNafHT yIsnW89wtiZ1mbWkM9+SoWVwHBLfwy1z3gr0rTq8jmcyRzNk+ULIlSTcEj80Tu2RcF 6KR9lRKWqY1Y9I6em1f3mnT8efUfr/BY3zr/NEgEA48QL2H/AHUq7a1TKKh5MGbgvW PthIc4LnOGGSg== Received: from aws-us-west-2-korg-oddjob-1.ci.codeaurora.org (localhost.localdomain [127.0.0.1]) by aws-us-west-2-korg-oddjob-1.ci.codeaurora.org (Postfix) with ESMTP id D4D29C43336; Sat, 4 May 2024 00:30:29 +0000 (UTC) Content-Type: text/plain; charset="utf-8" Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: Re: [PATCH net-next v1] Revert "net: mirror skb frag ref/unref helpers" From: patchwork-bot+netdevbpf@kernel.org Message-Id: <171478262986.21471.5306831974379816981.git-patchwork-notify@kernel.org> Date: Sat, 04 May 2024 00:30:29 +0000 References: <20240502175423.2456544-1-almasrymina@google.com> In-Reply-To: <20240502175423.2456544-1-almasrymina@google.com> To: Mina Almasry Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, ayush.sawal@chelsio.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, borisp@nvidia.com, john.fastabend@gmail.com, dtatulea@nvidia.com, jianbol@nvidia.com, jacob.e.keller@intel.com Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski : On Thu, 2 May 2024 10:54:22 -0700 you wrote: > This reverts commit a580ea994fd37f4105028f5a85c38ff6508a2b25. > > This revert is to resolve Dragos's report of page_pool leak here: > https://lore.kernel.org/lkml/20240424165646.1625690-2-dtatulea@nvidia.com/ > > The reverted patch interacts very badly with commit 2cc3aeb5eccc ("skbuff: > Fix a potential race while recycling page_pool packets"). The reverted > commit hopes that the pp_recycle + is_pp_page variables do not change > between the skb_frag_ref and skb_frag_unref operation. If such a change > occurs, the skb_frag_ref/unref will not operate on the same reference type. > In the case of Dragos's report, the grabbed ref was a pp ref, but the unref > was a page ref, because the pp_recycle setting on the skb was changed. > > [...] Here is the summary with links: - [net-next,v1] Revert "net: mirror skb frag ref/unref helpers" https://git.kernel.org/netdev/net-next/c/173e7622ccb3 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html