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
| ||
|
Message-ID: <20171024193022.GA4251@linux.intel.com> Date: Tue, 24 Oct 2017 13:30:22 -0600 From: Ross Zwisler <ross.zwisler@...ux.intel.com> To: Mike Snitzer <snitzer@...hat.com> Cc: Ross Zwisler <ross.zwisler@...ux.intel.com>, linux-kernel@...r.kernel.org, Josef Bacik <jbacik@...com>, Alasdair Kergon <agk@...hat.com>, Dan Williams <dan.j.williams@...el.com>, Dave Chinner <david@...morbit.com>, Jan Kara <jack@...e.cz>, Shaohua Li <shli@...nel.org>, dm-devel@...hat.com, linux-nvdimm@...ts.01.org, linux-raid@...r.kernel.org, linux-fsdevel@...r.kernel.org, linux-ext4@...r.kernel.org, linux-xfs@...r.kernel.org, Christoph Hellwig <hch@...radead.org> Subject: Re: [PATCH 2/2] dm log writes: add support for DAX On Tue, Oct 24, 2017 at 03:22:23PM -0400, Mike Snitzer wrote: > On Fri, Oct 20 2017 at 1:24am -0400, > Ross Zwisler <ross.zwisler@...ux.intel.com> wrote: > > > Now that we have the ability log filesystem writes using a flat buffer, add > > support for DAX. Unfortunately we can't easily track data that has been > > written via mmap() now that the dax_flush() abstraction was removed by this > > commit: > > > > commit c3ca015fab6d ("dax: remove the pmem_dax_ops->flush abstraction") > > > > Otherwise we could just treat each flush as a big write, and store the data > > that is being synced to media. It may be worthwhile to add the dax_flush() > > entry point back, just as a notifier so we can do this logging. > > > > The motivation for this support is the need for an xfstest that can test > > the new MAP_SYNC DAX flag. By logging the filesystem activity with > > dm-log-writes we can show that the MAP_SYNC page faults are writing out > > their metadata as they happen, instead of requiring an explicit > > msync/fsync. > > > > Signed-off-by: Ross Zwisler <ross.zwisler@...ux.intel.com> > > I've picked this up, please see: > https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=for-4.15/dm&id=ae613bbb0144e84cb3c0ebfa9f4fd4d1507c2f0e > > I tweaked the header and tweaked a couple whitespace nits. Also > switched version bump from 1.0.1 to 1.1.0. > > Thanks, > Mike Sure, your tweaks look fine. Thanks!
Powered by blists - more mailing lists