From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Chinner Subject: Re: [PATCH v5] mm, dax: fix DAX deadlocks Date: Tue, 13 Oct 2015 09:53:27 +1100 Message-ID: <20151012225327.GF27164@dastard> References: <1444258729-21974-1-git-send-email-ross.zwisler@linux.intel.com> <1444258729-21974-2-git-send-email-ross.zwisler@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-kernel@vger.kernel.org, Alexander Viro , Matthew Wilcox , linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, Andrew Morton , Dan Williams , Jan Kara , "Kirill A. Shutemov" , linux-nvdimm@lists.01.org, Matthew Wilcox To: Ross Zwisler Return-path: Content-Disposition: inline In-Reply-To: <1444258729-21974-2-git-send-email-ross.zwisler@linux.intel.com> Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On Wed, Oct 07, 2015 at 04:58:49PM -0600, Ross Zwisler wrote: > The following two locking commits in the DAX code: > > commit 843172978bb9 ("dax: fix race between simultaneous faults") > commit 46c043ede471 ("mm: take i_mmap_lock in unmap_mapping_range() for DAX") > > introduced a number of deadlocks and other issues which need to be fixed > for the v4.3 kernel. The list of issues in DAX after these commits (some > newly introduced by the commits, some preexisting) can be found here: > > https://lkml.org/lkml/2015/9/25/602 > > This undoes most of the changes introduced by those two commits, > essentially returning us to the DAX locking scheme that was used in v4.2. > > Signed-off-by: Ross Zwisler I've run this through some testing, the deadlocks aren't present and there don't appear to be any new regressions, so IMO this is fine to go to Linus. Tested-by: Dave Chinner Cheers, Dave. -- Dave Chinner david@fromorbit.com -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org