* [PATCH v2] Documentation/dax: Update description of DAX policy changing
@ 2021-01-06 1:50 Hao Li
2021-01-06 5:45 ` Ira Weiny
0 siblings, 1 reply; 3+ messages in thread
From: Hao Li @ 2021-01-06 1:50 UTC (permalink / raw)
To: corbet
Cc: davem, gregkh, alexander.deucher, mchehab+huawei, lee.jones,
ira.weiny, linux-doc, linux-kernel, lihao2018.fnst
After commit 77573fa310d9 ("fs: Kill DCACHE_DONTCACHE dentry even if
DCACHE_REFERENCED is set"), changes to DAX policy will take effect
as soon as all references to this file are gone.
Update the documentation accordingly.
Signed-off-by: Hao Li <lihao2018.fnst@cn.fujitsu.com>
---
Changes in v2:
* simplify sentences and fix style problems.
Documentation/filesystems/dax.txt | 17 +++--------------
1 file changed, 3 insertions(+), 14 deletions(-)
diff --git a/Documentation/filesystems/dax.txt b/Documentation/filesystems/dax.txt
index 8fdb78f3c6c9..e03c20564f3a 100644
--- a/Documentation/filesystems/dax.txt
+++ b/Documentation/filesystems/dax.txt
@@ -83,20 +83,9 @@ Summary
directories. This has runtime constraints and limitations that are
described in 6) below.
- 6. When changing the S_DAX policy via toggling the persistent FS_XFLAG_DAX flag,
- the change in behaviour for existing regular files may not occur
- immediately. If the change must take effect immediately, the administrator
- needs to:
-
- a) stop the application so there are no active references to the data set
- the policy change will affect
-
- b) evict the data set from kernel caches so it will be re-instantiated when
- the application is restarted. This can be achieved by:
-
- i. drop-caches
- ii. a filesystem unmount and mount cycle
- iii. a system reboot
+ 6. When changing the S_DAX policy via toggling the persistent FS_XFLAG_DAX
+ flag, the change to existing regular files won't take effect until the
+ files are closed by all processes.
Details
--
2.29.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] Documentation/dax: Update description of DAX policy changing
2021-01-06 1:50 [PATCH v2] Documentation/dax: Update description of DAX policy changing Hao Li
@ 2021-01-06 5:45 ` Ira Weiny
2021-01-11 19:59 ` Jonathan Corbet
0 siblings, 1 reply; 3+ messages in thread
From: Ira Weiny @ 2021-01-06 5:45 UTC (permalink / raw)
To: Hao Li
Cc: corbet, davem, gregkh, alexander.deucher, mchehab+huawei,
lee.jones, linux-doc, linux-kernel
On Wed, Jan 06, 2021 at 09:50:00AM +0800, Hao Li wrote:
> After commit 77573fa310d9 ("fs: Kill DCACHE_DONTCACHE dentry even if
> DCACHE_REFERENCED is set"), changes to DAX policy will take effect
> as soon as all references to this file are gone.
>
> Update the documentation accordingly.
>
> Signed-off-by: Hao Li <lihao2018.fnst@cn.fujitsu.com>
LGTM
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
> ---
> Changes in v2:
> * simplify sentences and fix style problems.
>
> Documentation/filesystems/dax.txt | 17 +++--------------
> 1 file changed, 3 insertions(+), 14 deletions(-)
>
> diff --git a/Documentation/filesystems/dax.txt b/Documentation/filesystems/dax.txt
> index 8fdb78f3c6c9..e03c20564f3a 100644
> --- a/Documentation/filesystems/dax.txt
> +++ b/Documentation/filesystems/dax.txt
> @@ -83,20 +83,9 @@ Summary
> directories. This has runtime constraints and limitations that are
> described in 6) below.
>
> - 6. When changing the S_DAX policy via toggling the persistent FS_XFLAG_DAX flag,
> - the change in behaviour for existing regular files may not occur
> - immediately. If the change must take effect immediately, the administrator
> - needs to:
> -
> - a) stop the application so there are no active references to the data set
> - the policy change will affect
> -
> - b) evict the data set from kernel caches so it will be re-instantiated when
> - the application is restarted. This can be achieved by:
> -
> - i. drop-caches
> - ii. a filesystem unmount and mount cycle
> - iii. a system reboot
> + 6. When changing the S_DAX policy via toggling the persistent FS_XFLAG_DAX
> + flag, the change to existing regular files won't take effect until the
> + files are closed by all processes.
>
>
> Details
> --
> 2.29.2
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] Documentation/dax: Update description of DAX policy changing
2021-01-06 5:45 ` Ira Weiny
@ 2021-01-11 19:59 ` Jonathan Corbet
0 siblings, 0 replies; 3+ messages in thread
From: Jonathan Corbet @ 2021-01-11 19:59 UTC (permalink / raw)
To: Ira Weiny
Cc: Hao Li, davem, gregkh, alexander.deucher, mchehab+huawei,
lee.jones, linux-doc, linux-kernel
On Tue, 5 Jan 2021 21:45:41 -0800
Ira Weiny <ira.weiny@intel.com> wrote:
> On Wed, Jan 06, 2021 at 09:50:00AM +0800, Hao Li wrote:
> > After commit 77573fa310d9 ("fs: Kill DCACHE_DONTCACHE dentry even if
> > DCACHE_REFERENCED is set"), changes to DAX policy will take effect
> > as soon as all references to this file are gone.
> >
> > Update the documentation accordingly.
> >
> > Signed-off-by: Hao Li <lihao2018.fnst@cn.fujitsu.com>
>
> LGTM
>
> Reviewed-by: Ira Weiny <ira.weiny@intel.com>
Applied, thanks.
jon
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-01-11 20:00 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-06 1:50 [PATCH v2] Documentation/dax: Update description of DAX policy changing Hao Li
2021-01-06 5:45 ` Ira Weiny
2021-01-11 19:59 ` Jonathan Corbet
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.