This patchset doesn't have a cover letter, and it doesn't apply to f2fs/dev, so I'm not sure what its purpose is.
On Tue, Nov 17, 2020 at 01:02:22PM +0900, Daeho Jeong wrote: > From: Daeho Jeong <[email protected]> > > We will add a new disable_auto_compr mount option to turn off the > automaic compression on the compression enabled file, in order to > give discretion of choosing the target file and the timing of > compression to the user. > > Signed-off-by: Daeho Jeong <[email protected]> > --- > Documentation/filesystems/f2fs.rst | 4 ++++ > fs/f2fs/data.c | 10 +++++----- > fs/f2fs/f2fs.h | 8 ++++++++ > fs/f2fs/segment.c | 2 +- > fs/f2fs/super.c | 9 +++++++++ > 5 files changed, 27 insertions(+), 6 deletions(-) > > diff --git a/Documentation/filesystems/f2fs.rst > b/Documentation/filesystems/f2fs.rst > index b8ee761c9922..c36ca1e0f7ab 100644 > --- a/Documentation/filesystems/f2fs.rst > +++ b/Documentation/filesystems/f2fs.rst > @@ -260,6 +260,10 @@ compress_extension=%s Support adding specified > extension, so that f2fs can enab > For other files, we can still enable compression via > ioctl. > Note that, there is one reserved special extension > '*', it > can be set to enable compression for all files. > +disable_auto_compr Even if compression feature is enabled, this option > can > + disable automaic compression on the compression > enabled > + file to give discretion of choosing the target file > and > + the timing of compression to the user. I don't understand. How is this different from just not specifying the compress_extension mount option? And how can a file have compression both enabled and disabled at the same time? - Eric
