Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Subject: Re: [PATCH net] net: fec: Use page_pool_put_full_page when freeing rx
 buffers
From:   patchwork-bot+netdevbpf@kernel.org
Message-Id: <167447101529.31244.5946300408564570569.git-patchwork-notify@kernel.org>
Date:   Mon, 23 Jan 2023 10:50:15 +0000
References: <20230119043747.943452-1-wei.fang@nxp.com>
In-Reply-To: <20230119043747.943452-1-wei.fang@nxp.com>
To:     Wei Fang <wei.fang@nxp.com>
Cc:     shenwei.wang@nxp.com, xiaoning.wang@nxp.com, davem@davemloft.net,
        edumazet@google.com, kuba@kernel.org, pabeni@redhat.com,
        netdev@vger.kernel.org, linux-imx@nxp.com,
        linux-kernel@vger.kernel.org
Precedence: bulk

Hello:

This patch was applied to netdev/net.git (master)
by David S. Miller <davem@davemloft.net>:

On Thu, 19 Jan 2023 12:37:47 +0800 you wrote:
> From: Wei Fang <wei.fang@nxp.com>
> 
> The page_pool_release_page was used when freeing rx buffers, and this
> function just unmaps the page (if mapped) and does not recycle the page.
> So after hundreds of down/up the eth0, the system will out of memory.
> For more details, please refer to the following reproduce steps and
> bug logs. To solve this issue and refer to the doc of page pool, the
> page_pool_put_full_page should be used to replace page_pool_release_page.
> Because this API will try to recycle the page if the page refcnt equal to
> 1. After testing 20000 times, the issue can not be reproduced anymore
> (about testing 391 times the issue will occur on i.MX8MN-EVK before).
> 
> [...]

Here is the summary with links:
  - [net] net: fec: Use page_pool_put_full_page when freeing rx buffers
    https://git.kernel.org/netdev/net/c/e38553bdc377

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html