Hi,

I love your patch! Yet something to improve:

[auto build test ERROR on nfs/linux-next]
[also build test ERROR on v5.8 next-20200806]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/trondmy-kernel-org/NFS-Report-the-stateid-status-in-trace_nfs4_layoutreturn_on_close/20200806-040955
base:   git://git.linux-nfs.org/projects/trondmy/linux-nfs.git linux-next
config: microblaze-randconfig-r033-20200805 (attached as .config)
compiler: microblaze-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=microblaze 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from include/trace/define_trace.h:102,
                    from fs/nfs/nfs4trace.h:2305,
                    from fs/nfs/nfs4trace.c:13:
   fs/nfs/./nfs4trace.h: In function 'trace_event_raw_event_nfs4_read_event':
>> fs/nfs/./nfs4trace.h:1775:25: error: dereferencing pointer to incomplete type 'const struct pnfs_layout_hdr'
    1775 |     nfs_stateid_hash(&lo->plh_stateid);
         |                         ^~
   include/trace/trace_events.h:707:4: note: in definition of macro 'DECLARE_EVENT_CLASS'
     707 |  { assign; }       \
         |    ^~~~~~
   fs/nfs/./nfs4trace.h:1752:3: note: in expansion of macro 'TP_fast_assign'
    1752 |   TP_fast_assign(
         |   ^~~~~~~~~~~~~~

vim +1775 fs/nfs/./nfs4trace.h

  1729	
  1730	DECLARE_EVENT_CLASS(nfs4_read_event,
  1731			TP_PROTO(
  1732				const struct nfs_pgio_header *hdr,
  1733				int error
  1734			),
  1735	
  1736			TP_ARGS(hdr, error),
  1737	
  1738			TP_STRUCT__entry(
  1739				__field(dev_t, dev)
  1740				__field(u32, fhandle)
  1741				__field(u64, fileid)
  1742				__field(loff_t, offset)
  1743				__field(u32, arg_count)
  1744				__field(u32, res_count)
  1745				__field(unsigned long, error)
  1746				__field(int, stateid_seq)
  1747				__field(u32, stateid_hash)
  1748				__field(int, layoutstateid_seq)
  1749				__field(u32, layoutstateid_hash)
  1750			),
  1751	
  1752			TP_fast_assign(
  1753				const struct inode *inode = hdr->inode;
  1754				const struct nfs_inode *nfsi = NFS_I(inode);
  1755				const struct nfs_fh *fh = hdr->args.fh ?
  1756							  hdr->args.fh : &nfsi->fh;
  1757				const struct nfs4_state *state =
  1758					hdr->args.context->state;
  1759				const struct pnfs_layout_segment *lseg = hdr->lseg;
  1760				const struct pnfs_layout_hdr *lo = lseg->pls_layout;
  1761	
  1762				__entry->dev = inode->i_sb->s_dev;
  1763				__entry->fileid = nfsi->fileid;
  1764				__entry->fhandle = nfs_fhandle_hash(fh);
  1765				__entry->offset = hdr->args.offset;
  1766				__entry->arg_count = hdr->args.count;
  1767				__entry->res_count = hdr->res.count;
  1768				__entry->error = error < 0 ? -error : 0;
  1769				__entry->stateid_seq =
  1770					be32_to_cpu(state->stateid.seqid);
  1771				__entry->stateid_hash =
  1772					nfs_stateid_hash(&state->stateid);
  1773				__entry->layoutstateid_seq = lseg->pls_seq;
  1774				__entry->layoutstateid_hash =
> 1775					nfs_stateid_hash(&lo->plh_stateid);
  1776			),
  1777	
  1778			TP_printk(
  1779				"error=%ld (%s) fileid=%02x:%02x:%llu fhandle=0x%08x "
  1780				"offset=%lld count=%u res=%u stateid=%d:0x%08x "
  1781				"layoutstateid=%d:0x%08x",
  1782				-__entry->error,
  1783				show_nfsv4_errors(__entry->error),
  1784				MAJOR(__entry->dev), MINOR(__entry->dev),
  1785				(unsigned long long)__entry->fileid,
  1786				__entry->fhandle,
  1787				(long long)__entry->offset,
  1788				__entry->arg_count, __entry->res_count,
  1789				__entry->stateid_seq, __entry->stateid_hash,
  1790				__entry->layoutstateid_seq, __entry->layoutstateid_hash
  1791			)
  1792	);
  1793	#define DEFINE_NFS4_READ_EVENT(name) \
  1794		DEFINE_EVENT(nfs4_read_event, name, \
  1795				TP_PROTO( \
  1796					const struct nfs_pgio_header *hdr, \
  1797					int error \
  1798				), \
  1799				TP_ARGS(hdr, error))
  1800	DEFINE_NFS4_READ_EVENT(nfs4_read);
  1801	#ifdef CONFIG_NFS_V4_1
  1802	DEFINE_NFS4_READ_EVENT(nfs4_pnfs_read);
  1803	#endif /* CONFIG_NFS_V4_1 */
  1804	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org