From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qk1-f174.google.com (mail-qk1-f174.google.com [209.85.222.174]) (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 F255D2D3733 for ; Mon, 27 Oct 2025 16:38:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761583134; cv=none; b=GlW8AA/dkVNAMNbdDQAGg/iVg+DZMyNP+l/kQhF8EI0U0c7M0MIN1aED2gml8cCLHX/va8adqlXnxHUGb1eUNP1J2uAXa5peNor7iKYtCuHM6CveTyeNzfUB/xJMhENsiqVxgcvFxLuZiWWPwF4spAtNly8h/tBpc0Qp05xWEBE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761583134; c=relaxed/simple; bh=nAhmUlr/hO6AGJs14F5PTgWNTU6mEzb9QsSufy3So/Q=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=ZLdFE/msjNfVMgOE1/ZESClD6Bm6KaHLXDnsSMj0Yo8XuDZPcuMlBb5Lb1ghgFt7E2m6gGE7qWA9H4k6Gq1GOsm1fCJUSlNP2wIJjoZC6pZuv4dRrqOez2CPUoX5of71VDONoVhlK6LLB8Lq+ZS2mqmMhMFunUydYbUr0fODjZg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=gourry.net; spf=pass smtp.mailfrom=gourry.net; dkim=pass (2048-bit key) header.d=gourry.net header.i=@gourry.net header.b=BSCrR+pG; arc=none smtp.client-ip=209.85.222.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=gourry.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gourry.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gourry.net header.i=@gourry.net header.b="BSCrR+pG" Received: by mail-qk1-f174.google.com with SMTP id af79cd13be357-8a074e50b41so276736885a.3 for ; Mon, 27 Oct 2025 09:38:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gourry.net; s=google; t=1761583132; x=1762187932; 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=f2HGwzd/4lzqDYF3XkN+FOghRP60HTqO9KJmTXVpNDU=; b=BSCrR+pG+Y7/saGJaQX20uzc1Qo1+qKSDQNhfe3Jq5mp2ZVIuA1AfXqrkKMasdFzmW 7Qx+hR79rqhMBQVyg3XVENRsHeiXAbOlGwuG8YjTI4x6f+K86y+AUpDV6/Dwg0/i7ua4 dtX3BNjxD8vykbbF2cTq0jAEukuu2uPSRi4AnTxPKyd/oeBEMaEjFTSYhZWoGLnpeRyS RilIGepkRXt83NIzS1lFthelm2vYj9pSCBbeq7bWvbUbmksj5tBICCrCnngGIpRRkGPd xDPc57+tG5kSN+8jOfsZ+gwtFsIILrsnYXvwMFI43BJ1Ux4gumpj4AIgWoy/kGuTDcHp lqGQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1761583132; x=1762187932; 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=f2HGwzd/4lzqDYF3XkN+FOghRP60HTqO9KJmTXVpNDU=; b=g63kNRtNbljjNFc5NqrGd4wiGHfH1UIQGSS6tEWavJJ+el+Dv27fTO2KLI6EnYOIcY nZ5ah3JWg5OIxUzV8zxmP+Dm4G5nHIFAxxxbdEnNV44qS4Sc1LWxcH91jKDeOiVS8NHA tuLXWo/olRLTTc4JguGKN+qYcr1hbE5HASMxrEzsAWdbI/5DIGd+J2Rxpv9Ltgf8vG8s rbpaYo5H/Qlt6QQ44jenMurb5nQT2AZXMNG1zbEpvtXcBCpotHP3iJHiGGGhHTM4/sH+ 1nrkShKJC3LZAkW3iogR135ConhMP1ad3fg7s8VgKe9rgHf3hjmctTPMYguWDKd4ShhB iGPg== X-Forwarded-Encrypted: i=1; AJvYcCXEQh/0de1AW3ptKbjxhZpEvNhiKnsxdJG2cmI3ZC+/cxgWwwV7EmK3qADoPIP0uhEu5N7mcGw0lgoM@vger.kernel.org X-Gm-Message-State: AOJu0YyX55ItY3hk59bOd2dhqN7PRjEQbcDQdZIQBcZi19cS7VFzDw3t E3cswiRz2tAyX16WF/LgxykCdXhdjGL67zYSVQnRwIyJ/KXltNBtBst3lA7KhSfNe84= X-Gm-Gg: ASbGnctS/D2f9n6P+Z91guNE4hMWXPgI0vYsuNUzceyXtfnEdKCF8JcGJeTC93qfWiI T3Aq9KV740mSnZUxbPLgknDmGPu5+f6PEPog6UAkalf24aV9CfAoq/y2UN1Rf5GGmf8Wd4BnMfL 8xAKwsodmymbc5rfanhnAlWXSZvfgkaDGOYwEKspXBDDLx3jGDIpFEWpNDgTYostG+mX5507qAi W+WnOLM+hf0yKEy7e37Wi8wkV7LvltGpThwYIw5IFGB2fJ+M4mQ6kQWLwsK9waUTyb018QyG1Bs IGrSpsJPkcjgzvNm2Ev/Wu5AHqxNaWrW+hJTNIo9YloMBMp+Xjo7E1D8gRq6WHkOxyUQGW7hpOP InKyYgToyK1roLF+1E2ialOBeRSsBrZj69NPS2KIy7MuXMxg3vzQPbdAZXTwqGOIuYwTgJ/BuRe 1xzqKAd4fjhU/AA+wOa7Ag7LYMpme0ILmZm8HAGMjp/O+ZSDaJNVHP77GEhP5eD+K3G1CtXQ== X-Google-Smtp-Source: AGHT+IE8OFT35MTx0jxBdZigqHGpIP25NqCfl0+l6c8fCefLdIH0hfSup7UdNhOGnYzqPJc4qxyLmA== X-Received: by 2002:a05:620a:711c:b0:892:636a:5675 with SMTP id af79cd13be357-8a6f704a25amr58519385a.58.1761583131221; Mon, 27 Oct 2025 09:38:51 -0700 (PDT) Received: from gourry-fedora-PF4VCD3F (pool-96-255-20-138.washdc.ftas.verizon.net. [96.255.20.138]) by smtp.gmail.com with ESMTPSA id af79cd13be357-89f25798afesm626595985a.37.2025.10.27.09.38.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 27 Oct 2025 09:38:50 -0700 (PDT) Date: Mon, 27 Oct 2025 12:38:48 -0400 From: Gregory Price To: Lorenzo Stoakes Cc: Jason Gunthorpe , Andrew Morton , Christian Borntraeger , Janosch Frank , Claudio Imbrenda , David Hildenbrand , Alexander Gordeev , Gerald Schaefer , Heiko Carstens , Vasily Gorbik , Sven Schnelle , Zi Yan , Baolin Wang , "Liam R . Howlett" , Nico Pache , Ryan Roberts , Dev Jain , Barry Song , Lance Yang , Kemeng Shi , Kairui Song , Nhat Pham , Baoquan He , Chris Li , Peter Xu , Matthew Wilcox , Leon Romanovsky , Muchun Song , Oscar Salvador , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , Jann Horn , Matthew Brost , Joshua Hahn , Rakie Kim , Byungchul Park , Ying Huang , Alistair Popple , Pedro Falcato , Pasha Tatashin , Rik van Riel , Harry Yoo , kvm@vger.kernel.org, linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [RFC PATCH 05/12] fs/proc/task_mmu: refactor pagemap_pmd_range() Message-ID: References: <2ce1da8c64bf2f831938d711b047b2eba0fa9f32.1761288179.git.lorenzo.stoakes@oracle.com> <76348b1f-2626-4010-8269-edd74a936982@lucifer.local> <3f3e5582-d707-41d0-99a7-4e9c25f1224d@lucifer.local> <20251027161146.GG760669@ziepe.ca> <27a5ea4e-155c-40d1-87d7-e27e98b4871d@lucifer.local> 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: <27a5ea4e-155c-40d1-87d7-e27e98b4871d@lucifer.local> On Mon, Oct 27, 2025 at 04:26:54PM +0000, Lorenzo Stoakes wrote: > On Mon, Oct 27, 2025 at 01:11:46PM -0300, Jason Gunthorpe wrote: > > On Fri, Oct 24, 2025 at 04:37:18PM -0400, Gregory Price wrote: > > > On Fri, Oct 24, 2025 at 09:15:59PM +0100, Lorenzo Stoakes wrote: > > > > On Fri, Oct 24, 2025 at 03:12:08PM -0400, Gregory Price wrote: > > > > > > > > So maybe actually that isn't too bad of an idea... > > > > > > > > Could also be > > > > > > > > nonpresent_or_swap_t but that's kinda icky... > > > > > > clearly we need: > > > > > > union { > > > swp_entry_t swap; > > > nonpresent_entry_t np; > > > pony_entry_t pony; > > > plum_emtry_t beer; > > > } leaf_entry_t; > > I think Greg meant this as a joke [correct me if wrong] :) that was my > impression anyway (see original end of email...) > > I like leaf_entry_t name-wise. > > I don't love the union. > The union was definitely a joke - see `plum_entry_t beer` There definitely shouldn't be enough extensions to warrant a union here, that seems like negative value. leaf_entry_t naming replacing swp_entry_t seems reasonable since that's basically all swp_entry_t is in its current form - even according to the this set's cover letter: ``` There's an established convention in the kernel that we treat leaf page tables (so far at the PTE, PMD level) as containing 'swap entries' should they be neither empty (i.e. p**_none() evaluating true) nor present (i.e. p**_present() evaluating true). ``` ~Gregory