Re: [PATCH] tracing: silence noisy warnings about struct inode

From: Steven Rostedt
Date: Sat Aug 31 2019 - 11:46:38 EST


On Thu, 29 Aug 2019 14:34:18 -0700
Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote:

> From: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
>
> Fix 30 warnings for missing "struct inode" declaration (like these) by
> adding a forward reference for it.
> These warnings come from 'headers_check' (CONFIG_HEADERS_CHECK):
> CC include/trace/events/iomap.h.s
>
> ./../include/trace/events/iomap.h:49:18: warning: 'struct inode' declared inside parameter list will not be visible outside of this definition or declaration
> ./../include/trace/events/iomap.h:77:18: warning: 'struct inode' declared inside parameter list will not be visible outside of this definition or declaration
>
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Cc: Steven Rostedt <rostedt@xxxxxxxxxxx>

Thanks!

-- Steve

> ---
> include/trace/events/iomap.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> --- linux-next-20190829.orig/include/trace/events/iomap.h
> +++ linux-next-20190829/include/trace/events/iomap.h
> @@ -44,6 +44,8 @@ DECLARE_EVENT_CLASS(iomap_page_class,
> __entry->length)
> )
>
> +struct inode;
> +
> #define DEFINE_PAGE_EVENT(name) \
> DEFINE_EVENT(iomap_page_class, name, \
> TP_PROTO(struct inode *inode, struct page *page, unsigned long off, \