Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756981AbZDUL01 (ORCPT ); Tue, 21 Apr 2009 07:26:27 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754318AbZDUL0N (ORCPT ); Tue, 21 Apr 2009 07:26:13 -0400 Received: from palinux.external.hp.com ([192.25.206.14]:47796 "EHLO mail.parisc-linux.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754295AbZDUL0M (ORCPT ); Tue, 21 Apr 2009 07:26:12 -0400 Date: Tue, 21 Apr 2009 05:26:08 -0600 From: Matthew Wilcox To: Jeff Garzik Cc: Benny Halevy , Jeff Garzik , lkml , linux-ide@vger.kernel.org, linux-scsi , Arjan van de Ven , Andrew Morton Subject: Re: Is sata_nv compatible with async scsi scan? Message-ID: <20090421112608.GV1926@parisc-linux.org> References: <49ED6FD7.3070307@panasas.com> <49ED9FAC.1080303@garzik.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <49ED9FAC.1080303@garzik.org> User-Agent: Mutt/1.5.13 (2006-08-11) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1727 Lines: 46 On Tue, Apr 21, 2009 at 06:27:56AM -0400, Jeff Garzik wrote: > Benny Halevy wrote: > >Hi Jeff, > > > >Since 2.6.29 I'm having intermittent problems with booting kernels. > >After supposedly waiting for scsi async scan to complete, > >quite frequently I see errors from the init resume process > >when it fails to find the swap partition and later the root > >file system fails to load. > > > >A workaround that I found to be helpful is booting the kernel > >with scsi_mod.scan=sync so I suspect sata_nv has a problem > >with asynchronous scanning. > > Personally, I think the whole system is broken, so continue to use this > workaround until it gets fixed upstream. This sounds like some timing > issues related to waiting for the device probe to finish, something that > people keep breaking (witness my USB flash drive boot breakage). No, it's 4ace92fc112c6069b4fcb95a31d3142d4a43ff2a. Specifically, this bit: @@ -179,6 +180,8 @@ int scsi_complete_async_scans(void) spin_unlock(&async_scan_lock); kfree(data); + /* Synchronize async operations globally */ + async_synchronize_full(); return 0; } Vegard Nossum has a patch that seems to have been ignored: http://marc.info/?l=linux-kernel&m=123920746830420&w=2 -- Matthew Wilcox Intel Open Source Technology Centre "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." -- 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/