linux-kernel - Re: [PATCH 02/11 linux-next] exofs: use magic.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <a55d1404-3ad8-cd25-8686-5739a793d2db@electrozaur.com>
Date:   Tue, 23 May 2017 11:06:33 +0300
From:   Boaz Harrosh <ooo@...ctrozaur.com>
To:     Fabian Frederick <fabf@...net.be>,
        Alexander Viro <viro@...iv.linux.org.uk>
Cc:     linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 02/11 linux-next] exofs: use magic.h

On 05/21/2017 06:40 PM, Fabian Frederick wrote:
> Filesystems generally use SUPER_MAGIC values from magic.h
> instead of a local definition.
> 

Is fine by me to move to magic.h but ...

> Signed-off-by: Fabian Frederick <fabf@...net.be>
> ---
>  fs/exofs/common.h          | 6 +-----
>  include/uapi/linux/magic.h | 1 +
>  2 files changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/fs/exofs/common.h b/fs/exofs/common.h
> index 7d88ef5..fb988f9 100644
> --- a/fs/exofs/common.h
> +++ b/fs/exofs/common.h
> @@ -37,6 +37,7 @@
>  #define __EXOFS_COM_H__
>  
>  #include <linux/types.h>
> +#include <linux/magic.h>
>  
>  #include <scsi/osd_attributes.h>
>  #include <scsi/osd_initiator.h>
> @@ -79,11 +80,6 @@ enum {
>  #define EXOFS_BLKSHIFT	12
>  #define EXOFS_BLKSIZE	(1UL << EXOFS_BLKSHIFT)
>  
> -/****************************************************************************
> - * superblock-related things
> - ****************************************************************************/

Please do not remove this comment, it acts as a separator and is related to
everything up to the comment that says:
 * inode-related things
	
And not only to this definition

Thanks
Boaz

> -#define EXOFS_SUPER_MAGIC	0x5DF5
> -
>  /*
>   * The file system control block - stored in object EXOFS_SUPER_ID's data.
>   * This is where the in-memory superblock is stored on disk.
> diff --git a/include/uapi/linux/magic.h b/include/uapi/linux/magic.h
> index e3bb43b..1265355 100644
> --- a/include/uapi/linux/magic.h
> +++ b/include/uapi/linux/magic.h
> @@ -26,6 +26,7 @@
>  #define EXT2_SUPER_MAGIC	0xEF53
>  #define EXT3_SUPER_MAGIC	0xEF53
>  #define EXT4_SUPER_MAGIC	0xEF53
> +#define EXOFS_SUPER_MAGIC	0x5DF5
>  #define F2FS_SUPER_MAGIC	0xF2F52010
>  #define FUTEXFS_SUPER_MAGIC	0xBAD1DEA
>  #define HOSTFS_SUPER_MAGIC	0x00c0ffee
>