Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1646375 > unrolled thread

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

Back to article view | Back to linux.kernel

Contents

  [PATCH 02/11 linux-next] exofs: use magic.h Fabian Frederick <fabf@skynet.be> - 2017-05-21 17:50 +0200
    Re: [PATCH 02/11 linux-next] exofs: use magic.h Boaz Harrosh <ooo@electrozaur.com> - 2017-05-23 10:10 +0200

#1646375 — [PATCH 02/11 linux-next] exofs: use magic.h

Filesystems generally use SUPER_MAGIC values from magic.h
instead of a local definition.

Signed-off-by: Fabian Frederick <fabf@skynet.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
- ****************************************************************************/
-#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
-- 
2.9.3

[toc] | [next] | [standalone]

#1647769

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@skynet.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
> 

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web