Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 23 Sep 2021 10:07:14 +0300
From:   Ilias Apalodimas <ilias.apalodimas@linaro.org>
To:     Yunsheng Lin <linyunsheng@huawei.com>
Cc:     davem@davemloft.net, kuba@kernel.org, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org, linuxarm@openeuler.org,
        hawk@kernel.org, jonathan.lemon@gmail.com, alobakin@pm.me,
        willemb@google.com, cong.wang@bytedance.com, pabeni@redhat.com,
        haokexin@gmail.com, nogikh@google.com, elver@google.com,
        memxor@gmail.com, edumazet@google.com, alexander.duyck@gmail.com,
        dsahern@gmail.com
Subject: Re: [PATCH net-next 0/7] some optimization for page pool
Message-ID: <YUwnogBl/qbNbQ7X@apalos.home>
References: <20210922094131.15625-1-linyunsheng@huawei.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210922094131.15625-1-linyunsheng@huawei.com>
Precedence: bulk

Hi Yunsheng, 

On Wed, Sep 22, 2021 at 05:41:24PM +0800, Yunsheng Lin wrote:
> Patch 1: disable dma mapping support for 32-bit arch with 64-bit
>          DMA.
> Patch 2: support non-split page when PP_FLAG_PAGE_FRAG is set.
> patch 3: avoid calling compound_head() for skb frag page
> Patch 4-7: use pp_magic to identify pp page uniquely.

There's some subtle changes in this patchset that might affect XDP.

What I forgot when I proposed removing the recycling bit,  is that it also
serves as an 'opt-in' mechanism for drivers that want to use page_pool but 
do the recycling internally.  With that removed we need to make sure
nothing bad happens to them.  In theory the page refcnt for mlx5
specifically will be elevated, so we'll just end up unmapping the buffer.
Arguably we could add a similar mechanism internally into page pool,  
which would allow us to enable and disable recycling,  but that's
an extra if per packet allocation and I don't know if we want that on the XDP 
case.
A few numbers pre/post patch for XDP would help, but iirc hns3 doesn't have
XDP support yet?

It's plumbers week so I'll do some testing starting Monday.

Thanks
/Ilias

> 
> V3:
>     1. add patch 1/4/6/7.
>     2. use pp_magic to identify pp page uniquely too.
>     3. avoid unnecessary compound_head() calling.
> 
> V2: add patch 2, adjust the commit log accroding to the discussion
>     in V1, and fix a compiler error reported by kernel test robot.
> 
> Yunsheng Lin (7):
>   page_pool: disable dma mapping support for 32-bit arch with 64-bit DMA
>   page_pool: support non-split page with PP_FLAG_PAGE_FRAG
>   pool_pool: avoid calling compound_head() for skb frag page
>   page_pool: change BIAS_MAX to support incrementing
>   skbuff: keep track of pp page when __skb_frag_ref() is called
>   skbuff: only use pp_magic identifier for a skb' head page
>   skbuff: remove unused skb->pp_recycle
> 
>  .../net/ethernet/hisilicon/hns3/hns3_enet.c   |  6 ---
>  drivers/net/ethernet/marvell/mvneta.c         |  2 -
>  .../net/ethernet/marvell/mvpp2/mvpp2_main.c   |  4 +-
>  drivers/net/ethernet/marvell/sky2.c           |  2 +-
>  drivers/net/ethernet/mellanox/mlx4/en_rx.c    |  2 +-
>  drivers/net/ethernet/ti/cpsw.c                |  2 -
>  drivers/net/ethernet/ti/cpsw_new.c            |  2 -
>  include/linux/mm_types.h                      | 13 +-----
>  include/linux/skbuff.h                        | 39 ++++++++----------
>  include/net/page_pool.h                       | 31 ++++++++------
>  net/core/page_pool.c                          | 40 +++++++------------
>  net/core/skbuff.c                             | 36 ++++++-----------
>  net/tls/tls_device.c                          |  2 +-
>  13 files changed, 67 insertions(+), 114 deletions(-)
> 
> -- 
> 2.33.0
>