From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qv1-f41.google.com (mail-qv1-f41.google.com [209.85.219.41]) (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 C6CA6320CA3 for ; Wed, 5 Nov 2025 17:21:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762363279; cv=none; b=ENEQfHWJsLIhis959gmwWzpLsmiCj1FXBYObIqnYf0J0XYvH7uXsmmFMW5h8duzw8hIdMQuSLxgJOG/0//I3btqgEIzqrWi1fBs0g6gSpFD7VDTHByZzCUkv2AsopWPlwc0mJXbnArcF+RcCfvNq9z/nRcYvSnOLog6bN43nBnM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762363279; 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=dn0nKN0yTvwbVZqco0P35lnzOigeaA1tZpcq8sGD5cP1QJNJW3m889pBNgn394rbuUCpEXY3DcGwig5SU0OPpNMLdtNoK+k32SjJtkpF3zFqiuwg4HY8nHMGEQHI0/7bOI+U2jkZ/Cm2D+BAnmk8PAuemOVgbfA+0FTdrrXqWLY= 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.41 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-f41.google.com with SMTP id 6a1803df08f44-87bb66dd224so1173446d6.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=ma/L8PDepJ1vO/gpdYEMLwuCx2oXpbvaqVyMSgVgcrR7U3oGSh5k3etJIx2CXvNooG 6Kd3Jn7v0HF2l7/FTupomCOgdoZIx9YGGVdVN+Mrw48vhj0jIWm8fAnl2lmCctt0qDg1 /K6m7bU8N0ubXrFQN4/LnMBZ+9m0u+iuZXdNqOMt2Nh7zPAm2ueU92kOAboQAIHfBjAH t2xzXBlcSHrjU4IjSEv8eRS30MZ/A/D1QKJEvPBYPcs7iDpeAQHRG/VjVDI25Oa2IHSg J3nsJwJz7s0r+AZIfi7Ph8cULkKoaF32eyIDPT+U3U24+7LsaJZaH+bHbOlDjTlDHgYd dzYw== X-Forwarded-Encrypted: i=1; AJvYcCVsQF0wVXSDOdl1zCXfPlTRMfITs0FCDcQEtjJKQ1zQ+1lpyCCOGqqlUFc7jewhQ8pzUAqGrJzL6g9h@vger.kernel.org X-Gm-Message-State: AOJu0YxJk4x2Bo/isGJFPNqJSy31lTI3gMFhHjfOOgBAsZ4VkLtjEFTm ILEj7nYc3Ftf9BbYDcrJtgUXGPwwe/42XAb5Rsa/VTwzcBG4siw1Na6c19cCQHeZXQQ= X-Gm-Gg: ASbGncviM6cvZJwIQ8GKBn0qJlmESqK2cZEIrcAjK7T9LqIKtLUM9dC7wMUbaoHP4Ei 1KQJMyVe7aPdKDOLN4QPV+pdgCOpFkrLbqGc0BJ/VjZEAlBcvMQCdPZU4D1DCUUKbXZ0O+kDbzA ih1E3UfLtDXrK3JsVnIHmwM/CvLdc27x3e5/OESpWkSbVO4sQEbHP9yJVDS+UrvAwfjhcGNoOz2 Fx/KD9hLmKg48IQQb7YI9tMgGQspSxT+6dvOjS/7lY1cpuxN3PfixiaTUvjN7Tv9puwfaZK5wvp leYWtcnaq+cFGeTwzTi34toX7+Rhmhz+i/AXNjCfS3fvjNUIEwM8hJeu43IKu9ja1N8Q0M3OmeS PwznJM1Sg6IK1/0bfTvetNGBA3oCma9Nd6gY/UhwuHZ0fWpejBN4Qf7+z2nGtVtoXJbhTxjsX8q ohm/pPXdsP4bZIUChgRQK+qL5zvp0Vpb70KlJJ3bX+LLVmuw== 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-s390@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