mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jeff Layton <jlayton@kernel.org>
To: Jan Kara <jack@suse.cz>
Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
	Alexander Viro	 <viro@zeniv.linux.org.uk>,
	Christian Brauner <brauner@kernel.org>,
	Steven Rostedt <rostedt@goodmis.org>,
	Masami Hiramatsu <mhiramat@kernel.org>,
	"Theodore Y. Ts'o"	 <tytso@mit.edu>,
	linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 02/61] vfs: change i_ino from unsigned long to u64
Date: Thu, 26 Feb 2026 12:59:31 -0500	[thread overview]
Message-ID: <b8cbab6fecc6a2743538d139eb1b054965b74841.camel@kernel.org> (raw)
In-Reply-To: <v2gibkxuingitgmoxevqhh7fxsnceioos2xrhycwnilep3fcni@jkgct7onqolx>

On Thu, 2026-02-26 at 18:19 +0100, Jan Kara wrote:
> On Thu 26-02-26 11:45:16, Jeff Layton wrote:
> > On Thu, 2026-02-26 at 11:40 -0500, Mathieu Desnoyers wrote:
> > > On 2026-02-26 11:35, Jeff Layton wrote:
> > > > 
> > > > I'll let others chime in first, but I'm open to going back and doing it
> > > > that way if we don't want to live with the compiler warnings during a
> > > > bisect.
> > > 
> > > On 32-bit archs, I suspect it will do more than emit compiler warnings.
> > > Trying to boot a kernel in the middle of the series is likely to lead to
> > > interesting inode value printout results.
> > > 
> > 
> > Definitely. None of that would be trustworthy in the middle of the
> > series.
> 
> Yeah, I think defining kino_t in the beginning as unsigned long, convert
> everything (including special format string specifier) to it, then switch
> it to u64 and finally just 'sed-out' the format string specifier in the
> final patch shouldn't be harder than what you do here and should keep
> bisectability?
> 

Ok. I'll look at doing it in a way that doesn't break bisectability.

-- 
Jeff Layton <jlayton@kernel.org>

  reply	other threads:[~2026-02-26 17:59 UTC|newest]

Thread overview: 111+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-26 15:55 [PATCH 00/61] vfs: change inode->i_ino " Jeff Layton
2026-02-26 15:55 ` [PATCH 01/61] vfs: widen inode hash/lookup functions " Jeff Layton
2026-02-26 17:00   ` Jan Kara
2026-02-26 17:14     ` Jan Kara
2026-02-26 22:56   ` Damien Le Moal
2026-02-26 15:55 ` [PATCH 02/61] vfs: change i_ino from unsigned long " Jeff Layton
2026-02-26 16:16   ` Mathieu Desnoyers
2026-02-26 16:35     ` Jeff Layton
2026-02-26 16:40       ` Mathieu Desnoyers
2026-02-26 16:45         ` Jeff Layton
2026-02-26 16:48           ` Mathieu Desnoyers
2026-02-26 17:19           ` Jan Kara
2026-02-26 17:59             ` Jeff Layton [this message]
2026-02-26 18:12     ` David Laight
2026-02-26 15:55 ` [PATCH 03/61] trace: update VFS-layer trace events for u64 i_ino Jeff Layton
2026-02-26 17:11   ` Jan Kara
2026-02-26 17:53     ` Jeff Layton
2026-02-26 17:48   ` Steven Rostedt
2026-02-27 21:05     ` Jeff Layton
2026-02-26 22:51   ` Damien Le Moal
2026-02-26 15:55 ` [PATCH 04/61] ext4: update " Jeff Layton
2026-02-26 15:55 ` [PATCH 05/61] jbd2: update format strings " Jeff Layton
2026-02-26 15:55 ` [PATCH 06/61] f2fs: update " Jeff Layton
2026-02-26 15:55 ` [PATCH 07/61] lockd: update format strings " Jeff Layton
2026-02-26 15:55 ` [PATCH 08/61] nfs: update " Jeff Layton
2026-02-26 15:55 ` [PATCH 09/61] nfs: remove nfs_fattr_to_ino_t() and nfs_fileid_to_ino_t() Jeff Layton
2026-02-26 15:55 ` [PATCH 10/61] nfs: remove nfs_compat_user_ino64() Jeff Layton
2026-02-26 15:55 ` [PATCH 11/61] nfs: remove enable_ino64 module parameter Jeff Layton
2026-02-26 15:55 ` [PATCH 12/61] nfsd: update format strings for u64 i_ino Jeff Layton
2026-02-26 19:48   ` Chuck Lever
2026-02-26 20:06     ` Jeff Layton
2026-02-26 15:55 ` [PATCH 13/61] smb: store full 64-bit uniqueid in i_ino Jeff Layton
2026-02-26 16:57   ` Paulo Alcantara
2026-02-26 15:55 ` [PATCH 14/61] smb: remove cifs_uniqueid_to_ino_t() Jeff Layton
2026-02-26 16:58   ` Paulo Alcantara
2026-02-26 15:55 ` [PATCH 15/61] locks: update /proc/locks format for u64 i_ino Jeff Layton
2026-02-26 15:55 ` [PATCH 16/61] proc: update /proc/PID/maps " Jeff Layton
2026-02-26 15:55 ` [PATCH 17/61] nilfs2: update " Jeff Layton
2026-02-26 19:46   ` Viacheslav Dubeyko
2026-02-26 20:06     ` Jeff Layton
2026-02-26 20:09       ` Viacheslav Dubeyko
2026-02-26 15:55 ` [PATCH 18/61] 9p: update format strings " Jeff Layton
2026-02-26 15:55 ` [PATCH 19/61] affs: " Jeff Layton
2026-02-26 16:49   ` David Sterba
2026-02-26 15:55 ` [PATCH 20/61] afs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 21/61] autofs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 22/61] befs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 23/61] bfs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 24/61] cachefiles: " Jeff Layton
2026-02-26 16:08   ` Mathieu Desnoyers
2026-02-26 15:55 ` [PATCH 25/61] ceph: " Jeff Layton
2026-02-26 19:37   ` Viacheslav Dubeyko
2026-02-26 15:55 ` [PATCH 26/61] coda: " Jeff Layton
2026-02-26 15:55 ` [PATCH 27/61] cramfs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 28/61] ecryptfs: " Jeff Layton
2026-02-26 18:21   ` Tyler Hicks
2026-02-26 15:55 ` [PATCH 29/61] efs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 30/61] exportfs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 31/61] ext2: " Jeff Layton
2026-02-26 15:55 ` [PATCH 32/61] freevxfs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 33/61] hfs: " Jeff Layton
2026-02-26 19:33   ` Viacheslav Dubeyko
2026-02-26 15:55 ` [PATCH 34/61] hfsplus: " Jeff Layton
2026-02-26 19:35   ` Viacheslav Dubeyko
2026-02-26 15:55 ` [PATCH 35/61] hpfs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 36/61] isofs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 37/61] jffs2: " Jeff Layton
2026-02-26 21:06   ` Richard Weinberger
2026-02-26 15:55 ` [PATCH 38/61] jfs: " Jeff Layton
2026-02-26 16:30   ` Dave Kleikamp
2026-02-26 15:55 ` [PATCH 39/61] minix: " Jeff Layton
2026-02-26 15:55 ` [PATCH 40/61] nsfs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 41/61] ntfs3: " Jeff Layton
2026-02-26 15:55 ` [PATCH 42/61] ocfs2: " Jeff Layton
2026-02-26 15:55 ` [PATCH 43/61] orangefs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 44/61] overlayfs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 45/61] qnx4: " Jeff Layton
2026-02-26 15:55 ` [PATCH 46/61] qnx6: " Jeff Layton
2026-02-26 15:55 ` [PATCH 47/61] ubifs: " Jeff Layton
2026-02-26 21:06   ` Richard Weinberger
2026-02-27  8:16   ` Zhihao Cheng
2026-02-26 15:55 ` [PATCH 48/61] udf: " Jeff Layton
2026-02-26 15:55 ` [PATCH 49/61] ufs: " Jeff Layton
2026-02-26 15:55 ` [PATCH 50/61] zonefs: " Jeff Layton
2026-02-26 22:45   ` Damien Le Moal
2026-02-27 11:55   ` Johannes Thumshirn
2026-02-26 15:55 ` [PATCH 51/61] security: update audit " Jeff Layton
2026-02-26 21:11   ` Paul Moore
2026-02-27 16:46   ` Ryan Lee
2026-02-26 15:55 ` [PATCH 52/61] drm/amdgpu: update " Jeff Layton
2026-02-26 15:55 ` [PATCH 53/61] fsnotify: update fdinfo format strings " Jeff Layton
2026-02-26 15:55 ` [PATCH 54/61] net: update socket dname format " Jeff Layton
2026-02-26 15:55 ` [PATCH 55/61] uprobes: update format strings " Jeff Layton
2026-02-26 15:55 ` [PATCH 56/61] dma-buf: update format string " Jeff Layton
2026-02-26 15:55 ` [PATCH 57/61] fscrypt: update format strings " Jeff Layton
2026-02-26 17:10   ` Eric Biggers
2026-02-26 15:56 ` [PATCH 58/61] fsverity: update format string " Jeff Layton
2026-02-26 15:56 ` [PATCH 59/61] iomap: " Jeff Layton
2026-02-26 16:21   ` Darrick J. Wong
2026-02-26 15:56 ` [PATCH 60/61] net: update legacy protocol format strings " Jeff Layton
2026-02-26 15:56 ` [PATCH 61/61] vfs: update core " Jeff Layton
2026-02-26 16:11   ` Mathieu Desnoyers
2026-02-26 16:22   ` Darrick J. Wong
2026-02-26 16:49 ` [PATCH 00/61] vfs: change inode->i_ino from unsigned long to u64 Matthew Wilcox
2026-02-26 17:01   ` Jeff Layton
2026-02-27 17:19   ` Jeff Layton
2026-02-27 19:01     ` Mathieu Desnoyers
2026-02-27 19:35       ` Jeff Layton
2026-02-27  9:30 ` Christian König
2026-02-27 11:52   ` Jeff Layton
2026-02-27 10:06 ` Christian Brauner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b8cbab6fecc6a2743538d139eb1b054965b74841.camel@kernel.org \
    --to=jlayton@kernel.org \
    --cc=brauner@kernel.org \
    --cc=jack@suse.cz \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mathieu.desnoyers@efficios.com \
    --cc=mhiramat@kernel.org \
    --cc=rostedt@goodmis.org \
    --cc=tytso@mit.edu \
    --cc=viro@zeniv.linux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®