From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nick Piggin Subject: [patch 5/9] mm: debug write deadlocks Date: Sun, 4 Feb 2007 09:50:28 +0100 (CET) Message-ID: <20070204063754.23659.28403.sendpatchset@linux.site> References: <20070204063707.23659.20741.sendpatchset@linux.site> Cc: Linux Kernel , Linux Filesystems , Nick Piggin , Linux Memory Management To: Andrew Morton Return-path: Received: from mx2.suse.de ([195.135.220.15]:40973 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752191AbXBDIuc (ORCPT ); Sun, 4 Feb 2007 03:50:32 -0500 In-Reply-To: <20070204063707.23659.20741.sendpatchset@linux.site> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org Allow CONFIG_DEBUG_VM to switch off the prefaulting logic, to simulate the difficult race where the page may be unmapped before calling copy_from_user. Makes the race much easier to hit. This is useful for demonstration and testing purposes, but is removed in a subsequent patch. Signed-off-by: Nick Piggin Index: linux-2.6/mm/filemap.c =================================================================== --- linux-2.6.orig/mm/filemap.c +++ linux-2.6/mm/filemap.c @@ -2103,6 +2103,7 @@ generic_file_buffered_write(struct kiocb if (maxlen > bytes) maxlen = bytes; +#ifndef CONFIG_DEBUG_VM /* * Bring in the user page that we will copy from _first_. * Otherwise there's a nasty deadlock on copying from the @@ -2110,6 +2111,7 @@ generic_file_buffered_write(struct kiocb * up-to-date. */ fault_in_pages_readable(buf, maxlen); +#endif page = __grab_cache_page(mapping,index,&cached_page,&lru_pvec); if (!page) {