Commit 88fd4515 authored by Steven Rostedt's avatar Steven Rostedt Committed by Carlos Maiolino
Browse files

xfs: remove unused event xfs_pagecache_inval 



When the function xfs_flushinval_pages() was removed, it removed the only
caller to the trace event xfs_pagecache_inval. As trace events can take up
to 5K of memory in text and meta data each regardless if they are used or
not, they should not be created when unused. Remove the unused event.

Fixes: fb595814 ("xfs: remove xfs_flushinval_pages")
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Signed-off-by: default avatarSteven Rostedt (Google) <rostedt@goodmis.org>
Signed-off-by: default avatarCarlos Maiolino <cem@kernel.org>
parent f1100605

fs/xfs/xfs_trace.h

+0 −25
Original line number Diff line number Diff line
@@ -1896,31 +1896,6 @@ DEFINE_EVENT(xfs_itrunc_class, name, \ 
DEFINE_ITRUNC_EVENT(xfs_itruncate_extents_start);

DEFINE_ITRUNC_EVENT(xfs_itruncate_extents_end);



TRACE_EVENT(xfs_pagecache_inval,

	TP_PROTO(struct xfs_inode *ip, xfs_off_t start, xfs_off_t finish),

	TP_ARGS(ip, start, finish),

	TP_STRUCT__entry(

		__field(dev_t, dev)

		__field(xfs_ino_t, ino)

		__field(xfs_fsize_t, size)

		__field(xfs_off_t, start)

		__field(xfs_off_t, finish)

	),

	TP_fast_assign(

		__entry->dev = VFS_I(ip)->i_sb->s_dev;

		__entry->ino = ip->i_ino;

		__entry->size = ip->i_disk_size;

		__entry->start = start;

		__entry->finish = finish;

	),

	TP_printk("dev %d:%d ino 0x%llx disize 0x%llx start 0x%llx finish 0x%llx",

		  MAJOR(__entry->dev), MINOR(__entry->dev),

		  __entry->ino,

		  __entry->size,

		  __entry->start,

		  __entry->finish)

);



TRACE_EVENT(xfs_bunmap,

	TP_PROTO(struct xfs_inode *ip, xfs_fileoff_t fileoff, xfs_filblks_t len,

		 int flags, unsigned long caller_ip),