From: Kari Argillander <kari.argillander@gmail.com>
To: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
Cc: ntfs3@lists.linux.dev, linux-kernel@vger.kernel.org,
linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH 2/2] fs/ntfs3: Remove unnecessary includes
Date: Sun, 3 Oct 2021 20:42:44 +0300 [thread overview]
Message-ID: <20211003174244.ijg3uxoqdt2ywyqe@kari-VirtualBox> (raw)
In-Reply-To: <43e50860-3708-2887-86f7-e201782a2001@paragon-software.com>
On Fri, Oct 01, 2021 at 07:02:51PM +0300, Konstantin Komarov wrote:
> All removed includes already included from other headers.
Nack from me. Usually in kernel both header and source file should make
they own includes even when they both include same header file.
> Signed-off-by: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
> ---
> fs/ntfs3/attrib.c | 2 --
> fs/ntfs3/attrlist.c | 2 --
> fs/ntfs3/dir.c | 2 --
> fs/ntfs3/file.c | 2 --
> fs/ntfs3/frecord.c | 2 --
> fs/ntfs3/fslog.c | 2 --
> fs/ntfs3/fsntfs.c | 2 --
> fs/ntfs3/index.c | 3 ---
> fs/ntfs3/inode.c | 2 --
> fs/ntfs3/lznt.c | 1 -
> fs/ntfs3/namei.c | 2 --
> fs/ntfs3/ntfs_fs.h | 1 -
> fs/ntfs3/record.c | 2 --
> fs/ntfs3/run.c | 2 --
> fs/ntfs3/super.c | 4 +---
> fs/ntfs3/xattr.c | 2 --
> 16 files changed, 1 insertion(+), 32 deletions(-)
>
> diff --git a/fs/ntfs3/attrib.c b/fs/ntfs3/attrib.c
> index 8a00fa978f5f..dd4f1613081d 100644
> --- a/fs/ntfs3/attrib.c
> +++ b/fs/ntfs3/attrib.c
> @@ -10,8 +10,6 @@
> #include <linux/slab.h>
> #include <linux/kernel.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> /*
> diff --git a/fs/ntfs3/attrlist.c b/fs/ntfs3/attrlist.c
> index bad6d8a849a2..c3934a2a28a9 100644
> --- a/fs/ntfs3/attrlist.c
> +++ b/fs/ntfs3/attrlist.c
> @@ -7,8 +7,6 @@
>
> #include <linux/fs.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> /*
> diff --git a/fs/ntfs3/dir.c b/fs/ntfs3/dir.c
> index 785e72d4392e..293303f00b66 100644
> --- a/fs/ntfs3/dir.c
> +++ b/fs/ntfs3/dir.c
> @@ -10,8 +10,6 @@
> #include <linux/fs.h>
> #include <linux/nls.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> /* Convert little endian UTF-16 to NLS string. */
> diff --git a/fs/ntfs3/file.c b/fs/ntfs3/file.c
> index 5fb3508e5422..13789543a0fb 100644
> --- a/fs/ntfs3/file.c
> +++ b/fs/ntfs3/file.c
> @@ -13,8 +13,6 @@
> #include <linux/falloc.h>
> #include <linux/fiemap.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> static int ntfs_ioctl_fitrim(struct ntfs_sb_info *sbi, unsigned long arg)
> diff --git a/fs/ntfs3/frecord.c b/fs/ntfs3/frecord.c
> index 007602badd90..b27f3ca2704b 100644
> --- a/fs/ntfs3/frecord.c
> +++ b/fs/ntfs3/frecord.c
> @@ -9,8 +9,6 @@
> #include <linux/fs.h>
> #include <linux/vmalloc.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
> #ifdef CONFIG_NTFS3_LZX_XPRESS
> #include "lib/lib.h"
> diff --git a/fs/ntfs3/fslog.c b/fs/ntfs3/fslog.c
> index 06492f088d60..4bf340babb32 100644
> --- a/fs/ntfs3/fslog.c
> +++ b/fs/ntfs3/fslog.c
> @@ -10,8 +10,6 @@
> #include <linux/random.h>
> #include <linux/slab.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> /*
> diff --git a/fs/ntfs3/fsntfs.c b/fs/ntfs3/fsntfs.c
> index 4de9acb16968..85cbbb8f41ea 100644
> --- a/fs/ntfs3/fsntfs.c
> +++ b/fs/ntfs3/fsntfs.c
> @@ -10,8 +10,6 @@
> #include <linux/fs.h>
> #include <linux/kernel.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> // clang-format off
> diff --git a/fs/ntfs3/index.c b/fs/ntfs3/index.c
> index 6f81e3a49abf..a25f04dcb85b 100644
> --- a/fs/ntfs3/index.c
> +++ b/fs/ntfs3/index.c
> @@ -8,10 +8,7 @@
> #include <linux/blkdev.h>
> #include <linux/buffer_head.h>
> #include <linux/fs.h>
> -#include <linux/kernel.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> static const struct INDEX_NAMES {
> diff --git a/fs/ntfs3/inode.c b/fs/ntfs3/inode.c
> index 7dd162f6a7e2..06113610c529 100644
> --- a/fs/ntfs3/inode.c
> +++ b/fs/ntfs3/inode.c
> @@ -13,8 +13,6 @@
> #include <linux/uio.h>
> #include <linux/writeback.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> /*
> diff --git a/fs/ntfs3/lznt.c b/fs/ntfs3/lznt.c
> index 28f654561f27..d9614b8e1b4e 100644
> --- a/fs/ntfs3/lznt.c
> +++ b/fs/ntfs3/lznt.c
> @@ -11,7 +11,6 @@
> #include <linux/string.h>
> #include <linux/types.h>
>
> -#include "debug.h"
> #include "ntfs_fs.h"
>
> // clang-format off
> diff --git a/fs/ntfs3/namei.c b/fs/ntfs3/namei.c
> index bc741213ad84..ed29cd3e98f4 100644
> --- a/fs/ntfs3/namei.c
> +++ b/fs/ntfs3/namei.c
> @@ -8,8 +8,6 @@
> #include <linux/fs.h>
> #include <linux/nls.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> /*
> diff --git a/fs/ntfs3/ntfs_fs.h b/fs/ntfs3/ntfs_fs.h
> index 38b7c1a9dc52..e6f37f9993a0 100644
> --- a/fs/ntfs3/ntfs_fs.h
> +++ b/fs/ntfs3/ntfs_fs.h
> @@ -29,7 +29,6 @@
> #include <asm/div64.h>
> #include <asm/page.h>
>
> -#include "debug.h"
> #include "ntfs.h"
>
> struct dentry;
> diff --git a/fs/ntfs3/record.c b/fs/ntfs3/record.c
> index 861e35791506..3dd7b960ac8d 100644
> --- a/fs/ntfs3/record.c
> +++ b/fs/ntfs3/record.c
> @@ -7,8 +7,6 @@
>
> #include <linux/fs.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> static inline int compare_attr(const struct ATTRIB *left, enum ATTR_TYPE type,
> diff --git a/fs/ntfs3/run.c b/fs/ntfs3/run.c
> index a8fec651f973..f5a5ce7aa206 100644
> --- a/fs/ntfs3/run.c
> +++ b/fs/ntfs3/run.c
> @@ -10,8 +10,6 @@
> #include <linux/fs.h>
> #include <linux/log2.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> /* runs_tree is a continues memory. Try to avoid big size. */
> diff --git a/fs/ntfs3/super.c b/fs/ntfs3/super.c
> index 705d8b4f4894..bd8d39992b35 100644
> --- a/fs/ntfs3/super.c
> +++ b/fs/ntfs3/super.c
> @@ -35,8 +35,6 @@
> #include <linux/seq_file.h>
> #include <linux/statfs.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
> #ifdef CONFIG_NTFS3_LZX_XPRESS
> #include "lib/lib.h"
> @@ -772,7 +770,7 @@ static int ntfs_init_from_boot(struct super_block *sb, u32 sector_size,
> /* No way to use ntfs_get_block in this case. */
> ntfs_err(
> sb,
> - "Failed to mount 'cause NTFS's cluster size (%u) is less than media sector size (%u)",
> + "Failed to mount 'cause NTFS's cluster size (%u) is less than media's sector size (%u)",
> sbi->cluster_size, sector_size);
> goto out;
> }
> diff --git a/fs/ntfs3/xattr.c b/fs/ntfs3/xattr.c
> index 111355692163..0673ba5e8c43 100644
> --- a/fs/ntfs3/xattr.c
> +++ b/fs/ntfs3/xattr.c
> @@ -10,8 +10,6 @@
> #include <linux/posix_acl_xattr.h>
> #include <linux/xattr.h>
>
> -#include "debug.h"
> -#include "ntfs.h"
> #include "ntfs_fs.h"
>
> // clang-format off
> --
> 2.33.0
>
prev parent reply other threads:[~2021-10-03 17:42 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-01 16:01 [PATCH 0/2] Removing of old code Konstantin Komarov
2021-10-01 16:02 ` [PATCH 1/2] fs/ntfs3: Remove unnecessary functions Konstantin Komarov
2021-10-16 10:42 ` Kari Argillander
2021-10-19 17:11 ` Konstantin Komarov
2021-10-01 16:02 ` [PATCH 2/2] fs/ntfs3: Remove unnecessary includes Konstantin Komarov
2021-10-03 17:42 ` Kari Argillander [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211003174244.ijg3uxoqdt2ywyqe@kari-VirtualBox \
--to=kari.argillander@gmail.com \
--cc=almaz.alexandrovich@paragon-software.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=ntfs3@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®