Re: [PATCH V7 3/9] fs/stat: Define DAX statx attribute

From: Ira Weiny
Date: Tue Apr 14 2020 - 16:39:26 EST


On Tue, Apr 14, 2020 at 08:23:06AM +0200, Christoph Hellwig wrote:
> On Sun, Apr 12, 2020 at 10:40:40PM -0700, ira.weiny@xxxxxxxxx wrote:
> > STATX_ATTR_DAX
> >
> > The file is in the DAX (cpu direct access) state. DAX state
> > attempts to minimize software cache effects for both I/O and
> > memory mappings of this file. It requires a file system which
> > has been configured to support DAX.
>
> Can we remove the misleading DAX name? Something like
> STATX_ATTR_DIRECT_LOAD_STORE?

This is easy enough to change but...

Honestly I feel like this ship has already sailed. We have so much out there
which uses the term "DAX". Is it really better to introduce a new terminology
for the same thing?

Ira