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 83448C433F5 for ; Thu, 3 Mar 2022 22:41:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237021AbiCCWma (ORCPT ); Thu, 3 Mar 2022 17:42:30 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53490 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231782AbiCCWm3 (ORCPT ); Thu, 3 Mar 2022 17:42:29 -0500 Received: from casper.infradead.org (casper.infradead.org [IPv6:2001:8b0:10b:1236::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 539751662FF for ; Thu, 3 Mar 2022 14:41:43 -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=52IPoiZlULRFsRfp7a+MOcHPPK5OVaKArRJeKTkzhvo=; b=KRjqs5qByXCCtDLJuVi6CXCPEi Lpe0SgN11Y9rnKZhPft2uhbuREDPHnG+3PYR9R9faV5PJG2TCg9lOpWNOZHWAzJaT1lfLmOojK3xL 1P00Mu4GJnFpxvLRLiGHRJg1rmHpX3zEgakW1IWBZbdAnXP77Kp6GEI0XJGNfYbprut/I6tYMccVY R7AiMS81A4YWCTxb2MK1mCg6sLoiWcWzmYpvJiv+sagrOqn0TYMAIW8dpAC7qCWSThbsurU/ahF3P nSvw2sANT59tduACOlNAgWIVloDzRiiJPl2oDSOmqF6/3QLpFKZY5Epf3+UtBH2DVIFL2M4/PhNyt gHkqkyrw==; Received: from willy by casper.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1nPu8g-00C3LQ-Kt; Thu, 03 Mar 2022 22:41:38 +0000 Date: Thu, 3 Mar 2022 22:41:38 +0000 From: Matthew Wilcox To: Hugh Dickins Cc: David Hildenbrand , Andrew Morton , Nicholas Piggin , Yu Zhao , Yang Shi , Michal Hocko , linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH mmotm] mm: delete __ClearPageWaiters() Message-ID: References: <351df0af-78f2-c20-2a6d-e5f978e5ca1@google.com> <9dda55d5-eafa-3177-2a4c-32ccb7e146e3@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <9dda55d5-eafa-3177-2a4c-32ccb7e146e3@google.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Mar 03, 2022 at 02:28:46PM -0800, Hugh Dickins wrote: > On Thu, 3 Mar 2022, David Hildenbrand wrote: > > On 03.03.22 02:56, Hugh Dickins wrote: > > > The PG_waiters bit is not included in PAGE_FLAGS_CHECK_AT_FREE, and > > > vmscan.c's free_unref_page_list() callers rely on that not to generate > > > bad_page() alerts. So __page_cache_release() and release_pages() (and > > > the presumably copy-and-pasted put_zone_device_private_or_public_page()) > > Hah, I'm showing my age there, or the patch's age: it's been rebranded > frequently since then, with linux-next calling it free_zone_device_page(), > as you kindly point out. How long before it's free_zone_device_folio()? Probably not a serious question, but within the next year, I expect. I have a prototype patch to do the entire page freeing path, but it wasn't a priority for this merge window.