Re: Unused event xfs_growfs_check_rtgeom

[Steven Rostedt <rostedt@goodmis.org>]

On Thu, 12 Jun 2025 22:59:47 -0700
Christoph Hellwig <hch@infradead.org> wrote:

> On Thu, Jun 12, 2025 at 02:46:08PM -0400, Steven Rostedt wrote:
> > On Thu, 12 Jun 2025 10:47:37 -0700
> > "Darrick J. Wong" <djwong@kernel.org> wrote:
> >   
> > > On Thu, Jun 12, 2025 at 01:16:51PM -0400, Steven Rostedt wrote:  
> > > > I also found events: xfs_metadir_link and xfs_metadir_start_link are
> > > > defined in fs/xfs/libxfs/xfs_metadir.c in a #ifndef __KERNEL__ section.
> > > > 
> > > > Are these events ever used? Why are they called in !__KERNEL__ code?    
> > > 
> > > libxfs is shared with userspace, and xfs_repair uses them to relink old
> > > quota files.
> > >  
> > 
> > Does this userspace use these trace events? If so, I think the events
> > need to have an:  
> 
> They have stubs for them.

If user space only has stubs for them, then why do they exist? The call
to this tracepoints are within #ifndef __KERNEL__ so the kernel will
never call them. Or can the user space stubs be replaced by actual content?

Either case, the events in the header file should be hidden from the kernel.

-- Steve