linux-ext4 - Re: [PATCH 3/3] ext2: remove dax EXPERIMENTAL warning

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20210922023622.GC570615@magnolia>
Date:   Tue, 21 Sep 2021 19:36:22 -0700
From:   "Darrick J. Wong" <djwong@...nel.org>
To:     Christoph Hellwig <hch@...radead.org>
Cc:     Jan Kara <jack@...e.cz>, Eric Sandeen <sandeen@...hat.com>,
        linux-xfs@...r.kernel.org, linux-ext4@...r.kernel.org,
        linux-fsdevel@...r.kernel.org, dan.j.williams@...el.com
Subject: Re: [PATCH 3/3] ext2: remove dax EXPERIMENTAL warning

On Fri, Sep 17, 2021 at 01:59:10PM +0100, Christoph Hellwig wrote:
> On Fri, Sep 17, 2021 at 11:47:07AM +0200, Jan Kara wrote:
> > On Wed 15-09-21 12:22:41, Eric Sandeen wrote:
> > > As there seems to be no significant outstanding concern about
> > > dax on ext2 at this point, remove the scary EXPERIMENTAL
> > > warning when in use.
> > > 
> > > Signed-off-by: Eric Sandeen <sandeen@...hat.com>
> > 
> > Agreed. Do you want my ack or should I just merge this patch?
> 
> Please do not merge it.  The whole DAX path is still a mess and should
> not be elevated to non-EXPERMINTAL state in this form.

Hi Christoph,

'still a mess' isn't all that useful for figuring out what still needs
to be done and splitting up the work.  Do you have items beyond my own
list below?

 - still arguing over what exactly FALLOC_FL_ZERO_REINIT_WHATEVER_PONIES
   should be doing
 - no reflink support, encompassing:
 - hwpoison from mmap regions really ought to tell the fs that bad stuff
   happened
 - mm rmap can't handle more than one owner

--D