From: Josef Bacik Subject: Re: [PATCH 2/2] dm log writes: add support for DAX Date: Mon, 23 Oct 2017 13:34:09 -0400 Message-ID: <20171023173408.z6gog67bdp5s7jzy@destiny> References: <20171020052404.13762-1-ross.zwisler@linux.intel.com> <20171020052404.13762-2-ross.zwisler@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-kernel@vger.kernel.org, Josef Bacik , Alasdair Kergon , Dan Williams , Dave Chinner , Jan Kara , Mike Snitzer , Shaohua Li , dm-devel@redhat.com, linux-nvdimm@lists.01.org, linux-raid@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org, linux-xfs@vger.kernel.org, Christoph Hellwig To: Ross Zwisler Return-path: Received: from mail-qk0-f193.google.com ([209.85.220.193]:52987 "EHLO mail-qk0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751449AbdJWReM (ORCPT ); Mon, 23 Oct 2017 13:34:12 -0400 Received: by mail-qk0-f193.google.com with SMTP id b15so22961992qkg.9 for ; Mon, 23 Oct 2017 10:34:11 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20171020052404.13762-2-ross.zwisler@linux.intel.com> Sender: linux-ext4-owner@vger.kernel.org List-ID: On Thu, Oct 19, 2017 at 11:24:04PM -0600, Ross Zwisler 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 > --- Ok this is just my ignorance of how DAX works shining through, but do we need a new flag to indicate this is DAX data? You are logging it like it's just normal data going to a certain sector, is that good enough? If it is then hooray this looks fine to me, I'm just slightly confused. Thanks, Josef