From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.223.130]) (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 33F0943CE77 for ; Fri, 24 Jul 2026 15:32:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=195.135.223.130 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784907172; cv=none; b=SHlzwvVbrcDbiucI+R7VD7VyNkfff6dfBaa8aUuu7hqOECar4scsCRrz/pyUVQ7QyXGm9n4zxtWIgNpGIbxVzpysxOzA36fPU6SCCORG9zv1kSF5nOlrBaPaNfnyL2lkvzoEh0pZKkiLhx2icO5UFxC+aFSjZyiYMTe+2F87HZQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784907172; c=relaxed/simple; bh=0ek21hUL3RIx2a+fDQWmsN+YSvNtt0UmwbcsoNF7Lbk=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=QwXosx4mERw9VicPzRSrQIgRwNGXLyZ3nD9TFGb+P4dj5Nr0Rw00Q8m6Q+Ejhz/8hnfMY7k/EhijnOaGQ1NN61exsqGdRIXVXVO9sWHuWhseG3GR8jRIl1vYN6Kr+lG7L6s7hvqCGcxGBvzpBfxBTE1iz6EWvBxx7y2G6bSG4HY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de; spf=pass smtp.mailfrom=suse.de; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=2LOVYft9; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=ngn5ZFPe; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=2LOVYft9; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=ngn5ZFPe; arc=none smtp.client-ip=195.135.223.130 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="2LOVYft9"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="ngn5ZFPe"; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="2LOVYft9"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="ngn5ZFPe" Received: from imap1.dmz-prg2.suse.org (unknown [10.150.64.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 401727BA2C; Fri, 24 Jul 2026 15:32:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1784907168; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=MgLR94cnov6/P8Yl4U/7T81H0kr3VMn5Pq5dQ8bzHh8=; b=2LOVYft95wMJIyhJa6PZSbwOiNmbFFPbcqUTuJ8eG3N/5dEfFfS1xnrE15U2j40DHgjDmq 6SVMdKicnUTzPnTNepl6LS43ziFb719IQZ/YpmgYwEQMWN5wsTERRtQjRxILqRY7bav/0w AiPtTwFDpTdCdCi2BNy9lyN9DKbBWuU= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1784907168; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=MgLR94cnov6/P8Yl4U/7T81H0kr3VMn5Pq5dQ8bzHh8=; b=ngn5ZFPemuxkfH460JBtHk6gBgwLnC/4m+F/Wm5dLwshckWwMM8hRMhkcYUYLvLbPxDwRh a99cvpioZ+OPOGDQ== Authentication-Results: smtp-out1.suse.de; none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1784907168; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=MgLR94cnov6/P8Yl4U/7T81H0kr3VMn5Pq5dQ8bzHh8=; b=2LOVYft95wMJIyhJa6PZSbwOiNmbFFPbcqUTuJ8eG3N/5dEfFfS1xnrE15U2j40DHgjDmq 6SVMdKicnUTzPnTNepl6LS43ziFb719IQZ/YpmgYwEQMWN5wsTERRtQjRxILqRY7bav/0w AiPtTwFDpTdCdCi2BNy9lyN9DKbBWuU= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1784907168; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=MgLR94cnov6/P8Yl4U/7T81H0kr3VMn5Pq5dQ8bzHh8=; b=ngn5ZFPemuxkfH460JBtHk6gBgwLnC/4m+F/Wm5dLwshckWwMM8hRMhkcYUYLvLbPxDwRh a99cvpioZ+OPOGDQ== Received: from imap1.dmz-prg2.suse.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by imap1.dmz-prg2.suse.org (Postfix) with ESMTPS id C990E779BA; Fri, 24 Jul 2026 15:32:45 +0000 (UTC) Received: from dovecot-director2.suse.de ([2a07:de40:b281:106:10:150:64:167]) by imap1.dmz-prg2.suse.org with ESMTPSA id KCyrLZ2FY2ozGgAAD6G6ig (envelope-from ); Fri, 24 Jul 2026 15:32:45 +0000 Date: Fri, 24 Jul 2026 16:32:43 +0100 From: Pedro Falcato To: Usama Arif Cc: Matthew Wilcox , Andrew Morton , David Hildenbrand , Catalin Marinas , Will Deacon , "James E.J. Bottomley" , Helge Deller , Madhavan Srinivasan , Michael Ellerman , Lorenzo Stoakes , "Liam R. Howlett" , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , Jan Kara , Zi Yan , Baolin Wang , Nico Pache , Ryan Roberts , Dev Jain , Barry Song , Lance Yang , Kevin Brodsky , Muhammad Usama Anjum , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-parisc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH 3/5] mm/parisc: constify ptep_get() argument Message-ID: References: <20260724102017.141532-1-pfalcato@suse.de> <20260724102017.141532-4-pfalcato@suse.de> <0fd713a8-e4ab-4c07-b36e-79dd138b4a86@linux.dev> <253f7ac0-205b-42f1-9fb5-d19eceb581b6@linux.dev> Precedence: bulk X-Mailing-List: linux-parisc@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: <253f7ac0-205b-42f1-9fb5-d19eceb581b6@linux.dev> X-Spam-Score: -6.30 X-Spam-Level: X-Spam-Flag: NO X-Spamd-Result: default: False [-6.30 / 50.00]; REPLY(-4.00)[]; BAYES_HAM(-3.00)[100.00%]; SUSPICIOUS_RECIPS(1.50)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; MID_RHS_NOT_FQDN(0.50)[]; NEURAL_HAM_SHORT(-0.20)[-1.000]; MIME_GOOD(-0.10)[text/plain]; MISSING_XM_UA(0.00)[]; RCVD_VIA_SMTP_AUTH(0.00)[]; MIME_TRACE(0.00)[0:+]; ARC_NA(0.00)[]; RCPT_COUNT_TWELVE(0.00)[32]; TO_DN_SOME(0.00)[]; FUZZY_RATELIMITED(0.00)[rspamd.com]; FREEMAIL_ENVRCPT(0.00)[gmx.de]; R_RATELIMIT(0.00)[to_ip_from(RLxu57a9hfgn7tttf5jiwuqe5o)]; FROM_HAS_DN(0.00)[]; FREEMAIL_CC(0.00)[infradead.org,linux-foundation.org,kernel.org,arm.com,hansenpartnership.com,gmx.de,linux.ibm.com,ellerman.id.au,google.com,suse.com,suse.cz,nvidia.com,linux.alibaba.com,redhat.com,linux.dev,lists.infradead.org,vger.kernel.org,lists.ozlabs.org,kvack.org]; RCVD_TLS_ALL(0.00)[]; FROM_EQ_ENVFROM(0.00)[]; RCVD_COUNT_TWO(0.00)[2]; TO_MATCH_ENVRCPT_ALL(0.00)[]; DKIM_SIGNED(0.00)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; DBL_BLOCKED_OPENRESOLVER(0.00)[suse.de:email,imap1.dmz-prg2.suse.org:helo] On Fri, Jul 24, 2026 at 04:03:44PM +0100, Usama Arif wrote: > > > On 24/07/2026 15:14, Matthew Wilcox wrote: > > On Fri, Jul 24, 2026 at 02:36:59PM +0100, Usama Arif wrote: > >> > >> > >> On 24/07/2026 11:20, Pedro Falcato wrote: > >>> ptep_get() does not need write access to the PTE. > >>> > >>> Signed-off-by: Pedro Falcato > >>> --- > >>> arch/parisc/include/asm/pgtable.h | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >> hmm I think you might break build bisectibility if > >> you separate out the patches, you should squash patch 3 and 4, > >> with patch 1. > > > > I don't see how this breaks bisectability. Can you elaborate on what > > you think would break? > > Patch 1 does: > > -static inline pte_t ptep_get_lockless(pte_t *ptep) > +static inline pte_t ptep_get_lockless(const pte_t *ptep) > { > return ptep_get(ptep); > } > > > ptep_get() takes a non-const arg till patch 3 for example for parsic. > I don't think you can pass a const variable to non const function arg? > > So parsic won't compile in patch 1 and 2, powerprc wont compile > for patches 1, 2 and 3.. Aha, yes, nice catch! I'll have to rethink that. Maybe squashing the patches would be the cleanest way forward. FWIW, the vast majority of architectures aren't doing funny things on ptep_get(). E.g PA-RISC, loongarch only define these so they can use it in their own asm/pgtable.h. I'd really like to delete these but I can't tell if they are *actually* required. With those gone only PPC 8xx and generic would need to be squashed (arm64 also defines its own version of ptep_get_lockless()). -- Pedro