Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
To:     Kari Argillander <kari.argillander@gmail.com>,
        "ntfs3@lists.linux.dev" <ntfs3@lists.linux.dev>
CC:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH] fs/ntfs3: Add ifndef + define to all header files
Thread-Topic: [PATCH] fs/ntfs3: Add ifndef + define to all header files
Thread-Index: AQHXkpaNnll4ojWsX0ux0GOVaSMR8auHlqNw
Date:   Fri, 27 Aug 2021 16:09:54 +0000
Message-ID: <e73f5cebaee948568c2738bd1b9d9edf@paragon-software.com>
References: <20210816120156.186193-1-kari.argillander@gmail.com>
In-Reply-To: <20210816120156.186193-1-kari.argillander@gmail.com>
Accept-Language: ru-RU, en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Precedence: bulk

> From: Kari Argillander <kari.argillander@gmail.com>
> Sent: Monday, August 16, 2021 3:02 PM
> To: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>; ntfs3@=
lists.linux.dev
> Cc: Kari Argillander <kari.argillander@gmail.com>; linux-kernel@vger.kern=
el.org
> Subject: [PATCH] fs/ntfs3: Add ifndef + define to all header files
>=20
> Add guards so that compiler will only include header files once.
>=20
> Signed-off-by: Kari Argillander <kari.argillander@gmail.com>
> ---
> I do not mind if this get's merged to V28/V29 without commit.
> ---
>  fs/ntfs3/debug.h   | 5 +++++
>  fs/ntfs3/ntfs.h    | 3 +++
>  fs/ntfs3/ntfs_fs.h | 5 +++++
>  3 files changed, 13 insertions(+)
>=20
> diff --git a/fs/ntfs3/debug.h b/fs/ntfs3/debug.h
> index dfaa4c79dc6d..465466631d38 100644
> --- a/fs/ntfs3/debug.h
> +++ b/fs/ntfs3/debug.h
> @@ -7,6 +7,9 @@
>   */
>=20
>  // clang-format off
> +#ifndef _LINUX_NTFS3_DEBUG_H
> +#define _LINUX_NTFS3_DEBUG_H
> +
>  #ifndef Add2Ptr
>  #define Add2Ptr(P, I)		((void *)((u8 *)(P) + (I)))
>  #define PtrOffset(B, O)		((size_t)((size_t)(O) - (size_t)(B)))
> @@ -61,4 +64,6 @@ void ntfs_inode_printk(struct inode *inode, const char =
*fmt, ...)
>  #define ntfs_free(p)		kfree(p)
>  #define ntfs_vfree(p)		kvfree(p)
>  #define ntfs_memdup(src, len)	kmemdup(src, len, GFP_NOFS)
> +
> +#endif /* _LINUX_NTFS3_DEBUG_H */
>  // clang-format on
> diff --git a/fs/ntfs3/ntfs.h b/fs/ntfs3/ntfs.h
> index 40398e6c39c9..16da514af124 100644
> --- a/fs/ntfs3/ntfs.h
> +++ b/fs/ntfs3/ntfs.h
> @@ -7,6 +7,8 @@
>   */
>=20
>  // clang-format off
> +#ifndef _LINUX_NTFS3_NTFS_H
> +#define _LINUX_NTFS3_NTFS_H
>=20
>  /* TODO:
>   * - Check 4K mft record and 512 bytes cluster
> @@ -1235,4 +1237,5 @@ struct SID {
>  };
>  static_assert(offsetof(struct SID, SubAuthority) =3D=3D 8);
>=20
> +#endif /* _LINUX_NTFS3_NTFS_H */
>  // clang-format on
> diff --git a/fs/ntfs3/ntfs_fs.h b/fs/ntfs3/ntfs_fs.h
> index 0c3ac89c3115..80e1dea2579a 100644
> --- a/fs/ntfs3/ntfs_fs.h
> +++ b/fs/ntfs3/ntfs_fs.h
> @@ -6,6 +6,9 @@
>   */
>=20
>  // clang-format off
> +#ifndef _LINUX_NTFS3_NTFS_FS_H
> +#define _LINUX_NTFS3_NTFS_FS_H
> +
>  #define MINUS_ONE_T			((size_t)(-1))
>  /* Biggest MFT / smallest cluster */
>  #define MAXIMUM_BYTES_PER_MFT		4096
> @@ -1090,3 +1093,5 @@ static inline void le64_sub_cpu(__le64 *var, u64 va=
l)
>  {
>  	*var =3D cpu_to_le64(le64_to_cpu(*var) - val);
>  }
> +
> +#endif /* _LINUX_NTFS3_NTFS_FS_H */
> --
> 2.30.2

Thanks again - applied!

Best regards