Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <16b4ab57-dfb0-2c1d-9be1-57da30dff3c3@intel.com>
Date:   Wed, 26 Jul 2023 12:43:30 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.12.0
Subject: Re: [PATCH net-next v2] page_pool: split types and declarations from
 page_pool.h
Content-Language: en-US
To:     Alexander H Duyck <alexander.duyck@gmail.com>
CC:     Yunsheng Lin <linyunsheng@huawei.com>, <davem@davemloft.net>,
        <kuba@kernel.org>, <pabeni@redhat.com>, <netdev@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, Eric Dumazet <edumazet@google.com>,
        Wei Fang <wei.fang@nxp.com>,
        Shenwei Wang <shenwei.wang@nxp.com>,
        Clark Wang <xiaoning.wang@nxp.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        Sunil Goutham <sgoutham@marvell.com>,
        Geetha sowjanya <gakula@marvell.com>,
        "Subbaraya Sundeep" <sbhatta@marvell.com>,
        hariprasad <hkelam@marvell.com>,
        "Saeed Mahameed" <saeedm@nvidia.com>,
        Leon Romanovsky <leon@kernel.org>,
        "Alexei Starovoitov" <ast@kernel.org>,
        Daniel Borkmann <daniel@iogearbox.net>,
        "Jesper Dangaard Brouer" <hawk@kernel.org>,
        John Fastabend <john.fastabend@gmail.com>,
        Felix Fietkau <nbd@nbd.name>,
        Lorenzo Bianconi <lorenzo@kernel.org>,
        "Ryder Lee" <ryder.lee@mediatek.com>,
        Shayne Chen <shayne.chen@mediatek.com>,
        "Sean Wang" <sean.wang@mediatek.com>,
        Kalle Valo <kvalo@kernel.org>,
        "Matthias Brugger" <matthias.bgg@gmail.com>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>,
        Ilias Apalodimas <ilias.apalodimas@linaro.org>,
        <linux-rdma@vger.kernel.org>, <bpf@vger.kernel.org>,
        <linux-wireless@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-mediatek@lists.infradead.org>
References: <20230725131258.31306-1-linyunsheng@huawei.com>
 <94272ffed7636c4c92fcc73ccfc15236dd8e47dc.camel@gmail.com>
From:   Alexander Lobakin <aleksander.lobakin@intel.com>
In-Reply-To: <94272ffed7636c4c92fcc73ccfc15236dd8e47dc.camel@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TGxpc0t6bG44UkFING9zQ3lRckZDcnB1Y0dmcG5kT0Vtb3VFR0U2TmZ6cWtG?=
 =?utf-8?B?dGtJYlJGRnRudHZEV01NSG0xb1VkV3VHdjA5ODZLM3NXUVIzUzM0MC9wcEsz?=
 =?utf-8?B?Ynp0bDBQY2kxZ2h5YitjL0RUUm1KdkpuZzZaeS9oSkp1VnRzY2Ywb0UyRjRT?=
 =?utf-8?B?Wi9JT0paNmxoTTUxdm0ya3g1WmgwRUgvV0pHL2J6OWd4MkdqQjVUUHNETU9X?=
 =?utf-8?B?T2I4MHJiVEFaWk1LZUFZeDhJMmFkWHlaYWtLdTNnWUtVQzlJVmZWU08vMVRm?=
 =?utf-8?B?bkpCZFhJWlBmdjZGcWtMdkpvVGluYzErbTZxWEpzMWRHanhESmdCTjhFaVU2?=
 =?utf-8?B?SEVBTTZDWU45YTUxbk1sUVkzWlBxVm02ZGZqM0w4VGdzeDJHczJZcjlQSkc4?=
 =?utf-8?B?V2pBRXgvcDNQTTh4b0pnUVpRSnlxMGlpTzRWMU9LMm1xeXhyMU5MYWphSENy?=
 =?utf-8?B?dHZPR2pBNHN2T01aYnI2Nll3TjNwa1h4RlNlZDlTVVVyaXoxK3N2eDA1V0po?=
 =?utf-8?B?cDhGYjR6L21vU0xVeDJLOWZFRG0wZzladFlNbDFLRTl4MlJwOFlTWnl6L0F0?=
 =?utf-8?B?Njl2UW9TWkdocU1FZ1dDWGYvdnFKc1I2ME9HbGIvMk9kVm1MbCtkbGxDYm16?=
 =?utf-8?B?TGdMa3NDMDZlTXIzMnc2UC9vOHlXWWNKbTQ2UGQvQ0VlUGhKQVJkT0YvT3Fl?=
 =?utf-8?B?aDFBU1RueDhVRlk2dHBIUmdYQmN0OXB6SXZXeVhyS1lXc2hTSmpMdFZ3MGta?=
 =?utf-8?B?d2xGNnpST2lRV1ZiUkNsYzREWGtHVkRHNmozQjU3OUx4d3Q2N29IZjRJN3pJ?=
 =?utf-8?B?VFRVaURpb2lhdDlpYmFIU1BXYkt3ZEFnTm1QdTVqQmQ1NHpiemM0ZjRpdHBX?=
 =?utf-8?B?TjM2NnkvcGFtYWhPVVp2citTQXJ0emdsblpnMVIraW9mRnFqYjFocGtqUm44?=
 =?utf-8?B?Tk9HUU9MN2ovMXh1MExQMURYT092Z3RmTnl4bHpDeXd0TENSL0Rnam4wUWhJ?=
 =?utf-8?B?RWh3c2w0YkVOUXc1dWlKVmJVN2ZFaUR1bHpIOFdaQnVJZHdkKzkwZG53aEtn?=
 =?utf-8?B?TXJHYk5WN1lwWVFydTFUTHgwYmpTMm13UXYzZFpkdFpXUGVyVXNYcXBDNDdT?=
 =?utf-8?B?YXFtcVBJV3JUTGZzcmdUaTZybHA1WUlRY2R1UXNnckJVOWVGUDYzMEJXWFFu?=
 =?utf-8?B?N04zcVZKaXp4SmRrTXlpSDgxbFA3Q0dLMkZuQjZHTDhLZU1zR0FYNTdpK1Ju?=
 =?utf-8?B?VllEQ2Vuc281d0pUMUJseURMb3lrK1krdFBpY2lkODIxVHJSQlRrbE1JZFFh?=
 =?utf-8?B?UmdJbGh4SnYyT3pXb1ZHYUZYNEl1aVEvQXE2NjRrS0hNRmJpYmZ2WWlWL1pY?=
 =?utf-8?B?MTN3ejdoNFpFajArem43RnFjanZZb1J4Z05ySXUvRVFQM3B6Tng4K0F5YVZw?=
 =?utf-8?B?Q2NTMkhtRXRpMUhXK1FjMUtqV0NVWGJuZEJ6clliR0pFMUVLQzRuVFRmVDRM?=
 =?utf-8?B?bXR3NnUvZC93YktMNmpwZEFJQUE2aVh2eUgwN1hVM2RnajBJRGpkRkNVV1Mr?=
 =?utf-8?B?eWVXUXg4OG1zSGJRM2lTa2RpcVlRWGp4TlFqc0orTlZ4Q3NVSERsbk1teUZx?=
 =?utf-8?B?NTVHRGRYdXJkWFdvSTlORXVMQ0g3Y2NCNmk3ZlVGbWdCdEdKaDhDMFhNT1pv?=
 =?utf-8?B?T1ZpM2FJUzFYYkEwLzdNV3IyQWFWUGMyWERrenloT0tES1lZQ1NlRldFTW51?=
 =?utf-8?B?RU0waE1QUTdGaU9RMCtqK29TL2RUdFRuQkRvY0VJdXBwNkdnQktYNzUxWjZ6?=
 =?utf-8?B?R3QvRDZrM3RrbDMxZm9pc2tXeFBsaU0wN1c5VzhWVHVZWnpLbVAzVUFoTm8x?=
 =?utf-8?B?TUhHVlJJQ2tRNDRsMVV4UHo1dWNqdFZwMjYya0FwamhrTUtmVktFcjdPOWFW?=
 =?utf-8?B?NnhFRXBQUVdZd3hnMHdpc1RGbVBuNlNjZ0tlR3BtalByakhTamRUQ0gyUFhp?=
 =?utf-8?B?eEc3UGFwS1BmNzF3blFHV1BDRGtTYzJGVlJVUmlnYVZTb0xNNGgwTUVZbWls?=
 =?utf-8?B?RkhOTGNWdTFzOUpzYnlXVHN0eWVzaUc0KzRyK2lhOXc2MTFXd0paTk54WGNC?=
 =?utf-8?B?Y1hYSldHK3FTUmhHVldYZXhvcUpmTXZRSUFxbnpEU0NMMlErSmo4ZGlyUUVi?=
 =?utf-8?B?bWc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 0a3b3055-9f09-4c6c-9885-08db8dc55aaf
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB3625.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jul 2023 10:44:57.5228
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: cATLo912uCZa0Ogw9xRcIXkj6m01KbNJNmkWqVo7dLNzKwXW9zaCkt5eWbFo2o9TaTX+FGjEGR3RPQCnE3meSIal8/uXlQuiwZ3S0gb+ycE=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB6852
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,
        SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-wireless.vger.kernel.org>
X-Mailing-List: linux-wireless@vger.kernel.org

From: Alexander H Duyck <alexander.duyck@gmail.com>
Date: Tue, 25 Jul 2023 08:47:46 -0700

> On Tue, 2023-07-25 at 21:12 +0800, Yunsheng Lin wrote:
>> Split types and pure function declarations from page_pool.h
>> and add them in page_pool/types.h, so that C sources can
>> include page_pool.h and headers should generally only include
>> page_pool/types.h as suggested by jakub.
>>
>> Signed-off-by: Yunsheng Lin <linyunsheng@huawei.com>
>> Suggested-by: Jakub Kicinski <kuba@kernel.org>
>> CC: Alexander Lobakin <aleksander.lobakin@intel.com>

[...]

>> +/* Caller must provide appropriate safe context, e.g. NAPI. */
>> +void page_pool_update_nid(struct page_pool *pool, int new_nid);
>> +
>> +#endif /* _NET_PAGE_POOL_H */
> 
> 
> This seems kind of overkill for what is needed. It seems like the
> general thought process with splitting this was so that you had just
> the minimum of what is needed to support skbuff.h and the functions
> declared there. The rest of this would then be added via the .h to the
> .c files that will actually be calling the functions.
> 
> By that logic I think the only thing we really need is the function
> declaration for page_pool_return_skb_page moved into skbuff.h. We could
> then just remove page_pool.h from skbuff.h couldn't we?

This patch is not to drop page_pool.h include from skbuff.h.
This is more future-proof (since I'm dropping this include anyway in my
series) to have includes organized and prevent cases like that one with
skbuff.h from happening. And to save some CPU cycles on preprocessing if
that makes sense.

> 
> Another thing we could consider doing is looking at splitting things up
> so that we had a include file in net/core/page_pool.h to handle some of
> the cases where we are just linking the page_pool bits to other core
> file bits such as xdp.c and skbuff.c.

Thanks,
Olek