Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965017AbXAJSFp (ORCPT ); Wed, 10 Jan 2007 13:05:45 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965015AbXAJSFp (ORCPT ); Wed, 10 Jan 2007 13:05:45 -0500 Received: from 3a.49.1343.static.theplanet.com ([67.19.73.58]:47592 "EHLO pug.o-hand.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965013AbXAJSFo (ORCPT ); Wed, 10 Jan 2007 13:05:44 -0500 Subject: [PATCH 3/4] swap: Catch pages with errors and mark as bad From: Richard Purdie To: Nick Piggin , Hugh Dickins , kernel list Cc: Andrew Morton Content-Type: text/plain Date: Wed, 10 Jan 2007 18:05:38 +0000 Message-Id: <1168452339.5801.61.camel@localhost.localdomain> Mime-Version: 1.0 X-Mailer: Evolution 2.6.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2242 Lines: 63 Check for pages with errors within shrink_page_list() and if found, try to unuse them which will trigger the functions to mark the page bad. The page will then be allocated a new swap page. If a swap page write error occurs, don't disable page reclaim. Based on a patch by Nick Piggin and some of my own changes as discussed on LKML. Signed-off-by: Richard Purdie --- mm/page_io.c | 3 --- mm/vmscan.c | 9 +++++++++ 2 files changed, 9 insertions(+), 3 deletions(-) Index: git/mm/page_io.c =================================================================== --- git.orig/mm/page_io.c 2007-01-07 21:39:29.000000000 +0000 +++ git/mm/page_io.c 2007-01-08 11:40:38.000000000 +0000 @@ -59,15 +59,12 @@ static int end_swap_bio_write(struct bio * Re-dirty the page in order to avoid it being reclaimed. * Also print a dire warning that things will go BAD (tm) * very quickly. - * - * Also clear PG_reclaim to avoid rotate_reclaimable_page() */ set_page_dirty(page); printk(KERN_ALERT "Write-error on swap-device (%u:%u:%Lu)\n", imajor(bio->bi_bdev->bd_inode), iminor(bio->bi_bdev->bd_inode), (unsigned long long)bio->bi_sector); - ClearPageReclaim(page); } end_page_writeback(page); bio_put(bio); Index: git/mm/vmscan.c =================================================================== --- git.orig/mm/vmscan.c 2007-01-07 21:39:29.000000000 +0000 +++ git/mm/vmscan.c 2007-01-08 11:40:00.000000000 +0000 @@ -490,6 +490,15 @@ static unsigned long shrink_page_list(st #ifdef CONFIG_SWAP /* + * Encountered an error last time? Try to remove the + * page from its current position, which will notice + * the error and mark that swap entry bad. Then we can + * try allocating another swap entry. + */ + if (PageSwapCache(page) && unlikely(PageError(page))) + try_to_unuse_page_entry(page); + + /* * Anonymous process memory has backing store? * Try to allocate it some swap space here. */ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/