Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753553AbZIEWmQ (ORCPT ); Sat, 5 Sep 2009 18:42:16 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753487AbZIEWmP (ORCPT ); Sat, 5 Sep 2009 18:42:15 -0400 Received: from mail.parknet.ad.jp ([210.171.162.6]:53793 "EHLO mail.officemail.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751918AbZIEWmO (ORCPT ); Sat, 5 Sep 2009 18:42:14 -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> <87iqfx5mss.fsf@devron.myhome.or.jp> Date: Sun, 06 Sep 2009 07:42:13 +0900 In-Reply-To: (Zdenek Kabelac's message of "Sat, 5 Sep 2009 21:53:49 +0200") Message-ID: <873a7157zu.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: 4246 Lines: 99 Zdenek Kabelac writes: > 2009/9/5 OGAWA Hirofumi : >> 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? >> > > So I've tested your patch - it seems to fix the problem in suspend - > the machine sleeps - however after resume the mmc SD card becomes > unusable to the system and appended call trace filled my message log > very quickly: > > After reboot the filesystem appeared to be usable again without errors. Thanks for testing. "logical block size: 27499" is wrong value completely. Of course, fatfs is assuming the blocksize is not changed. (mmc wasn't resumed correctly?) Well, this problem seems to be completely different problem, and it seems the problem of suspend or mmc (or block?) stuff, or something. It would need to be analyzed by those people. Meanwhile, I'll apply this patch to workaround suspend problem and to remove unneeded write for next window. Thanks. > Call Trace: > [] __getblk+0x2cb/0x300 > [] ? _spin_unlock_irqrestore+0x38/0x80 > [] __breadahead+0x12/0x40 > [] fat_count_free_clusters+0x307/0x320 [fat] > [] ? check_object+0xd8/0x260 > [] ? trace_hardirqs_on+0xd/0x10 > [] fat_statfs+0xf5/0x110 [fat] > [] vfs_statfs+0x7c/0xa0 > [] vfs_statfs_native+0x20/0xb0 > [] sys_statfs+0x73/0xb0 > [] ? __up_write+0xcb/0x120 > [] ? sysret_check+0x27/0x62 > [] ? audit_syscall_entry+0x28b/0x2b0 > [] ? trace_hardirqs_on_caller+0x15d/0x1a0 > [] ? trace_hardirqs_on_thunk+0x3a/0x3f > [] system_call_fastpath+0x16/0x1b > getblk(): invalid block size 512 requested > logical block size: 27499 > > [] __breadahead+0x12/0x40 > [] fat_count_free_clusters+0x307/0x320 [fat] > [] ? check_object+0xd8/0x260 > [] ? trace_hardirqs_on+0xd/0x10 > [] fat_statfs+0xf5/0x110 [fat] > [] vfs_statfs+0x7c/0xa0 > [] vfs_statfs_native+0x20/0xb0 > [] sys_statfs+0x73/0xb0 > [] ? __up_write+0xcb/0x120 > [] ? sysret_check+0x27/0x62 > [] ? audit_syscall_entry+0x28b/0x2b0 > [] ? trace_hardirqs_on_caller+0x15d/0x1a0 > [] ? trace_hardirqs_on_thunk+0x3a/0x3f > [] system_call_fastpath+0x16/0x1b -- OGAWA Hirofumi -- 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/