From: Ira Weiny <ira.weiny@intel.com> To: JeffleXu <jefflexu@linux.alibaba.com> Cc: Vivek Goyal <vgoyal@redhat.com>, Dave Chinner <dchinner@redhat.com>, stefanha@redhat.com, miklos@szeredi.hu, linux-fsdevel@vger.kernel.org, virtio-fs@redhat.com, bo.liu@linux.alibaba.com, joseph.qi@linux.alibaba.com Subject: Re: [PATCH v6 2/7] fuse: make DAX mount option a tri-state Date: Mon, 25 Oct 2021 10:52:51 -0700 [thread overview] Message-ID: <20211025175251.GF3465596@iweiny-DESK2.sc.intel.com> (raw) In-Reply-To: <96956132-fced-5739-d69a-7b424dc65f7c@linux.alibaba.com> On Fri, Oct 22, 2021 at 02:54:03PM +0800, JeffleXu wrote: > cc [Ira Weiny], author of per inode DAX on xfs/ext4 > > On 10/20/21 11:17 PM, Vivek Goyal wrote: > > On Wed, Oct 20, 2021 at 10:52:38AM +0800, JeffleXu wrote: > >> > >> > >> On 10/18/21 10:10 PM, Vivek Goyal wrote: > >>> On Mon, Oct 11, 2021 at 11:00:47AM +0800, Jeffle Xu wrote: > >>>> We add 'always', 'never', and 'inode' (default). '-o dax' continues to > >>>> operate the same which is equivalent to 'always'. To be consistemt with > >>>> ext4/xfs's tri-state mount option, when neither '-o dax' nor '-o dax=' > >>>> option is specified, the default behaviour is equal to 'inode'. > >>> > >>> Hi Jeffle, > >>> > >>> I am not sure when -o "dax=inode" is used as a default? If user > >>> specifies, "-o dax" then it is equal to "-o dax=always", otherwise > >>> user will explicitly specify "-o dax=always/never/inode". So when > >>> is dax=inode is used as default? > >> > >> That means when neither '-o dax' nor '-o dax=always/never/inode' is > >> specified, it is actually equal to '-o dax=inode', which is also how > >> per-file DAX on ext4/xfs works. > > It's been a while so I'm fuzzy on the details of the discussions but yes that is the way things are now in the code. > > [ CC dave chinner] > > > > Is it not change of default behavior for ext4/xfs as well. My > > understanding is that prior to this new dax options, "-o dax" enabled > > dax on filesystem and if user did not specify it, DAX is disbaled > > by default. Technically it does change default behavior... However, NOT in a way which breaks anything. See below. > > > > Now after introduction of "-o dax=always/never/inode", if suddenly > > "-o dax=inode" became the default if user did not specify anything, > > that's change of behavior. Technically yes but not in a broken way. > > > > Is that intentional. If given a choice, > > I would rather not change default and ask user to opt-in for > > appropriate dax functionality. There is no need for this. > > > > Dave, you might have thoughts on this. It makes me uncomfortable to > > change virtiofs dax default now just because other filesytems did it. > > > > I can only find the following discussions about the earliest record on > this tri-state mount option: > > https://lore.kernel.org/lkml/20200316095509.GA13788@lst.de/ > https://lore.kernel.org/lkml/20200401040021.GC56958@magnolia/ > > > Hi, Ira Weiny, > > Do you have any thought on this, i.e. why the default behavior has > changed after introduction of per inode dax? While this is 'technically' different behavior the end user does not see any difference in behavior if they continue without software changes. Specifically specifying nothing continues to operate with all the files on the FS to be '_not_ DAX'. While specifying '-o dax' forces DAX on all files. This expands the default behavior in a backwards compatible manner. The user can now enable DAX on some files. But this is an opt-in on the part of the user of the FS and again does not change with existing software/scripts/etc. Does that make sense? Ira
WARNING: multiple messages have this Message-ID (diff)
From: Ira Weiny <ira.weiny@intel.com> To: JeffleXu <jefflexu@linux.alibaba.com> Cc: miklos@szeredi.hu, virtio-fs@redhat.com, joseph.qi@linux.alibaba.com, Dave Chinner <dchinner@redhat.com>, linux-fsdevel@vger.kernel.org, Vivek Goyal <vgoyal@redhat.com> Subject: Re: [Virtio-fs] [PATCH v6 2/7] fuse: make DAX mount option a tri-state Date: Mon, 25 Oct 2021 10:52:51 -0700 [thread overview] Message-ID: <20211025175251.GF3465596@iweiny-DESK2.sc.intel.com> (raw) In-Reply-To: <96956132-fced-5739-d69a-7b424dc65f7c@linux.alibaba.com> On Fri, Oct 22, 2021 at 02:54:03PM +0800, JeffleXu wrote: > cc [Ira Weiny], author of per inode DAX on xfs/ext4 > > On 10/20/21 11:17 PM, Vivek Goyal wrote: > > On Wed, Oct 20, 2021 at 10:52:38AM +0800, JeffleXu wrote: > >> > >> > >> On 10/18/21 10:10 PM, Vivek Goyal wrote: > >>> On Mon, Oct 11, 2021 at 11:00:47AM +0800, Jeffle Xu wrote: > >>>> We add 'always', 'never', and 'inode' (default). '-o dax' continues to > >>>> operate the same which is equivalent to 'always'. To be consistemt with > >>>> ext4/xfs's tri-state mount option, when neither '-o dax' nor '-o dax=' > >>>> option is specified, the default behaviour is equal to 'inode'. > >>> > >>> Hi Jeffle, > >>> > >>> I am not sure when -o "dax=inode" is used as a default? If user > >>> specifies, "-o dax" then it is equal to "-o dax=always", otherwise > >>> user will explicitly specify "-o dax=always/never/inode". So when > >>> is dax=inode is used as default? > >> > >> That means when neither '-o dax' nor '-o dax=always/never/inode' is > >> specified, it is actually equal to '-o dax=inode', which is also how > >> per-file DAX on ext4/xfs works. > > It's been a while so I'm fuzzy on the details of the discussions but yes that is the way things are now in the code. > > [ CC dave chinner] > > > > Is it not change of default behavior for ext4/xfs as well. My > > understanding is that prior to this new dax options, "-o dax" enabled > > dax on filesystem and if user did not specify it, DAX is disbaled > > by default. Technically it does change default behavior... However, NOT in a way which breaks anything. See below. > > > > Now after introduction of "-o dax=always/never/inode", if suddenly > > "-o dax=inode" became the default if user did not specify anything, > > that's change of behavior. Technically yes but not in a broken way. > > > > Is that intentional. If given a choice, > > I would rather not change default and ask user to opt-in for > > appropriate dax functionality. There is no need for this. > > > > Dave, you might have thoughts on this. It makes me uncomfortable to > > change virtiofs dax default now just because other filesytems did it. > > > > I can only find the following discussions about the earliest record on > this tri-state mount option: > > https://lore.kernel.org/lkml/20200316095509.GA13788@lst.de/ > https://lore.kernel.org/lkml/20200401040021.GC56958@magnolia/ > > > Hi, Ira Weiny, > > Do you have any thought on this, i.e. why the default behavior has > changed after introduction of per inode dax? While this is 'technically' different behavior the end user does not see any difference in behavior if they continue without software changes. Specifically specifying nothing continues to operate with all the files on the FS to be '_not_ DAX'. While specifying '-o dax' forces DAX on all files. This expands the default behavior in a backwards compatible manner. The user can now enable DAX on some files. But this is an opt-in on the part of the user of the FS and again does not change with existing software/scripts/etc. Does that make sense? Ira
next prev parent reply other threads:[~2021-10-25 17:52 UTC|newest] Thread overview: 74+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-10-11 3:00 [PATCH v6 0/7] fuse,virtiofs: support per-file DAX Jeffle Xu 2021-10-11 3:00 ` [Virtio-fs] " Jeffle Xu 2021-10-11 3:00 ` [PATCH v6 1/7] fuse: add fuse_should_enable_dax() helper Jeffle Xu 2021-10-11 3:00 ` [Virtio-fs] " Jeffle Xu 2021-10-11 3:00 ` [PATCH v6 2/7] fuse: make DAX mount option a tri-state Jeffle Xu 2021-10-11 3:00 ` [Virtio-fs] " Jeffle Xu 2021-10-18 14:10 ` Vivek Goyal 2021-10-18 14:10 ` [Virtio-fs] " Vivek Goyal 2021-10-20 2:52 ` JeffleXu 2021-10-20 2:52 ` [Virtio-fs] " JeffleXu 2021-10-20 14:48 ` Vivek Goyal 2021-10-20 14:48 ` [Virtio-fs] " Vivek Goyal 2021-10-29 8:33 ` JeffleXu 2021-10-29 8:33 ` [Virtio-fs] " JeffleXu 2021-10-29 13:03 ` Vivek Goyal 2021-10-29 13:03 ` [Virtio-fs] " Vivek Goyal 2021-11-01 8:21 ` JeffleXu 2021-11-01 8:21 ` [Virtio-fs] " JeffleXu 2021-10-20 15:17 ` Vivek Goyal 2021-10-20 15:17 ` [Virtio-fs] " Vivek Goyal 2021-10-22 6:54 ` JeffleXu 2021-10-22 6:54 ` [Virtio-fs] " JeffleXu 2021-10-25 17:52 ` Ira Weiny [this message] 2021-10-25 17:52 ` Ira Weiny 2021-10-25 18:12 ` Vivek Goyal 2021-10-25 18:12 ` [Virtio-fs] " Vivek Goyal 2021-10-25 19:02 ` Ira Weiny 2021-10-25 19:02 ` [Virtio-fs] " Ira Weiny 2021-10-25 19:33 ` Vivek Goyal 2021-10-25 19:33 ` [Virtio-fs] " Vivek Goyal 2021-10-25 20:41 ` Ira Weiny 2021-10-25 20:41 ` [Virtio-fs] " Ira Weiny 2021-10-26 13:45 ` Vivek Goyal 2021-10-26 13:45 ` [Virtio-fs] " Vivek Goyal 2021-10-27 6:00 ` JeffleXu 2021-10-27 6:00 ` [Virtio-fs] " JeffleXu 2021-10-11 3:00 ` [PATCH v6 3/7] fuse: support per-file DAX in fuse protocol Jeffle Xu 2021-10-11 3:00 ` [Virtio-fs] " Jeffle Xu 2021-10-18 14:14 ` Vivek Goyal 2021-10-18 14:14 ` [Virtio-fs] " Vivek Goyal 2021-10-18 14:20 ` Vivek Goyal 2021-10-18 14:20 ` [Virtio-fs] " Vivek Goyal 2021-10-20 3:04 ` JeffleXu 2021-10-20 3:04 ` [Virtio-fs] " JeffleXu 2021-10-20 14:54 ` Vivek Goyal 2021-10-20 14:54 ` [Virtio-fs] " Vivek Goyal 2021-10-11 3:00 ` [PATCH v6 4/7] fuse: negotiate per-file DAX in FUSE_INIT Jeffle Xu 2021-10-11 3:00 ` [Virtio-fs] " Jeffle Xu 2021-10-18 14:30 ` Vivek Goyal 2021-10-18 14:30 ` [Virtio-fs] " Vivek Goyal 2021-10-20 3:10 ` JeffleXu 2021-10-20 3:10 ` [Virtio-fs] " JeffleXu 2021-10-20 15:44 ` Vivek Goyal 2021-10-20 15:44 ` [Virtio-fs] " Vivek Goyal 2021-10-11 3:00 ` [PATCH v6 5/7] fuse: enable per-file DAX Jeffle Xu 2021-10-11 3:00 ` [Virtio-fs] " Jeffle Xu 2021-10-18 15:11 ` Vivek Goyal 2021-10-18 15:11 ` [Virtio-fs] " Vivek Goyal 2021-10-11 3:00 ` [PATCH v6 6/7] fuse: mark inode DONT_CACHE when per-file DAX hint changes Jeffle Xu 2021-10-11 3:00 ` [Virtio-fs] " Jeffle Xu 2021-10-18 15:19 ` Vivek Goyal 2021-10-18 15:19 ` [Virtio-fs] " Vivek Goyal 2021-10-27 5:05 ` JeffleXu 2021-10-27 5:05 ` [Virtio-fs] " JeffleXu 2021-10-11 3:00 ` [PATCH v6 7/7] Documentation/filesystem/dax: record DAX on virtiofs Jeffle Xu 2021-10-11 3:00 ` [Virtio-fs] " Jeffle Xu 2021-10-15 3:33 ` [PATCH v6 0/7] fuse,virtiofs: support per-file DAX JeffleXu 2021-10-15 3:33 ` [Virtio-fs] " JeffleXu 2021-10-18 15:21 ` Vivek Goyal 2021-10-18 15:21 ` [Virtio-fs] " Vivek Goyal 2021-10-20 5:22 ` JeffleXu 2021-10-20 5:22 ` [Virtio-fs] " JeffleXu 2021-10-20 16:06 ` Vivek Goyal 2021-10-20 16:06 ` [Virtio-fs] " Vivek Goyal
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20211025175251.GF3465596@iweiny-DESK2.sc.intel.com \ --to=ira.weiny@intel.com \ --cc=bo.liu@linux.alibaba.com \ --cc=dchinner@redhat.com \ --cc=jefflexu@linux.alibaba.com \ --cc=joseph.qi@linux.alibaba.com \ --cc=linux-fsdevel@vger.kernel.org \ --cc=miklos@szeredi.hu \ --cc=stefanha@redhat.com \ --cc=vgoyal@redhat.com \ --cc=virtio-fs@redhat.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.