[PATCH 1/7] fs: remove trailing whitespace from include/linux/fs.h

[Jan Kara]

On Wed 09-09-26 17:01:06, Julian Sun wrote:
> Remove trailing whitespace before introducing the inode iterator API.
> This avoids unrelated whitespace changes from automatic editor cleanup
> in the following patches. This also fixes formatting issues in fs.h
> reported by scripts/checkpatch.pl.
> 
> Signed-off-by: Julian Sun <sunjunchao at bytedance.com>

We generally try to avoid whitespace only or style only changes as they
hurt backportability mostly unnecessarily. But this patch looks harmless
enough so feel free to add:

Reviewed-by: Jan Kara <jack at suse.cz>

								Honza

> ---
>  include/linux/fs.h | 14 ++++++++------
>  1 file changed, 8 insertions(+), 6 deletions(-)
> 
> diff --git a/include/linux/fs.h b/include/linux/fs.h
> index f9d1e05e8ae6..09c4db5e9ae0 100644
> --- a/include/linux/fs.h
> +++ b/include/linux/fs.h
> @@ -296,7 +296,7 @@ struct iattr {
>   */
>  #define FILESYSTEM_MAX_STACK_DEPTH 2
>  
> -/** 
> +/**
>   * enum positive_aop_returns - aop return codes with specific semantics
>   *
>   * @AOP_WRITEPAGE_ACTIVATE: Informs the caller that page writeback has
> @@ -306,7 +306,7 @@ struct iattr {
>   * 			    be a candidate for writeback again in the near
>   * 			    future.  Other callers must be careful to unlock
>   * 			    the page if they get this return.  Returned by
> - * 			    writepage(); 
> + *			    writepage();
>   *
>   * @AOP_TRUNCATED_PAGE: The AOP method that was handed a locked page has
>   *  			unlocked it and the page might have been truncated.
> @@ -1315,8 +1315,10 @@ struct file *get_file_active(struct file **f);
>  
>  #define	MAX_NON_LFS	((1UL<<31) - 1)
>  
> -/* Page cache limit. The filesystems should put that into their s_maxbytes 
> -   limits, otherwise bad things can happen in VM. */ 
> +/*
> + * Page cache limit. The filesystems should put that into their s_maxbytes
> + * limits, otherwise bad things can happen in VM.
> + */
>  #if BITS_PER_LONG==32
>  #define MAX_LFS_FILESIZE	((loff_t)ULONG_MAX << PAGE_SHIFT)
>  #elif BITS_PER_LONG==64
> @@ -2285,7 +2287,7 @@ int sync_inode_metadata(struct inode *inode, int wait);
>  struct file_system_type {
>  	const char *name;
>  	int fs_flags;
> -#define FS_REQUIRES_DEV		1 
> +#define FS_REQUIRES_DEV		1
>  #define FS_BINARY_MOUNTDATA	2
>  #define FS_HAS_SUBTYPE		4
>  #define FS_USERNS_MOUNT		8	/* Can be mounted by userns root */
> @@ -2907,7 +2909,7 @@ ssize_t __kernel_read(struct file *file, void *buf, size_t count, loff_t *pos);
>  extern ssize_t kernel_write(struct file *, const void *, size_t, loff_t *);
>  extern ssize_t __kernel_write(struct file *, const void *, size_t, loff_t *);
>  extern struct file * open_exec(const char *);
> - 
> +
>  /* fs/dcache.c -- generic fs support functions */
>  extern bool is_subdir(struct dentry *, struct dentry *);
>  extern bool path_is_under(const struct path *, const struct path *);
> -- 
> 2.39.5
> 
-- 
Jan Kara <jack at suse.com>
SUSE Labs, CR