Bug 11343

Summary: SATA Cold Boot Problems with 2.6.27-rc[23] on nVidia 680i
Product: IO/Storage Reporter: Rafael J. Wysocki (rjw)
Component: Serial ATAAssignee: Jeff Garzik (jgarzik)
Status: CLOSED CODE_FIX    
Severity: normal    
Priority: P1    
Hardware: All   
OS: Linux   
Kernel Version: 2.6.27-rc[23] Subsystem:
Regression: Yes Bisected commit-id:
Bug Depends on:    
Bug Blocks: 11167    
Attachments: nv-nohrst.patch

Description Rafael J. Wysocki 2008-08-14 09:16:27 UTC
Subject    : SATA Cold Boot Problems with 2.6.27-rc[23] on nVidia 680i
Submitter  : "Manny Maxwell" <mannymax@mannymax.net>
Date       : 2008-08-14 4:16
References : http://marc.info/?l=linux-kernel&m=121868782917600&w=4

This entry is being used for tracking a regression from 2.6.26.  Please don't
close it until the problem is fixed in the mainline.
Comment 1 Rafael J. Wysocki 2008-08-24 14:05:41 UTC
On Sunday, 24 of August 2008, Jeff Garzik wrote:
> Rafael J. Wysocki wrote:
> > This message has been generated automatically as a part of a report
> > of recent regressions.
> > 
> > The following bug entry is on the current list of known regressions
> > from 2.6.26.  Please verify if it still should be listed and let me know
> > (either way).
> > 
> > 
> > Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=11343
> > Subject             : SATA Cold Boot Problems with 2.6.27-rc[23] on nVidia
> 680i
> > Submitter   : Manny Maxwell <mannymax@mannymax.net>
> > Date                : 2008-08-14 4:16 (10 days old)
> > References  : http://marc.info/?l=linux-kernel&m=121868782917600&w=4
> 
> 
> hmmmm.  Looking at changes between the two csets listed in the email 
> (623fa57..8f616cd), all of them are driver-specific and unrelated to 
> Manny's hardware except for
> 
>       commit 2486fa561a3192bbbec39c7feef87a1e07bd6342
>       Author: Tejun Heo <tj@kernel.org>
>       Date:   Thu Jul 31 07:52:40 2008 +0900
> 
>           libata: update atapi disable handling
> 
> So you could try to revert that and see what happens.  But given that 
> small range of changes, it really seems like something else, maybe in 
> the PCI subsystem (random guess).
> 
> Looking at the entire kernel, nothing jumps out, either.  Its mostly fs 
> updates (ext4, xfs), a networking update, an ARM update, and a libata 
> update.
> 
> Also, some reset-related fixes just went in, so re-testing the latest 
> -git would be helpful as well.
Comment 2 Tejun Heo 2008-08-25 01:51:55 UTC
Created attachment 17431 [details]
nv-nohrst.patch

Can you please give a shot at the attached patch?
Comment 3 Jeff Garzik 2008-09-16 12:14:16 UTC
Ping?

Tejun's patch is now in the kernel as

commit 2fd673ecf0378ddeeeb87b3605e50212e0c0ddc6
Author: Tejun Heo <tj@kernel.org>
Date:   Fri Aug 29 16:13:12 2008 +0200

    sata_nv: disable hardreset for generic