* [PATCH] xfs: remove deprecated barrier/nobarrier mount options
@ 2018-06-26 0:03 Eric Sandeen
2018-06-26 0:16 ` Darrick J. Wong
2018-06-26 0:43 ` [PATCH 4.19 V2] " Eric Sandeen
0 siblings, 2 replies; 10+ messages in thread
From: Eric Sandeen @ 2018-06-26 0:03 UTC (permalink / raw)
To: linux-xfs
The barrier mount options have been no-ops and deprecated since
4cf4573 xfs: deprecate barrier/nobarrier mount option
i.e. kernel 4.10 / December 2016, with a stated deprecation schedule
after v4.15. Should be fair game to remove them now.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---
Documentation/filesystems/xfs.txt | 4 ++--
fs/xfs/xfs_mount.h | 1 -
fs/xfs/xfs_super.c | 34 ++++------------------------------
3 files changed, 6 insertions(+), 33 deletions(-)
diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
index 4d9ff0a..07dbd4e 100644
--- a/Documentation/filesystems/xfs.txt
+++ b/Documentation/filesystems/xfs.txt
@@ -223,8 +223,6 @@ Deprecated Mount Options
Name Removal Schedule
---- ----------------
- barrier no earlier than v4.15
- nobarrier no earlier than v4.15
Removed Mount Options
@@ -236,6 +234,8 @@ Removed Mount Options
ihashsize v4.0
irixsgid v4.0
osyncisdsync/osyncisosync v4.0
+ barrier v4.18
+ nobarrier v4.18
sysctls
diff --git a/fs/xfs/xfs_mount.h b/fs/xfs/xfs_mount.h
index 245349d..0f9115c 100644
--- a/fs/xfs/xfs_mount.h
+++ b/fs/xfs/xfs_mount.h
@@ -216,7 +216,6 @@ struct xfs_error_cfg {
#define XFS_MOUNT_SMALL_INUMS (1ULL << 14) /* user wants 32bit inodes */
#define XFS_MOUNT_32BITINODES (1ULL << 15) /* inode32 allocator active */
#define XFS_MOUNT_NOUUID (1ULL << 16) /* ignore uuid during mount */
-#define XFS_MOUNT_BARRIER (1ULL << 17)
#define XFS_MOUNT_IKEEP (1ULL << 18) /* keep empty inode clusters*/
#define XFS_MOUNT_SWALLOC (1ULL << 19) /* turn on stripe width
* allocation */
diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
index 9d791f1..03c5677 100644
--- a/fs/xfs/xfs_super.c
+++ b/fs/xfs/xfs_super.c
@@ -65,11 +65,10 @@ enum {
Opt_logbufs, Opt_logbsize, Opt_logdev, Opt_rtdev, Opt_biosize,
Opt_wsync, Opt_noalign, Opt_swalloc, Opt_sunit, Opt_swidth, Opt_nouuid,
Opt_mtpt, Opt_grpid, Opt_nogrpid, Opt_bsdgroups, Opt_sysvgroups,
- Opt_allocsize, Opt_norecovery, Opt_barrier, Opt_nobarrier,
- Opt_inode64, Opt_inode32, Opt_ikeep, Opt_noikeep,
- Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2, Opt_filestreams,
- Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota, Opt_prjquota,
- Opt_uquota, Opt_gquota, Opt_pquota,
+ Opt_allocsize, Opt_norecovery, Opt_inode64, Opt_inode32, Opt_ikeep,
+ Opt_noikeep, Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2,
+ Opt_filestreams, Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota,
+ Opt_prjquota, Opt_uquota, Opt_gquota, Opt_pquota,
Opt_uqnoenforce, Opt_gqnoenforce, Opt_pqnoenforce, Opt_qnoenforce,
Opt_discard, Opt_nodiscard, Opt_dax, Opt_err,
};
@@ -118,14 +117,7 @@ enum {
{Opt_qnoenforce, "qnoenforce"}, /* same as uqnoenforce */
{Opt_discard, "discard"}, /* Discard unused blocks */
{Opt_nodiscard, "nodiscard"}, /* Do not discard unused blocks */
-
{Opt_dax, "dax"}, /* Enable direct access to bdev pages */
-
- /* Deprecated mount options scheduled for removal */
- {Opt_barrier, "barrier"}, /* use writer barriers for log write and
- * unwritten extent conversion */
- {Opt_nobarrier, "nobarrier"}, /* .. disable */
-
{Opt_err, NULL},
};
@@ -209,7 +201,6 @@ enum {
* Set some default flags that could be cleared by the mount option
* parsing.
*/
- mp->m_flags |= XFS_MOUNT_BARRIER;
mp->m_flags |= XFS_MOUNT_COMPAT_IOSIZE;
/*
@@ -362,14 +353,6 @@ enum {
mp->m_flags |= XFS_MOUNT_DAX;
break;
#endif
- case Opt_barrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags |= XFS_MOUNT_BARRIER;
- break;
- case Opt_nobarrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags &= ~XFS_MOUNT_BARRIER;
- break;
default:
xfs_warn(mp, "unknown mount option [%s].", p);
return -EINVAL;
@@ -487,7 +470,6 @@ struct proc_xfs_info {
static struct proc_xfs_info xfs_info_unset[] = {
/* the few simple ones we can get from the mount struct */
{ XFS_MOUNT_COMPAT_IOSIZE, ",largeio" },
- { XFS_MOUNT_BARRIER, ",nobarrier" },
{ XFS_MOUNT_SMALL_INUMS, ",inode64" },
{ 0, NULL }
};
@@ -1278,14 +1260,6 @@ struct proc_xfs_info {
token = match_token(p, tokens, args);
switch (token) {
- case Opt_barrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags |= XFS_MOUNT_BARRIER;
- break;
- case Opt_nobarrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags &= ~XFS_MOUNT_BARRIER;
- break;
case Opt_inode64:
mp->m_flags &= ~XFS_MOUNT_SMALL_INUMS;
mp->m_maxagi = xfs_set_inode_alloc(mp, sbp->sb_agcount);
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] xfs: remove deprecated barrier/nobarrier mount options
2018-06-26 0:03 [PATCH] xfs: remove deprecated barrier/nobarrier mount options Eric Sandeen
@ 2018-06-26 0:16 ` Darrick J. Wong
2018-06-26 0:34 ` Eric Sandeen
2018-06-26 0:43 ` [PATCH 4.19 V2] " Eric Sandeen
1 sibling, 1 reply; 10+ messages in thread
From: Darrick J. Wong @ 2018-06-26 0:16 UTC (permalink / raw)
To: Eric Sandeen; +Cc: linux-xfs
On Mon, Jun 25, 2018 at 07:03:57PM -0500, Eric Sandeen wrote:
> The barrier mount options have been no-ops and deprecated since
>
> 4cf4573 xfs: deprecate barrier/nobarrier mount option
>
> i.e. kernel 4.10 / December 2016, with a stated deprecation schedule
> after v4.15. Should be fair game to remove them now.
>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
> ---
> Documentation/filesystems/xfs.txt | 4 ++--
> fs/xfs/xfs_mount.h | 1 -
> fs/xfs/xfs_super.c | 34 ++++------------------------------
> 3 files changed, 6 insertions(+), 33 deletions(-)
>
>
> diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
> index 4d9ff0a..07dbd4e 100644
> --- a/Documentation/filesystems/xfs.txt
> +++ b/Documentation/filesystems/xfs.txt
> @@ -223,8 +223,6 @@ Deprecated Mount Options
>
> Name Removal Schedule
> ---- ----------------
> - barrier no earlier than v4.15
> - nobarrier no earlier than v4.15
>
>
> Removed Mount Options
> @@ -236,6 +234,8 @@ Removed Mount Options
> ihashsize v4.0
> irixsgid v4.0
> osyncisdsync/osyncisosync v4.0
> + barrier v4.18
> + nobarrier v4.18
I'm not removing mount options this far into the 4.18 cycle. ;)
4.19, perhaps?
--D
>
>
> sysctls
> diff --git a/fs/xfs/xfs_mount.h b/fs/xfs/xfs_mount.h
> index 245349d..0f9115c 100644
> --- a/fs/xfs/xfs_mount.h
> +++ b/fs/xfs/xfs_mount.h
> @@ -216,7 +216,6 @@ struct xfs_error_cfg {
> #define XFS_MOUNT_SMALL_INUMS (1ULL << 14) /* user wants 32bit inodes */
> #define XFS_MOUNT_32BITINODES (1ULL << 15) /* inode32 allocator active */
> #define XFS_MOUNT_NOUUID (1ULL << 16) /* ignore uuid during mount */
> -#define XFS_MOUNT_BARRIER (1ULL << 17)
> #define XFS_MOUNT_IKEEP (1ULL << 18) /* keep empty inode clusters*/
> #define XFS_MOUNT_SWALLOC (1ULL << 19) /* turn on stripe width
> * allocation */
> diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
> index 9d791f1..03c5677 100644
> --- a/fs/xfs/xfs_super.c
> +++ b/fs/xfs/xfs_super.c
> @@ -65,11 +65,10 @@ enum {
> Opt_logbufs, Opt_logbsize, Opt_logdev, Opt_rtdev, Opt_biosize,
> Opt_wsync, Opt_noalign, Opt_swalloc, Opt_sunit, Opt_swidth, Opt_nouuid,
> Opt_mtpt, Opt_grpid, Opt_nogrpid, Opt_bsdgroups, Opt_sysvgroups,
> - Opt_allocsize, Opt_norecovery, Opt_barrier, Opt_nobarrier,
> - Opt_inode64, Opt_inode32, Opt_ikeep, Opt_noikeep,
> - Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2, Opt_filestreams,
> - Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota, Opt_prjquota,
> - Opt_uquota, Opt_gquota, Opt_pquota,
> + Opt_allocsize, Opt_norecovery, Opt_inode64, Opt_inode32, Opt_ikeep,
> + Opt_noikeep, Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2,
> + Opt_filestreams, Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota,
> + Opt_prjquota, Opt_uquota, Opt_gquota, Opt_pquota,
> Opt_uqnoenforce, Opt_gqnoenforce, Opt_pqnoenforce, Opt_qnoenforce,
> Opt_discard, Opt_nodiscard, Opt_dax, Opt_err,
> };
> @@ -118,14 +117,7 @@ enum {
> {Opt_qnoenforce, "qnoenforce"}, /* same as uqnoenforce */
> {Opt_discard, "discard"}, /* Discard unused blocks */
> {Opt_nodiscard, "nodiscard"}, /* Do not discard unused blocks */
> -
> {Opt_dax, "dax"}, /* Enable direct access to bdev pages */
> -
> - /* Deprecated mount options scheduled for removal */
> - {Opt_barrier, "barrier"}, /* use writer barriers for log write and
> - * unwritten extent conversion */
> - {Opt_nobarrier, "nobarrier"}, /* .. disable */
> -
> {Opt_err, NULL},
> };
>
> @@ -209,7 +201,6 @@ enum {
> * Set some default flags that could be cleared by the mount option
> * parsing.
> */
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> mp->m_flags |= XFS_MOUNT_COMPAT_IOSIZE;
>
> /*
> @@ -362,14 +353,6 @@ enum {
> mp->m_flags |= XFS_MOUNT_DAX;
> break;
> #endif
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> default:
> xfs_warn(mp, "unknown mount option [%s].", p);
> return -EINVAL;
> @@ -487,7 +470,6 @@ struct proc_xfs_info {
> static struct proc_xfs_info xfs_info_unset[] = {
> /* the few simple ones we can get from the mount struct */
> { XFS_MOUNT_COMPAT_IOSIZE, ",largeio" },
> - { XFS_MOUNT_BARRIER, ",nobarrier" },
> { XFS_MOUNT_SMALL_INUMS, ",inode64" },
> { 0, NULL }
> };
> @@ -1278,14 +1260,6 @@ struct proc_xfs_info {
>
> token = match_token(p, tokens, args);
> switch (token) {
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> case Opt_inode64:
> mp->m_flags &= ~XFS_MOUNT_SMALL_INUMS;
> mp->m_maxagi = xfs_set_inode_alloc(mp, sbp->sb_agcount);
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] xfs: remove deprecated barrier/nobarrier mount options
2018-06-26 0:16 ` Darrick J. Wong
@ 2018-06-26 0:34 ` Eric Sandeen
2018-06-26 0:40 ` Darrick J. Wong
0 siblings, 1 reply; 10+ messages in thread
From: Eric Sandeen @ 2018-06-26 0:34 UTC (permalink / raw)
To: Darrick J. Wong, Eric Sandeen; +Cc: linux-xfs
On 6/25/18 7:16 PM, Darrick J. Wong wrote:
> On Mon, Jun 25, 2018 at 07:03:57PM -0500, Eric Sandeen wrote:
>> The barrier mount options have been no-ops and deprecated since
>>
>> 4cf4573 xfs: deprecate barrier/nobarrier mount option
>>
>> i.e. kernel 4.10 / December 2016, with a stated deprecation schedule
>> after v4.15. Should be fair game to remove them now.
>>
>> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
>> ---
>> Documentation/filesystems/xfs.txt | 4 ++--
>> fs/xfs/xfs_mount.h | 1 -
>> fs/xfs/xfs_super.c | 34 ++++------------------------------
>> 3 files changed, 6 insertions(+), 33 deletions(-)
>>
>>
>> diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
>> index 4d9ff0a..07dbd4e 100644
>> --- a/Documentation/filesystems/xfs.txt
>> +++ b/Documentation/filesystems/xfs.txt
>> @@ -223,8 +223,6 @@ Deprecated Mount Options
>>
>> Name Removal Schedule
>> ---- ----------------
>> - barrier no earlier than v4.15
>> - nobarrier no earlier than v4.15
>>
>>
>> Removed Mount Options
>> @@ -236,6 +234,8 @@ Removed Mount Options
>> ihashsize v4.0
>> irixsgid v4.0
>> osyncisdsync/osyncisosync v4.0
>> + barrier v4.18
>> + nobarrier v4.18
>
> I'm not removing mount options this far into the 4.18 cycle. ;)
>
> 4.19, perhaps?
Yeah I kinda figured. Resend V2 later? Edit on the way in and stash it
for a while? V2 now?
-Eric
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] xfs: remove deprecated barrier/nobarrier mount options
2018-06-26 0:34 ` Eric Sandeen
@ 2018-06-26 0:40 ` Darrick J. Wong
0 siblings, 0 replies; 10+ messages in thread
From: Darrick J. Wong @ 2018-06-26 0:40 UTC (permalink / raw)
To: Eric Sandeen; +Cc: Eric Sandeen, linux-xfs
On Mon, Jun 25, 2018 at 07:34:20PM -0500, Eric Sandeen wrote:
> On 6/25/18 7:16 PM, Darrick J. Wong wrote:
> > On Mon, Jun 25, 2018 at 07:03:57PM -0500, Eric Sandeen wrote:
> >> The barrier mount options have been no-ops and deprecated since
> >>
> >> 4cf4573 xfs: deprecate barrier/nobarrier mount option
> >>
> >> i.e. kernel 4.10 / December 2016, with a stated deprecation schedule
> >> after v4.15. Should be fair game to remove them now.
> >>
> >> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
> >> ---
> >> Documentation/filesystems/xfs.txt | 4 ++--
> >> fs/xfs/xfs_mount.h | 1 -
> >> fs/xfs/xfs_super.c | 34 ++++------------------------------
> >> 3 files changed, 6 insertions(+), 33 deletions(-)
> >>
> >>
> >> diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
> >> index 4d9ff0a..07dbd4e 100644
> >> --- a/Documentation/filesystems/xfs.txt
> >> +++ b/Documentation/filesystems/xfs.txt
> >> @@ -223,8 +223,6 @@ Deprecated Mount Options
> >>
> >> Name Removal Schedule
> >> ---- ----------------
> >> - barrier no earlier than v4.15
> >> - nobarrier no earlier than v4.15
> >>
> >>
> >> Removed Mount Options
> >> @@ -236,6 +234,8 @@ Removed Mount Options
> >> ihashsize v4.0
> >> irixsgid v4.0
> >> osyncisdsync/osyncisosync v4.0
> >> + barrier v4.18
> >> + nobarrier v4.18
> >
> > I'm not removing mount options this far into the 4.18 cycle. ;)
> >
> > 4.19, perhaps?
>
> Yeah I kinda figured. Resend V2 later? Edit on the way in and stash it
> for a while? V2 now?
v2 now, I'll stuff it in the 4.19 branch when I get around to creating
it......
--D
>
> -Eric
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 4.19 V2] xfs: remove deprecated barrier/nobarrier mount options
2018-06-26 0:03 [PATCH] xfs: remove deprecated barrier/nobarrier mount options Eric Sandeen
2018-06-26 0:16 ` Darrick J. Wong
@ 2018-06-26 0:43 ` Eric Sandeen
2018-06-26 11:23 ` Carlos Maiolino
2018-06-26 14:32 ` Darrick J. Wong
1 sibling, 2 replies; 10+ messages in thread
From: Eric Sandeen @ 2018-06-26 0:43 UTC (permalink / raw)
To: Eric Sandeen, linux-xfs
The barrier mount options have been no-ops and deprecated since
4cf4573 xfs: deprecate barrier/nobarrier mount option
i.e. kernel 4.10 / December 2016, with a stated deprecation schedule
after v4.15. Should be fair game to remove them now.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---
V2: plan for 4.19.
(now wait, it'll be 5.0)
Documentation/filesystems/xfs.txt | 4 ++--
fs/xfs/xfs_mount.h | 1 -
fs/xfs/xfs_super.c | 34 ++++------------------------------
3 files changed, 6 insertions(+), 33 deletions(-)
diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
index 4d9ff0a..07dbd4e 100644
--- a/Documentation/filesystems/xfs.txt
+++ b/Documentation/filesystems/xfs.txt
@@ -223,8 +223,6 @@ Deprecated Mount Options
Name Removal Schedule
---- ----------------
- barrier no earlier than v4.15
- nobarrier no earlier than v4.15
Removed Mount Options
@@ -236,6 +234,8 @@ Removed Mount Options
ihashsize v4.0
irixsgid v4.0
osyncisdsync/osyncisosync v4.0
+ barrier v4.19
+ nobarrier v4.19
sysctls
diff --git a/fs/xfs/xfs_mount.h b/fs/xfs/xfs_mount.h
index 245349d..0f9115c 100644
--- a/fs/xfs/xfs_mount.h
+++ b/fs/xfs/xfs_mount.h
@@ -216,7 +216,6 @@ struct xfs_error_cfg {
#define XFS_MOUNT_SMALL_INUMS (1ULL << 14) /* user wants 32bit inodes */
#define XFS_MOUNT_32BITINODES (1ULL << 15) /* inode32 allocator active */
#define XFS_MOUNT_NOUUID (1ULL << 16) /* ignore uuid during mount */
-#define XFS_MOUNT_BARRIER (1ULL << 17)
#define XFS_MOUNT_IKEEP (1ULL << 18) /* keep empty inode clusters*/
#define XFS_MOUNT_SWALLOC (1ULL << 19) /* turn on stripe width
* allocation */
diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
index 9d791f1..03c5677 100644
--- a/fs/xfs/xfs_super.c
+++ b/fs/xfs/xfs_super.c
@@ -65,11 +65,10 @@ enum {
Opt_logbufs, Opt_logbsize, Opt_logdev, Opt_rtdev, Opt_biosize,
Opt_wsync, Opt_noalign, Opt_swalloc, Opt_sunit, Opt_swidth, Opt_nouuid,
Opt_mtpt, Opt_grpid, Opt_nogrpid, Opt_bsdgroups, Opt_sysvgroups,
- Opt_allocsize, Opt_norecovery, Opt_barrier, Opt_nobarrier,
- Opt_inode64, Opt_inode32, Opt_ikeep, Opt_noikeep,
- Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2, Opt_filestreams,
- Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota, Opt_prjquota,
- Opt_uquota, Opt_gquota, Opt_pquota,
+ Opt_allocsize, Opt_norecovery, Opt_inode64, Opt_inode32, Opt_ikeep,
+ Opt_noikeep, Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2,
+ Opt_filestreams, Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota,
+ Opt_prjquota, Opt_uquota, Opt_gquota, Opt_pquota,
Opt_uqnoenforce, Opt_gqnoenforce, Opt_pqnoenforce, Opt_qnoenforce,
Opt_discard, Opt_nodiscard, Opt_dax, Opt_err,
};
@@ -118,14 +117,7 @@ enum {
{Opt_qnoenforce, "qnoenforce"}, /* same as uqnoenforce */
{Opt_discard, "discard"}, /* Discard unused blocks */
{Opt_nodiscard, "nodiscard"}, /* Do not discard unused blocks */
-
{Opt_dax, "dax"}, /* Enable direct access to bdev pages */
-
- /* Deprecated mount options scheduled for removal */
- {Opt_barrier, "barrier"}, /* use writer barriers for log write and
- * unwritten extent conversion */
- {Opt_nobarrier, "nobarrier"}, /* .. disable */
-
{Opt_err, NULL},
};
@@ -209,7 +201,6 @@ enum {
* Set some default flags that could be cleared by the mount option
* parsing.
*/
- mp->m_flags |= XFS_MOUNT_BARRIER;
mp->m_flags |= XFS_MOUNT_COMPAT_IOSIZE;
/*
@@ -362,14 +353,6 @@ enum {
mp->m_flags |= XFS_MOUNT_DAX;
break;
#endif
- case Opt_barrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags |= XFS_MOUNT_BARRIER;
- break;
- case Opt_nobarrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags &= ~XFS_MOUNT_BARRIER;
- break;
default:
xfs_warn(mp, "unknown mount option [%s].", p);
return -EINVAL;
@@ -487,7 +470,6 @@ struct proc_xfs_info {
static struct proc_xfs_info xfs_info_unset[] = {
/* the few simple ones we can get from the mount struct */
{ XFS_MOUNT_COMPAT_IOSIZE, ",largeio" },
- { XFS_MOUNT_BARRIER, ",nobarrier" },
{ XFS_MOUNT_SMALL_INUMS, ",inode64" },
{ 0, NULL }
};
@@ -1278,14 +1260,6 @@ struct proc_xfs_info {
token = match_token(p, tokens, args);
switch (token) {
- case Opt_barrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags |= XFS_MOUNT_BARRIER;
- break;
- case Opt_nobarrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags &= ~XFS_MOUNT_BARRIER;
- break;
case Opt_inode64:
mp->m_flags &= ~XFS_MOUNT_SMALL_INUMS;
mp->m_maxagi = xfs_set_inode_alloc(mp, sbp->sb_agcount);
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 4.19 V2] xfs: remove deprecated barrier/nobarrier mount options
2018-06-26 0:43 ` [PATCH 4.19 V2] " Eric Sandeen
@ 2018-06-26 11:23 ` Carlos Maiolino
2018-06-26 14:32 ` Darrick J. Wong
1 sibling, 0 replies; 10+ messages in thread
From: Carlos Maiolino @ 2018-06-26 11:23 UTC (permalink / raw)
To: Eric Sandeen; +Cc: Eric Sandeen, linux-xfs
On Mon, Jun 25, 2018 at 07:43:36PM -0500, Eric Sandeen wrote:
> The barrier mount options have been no-ops and deprecated since
>
> 4cf4573 xfs: deprecate barrier/nobarrier mount option
>
> i.e. kernel 4.10 / December 2016, with a stated deprecation schedule
> after v4.15. Should be fair game to remove them now.
>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
> ---
>
> V2: plan for 4.19.
> (now wait, it'll be 5.0)
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
>
> Documentation/filesystems/xfs.txt | 4 ++--
> fs/xfs/xfs_mount.h | 1 -
> fs/xfs/xfs_super.c | 34 ++++------------------------------
> 3 files changed, 6 insertions(+), 33 deletions(-)
>
> diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
> index 4d9ff0a..07dbd4e 100644
> --- a/Documentation/filesystems/xfs.txt
> +++ b/Documentation/filesystems/xfs.txt
> @@ -223,8 +223,6 @@ Deprecated Mount Options
>
> Name Removal Schedule
> ---- ----------------
> - barrier no earlier than v4.15
> - nobarrier no earlier than v4.15
>
>
> Removed Mount Options
> @@ -236,6 +234,8 @@ Removed Mount Options
> ihashsize v4.0
> irixsgid v4.0
> osyncisdsync/osyncisosync v4.0
> + barrier v4.19
> + nobarrier v4.19
>
>
> sysctls
> diff --git a/fs/xfs/xfs_mount.h b/fs/xfs/xfs_mount.h
> index 245349d..0f9115c 100644
> --- a/fs/xfs/xfs_mount.h
> +++ b/fs/xfs/xfs_mount.h
> @@ -216,7 +216,6 @@ struct xfs_error_cfg {
> #define XFS_MOUNT_SMALL_INUMS (1ULL << 14) /* user wants 32bit inodes */
> #define XFS_MOUNT_32BITINODES (1ULL << 15) /* inode32 allocator active */
> #define XFS_MOUNT_NOUUID (1ULL << 16) /* ignore uuid during mount */
> -#define XFS_MOUNT_BARRIER (1ULL << 17)
> #define XFS_MOUNT_IKEEP (1ULL << 18) /* keep empty inode clusters*/
> #define XFS_MOUNT_SWALLOC (1ULL << 19) /* turn on stripe width
> * allocation */
> diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
> index 9d791f1..03c5677 100644
> --- a/fs/xfs/xfs_super.c
> +++ b/fs/xfs/xfs_super.c
> @@ -65,11 +65,10 @@ enum {
> Opt_logbufs, Opt_logbsize, Opt_logdev, Opt_rtdev, Opt_biosize,
> Opt_wsync, Opt_noalign, Opt_swalloc, Opt_sunit, Opt_swidth, Opt_nouuid,
> Opt_mtpt, Opt_grpid, Opt_nogrpid, Opt_bsdgroups, Opt_sysvgroups,
> - Opt_allocsize, Opt_norecovery, Opt_barrier, Opt_nobarrier,
> - Opt_inode64, Opt_inode32, Opt_ikeep, Opt_noikeep,
> - Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2, Opt_filestreams,
> - Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota, Opt_prjquota,
> - Opt_uquota, Opt_gquota, Opt_pquota,
> + Opt_allocsize, Opt_norecovery, Opt_inode64, Opt_inode32, Opt_ikeep,
> + Opt_noikeep, Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2,
> + Opt_filestreams, Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota,
> + Opt_prjquota, Opt_uquota, Opt_gquota, Opt_pquota,
> Opt_uqnoenforce, Opt_gqnoenforce, Opt_pqnoenforce, Opt_qnoenforce,
> Opt_discard, Opt_nodiscard, Opt_dax, Opt_err,
> };
> @@ -118,14 +117,7 @@ enum {
> {Opt_qnoenforce, "qnoenforce"}, /* same as uqnoenforce */
> {Opt_discard, "discard"}, /* Discard unused blocks */
> {Opt_nodiscard, "nodiscard"}, /* Do not discard unused blocks */
> -
> {Opt_dax, "dax"}, /* Enable direct access to bdev pages */
> -
> - /* Deprecated mount options scheduled for removal */
> - {Opt_barrier, "barrier"}, /* use writer barriers for log write and
> - * unwritten extent conversion */
> - {Opt_nobarrier, "nobarrier"}, /* .. disable */
> -
> {Opt_err, NULL},
> };
>
> @@ -209,7 +201,6 @@ enum {
> * Set some default flags that could be cleared by the mount option
> * parsing.
> */
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> mp->m_flags |= XFS_MOUNT_COMPAT_IOSIZE;
>
> /*
> @@ -362,14 +353,6 @@ enum {
> mp->m_flags |= XFS_MOUNT_DAX;
> break;
> #endif
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> default:
> xfs_warn(mp, "unknown mount option [%s].", p);
> return -EINVAL;
> @@ -487,7 +470,6 @@ struct proc_xfs_info {
> static struct proc_xfs_info xfs_info_unset[] = {
> /* the few simple ones we can get from the mount struct */
> { XFS_MOUNT_COMPAT_IOSIZE, ",largeio" },
> - { XFS_MOUNT_BARRIER, ",nobarrier" },
> { XFS_MOUNT_SMALL_INUMS, ",inode64" },
> { 0, NULL }
> };
> @@ -1278,14 +1260,6 @@ struct proc_xfs_info {
>
> token = match_token(p, tokens, args);
> switch (token) {
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> case Opt_inode64:
> mp->m_flags &= ~XFS_MOUNT_SMALL_INUMS;
> mp->m_maxagi = xfs_set_inode_alloc(mp, sbp->sb_agcount);
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Carlos
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 4.19 V2] xfs: remove deprecated barrier/nobarrier mount options
2018-06-26 0:43 ` [PATCH 4.19 V2] " Eric Sandeen
2018-06-26 11:23 ` Carlos Maiolino
@ 2018-06-26 14:32 ` Darrick J. Wong
1 sibling, 0 replies; 10+ messages in thread
From: Darrick J. Wong @ 2018-06-26 14:32 UTC (permalink / raw)
To: Eric Sandeen; +Cc: Eric Sandeen, linux-xfs
On Mon, Jun 25, 2018 at 07:43:36PM -0500, Eric Sandeen wrote:
> The barrier mount options have been no-ops and deprecated since
>
> 4cf4573 xfs: deprecate barrier/nobarrier mount option
>
> i.e. kernel 4.10 / December 2016, with a stated deprecation schedule
> after v4.15. Should be fair game to remove them now.
>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Looks ok,
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
--D
> ---
>
> V2: plan for 4.19.
> (now wait, it'll be 5.0)
>
> Documentation/filesystems/xfs.txt | 4 ++--
> fs/xfs/xfs_mount.h | 1 -
> fs/xfs/xfs_super.c | 34 ++++------------------------------
> 3 files changed, 6 insertions(+), 33 deletions(-)
>
> diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
> index 4d9ff0a..07dbd4e 100644
> --- a/Documentation/filesystems/xfs.txt
> +++ b/Documentation/filesystems/xfs.txt
> @@ -223,8 +223,6 @@ Deprecated Mount Options
>
> Name Removal Schedule
> ---- ----------------
> - barrier no earlier than v4.15
> - nobarrier no earlier than v4.15
>
>
> Removed Mount Options
> @@ -236,6 +234,8 @@ Removed Mount Options
> ihashsize v4.0
> irixsgid v4.0
> osyncisdsync/osyncisosync v4.0
> + barrier v4.19
> + nobarrier v4.19
>
>
> sysctls
> diff --git a/fs/xfs/xfs_mount.h b/fs/xfs/xfs_mount.h
> index 245349d..0f9115c 100644
> --- a/fs/xfs/xfs_mount.h
> +++ b/fs/xfs/xfs_mount.h
> @@ -216,7 +216,6 @@ struct xfs_error_cfg {
> #define XFS_MOUNT_SMALL_INUMS (1ULL << 14) /* user wants 32bit inodes */
> #define XFS_MOUNT_32BITINODES (1ULL << 15) /* inode32 allocator active */
> #define XFS_MOUNT_NOUUID (1ULL << 16) /* ignore uuid during mount */
> -#define XFS_MOUNT_BARRIER (1ULL << 17)
> #define XFS_MOUNT_IKEEP (1ULL << 18) /* keep empty inode clusters*/
> #define XFS_MOUNT_SWALLOC (1ULL << 19) /* turn on stripe width
> * allocation */
> diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
> index 9d791f1..03c5677 100644
> --- a/fs/xfs/xfs_super.c
> +++ b/fs/xfs/xfs_super.c
> @@ -65,11 +65,10 @@ enum {
> Opt_logbufs, Opt_logbsize, Opt_logdev, Opt_rtdev, Opt_biosize,
> Opt_wsync, Opt_noalign, Opt_swalloc, Opt_sunit, Opt_swidth, Opt_nouuid,
> Opt_mtpt, Opt_grpid, Opt_nogrpid, Opt_bsdgroups, Opt_sysvgroups,
> - Opt_allocsize, Opt_norecovery, Opt_barrier, Opt_nobarrier,
> - Opt_inode64, Opt_inode32, Opt_ikeep, Opt_noikeep,
> - Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2, Opt_filestreams,
> - Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota, Opt_prjquota,
> - Opt_uquota, Opt_gquota, Opt_pquota,
> + Opt_allocsize, Opt_norecovery, Opt_inode64, Opt_inode32, Opt_ikeep,
> + Opt_noikeep, Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2,
> + Opt_filestreams, Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota,
> + Opt_prjquota, Opt_uquota, Opt_gquota, Opt_pquota,
> Opt_uqnoenforce, Opt_gqnoenforce, Opt_pqnoenforce, Opt_qnoenforce,
> Opt_discard, Opt_nodiscard, Opt_dax, Opt_err,
> };
> @@ -118,14 +117,7 @@ enum {
> {Opt_qnoenforce, "qnoenforce"}, /* same as uqnoenforce */
> {Opt_discard, "discard"}, /* Discard unused blocks */
> {Opt_nodiscard, "nodiscard"}, /* Do not discard unused blocks */
> -
> {Opt_dax, "dax"}, /* Enable direct access to bdev pages */
> -
> - /* Deprecated mount options scheduled for removal */
> - {Opt_barrier, "barrier"}, /* use writer barriers for log write and
> - * unwritten extent conversion */
> - {Opt_nobarrier, "nobarrier"}, /* .. disable */
> -
> {Opt_err, NULL},
> };
>
> @@ -209,7 +201,6 @@ enum {
> * Set some default flags that could be cleared by the mount option
> * parsing.
> */
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> mp->m_flags |= XFS_MOUNT_COMPAT_IOSIZE;
>
> /*
> @@ -362,14 +353,6 @@ enum {
> mp->m_flags |= XFS_MOUNT_DAX;
> break;
> #endif
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> default:
> xfs_warn(mp, "unknown mount option [%s].", p);
> return -EINVAL;
> @@ -487,7 +470,6 @@ struct proc_xfs_info {
> static struct proc_xfs_info xfs_info_unset[] = {
> /* the few simple ones we can get from the mount struct */
> { XFS_MOUNT_COMPAT_IOSIZE, ",largeio" },
> - { XFS_MOUNT_BARRIER, ",nobarrier" },
> { XFS_MOUNT_SMALL_INUMS, ",inode64" },
> { 0, NULL }
> };
> @@ -1278,14 +1260,6 @@ struct proc_xfs_info {
>
> token = match_token(p, tokens, args);
> switch (token) {
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> case Opt_inode64:
> mp->m_flags &= ~XFS_MOUNT_SMALL_INUMS;
> mp->m_maxagi = xfs_set_inode_alloc(mp, sbp->sb_agcount);
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] xfs: remove deprecated barrier/nobarrier mount options
2018-07-26 16:01 [PATCH] " Darrick J. Wong
2018-07-26 16:10 ` Eric Sandeen
@ 2018-07-26 16:10 ` Darrick J. Wong
1 sibling, 0 replies; 10+ messages in thread
From: Darrick J. Wong @ 2018-07-26 16:10 UTC (permalink / raw)
To: xfs
On Thu, Jul 26, 2018 at 09:01:19AM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
>
> Remove the deprecated barrier and nobarrier mount options since they do
> nothing and were listed for removal some time around v4.15.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
NAK, Eric Sandeen already sent a patch doing this, will take that
instead.
--D
> ---
> Documentation/filesystems/xfs.txt | 4 ++--
> fs/xfs/xfs_mount.h | 1 -
> fs/xfs/xfs_super.c | 26 +-------------------------
> 3 files changed, 3 insertions(+), 28 deletions(-)
>
> diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
> index 4d9ff0a7f8e1..a9ae82fb9d13 100644
> --- a/Documentation/filesystems/xfs.txt
> +++ b/Documentation/filesystems/xfs.txt
> @@ -223,8 +223,6 @@ Deprecated Mount Options
>
> Name Removal Schedule
> ---- ----------------
> - barrier no earlier than v4.15
> - nobarrier no earlier than v4.15
>
>
> Removed Mount Options
> @@ -236,6 +234,8 @@ Removed Mount Options
> ihashsize v4.0
> irixsgid v4.0
> osyncisdsync/osyncisosync v4.0
> + barrier v4.19
> + nobarrier v4.19
>
>
> sysctls
> diff --git a/fs/xfs/xfs_mount.h b/fs/xfs/xfs_mount.h
> index 16869d771b71..387b3868c66a 100644
> --- a/fs/xfs/xfs_mount.h
> +++ b/fs/xfs/xfs_mount.h
> @@ -229,7 +229,6 @@ typedef struct xfs_mount {
> #define XFS_MOUNT_SMALL_INUMS (1ULL << 14) /* user wants 32bit inodes */
> #define XFS_MOUNT_32BITINODES (1ULL << 15) /* inode32 allocator active */
> #define XFS_MOUNT_NOUUID (1ULL << 16) /* ignore uuid during mount */
> -#define XFS_MOUNT_BARRIER (1ULL << 17)
> #define XFS_MOUNT_IKEEP (1ULL << 18) /* keep empty inode clusters*/
> #define XFS_MOUNT_SWALLOC (1ULL << 19) /* turn on stripe width
> * allocation */
> diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
> index 9dd7df216fc3..afe59ce33876 100644
> --- a/fs/xfs/xfs_super.c
> +++ b/fs/xfs/xfs_super.c
> @@ -65,7 +65,7 @@ enum {
> Opt_logbufs, Opt_logbsize, Opt_logdev, Opt_rtdev, Opt_biosize,
> Opt_wsync, Opt_noalign, Opt_swalloc, Opt_sunit, Opt_swidth, Opt_nouuid,
> Opt_mtpt, Opt_grpid, Opt_nogrpid, Opt_bsdgroups, Opt_sysvgroups,
> - Opt_allocsize, Opt_norecovery, Opt_barrier, Opt_nobarrier,
> + Opt_allocsize, Opt_norecovery,
> Opt_inode64, Opt_inode32, Opt_ikeep, Opt_noikeep,
> Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2, Opt_filestreams,
> Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota, Opt_prjquota,
> @@ -120,12 +120,6 @@ static const match_table_t tokens = {
> {Opt_nodiscard, "nodiscard"}, /* Do not discard unused blocks */
>
> {Opt_dax, "dax"}, /* Enable direct access to bdev pages */
> -
> - /* Deprecated mount options scheduled for removal */
> - {Opt_barrier, "barrier"}, /* use writer barriers for log write and
> - * unwritten extent conversion */
> - {Opt_nobarrier, "nobarrier"}, /* .. disable */
> -
> {Opt_err, NULL},
> };
>
> @@ -209,7 +203,6 @@ xfs_parseargs(
> * Set some default flags that could be cleared by the mount option
> * parsing.
> */
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> mp->m_flags |= XFS_MOUNT_COMPAT_IOSIZE;
>
> /*
> @@ -362,14 +355,6 @@ xfs_parseargs(
> mp->m_flags |= XFS_MOUNT_DAX;
> break;
> #endif
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> default:
> xfs_warn(mp, "unknown mount option [%s].", p);
> return -EINVAL;
> @@ -487,7 +472,6 @@ xfs_showargs(
> static struct proc_xfs_info xfs_info_unset[] = {
> /* the few simple ones we can get from the mount struct */
> { XFS_MOUNT_COMPAT_IOSIZE, ",largeio" },
> - { XFS_MOUNT_BARRIER, ",nobarrier" },
> { XFS_MOUNT_SMALL_INUMS, ",inode64" },
> { 0, NULL }
> };
> @@ -1322,14 +1306,6 @@ xfs_fs_remount(
>
> token = match_token(p, tokens, args);
> switch (token) {
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> case Opt_inode64:
> mp->m_flags &= ~XFS_MOUNT_SMALL_INUMS;
> mp->m_maxagi = xfs_set_inode_alloc(mp, sbp->sb_agcount);
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] xfs: remove deprecated barrier/nobarrier mount options
2018-07-26 16:01 [PATCH] " Darrick J. Wong
@ 2018-07-26 16:10 ` Eric Sandeen
2018-07-26 16:10 ` Darrick J. Wong
1 sibling, 0 replies; 10+ messages in thread
From: Eric Sandeen @ 2018-07-26 16:10 UTC (permalink / raw)
To: Darrick J. Wong, xfs
On 7/26/18 9:01 AM, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
>
> Remove the deprecated barrier and nobarrier mount options since they do
> nothing and were listed for removal some time around v4.15.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Looks at least as good as the one you reviewed from me on
6/26/18 ;)
Reviewed-by: Eric Sandeen <sandeen@redhat.com>
if you prefer this flavor ;)
> ---
> Documentation/filesystems/xfs.txt | 4 ++--
> fs/xfs/xfs_mount.h | 1 -
> fs/xfs/xfs_super.c | 26 +-------------------------
> 3 files changed, 3 insertions(+), 28 deletions(-)
>
> diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
> index 4d9ff0a7f8e1..a9ae82fb9d13 100644
> --- a/Documentation/filesystems/xfs.txt
> +++ b/Documentation/filesystems/xfs.txt
> @@ -223,8 +223,6 @@ Deprecated Mount Options
>
> Name Removal Schedule
> ---- ----------------
> - barrier no earlier than v4.15
> - nobarrier no earlier than v4.15
>
>
> Removed Mount Options
> @@ -236,6 +234,8 @@ Removed Mount Options
> ihashsize v4.0
> irixsgid v4.0
> osyncisdsync/osyncisosync v4.0
> + barrier v4.19
> + nobarrier v4.19
>
>
> sysctls
> diff --git a/fs/xfs/xfs_mount.h b/fs/xfs/xfs_mount.h
> index 16869d771b71..387b3868c66a 100644
> --- a/fs/xfs/xfs_mount.h
> +++ b/fs/xfs/xfs_mount.h
> @@ -229,7 +229,6 @@ typedef struct xfs_mount {
> #define XFS_MOUNT_SMALL_INUMS (1ULL << 14) /* user wants 32bit inodes */
> #define XFS_MOUNT_32BITINODES (1ULL << 15) /* inode32 allocator active */
> #define XFS_MOUNT_NOUUID (1ULL << 16) /* ignore uuid during mount */
> -#define XFS_MOUNT_BARRIER (1ULL << 17)
> #define XFS_MOUNT_IKEEP (1ULL << 18) /* keep empty inode clusters*/
> #define XFS_MOUNT_SWALLOC (1ULL << 19) /* turn on stripe width
> * allocation */
> diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
> index 9dd7df216fc3..afe59ce33876 100644
> --- a/fs/xfs/xfs_super.c
> +++ b/fs/xfs/xfs_super.c
> @@ -65,7 +65,7 @@ enum {
> Opt_logbufs, Opt_logbsize, Opt_logdev, Opt_rtdev, Opt_biosize,
> Opt_wsync, Opt_noalign, Opt_swalloc, Opt_sunit, Opt_swidth, Opt_nouuid,
> Opt_mtpt, Opt_grpid, Opt_nogrpid, Opt_bsdgroups, Opt_sysvgroups,
> - Opt_allocsize, Opt_norecovery, Opt_barrier, Opt_nobarrier,
> + Opt_allocsize, Opt_norecovery,
> Opt_inode64, Opt_inode32, Opt_ikeep, Opt_noikeep,
> Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2, Opt_filestreams,
> Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota, Opt_prjquota,
> @@ -120,12 +120,6 @@ static const match_table_t tokens = {
> {Opt_nodiscard, "nodiscard"}, /* Do not discard unused blocks */
>
> {Opt_dax, "dax"}, /* Enable direct access to bdev pages */
> -
> - /* Deprecated mount options scheduled for removal */
> - {Opt_barrier, "barrier"}, /* use writer barriers for log write and
> - * unwritten extent conversion */
> - {Opt_nobarrier, "nobarrier"}, /* .. disable */
> -
> {Opt_err, NULL},
> };
>
> @@ -209,7 +203,6 @@ xfs_parseargs(
> * Set some default flags that could be cleared by the mount option
> * parsing.
> */
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> mp->m_flags |= XFS_MOUNT_COMPAT_IOSIZE;
>
> /*
> @@ -362,14 +355,6 @@ xfs_parseargs(
> mp->m_flags |= XFS_MOUNT_DAX;
> break;
> #endif
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> default:
> xfs_warn(mp, "unknown mount option [%s].", p);
> return -EINVAL;
> @@ -487,7 +472,6 @@ xfs_showargs(
> static struct proc_xfs_info xfs_info_unset[] = {
> /* the few simple ones we can get from the mount struct */
> { XFS_MOUNT_COMPAT_IOSIZE, ",largeio" },
> - { XFS_MOUNT_BARRIER, ",nobarrier" },
> { XFS_MOUNT_SMALL_INUMS, ",inode64" },
> { 0, NULL }
> };
> @@ -1322,14 +1306,6 @@ xfs_fs_remount(
>
> token = match_token(p, tokens, args);
> switch (token) {
> - case Opt_barrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags |= XFS_MOUNT_BARRIER;
> - break;
> - case Opt_nobarrier:
> - xfs_warn(mp, "%s option is deprecated, ignoring.", p);
> - mp->m_flags &= ~XFS_MOUNT_BARRIER;
> - break;
> case Opt_inode64:
> mp->m_flags &= ~XFS_MOUNT_SMALL_INUMS;
> mp->m_maxagi = xfs_set_inode_alloc(mp, sbp->sb_agcount);
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] xfs: remove deprecated barrier/nobarrier mount options
@ 2018-07-26 16:01 Darrick J. Wong
2018-07-26 16:10 ` Eric Sandeen
2018-07-26 16:10 ` Darrick J. Wong
0 siblings, 2 replies; 10+ messages in thread
From: Darrick J. Wong @ 2018-07-26 16:01 UTC (permalink / raw)
To: xfs
From: Darrick J. Wong <darrick.wong@oracle.com>
Remove the deprecated barrier and nobarrier mount options since they do
nothing and were listed for removal some time around v4.15.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
---
Documentation/filesystems/xfs.txt | 4 ++--
fs/xfs/xfs_mount.h | 1 -
fs/xfs/xfs_super.c | 26 +-------------------------
3 files changed, 3 insertions(+), 28 deletions(-)
diff --git a/Documentation/filesystems/xfs.txt b/Documentation/filesystems/xfs.txt
index 4d9ff0a7f8e1..a9ae82fb9d13 100644
--- a/Documentation/filesystems/xfs.txt
+++ b/Documentation/filesystems/xfs.txt
@@ -223,8 +223,6 @@ Deprecated Mount Options
Name Removal Schedule
---- ----------------
- barrier no earlier than v4.15
- nobarrier no earlier than v4.15
Removed Mount Options
@@ -236,6 +234,8 @@ Removed Mount Options
ihashsize v4.0
irixsgid v4.0
osyncisdsync/osyncisosync v4.0
+ barrier v4.19
+ nobarrier v4.19
sysctls
diff --git a/fs/xfs/xfs_mount.h b/fs/xfs/xfs_mount.h
index 16869d771b71..387b3868c66a 100644
--- a/fs/xfs/xfs_mount.h
+++ b/fs/xfs/xfs_mount.h
@@ -229,7 +229,6 @@ typedef struct xfs_mount {
#define XFS_MOUNT_SMALL_INUMS (1ULL << 14) /* user wants 32bit inodes */
#define XFS_MOUNT_32BITINODES (1ULL << 15) /* inode32 allocator active */
#define XFS_MOUNT_NOUUID (1ULL << 16) /* ignore uuid during mount */
-#define XFS_MOUNT_BARRIER (1ULL << 17)
#define XFS_MOUNT_IKEEP (1ULL << 18) /* keep empty inode clusters*/
#define XFS_MOUNT_SWALLOC (1ULL << 19) /* turn on stripe width
* allocation */
diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
index 9dd7df216fc3..afe59ce33876 100644
--- a/fs/xfs/xfs_super.c
+++ b/fs/xfs/xfs_super.c
@@ -65,7 +65,7 @@ enum {
Opt_logbufs, Opt_logbsize, Opt_logdev, Opt_rtdev, Opt_biosize,
Opt_wsync, Opt_noalign, Opt_swalloc, Opt_sunit, Opt_swidth, Opt_nouuid,
Opt_mtpt, Opt_grpid, Opt_nogrpid, Opt_bsdgroups, Opt_sysvgroups,
- Opt_allocsize, Opt_norecovery, Opt_barrier, Opt_nobarrier,
+ Opt_allocsize, Opt_norecovery,
Opt_inode64, Opt_inode32, Opt_ikeep, Opt_noikeep,
Opt_largeio, Opt_nolargeio, Opt_attr2, Opt_noattr2, Opt_filestreams,
Opt_quota, Opt_noquota, Opt_usrquota, Opt_grpquota, Opt_prjquota,
@@ -120,12 +120,6 @@ static const match_table_t tokens = {
{Opt_nodiscard, "nodiscard"}, /* Do not discard unused blocks */
{Opt_dax, "dax"}, /* Enable direct access to bdev pages */
-
- /* Deprecated mount options scheduled for removal */
- {Opt_barrier, "barrier"}, /* use writer barriers for log write and
- * unwritten extent conversion */
- {Opt_nobarrier, "nobarrier"}, /* .. disable */
-
{Opt_err, NULL},
};
@@ -209,7 +203,6 @@ xfs_parseargs(
* Set some default flags that could be cleared by the mount option
* parsing.
*/
- mp->m_flags |= XFS_MOUNT_BARRIER;
mp->m_flags |= XFS_MOUNT_COMPAT_IOSIZE;
/*
@@ -362,14 +355,6 @@ xfs_parseargs(
mp->m_flags |= XFS_MOUNT_DAX;
break;
#endif
- case Opt_barrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags |= XFS_MOUNT_BARRIER;
- break;
- case Opt_nobarrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags &= ~XFS_MOUNT_BARRIER;
- break;
default:
xfs_warn(mp, "unknown mount option [%s].", p);
return -EINVAL;
@@ -487,7 +472,6 @@ xfs_showargs(
static struct proc_xfs_info xfs_info_unset[] = {
/* the few simple ones we can get from the mount struct */
{ XFS_MOUNT_COMPAT_IOSIZE, ",largeio" },
- { XFS_MOUNT_BARRIER, ",nobarrier" },
{ XFS_MOUNT_SMALL_INUMS, ",inode64" },
{ 0, NULL }
};
@@ -1322,14 +1306,6 @@ xfs_fs_remount(
token = match_token(p, tokens, args);
switch (token) {
- case Opt_barrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags |= XFS_MOUNT_BARRIER;
- break;
- case Opt_nobarrier:
- xfs_warn(mp, "%s option is deprecated, ignoring.", p);
- mp->m_flags &= ~XFS_MOUNT_BARRIER;
- break;
case Opt_inode64:
mp->m_flags &= ~XFS_MOUNT_SMALL_INUMS;
mp->m_maxagi = xfs_set_inode_alloc(mp, sbp->sb_agcount);
^ permalink raw reply related [flat|nested] 10+ messages in thread
end of thread, other threads:[~2018-07-26 17:28 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-26 0:03 [PATCH] xfs: remove deprecated barrier/nobarrier mount options Eric Sandeen
2018-06-26 0:16 ` Darrick J. Wong
2018-06-26 0:34 ` Eric Sandeen
2018-06-26 0:40 ` Darrick J. Wong
2018-06-26 0:43 ` [PATCH 4.19 V2] " Eric Sandeen
2018-06-26 11:23 ` Carlos Maiolino
2018-06-26 14:32 ` Darrick J. Wong
2018-07-26 16:01 [PATCH] " Darrick J. Wong
2018-07-26 16:10 ` Eric Sandeen
2018-07-26 16:10 ` Darrick J. Wong
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.