On Mon, Jul 31, 2023 at 02:28:49PM +0200, Christian Brauner wrote: > On Mon, Jul 31, 2023 at 01:18:55PM +0100, Matthew Wilcox wrote: > > On Mon, Jul 31, 2023 at 06:37:10AM -0400, Jeff Layton wrote: > > > /** > > > * generic_fillattr - Fill in the basic attributes from the inode struct > > > - * @idmap: idmap of the mount the inode was found from > > > - * @req_mask statx request_mask > > > - * @inode: Inode to use as the source > > > - * @stat: Where to fill in the attributes > > > + * @idmap: idmap of the mount the inode was found from > > > + * @request_mask statx request_mask > > > > Missing the colon after request_mask. > > Fixed in-tree, thanks! FWIW, a W=1 build will catch this: ../mm/filemap.c:254: warning: Function parameter or member 'folio' not described in 'filemap_remove_folio' (after having deliberately removed the colon on that line)
