On Fri, 26 Jan 2024 13:36:20 -0800 Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote: > On Fri, 26 Jan 2024 at 13:26, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote: > > > > I'd be happy to change that patch to what I originally did before deciding > > to copy get_next_ino(): > > > > unsigned int tracefs_get_next_ino(int files) > > { > > static atomic_t next_inode; > > unsigned int res; > > > > do { > > res = atomic_add_return(files + 1, &next_inode); > > > > /* Check for overflow */ > > } while (unlikely(res < files + 1)); > > > > return res - files; > > Still entirely pointless. > > If you have more than 4 billion inodes, something is really really wrong. No, but you can trivially make a loop that creates and destroys directories that will eventually overflow the count. -- Steve