Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752781AbZIERWe (ORCPT ); Sat, 5 Sep 2009 13:22:34 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752576AbZIERWc (ORCPT ); Sat, 5 Sep 2009 13:22:32 -0400 Received: from mail.parknet.ad.jp ([210.171.162.6]:36467 "EHLO mail.officemail.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752555AbZIERWb (ORCPT ); Sat, 5 Sep 2009 13:22:31 -0400 From: OGAWA Hirofumi To: Zdenek Kabelac Cc: Christoph Hellwig , "Rafael J. Wysocki" , Linux Kernel Mailing List , linux-mmc@vger.kernel.org, viro@zeniv.linux.org.uk Subject: Re: Regression in suspend to ram in 2.6.31-rc kernels References: <200908312119.12121.rjw@sisk.pl> <20090903232317.GA6760@lst.de> <87ljkvmt71.fsf@devron.myhome.or.jp> Date: Sun, 06 Sep 2009 02:22:27 +0900 In-Reply-To: (Zdenek Kabelac's message of "Fri, 4 Sep 2009 11:13:02 +0200") Message-ID: <87iqfx5mss.fsf@devron.myhome.or.jp> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4154 Lines: 125 Zdenek Kabelac writes: > 2009/9/4 OGAWA Hirofumi : >> Christoph Hellwig writes: >> >>> Note that when you rever this patch on a current kernel you do actually >>> get different behvaviour than when going back to before this commit. >>> >>> In 2.6.30 we called ->write_super in the various sync functions and >>> then ->sync_fs, in 2.6.31-rc8 you would not call any syncing at all >>> anymore. ?I think this patch might just be a symptom for a situation >>> where the suspend code causes a sync and the mmc driver can't handle >>> it anymore. > > So - here is the console trace from suspend when I've added > dump_stack() to the fat_sync_fs() (and also added debug prints > around each call in this function -so its obvious the function is > actually left - but then it freezes later somewhere.) > > It's interesting that 3 calls to sync happens. It seems 1) sync() (probabry "sync" command) 2) sync as part of suspend sequence 3) sync_filesystem() by mmc remove event I guess the root-cause of the problem would be 3). However, it would not be easy to fix, at least, we would need to think about what we want to do for it. So, to workaround it for now, I've made this patch. Can you try whether this patch fixes this problem? Thanks. -- OGAWA Hirofumi Signed-off-by: OGAWA Hirofumi --- fs/fat/fat.h | 2 +- fs/fat/inode.c | 14 +++++++++----- fs/fat/misc.c | 8 +++++--- 3 files changed, 15 insertions(+), 9 deletions(-) diff -puN fs/fat/inode.c~fat_sync_fs fs/fat/inode.c --- linux-2.6/fs/fat/inode.c~fat_sync_fs 2009-09-03 21:24:03.000000000 +0900 +++ linux-2.6-hirofumi/fs/fat/inode.c 2009-09-06 02:07:07.000000000 +0900 @@ -451,12 +451,16 @@ static void fat_write_super(struct super static int fat_sync_fs(struct super_block *sb, int wait) { - lock_super(sb); - fat_clusters_flush(sb); - sb->s_dirt = 0; - unlock_super(sb); + int err = 0; - return 0; + if (sb->s_dirt) { + lock_super(sb); + sb->s_dirt = 0; + err = fat_clusters_flush(sb); + unlock_super(sb); + } + + return err; } static void fat_put_super(struct super_block *sb) diff -puN fs/fat/misc.c~fat_sync_fs fs/fat/misc.c --- linux-2.6/fs/fat/misc.c~fat_sync_fs 2009-09-03 21:24:03.000000000 +0900 +++ linux-2.6-hirofumi/fs/fat/misc.c 2009-09-06 02:02:56.000000000 +0900 @@ -43,19 +43,19 @@ EXPORT_SYMBOL_GPL(fat_fs_error); /* Flushes the number of free clusters on FAT32 */ /* XXX: Need to write one per FSINFO block. Currently only writes 1 */ -void fat_clusters_flush(struct super_block *sb) +int fat_clusters_flush(struct super_block *sb) { struct msdos_sb_info *sbi = MSDOS_SB(sb); struct buffer_head *bh; struct fat_boot_fsinfo *fsinfo; if (sbi->fat_bits != 32) - return; + return 0; bh = sb_bread(sb, sbi->fsinfo_sector); if (bh == NULL) { printk(KERN_ERR "FAT: bread failed in fat_clusters_flush\n"); - return; + return -EIO; } fsinfo = (struct fat_boot_fsinfo *)bh->b_data; @@ -74,6 +74,8 @@ void fat_clusters_flush(struct super_blo mark_buffer_dirty(bh); } brelse(bh); + + return 0; } /* diff -puN fs/fat/fat.h~fat_sync_fs fs/fat/fat.h --- linux-2.6/fs/fat/fat.h~fat_sync_fs 2009-09-03 21:24:03.000000000 +0900 +++ linux-2.6-hirofumi/fs/fat/fat.h 2009-09-06 02:02:56.000000000 +0900 @@ -323,7 +323,7 @@ extern int fat_flush_inodes(struct super /* fat/misc.c */ extern void fat_fs_error(struct super_block *s, const char *fmt, ...) __attribute__ ((format (printf, 2, 3))) __cold; -extern void fat_clusters_flush(struct super_block *sb); +extern int fat_clusters_flush(struct super_block *sb); extern int fat_chain_add(struct inode *inode, int new_dclus, int nr_cluster); extern void fat_time_fat2unix(struct msdos_sb_info *sbi, struct timespec *ts, __le16 __time, __le16 __date, u8 time_cs); _ -- 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/