From: Dan Williams Subject: Re: [PATCH 0/6 v3] dax: Page invalidation fixes Date: Tue, 13 Dec 2016 10:57:23 -0800 Message-ID: References: <20161212164708.23244-1-jack@suse.cz> <20161213115209.GG15362@quack2.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: "linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org" , Linux MM , Johannes Weiner , linux-fsdevel , linux-ext4 To: Jan Kara Return-path: In-Reply-To: <20161213115209.GG15362-4I4JzKEfoa/jFM9bn6wA6Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linux-nvdimm-bounces-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org Sender: "Linux-nvdimm" List-Id: linux-ext4.vger.kernel.org On Tue, Dec 13, 2016 at 3:52 AM, Jan Kara wrote: > On Mon 12-12-16 17:47:02, Jan Kara wrote: >> Hello, >> >> this is the third revision of my fixes of races when invalidating hole pages in >> DAX mappings. See changelogs for details. The series is based on my patches to >> write-protect DAX PTEs which are currently carried in mm tree. This is a hard >> dependency because we really need to closely track dirtiness (and cleanness!) >> of radix tree entries in DAX mappings in order to avoid discarding valid dirty >> bits leading to missed cache flushes on fsync(2). >> >> The tests have passed xfstests for xfs and ext4 in DAX and non-DAX mode. >> >> Johannes, are you OK with patch 2/6 in its current form? I'd like to push these >> patches to some tree once DAX write-protection patches are merged. I'm hoping >> to get at least first three patches merged for 4.10-rc2... Thanks! > > OK, with the final ack from Johannes and since this is mostly DAX stuff, > can we take this through NVDIMM tree and push to Linus either late in the > merge window or for -rc2? These patches require my DAX patches sitting in mm > tree so they can be included in any git tree only once those patches land > in Linus' tree (which may happen only once Dave and Ted push out their > stuff - this is the most convoluted merge window I'd ever to deal with ;-)... > Dan? > I like the -rc2 plan better than sending a pull request based on some random point in the middle of the merge window. I can give Linus a heads up in my initial nvdimm pull request for -rc1 that for coordination purposes we'll be sending this set of follow-on DAX cleanups for -rc2.