From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qv1-f42.google.com (mail-qv1-f42.google.com [209.85.219.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D873D32ED3B for ; Wed, 5 Nov 2025 17:21:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762363280; cv=none; b=dWwB1OS7bh1XkhgHV/U+T8d4qKfGL9VpeSMzUC8M9rnnqKqm7eXhxgFy61rC3kbih3Eo4KsYWEJUpVetUpcbcBBZK8wLtemJnw3abyz9tj1FRoXfqDOj/NsFLrVVyRgUW+1JSwTbKjfYZO6BbC3jwGqakR6SDPHNo2c1mwYdnzI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762363280; c=relaxed/simple; bh=R+scxNFIJ7A2QKHZGih1Ehc3wdUpOLgJ/krGtFdCacU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=kKEGu/heGmA5yJMmVOTx+8Ujmn+fFVfYN43p0hUtIxey48LGKPjcgQ8IMLjs6y2qrG3wGPZBzEF0OslOrOXVX4erAWNlLdJiGxX7pkEmZ6dE2YNlY5ZCrxvzfmCxAkxBapgFpZfk8uwZ5kLR/gqIklWjZn4mLltTEv3943jY5y8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ziepe.ca; spf=pass smtp.mailfrom=ziepe.ca; dkim=pass (2048-bit key) header.d=ziepe.ca header.i=@ziepe.ca header.b=SH8jp2DI; arc=none smtp.client-ip=209.85.219.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ziepe.ca Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ziepe.ca Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=ziepe.ca header.i=@ziepe.ca header.b="SH8jp2DI" Received: by mail-qv1-f42.google.com with SMTP id 6a1803df08f44-87bb66dd224so1173456d6.3 for ; Wed, 05 Nov 2025 09:21:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ziepe.ca; s=google; t=1762363277; x=1762968077; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=2cq2mS+yWhZev0zm4YEm/j+lSP0FbcQSTW4gUm6tSNU=; b=SH8jp2DIufEX3vRN+bltek/lJi75b+p+z6AIN9nyUDWTVYCvATPDIaqoM3LESvI4FD 8nquf5IVH6r6cI33uAXnizUpTqbHvleYQT9kAAY4b0ZbCwQlwYzE3GQuW2Y6sut/vNOW o3iLPOiHZJM+c3dlQjap/e5ySWpcEqF+1Rfwrb/26DtOzWcSjuSoveArv3xqAA9c56eX Wf8nxoHDCzeCmeLzkzXWAQ7WhGjvqn8Wk7ZgL3TfZvDJRS8xIeJ2VQMyPMS2Q/xKQqPz hHDtgt+7wOO5Flg+YPcQTR+oqDeKyusFE0U4bnVdqJ4pi9l3rGcyGzEmTGvr1ZnLdMkg U/QQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762363277; x=1762968077; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=2cq2mS+yWhZev0zm4YEm/j+lSP0FbcQSTW4gUm6tSNU=; b=iMoFz6IIofZBfsVNl2+xM1vOY1VnyRvjWmD8zsyPIp0XPINoMAC/7M2pWIz0NCwOHw wLDyqljU1tzx+62zK+mDGcOk2fNiqJuc6Rq3vsUcoFiLpefJjW8rHBpSbK96DHOTYr/N eLFpd0AxbAyEHbZDZTqZvNrCgIfSGwkaQjEB+0PdgFstl3PV15fqD3lne6gxnudIqXpA pjJS9bcNcl0HcLQkHHZOwwHFZ+zbT5BvlWiq1Lrb2eU96PpdODkb8/AtAnhu+v7/pc68 G62BehBJymD2u6b61lRVVr6hiYCpfxWO7RklU/ibUb5eyT14tbnXqWxSeG+ePQR0fFQ2 V1wg== X-Forwarded-Encrypted: i=1; AJvYcCXSBKmsGJmU0edrPgj0Yij1f8mmr0nitrLBKa2VnRSxaxDtEaTdMAjp2f6/X3IpgxCBHNpKkKvUqp6X@vger.kernel.org X-Gm-Message-State: AOJu0YzP8yU40fACXGUfzmLTE+wOcszQOsCvjMluwVsGK0igLu+co0Gn 3h8HIiENhL/SL6o3caKeMyR9+LIDvyhW2gpY+L2tiMK7Ealb3wfV0g5RC+PKzhDxFRs= X-Gm-Gg: ASbGncsfu6bCcPEKJJLa/sQ+ykl+93Q2NO5L0sDgGtdxHZm5YQEsxM1kqyhOIwnP35c bzycb1IWnbnpe3BZ1tisr0wkZVmM+md5rtRKOxBQzrmYieScczIhP6xzvB+oZMebowX6/UuFZoN G8lXXmRUNqIem8uVy6jZnqquKu3Lbp7Jp+Lw7V76IR7yB41KE/MqS0cwMvD70NIEMNvuxGBe9Ym wDT3MsmpNS2B0GApfJKu/C+dk4uuSDRbTLkIRxoaa0oGslL7MJEL+u7cTyVx+zDbWgUcjeb5qbu RHUkw0WIZA8ruLLwB605gggz0HWODACJ8rgAqlyFkZDQm/XdRPKvspdFWog9xZ2Hl3v2MnPtDcD 06i7e5mHjSUCOnlcbfR/H4n0dDvpsgoyXq6+VLnDsgbH3CDJXJ/N9q4OUwD2si+hhm7o450n04C udkNC/ZSAW5HfmdwcijNVV57fdCk2b5hHGf7mYMrXjjF7Tbw== X-Google-Smtp-Source: AGHT+IGCK5Ht7P6eoqM7YgVZ6ef4jlofMTj5LlbxF9x0JfdkYp0y/H774e8UOGf7CmqXFow5BgQdlA== X-Received: by 2002:ad4:576e:0:b0:880:5779:d489 with SMTP id 6a1803df08f44-88071095623mr52524556d6.3.1762363276490; Wed, 05 Nov 2025 09:21:16 -0800 (PST) Received: from ziepe.ca (hlfxns017vw-47-55-120-4.dhcp-dynamic.fibreop.ns.bellaliant.net. [47.55.120.4]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-880828c457bsm996916d6.5.2025.11.05.09.21.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 05 Nov 2025 09:21:15 -0800 (PST) Received: from jgg by wakko with local (Exim 4.97) (envelope-from ) id 1vGhC7-00000007Bv7-1e20; Wed, 05 Nov 2025 13:21:15 -0400 Date: Wed, 5 Nov 2025 13:21:15 -0400 From: Jason Gunthorpe To: Gregory Price Cc: Lorenzo Stoakes , Andrew Morton , Christian Borntraeger , Janosch Frank , Claudio Imbrenda , David Hildenbrand , Alexander Gordeev , Gerald Schaefer , Heiko Carstens , Vasily Gorbik , Sven Schnelle , Peter Xu , Alexander Viro , Christian Brauner , Jan Kara , Arnd Bergmann , Zi Yan , Baolin Wang , "Liam R . Howlett" , Nico Pache , Ryan Roberts , Dev Jain , Barry Song , Lance Yang , Muchun Song , Oscar Salvador , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , Matthew Brost , Joshua Hahn , Rakie Kim , Byungchul Park , Ying Huang , Alistair Popple , Axel Rasmussen , Yuanchu Xie , Wei Xu , Kemeng Shi , Kairui Song , Nhat Pham , Baoquan He , Chris Li , SeongJae Park , Matthew Wilcox , Leon Romanovsky , Xu Xin , Chengming Zhou , Jann Horn , Miaohe Lin , Naoya Horiguchi , Pedro Falcato , Pasha Tatashin , Rik van Riel , Harry Yoo , Hugh Dickins , linux-kernel@vger.kernel.org, kvm@vger.kernel.org, linux-s390@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-arch@vger.kernel.org, damon@lists.linux.dev Subject: Re: [PATCH 02/16] mm: introduce leaf entry type and use to simplify leaf entry logic Message-ID: <20251105172115.GQ1204670@ziepe.ca> References: <2c75a316f1b91a502fad718de9b1bb151aafe717.1762171281.git.lorenzo.stoakes@oracle.com> Precedence: bulk X-Mailing-List: linux-arch@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Wed, Nov 05, 2025 at 09:42:24AM -0500, Gregory Price wrote: > On Mon, Nov 03, 2025 at 12:31:43PM +0000, Lorenzo Stoakes wrote: > > +typedef swp_entry_t leaf_entry_t; > > + > > +#ifdef CONFIG_MMU > > + > > +/* Temporary until swp_entry_t eliminated. */ > > +#define LEAF_TYPE_SHIFT SWP_TYPE_SHIFT > > + > > +enum leaf_entry_type { > > + /* Fundamental types. */ > > + LEAFENT_NONE, > > + LEAFENT_SWAP, > > + /* Migration types. */ > > + LEAFENT_MIGRATION_READ, > > + LEAFENT_MIGRATION_READ_EXCLUSIVE, > > + LEAFENT_MIGRATION_WRITE, > > + /* Device types. */ > > + LEAFENT_DEVICE_PRIVATE_READ, > > + LEAFENT_DEVICE_PRIVATE_WRITE, > > + LEAFENT_DEVICE_EXCLUSIVE, > > + /* H/W posion types. */ > > + LEAFENT_HWPOISON, > > + /* Marker types. */ > > + LEAFENT_MARKER, > > +}; > > + > > Have been browsing the patch set again, will get around a deeper review, > but just wanted to say this is a thing of beauty :] +1 I thought the same thing. So much clearer what is going on here, and I didn't realize we had so many types already.. Jason