From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Layton Subject: OOM kills when running fsstress on CIFS Date: Tue, 25 May 2010 06:57:05 -0400 Message-ID: <20100525065705.5a5d95e1@corrin.poochiereds.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, linux-cifs-client@lists.samba.org To: Nick Piggin , Steve French , Dave Kleikamp Return-path: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-cifs-client-bounces@lists.samba.org Errors-To: linux-cifs-client-bounces@lists.samba.org List-Id: linux-fsdevel.vger.kernel.org Since 2.6.34, I've been able to consistently reproduce OOM kills when running fsstress (from the LTP suite) on CIFS. I spent some time yesterday and bisected it down to this patch: ---------------------[snip]--------------------- commit 315e995c63a15cb4d4efdbfd70fe2db191917f7a Author: Nick Piggin Date: Wed Apr 21 03:18:28 2010 +0000 [CIFS] use add_to_page_cache_lru add_to_page_cache_lru is exported, so it should be used. Benefits over using a private pagevec: neater code, 128 bytes fewer stack used, percpu lru ordering is preserved, and finally don't need to flush pagevec before returning so batching may be shared with other LRU insertions. Signed-off-by: Nick Piggin Reviewed-by: Dave Kleikamp Signed-off-by: Steve French ---------------------[snip]--------------------- Here's how I've been reproducing it: Mount up a samba share with -o sec=krb5i,nounix,noserverino Run: fsstress -d /path/to/dir/on/cifs/ -n 1000 -l0 -p8 ...within an hour or two, I start getting OOM kills. After backing out the patch above, I was able to run the test overnight. I'm not sure yet what the actual problem is, but there seems to be something wrong with that patch. Thoughts? -- Jeff Layton