From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A717AC61DB3 for ; Tue, 10 Jan 2023 18:53:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234967AbjAJSw7 (ORCPT ); Tue, 10 Jan 2023 13:52:59 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37186 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235073AbjAJSwd (ORCPT ); Tue, 10 Jan 2023 13:52:33 -0500 Received: from casper.infradead.org (casper.infradead.org [IPv6:2001:8b0:10b:1236::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C06CF4FCC8 for ; Tue, 10 Jan 2023 10:47:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=i/9o7iNl6zIhJocKYwkxzObk5xjtVqd6JUFtOY8FyKQ=; b=AELf9y87WqHFA8TfIY326rRMk5 Jr+ZcgRGBSwPm19GtkO7qNAS9E4q8pY+DcVleHVEQuFxit9tsYl1No0w8Wwj0XipdcLEU+xrqzwct pTy9wDqKztiLytpcUIeVN9MTgQxnNUX/nAJoHDuqz8FtEuRI/8+uJDvDHvoL4HogPy2b3e5SRWPAN Gl+6R1Z0KYr3IKWbdtTbAcHeIULTXCLVCxUM1aIQWQcwHpem20AMWnhlubqjN8SQ6omPOdXzwTHAi Y6GTs462QeuHu1yTT7T2AoL80KXaxIRCVnU859fvFoX6pv7AXBXi+vyx9XDGmzMRvXvNEW09u2Q+4 9si2KJuA==; Received: from willy by casper.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1pFJem-003SHj-Qf; Tue, 10 Jan 2023 18:47:32 +0000 Date: Tue, 10 Jan 2023 18:47:32 +0000 From: Matthew Wilcox To: Ilias Apalodimas Cc: Jesper Dangaard Brouer , netdev@vger.kernel.org, linux-mm@kvack.org, Shakeel Butt Subject: Re: [PATCH v2 04/24] page_pool: Convert page_pool_release_page() to page_pool_release_netmem() Message-ID: References: <20230105214631.3939268-1-willy@infradead.org> <20230105214631.3939268-5-willy@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Tue, Jan 10, 2023 at 11:28:10AM +0200, Ilias Apalodimas wrote: > > -static inline void page_pool_release_page(struct page_pool *pool, > > - struct page *page) > > +static inline void page_pool_release_netmem(struct page_pool *pool, > > + struct netmem *nmem) > > { > > } > > > > @@ -378,6 +378,12 @@ static inline void page_pool_put_page_bulk(struct page_pool *pool, void **data, > > } > > #endif > > > > I think it's worth commenting here that page_pool_release_page() is > eventually going to be removed once we convert all drivers and shouldn't > be used anymore OK. How about I add this to each function in this category: /* Compat, remove when all users gone */ and then we can find them all by grepping for 'Compat'?